Cc Update Review

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/ のいずれかに実体差分がある
• 対象テストまたは検証スクリプトで固定されている

例:

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

(B) 書いただけ

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

判定条件:

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

例:

結果: 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 判定しない

例:

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

(P) Plans 化

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

判定条件:

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

例:

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

Upstream update PR チェックリスト

markdownCopy code
## 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 件以上検出された場合、以下のフォーマットで実装案を出力する。 このフォーマットの出力は必須であり、省略は許可されない。

markdownCopy code
## カテゴリ B 検出: 実装案

### B-{番号}. {Feature Table の項目名}

**現状**: Feature Table に記載のみ。Harness 側の実装 / 検証 / Plans 化なし。

**Harness ならではの付加価値**:
{この機能を Harness がどう活用すべきかの具体的な説明}

**実装案**:

| 対象ファイル | 変更内容 |
|------------|---------|
| `{ファイルパス}` | {具体的な変更内容} |
| `{ファイルパス}` | {具体的な変更内容} |

**ユーザー体験の改善**:
- 今まで: {現在のユーザー体験}
- 今後: {実装後のユーザー体験}

**実装の優先度**: {高 / 中 / 低}
**推定工数**: {小 / 中 / 大}

関連スキル

• claude-codex-upstream-update - upstream 差分調査と実装 cycle
• harness-review - コードレビュー
• harness-work - カテゴリ B / P の実装
• memory - 分類基準の SSOT 化