Skip to content

ci: reuse verified PR artifacts across unrelated main changes - #880

Merged
simonvanlaak merged 4 commits into
mainfrom
ci/pr-reuse-input-closure
Nov 6, 2026
Merged

simonvanlaak merged 4 commits into
mainfrom
ci/pr-reuse-input-closure

Conversation

@simonvanlaak

@simonvanlaak simonvanlaak commented Nov 6, 2026 •

Copy link
Copy Markdown
Collaborator

Successful PR artifacts currently rebuild when unrelated commits land after their tested base. Admit the original artifact when its complete registered component source closure matches main, its tested base is an ancestor of main's landing parent, and the existing owner/target/runtime/toolchain digest and authenticated producer checks match. Missing commit or artifact evidence falls back to fresh work; original bytes and build provenance remain unchanged.

The input audit adds two external App Shell test dependencies to its registration, trigger filters and platform selector. No workflow jobs or cancellation rules are added. Public CI also exposed a reproducible test setup race: resizing left a smooth scroll running during the frog double-click. Flush resize layout and finish scrolling before the unchanged real pointer action and reveal/QR viewport assertions; no product behavior or timeouts change.

Validation: 84 delivery/workflow tests, 21 architecture tests, strict TypeScript, repository architecture checks and actionlint passed. Real Git fixtures cover advanced main, missing tested commits, changed inputs/inventory/modes, divergent history and failed retrieval. Public's failure reproduced with full Chrome on two CPUs; the final clean repair passed its full verifier with Chromium/Firefox (240 + 26 tests and analytics smoke). Hook fixtures include the newly consumed registration. Independent input audit, admission review, and final repair/integration review found no unresolved findings. Normal PR platform checks and post-merge main receipts remain delivery gates.

@simonvanlaak
simonvanlaak force-pushed the ci/pr-reuse-input-closure branch from f9ff67b to 8a5b8ee Compare November 6, 2026 17:19
@simonvanlaak simonvanlaak changed the title ci: reuse PR artifacts when unrelated main inputs change ci: reuse verified PR artifacts across unrelated main changes Nov 6, 2026
@simonvanlaak
simonvanlaak merged commit 1d50042 into main Nov 6, 2026
44 checks passed
@simonvanlaak
simonvanlaak deleted the ci/pr-reuse-input-closure branch November 6, 2026 17:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant