Menú
PR 完成時にコードレビューを行い、指摘修正→再レビューのサイクルを回すプロトコル (Claude Code)。
Instalar con Codex o Claude Copia este prompt, pégalo en Codex, Claude u otro asistente, y deja que revise la página de la skill y la instale por ti.
Basado en la clasificación ocupacional SOC
| name | pr_review_cycle |
| description | PR 完成時にコードレビューを行い、指摘修正→再レビューのサイクルを回すプロトコル (Claude Code)。 |
| 手段 | 用途 |
|---|---|
/code-review スラッシュコマンド | 現在の差分 (作業ブランチ) を Claude が観点別にレビュー。--comment で PR にインラインコメント投稿、--fix で修正適用も可 |
レビュー用サブエージェント (Task/Agent) | 独立した観点 (correctness / security / 可読性) で並列にレビューさせ、結論を統合 |
| GitHub の PR レビュー | リポジトリに設定された自動レビューア (例: GitHub Actions / 外部ボット) があればそれも併用。gh pr view --comments で取得 |
[!WARNING] 多重レビューの防止: 同じ差分に重複してレビューを走らせない。再レビュー前に必ず
gh pr view <PR番号> --commentsで未対応の指摘が残っていないか確認する。
/code-review を実行する (またはレビュー用サブエージェントを起動)。gh pr view <PR番号> --comments で回収する。cargo fmt → cargo clippy --workspace --all-targets -- -D warnings → cargo test --workspace を全実行する。gh pr comment <PR番号> --body "...")。/code-review を実行する (または GitHub 側に再レビューを依頼)。指摘レベル別の対応ルール:
| レベル | 対応 |
|---|---|
| Critical | 必ず修正。修正後に再レビュー必須 |
| High | 必ず修正。修正後に再レビュー必須 |
| Medium | 内容が妥当であれば修正する |
| Low | 判断して対応。棄却する場合は PR コメントで理由を残す |
マージ判定フロー:
main への直接 push / 直接マージは禁止。必ず PR 経由でマージする。CSW のユーザー向け UI・コピーの正典。ユーザー向け語彙(「環境」)、共有モードの3語(共有/分離/コピー)、選択行の見せ方(塗りのみ)、メニューバーを主機構にしない、コピーの一次ソースは LP、Claude 本体の現行用語への追従、並列 UI の分量・文体、禁止記号(em-dash/※/絵文字)を定める。アプリ UI・LP・docs のラベル/モード名/説明文/マイクロコピーを作る/変える前に読む。
用語・概念・UI・コピーを1箇所変えたら、全サーフェス（アプリ UI・トレイ・docs ja/en・README ja/en・LP ja/en・スクリーンショット・OG 画像）へ同時反映し、旧表現を全リポ grep で残存ゼロ確認する。スクショ等の非テキスト派生物は grep に映らないので明示列挙して再生成する。UI・用語・コピーを変えたら発動。
PR をマージする際のレビュー・マージ実行チェックリスト (Rust + Tauri v2 / CSW)。スコープ外変更・安全装置の格下げ・テスト負債・高リスクインフラファイル改変を検出し、CI green を確認したうえで自律的に squash merge する手順を定義する。
CI (GitHub Actions の Test / Build / Security job) が想定時間を大幅に超過した場合のハング検出と cancel / rerun フロー。macos-latest runner の混雑や cargo の依存取得・コンパイル stall 等、transient な GHA 側障害の典型対処。CI が想定より長く `in_progress` のまま動かない時に発動。
日本語の見出し・本文の折り返し / タイポグラフィ品質ゲート。日本語 UI・LP・ドキュメントのコピーやフォントサイズ、改行、行間に触れる前に必ず読む。英語の折り返しルールをそのまま日本語に当てると崩れる(禁則・文節・字面密度が違う)ため、JP 専用の規則・モダン CSS・モジュラースケール・実機検証手順を正典化する。design-taste-frontend を補完する JP 特化スキル。
LP(website/) と人間向けドキュメント(docs/・README) と実装(crates/) が「同じ一つの事実」を語っているかを横断監査する。用語・アーキテクチャ・CLI 表面・機能主張・ja/en 整合・禁止表現を点検し、不整合を file:line と修正案つきで報告する。リリース前やドキュメント/LP/実装を変えた後に実行する read-only 監査。