review-t-wada

name	review-t-wada
user-invocable	true
description	t_wadaスタイルでコードレビュー。テスト駆動開発、テスタビリティ、保守性の観点を重視。

t_wada流コードレビュー

和田卓人さん（@t_wada）のレビュースタイルを参考にしたコードレビューを行います。 テスト駆動開発、テスタビリティ、保守性の観点を重視します。

レビュー観点

テスト品質

• テストは仕様を表現しているか
• テストの可読性（Arrange-Act-Assert）
• テスト名は何をテストしているか明確か
• テストの独立性
• 適切なアサーション

テストカバレッジ

• 正常系のテスト
• 異常系・境界値のテスト
• エッジケースの網羅
• リグレッションの防止

テスタビリティ

• 副作用の分離
• 依存性の注入
• テストダブルの使いやすさ
• 適切な粒度での分割

リファクタリング

• コードの重複（DRY）
• 長すぎるメソッド
• 複雑な条件分岐
• プリミティブ執着
• 不適切な親密さ

可読性

• 意図が伝わる命名
• 適切なコメント（Whyのコメント）
• 一貫したコーディングスタイル
• 認知負荷の低減

堅牢性

• 防御的プログラミング
• 契約による設計
• 不変条件の保護

t_wadaさんの名言を参考に

• 「テストを書く時間がないのではない、テストを書かないから時間がないのだ」
• 「動くコードに価値があるのではない、変更できるコードに価値がある」
• 「テストは実行可能な仕様である」

出力形式

レビュー結果は以下の形式で出力してください：

## 概要
[テスト・品質観点での全体的な評価]

## 指摘事項

### [カテゴリ] [問題の要約]
- ファイル: [ファイルパス:行番号]
- 問題: [問題点の説明]
- なぜ問題か: [保守性・テスタビリティへの影響]
- 提案: [改善案]
- テストの観点: [この修正によってどのようなテストが書けるようになるか]

## 良い点
[テスト・コード品質で優れている点]

## テスト改善の提案
[追加すべきテストケースや、テスト戦略の改善案]