/cc-update-review

name: cc-update-review
description: "Quality guardrail for Claude/Codex update integration. Detects doc-only Feature Table additions and requires implementation or explicit planning. Internal use only."
description-en: "Quality guardrail for Claude/Codex update integration. Detects doc-only Feature Table additions and requires implementation or explicit planning. Internal use only."
description-ja: "Claude/Codex upstream update 統合の品質ガードレール。Feature Table 追加時に「書いただけ」を検出し、実装または Plans 化を強制する。内部専用。"
user-invocable: false
disable-model-invocation: true
allowed-tools: ["Read", "Grep", "Glob", "Bash"]

Claude/Codex Update Review ガードレール

Claude Code / OpenAI Codex のアップデート統合時に「Feature Table に書いただけ」を防止する品質ガードレール。 Feature Table への追加が実装・検証・明示的な将来タスク化を伴っているかを分類し、不足があれば実装案を強制出力する。

Quick Reference

以下の状況でこのスキルがトリガーされる:

• Claude Code / Codex upstream update 統合 PR のレビュー時
• `docs/CLAUDE-feature-table.md` に新行が追加された diff を検出した時
• `/harness-review` が upstream update 統合 PR と判定した場合の内部呼び出し
• `claude-codex-upstream-update` スキル更新のレビュー時

トリガーしない状況:

• 通常の実装作業
• Feature Table / upstream 追従に関係しない変更
• セットアップ・初期化作業

差分入力の取得

このスキルは diff-aware review 専用のため、必ず以下のどちらかでレビュー対象差分を確定する。

1. 呼び出し元の `/harness-review` が PR diff / changed files / Feature Table 追加行を渡す 2. このスキル自身が read-only Bash で `git status --short`, `git diff --name-only`, `git diff -- docs/CLAUDE-feature-table.md`, `git show --stat --name-only` などを実行して確認する

Bash は read-only git inspection のみに使う。テスト実行、format、生成、network access、ファイル変更を伴うコマンドは実行しない。 diff が取得できない場合は `B: 書いただけ 0 件` と推定せず、「差分未提供のため分類不能」としてレビューを止める。

前提チェック

レビュー冒頭で必ず確認する:

• diff source が呼び出し元提供または read-only git inspection のどちらかで確定しているか
• `skills/` や `hooks/` を編集した PR の場合、直後に `/reload-plugins` を実行して runtime cache を更新したか（`{skills,hooks}/**` ガイドライン準拠）
• upstream のバージョン別分解表があるか
• Claude Code の一次情報 URL が `anthropics/claude-code` または公式 docs になっているか
• Codex の一次情報 URL が `openai/codex/releases` または OpenAI 公式記事になっているか
• `B: 書いただけ` が残っていないか
• skill mirror を触る場合、`skills/`, `codex/.codex/skills/`, `.agents/skills/` の差分が意図通りか

禁止する古い参照:

• 旧 TypeScript guardrail path
• 旧 TypeScript implementation glob
• 旧 Codex feature-table path
• 旧 Codex plugin directory
• 旧 Codex state directory を現行正本として扱う記述
• 存在しない Anthropic 側 Codex repo URL

A/B/C/P 分類

Feature Table に追加された各項目を、以下の A/B/C/P のいずれかに分類する。

(A) 実装あり

定義: Feature Table の追加に対応する hooks / settings / Go / scripts / agents / skills / tests の変更が同じ PR に含まれている。

判定条件:

• Feature Table の行で言及されている機能に関連するファイルが変更されている
• `hooks/hooks.json`, `.claude-plugin/hooks.json`, `.claude-plugin/settings.json`, `go/internal/guardrail/`, `go/internal/hookhandler/`, `scripts/`, `agents/`, `skills/`, `tests/` のいずれかに実体差分がある
• 対象テストまたは検証スクリプトで固定されている

例:

| Feature Table 追加 | 対応する実装変更 | 判定 | |-------------------|----------------|------| | `AskUserQuestion updatedInput` | Go handler + hooks wiring + upstream integration test | A | | `sandbox.network.deniedDomains` | `.claude-plugin/settings.json` + jq test | A | | `find -delete hardening` | `go/internal/guardrail/` + unit test | A |

結果: OK。追加のアクション不要。

---

(B) 書いただけ

定義: Feature Table にのみ行が追加され、Harness 側の実装変更も Plans 化もない。かつ upstream 自動継承にも該当しない。

判定条件:

• Feature Table に新行がある
• 同じ PR 内で関連する実装 / test / skill / Plans の変更がない
• Harness が独自の付加価値を提供すべき機能である

例:

| Feature Table 追加 | 対応する実装変更 | 判定 | |-------------------|----------------|------| | `PreCompact hook` | なし | B | | `permission hardening` | settings / guardrail / tests の確認なし | B | | `Codex marketplace` | Plans への切り出しなし | B |

結果: NG。PR をブロックし、実装案または Plans 化を要求する。

---

(C) upstream 自動継承

定義: Claude Code / Codex 本体のパフォーマンス改善・バグ修正・内部最適化等で、Harness 側の変更が不要な項目。

判定条件:

• upstream 本体の修正であり、Harness がラップ・拡張する余地がない
• Harness の settings / hooks / guardrail / workflow / tests に影響しない
• Feature Table に「upstream 自動継承」または「CC 自動継承 / Codex 側自動継承」と明記されている

注意:

• permission / sandbox / security / Bash allowlist / MCP trust boundary は安易に C にしない
• その項目が Harness の独自 guardrail や settings に影響しないことを確認してから C にする
• Claude Code 2.1.113 の hardening は、`sandbox.network.deniedDomains`, wrapper Bash deny, `find -exec/-delete`, macOS dangerous rm paths を確認するまで C 判定しない

例:

| Feature Table 追加 | 理由 | 判定 | |-------------------|------|------| | `Agent Teams permission dialog crash fix` | CC 本体の crash fix。Harness 側変更不要 | C | | `Codex Guardian timeout wording` | Codex 側 UX 修正。Harness surface なし | C |

結果: OK。ただし理由を明記する。

---

(P) Plans 化

定義: 今回は直接実装しないが、Harness に取り込む価値があるため `Plans.md` に明示タスクとして残す項目。

判定条件:

• Feature Table の付加価値列が `A: 将来タスク化` または `P: Plans 化` として読める
• `Plans.md` に対応タスクがあり、setup / guardrails / memory / Codex workflow など実装面が明記されている
• alpha release や大規模設計変更など、即実装しない理由が書かれている

例:

| Feature Table 追加 | Plans への切り出し | 判定 | |-------------------|-------------------|------| | `Codex marketplace / MCP Apps` | Codex workflow 比較軸タスク | P | | `Codex 0.122.0-alpha` | stable 化後の compare 調査タスク | P |

結果: OK。次回 cycle で拾える。

Upstream update PR チェックリスト

## Claude/Codex update 統合チェックリスト

### 1. 一次情報と分解表
- [ ] diff source が呼び出し元提供または read-only git inspection のどちらかで確定している
- [ ] Claude / Codex の公式 URL を確認した
- [ ] Version / Upstream item / Category / Harness surface / Action の表がある
- [ ] alpha / stable / docs-only の区別がある

### 2. Feature Table 差分
- [ ] `docs/CLAUDE-feature-table.md` の追加行を列挙した
- [ ] 各行に A / C / P のいずれかが付いている
- [ ] B が 0 件である

### 3. カテゴリ別の確認
- [ ] (A) 実装あり: 対応する実装ファイルとテストがある
- [ ] (B) 書いただけ: 0 件。残る場合は PR ブロック
- [ ] (C) 自動継承: permission / sandbox / security / workflow 影響を確認済み
- [ ] (P) Plans 化: `Plans.md` に将来タスクがある

### 4. Mirror と stale path
- [ ] `skills/` と `codex/.codex/skills/` の意図しない drift がない
- [ ] `.agents/skills/` が存在する場合、Claude/Codex 表記が壊れていない
- [ ] 旧 TypeScript guardrail path、旧 Codex plugin directory、旧 Codex feature-table path などの旧参照がない

### 5. CHANGELOG / tests
- [ ] CHANGELOG に「今まで / 今後」または相当する user-facing 説明がある
- [ ] upstream integration test または対象 unit test が追加 / 更新されている

カテゴリ B 検出時の出力フォーマット

カテゴリ B が 1 件以上検出された場合、以下のフォーマットで実装案を出力する。 このフォーマットの出力