river-review-performance

Performance Review（パフォーマンスレビュー）

パフォーマンスに影響する変更を検出し、適切な個別スキルで検証する。

When to Use / いつ使うか

• データベースクエリの追加・変更時
• ループ処理やバッチ処理の変更時
• キャッシュ戦略の変更時
• 大量データの処理ロジック変更時

Routing / ルーティング

キーワード	スキルID	説明
キャッシュ, TTL	rr-upstream-cache-strategy-consistency-001	キャッシュ戦略の一貫性
障害, 監視, メトリクス	rr-upstream-failure-modes-observability-001	障害モードと可観測性
ログ, トレース	rr-midstream-logging-observability-001	ロギング・可観測性
SLO, レイテンシ	rr-upstream-operability-slo-001	運用性・SLO

デフォルト動作

• キーワード指定なし → 以下のヒューリスティクスで判定:
  • ループ内I/O → N+1クエリ検出
  • 大量データ処理 → メモリ効率チェック
  • 外部API呼び出し → タイムアウト・リトライ検証

Execution Flow / 実行フロー

1. 変更内容の分析
   ├─ ループ内I/O → N+1クエリ検出を優先
   ├─ 大量データ処理 → メモリ効率チェックを優先
   ├─ 外部API呼び出し → タイムアウト・リトライ検証を優先
   └─ キーワード指定あり → 該当スキルを直接選択

2. スキルの実行
   ├─ cache-strategy-consistency: キャッシュ戦略の一貫性
   ├─ failure-modes-observability: 障害モードと可観測性
   ├─ logging-observability: ロギング・可観測性
   └─ operability-slo: 運用性・SLO

3. 統合
   ├─ 重複する指摘の除去
   └─ Checklistに基づくパフォーマンスチェックの補完

Checklist / チェックリスト

パフォーマンスレビューでは以下を確認する:

クエリ効率

• N+1クエリが発生していないか
• 必要なeager loadingが設定されているか
• 不要なカラムを取得していないか（SELECT *）

メモリ効率

• ループ内での不要なオブジェクト生成がないか
• 大量データのストリーム処理が適切か
• メモリリークのパターンがないか

I/O効率

• 外部API呼び出しのタイムアウト設定
• リトライ戦略の妥当性
• 並列化可能なI/Oの逐次実行

キャッシュ

• キャッシュキーの設計が適切か
• TTLが妥当か
• キャッシュの無効化戦略

Output Format / 出力形式

<file>:<line>: <message>

• Finding: 何が問題か（1文）
• Impact: 推定される影響（レイテンシ増加、メモリ消費等）
• Fix: 次の一手（最小の修正案）

他スキルとの関係

スキル	関係	棲み分け
river-review-architecture	補完	performance は「実行時効率」、architecture は「構造的スケーラビリティ」
river-review-code	補完	performance は「速度・効率」、code は「可読性・保守性」

References

• ROUTING.md: 詳細なルーティングルール