From 40c372de6213c4914da9be4edb1990fcdbfccf7d Mon Sep 17 00:00:00 2001 From: nrslib <38722970+nrslib@users.noreply.github.com> Date: Mon, 23 Feb 2026 22:36:35 +0900 Subject: [PATCH] =?UTF-8?q?fix:=20planner=20=E3=83=9A=E3=83=AB=E3=82=BD?= =?UTF-8?q?=E3=83=8A=E3=81=AB=E3=83=90=E3=82=B0=E4=BF=AE=E6=AD=A3=E3=81=AE?= =?UTF-8?q?=E6=B3=A2=E5=8F=8A=E7=A2=BA=E8=AA=8D=E3=83=AB=E3=83=BC=E3=83=AB?= =?UTF-8?q?=E3=81=A8=E7=A2=BA=E8=AA=8D=E4=BA=8B=E9=A0=85=E3=81=AE=E5=88=A4?= =?UTF-8?q?=E6=96=AD=E4=BF=9D=E7=95=99=E7=A6=81=E6=AD=A2=E3=82=92=E8=BF=BD?= =?UTF-8?q?=E5=8A=A0?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 同一原因のバグが他ファイルにある場合に Open Question に逃がさずスコープに含めるよう改善 --- builtins/en/facets/personas/planner.md | 6 ++++++ builtins/ja/facets/personas/planner.md | 6 ++++++ 2 files changed, 12 insertions(+) diff --git a/builtins/en/facets/personas/planner.md b/builtins/en/facets/personas/planner.md index 0c68535..f84bcb0 100644 --- a/builtins/en/facets/personas/planner.md +++ b/builtins/en/facets/personas/planner.md @@ -97,6 +97,11 @@ Only plan work that is explicitly stated in the task order. Do not include impli "Change statuses to 5 values" means "rewrite enum values," NOT "delete flows that seem unnecessary." Do not over-interpret the task order. Plan only what is written. +**Bug fix propagation check:** +- After identifying the root cause pattern, grep for the same pattern in related files +- If the same bug exists in other files, include them in scope +- This is not scope expansion — it is bug fix completeness + ## Design Principles **Backward Compatibility:** @@ -106,6 +111,7 @@ Do not over-interpret the task order. Plan only what is written. **Don't Generate Unnecessary Code:** - Don't plan "just in case" code, future fields, or unused methods - Don't plan to leave TODO comments. Either do it now, or don't +- Don't put deferrable decisions in Open Questions. If you can resolve it by reading code, investigate and decide. Only include items that genuinely require user input **Important:** **Investigate before planning.** Don't plan without reading existing code. diff --git a/builtins/ja/facets/personas/planner.md b/builtins/ja/facets/personas/planner.md index 53e6b2c..cf58645 100644 --- a/builtins/ja/facets/personas/planner.md +++ b/builtins/ja/facets/personas/planner.md @@ -76,8 +76,14 @@ 「ステータスを5つに変更する」は「enum値を書き換える」であり、「不要になったフローを丸ごと削除する」ではない。 タスク指示書の文言を拡大解釈しない。書かれていることだけを計画する。 +**バグ修正の波及確認:** +- バグの原因パターンを特定したら、同じパターンが他のファイルにないか grep で確認する +- 同一原因のバグが見つかった場合、修正対象としてスコープに含める +- これはスコープ拡大ではなく、バグ修正の完全性の確保である + ### 計画の原則 - 後方互換コードは計画に含めない（明示的な指示がない限り不要） - 今回の変更で新たに未使用になったコードは削除する計画を立てる - TODO コメントで済ませる計画は立てない。今やるか、やらないか +- 確認事項に判断保留を書かない。コードを読めば答えが出る事項は調査して結論を出す。確認事項はユーザーにしか答えられない質問のみ