lsm / neokai / 26667979831

82%

push

github

fix(web): prevent stale onInput from restoring draft after send (#2045)

* fix(web): prevent stale onInput from restoring draft after send

Race between clearDraft() and browser onInput in uncontrolled textarea.
When handleSubmit cleared the signal, useLayoutEffect hadn't flushed yet,
so a stale onInput reading textarea.value could call setContent and
restore the old content — leaving a single character in the composer.

Fix:
- submittingRef guard in handleContentChange drops onInput during submit
- Direct textarea.value = '' clear in handleSubmit, bypassing batch flush

Closes #502

* fix(web): wrap submit in try/finally to reset submittingRef on throw

submittingRef.current = false was placed after await onSend(...).
If onSend rejected, the ref stayed true forever, making the textarea
permanently unresponsive.

Fix: move the reset into a finally block so it always runs regardless
of success, failure, or exception.

Also adds a test verifying the textarea remains functional after onSend
throws an error.

9798 of 14272 branches covered (68.65%)

Branch coverage included in aggregate %.

14 of 15 new or added lines in 1 file covered. (93.33%)

82164 of 96239 relevant lines covered (85.37%)

297.09 hits per line