takt/builtins/ja/instructions/review-arch.md
nrslib fa42ef7561 facet: 契約文字列のハードコード散在を防止するルールを追加
ポリシー・実装インストラクション・アーキテクチャレビューの3箇所に
ファイル名・設定キー名などの契約文字列を定数で一箇所管理するルールを追加。
2026-02-22 00:32:11 +09:00

1.3 KiB
Raw Blame History

アーキテクチャと設計のレビューに集中してください。 AI特有の問題はレビューしないでくださいai_reviewムーブメントで実施済み

レビュー観点:

  • 構造・設計の妥当性
  • コード品質
  • 変更スコープの適切性
  • テストカバレッジ
  • デッドコード
  • 呼び出しチェーン検証
  • 契約文字列(ファイル名・設定キー名)のハードコード散在

前回指摘の追跡(必須):

  • まず「Previous Response」から前回の open findings を抽出する
  • 各 finding に finding_id を付け、今回の状態を new / persists / resolved で判定する
  • persists と判定する場合は、未解決である根拠(ファイル/行)を必ず示す

判定手順

  1. まず前回open findingsを抽出し、new / persists / resolved を仮判定する
  2. 変更差分を確認し、構造・設計の観点に基づいて問題を検出する
    • ナレッジの判定基準テーブルREJECT条件と変更内容を照合する
  3. 検出した問題ごとに、Policyのスコープ判定表と判定ルールに基づいてブロッキング/非ブロッキングを分類する
  4. ブロッキング問題(new または persistsが1件でもあればREJECTと判定する