レビュー
このスキルは、すでに存在する作業を評価するために使用します。レビューは評価対象の変更を執筆しません。
これは標準的なレビューワークフローです。merge-readinessはここにルーティングされ、ai-slop-cleanerはその中のオプトインレーンです。
目標
実際に問題がある箇所を、深刻度順に、行動に移せるだけの詳細とともに特定します。
ワークフロー
- 何が変更され、何を意図していたのかを把握する。
- その意図に照らして変更を評価する。
- 最初に正しさを確認し、次にリスク、最後に簡略化を確認する。
- 報告する前に、候補となる各発見事項を検証する。
- 最も深刻な発見事項から先に報告する。
確認項目
- 正しさ — ロジックの欠陥、エッジケース、エラーパス、並行処理
- リスク — セキュリティ境界、破壊的操作、データ整合性
- 再利用 — 変更が使用すべきだった既存のユーティリティやパターン
- 簡略化 — 削除または統合できるコード
- カバレッジ — テストされずに出荷される動作
ルール
- コンテキストを分離する：レビュー担当者は、作成者と同じアクティブコンテキストであってはなりません。
- 報告前に検証する：もっともらしく聞こえるが再現できない発見はノイズです。
- 深刻度を正直に述べ、リストを水増しして見栄えを良くしないでください。
- 「発見事項なし」は、作業内容が適切である場合の正当な結果です。
- デフォルトはアドバイザリ（助言）です。レビューは情報を提供するものであり、ゲート（必須の承認）にはなりません。リリース、セキュリティ、破壊的操作などのハードゲートは別途管理し、閉じた障害として扱います。
出力
- 発見事項（最も深刻なものから順に、ファイル、行番号、具体的な失敗シナリオを含む）
- 確認したが問題がなかった項目
- 評価できなかった項目