Back to skills
extension
Category: Development & EngineeringNo API key required

reviewer-common

Define the common settings, operation modes, output formats, and scoring criteria for each reviewer agent (backend, frontend, database, security, infra, req, basic-design, detailed-design)

personAuthor: jakexiaohubgithub

レビュアー共通ガイドライン

参照元: 全レビュアーエージェントの共通パターン


1. 共通設定

1.1 Frontmatter テンプレート(実装レビュアー用)

---
description: {専門領域}を専門的にレビューする{ペルソナ}
mode: subagent
model: google/antigravity-gemini-3-pro
temperature: 0.2
tools:
  read: true
  glob: true
  grep: true
  mcp__container-use__environment_file_read: true
  mcp__container-use__environment_file_list: true
  write: false
  edit: false
  bash: false
---

1.2 Frontmatter テンプレート(設計レビュアー用)

---
description: {設計書種別}をレビューし10点満点でスコアリングする
model: google/antigravity-gemini-3-pro
mode: subagent
temperature: 0.2
tools:
  read: true
  glob: true
  grep: true
  write: false
  edit: false
  bash: false
---

2. 共通動作モード

2.1 モード切り替え

| モード | 対象 | 目的 | |-------|------|------| | Mode A: 設計レビュー | *設計書.md | 設計フェーズでの品質保証 | | Mode B: 実装レビュー | ソースコード、Git Diff | 実装コードの品質保証 |

2.2 Diff-Driven Review (Mode B)

Token最適化: 実装レビュー時は、必ず「差分(Diff)」を最優先で確認する。

手順:

  1. git diff origin/main...HEAD を読む
  2. 変更されたファイルと行を特定
  3. 文脈が必要な場合のみ read(offset, limit) で周辺を読む
  4. 禁止: 変更がないファイルの全文読み込み

3. 共通出力形式

3.1 実装レビュー出力

## {専門領域}実装レビュー結果

### スコア: X/10点

### 各項目の評価
| 項目 | スコア | 詳細 |
|------|--------|------|
| {観点1} | 0-N | ... |
| {観点2} | 0-N | ... |

### 指摘事項(修正必須)
1. [ファイル名] 行番号: 問題の説明
   - 問題: 
   - 修正案: 
   - 理由: 

### 判定
[PASS / FAIL] (9点以上で合格)

3.2 設計レビュー出力

## レビュー結果

### スコア: X/10点

### 各項目の評価
| 項目 | スコア | 詳細 |
|------|--------|------|
| {観点1} | 0-N | コメント |
| {観点2} | 0-N | コメント |

### 問題点
1. [重大度: 高/中/低] 問題の説明
   - 該当箇所: 
   - 修正提案: 

### 改善提案
- 提案1
- 提案2

### 良い点
- 良い点1

### 合格判定: [合格/不合格]

3.3 最終出力(全レビュアー共通)

SCORE: [数値]/10
PASSED: [true/false]

4. スコアリング基準

参照: 合格閾値の正式定義は @.claude/skills/workflow-phase-convention/SKILL.md §レビュースコア閾値を参照

4.1 合格閾値

| レビュアー | 合格閾値 | 備考 | |-----------|---------|------| | req-reviewer | 8点以上 | 要件定義 | | basic-design-reviewer | 9点以上 | 基本設計 | | detailed-design-reviewer | 9点以上 | 詳細設計 | | backend-reviewer | 9点以上 | 実装 | | frontend-reviewer | 9点以上 | 実装 | | database-reviewer | 9点以上 | 実装 | | security-reviewer | 9点以上 + Critical/High=0 | 実装 | | infra-reviewer | 8点以上 | 複雑性考慮 |

4.2 配点ガイドライン

| 合計点 | 領域数 | 推奨配点例 | |--------|--------|-----------| | 10点 | 4領域 | 3+3+2+2 | | 10点 | 5領域 | 2+2+2+2+2 |

4.3 スコア定義(2点満点の場合)

| スコア | 状態 | 説明 | |--------|------|------| | 2点 | 優秀 | 完璧またはそれに近い | | 1点 | 普通 | 基本要件は満たす | | 0点 | 不十分 | 重大な問題あり |

4.4 重大度定義

| 重大度 | 説明 | |--------|------| | 高 | 根幹に関わる問題。修正必須 | | 中 | 品質に影響。修正推奨 | | 低 | 軽微。修正でより良くなる |


5. 共通ルール

5.1 全レビュアー共通

  1. 建設的: 問題だけでなく修正案を必ず提示
  2. 具体的: ファイル名、行番号、コード例を含める
  3. 優先度明示: 修正必須と任意提案を区別
  4. 一貫性: プロジェクトの既存パターンを尊重

5.2 実装レビュアー追加ルール

  • Diff-Driven Review を使用
  • 変更がないファイルの全文読み込み禁止
  • 常に具体的なファイルパスと行番号を提示

5.3 良い指摘例

1. [src/auth/service.ts] 45行目: 認証トークンの検証漏れ
   - 問題: JWT有効期限チェックが未実装
   - 修正案: `jwt.verify()``maxAge` を追加
   - 理由: 期限切れトークンでのアクセス許可リスク

5.4 悪い指摘例

1. 認証の実装が不十分  ← 曖昧、修正箇所不明