flutter-review-request

Flutter コードレビューの依頼

問題が連鎖する前に検出するため、flutter-craft:flutter-code-reviewer サブエージェントを呼び出します。

基本原則： 早めに、頻繁にレビューします。

開始時に宣言：「コードレビューを依頼するため、flutter-review-request スキルを使用します。」

レビューを依頼するタイミング

必須：

• サブエージェント主導の開発では、各タスクの完了後
• 大きな機能の完成後
• main にマージする前
• 新しいクリーンアーキテクチャレイヤーの実装後

任意ですが有益：

• 行き詰まったとき（新たな視点を得るため）
• リファクタリング前（ベースラインの確認）
• 複雑なバグの修正後
• 新しい状態管理の追加後

依頼方法

1. git SHA を取得する

BASE_SHA=$(git rev-parse HEAD~1)  # or origin/main
HEAD_SHA=$(git rev-parse HEAD)

2. flutter-code-reviewer サブエージェントを呼び出す

Task ツールで flutter-craft:flutter-code-reviewer タイプを使用します：

Task tool:
  subagent_type: "flutter-craft:flutter-code-reviewer"
  prompt: |
    ## What Was Implemented
    {WHAT_WAS_IMPLEMENTED}

    ## Requirements/Plan
    {PLAN_OR_REQUIREMENTS}

    ## Git Range to Review
    Base: {BASE_SHA}
    Head: {HEAD_SHA}

    ## Files Changed
    {FILES_CHANGED}

    Please review for:
    - Clean Architecture compliance
    - State management patterns
    - Flutter best practices
    - Test coverage (priority-based)

3. フィードバックに対応する

問題の重大度	対応
Critical	作業を続ける前にただちに修正します
Important	次のタスクに進む前に修正します
Minor	後で対応する項目として記録し、作業を続けられます

Flutter 固有のレビューチェックリスト

クリーンアーキテクチャ、状態管理、ウィジェットの構成、Dart のスタイル、優先度に基づくテストカバレッジ（ゴールデンテストを含む）についての完全なチェックリストは、レビュー担当エージェント自身（agents/flutter-code-reviewer.md）にあります。ここが唯一の信頼できる情報源です。内容をここに重複して記載しないでください。呼び出されたエージェントがチェックリストを適用します。

例

[Just completed Task 3: Add AuthBloc]

You: Let me request code review before proceeding.

BASE_SHA=$(git rev-parse HEAD~3)
HEAD_SHA=$(git rev-parse HEAD)

[Dispatch flutter-craft:flutter-code-reviewer subagent]
  WHAT_WAS_IMPLEMENTED: Authentication feature with login/logout
  PLAN_OR_REQUIREMENTS: Task 1-3 from docs/plans/auth-plan.md
  BASE_SHA: a7981ec
  HEAD_SHA: 3df7661
  FILES_CHANGED:
    - lib/features/auth/domain/entities/user.dart
    - lib/features/auth/domain/repositories/auth_repository.dart
    - lib/features/auth/data/repositories/auth_repository_impl.dart
    - lib/features/auth/presentation/bloc/auth_bloc.dart

[Subagent returns]:
  ### Strengths
  - Clean layer separation
  - Proper BLoC state handling
  - Good error states

  ### Issues
  #### Important
  - Missing AuthBloc unit tests (auth_bloc_test.dart)
  - No error boundary in LoginScreen

  #### Minor
  - Consider using freezed for state classes

  ### Assessment
  **Ready to merge: With fixes**
  Fix the Important issues before proceeding.

You: [Add AuthBloc tests, add error boundary]
[Continue to Task 4]

ワークフローとの連携

サブエージェント主導の開発

• 各タスクの後にレビューする
• 問題が積み重なる前に検出する
• 次のタスクに進む前に修正する

プランの実行

• 各バッチ（3 タスク）の後にレビューする
• フィードバックを受け取り、反映してから続ける

アドホックな開発

• マージ前にレビューする
• 行き詰まったときにレビューする

危険信号

絶対にしないこと：

• 「単純な Flutter コードだから」という理由でレビューを省略する
• Critical の問題を無視する
• Important の問題を未修正のまま作業を続ける
• テストに関するフィードバックを無視する

レビュー担当者の指摘が誤っている場合：

• 技術的な根拠を示して異議を唱える
• 動作を証明する flutter test の出力を提示する
• Flutter 固有の懸念点について説明を求める

必須のサブスキル

レビューのフィードバックを受け取った後、必ず以下を呼び出してください： → flutter-craft:flutter-review-receive

続行する前に、フィードバックを順序立てて処理します。