Menu
コードの可読性レビュー。セキュリティは use-context-reviewer-security、エラーハンドリングは use-context-reviewer-silence、テスト設計は use-context-reviewer-testability に使う。
Install with Codex or Claude Copy this prompt, paste it into Codex, Claude, or another assistant, and let it review the skill page and install it for you.
Based on SOC occupation classification
issue が build に投入できる形かを検分し、verdict (build-ready / needs-plan / needs-fix) と指摘を返す。起票には使わない (/issue)。PR のスクリーニングには使わない (/preview)。
Inspect whether an issue is in shape to hand to build, returning a verdict (build-ready / needs-plan / needs-fix) and the findings. Do NOT use to file an issue (use /issue) or to screen a PR (use /preview).
critic-design による敵対的批判を伴う設計探索。生き残った案を構造化 plan にまとめ、自己点検して呼び出し元に返す。plan の永続先は issue の Plan 節が唯一。計画意図のないコードベース調査には使わない (代わりに /research)。
Design exploration with adversarial critique by critic-design. Assembles the surviving approach into a structured plan, self-checks it, and returns it to the caller. The issue's Plan section is the plan's only persistent home. Do NOT use for codebase investigation without planning intent (use /research instead).
構造化されたタイトルと本文で GitHub Issue を生成する。単独で成立し、前段を要求しない。challenge / research の成果物が会話にあれば本文の根拠に使い、/think の plan 下書きがあれば `## Plan` 節へ移設する。issue 番号を渡すと、起票済みで Plan 節を持たない issue へ plan を転記する。
Generate GitHub Issue with structured title and body. Standalone; requires no upstream stage. When challenge / research artifacts exist in the conversation, they feed the body's evidence; when a /think plan draft exists, it is transferred into the `## Plan` section. Given an issue number, it transfers a plan into a filed issue that has no Plan section.
| name | use-context-reviewer-readability |
| description | コードの可読性レビュー。セキュリティは use-context-reviewer-security、エラーハンドリングは use-context-reviewer-silence、テスト設計は use-context-reviewer-testability に使う。 |
| when_to_use | 可読性, 明確, 命名, 変数名, 関数名, ネスト, 関数設計, コメント, 複雑, Miller's Law, ミラーの法則, 認知負荷, AI-generated, 過剰設計 |
| allowed-tools | Read Task Bash(ugrep:*) Bash(bfs:*) |
| agent | reviewer-readability |
| context | fork |
| background | false |
| user-invocable | false |
しきい値は、作業記憶や 1 画面集中といった認知限界と、McCabe complexity のような確立されたメトリクスに基づく。
| 目標 | 推奨値 | 根拠 |
|---|---|---|
| 関数行数 | ≤30 | 1 画面の可読性 |
| ファイル行数 | ≤400 | モジュールレベルの認知上限 |
| ネスト深度 | ≤3 | 作業記憶内での分岐追跡 |
| 関数引数 | ≤3 | 引数順の暗記限界 |
| 循環的複雑度 | ≤10 | McCabe 1976: パス爆発なくテスト可能 |
| ID | パターン | 修正 |
|---|---|---|
| RD1 | 曖昧な processData() | validateUserEmail() |
| RD1 | 誤解を招く識別子 | 名前は意図を表す |
| RD2 | ネスト > 3 階層 | guard clause、関数抽出 |
| RD2 | 関数 > 30 行 | 分解 |
| RD3 | コメント: // increment i | 自明なので削除 |
| RD3 | コメント: // TODO: fix later | issue 化または今修正 |
| RD4 | 単一実装の interface | 2 つ目の実装が出るまで削除 |
| RD4 | 状態を持たないロジックの class | 純粋関数 |
| RD5 | 関数引数 > 5 | 設定オブジェクトまたは分解 |
| トピック | ファイル |
|---|---|
| Control Flow | ${CLAUDE_SKILL_DIR}/references/control-flow.md |
| Comments | ${CLAUDE_SKILL_DIR}/references/comments-clarity.md |
| AI Antipatterns | ${CLAUDE_SKILL_DIR}/references/ai-antipatterns.md |
| Naming | ${CLAUDE_SKILL_DIR}/references/naming.md |
