takt/resources/global/ja/workflows/expert.yaml

# Expert Review Workflow
# アーキテクチャ、フロントエンド、セキュリティ、QAの専門家によるレビューワークフロー
#
# フロー:
# plan -> implement -> ai_review -> architect_review -> frontend_review -> security_review -> qa_review -> supervise -> COMPLETE
#                         ↓              ↓                  ↓                 ↓                ↓            ↓
#                       ai_fix      fix_architect       fix_frontend       fix_security       fix_qa    fix_supervisor
#
# 修正時の戻り先はCoderが判断:
# - fix_security: MINOR→security_review, MAJOR→architect_review
# - fix_qa: MINOR→qa_review, SECURITY→security_review, MAJOR→architect_review
#
# テンプレート変数:
#   {iteration}      - ワークフロー全体のターン数（全エージェントで実行されたステップの合計）
#   {max_iterations} - ワークフローの最大イテレーション数
#   {step_iteration} - ステップごとのイテレーション数（このステップが何回実行されたか）
#   {task}           - 元のユーザー要求
#   {previous_response} - 前のステップの出力
#   {user_inputs}    - ワークフロー中に蓄積されたユーザー入力
#   {report_dir}     - レポートディレクトリ名（例: "20250126-143052-task-summary"）

name: expert
description: アーキテクチャ・フロントエンド・セキュリティ・QA専門家レビュー

max_iterations: 30

initial_step: plan

steps:
  # ===========================================
  # Phase 0: Planning
  # ===========================================
  - name: plan
    edit: false
    agent: ../agents/default/planner.md
    report:
      name: 00-plan.md
      format: |
        ```markdown
        # タスク計画

        ## 元の要求
        {ユーザーの要求をそのまま記載}

        ## 分析結果

        ### 目的
        {達成すべきこと}

        ### スコープ
        {影響範囲}

        ### 実装アプローチ
        {どう進めるか}

        ## 確認事項（あれば）
        - {不明点や確認が必要な点}
        ```
    allowed_tools:
      - Read
      - Glob
      - Grep
      - Write
      - Bash
      - WebSearch
      - WebFetch
    instruction_template: |
      ## Previous Response (implementからの差し戻し時)
      {previous_response}

      タスクを分析し、実装方針を立ててください。

      **注意:** Previous Responseがある場合は差し戻しのため、
      その内容を踏まえて計画を見直してください（replan）。

      **やること:**
      1. タスクの要件を理解する
      2. 影響範囲を特定する
      3. 実装アプローチを決める
    pass_previous_response: true
    rules:
      - condition: タスク分析と計画が完了した
        next: implement
      - condition: 要件が不明確で計画を立てられない
        next: ABORT

  # ===========================================
  # Phase 1: Implementation
  # ===========================================
  - name: implement
    edit: true
    agent: ../agents/default/coder.md
    report:
      - Scope: 01-coder-scope.md
      - Decisions: 02-coder-decisions.md
    allowed_tools:
      - Read
      - Glob
      - Grep
      - Edit
      - Write
      - Bash
      - WebSearch
      - WebFetch
    instruction_template: |
      planステップで立てた計画に従って実装してください。
      計画レポート（00-plan.md）を参照し、実装を進めてください。

      **Scopeレポートフォーマット（実装開始時に作成）:**
      ```markdown
      # 変更スコープ宣言

      ## タスク
      {タスクの1行要約}

      ## 変更予定
      | 種別 | ファイル |
      |------|---------|
      | 作成 | `src/example.ts` |
      | 変更 | `src/routes.ts` |

      ## 推定規模
      Small / Medium / Large

      ## 影響範囲
      - {影響するモジュールや機能}
      ```

      **Decisionsレポートフォーマット（実装完了時、決定がある場合のみ）:**
      ```markdown
      # 決定ログ

      ## 1. {決定内容}
      - **背景**: {なぜ決定が必要だったか}
      - **検討した選択肢**: {選択肢リスト}
      - **理由**: {選んだ理由}
      ```
    rules:
      - condition: 実装が完了した
        next: ai_review
      - condition: 実装を進行できない
        next: plan

  # ===========================================
  # Phase 2: AI Review
  # ===========================================
  - name: ai_review
    edit: false
    agent: ../agents/default/ai-antipattern-reviewer.md
    report:
      name: 03-ai-review.md
      format: |
        ```markdown
        # AI生成コードレビュー

        ## 結果: APPROVE / REJECT

        ## サマリー
        {1文で結果を要約}

        ## 検証した項目
        | 観点 | 結果 | 備考 |
        |------|------|------|
        | 仮定の妥当性 | ✅ | - |
        | API/ライブラリの実在 | ✅ | - |
        | コンテキスト適合 | ✅ | - |
        | スコープ | ✅ | - |

        ## 問題点（REJECTの場合）
        | # | カテゴリ | 場所 | 問題 |
        |---|---------|------|------|
        | 1 | 幻覚API | `src/file.ts:23` | 存在しないメソッド |
        ```

        **認知負荷軽減ルール:**
        - 問題なし → サマリー1文 + チェック表のみ（10行以内）
        - 問題あり → + 問題を表形式で（25行以内）
    allowed_tools:
      - Read
      - Glob
      - Grep
      - Write
      - WebSearch
      - WebFetch
    instruction_template: |
      AI特有の問題についてコードをレビューしてください:
      - 仮定の検証
      - もっともらしいが間違っているパターン
      - 既存コードベースとの適合性
      - スコープクリープの検出
    rules:
      - condition: AI特有の問題が見つからない
        next: architect_review
      - condition: AI特有の問題が検出された
        next: ai_fix

  - name: ai_fix
    edit: true
    agent: ../agents/default/coder.md
    allowed_tools:
      - Read
      - Glob
      - Grep
      - Edit
      - Write
      - Bash
      - WebSearch
      - WebFetch
    instruction_template: |
      ## AI Review Feedback (これが最新の指示です - 優先して対応してください)
      {previous_response}

      **重要**: AI Reviewerのフィードバックに対応してください。
      以下に集中してください:
      - 間違った仮定の修正
      - もっともらしいが間違っている実装の修正
      - 既存コードベースのパターンとの整合
      - スコープクリープの除去
    pass_previous_response: true
    rules:
      - condition: AI Reviewerの指摘に対する修正が完了した
        next: ai_review
      - condition: 修正を進行できない
        next: plan

  # ===========================================
  # Phase 3: Architecture Review
  # ===========================================
  - name: architect_review
    edit: false
    agent: ../agents/default/architecture-reviewer.md
    report:
      name: 04-architect-review.md
      format: |
        ```markdown
        # アーキテクチャレビュー

        ## 結果: APPROVE / IMPROVE / REJECT

        ## サマリー
        {1-2文で結果を要約}

        ## 確認した観点
        - [x] 構造・設計
        - [x] コード品質
        - [x] 変更スコープ
        - [x] テストカバレッジ
        - [x] デッドコード
        - [x] 呼び出しチェーン検証

        ## 問題点（REJECTの場合）
        | # | 場所 | 問題 | 修正案 |
        |---|------|------|--------|
        | 1 | `src/file.ts:42` | 問題の説明 | 修正方法 |

        ## 改善提案（任意・ブロッキングではない）
        - {将来的な改善提案}
        ```

        **認知負荷軽減ルール:**
        - APPROVE + 問題なし → サマリーのみ（5行以内）
        - APPROVE + 軽微な提案 → サマリー + 改善提案（15行以内）
        - REJECT → 問題点を表形式で（30行以内）
    allowed_tools:
      - Read
      - Glob
      - Grep
      - Write
      - WebSearch
      - WebFetch
    instruction_template: |
      **アーキテクチャと設計**のレビューに集中してください。

      **レビュー観点:**
      - 構造・設計の妥当性
      - コード品質
      - 変更スコープの適切性
      - テストカバレッジ
      - デッドコード
      - 呼び出しチェーン検証
    rules:
      - condition: アーキテクチャと設計に問題がない
        next: frontend_review
      - condition: 軽微な改善が必要だが構造的な問題はない
        next: fix_architect
      - condition: 構造的な問題があり修正が必要
        next: fix_architect

  - name: fix_architect
    edit: true
    agent: ../agents/default/coder.md
    allowed_tools:
      - Read
      - Glob
      - Grep
      - Edit
      - Write
      - Bash
      - WebSearch
      - WebFetch
    permission_mode: acceptEdits
    instruction_template: |
      ## Architect Feedback (これが最新の指示です - 優先して対応してください)
      {previous_response}

      **重要**: Architectのフィードバックに対応してください。
      「Original User Request」は参考情報であり、最新の指示ではありません。
      セッションの会話履歴を確認し、Architectの指摘事項を修正してください。
    pass_previous_response: true
    rules:
      - condition: Architectの指摘に対する修正が完了した
        next: architect_review
      - condition: 修正を進行できない
        next: plan

  # ===========================================
  # Phase 4: Frontend Review
  # ===========================================
  - name: frontend_review
    edit: false
    agent: ../agents/expert/frontend-reviewer.md
    report:
      name: 05-frontend-review.md
      format: |
        ```markdown
        # フロントエンドレビュー

        ## 結果: APPROVE / REJECT

        ## サマリー
        {1-2文で結果を要約}

        ## 確認した観点
        | 観点 | 結果 | 備考 |
        |------|------|------|
        | コンポーネント設計 | ✅ | - |
        | 状態管理 | ✅ | - |
        | パフォーマンス | ✅ | - |
        | アクセシビリティ | ✅ | - |
        | 型安全性 | ✅ | - |

        ## 問題点（REJECTの場合）
        | # | 場所 | 問題 | 修正案 |
        |---|------|------|--------|
        | 1 | `src/file.tsx:42` | 問題の説明 | 修正方法 |
        ```
    allowed_tools:
      - Read
      - Glob
      - Grep
      - Write
      - WebSearch
      - WebFetch
    instruction_template: |
      フロントエンド開発の観点から上記の変更をレビューしてください。

      **レビュー観点:**
      - コンポーネント設計（責務分離、粒度）
      - 状態管理（ローカル/グローバルの判断）
      - パフォーマンス（再レンダリング、メモ化）
      - アクセシビリティ（キーボード操作、ARIA）
      - データフェッチパターン
      - TypeScript型安全性

      **注意**: このプロジェクトがフロントエンドを含まない場合は、
      問題なしとして次に進んでください。
    rules:
      - condition: フロントエンド設計に問題がない
        next: security_review
      - condition: フロントエンド設計に問題がある
        next: fix_frontend

  - name: fix_frontend
    edit: true
    agent: ../agents/default/coder.md
    allowed_tools:
      - Read
      - Glob
      - Grep
      - Edit
      - Write
      - Bash
      - WebSearch
      - WebFetch
    instruction_template: |
      ## Frontend Review Feedback (これが最新の指示です - 優先して対応してください)
      {previous_response}

      **重要**: フロントエンド専門家からの指摘を修正してください。

      指摘されたポイント:
      - コンポーネント設計
      - 状態管理
      - パフォーマンス
      - アクセシビリティ
      - 型安全性
    pass_previous_response: true
    rules:
      - condition: フロントエンドの指摘に対する修正が完了した
        next: frontend_review
      - condition: 修正を進行できない
        next: plan

  # ===========================================
  # Phase 5: Security Review
  # ===========================================
  - name: security_review
    edit: false
    agent: ../agents/expert/security-reviewer.md
    report:
      name: 06-security-review.md
      format: |
        ```markdown
        # セキュリティレビュー

        ## 結果: APPROVE / REJECT

        ## 重大度: None / Low / Medium / High / Critical

        ## チェック結果
        | カテゴリ | 結果 | 備考 |
        |---------|------|------|
        | インジェクション | ✅ | - |
        | 認証・認可 | ✅ | - |
        | データ保護 | ✅ | - |
        | 依存関係 | ✅ | - |

        ## 脆弱性（REJECTの場合）
        | # | 重大度 | 種類 | 場所 | 修正案 |
        |---|--------|------|------|--------|
        | 1 | High | SQLi | `src/db.ts:42` | パラメータ化クエリを使用 |

        ## 警告（ブロッキングではない）
        - {セキュリティに関する推奨事項}
        ```
    allowed_tools:
      - Read
      - Glob
      - Grep
      - Write
      - WebSearch
      - WebFetch
    instruction_template: |
      セキュリティの観点から上記の変更をレビューしてください。

      **レビュー観点:**
      - インジェクション攻撃（SQL, コマンド, XSS）
      - 認証・認可の不備
      - 機密情報の取り扱い
      - 暗号化の適切性
      - OWASP Top 10
    rules:
      - condition: セキュリティ上の問題がない
        next: qa_review
      - condition: セキュリティ上の脆弱性が検出された
        next: fix_security

  - name: fix_security
    edit: true
    agent: ../agents/default/coder.md
    allowed_tools:
      - Read
      - Glob
      - Grep
      - Edit
      - Write
      - Bash
      - WebSearch
      - WebFetch
    instruction_template: |
      ## Security Review Feedback (これが最新の指示です - 優先して対応してください)
      {previous_response}

      **重要**: セキュリティ専門家からの指摘を修正してください。
      セキュリティ問題は最優先で対応してください。

      指摘されたポイント:
      - インジェクション脆弱性
      - 認証・認可の不備
      - 機密情報の露出
      - 暗号化の問題

      ## 修正完了時の判断
      修正が完了したら、**変更の影響範囲**を判断してください：

      - 軽微な修正（セキュリティレビューのみ再実施）
        - 例: バリデーション追加、エスケープ処理追加、設定変更
      - 大きな修正（アーキテクチャレビューからやり直し）
        - 例: データフロー変更、API設計変更、認証方式変更、ドメインモデル変更
    pass_previous_response: true
    rules:
      - condition: 軽微なセキュリティ修正が完了した
        next: security_review
      - condition: 大きな修正を行い、アーキテクチャレビューからやり直す必要がある
        next: architect_review
      - condition: 修正を進行できない
        next: plan

  # ===========================================
  # Phase 6: QA Review
  # ===========================================
  - name: qa_review
    edit: false
    agent: ../agents/expert/qa-reviewer.md
    report:
      name: 07-qa-review.md
      format: |
        ```markdown
        # QAレビュー

        ## 結果: APPROVE / REJECT

        ## サマリー
        {1-2文で結果を要約}

        ## 確認した観点
        | 観点 | 結果 | 備考 |
        |------|------|------|
        | テストカバレッジ | ✅ | - |
        | テスト品質 | ✅ | - |
        | エラーハンドリング | ✅ | - |
        | ドキュメント | ✅ | - |
        | 保守性 | ✅ | - |

        ## 問題点（REJECTの場合）
        | # | カテゴリ | 問題 | 修正案 |
        |---|---------|------|--------|
        | 1 | テスト | 問題の説明 | 修正方法 |
        ```
    allowed_tools:
      - Read
      - Glob
      - Grep
      - Write
      - WebSearch
      - WebFetch
    instruction_template: |
      品質保証の観点から上記の変更をレビューしてください。

      **レビュー観点:**
      - テストカバレッジと品質
      - テスト戦略（単体/統合/E2E）
      - ドキュメント（コード内・外部）
      - エラーハンドリング
      - ログとモニタリング
      - 保守性
    rules:
      - condition: 品質基準を満たしている
        next: supervise
      - condition: 品質に問題がある
        next: fix_qa

  - name: fix_qa
    edit: true
    agent: ../agents/default/coder.md
    allowed_tools:
      - Read
      - Glob
      - Grep
      - Edit
      - Write
      - Bash
      - WebSearch
      - WebFetch
    instruction_template: |
      ## QA Review Feedback (これが最新の指示です - 優先して対応してください)
      {previous_response}

      **重要**: QA専門家からの指摘を修正してください。

      指摘されたポイント:
      - テストの追加・改善
      - ドキュメントの追加・修正
      - エラーハンドリング
      - ログ出力
      - コード品質

      ## 修正完了時の判断
      修正が完了したら、**変更の影響範囲**を判断してください：

      - 軽微な修正（QAレビューのみ再実施）
        - 例: テスト追加、ドキュメント追加、ログ追加、コメント追加
      - セキュリティに影響する修正（セキュリティレビューからやり直し）
        - 例: エラーハンドリング変更（エラーメッセージの内容変更）、入力検証の変更
      - 大きな修正（アーキテクチャレビューからやり直し）
        - 例: ビジネスロジック変更、データモデル変更、API変更
    pass_previous_response: true
    rules:
      - condition: 軽微なQA修正が完了した
        next: qa_review
      - condition: セキュリティに影響する修正を行った
        next: security_review
      - condition: 大きな修正を行い、アーキテクチャレビューからやり直す必要がある
        next: architect_review
      - condition: 修正を進行できない
        next: plan

  # ===========================================
  # Phase 7: Supervision
  # ===========================================
  - name: supervise
    edit: false
    agent: ../agents/expert/supervisor.md
    report:
      - Validation: 08-supervisor-validation.md
      - Summary: summary.md
    allowed_tools:
      - Read
      - Glob
      - Grep
      - Write
      - WebSearch
      - WebFetch
    instruction_template: |
      ## Previous Reviews Summary
      このステップに到達したということは、以下のレビューがすべてAPPROVEされています：
      - AI Review: APPROVED
      - Architecture Review: APPROVED
      - Frontend Review: APPROVED
      - Security Review: APPROVED
      - QA Review: APPROVED

      テスト実行、ビルド確認、最終承認を行ってください。

      **ワークフロー全体の確認:**
      1. 計画（00-plan.md）と実装結果が一致しているか
      2. 各レビューステップの指摘が対応されているか
      3. 元のタスク目的が達成されているか

      **レポートの確認:** Report Directory内の全レポートを読み、
      未対応の改善提案がないか確認してください。

      **Validationレポートフォーマット:**
      ```markdown
      # 最終検証結果

      ## 結果: APPROVE / REJECT

      ## 検証サマリー
      | 項目 | 状態 | 確認方法 |
      |------|------|---------|
      | 要求充足 | ✅ | 要求リストと照合 |
      | テスト | ✅ | `npm test` (N passed) |
      | ビルド | ✅ | `npm run build` 成功 |
      | 動作確認 | ✅ | 主要フロー確認 |

      ## 成果物
      - 作成: {作成したファイル}
      - 変更: {変更したファイル}

      ## 未完了項目（REJECTの場合）
      | # | 項目 | 理由 |
      |---|------|------|
      | 1 | {項目} | {理由} |
      ```

      **Summaryレポートフォーマット（APPROVEの場合のみ）:**
      ```markdown
      # タスク完了サマリー

      ## タスク
      {元の要求を1-2文で}

      ## 結果
      ✅ 完了

      ## 変更内容
      | 種別 | ファイル | 概要 |
      |------|---------|------|
      | 作成 | `src/file.ts` | 概要説明 |

      ## レビュー結果
      | レビュー | 結果 |
      |---------|------|
      | AI Review | ✅ APPROVE |
      | Architecture | ✅ APPROVE |
      | Frontend | ✅ APPROVE |
      | Security | ✅ APPROVE |
      | QA | ✅ APPROVE |
      | Supervisor | ✅ APPROVE |

      ## 確認コマンド
      ```bash
      npm test
      npm run build
      ```
      ```
    rules:
      - condition: すべての検証が完了し、マージ可能な状態である
        next: COMPLETE
      - condition: 問題が検出された
        next: fix_supervisor

  - name: fix_supervisor
    edit: true
    agent: ../agents/default/coder.md
    allowed_tools:
      - Read
      - Glob
      - Grep
      - Edit
      - Write
      - Bash
      - WebSearch
      - WebFetch
    instruction_template: |
      ## Supervisor Feedback (これが最新の指示です - 優先して対応してください)
      {previous_response}

      **重要**: 監督者からの指摘を修正してください。

      監督者は全体を俯瞰した視点から問題を指摘しています。
      優先度の高い項目から順に対応してください。
    pass_previous_response: true
    rules:
      - condition: 監督者の指摘に対する修正が完了した
        next: supervise
      - condition: 修正を進行できない
        next: plan