solidjs / solid / 34825483586

78%

main: 89%

push

github

fix(signals): keep a mainline effect value out of its parked transaction (#3412) (#3420)

* fix(signals): keep a mainline effect value out of its parked transaction (#3412)

An effect stamped by a held transaction that recomputes on an unrelated
write, and no longer reads the held source, publishes its value mainline.
The forced re-run inside its own transaction (which only refreshes the
staged view) re-stamped `_valueTransition`, so when finalization re-entered
the transaction — a Loading `on` reset writing its `_disabled` signal — the
effect phase parked the mainline value with it. Restore the owner after the
re-run.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

* chore(size): raise the store-families cap for #3412's ownership save/restore (+8 B)

Co-authored-by: Cursor <cursoragent@cursor.com>

---------

Co-authored-by: Brenley Dueck <brenleydueck@Brenleys-Mac-mini.local>
Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
Co-authored-by: Ryan Carniato <ryansolid@gmail.com>
Co-authored-by: Cursor <cursoragent@cursor.com>

519 of 790 branches covered (65.7%)

Branch coverage included in aggregate %.

772 of 1007 relevant lines covered (76.66%)

15.02 hits per line