goodvibes-codebase-review について
Goodvibes エージェント向けのコードベースレビューと修正のスキルです。分析、報告、計画、並行修正を手順化し、ツール利用と編集範囲も定めます。
主な機能
- 01コード品質とセキュリティの体系的レビュー
- 02アーキテクチャと依存関係の分析
- 03レビュー・修正レポートの作成
- 04エージェントによる並行修正と確認
- 05追跡可能な編集範囲の制限
活用シーン
- 01リポジトリのコードとセキュリティ問題の監査
- 02優先度付き修正計画の作成
- 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.mdtool-priority.mdscripts/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 ツールで実行できることは、必ずそのツールを使ってください。例外はありません。
高精度ツール(必須)
重要:システムツールではなく、高精度ツールを使用してください。
| 代わりに使うもの | 使用するもの | 理由 |
|---|---|---|
Read | precision_read | 抽出モード、行範囲、アウトライン/シンボル |
Grep | precision_grep | 出力モード、バッチクエリ、コンテキスト制御 |
Glob | precision_glob | 出力モード、フィルター、プレビュー |
Edit | precision_edit | アトミックなトランザクション、検証、ヒント |
Write | precision_write | アトミック処理、テンプレート、検証 |
Bash | precision_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
ワークフローのパターン
-
探索 - クエリを実行して対象範囲を把握します
- まず
count_onlyを使って規模を把握します - 次に
files_onlyを使って対象ファイルの一覧を取得します
- まず
-
計画 - 探索結果を使ってバッチ操作を組み立てます
- バッチ操作では、探索で見つかったファイルを指定します
- 見つかった対象に作業範囲を正確に絞ります
-
実行 - 十分なコンテキストを使ってバッチを実行します
例:機能の実装
# 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_cases | jest --coverage |
| 設定 | read_config、env_audit | env ファイルをスキャン |
| 依存関係 | analyze_dependencies | npm audit |
| エラー | parse_error_stack、explain_type_error | tsc の出力 |
| スタイル | scan_patterns、get_conventions | eslint の出力 |
レビューチェックリスト
カテゴリごとの最低要件:
- 具体的なファイルと行番号を示す指摘が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)
エージェントの役割の対応付け
| フェーズ | エージェント | 目的 |
|---|---|---|
| WORK | goodvibes:engineer | 修正タスクを実装します |
| REVIEW | goodvibes:reviewer | 作業品質を検証します(100%必須) |
| FIX | goodvibes:engineer | レビューでのすべての指摘に対応します |
| CHECK | goodvibes:reviewer | 修正を再検証します |
エージェントの完了待ち(重要)
エージェントの状態確認に以下を使用しないでください:
- トランスクリプトファイルに対する
tailコマンド TaskOutputツール- あらゆる形式のポーリング
代わりに、オーケストレーターは次の手順を実行します:
run_in_background: trueを指定してエージェントを起動します- 操作を停止します(ターンが終了します)
- エージェントの完了時に SubagentStop フック通知を受け取ります
- フックメッセージがステータスとともにコンテキストに表示されます
同時実行ルール
| ルール | 値 |
|---|---|
| 同時実行できるエージェントの最大数 | 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}
指示
- 割り当てられたタスクのみを完了します
- bash の前に goodvibes MCP ツールを使用します
- すべての操作に高精度ツール(precision_edit、precision_exec)を使用します
- 編集のたびに検証ツールを実行します
- 編集によって新しいエラーが発生した場合は、完了前に修正します
ツールの優先順位(必須)
- 最初に
mcp-cli infoを確認してからmcp-cli call plugin_goodvibes_*を実行します - 次に、高精度ツール(precision_edit、precision_read など)を使用します
- 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% | テストに合格し、カバレッジが維持されていますか? |
レビュープロセス
- 変更したファイルをすべて読みます
- MCP 検証ツールを実行します
- 以下を確認します:
- 型エラー
- テストの失敗
- セキュリティ上の問題
- コード品質の問題
- 不完全な修正
- 基準の配点に基づいてスコアを算出します
- 判定を提示します
レビュー結果
{
"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}
指示
- レビューで指摘されたすべての問題に対応してください。省略しないでください
- 各問題について提示された
fix_guidanceに従ってください - すべての操作で精密ツールを使用してください
- 修正のたびに検証を実行してください
- 問題を修正できない場合は、その理由を記録し、ブロック中としてマークしてください
完了レポート
{
"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}
検証プロセス
- 変更されたすべてのファイルを読む
- MCP 検証ツールを実行する
- 前回の問題がそれぞれ解決されたことを確認する
- 修正によって新たな問題が発生していないか確認する
- 判定を提示する
判定結果
{
"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.md | code-review-{YYYYMMDD-HHMMSS}.md |
remediation-plan.md | remediation-plan-{YYYYMMDD-HHMMSS}.md |
remediation-log.md | remediation-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.md | WORK-REVIEW-FIX-CHECK のステータスをリアルタイムで追跡 |
注: すべての修正タスクが完了したら、これらのファイルはタイムスタンプのサフィックスを付けて .goodvibes/completed/ にアーカイブされます。詳しくは「完了後のクリーンアップ」セクションを参照してください。
メモリ統合
レビュー/修正後、.goodvibes/memory/ を更新してください:
| ファイル | 更新内容 |
|---|---|
decisions.md | 修正中に行われたアーキテクチャ上の決定 |
patterns.md | 発見または適用されたパターン |
failures.md | 根本原因の分析を含む、失敗した修正 |
preferences.md | 効果的だったツールの設定 |
制約
| 制約 | 値 | 理由 |
|---|---|---|
| 同時実行エージェントの最大数 | 6 | リソース管理 |
| エージェントの種類 | goodvibes background | フック経由のテレメトリ |
| エージェントあたりのタスク数 | 1 | タスクごとに新しいコンテキストを使用 |
| コンテキストの継承 | なし | 相互汚染を防止 |
| 監視方法 | 自己報告 | SubagentStop フックが処理 |
| ツールの優先順位 | MCP > precision > bash | 一貫性とテレメトリ |
| レビュー要件 | 100% | コミット前に品質を確保 |