Repository navigation
test(ui-react): separate narrow callout geometry and capture - #2393
Conversation
|
Run 39082286976, attempt 1, passed all 14 selected product jobs on head Actual checkout logs for metadata and every selected product job agree on Returned to draft and refreshed the branch onto this delivered base. Local candidate |
7ca8cd6 to
8e89864
Compare
|
Confirmation run 39084080316, attempt 1, passed all 14 selected product jobs and aggregate job 117550893618 at head Before merging, the independent live-main read found Returned to draft and refreshed locally onto current main. Candidate |
|
Custody handoff acknowledged to authorized claim Additional local receipt available to the takeover owner: candidate The latest independent live-main read is |
Browser shard 2 drifted from the 840 s qualification bar to its 900 s job bound (887 s on main 7f22bb6, 917 s cancelled on a ready PR). Refresh measured weights from successful main run 39109891338 attempt 1, which includes the #2393 split. Scored by the fresh weights, the slowest project per shard drops from 628 s to 573 s on both shards. Closes #2414 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…e bound and three shards (#2415) * chore(ci): refresh browser shard weights after the narrow callout split Browser shard 2 drifted from the 840 s qualification bar to its 900 s job bound (887 s on main 7f22bb6, 917 s cancelled on a ready PR). Refresh measured weights from successful main run 39109891338 attempt 1, which includes the #2393 split. Scored by the fresh weights, the slowest project per shard drops from 628 s to 573 s on both shards. Closes #2414 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * chore(ci): raise the browser job bound to 20 minutes Browser shard 2 reached 887 s on main and 917 s on a ready PR against the 900 s bound; refreshed weights brought the slower shard to 845 s, still above the 840 s qualification bar. The bound now leaves real buffer, and a test pins it so it cannot shrink silently. Refs #2414 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * chore(ci): run browser proof in three measured shards Refreshed weights brought the slower browser shard to 845 s, above the 840 s bar. Scored by fresh weights, three shards bring each shard's slowest project to 482 s, the cost of the single heaviest file. Layout stays at two shards. One table, PROOF_SHARDS in scripts/proof-shards.ts, now feeds retention receipts and source validation, the weight refresh, the input-partition fingerprint and a workflow guard that matches each domain's matrix and --shard divisor. A receipt from the two-shard era names another partition and is never admitted. Refs #2414 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * test(ci): reject a drifted layout shard matrix independently Also state that the inclusion fingerprint binds each domain's measured partitions rather than two, and that layout keeps two shards. Refs #2414 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Closes #2386
What changed
The authored narrow 1,024-note diagram scenarios now check full geometry and capture the raster in separate cases for light, dark and print. Each capture still mounts the complete 320px source and checks all note indices, one-column layout and overflow before writing the same artifact identifier. Full geometry and source-text assertions remain in the geometry cases, and whole-case teardown ownership is unchanged.
Why this shape
The original WebKit case combined two substantial operations under one 15-second budget. Hosted evidence records a 15,745-ms failed case with an 8,496-ms capture that eventually completed. A bounded local resource control reproduces cumulative deadline exhaustion with completed geometry and capture, rather than a universal rendering defect. Separate obligations follow the neighboring literal/grouped cases delivered under #2351. No timeout, default scheduling, production renderer or offline guard changes.
Proof
e700f7c2f46ed5996005be8c30e07a9eefc3ec38, base63fde3c8fe76e754aa4524f95b23f9786f4447fe, head8ad081da7c29f809c74c12638fb54a8bd14e2a72, tree048556cdbf4e7ce2e7d42fb721fec9833c21916d. Its failed aggregate remains failed evidence.bun run check:pr --base origin/mainpassed lint/leakage/environment/changeset gates, 956 fast invariant tests / 6,328 assertions and 170 import-graph tests / 1,512 assertions. The earlier test-file-only preflight had 54 invariant tests and no import-graph matches; the final result includes the reviewed pin update.REVIEWED_DIAGRAM_DOCUMENT_BINDSas expected. After re-auditing all eight direct call sites, unchanged capture arguments, sink and producer closure, the exact caller hash is updated; all 13 binding tests / 69 assertions pass, including drift and unreviewed-caller refusal controls.8e89864ec56e4818027b06dc78e6d3fa180e167f, parent68e12f27eb249c9fe7262a498a0397c04bbda8f4, tree6333172bcc6a8dd16bd2e239d6bcf9adc4d6b420. The two changed files are byte-identical across the rebase; refreshed local preflight passed 956 invariants / 6,328 assertions and 170 affected import-graph tests / 1,512 assertions. Complete selected hosted confirmation on this base remains required.7ca8cd68517e38f29bf617133c27305810f2ecde. Metadata, every selected job and aggregate checkout logs agree on3ccc6b4534fb7ccfea388c038974bc7b8fc07c1d, parents base775eb17aab740fa759c2182f306c38fc17acbcbeand that head, tree57d10cf4ef552ed8082d6afd900053126add393f. Proof job 117545272735 correctly failed live-base freshness after test(ui-sdk): exercise literal /tmp regardless of runner TMPDIR #2389 changed private-tmp native placement proof. This failed aggregate is preserved and does not authorize merging or relabel old jobs as new executions.Checklist
docs/reflect the change, or nothing documented moved.Anything left open
Complete selected current-candidate hosted proof and matching-head merge remain required. #2356/#2359 stay blocked until this correction is delivered; their Gemini implementation is outside this PR. No original failed run is admitted as a passing baseline.