M3E: Skill for Claude Code

pr-review — PR レビュー・マージ・事後処理

PR作成後のマネージャー側フロー全体を扱う。

PR受領 → 差分レビュー → 検証 → 判定 → マージ → 事後処理

---

Step 1: PR の読み込み

PR番号 or URLが指定された場合:

gh pr view <number> --json title,body,headRefName,baseRefName,files,commits,state
gh pr diff <number>

PR番号が不明な場合:

# open なPR一覧（base=dev-beta）
gh pr list --base dev-beta --state open

読み取る情報:

• タイトル・概要
• ソースブランチ → ベースブランチ
• コミット一覧
• 変更ファイル一覧と差分
• 既存のレビューコメント

---

Step 2: レビュー

3つの観点で差分を評価する。devM3E の reviewer agent と同じ基準だが、PR文脈に特化。

2a. Spec整合チェック

変更が関連specに準拠しているか確認する。

1. 変更ファイルから関連specを特定（references/spec_index.md 参照）
2. データモデル不変条件の維持
3. Commandパターン（undo/redo）の維持
4. scope/alias ルールへの適合

specを読む必要がある場合は docs/03_Spec/ から該当文書を読む。

2b. コード品質チェック

• 型安全性（不必要な any、型アサーション）
• エラーハンドリング（fail-closed か）
• パフォーマンス懸念（O(n²)ループ、不要な再レンダリング）
• テストカバレッジ（新規コードにテストがあるか）

2c. 運用ルール準拠

• コミットメッセージが imperative 形式か
• daily note（docs/daily/YYMMDD.md）が更新されているか
• Decision Pool に記録すべき設計判断が埋もれていないか
• ブランチ名がロールに合致しているか（visual→dev-visual 等）

---

Step 3: 検証（オプション）

差分の性質に応じて検証を実行する。

変更の性質	実行する検証
TypeScript コード	npx tsc --noEmit
テスト関連	npx vitest run
ビルド影響あり	npm run build
UI変更	Playwright（該当テストがあれば）
ドキュメントのみ	スキップ

検証を実行する場合は、PR のブランチをチェックアウトして行う:

gh pr checkout <number>
cd beta
npx tsc --noEmit && npx vitest run && npm run build

---

Step 4: 判定

レビュー結果を以下の形式でユーザーに報告する。

## PR #{number}: {title}

### 判定: ✅ approve / ⚠️ comment / ❌ request-changes

### 変更サマリー
{ファイル数}ファイル, +{追加行} -{削除行}
影響範囲: {beta/ | docs/ | scripts/ 等}

### レビュー結果
- [pass|fail] Spec整合: {detail}
- [pass|fail] コード品質: {detail}
- [pass|fail] 運用ルール: {detail}

### 検証結果
- tsc: pass/fail
- test: pass/fail ({n}/{total})
- build: pass/fail

### issue（あれば）
- [{severity}] {description}

### 推奨アクション
{マージしてよい / 修正が必要 / 要議論}

判定基準

判定	条件
approve	issue なし or nit のみ。検証パス
comment	minor issue あるがマージをブロックしない
request-changes	major/critical issue あり。修正が必要