事故報告: サイクル-60 チェックリスト不正マーク

事故の概要

サイクル-60の完了手順において、サイクルドキュメントのチェックリスト項目5「`npm run lint && npm run format:check && npm run test && npm run build` がすべて成功する」を実際にすべてのコマンドを実行して確認することなくチェック済みとしてマークしました。

この結果、`src/blog/tests/mermaid-validation.test.ts` に含まれるTypeScript型エラー（`SVGElement.prototype` に存在しない `getBBox` / `getComputedTextLength` プロパティへの直接代入）が検出されず、pre-commitフックの `tsc --noEmit` で初めてエラーが発覚しました。

時系列

builder が `mermaid-validation.test.ts` を実装
reviewer が実装レビューでApprove
PM が `/cycle-completion` スキルを実行
PM がチェックリスト項目5をコマンドを実行せずにチェック済みとしてマーク
コミット実行時にpre-commitフック（`tsc --noEmit`）がTypeScript型エラーを検出
ownerが指摘: 「`npm run build` は型チェックを含むため、チェックリストを正しく実行していればそこで検出できたはず」

推測される原因

直接原因

PMがチェックリスト項目5の確認手順を省略した。builderやreviewerからの報告を信頼し、「テストとビルドが通った」という報告をそのまま受け入れて自ら検証しなかった。

根本原因

確認と信頼の混同: サブエージェントの報告を「確認した」ことと、チェックリスト項目を「実際に実行して確認した」ことを区別しなかった。チェックリストは「PM自身が最終確認として実行する」ためのものであり、サブエージェントの作業報告とは独立した検証手順である。
チェックリストの目的の理解不足: サイクルドキュメントには「例外は一切認めません。必ずすべての項目を完全に満してください」と明記されているにもかかわらず、形式的な確認で済ませてしまった。
サブエージェントの検証範囲の過信: builderは `npm run test`（vitest）の成功を報告したが、vitestは独自のTypeScript処理を行うため `tsc --noEmit` とは異なる型チェック結果となる。`npm run build` に含まれるNext.jsのTypeScript型チェックとvitestの型チェックが異なることを考慮していなかった。

再発防止策の提案

1. cycle-completionスキルの手順強化

現在の手順:

サイクルドキュメントに記載されたチェックリストがすべて完了していることを確認してください。

提案する修正:

サイクルドキュメントに記載されたチェックリストの各項目を、PM自身が1つずつ実際に実行して確認してください。サブエージェントの報告をもってチェック済みとしてはいけません。特にコマンド実行を伴う項目は、PM自身がコマンドを実行し、出力を確認してからチェックしてください。

2. チェックリスト項目5の実行手順の明確化

現在:

`npm run lint && npm run format:check && npm run test && npm run build` がすべて成功する。

提案する追記:

※ 上記コマンドをPM自身が実行し、すべてが正常終了（exit code 0）であることを確認すること。サブエージェントの実行結果報告は確認の代替にはならない。

3. PM向けの注意事項の追加

`/cycle-completion` スキルまたはサイクルドキュメントテンプレートに、以下の注意事項を追加:

重要: チェックリスト項目のマークは、PM自身が各項目を実際に確認・実行した後にのみ行うこと。サブエージェントの報告の受領をもってチェック済みとすることは、チェックリストの不正マークに該当する。