スキル: コードレビュー

トリガー

/impl review [ファイルパス、PRリファレンス、または変更内容の説明]

説明

構造化されたチェックリストを使用してコード変更をレビューします。CRITICAL、IMPORTANT、MINORの分類に基づいた指摘を含むレビューを作成します。

入力

• target: string、ファイルパス、PRリファレンス、またはレビュー対象の説明(必須)

プロセス

1. コード変更を読む(差分または完全なファイル)
2. レビューチェックリストを適用する:
   • ロジックの正確性: コードが主張する動作をしているか?エッジケースは処理されているか?
   • コードスタイル: ESLintとPrettierへの準拠;TypeScript strict modeの遵守
   • セキュリティ(OWASP Top 10): 入力検証、出力エンコーディング、認証チェック、シークレット管理、依存関係の脆弱性
   • パフォーマンス: O(n)の計算量、N+1クエリパターン、無制限ループ、ページネーション欠落
   • テストカバレッジ: 新しいコードパスがカバーされているか?カバレッジが低下していないか?
   • ドキュメント: 公開インターフェースが文書化さ

...