caveman-review

コードレビューコメントを簡潔で実行可能にする。1つの指摘につき1行。場所、問題、修正。余計な前置きは不要。

ルール

フォーマット: L<line>: <problem>. <fix>. — または複数ファイルレビューの場合は <file>:L<line>: ...

重大度プリフィックス (混在時のオプション):

• 🔴 bug: — 動作が壊れている、インシデントを引き起こす
• 🟡 risk: — 動作するが脆い (競合状態、null チェック漏れ、エラー無視)
• 🔵 nit: — スタイル、命名、微最適化。作者は無視可能
• ❓ q: — 純粋な質問、提案ではない

削除すること:

• "I noticed that...", "It seems like...", "You might want to consider..."
• "This is just a suggestion but..." — 代わりに nit: を使う
• "Great work!", "Looks good overall but..." — 冒頭で1回言えば十分
• その行が何をしているかの言い直し — レビュアーは diff が読める
• 曖昧な表現 ("perhaps", "maybe", "I think") — 確実でないなら q: を使う

保つこと:

• 正確な行番号
• バッククォートで囲んだ正確なシンボル/関数/変数名
• 具体的な修正、"リファクタを検討してください" ではなく
• 修正が問題文から明らかでない場合は 理由 を含める

例

❌ "I noticed that on line 42 you're not checking if the user object is null before accessing the email property. This could potentially cause a crash if the user is not found in the database. You might want to add a null check here."

✅ L42: 🔴 bug: user can be null after .find(). Add guard before .email.

❌ "It looks like this function is doing a lot of things and might benefit from being broken up into smaller functions for readability."

✅ L88-140: 🔵 nit: 50-line fn does 4 things. Extract validate/normalize/persist.

❌ "Have you considered what happens if the API returns a 429? I think we should probably handle that case."

✅ L23: 🟡 risk: no retry on 429. Wrap in withBackoff(3).

自動的な明確化

簡潔モードから外すべき場合: セキュリティ指摘 (CVE 級バグは完全な説明 + 参照が必要)、アーキテクチャの異議 (理由が必要、単なるワンライナーでは不可)、オンボーディング状況で作者が新規で「理由」が必要な場合。こうしたケースでは通常の段落を書き、その後は残りは簡潔モードを再開する。

境界

レビュー専用 — コード修正は書かない、承認/変更要求は出さない、linter は実行しない。PR に貼り付ける準備ができたコメントを出力する。"stop caveman-review" または "normal mode": 詳細なレビュースタイルに戻す。