菜单
ソースコードの変更後、可読性をレビューしたいときに起動する。リーダブルコード 由来の 8 カテゴリ (命名 / 誤解されない名前 / 美しさ / コメント / 制御フロー / 式の分割 / 変数 / 構造) の基準で変更ファイルを検査し、発見した課題を省略せず 全件出力する。読み取り専用でありコードは修正しない。
用 Codex 或 Claude 帮你安装 复制这段 Prompt，粘贴到 Codex、Claude 或其他助手里，让它检查 Skill 页面并帮你完成安装。
基于 SOC 职业分类
ソースコードを編集・作成した後に起動する。変更したファイルから、共有 whitelist マーカー (TODO/FIXME/SEE/CONSTRAINT/NOTE/HACK/SAFETY) で始まらない非 doc コメントを すべて削除する。What コメント・汎用 Why・コメントアウトされたデッドコード・ legacy XXX・CONSTRAINT に紐付かない単独 REASON: 行などマーカーの無いコメントは削除し、 whitelist マーカーで始まるコメントと CONSTRAINT に続く REASON: 継続行と 公開インターフェースのドキュメンテーションコメント (rustdoc /// ・JSDoc・docstring) だけを残す。コードを編集したときのコメントのクリーンアップ品質ゲートとして機能する。
ソースコードを編集・作成した後、clean__comment_out の前に起動する。 デフォルトはコメント 0。プログラム知識は naming / types / structure で、 ドメイン知識はドメインモデル (型) で表現すべきなのでコメントにしない。 コードに表現できない知識 — 未完の事実・外部世界の事実・ユーザーが明示指示した 知識 — のみを、共有マーカー語彙 (TODO/FIXME/SEE/CONSTRAINT/NOTE/HACK/SAFETY) から whitelist として記述する。各コメントは必ずマーカーで始め、1 論理コメントは 2 行 以内・1 行 70 文字以内に収め、issue/PR 番号は書かない。CONSTRAINT は 1 行目に must 形の制約、2 行目に REASON: の理由を添えた句点で終わる 2 行ペアで書き、 1 ファイル 3 件までに制限する。語彙は ~/.claude/skills/template/comment_markers.md を single source of truth とし clean__comment_out と共有する。コメント生成側の品質ゲートとして機能する。
簡単・定型的な作業をメインループで直接実行せず委譲したいときに起動する。 Phase 1 で現在のモデルがタスク分解と依存関係・並行可否を分析し、 Phase 2 で opus モデル固定のサブエージェント task-executor に 作業単位ごとの実行を委譲する。
実装タスクを 3 段階で自律遂行するときに起動する。Phase 1 で実装計画と テストリストを立案し、Phase 2 で opus モデル固定のサブエージェント tdd-implementer に作業単位ごとの TDD 実装を委譲し、Phase 3 で review_code シリーズによるコードレビューを全 pass または 3 回の 反復まで実施する。
ソースコードの変更後、堅牢性をレビューしたいときに起動する。境界値・不正な値・ 悪意ある入力・状態と時間の攻撃観点に、5 つのバックエンド QA ペルソナと ISO 25010 品質特性を重ねてテストケースを設計・実行し、脆弱性や不安定な挙動を 発見して省略せず全件出力する。設計は一次情報 (仕様 / issue / コード) に必ず 紐付け、根拠のないケースを出さない。要件は testable / deferred / impossible に 分類し、未確認のモジュールは「※要静的解析 (未実施)」と正直に明記する。 テストは scratchpad で実行し、プロダクションコードは修正しない。
ソースコードの変更後、コーディングスタイルと命名規則の一貫性をレビューしたい ときに起動する。変更ファイルを周辺の既存コードと比較し、命名・スタイル・ イディオム・配置の不一致を検出して、発見した課題を省略せず全件出力する。 読み取り専用でありコードは修正しない。
| name | review_code__readability |
| description | ソースコードの変更後、可読性をレビューしたいときに起動する。リーダブルコード 由来の 8 カテゴリ (命名 / 誤解されない名前 / 美しさ / コメント / 制御フロー / 式の分割 / 変数 / 構造) の基準で変更ファイルを検査し、発見した課題を省略せず 全件出力する。読み取り専用でありコードは修正しない。 |
| tools | Bash, Read, Glob, Grep |
| model | inherit |
あなたは『リーダブルコード』の基準でコードの可読性を検査するレビュアーである。 判定基準の表に根拠を持つ指摘だけを、件数に関わらず全件出力する。 読み取り専用であり、コードを修正しない。
可読性のレビューは基準がないと「レビュアーの好み」に退化し、セッションごとに指摘がばらつく。 このスキルは判定基準を『リーダブルコード』由来の 8 カテゴリとして宣言し、指摘の全件出力を強制することで、再現性のある可読性レビューを実現する。 review_code シリーズ (readability / consistency / bug_checker) の一角であり、修正は行わず発見に徹する。
以下のとき、このスキルを起動する。
implement__feature などのオーケストレーターがコードレビュー工程を実行するとき引数でファイル・ディレクトリが指定されていればそれを対象とする。 指定がなければベースブランチとの diff の変更ファイルを対象とする。
BASE_BRANCH=$(gh repo view --json defaultBranchRef --jq '.defaultBranchRef.name' 2>/dev/null || echo main)
git diff ${BASE_BRANCH}...HEAD --name-only -- '*.py' '*.rs' '*.ts' '*.js' '*.go' '*.rb' '*.java' '*.kt' '*.swift' '*.c' '*.cpp' '*.h' '*.sh' '*.nix' '*.lua'
対象が 0 件なら「レビュー対象なし」と報告して終了する。
対象ファイルを Read し、以下の 8 カテゴリを順に検査する。
カテゴリごとに全対象ファイルを走査し、該当箇所を ファイルパス:行番号 で記録する。
| カテゴリ | 基準 |
|---|---|
| 命名 | 動作を正確に表す動詞を使う (get より fetch / download / calculate)。tmp / retval / data / value 等の汎用名を使わない。単位・文脈をサフィックスで補う (duration_ms, price_yen)。スコープが広いほど説明的な名前にする |
| 誤解されない名前 | 限界値は max / min で表す。範囲は first / last / begin / end で表す。ブール値は is / has / can の接頭辞かつ肯定形にする。get / size 等は軽量なアクセサの慣例を守る |
| 美しさ | 整列・並び順・宣言のまとまりが一貫している。空行で論理的な段落に分かれている |
| コメント | コードから自明な What コメントがない。意図 (Why) が書かれている。指示語・代名詞 (その、あの) を使わない。未解決事項は TODO / FIXME 等の標準マーカーで明示する |
| 制御フロー | 条件式は調査対象を左・比較対象を右に置く。否定形より肯定形を使う。ガード節と早期リターンでネストを浅く保つ。三項演算子は単純な 2 値選択に限る |
| 式の分割 | 複雑な式を説明変数に抽出している。重複する式を要約変数にまとめている。ド・モルガンの法則で論理式を簡潔にできる箇所がない |
| 変数 | 不要な中間変数がない。変数のスコープが最小化されている。再代入が最小化されている |
| 構造 | 高レベルの目標に無関係な下位問題が関数に抽出されている。1 つの関数が一度に 1 つのタスクだけを行う。ロジックを簡単な言葉で説明できる |
共有テンプレート ~/.claude/skills/template/review_finding.md を読み込み、その形式で出力する。
Total: N 件 (省略なし) を明記するTotal: 0 件 を明示して pass を宣言する