ci: allow snapshot harness more time - #20
Draft
fullofcaffeine wants to merge 1 commit into
Draft
fullofcaffeine wants to merge 1 commit into
fullofcaffeine wants to merge 1 commit into
Conversation
Two hosted snapshot runs completed setup normally and kept passing cases until GitHub cancelled each job at the exact 60-minute ceiling. The suite had reached its late curated Clippy cases, so the cancellation did not identify a compiler or snapshot failure.\n\nRaise only the snapshot shard ceiling to 90 minutes and update the focused workflow contract to prevent an accidental reduction below the newly observed requirement. The harness stages, six-worker parallelism, cleanup policy, and test coverage remain unchanged.\n\nVerified with the focused CI workflow contract, JavaScript syntax checking, and git diff validation. Hosted CI on this branch provides the final full-suite proof.
This was referenced Sep 6, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why\n\nTwo hosted snapshot jobs completed setup normally and kept passing cases until GitHub cancelled each at the exact 60-minute job ceiling. Both runs had reached late curated Clippy cases, so the cancellation did not identify a compiler, snapshot, or assertion failure.\n\n## What changed\n\n- Raise only the Harness / snapshots job timeout from 60 to 90 minutes.\n- Raise the focused workflow contract minimum to 90 minutes.\n- Keep the existing stages, six-worker parallelism, cold-output cleanup, and assertions unchanged.\n\nThis is a stacked draft based on fix/fast-uri-audit. That base supplies the independent fast-uri audit repair required for hosted CI. The intended order is:\n\n1. Review and merge this PR into fix/fast-uri-audit.\n2. Let PR #19 rerun at its new exact head with the longer snapshot ceiling.\n3. Merge PR #19 into main only after all required checks pass.\n\n## Verification\n\n- node test/scripts/ci-harness-workflow.test.js\n- node --check test/scripts/ci-harness-workflow.test.js\n- git diff --check\n- The two failing hosted jobs were cancelled at exactly 60 minutes while the harness was still advancing.\n\nHosted CI on this draft is the final full-suite proof.\n\n## Agent ownership and provenance\n\nA repository-maintenance agent prepared this focused CI change after reviewing the exact hosted job metadata and logs. That agent owns review fixes and rebasing for this branch. Final approval and merge remain maintainer-only. No personal full-name attribution is required.