Phase 1: A dirty review-state card blocks the checkout in phase-advance #588

Merged

claude merged 6 commits from IW-583-phase-01 into IW-583 2026-09-21 10:52:41 +00:00

Conversation 0 Commits 6 Files changed 7 +433 -43

claude commented 2026-09-21 09:26:37 +00:00

Member

Copy link

Phase 01 implementation for IW-583.

Files

Browse changed files: https://code.iterative.works/iterative-works/iw-cli/-/blob/IW-583-phase-01/

Phase 01 implementation for IW-583. ### Files Browse changed files: https://code.iterative.works/iterative-works/iw-cli/-/blob/IW-583-phase-01/

claude added 2 commits 2026-09-21 09:26:37 +00:00

chore(IW-583): update review-state for phase 01 2dc199fd84

A dirty review-state card blocks the checkout in phase-advance ... 3de2b2ed24

- phase-advance commits the review-state card before it checks out the feature branch, only after the PR is confirmed merged
- One private reviewStateCommitMessage serves both card commits in PhaseAdvance
- Harness tests for a dirty card, no flush before merge, and a dirty user file; BATS regression test with real git

mph added 1 commit 2026-09-21 09:27:55 +00:00

chore(IW-583): update review-state for phase 01 145bd0f8b1

mph added 1 commit 2026-09-21 10:29:43 +00:00

chore(IW-583): recover workflow state from interrupted run 387218d8c2

mph added 1 commit 2026-09-21 10:30:57 +00:00

chore(IW-583): update review-state for phase 01 e3da7326fb

mph added 1 commit 2026-09-21 10:52:18 +00:00

chore(IW-583): update review-state for phase 01 35372c76a0

claude merged commit 53a56c7bc8 into IW-583 2026-09-21 10:52:41 +00:00

claude deleted branch IW-583-phase-01 2026-09-21 10:52:41 +00:00

claude referenced this pull request from a commit 2026-09-21 10:52:41 +00:00

Phase 1: A dirty review-state card blocks the checkout in phase-advance (#588)

claude referenced this pull request 2026-09-21 10:56:26 +00:00

IW-583: phase-advance commits the review-state card before it checks out the feature branch #589

claude referenced this pull request 2026-09-21 11:26:27 +00:00

phase-pr pushes a commit after reading CI, so its green can describe a stale head #491

claude referenced this pull request 2026-09-21 12:05:33 +00:00

phase-pr pushes a commit after reading CI, so its green can describe a stale head #491

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

2 participants

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!588

No description provided.