Menu
プロジェクトガイドラインへの準拠をチェックし、バグ、スタイル違反、コード品質問題を検出する。PRレビュー時に使用。
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
| name | code-review |
| description | プロジェクトガイドラインへの準拠をチェックし、バグ、スタイル違反、コード品質問題を検出する。PRレビュー時に使用。 |
プロジェクトガイドラインへの準拠をチェックし、高信頼度の問題のみを報告する。
AGENTS.md の「よくある間違いと回避方法」docs-template/MASTER.md のコード生成ルールdocs-template/03-implementation/PATTERNS.md の実装パターン各問題には0-100の信頼度スコアを付与してください：
| スコア | 意味 | 報告 |
|---|---|---|
| 0-25 | 誤検出または既存の問題 | 報告しない |
| 26-50 | マイナーな指摘（ガイドラインに明記なし） | 報告しない |
| 51-79 | 有効だが低影響 | 報告しない |
| 80-90 | 重要な問題 | 報告する |
| 91-100 | クリティカルなバグまたは明示的な違反 | 必ず報告 |
報告閾値: 信頼度80以上のみ報告
結果は以下のMarkdown形式で出力してください。Copilot CLI からのファイル出力に使用されます。
# Code Review Results
## Critical Issues (信頼度 91-100)
- [ファイル名:行番号] 問題の説明
- 信頼度: XX
- 理由: なぜこれが問題か
- 修正提案: どう修正すべきか
## Important Issues (信頼度 80-90)
- [ファイル名:行番号] 問題の説明
- 信頼度: XX
- 理由: なぜこれが問題か
- 修正提案: どう修正すべきか
## Summary
- 検出された問題数: X
- Critical: X
- Important: X
Enforces error handling standards: silent error prohibition, custom error class hierarchy (AppError base with ValidationError, NotFoundError, InternalError), Result pattern (Result.ok/Result.fail), proper try-catch with error type checking, structured error logging with context metadata, and HTTP status code mapping. Use when implementing error handling, reviewing catch blocks, or designing error responses.
Enforces project coding standards during code review: naming conventions (PascalCase for classes, camelCase for methods/variables, UPPER_SNAKE_CASE for constants, kebab-case for files), standard file structure, anti-magic-number policy, file size limits (500/800 lines), security patterns, and design patterns. Use when reviewing code, writing new code, or checking adherence to project conventions.
Guides test implementation following the test pyramid (75% unit, 20% integration, 5% E2E), Arrange-Act-Assert pattern, and project coverage thresholds (branches 70%, functions 80%, lines 80%, statements 80%). Covers test naming conventions, test independence, mock and stub patterns, test data management with fixtures and builders, and CI integration. Use when writing tests, reviewing test code, or setting up test infrastructure.
Creates and reviews agent skills safely by enforcing deletion-prevention harness rules: no fixed absolute temp paths, no cwd-dependent cleanup, no wildcard deletes, no workspace-external file operations, no silent cleanup failures, and mandatory temp-root validation before any destructive command. Use when creating a new skill, updating an existing SKILL.md, or reviewing prompts/runbooks that include temp files, extraction, cleanup, PowerShell, bash, rm, del, Remove-Item, or archive expansion.
機能を保持したまま不要な複雑性を排除し、コードの簡潔性と可読性を向上させる。30行超の関数や3段以上のネスト検出時に使用。
コードコメントの正確性、完全性、長期的な保守性を分析する。JSDoc、コメント、README等のドキュメントが変更された場合に使用。