solidjs / solid / 41613720232

78%

main: 89%

push

github

fix(signals): an Errored heals when nothing reads the failed source any more (#4644) (#4653)

* fix(signals): an Errored heals when nothing reads the failed source any more (#4644)

An error boundary waits on the source that threw. When the reader it caught
was its own tree (the error reached it only through memos, e.g. a Show/Match
condition), the tree re-ran clean after navigating away but prune kept it
because the source itself was still errored. A source read by nothing now
settles: no live reader holds the error (D-39, F-3, F-12).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0117BoWNSSgtemsSYsaX2hXE

* test(web): import JSX from @solidjs/web in the #4644 spec

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0117BoWNSSgtemsSYsaX2hXE

* fix(signals): only readers the Errored caught keep its fallback up

prune kept an error boundary's reader while the source that threw was
errored and subscribed by anything at all. A memo hoisted outside the
boundary, a derived store or projection, or a reader under another
Errored kept the source subscribed, so the fallback stayed up after
every reader under the boundary was gone. D-39: the boundary waits on
what threw only while a reader that caught the error is alive. The
source now keeps the reader only while that reader still reads it,
directly or through derivations (#4644).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0117BoWNSSgtemsSYsaX2hXE

* fix(signals): walk each upstream node once in the Errored heal check (#4644)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0117BoWNSSgtemsSYsaX2hXE

* refactor(signals): shorter heal-check walk (#4644)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0117BoWNSSgtemsSYsaX2hXE

---------

... (continued)

807 of 1108 branches covered (72.83%)

Branch coverage included in aggregate %.

1132 of 1370 relevant lines covered (82.63%)

30.14 hits per line