sadokulisted

# 査読（sadoku） 本番コードとエージェントが実行するMarkdownを、実装者の説明から独立してレビューする。レビュー中は対象を変更しない。 <!-- hikizan:contract:start --> ## 共通ルール 全スキル共通。正本は`scripts/contract.md`で、`scripts/gen-contract.sh`が各`SKILL.md`のこの区間に書き込む（手で編集しない）。 - 各スキルを起動したら、そのスキルの作業を始める直前に1行だけ`🌲 <スキル名>（日本語名）：<今回の目的>`と伝える。複数スキルを1行にまとめず、まだ始めないスキルを予告しない。同じスキル内の局所作業では繰り返さない - 調査、相談、設計、レビューだけの依頼では対象を変更しない。修正、追加、削除、実行、PR提出が依頼に含まれる場合だけ、必要なスキルをつないで明示された終点まで進む - スキルを固定順に通さず、依頼された成果に必要な観点だけを使う。明示済みの終点へ向かう途中で、形式的な承認を追加しない - 利用者が示した例や問題箇所を変更範囲そのものとみなさない。背景の原因と守るべき規則を確認し、同じ原因を防ぐ最小の共通箇所を変更する。要求外の一般化はしない - 検証はリスクに比例させ、未検証の状態を成功や完了と書かない - 人へ渡す日本語は結果か判断を先に置き、簡潔で分かりやすく書く。文章の表現や構成自体が成果なら`houkoku`を使う - PRのマージと既定ブランチへの直接のpush、公開・配布・本番環境や共有データを変更する操作は、利用者が依頼の終点として明示した場合だけ行う。「PRまで」はマージを含めない。明示済みなら作業判断のために再確認せず、ハーネスが実行直前の確認を表示した場合はその結果に従う - 停止するときに意味のある次の進め方があれば、最大3件を推奨順に`A（あ）`、`I（い）`、`U（う）`で示し、英字とひらがなのどちらの回答も同じ選択として扱う <!-- hikizan:contract:end --> ## 使い分け - レビュー：バグ、回帰、セキュリティ、既存コードベースとの不整合を探す - 簡略化：振る舞いを変えずに減らせる分岐・層・重複・不要コードを探す ## 手順 1. 対象時点を固定する。ブランチ全体の比較元は利用者・PR指定、なければ選んだリモートの既定ブランチとし、機能ブランチの上流ブランチを比較元にしない。`merge-base`からのコミット済み・ステージ済み・未ステージ・未追跡ファイルを含め、途中で別の対象範囲へ読み替えない 2. 変更意図、関連テスト、近隣の類似実装、リポジトリ規約を必要な範囲だけ読む 3. リスクで深さを決める。局所的で可逆なら軽量、振る舞い・API・複数モジュールへ波及するなら標準、セキュリティ・権限・スキーマ・移行・データ損失・並行性・取り消し費用が高い変更なら重点とする 4. 正しさ、失敗経路、既存パターン、より単純な表現、追加行への秘密情報・個人情報混入を確認する。修正では、提示されたケースだけを通す局所パッチになっていないか、反対に要求と無関係な一般化へ広がっていないかも確認する。行数やファイル数だけでレビュー担当を増やさない 5. 独立した専門性が結果を改善する場合だけ`references/persona-catalog.md`の契約で専門レビューを行う 6. UI・スタイル・配置・操作に触れる場合は、`references/visual-