frontend-review-hygiene

フロントエンドレビュー — 品質管理

フロントエンドプロジェクトのコード品質の基本項目である、型、Lint、不要なコード、重複を評価します。これらは毎週改善目標を引き上げていく KPI です。

手順

1. 以下を並行して実行します。
   • scripts/audit-typescript.sh --repo <client-repo>
   • scripts/audit-lint.sh --repo <client-repo>
   • scripts/audit-similarity.sh --repo <client-repo>
2. 各 raw/*.json を確認します。
3. <client-repo>/.frontend-review/kpi/baseline.json が存在する場合は、前回の実行結果と比較します。

出力

<client-repo>/.frontend-review/report/latest/md/hygiene-summary.md に以下を記載します。

• KPI テーブル：any の数、@ts-ignore の数、Lint のエラー／警告、knip で検出された未使用のファイル／エクスポート／依存関係、重複コードのペア数
• ベースラインとの差分（悪化は太字で示し、改善には ✅ を付けます）
• 影響度別の修正バッチ — 1 つの PR でまとめて修正するとよい項目
• 含めない項目：ファイルごとの指摘（これらは raw JSON に残します）。このレポートはコードレビューではなく、意思決定のためのものです。

クライアントにベースラインがまだない場合は、現在の JSON を <client-repo>/.frontend-review/kpi/baseline.json にコピーして作成し、その旨をレポートに記載します。

ツールチェーンにおける役割の分担

Lint／フォーマットのスタックを評価する際は、各ツールの役割が明確に分かれているか確認します。

ツール	想定される役割
TypeScript	型の正しさのみ
ESLint / Biome / oxlint	AST レベルのバグパターン、インポートルール
Prettier / Biome / oxfmt	フォーマットのみ — Lint と役割を重複させない
ast-grep / カスタムルール	プロジェクト固有の構造ルール
vitest / jest	動作

役割が重複していると（例：ESLint でフォーマットも処理する）、競合が起き、CI が遅くなります。分離を指摘し、推奨してください。

自動生成ファイル（ロックファイル、生成されたスキーマ、ツールの生成物）は、フォーマットの実行対象から除外する必要があります。生成ファイルを繰り返しフォーマットして元に戻している場合は、.prettierignore／.biomeignore／同等のファイルから除外されていない可能性があります。

境界

• 依存関係の鮮度や CVE は評価しないでください — これは frontend-review-deps の担当です。
• コードレベルの修正を提案しないでください。これは 5 つの観点別スキルの担当です。
• クライアントリポジトリのソースファイルを変更しないでください。
• audit-security.sh を実行しないでください — これはセキュリティスキルの担当です。

参考資料

• チェックリスト：03-typescript.md、04-lint-format.md、05-deadcode-knip.md、06-similarity.md、13-kpi-tracking.md
• 関連：frontend-review-deps（依存関係の健全性）、frontend-review-security（CVE + 認証 + 環境変数）
• フェーズ：week-1-ci-baseline.md、week-4-ai-review.md