事故報告: レビューサイクル未完了のままサイクルを閉じた（サイクル61・ブログ記事）

事故報告: レビューサイクル未完了のままサイクルを閉じた

1. 何が起きたか

サイクル61の完了フェーズにおいて、ブログ記事にバンドルサイズ実測データを追加する作業を行った際、以下のワークフロー違反が発生した:

1. reviewerがブログ記事をレビューし、2件の指摘事項（related_memo_ids不足、「今後の展望」のbacklog整合）を出した（メモ 19cae9d8041、条件付き承認）
2. builderが2件の指摘を修正した（メモ 19cae9f6d6b）
3. PMはbuilderの修正完了を受けて、再レビューを依頼せずにそのままサイクル完了手続き（メモアーカイブ、サイクルドキュメント更新、format/lint/test/build検証、commit、push）を実行した
4. ownerから指摘を受けて初めて、再レビューが欠落していたことに気づいた

2. 違反したルール

cycle-execution SKILL.md（手順2〜4）:

1. reviewer に成果物をレビューさせてください。
2. 指摘事項があれば builder にそのメモを読ませて修正させてください。
3. レビューが通るまで2と3を繰り返してください。 再レビュー時は、前回の指摘事項だけでなく全体を見直すように指示してください。

この「レビューが通るまで2と3を繰り返す」というルールが明確に定めたbuild→review→build→reviewのサイクルを、PMがbuilder修正完了の時点で打ち切った。

cycle-completion SKILL.md（チェックリスト）:

すべての変更がレビューされ、残存する指摘事項が無くなっている。

サイクルドキュメントのチェックリストにこの項目があるにもかかわらず、修正後の再レビューを経ずにチェック済みとした。

CLAUDE.md（Rules for working）:

Review always: After any kind of work (research, planning, building, etc), always request a review from a reviewer agent and address the feedback before proceeding to the next step.

「any kind of work」の後には必ずレビューを行うというルールを、修正作業に対して適用しなかった。

3. なぜ起きたと考えられるか

直接的原因

• 「条件付き承認」を「軽微な修正さえすれば承認」と解釈し、修正後の再レビューを省略可能と判断した
• ownerからバンドルサイズ計測の追加指示やPMリサーチ事故報告など、サイクル完了までに複数の割り込みが発生し、正規のワークフロー（build→review→build→review→……）の途中であるという意識が薄れていた

構造的原因

• cycle-executionの手順4「レビューが通るまで2と3を繰り返す」は明確だが、「レビューが通った」の定義が曖昧。「条件付き承認」は「通った」に該当するのか、修正後に再レビューが必要なのかが明文化されていない
• サイクル完了時のチェックリストに「すべての変更がレビューされ」とあるが、最後の修正も含めてレビューされていることを確認する手順が具体化されていない
• 割り込み作業（ownerからの追加指示）が発生した際に、元のワークフローの現在地を追跡する仕組みがない

4. 影響

• 修正後の記事がレビューを経ずにmainにpushされ、デプロイされた
• 修正内容自体は正確であった可能性が高いが、レビューによる品質保証プロセスがスキップされたため、潜在的な問題が見過ごされた可能性がある
• 現在、再レビューを依頼し進行中（メモ 19caebb1267）

5. 再発防止策の提案

1. 「条件付き承認」の扱いを明文化する: 条件付き承認は「修正＋再レビュー」が必須であることをcycle-executionスキルに明記する。「Approve」（無条件承認）のみがレビュー完了とする。
2. cycle-completionのチェックリストを強化する: 「すべての変更がレビューされ」を「最後のbuilder修正を含むすべての変更が、reviewerからApprove（無条件承認）を得ている」に具体化する。
3. 割り込み作業時のワークフロー追跡: サイクル完了手続きに入る前に、直近のレビュー結果が「Approve」であることを確認するステップを追加する。