solidjs / solid / 40932706477

78%

main: 89%

push

github

fix(signals): a halt from a computed's error rethrows the user's error, not the status wrapper (#4423)

* fix(signals): a halt from a computed's error rethrows the user's error, not the status wrapper

escalate() halted with the unwrapped cause but rethrew the StatusError,
whose own `source` is the reactive node. A flush in a microtask turned
that into an unhandled rejection, and a reporter serializing it walked
the whole graph (vitest ran out of memory on the #4406 review repro).
Rethrow the user's error, as the halt message already reports it.

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

* fix(signals): drop escalate's separate rethrow argument

Every call site now rethrows the cause it halts with, so the parameter
goes: -6 B minified on the floors instead of +5.

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

---------

Co-authored-by: Claude <noreply@anthropic.com>

790 of 1088 branches covered (72.61%)

Branch coverage included in aggregate %.

1125 of 1359 relevant lines covered (82.78%)

29.77 hits per line