AIコードのレビュー観点
なぜ学ぶか: Phase 01〜06で学んだ個別の観点を、1つのチェックリストに束ねるのがこのトピックです。レビューは『気合い』ではなく『手順』。決まった観点で順に見れば、見落としが減り、AIにも同じ基準で自己点検させられます。読む力の、実戦での使い方です。
4つの問い:動くか・読めるか・安全か・シンプルか
この順で見るのが基本。動作 → 読みやすさ → 安全性 → 単純さ。優先度は状況次第ですが、『安全か』は妥協しない一線です。
動くか:正常時だけでなく失敗時も
成功時だけでなく、失敗・空・境界値で壊れないか。4状態・エラー処理の観点です。
読めるか:未来の自分が読者
命名が役割を表しているか、1つの関数が長すぎないか。今読めないコードは、直すときにもっと読めません。
安全か:最優先の一線
秘密の露出・RLS・入力の扱い・asの乱用。他が多少雑でも、ここだけは通してはいけません。
シンプルか:やりすぎを削る
不要なuseState/useEffect・過剰なuseMemo・要らない抽象化。AIは『やりすぎる』ことがあります。
チェックリスト
AIコード レビューチェックリスト □ 動くか ・成功・失敗・空・境界値で壊れないか ・エラーを握りつぶしていないか □ 読めるか ・命名は役割を表しているか ・巨大コンポーネント/関数になっていないか □ 安全か ← 最優先・妥協しない ・秘密がクライアントに露出していないか ・Supabase テーブルに RLS はあるか ・ユーザー入力を信頼して実行/描画していないか ・as any / ! で型を握りつぶしていないか □ シンプルか ・不要な state / useEffect / memo はないか ・過剰な抽象化・重複はないか
レビューは気合いではなく手順です。このリストをCLAUDE.mdに入れておけば、AIが生成時に自分でチェックし、あなたは最終確認に集中できます。
ステップ 1 / 2
※ カリキュラムの内容はAIが生成しており、誤りが含まれる場合があります。正確な情報は公式ドキュメント等でご確認ください。