Uh oh!
There was an error while loading. Please reload this page.
refactor: レビューと PR コメント対応の Skill を統合 (0-2) - #68
Conversation
review + review-branch → review (466 → 329 行) - --branch でローカル差分のセルフレビューへ切り替える 2 モード構成にする - review-branch 側にしかなかったセキュリティ・エラーハンドリングの観点を 共通の「観点」へ取り込み、review 側と重複していた品質観点を 1 本化する review-pr-comments + fix + resolve-pr-comments → fix (559 → 324 行) - --classify-only で分類のみの読み取り専用モードを提供する - 返信の書き分けと Resolve 手順を fix の手順に統合し、3 Skill に分散していた コメント取得・重要度判定・Resolve の記述を 1 本化する - コメント ID から thread node_id を引く手順を残す あわせて manifest 3 種、plugin.json の skills 配列、build-runtime-plugins.sh の パス書き換え対象、および他 Skill・文書からの参照を更新した。
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 1 | codex | REQUEST_CHANGES
外部 AI 委譲の統合後手順で存在しない Skill を参照しているため、codex / gemini 指定レビューの実行手順が途切れます。あわせて削除後の Skill 数を README の公開値と総数へ反映してください。
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 1 | gemini | REQUEST_CHANGES
統合の網羅性と、manifest・関連スクリプト群への波及も漏れなく対応されています。
しかし、review からの外部 AI 呼び出し手順の委譲先として、まだ存在しない /ndf:external-ai が指定されているため、Task 0-3（次 PR）での統合前にエラーとなる問題があります。該当箇所を既存の codex / gemini 参照に戻すよう修正をお願いします。
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
- review Skill が参照していた `/ndf:external-ai` は Task 0-3 で統合予定であり 本 PR 時点では存在しないため、`/ndf:codex` / `/ndf:gemini` 参照に戻した - README の公開 Skill 数（Claude 26 / Kiro 25 / Codex 27）と元 Skill 数（46）を 本 PR 後の実態に合わせて更新した Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AGejnYyYFuSkQjBhW2KQNy
takemi-ohama
commented
Aug 8, 2026
🔧 cross-review round 1 — 指摘対応サマリ修正コミット:
修正内容
配布物 3 種は 本 PR で意図的に対応していない事項
検証結果
|
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 2 | gemini | APPROVE
Skill の統廃合（review-branch, review-pr-comments, resolve-pr-comments の削除と機能統合）について網羅的に確認しました。
各ランタイム（Claude Code / Codex / Kiro）の manifest や設定ファイル、および build-runtime-plugins.sh による生成プロセスへの反映に漏れがないことを確認しています。
コードの複雑性も適切に削減されており、既存の参照も正しく更新されています。対応に問題ありません。
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 2 | codex | REQUEST_CHANGES
Codex/Kiro 配布物の review が、そのランタイムに同梱されていない外部 AI Skill を参照しているため、配布物ごとの手順か manifest を揃える修正が必要です。
Uh oh!
There was an error while loading. Please reload this page.
codex / kiro runtime の manifest には `codex` / `gemini` Skill が含まれない ため、`/ndf:codex` / `/ndf:gemini` への無条件参照ではインストール後に 呼び出し手順を解決できなかった。ベースブランチと同じく「同梱されている 場合はその skill に従う」条件付き記述に戻し、非同梱 runtime 向けの要点 （起動コマンド・完了検知・成果物回収・サンドボックスに関する警告）を インラインで復元した。「関連」節のリンクにも同梱 runtime 限定である旨を 注記した。 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AGejnYyYFuSkQjBhW2KQNy
takemi-ohama
commented
Aug 8, 2026
🔧 cross-review round 2 — 指摘対応サマリ修正コミット:
gemini (round 2) は APPROVE・指摘なしでした。codex (round 2) の major 1 件に対応しています。 修正内容[major / 整合性] Codex / Kiro 配布物の manifest の実態を確認したところ、
配布物 3 種は 本 PR で意図的に対応していない事項
検証結果
|
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 3 | codex | REQUEST_CHANGES
fix Skill の統合後手順に、フラグ付き実行時の PR 番号解釈と Resolve Conversation の ID 取得方法で修正が必要な箇所があります。
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 3 | gemini | APPROVE
統合された review と fix の内容、削除された Skill の網羅的な削除状況（各マニフェスト、plugin.json、build-runtime-plugins.sh の変更）、および 3 ランタイムへの影響を確認しました。
既存の責務（分類・返信・Resolve 等）が欠落なく統合先に吸収されており、孤立した参照も残っていません。また、外部 AI 委譲の自己完結手順が意図通り復元されていることも確認しました。設計と正確性に問題ないため Approve とします。
- PR 番号とオプションを分離して解釈する手順に修正（`--classify-only` / `--defer-nit` / `--severity-min` 付き実行で PR 番号が壊れていた） - Resolve Conversation で使う ID を review thread ID (PRRT_) と明記し、 レビューコメントの node_id (PRRC_) を引くショートカットを削除 - reviewThreads query に databaseId を追加し、コメント ID と thread の 対応付けができるようにした Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AGejnYyYFuSkQjBhW2KQNy
takemi-ohama
commented
Aug 8, 2026
🔧 cross-review round 3 — 指摘対応サマリ修正コミット:
gemini (round 3) は APPROVE・指摘なし でした。codex (round 3) の major 2 件に対応しています。いずれも統合元 Skill ( 修正内容1. [major / 正確性] 統合元は引数が PR 番号のみだったため成立していましたが、統合後の 2. [major / 正確性] Resolve 用の ID 取得にコメントの
配布物 3 種 ( 本 PR で意図的に対応していない事項
検証結果
|
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 4 | gemini | APPROVE
Skill の統廃合と各ランタイムの manifest や設定ファイル等への反映状況について網羅的に確認し、参照漏れ等修正が必要な箇所がないことを確認しました。
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 4 | codex | APPROVE
指摘事項はありません。
release ブランチ側の削除 (PR #71 ブラウザ自動テスト集約 / PR #72 Skill 削除) と 本ブランチ側の統合 (review / fix) の双方を反映した。 - manifest 3 種と plugin.json は両者の削除を反映した積集合にした (claude 24 / codex 24 / kiro 23、shared 33) - browser-test は playwright-authoring へ統合済みのため削除し、 同 Skill に残っていた /ndf:review-branch 参照を /ndf:review --branch へ更新 - README の Skill 一覧と個数を実体に一致させた
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 5 | gemini | APPROVE
Skill の統廃合および各ランタイムのマニフェスト・ビルドスクリプトへの反映が網羅的に行われていることを確認しました。追加の修正提案はありません。
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 5 | codex | APPROVE
修正が必要な新規指摘はありません。
Uh oh!
There was an error while loading. Please reload this page.
並行 PR (#68 review/fix 系統合、#71 ブラウザ自動テスト集約、#72 Skill 削除) が 先に release へ入ったため、本ブランチの external-ai 統合と競合していた。 競合解消方針: - manifest / plugin.json の skills は両ブランチの削除を両方反映 (積集合)。 release が消した review-branch / review-pr-comments / resolve-pr-comments / browser-test と、本ブランチが消した codex / gemini をいずれも除去し、 release が追加した playwright-authoring と本ブランチの external-ai を残した。 - review / cross-review SKILL.md は release 側の再構成をベースに採用し、 本ブランチの /ndf:codex /ndf:gemini → /ndf:external-ai 参照差し替えを再適用。 - README.md の Skill 一覧と個数は plugins/ndf-shared/skills/ と manifest の 実数から数え直した (元Skills 32個 / Claude 23・Kiro 23・Codex 25)。 - plugins/ndf-{claude,codex,kiro}/skills/ は共有側を直したうえで scripts/build-runtime-plugins.sh で再生成。
Summary
対象が違うだけでレビュー観点が同一だった 2 個と、分類・修正・返信で 3 分割されていた PR コメント対応を、それぞれ 1 個へ統合する。
review←review-branch--branchを付けるとローカルブランチの差分をレビューする 2 モード構成にした。--branchモードgit diff <既定ブランチ>の差分APPROVE/REQUEST_CHANGES統合にあたり、
review-branch側にしかなかったセキュリティ（SQL インジェクション / XSS / CSRF / 認証認可 / 機密情報）とエラーハンドリングの観点を共通の「観点」へ取り込み、両者に重複していた品質観点（重複排除、関数長、N+1）を 1 本化した。--focusはreview-branchの第一引数を PR モードでも使えるようにしたもの。fix←review-pr-comments+resolve-pr-comments分類 → 修正 → 返信 → Resolve を 1 本の流れにし、
--classify-onlyで分類のみの読み取り専用モードを提供する。review-pr-commentsの 5 分類テーブル（🔴 重大 / 🟡 改善推奨 / 🟢 軽微 / ⚪ 参考 / 🔵 別 PR 対応）と出力形式を--classify-onlyの出力として取り込んだresolve-pr-commentsの返信テンプレートを「返信の書き分け」表に統合し、コメント ID から thread node_id を引く手順（gh api "repos/.../pulls/comments/<id>" --jq '.node_id'）を残したcross-reviewがループ内でfixを呼ぶ規約（--defer-nit、戻り値fix-pr<PR>-result.jsonのスキーマ）は変更していない。統合前後の行数
reviewfixいずれも 500 行上限内。
併せて更新した箇所
plugins/ndf-shared/manifests/{claude,codex,kiro}-skills.txt— 統合元 3 個を削除（29→26 / 30→27 / 28→25）plugins/ndf-claude/.claude-plugin/plugin.jsonのskills配列 — 存在しないパスを指さないよう 29→26 件へscripts/build-runtime-plugins.sh—rewrite_codex_skill_paths/rewrite_kiro_skill_pathsのパス書き換え対象からreview-pr-comments/SKILL.mdを除去issue-plan-strategy/browser-test/cross-review/README.md/docs/ndf-plugin-reference.mdの参照を新コマンド名へ既知の残課題（Task 0-7 の担当範囲）
reviewはdisable-model-invocation: trueを持つため、統合で引き継いだwhen_to_useのトリガ（セルフレビュー等）は現時点では発動しない。reviewからdisable-model-invocationを外すのは Task 0-7 の担当範囲のため、この PR ではトリガ語を frontmatter に残すにとどめている。0-7 で外した時点でreview-branch相当の自動発動が復活する。Test plan
SKILL.mdが 500 行を超えていないcross-reviewからfixを呼ぶ規約（引数・戻り値スキーマ）が変わっていないgrep -rnで旧 Skill 名の参照がissues/の履歴を除いて残っていないplugin.jsonに反映されているbash scripts/build-runtime-plugins.sh --checkが差異を検出しないpython3 scripts/check-markdown-links.py --root .が成功するbash scripts/validate-runtime-plugins.shが成功するrelease PR: #66
plan: issues/ndf-development-skills/07-tasks.md