← inaro.devAI駆動開発 学習ツール

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が生成しており、誤りが含まれる場合があります。正確な情報は公式ドキュメント等でご確認ください。