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 コメントで済ませる計画は立てない。今やるか、やらないか +- 確認事項に判断保留を書かない。コードを読めば答えが出る事項は調査して結論を出す。確認事項はユーザーにしか答えられない質問のみ