mcpField
G
goodvibes-codebase-review

goodvibes-codebase-review

github.com/mgd34msuテスト・品質637 回インストール9 週間前に更新
リポジトリ

goodvibes-codebase-review について

Goodvibes エージェント向けのコードベースレビューと修正のスキルです。分析、報告、計画、並行修正を手順化し、ツール利用と編集範囲も定めます。

主な機能

  1. 01コード品質とセキュリティの体系的レビュー
  2. 02アーキテクチャと依存関係の分析
  3. 03レビュー・修正レポートの作成
  4. 04エージェントによる並行修正と確認
  5. 05追跡可能な編集範囲の制限

活用シーン

  1. 01リポジトリのコードとセキュリティ問題の監査
  2. 02優先度付き修正計画の作成
  3. 03並行するコード修正の調整と検証

MCP 対応

コード分析、問題検出、編集検証に Goodvibes MCP ツールを使用するよう指示します。

インストール

コマンドをコピーしてターミナルで実行します。

$npx skills add https://github.com/mgd34msu/goodvibes-plugin --skill goodvibes-codebase-review

インストール後、Claude Code で/goodvibes-codebase-reviewを入力して読み込まれたか確認します。

同梱ファイル

SKILL.md と一緒にインストールされる参考資料やスクリプト。

references/
review-checklist.md
tool-priority.md
scripts/
orchestrator.py

詳細

Goodvibes コードベースのレビューと修正

goodvibes エージェントを並列で動作させ、体系的にコードベースを分析して修正します。

重要:客観的かつ完全に正直であること。決して、絶対に、取り繕わないでください。

  • 正直なレビュー:正直なレビューは、より良い結果につながります。評価の際に、感情や気持ちは一切考慮しないでください。

ファイルシステムの境界

重要:書き込みはローカル、読み取りはグローバルです。

  • 書き込み/編集/作成:現在の作業ディレクトリとそのサブディレクトリ内でのみ行ってください。ここがプロジェクトルートです。すべての変更を Git で追跡可能にする必要があります。
  • 読み取り:コンテキストを把握するため、どこにあるファイルでも読み取れます(node_modules、グローバル設定、参考用の他のプロジェクトなど)。
  • 次の場所には決して書き込まないでください:親ディレクトリ、ホームディレクトリ、システムファイル、他のプロジェクト、プロジェクトルート外のすべての場所。

起動されたときの作業ディレクトリがプロジェクトルートです。変更はすべてその範囲内で行ってください。

MCP ツールのチェックリスト(必須)

停止してください。何かを行う前に、必ずこのチェックリストを完了してください。

重要な前提条件: mcp-cli call <server>/<tool> を実行する前に、必ず mcp-cli info <server>/<tool> を呼び出す必要があります。これは必須要件です。

タスク開始時
# ALWAYS check schema first
mcp-cli info plugin_goodvibes_analysis-engine/detect_stack
mcp-cli info plugin_goodvibes_registry-engine/recommend_skills
mcp-cli info plugin_goodvibes_project-engine/project_issues

# Then make the calls
mcp-cli call plugin_goodvibes_analysis-engine/detect_stack '{}'
mcp-cli call plugin_goodvibes_registry-engine/recommend_skills '{"task":"codebase review quality audit"}'
mcp-cli call plugin_goodvibes_project-engine/project_issues '{}'
レビューフェーズ
# Check schemas first
mcp-cli info plugin_goodvibes_analysis-engine/find_circular_deps
mcp-cli info plugin_goodvibes_analysis-engine/scan_for_secrets
mcp-cli info plugin_goodvibes_project-engine/analyze_dependencies
mcp-cli info plugin_goodvibes_analysis-engine/find_dead_code
mcp-cli info plugin_goodvibes_project-engine/get_test_coverage

# Then execute
mcp-cli call plugin_goodvibes_analysis-engine/find_circular_deps '{}'
mcp-cli call plugin_goodvibes_analysis-engine/scan_for_secrets '{}'
mcp-cli call plugin_goodvibes_project-engine/analyze_dependencies '{}'
mcp-cli call plugin_goodvibes_analysis-engine/find_dead_code '{}'
mcp-cli call plugin_goodvibes_project-engine/get_test_coverage '{}'
編集の前に毎回
# Check schemas
mcp-cli info plugin_goodvibes_analysis-engine/scan_patterns
mcp-cli info plugin_goodvibes_project-engine/find_tests_for_file
mcp-cli info plugin_goodvibes_analysis-engine/validate_edits_preview

# Execute
mcp-cli call plugin_goodvibes_analysis-engine/scan_patterns '{}'
mcp-cli call plugin_goodvibes_project-engine/find_tests_for_file '{"file":"path/to/file.ts"}'
mcp-cli call plugin_goodvibes_analysis-engine/validate_edits_preview '{}'
編集の後に毎回
# Check schema
mcp-cli info plugin_goodvibes_project-engine/project_issues

# Execute
mcp-cli call plugin_goodvibes_project-engine/project_issues '{}'

絶対厳守:goodvibes MCP ツールで実行できることは、必ずそのツールを使ってください。例外はありません。


高精度ツール(必須)

重要:システムツールではなく、高精度ツールを使用してください。

代わりに使うもの使用するもの理由
Readprecision_read抽出モード、行範囲、アウトライン/シンボル
Grepprecision_grep出力モード、バッチクエリ、コンテキスト制御
Globprecision_glob出力モード、フィルター、プレビュー
Editprecision_editアトミックなトランザクション、検証、ヒント
Writeprecision_writeアトミック処理、テンプレート、検証
Bashprecision_execバッチコマンド、期待値、出力制御
高精度ツールの使用パターン
# Find files with pattern (minimal output)
precision_grep:
  queries:
    - pattern: "TODO|FIXME|HACK"
      glob: "src/**/*.ts"
  output:
    mode: files_only

# Read file structure without content
precision_read:
  files: ["src/index.ts", "src/app.ts"]
  extract: outline
  output:
    mode: minimal

# Batch edit multiple files atomically
precision_edit:
  edits:
    - file: "src/api.ts"
      find: "const API_URL = 'http://localhost'"
      replace: "const API_URL = process.env.API_URL"
    - file: "src/config.ts"
      find: "debug: true"
      replace: "debug: process.env.NODE_ENV !== 'production'"
  transaction:
    mode: atomic
    rollback_on_fail: true
  output:
    mode: minimal

# Execute commands with expectations
precision_exec:
  commands:
    - cmd: "npm run typecheck"
      expect:
        exit_code: 0
    - cmd: "npm run lint"
      expect:
        exit_code: 0
  output:
    mode: minimal

探索 -> バッチのワークフロー

重要:バッチ処理の前に、必ず探索してください。

discover ツールは複数のクエリを並列で実行し、バッチを組み立てる前にコンテキストを収集します。これにより、無駄な操作を防ぎ、対象ファイルを正確に絞り込めます。

Discovery ツールの使用方法
# Run parallel discovery queries
discover:
  queries:
    - id: find_components
      type: glob
      patterns: ["src/components/**/*.tsx"]
    - id: find_api_routes
      type: glob
      patterns: ["src/api/**/*.ts", "src/app/api/**/*.ts"]
    - id: find_issues
      type: grep
      pattern: "TODO|FIXME|HACK"
      glob: "src/**/*.{ts,tsx}"
    - id: find_hooks
      type: symbols
      query: "use"
      kinds: ["function"]
  output_mode: files_only  # count_only | files_only | locations
ワークフローのパターン
  1. 探索 - クエリを実行して対象範囲を把握します

    • まず count_only を使って規模を把握します
    • 次に files_only を使って対象ファイルの一覧を取得します
  2. 計画 - 探索結果を使ってバッチ操作を組み立てます

    • バッチ操作では、探索で見つかったファイルを指定します
    • 見つかった対象に作業範囲を正確に絞ります
  3. 実行 - 十分なコンテキストを使ってバッチを実行します

例:機能の実装
# Step 1: Discover current state
discover:
  queries:
    - id: existing_files
      type: glob
      patterns: ["src/features/auth/**/*.ts"]
    - id: existing_patterns
      type: grep
      pattern: "export (function|const|class)"
      glob: "src/features/**/*.ts"
  output_mode: files_only

# Step 2: Use results to build targeted batch
batch:
  id: implement-feature
  operations:
    read:
      - id: analyze
        type: files
        targets: "{{existing_files.files}}"  # From discovery
        extract: outline

メリット:

  • 誤ったファイルに対する見当違いの操作を防ぎます
  • コードベース全体で一貫したパターンを確保します
  • 必要な対象だけに絞ることで、トークン使用量を削減します
  • 実装方針について、十分な情報に基づいた判断が可能になります

モードに応じた動作

現在のモードに応じて動作が変わります。

vibecoding モード
  • コミュニケーション:進捗を示し、判断の理由を説明し、結果を詳しく報告します
  • 確認:曖昧さやリスクがある場合は、先にユーザーに確認します
  • チェックポイント:バッチごとにチェックポイントを作成します
  • 出力:標準的な詳しさで、差分を表示します
justvibes モード
  • 静音:必要最小限のコミュニケーションとし、.goodvibes/logs/activity.md に記録します
  • 自律動作:最善と考えられる判断を行い、リスクがある場合はチェックポイントを作成して続行します
  • 自動連鎖:次の論理的なバッチへ自動的に進みます
  • 出力:必要最小限の詳しさで、差分は表示しません

ワークフローの概要

┌─────────────────────────────────────────────────────────────┐
│                      CODEBASE REVIEW                        │
├─────────────────────────────────────────────────────────────┤
│  Phase 1: Review      │  Run MCP tools, analyze all code   │
│  Phase 2: Report      │  Generate codebase-review-report.md │
│  Phase 3: Plan        │  Create remediation-plan.md        │
│  Phase 4: Execute     │  WORK-REVIEW-FIX-CHECK (max 6)     │
└─────────────────────────────────────────────────────────────┘

フェーズ 1:コードベースのレビュー

MCP ツールの対応付け
カテゴリ主なツール代替手段
品質find_dead_code、scan_patternsパターンを grep で検索
アーキテクチャfind_circular_deps、get_api_surface手動分析
セキュリティscan_for_secrets、check_permissionsパターンを grep で検索
パフォーマンスget_prisma_operations、analyze_bundleなし
ドキュメントexplain_codebaseファイルをスキャン
テストget_test_coverage、find_tests_for_file、suggest_test_casesjest --coverage
設定read_config、env_auditenv ファイルをスキャン
依存関係analyze_dependenciesnpm audit
エラーparse_error_stack、explain_type_errortsc の出力
スタイルscan_patterns、get_conventionseslint の出力
レビューチェックリスト

カテゴリごとの最低要件:

  • 具体的なファイルと行番号を示す指摘が5件、または「問題は見つかりませんでした」と明示すること
  • 定量的な測定値(件数、割合、LOC)
  • 重大度の分類(critical/high/medium/low)

フェーズ 2:マスターレポート

codebase-review-report.md を生成します:

# Codebase Review Report

**Project**: {name from detect_stack}
**Stack**: {technologies detected}
**Generated**: {ISO 8601 timestamp}
**Overall Score**: {X.X}/10

## Executive Summary

| Severity | Count | Description |
|----------|-------|-------------|
| 🔴 Critical | N | Issues requiring immediate attention |
| 🟠 High | N | Issues blocking production readiness |
| 🟡 Medium | N | Issues impacting maintainability |
| 🔵 Low | N | Minor improvements |

## Score Breakdown

| Category | Weight | Raw | Deductions | Score | Grade |
|----------|--------|-----|------------|-------|-------|
| Quality | 15% | 10 | -X.X | X.X | A-F |
| Architecture | 15% | 10 | -X.X | X.X | A-F |
| Security | 20% | 10 | -X.X | X.X | A-F |
| Performance | 10% | 10 | -X.X | X.X | A-F |
| Documentation | 5% | 10 | -X.X | X.X | A-F |
| Testing | 15% | 10 | -X.X | X.X | A-F |
| Config | 5% | 10 | -X.X | X.X | A-F |
| Dependencies | 5% | 10 | -X.X | X.X | A-F |
| Errors | 5% | 10 | -X.X | X.X | A-F |
| Style | 5% | 10 | -X.X | X.X | A-F |
| **TOTAL** | 100% | - | - | **X.X** | **X** |

## Score Calculation

{Show the math for each category deduction}

## Detailed Findings

### {Category Name}

#### Finding: {Title}

| Field | Value |
|-------|-------|
| **Severity** | critical\|high\|medium\|low |
| **Location** | `file:line` or `file:startLine-endLine` |
| **Measurement** | {exact number or percentage} |
| **Threshold** | {acceptable value} |
| **Impact** | {business/technical consequence} |
| **Points Deducted** | {-X.X from Category} |

**Evidence:**

{code snippet or tool output}


**Required Fix:**

{specific remediation code or steps}


---

{repeat for all findings}

## What's Working Well

{List genuinely good patterns found, with file references}

## Improvement Roadmap

| Phase | Focus | Expected Impact | New Score |
|-------|-------|-----------------|-----------|
| 1 | Critical fixes | +X.X points | X.X |
| 2 | High priority | +X.X points | X.X |
| 3 | Medium priority | +X.X points | X.X |
| 4 | Polish | +X.X points | X.X |

フェーズ 3:修正計画

remediation-plan.md を生成します:

# Remediation Plan

**Generated**: {timestamp}
**Total Tasks**: {N}
**Execution Strategy**: WORK-REVIEW-FIX-CHECK with parallel goodvibes agents (max 6 concurrent)

## Execution Rules

| Rule | Value |
|------|-------|
| Max concurrent agents | 6 |
| Agent type | goodvibes background |
| Context model | Fresh per task (no accumulation) |
| Tool priority | MCP tools > bash |
| Monitoring | Self-report via SubagentStop hook |
| Workflow | WORK → REVIEW → PASS/FAIL → (FIX → CHECK) |

## Task Definitions

### Wave 1: Critical [P0] - Execute Immediately

#### TASK-001: {Description}

| Field | Value |
|-------|-------|
| **Severity** | critical |
| **Target Files** | `file1.ts`, `file2.ts` |
| **Related Finding** | {reference to report finding} |
| **MCP Tools** | `tool1`, `tool2` |
| **Estimated Duration** | Xm |

**Context from Report:**

{relevant finding details}


---

{repeat for all tasks, grouped by wave}

### Wave 2: High [P1]

### Wave 3: Medium [P2]

### Wave 4: Low [P3]

## Dependency Graph

{Show task dependencies if any exist}

フェーズ 4:エージェントの並列実行

WORK-REVIEW-FIX-CHECK ワークフロー
For each remediation task:

  WORK Agent ─────────────────────> REVIEW Agent
  (goodvibes:engineer)              (goodvibes:reviewer)
       │                                  │
       │                                  ├─> PASS ─> Commit ─> Update Log ─> Next Task
       │                                  │
       │                                  └─> FAIL ─> FIX Agent ─> CHECK Agent
                                                         │              │
                                                         │              ├─> PASS ─> Commit
                                                         │              │
                                                         └──────────────┴─> FAIL (loop)
エージェントの役割の対応付け
フェーズエージェント目的
WORKgoodvibes:engineer修正タスクを実装します
REVIEWgoodvibes:reviewer作業品質を検証します(100%必須)
FIXgoodvibes:engineerレビューでのすべての指摘に対応します
CHECKgoodvibes:reviewer修正を再検証します
エージェントの完了待ち(重要)

エージェントの状態確認に以下を使用しないでください:

  • トランスクリプトファイルに対する tail コマンド
  • TaskOutput ツール
  • あらゆる形式のポーリング

代わりに、オーケストレーターは次の手順を実行します:

  1. run_in_background: true を指定してエージェントを起動します
  2. 操作を停止します(ターンが終了します)
  3. エージェントの完了時に SubagentStop フック通知を受け取ります
  4. フックメッセージがステータスとともにコンテキストに表示されます
同時実行ルール
ルール値
同時実行できるエージェントの最大数6
タスクあたりのエージェント数1(同時に1つのエージェントだけがタスクに取り組みます)
完了要件100%(99.9%ではありません)

WORK エージェントのプロンプトテンプレート
# Remediation Task: {TASK_ID}

## MCP Tool Checklist (MANDATORY)

**CRITICAL PREREQUISITE:** You MUST call `mcp-cli info <server>/<tool>` BEFORE ANY `mcp-cli call <server>/<tool>`.

Before ANY edit:
```bash
# Check schemas first
mcp-cli info plugin_goodvibes_analysis-engine/scan_patterns
mcp-cli info plugin_goodvibes_project-engine/find_tests_for_file

# Then execute
mcp-cli call plugin_goodvibes_analysis-engine/scan_patterns '{}'
mcp-cli call plugin_goodvibes_project-engine/find_tests_for_file '{"file":"{TARGET_FILE}"}'

編集のたびに必ず次を実行します:

# Check schema first
mcp-cli info plugin_goodvibes_project-engine/project_issues

# Then execute
mcp-cli call plugin_goodvibes_project-engine/project_issues '{}'

割り当て

フィールド値
タスク ID{TASK_ID}
重大度{SEVERITY}
説明{DESCRIPTION}
ファイル{FILE_LIST}

レポートからのコンテキスト

{FINDING_DETAILS}

指示

  1. 割り当てられたタスクのみを完了します
  2. bash の前に goodvibes MCP ツールを使用します
  3. すべての操作に高精度ツール(precision_edit、precision_exec)を使用します
  4. 編集のたびに検証ツールを実行します
  5. 編集によって新しいエラーが発生した場合は、完了前に修正します

ツールの優先順位(必須)

  1. 最初に mcp-cli info を確認してから mcp-cli call plugin_goodvibes_* を実行します
  2. 次に、高精度ツール(precision_edit、precision_read など)を使用します
  3. MCP/高精度ツールが存在しない場合に限り、その後で標準ツールに切り替えます

完了レポート

完了したら、次を出力します:

{
  "task_id": "{TASK_ID}",
  "status": "success|failed",
  "files_modified": ["file1.ts", "file2.ts"],
  "tests_passed": true|false,
  "type_check_passed": true|false,
  "notes": "Any relevant context"
}

---

### REVIEW Agent Prompt Template

```markdown
# Review Task: {TASK_ID}

## MCP Tool Checklist (MANDATORY)

**CRITICAL PREREQUISITE:** You MUST call `mcp-cli info <server>/<tool>` BEFORE ANY `mcp-cli call <server>/<tool>`.

```bash
# Check schemas first
mcp-cli info plugin_goodvibes_project-engine/project_issues
mcp-cli info plugin_goodvibes_analysis-engine/scan_patterns
mcp-cli info plugin_goodvibes_project-engine/get_test_coverage

# Then execute
mcp-cli call plugin_goodvibes_project-engine/project_issues '{}'
mcp-cli call plugin_goodvibes_analysis-engine/scan_patterns '{}'
mcp-cli call plugin_goodvibes_project-engine/get_test_coverage '{}'

割り当て

フィールド値
タスク ID{TASK_ID}
WORK エージェント{WORK_AGENT_ID}
変更したファイル{FILE_LIST}

WORK エージェントからのコンテキスト

{WORK_COMPLETION_REPORT}

レビュー基準

基準配点説明
正確性30%問題を正しく解決していますか?
完全性25%すべての側面に対応していますか?
品質20%コードはクリーンで保守しやすいですか?
安全性15%新たなエラーやセキュリティ上の問題はありませんか?
テスト10%テストに合格し、カバレッジが維持されていますか?

レビュープロセス

  1. 変更したファイルをすべて読みます
  2. MCP 検証ツールを実行します
  3. 以下を確認します:
    • 型エラー
    • テストの失敗
    • セキュリティ上の問題
    • コード品質の問題
    • 不完全な修正
  4. 基準の配点に基づいてスコアを算出します
  5. 判定を提示します

レビュー結果

{
  "task_id": "{TASK_ID}",
  "verdict": "PASS|FAIL",
  "score": 8.5,
  "issues": [
    {
      "severity": "critical|major|minor",
      "file": "path/to/file.ts",
      "line": 47,
      "issue": "Description of the issue",
      "fix_guidance": "Specific guidance on how to fix"
    }
  ],
  "recommendation": "Details for FIX agent if FAIL"
}

判定ルール

  • PASS:スコアが 8.0 以上で、重大な問題がなく、すべての基準に合格
  • FAIL:スコアが 8.0 未満、または重大な問題がある、またはいずれかの基準に不合格

重大:正直に評価してください。本番環境に出す前に問題を見つけることが重要です。


---

### FIX Agent Prompt Template

```markdown
# Fix Task: {TASK_ID} (Iteration {N})

## MCP Tool Checklist (MANDATORY)

**CRITICAL PREREQUISITE:** You MUST call `mcp-cli info <server>/<tool>` BEFORE ANY `mcp-cli call <server>/<tool>`.

Before ANY edit:
```bash
# Check schemas first
mcp-cli info plugin_goodvibes_analysis-engine/scan_patterns
mcp-cli info plugin_goodvibes_project-engine/find_tests_for_file

# Then execute
mcp-cli call plugin_goodvibes_analysis-engine/scan_patterns '{}'
mcp-cli call plugin_goodvibes_project-engine/find_tests_for_file '{"file":"{TARGET_FILE}"}'

編集するたびに必ず実行してください:

# Check schema first
mcp-cli info plugin_goodvibes_project-engine/project_issues

# Then execute
mcp-cli call plugin_goodvibes_project-engine/project_issues '{}'

レビューフィードバック

{ISSUES_FROM_REVIEW}

指示

  1. レビューで指摘されたすべての問題に対応してください。省略しないでください
  2. 各問題について提示された fix_guidance に従ってください
  3. すべての操作で精密ツールを使用してください
  4. 修正のたびに検証を実行してください
  5. 問題を修正できない場合は、その理由を記録し、ブロック中としてマークしてください

完了レポート

{
  "task_id": "{TASK_ID}",
  "iteration": {N},
  "status": "success|blocked",
  "issues_addressed": N,
  "issues_remaining": N,
  "blocked_issues": [
    {
      "issue": "Description",
      "reason": "Why it cannot be fixed"
    }
  ],
  "notes": "Any relevant context"
}

---

### CHECK Agent Prompt Template

```markdown
# Check Task: {TASK_ID} (Verification Round {N})

## MCP Tool Checklist (MANDATORY)

**CRITICAL PREREQUISITE:** You MUST call `mcp-cli info <server>/<tool>` BEFORE ANY `mcp-cli call <server>/<tool>`.

```bash
# Check schemas first
mcp-cli info plugin_goodvibes_project-engine/project_issues
mcp-cli info plugin_goodvibes_analysis-engine/scan_patterns

# Then execute
mcp-cli call plugin_goodvibes_project-engine/project_issues '{}'
mcp-cli call plugin_goodvibes_analysis-engine/scan_patterns '{}'

割り当て

フィールド値
Task ID{TASK_ID}
FIX エージェント{FIX_AGENT_ID}
イテレーション{N}

前回の問題

{ISSUES_TO_VERIFY}

FIX エージェントからのコンテキスト

{FIX_COMPLETION_REPORT}

検証プロセス

  1. 変更されたすべてのファイルを読む
  2. MCP 検証ツールを実行する
  3. 前回の問題がそれぞれ解決されたことを確認する
  4. 修正によって新たな問題が発生していないか確認する
  5. 判定を提示する

判定結果

{
  "task_id": "{TASK_ID}",
  "iteration": {N},
  "verdict": "PASS|FAIL",
  "resolved_issues": N,
  "unresolved_issues": [
    {
      "issue": "Description",
      "status": "still_present|new_issue"
    }
  ],
  "recommendation": "Next action if FAIL"
}

判定ルール

  • PASS:前回の問題がすべて解決され、新たな重大な問題がない
  • FAIL:未解決の問題が1つでもある、または新たな重大な問題が見つかった

3 回のイテレーション後も FAIL の場合は、人によるレビューにエスカレーションしてください。


---

### Commit Protocol

After REVIEW or CHECK verdict is PASS, create commit:

fix({CATEGORY}): {TASK_ID} - {SHORT_DESCRIPTION}

{DETAILED_DESCRIPTION}

変更内容

  • {FILE_1}: {WHAT_CHANGED}
  • {FILE_2}: {WHAT_CHANGED}

レビュー

  • レビュー担当:REVIEW エージェント
  • スコア:{SCORE}/10
  • イテレーション数:{FIX_ITERATIONS}

Task-ID: {TASK_ID} Severity: {SEVERITY} Co-Authored-By: Claude Opus 4.5 [email protected]


### Completion Cleanup

After ALL remediation tasks have passed review and been committed, archive the output files:

**Archive Location:** `.goodvibes/completed/`

**Archive Commands:**
```bash
# Run all commands together in a single shell session:
# Create archive directory if it doesn't exist
mkdir -p .goodvibes/completed

# Generate timestamp
TIMESTAMP=$(date +%Y%m%d-%H%M%S)

# Move and rename files
mv codebase-review-report.md .goodvibes/completed/code-review-${TIMESTAMP}.md
mv remediation-plan.md .goodvibes/completed/remediation-plan-${TIMESTAMP}.md
mv remediation-log.md .goodvibes/completed/remediation-log-${TIMESTAMP}.md

アーカイブのチェックリスト:

  • すべての修正タスクのステータスが remediation-log.md で completed になっている
  • すべてのコミットがプッシュされている(該当する場合)
  • メモリファイルが更新されている(.goodvibes/memory/)
  • アーカイブディレクトリが作成されている
  • タイムスタンプのサフィックスを付けてファイルが移動されている

ファイル命名規則:

元のファイルアーカイブ後
codebase-review-report.mdcode-review-{YYYYMMDD-HHMMSS}.md
remediation-plan.mdremediation-plan-{YYYYMMDD-HHMMSS}.md
remediation-log.mdremediation-log-{YYYYMMDD-HHMMSS}.md

アーカイブする理由:

  • プロジェクトのルートディレクトリを整理できます
  • 過去のレビュー情報を保持できます
  • 時間の経過に伴うコードベースの健全性を追跡できます
  • 今後のレビューで混乱が生じるのを防げます
完了ログの記録

remediation-log.md を維持してください:

# Remediation Log

## Active Agents

| Task ID | Description | Agent | Phase | Started | Elapsed |
|---------|-------------|-------|-------|---------|---------|
| TASK-002 | Remove secrets | agent-abc | WORK | 10:05:00 | 5m |
| TASK-003 | Fix auth flow | agent-def | REVIEW | 10:08:00 | 2m |

## Completed Tasks

| Task ID | Description | Status | Duration | Iterations | Changes |
|---------|-------------|--------|----------|------------|---------|
| TASK-001 | Fix SQL injection | ✅ PASS | 15m32s | 1 | `api/users.ts` |

## Failed Tasks

| Task ID | Description | Status | Iterations | Reason | Retry? |
|---------|-------------|--------|------------|--------|--------|
| TASK-005 | Fix type errors | ❌ FAIL | 3 | Blocked by external dependency | No |

## Summary

- **Total Tasks**: N
- **Completed**: N (X%)
- **In Progress**: N
- **Failed**: N
- **Remaining**: N
- **Active Agents**: N/6

出力成果物

ファイル用途
codebase-review-report.md定量化された指標とスコアを含む、完全な調査結果
remediation-plan.mdコンテキストを含む、重大度別に優先順位付けされたタスクリスト
remediation-log.mdWORK-REVIEW-FIX-CHECK のステータスをリアルタイムで追跡

注: すべての修正タスクが完了したら、これらのファイルはタイムスタンプのサフィックスを付けて .goodvibes/completed/ にアーカイブされます。詳しくは「完了後のクリーンアップ」セクションを参照してください。


メモリ統合

レビュー/修正後、.goodvibes/memory/ を更新してください:

ファイル更新内容
decisions.md修正中に行われたアーキテクチャ上の決定
patterns.md発見または適用されたパターン
failures.md根本原因の分析を含む、失敗した修正
preferences.md効果的だったツールの設定

制約

制約値理由
同時実行エージェントの最大数6リソース管理
エージェントの種類goodvibes backgroundフック経由のテレメトリ
エージェントあたりのタスク数1タスクごとに新しいコンテキストを使用
コンテキストの継承なし相互汚染を防止
監視方法自己報告SubagentStop フックが処理
ツールの優先順位MCP > precision > bash一貫性とテレメトリ
レビュー要件100%コミット前に品質を確保

コメント