PJR-T46Y register の実行のあとに review の agent が判定し、todo と issue を自動で close する
1. 概要
段階 A。統合の後に review の agent が個票の完了条件・result・差分・検証の結果を照らして verdict を出し、complete なら close、incomplete なら waiting に戻す。runner 自身のコードを変えた項目の動作確認の扱いも決める。 PJR-3DY1 の決定に基づく
2. 完了条件
- PJR-7WME の完了後に行う。
exec run --registerの統合の後に、review の agent（既定はclaude-expert-review-executor）が、個票の完了条件・result・差分・検証の結果を照らして verdict を出す段を加える。review の agent は読み取り専用とし、確かめるためのコマンドは流してよい。- verdict が
complete・complete-with-findingsなら runner が close し、incompleteならwaitingに戻す。actor は review の agent とし、verdict の理由をイベントに残す。 - 自動で close するのは todo と issue だけとし、decision・question は close しない。
- この段は既定では動かさず、オプションか設定で有効にする。有効にする方法を specdojo:exec-operation-guide に書く。
- runner 自身のコードを変えた項目の扱い（統合の後に実際の
exec runで確かめる段を加えるか、人の close に残すか）を決めて実装し、理由を result に書く。 - 単体テストか統合テストで、
complete・complete-with-findings・incompleteの 3 通りと、decision を close しないことを確かめる。 - 作業を終える前に、
npm run typecheck・npm run lint:ts・npm run test:unitを実行して通す。 - 親の検証（lint-ts・lint-fm・lint-md・typecheck・test-unit・test-integration・validate-schema）が通る。
- CHANGELOG に記載する場合は、未公開の版の節（
## [0.4.0] - 未定）に書く。公開済みの版の節は変えない。
3. 作業内容
| No | 作業 | 担当 | 状態 | メモ |
|---|---|---|---|---|
| 1 | 統合の後の review の agent による自動の close を加える | DEV | done | --post-review と run.register_post_review で有効化 |
4. 対応結果
- register の worktree 統合後に、個票、edit plan/result、executor evidence、統合差分を照合する読み取り専用 review pipeline を追加した。既定の review executor は
claude-expert-review-executorで、--review-byにより変更できる。 --post-reviewまたはexec-defaults.yamlのrun.register_post_review: trueで有効にする。既定は無効であり、統合差分を確定できない in-place 実行では利用できない。- verdict が
complete/complete-with-findingsの todo・issue は review executor を actor として close し、incompleteまたは review 失敗は理由を event に残してwaitingへ戻す。decision・question・risk・change-request は完了 verdict でもreviewに残す。 - review executor がファイルを変更した場合は
incompleteとして扱う。post-review の plan/result/evidence と状態遷移は専用 commit にまとめる。 - runner 自身のコードを変更した項目は、実行中の親プロセスが統合後のコードを読み直せないため自動 close せず、人が新しいプロセスで smoke test を行って close する方式を採った。
- disposition の単体テストで
complete、complete-with-findings、incomplete、decision を close しない場合、および runner 実装変更を人の close に残す場合を確認する。