code-review
Freeプロジェクトガイドラインへの準拠をチェックし、バグ、スタイル違反、コード品質問題を検出する。PRレビュー時に使用。
The English summary is being prepared. Showing the original description.
サイレント失敗を検出し、堅牢なエラーハンドリングを確保する。PRレビューでのエラーハンドリング検査時に使用。
The English summary is being prepared. Showing the original description.
See how to installReview the instructions the agent will receive before you install.
サイレント失敗を検出し、堅牢なエラーハンドリングを確保する。
以下のコードパターンを重点的に検査:
.catch(), onError)if (error), if (!result))?., ??)catch (e) {}catch (e) { console.log(e); }catch (e) { return null; }| レベル | 説明 | 例 |
|---|---|---|
| CRITICAL | サイレント失敗、ブロード catch | 空のcatchブロック、catch(e) {} |
| HIGH | 不十分なエラーメッセージ | console.log("error") のみ |
| MEDIUM | コンテキスト不足 | エラーの原因が不明確 |
結果は以下のMarkdown形式で出力してください。
# Error Handler Hunt Results
## CRITICAL Issues
- [ファイル名:行番号] 問題の説明
- コード: 問題のあるコード
- 問題: 何が問題か
- リスク: ユーザーへの影響
- 修正提案: 推奨される修正
## HIGH Issues
- [ファイル名:行番号] 問題の説明
- コード: ...
- 問題: ...
- 修正提案: ...
## Summary
- CRITICAL: X
- HIGH: X
- MEDIUM: 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.
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.
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.
The English summary is being prepared. Showing the original description.