quality

品質

完了した調理作業をレビューします。評価を含む stage_message イベントを出力してください。イベントのスキーマについては、references/stage-message-schema.md を参照してください。

完全に自律して作業してください。ユーザーに質問してはいけません。

ステップ 0 — 原則を読み込む

brain/principles.md を読み、すべての [[wikilink]] に従ってください。これらはレビューの指針です。省略してはいけません。

評価パイプライン

順番に実行してください。重大度の高い指摘事項があれば、その時点で停止して却下してください。

1. スコープの確認

• 調理担当に割り当てられた計画フェーズを確認します（調理担当の最初のプロンプトまたは mise コンテキストから）。
• 調理担当のコミットについて、git diff --stat と git log --oneline を実行します。
• 計画フェーズで定められたスコープ外の変更ファイルがあれば、スコープ違反として指摘します。

2. コード品質

• go vet ./... と go test ./... が成功しなければなりません。
• sh scripts/lint-arch.sh が存在する場合は実行します。
• エラー処理がプロジェクトの規約（期待を示すメッセージではなく、失敗状態を示すメッセージ）に従っているか確認します。

3. 原則への準拠

• 変更されたファイルごとに、読み込んだ原則に照らして確認します。
• よくある違反：再設計ではなく変更を後付けしている（redesign-from-first-principles）、検証が不足している（prove-it-works）、不必要に複雑さを加えている（subtract-before-you-add）。

4. テストの証拠

• 新しい動作には、新しいテストが必要です。
• テストでは実装の詳細ではなく、結果を検証しなければなりません。
• 計画フェーズで言及されているエッジケースが網羅されているか確認します。

5. 実行時の検証

• 計画フェーズで実行時の確認が指定されている場合は、実施されたことを検証します。
• 「コンパイルできる」だけでは検証になりません。

判定

評価を含む stage_message イベントを出力してください。

noodle event emit --session $NOODLE_SESSION_ID stage_message --payload '<json>'

承認：blocking: false を指定して出力します。パイプラインが続行され、スケジューラーに承認が伝わります。

{ "message": "All checks pass. Tests green, scope clean.", "blocking": false }

却下：blocking: true を指定して出力します（または blocking を省略します）。スケジューラーは指摘事項を読み、対応（再試行、oops ステージの追加、または中止）を判断します。

{ "message": "Rejected: 3 high issues. [1] Missing test for edge case in handleCompletion. [2] Scope violation: modified cook_merge.go outside plan phase scope. [3] Error message uses expectation form.", "blocking": true }

スケジューラーが対応できる自然な文章でメッセージを記述してください。再試行する調理担当に指示できるよう、具体的なファイルパス、行の文脈、原則違反を含めてください。

ブロック対象外の問題

却下するほどではない問題（スタイル上の細かな指摘、軽微な改善、ドキュメントの不足）は、バックログアダプターを使ってバックログ項目として登録してください。

原則

• [[prove-it-works]]
• [[outcome-oriented-execution]]