cga-tf-refactoring

name: cga-tf-refactoring
description: Small, safe refactorings before feature work using Kent Beck's "Tidy First?" patterns (guard clauses, extract helper, move to model, etc.). Invoke as /cga-tf-refactoring [file] or use when preparing code for changes, like "tidy this file before adding feature", "refactor before implementing", "clean up structure before modifying", "整理してから機能追加", "リファクタリングしてから変更". NOT for large refactorings or code without tests.
allowed-tools: Read Glob Grep Edit Write Bash AskUserQuestion
disable-model-invocation: true

いつ使うか

• 新機能を追加する前に、コードの構造を整理したいとき
• 既存コードに手を入れる前に、変更しやすくしたいとき
• 「このコード、もう少し整理してから機能追加したい」と感じたとき

いつ使わないか

• テストがなく安全に変更できないとき（先にテストを書く）
• 大規模な書き換えが必要なとき（別タスクとしてプランニングする）
• 今回の変更に関係ない箇所の整理（スコープ外）

何をするか

Tidy First = 機能変更の前に、小さな構造改善を先に行う

1. 対象コードを分析し、変更を妨げる「構造の問題」を特定
2. 機能変更に必要な最小限の整理（Tidying）を提案
3. 整理を実施（テストが通ることを確認）
4. その後、本来の機能変更に進む

Tidyingのパターン

中断条件

以下の場合は整理を中断し、開発者に状況を報告する:

• テスト失敗: 整理後にテストが通らない場合はrevertして報告
• スコープ超過: 整理対象が3ファイルを超える場合は停止して確認を求める
• 振る舞い変更の懸念: 整理がロジックの振る舞いに影響する可能性がある場合

使用例

/cga-tf-refactoring src/ordering/usecases/create-order.ts に商品バリデーション追加したいので整理して
/cga-tf-refactoring このファイルをキャンセル機能追加しやすくして

ワークフロー

全体フロー

/cga-explore-planning  # 実装計画の作成
  ↓
/cga-tf-refactoring   # コード整理（本スキル）
  ↓
/cga-programming      # 機能実装
  ↓
/cga-review           # レビュー

本スキル内のフロー

1. 変更対象のコードを読む
   ↓
2. 「このままプログラミングできるか?」を考える
   ↓
3-A. できる → そのまま /cga-programming へ
3-B. 整理したい → Tidyingを特定
   ↓
4. Tidyingを実施（テストが通ることを確認）
   ↓
5. Tidyingをコミット（"tidy:" プレフィックス）
   ↓
6. /cga-programming でプログラミングを実施

成果物

• Tidyingコミット（"tidy:"プレフィックス）
• テスト実行結果（すべてPASS）
• 次ステップの提案（/cga-programmingへ移行 or 追加Tidying必要性）

トラブルシューティング

テストが失敗した場合

1. 変更を即座にrevert: git checkout -- <file>
2. テストログを確認し、振る舞いが変わった箇所を特定
3. より小さなTidyingに分解するか、今回は見送る

スコープが広がりすぎた場合

兆候: 3ファイル以上に変更が必要、または30分以上かかりそう

1. 現在の変更をstash: git stash
2. ユーザーに状況を報告し、分割案を提示
3. 承認を得てから続行、または見送り

振る舞い変更の懸念がある場合

例: ロジックの条件式を変更、エラーハンドリングの追加など

1. 変更を一時停止
2. ユーザーに「これは振る舞いを変える可能性があります」と報告
3. テストでカバーされているか確認し、承認を得る

Tidyingパターンが見つからない場合

1. 無理に整理しない（過剰なTidyingを避ける）
2. 「現状のまま機能追加可能」と判断し、/cga-programmingへ進む

詳細

guide.md を参照