IW-690: ci-gate --expected-head #763

Merged

mph merged 15 commits from IW-690 into main 2026-10-08 06:21:51 +00:00

Conversation 0 Commits 15 Files changed 17 +1538 -22

claude commented 2026-10-08 04:52:26 +00:00

Member

Copy link

Summary

iw ci-gate gets an optional flag, --expected-head SHA. The caller gives the full commit id of the head that it pushed. The gate then judges only that head: a reading of any other head is not judged, the gate keeps waiting, and the wait times out if the forge never reports that head. Without the flag, the behaviour does not change.

This lets kanon awaitCi stop a stale-head judgement after a ci-fix push, so a stale red does not use up a recovery attempt.

Changes

• core/model/CiGate.scala: pure fullCommitId validation (40 or 64 hex characters, any case, normalised to lower case).
• core/commands/CiGateCommand.scala: the flag, validation as the first step of resolve (exit 1 with a clear error, no other work), value passed into CiGateTarget.expectedHead; help and exit-code text updated.
• docs/api/CiGate.md: flag documented.
• Tests: unit (CiGateTest), harness (CiGateCommandHarnessTest), BATS smoke (test/ci-gate.bats).

Testing

• Phase PR #761 passed CI on its head.
• Pre-push checks on this branch: core compile (no warnings), scalafix, core unit tests, command compilation — all pass.

Review packet: project-management/issues/IW-690/review-packet.md.
Follow-up filed: #762.

Release notes

See project-management/issues/IW-690/release-notes.md (Czech).

🤖 Generated with Claude Code

Closes #690

## Summary `iw ci-gate` gets an optional flag, `--expected-head SHA`. The caller gives the full commit id of the head that it pushed. The gate then judges only that head: a reading of any other head is not judged, the gate keeps waiting, and the wait times out if the forge never reports that head. Without the flag, the behaviour does not change. This lets kanon `awaitCi` stop a stale-head judgement after a ci-fix push, so a stale red does not use up a recovery attempt. ## Changes - `core/model/CiGate.scala`: pure `fullCommitId` validation (40 or 64 hex characters, any case, normalised to lower case). - `core/commands/CiGateCommand.scala`: the flag, validation as the first step of `resolve` (exit 1 with a clear error, no other work), value passed into `CiGateTarget.expectedHead`; help and exit-code text updated. - `docs/api/CiGate.md`: flag documented. - Tests: unit (`CiGateTest`), harness (`CiGateCommandHarnessTest`), BATS smoke (`test/ci-gate.bats`). ## Testing - Phase PR #761 passed CI on its head. - Pre-push checks on this branch: core compile (no warnings), scalafix, core unit tests, command compilation — all pass. Review packet: `project-management/issues/IW-690/review-packet.md`. Follow-up filed: #762. ## Release notes See `project-management/issues/IW-690/release-notes.md` (Czech). 🤖 Generated with [Claude Code](https://claude.com/claude-code) Closes #690

claude added 15 commits 2026-10-08 04:52:26 +00:00

workflow(wf): Create analysis for IW-690 ... f42fbc35e0

ci-gate has no way to take an expected head, so kanon awaitCi cannot stop a stale-head judgement

Artifacts:
- analysis.md
- issue.md

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01B8ZD5foFWAE2zV1xYojPMa

chore(IW-690): update review-state to analysis_ready ... 190a91dba5

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01B8ZD5foFWAE2zV1xYojPMa

docs(IW-690): correct the analysis where review found errors ... 7526b9a56e

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01B8ZD5foFWAE2zV1xYojPMa

chore(IW-690): update review-state to tasks_ready fd892bda5e

workflow(wf): Create task index for IW-690 ... ff2c0eb06f

ci-gate has no way to take an expected head, so kanon awaitCi cannot stop a stale-head judgement

Artifacts:
- tasks.md
- review-state.json (tasks_ready)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01B8ZD5foFWAE2zV1xYojPMa

docs(IW-690): intake verdict — agent-ready ... d23ed9e0a6

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01B8ZD5foFWAE2zV1xYojPMa

Merge remote-tracking branch 'origin/main' into IW-690 d943cd2de9

Merge remote-tracking branch 'origin/main' into IW-690 1123cdcc2c

chore(IW-690): move review-state card out of the tree 4dbd4b384b

Phase 1: context ready ... 03499b61fb

Issue: IW-690
Phase: 1 - ci-gate --expected-head

This commit marks the checkpoint for phase 1 context generation.
Use wf-implement IW-690 to return to this point.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

workflow(wf): Generate Phase 1 tasks for IW-690 ... 36da0ae4b0

ci-gate --expected-head

Artifacts:
- phase-01-tasks.md

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Phase 1: ci-gate --expected-head (#761) e84bf0dc91

chore(IW-690): mark phase 01 complete in tasks.md bb4833fff5

chore(IW-690): file follow-up as #762 ... 65ddbb4ffe

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

docs(IW-690): review packet and release notes ... 7a7fd8e12e

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

mph merged commit 3abaf3e323 into main 2026-10-08 06:21:51 +00:00

mph referenced this pull request from a commit 2026-10-08 06:21:51 +00:00

Merge pull request 'IW-690: ci-gate --expected-head' (#763) from IW-690 into main

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

agent-ready

  

bug

Something isn't working

  

contract

Run the tool-contract suite on this PR

  

documentation

Improvements or additions to documentation

  

duplicate

This issue or pull request already exists

  

enhancement

New feature or request

  

good first issue

Good for newcomers

  

help wanted

Extra attention is needed

  

invalid

This doesn't seem right

  

needs-human

  

needs-input

Intake asked the human a question on the issue; waiting for the answer

  

question

Further information is requested

  

wontfix

This will not be worked on

No labels

agent-ready

bug

contract

documentation

duplicate

enhancement

good first issue

help wanted

invalid

needs-human

needs-input

question

wontfix

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

iterative-works/iw-cli!763

No description provided.