solidjs / solid / 38438266685

78%

main: 89%

push

github

fix(signals): a Show branch mounted by a held write reads the value that mounted it (#4010) (#4027)

* fix(signals): a Show branch mounted by a held write reads the value that mounted it (#4010)

A binding in a branch a held pass mounted is a member of that pass's held
frame (ruling A): the branch exists only in the transaction. Only a member
that had never run (STATUS_UNINITIALIZED) was treated as born into it; one
whose first pass published before the seam held it was served the
committed screen whenever it ran again before the landing — `frameRead`
made it a stale reader, `recompute` published it mainline, and
`stagedScreen` served a verdict reader the committed value in the
transaction's own flush. A non-keyed Show's narrowed accessor then read the
committed `null` and threw into <Errored>; since #3977 an unchanged landing
left the error up.

The park seam now flags every member of a held pass's frame
(CONFIG_HELD_FRAME, cleared by its commit), and `bornInto(c, t)` — held by
t and uninitialized or flagged — replaces the uninitialized-only test in
`recompute`, `frameRead` and `stagedScreen`. `observeFlight` likewise no
longer serves such a verdict reader the committed value of its own
transaction's flight: it waits with its frame. Without that, a member that
probes and reads the flight let the transaction land under it and was
re-derived into a fresh observation every round (an infinite flush loop).

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

* chore(size): raise the caps #4010's held-frame tag crosses

Accepted by the maintainer: +89 B minified in the always-loaded signals
core. Each cap is set at measured + 10 B rounded up to 0.01 KB; hello
world stays under 10 KB (9,966 B).

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

---------

Co-authored-by: Claude ... (continued)

746 of 1036 branches covered (72.01%)

Branch coverage included in aggregate %.

1079 of 1321 relevant lines covered (81.68%)

27.2 hits per line