Repository navigation
ci: fewer-redundant-runs - #930
fizzy-linux[bot] wants to merge 6 commits into
Conversation
Agent-Team: linux Worker: codex
Agent-Team: linux Worker: claude
Agent-Team: linux Worker: codex
PM note: how this PR will be handledWhat it is for: ci: fewer-redundant-runs Rules that apply: touches CI or workflows; changes agent rules or guides. Things to watch (checked by rule):
What happens next:
A living status comment below is kept current. Nothing here is a merge decision. |
Where this PR standsStage: Waiting for re-review What the PM needs before approving (ticks as it is met; the full CI runs only at the end, after the reviews are clean):
Rules that apply: touches the SDK or plugin ABI: the shape fingerprint must be recorded; touches build scripts; touches CI or workflows; changes agent rules or guides. Reviews at the current commit ( None yet. 4 earlier reviews are for older commits and will be redone. Latest fixes comment: https://github.com/fizzyedit/work/issues/523#issuecomment-6452818395 (12 findings answered) Updated 2026-10-27 20:30 UTC by the PM. This comment is edited in place. |
Merge origin/main (the draft/ready change landed in #903), taking its docs/AGENT_WORK.md wording. test-ci-ok.py no longer splits the workflow on fixed indents and blank lines, and drops the unused RESULTS variable. Agent-Team: linux Worker: claude Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Agent-Team: linux Worker: codex
|
Closed by the PM's ruling: the remaining change does not earn a ci.yml edit; work#103's last Done item (the measurement table) is the PM's. |
Part of fizzyedit/work#103
Rework: work#540
Goal
Protect the existing draft changes-filter and ci-ok policy with regression tests. Suggested title: ci: test the ci-ok and draft-filter shell.
What changed and why
The PR adds scripts/test-ci-ok.py and its format-job step. Tests execute the workflow's real shell for draft/ready filtering and selected-job outcomes. Extract run blocks by indentation so blank lines and indentation changes are accepted; remove the unused RESULTS environment variable. Correct the format-job tools comment. Draft scheduling and cache policy already landed in #903.
How it was verified
python3 -I scripts/test-ci-ok.py: 2 tests pass. Removing the draft early exit fails the draft case; accepting skipped selected jobs fails the gate cases. Formatting and lint pass; build 84/84, test 183/183, integration 94/94, check-web 44/44, SDK-version 35/35 steps succeeded. See GATES.md.
Not done / follow-ups
The required before/after CI-run and Zig-install measurements over the last ten merged PRs remain uncollected. A network-enabled harness or maintainer must obtain them from gh run list and setup-zig logs before the evidence criterion is satisfied. Hosted scheduling and cache hits remain unverified. The harness must update the PR title/body and regenerate its diffstat against the correct merge base; this worker cannot publish. Pre-existing staged changes were left intact.
Changes (101 files, +604/-3132)
Gates: 6 of 6 passed
scripts/check-fmt.sh(exit 0)scripts/zb build --summary all(exit 0)scripts/zb build test --summary all(exit 0)scripts/zb build test-integration --summary all(exit 0)scripts/zb build check-web --summary all(exit 0)scripts/zb build test-sdk-version --summary all(exit 0)