Kikobeats / superlock / 31020843318

100%

push

github

fix: treat null signal as no cancellation under contention (#24)

* fix: treat null signal as no cancellation under contention

Passing null as the AbortSignal under contention took the abortable
queue path (signal !== undefined), so addEventListener threw after the
waiter was enqueued. That left a zombie whose acquire() also threw on
release, rejecting successful withLock work and letting later acquirers
jump the FIFO queue.

Treat null like undefined, and unlink the waiter if subscribe throws.

* refactor: reject a malformed signal before enqueueing

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01MTbj6CYXGsk6rhXx7J8SUP

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Kiko Beats <josefrancisco.verdu@gmail.com>
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>

40 of 40 branches covered (100.0%)

Branch coverage included in aggregate %.

4 of 4 new or added lines in 1 file covered. (100.0%)

111 of 111 relevant lines covered (100.0%)

148.86 hits per line