You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Parent tracker for the adversarial review of the component-test migration (#274, follow-ups #300/#301), conducted post-merge with three independent review passes (maintainability/ergonomics, coverage deltas, CI/DX) plus measured old-vs-new timing. Every finding was validated against source (component-test read at the pinned rev); the census-gap and ratification findings were confirmed by two reviewers independently plus a direct read.
Context — what the migration got right (verified, for balance): cutover parity is exact (census fixtures byte-identical to the incumbent's final locks; probe tables/bodies, schedule logic, and the 1-in-20 straddle sample byte-identical); and the new stack gains matrix-check (a CI freshness gate the incumbent lacked), the decline-pair lint, per-case not-reached synthesis with trap attribution, dead-declaration errors, a ported mutation oracle, and the widened class-D gate. The e5e378e/f599fc3 aggregation wobble resolved coherently after upstream's hashes-are-provenance ruling; nothing ended weaker than pre-wobble.
Measured cost (local, warm, no browser): full gate 42.7s (incumbent, parallel targets) -> 70.7s (sequential targets + per-run transpile), while the wasmtime leg itself got 2x faster; CI common-case workflow wall roughly doubled, with the conformance job the critical path in every sampled run and zero artifact-cache hits in the sample (#307 has the tables).
Deliberately excluded (tracked elsewhere): the jco-browser optional/missing-results handling and the live browser-run results viewer.
Suggested order: #302 first (it changes what #303's re-ratification can say, and its fix shape may come from upstream component-test); #304/#305 are cheap doc/setup fixes with outsized onboarding effect; #307's three items are independent mechanical wins.
Parent tracker for the adversarial review of the component-test migration (#274, follow-ups #300/#301), conducted post-merge with three independent review passes (maintainability/ergonomics, coverage deltas, CI/DX) plus measured old-vs-new timing. Every finding was validated against source (component-test read at the pinned rev); the census-gap and ratification findings were confirmed by two reviewers independently plus a direct read.
Context — what the migration got right (verified, for balance): cutover parity is exact (census fixtures byte-identical to the incumbent's final locks; probe tables/bodies, schedule logic, and the 1-in-20 straddle sample byte-identical); and the new stack gains matrix-check (a CI freshness gate the incumbent lacked), the decline-pair lint, per-case not-reached synthesis with trap attribution, dead-declaration errors, a ported mutation oracle, and the widened class-D gate. The e5e378e/f599fc3 aggregation wobble resolved coherently after upstream's hashes-are-provenance ruling; nothing ended weaker than pre-wobble.
Measured cost (local, warm, no browser): full gate 42.7s (incumbent, parallel targets) -> 70.7s (sequential targets + per-run transpile), while the wasmtime leg itself got 2x faster; CI common-case workflow wall roughly doubled, with the conformance job the critical path in every sampled run and zero artifact-cache hits in the sample (#307 has the tables).
P0 — substance
P1 — workflow and cost
P2 — bundles by area
Deliberately excluded (tracked elsewhere): the jco-browser optional/missing-results handling and the live browser-run results viewer.
Suggested order: #302 first (it changes what #303's re-ratification can say, and its fix shape may come from upstream component-test); #304/#305 are cheap doc/setup fixes with outsized onboarding effect; #307's three items are independent mechanical wins.