القائمة
コードの可読性レビュー。セキュリティは use-context-reviewer-security、エラーハンドリングは use-context-reviewer-silence、テスト設計は use-context-reviewer-testability に使う。
التثبيت باستخدام Codex أو Claude انسخ هذا Prompt والصقه في Codex أو Claude أو مساعد آخر ليراجع صفحة Skill ويثبّتها لك.
استنادا إلى تصنيف SOC المهني
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 |
