requesting-code-review
品質確認のためのコードレビューをリクエスト
Também disponível em: davila7,DMJGilbert,Cygnusfear,obra,Cycleaddict,CodingCossack
コードの問題は早期に発見することで、後々の高昂な修正を防ぐことができます。このスキルは、要件に対する実装を確認するためにコードレビューアー用サブエージェントを派遣し、開発を進める前に検証を行います。
Baixar o ZIP da skill
Upload no Claude
Vá em Configurações → Capacidades → Skills → Upload skill
Ative e comece a usar
Testar
A utilizar "requesting-code-review". メインブランチにマージする前にユーザー認証モジュールのコードレビューをリクエストします
Resultado esperado:
- 強み: auth.ts:15-42のauthミドルウェアによる懸念事項の明確な分離。8つの認証フローに対する包括的なテストカバレッジ。特定のエラータイプによる適切なエラー処理。
- 重要: ログインエンドポイントでのレート制限が欠落（セキュリティ上の懸念）。要件にパスワードリセットフローが文書化されていない。
- 軽微: JWT署名の分離を検討。建议。モジュール間でエラーメッセージの形式が一貫していない。
- 評価: 修正によりマージ可能。コア実装は堅実。本番環境で使用する前に重要な問題に対処すべき。
A utilizar "requesting-code-review". タスク3を完了しました - 進捗レポート付きの検索インデックスを追加しました。タスク4の前にレビューします
Resultado esperado:
- 強み: 進捗インジケーターが適切に動作しています（indexer.ts:45-52）。クリーンな非同期処理。空の結果に対するエッジケースの対応。
- 重要: 長時間実行されるインデックス操作の取り消しサポートが欠落。プロセス割り込み時のクリーンアップがない。
- 軽微: 進捗カウンターが100アイテムにハードコードされている。設定可能にすることを検討してください。
- 評価: 進行可能。タスク4前に取り消しを修正、軽微な項目は後で対応。
Auditoria de Segurança
SeguroPure documentation and template skill containing only markdown files with code review guidelines. No executable code, network operations, or file system access beyond reading its own files. All 38 static findings are false positives triggered by git command documentation and markdown placeholders.
Fatores de risco
⚙️ Comandos externos (18)
🌐 Acesso à rede (1)
📁 Acesso ao sistema de arquivos (1)
Pontuação de qualidade
O Que Você Pode Construir
タスク完了の検証
次のタスクに進む前に、各完了したタスクをレビューして、問題が複合化しないようにする
マージ前の品質ゲート
メインブランチにコードをマージする前に、機能実装が要件を満たしていることを確認する
新鮮な視点でのデバッグ
行き詰まったときに客観的なコードレビューを受けて、問題の特定と解決策の提案を受ける
Tente Estes Prompts
実装のコードレビューをリクエストします。内容: 検証および修復関数の追加。計画: docs/plans/deployment-plan.mdのタスク2。BASE_SHA: a7981ec HEAD_SHA: 3df7661。説明: 4つの問題タイプを持つverifyIndex()とrepairIndex()を追加しました。
マージ前のフルコードレビュー。内容: ユーザー認証モジュール。要件: RFC-001-auth.mdを参照。差分: origin/mainと現在のHEADを比較。ファイル参照とともにクリティカル、重要な問題、軽微な問題を提供してください。
メモリリークの接続プールに関するバグ修正をレビューしてください。内容: finallyブロックでのクリーンアップを追加。差分: HEAD~1とHEADを比較。リソース管理とエッジケースに焦点を当ててください。
リファクタリング前の現在の状態をレビューしてください。内容: データベース層の抽象化。計画: docs/plans/refactor-db.md。現在のアーキテクチャとテストカバレッジのベースライン評価を取得します。
Melhores Práticas
- サブエージェント主導の開発では、各タスク後にレビューして、問題が複合化しないように早期に発見する
- 意味のあるレビューができるように、常に比較用の計画や要件文書を提供する
- フィードバックに比例して対応する：クリティカル���即座に、重要なものは進行前に、軽微なものは後でメモ
Evitar
- コードがシンプルに見えるからレビューをスキップする - シンプルなコードでもバグやセキュリティ問題が発生しうる
- 修正されていない重要な問題を進めて、それが下流で更大的な問題に波及する可能性がある
- 反証や反例を提供せずに有効な技術的フィードバックを無視する