Repository navigation
Deepen CI watch to own Delivery's per-head re-run policy - #1074
Merged
Merged
Conversation
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
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.
Summary
CI watch now owns observation and both per-head re-run allowances for one Delivery.
Closes #955.
Evidence
Watchdid not exist; the Repair loop selected three raw CI operations and tracked allowances.After: 52 CI watch tests and 47 Repair loop tests pass through the narrowed interface. The per-head policy matrix now uses scripted check/job observations; the loop tests retain ordering, stages, budgets and Base fix routing.
cargo check,cargo fmt --checkandcargo clippy --all-targets -- -D warningspass. The full Rust suite passed in CI on exact head4eb7e7f: 2,249 passed, 51 ignored. CI also passed site, Console and the macOS gate./usr/bin/time -p scripts/affected-tests mainselected the full Rust suite using nextest. The redundant local run was stopped after 1,182.07s, including compilation, once exact-head CI passed; its successful result is reused perdocs/testing.md.Test seams: CI watch assessment with the existing in-memory GitHub adapter; Repair loop assessment/completed-CI-fix notification; CLI Delivery with real Git and fake GitHub, including captured pull request gates.
Independent Standards and Spec reviews read all seven changed files. No generated exclusions or unread changed files remain.
Merge Danger
Door: two-way
An internal ownership refactor with no configuration, persistence or CI policy change.
Blast Radius: Delivery
Runs and Spec PRs share the CI assessment path.
Unaddressed findings
Built with codex · gpt-6.1-sol · xhigh
Security review outcome
No unaddressed introduced findings.