code-review
Freeプロジェクトガイドラインへの準拠をチェックし、バグ、スタイル違反、コード品質問題を検出する。PRレビュー時に使用。
The English summary is being prepared. Showing the original description.
テスト品質を動作カバレッジの観点から分析し、クリティカルなテストギャップを特定する。テストファイルの追加・変更時に使用。
The English summary is being prepared. Showing the original description.
See how to installReview the instructions the agent will receive before you install.
動作カバレッジの観点からテスト品質を分析し、クリティカルなギャップを特定する。
| 優先度 | 意味 | 例 |
|---|---|---|
| 9-10 | クリティカル | データ損失、セキュリティ、システム障害の可能性 |
| 7-8 | 重要 | ユーザー向けエラーの可能性 |
| 5-6 | エッジケース | 混乱や軽微な問題 |
| 3-4 | Nice-to-have | 完全性のため |
| 1-2 | オプショナル | 改善の余地 |
報告対象: 優先度7以上
# Test Analysis Results
## Critical Gaps (優先度 9-10)
### [機能/関数名]
- ファイル: path/to/file.ts
- テストされていない動作: エラー時のリトライロジック
- リスク: データ損失の可能性
- 優先度: 10
- 推奨テストケース:
- エラー発生時にリトライが実行されることを確認
- 最大リトライ回数後にエラーがスローされることを確認
## Important Gaps (優先度 7-8)
...
## Summary
- Critical Gaps: X
- Important Gaps: X
No reviews yet. Tell others what you thought.
No login needed. Share how it worked for you, ask a question, or note where it did not run.
No comments yet. Be the first to write one.
プロジェクトガイドラインへの準拠をチェックし、バグ、スタイル違反、コード品質問題を検出する。PRレビュー時に使用。
The English summary is being prepared. Showing the original description.
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.
The English summary is being prepared. Showing the original description.
機能を保持したまま不要な複雑性を排除し、コードの簡潔性と可読性を向上させる。30行超の関数や3段以上のネスト検出時に使用。
The English summary is being prepared. Showing the original description.
コードコメントの正確性、完全性、長期的な保守性を分析する。JSDoc、コメント、README等のドキュメントが変更された場合に使用。
The English summary is being prepared. Showing the original description.
サイレント失敗を検出し、堅牢なエラーハンドリングを確保する。PRレビューでのエラーハンドリング検査時に使用。
The English summary is being prepared. Showing the original description.
Enforces error handling standards: silent error prohibition, custom error class hierarchy (AppError base with category never-fallback / transient / permanent, cause preservation, ValidationError / UnauthorizedError / UpstreamError etc. — canonical definitions in ERRORS.md), external-boundary normalization (normalizeExternalError), Result pattern (Result.ok/Result.fail), proper try-catch with error type checking, structured error logging with a single Logger contract, fallback prohibition categories (isNeverFallback), and HTTP status code mapping. Use when implementing error handling, reviewing catch blocks, or designing error responses.
The English summary is being prepared. Showing the original description.