B-119 計画v2.2 全体再レビュー結果

総合評価: 承認

計画v2.2（v2.1 + v2.2差分の統合体）は、ownerの根本課題であったsrc/content/問題を含む全ての過去レビュー指摘（v2: Major 3件 + Minor 5件、v2.1: Critical 1件 + Minor 2件）が適切に反映されており、計画全体として高品質であり実行可能と評価する。Critical/Major指摘はなく、承認とする。

1. src/content/ 問題の完全解決（最重要チェック）: OK

1-1. src/content/ ディレクトリの完全廃止

v2.2差分により以下が全て計画されていることを確認した:

完成の定義に項目14「src/content/ ディレクトリが存在しないこと（Astroの Content Collections と誤認されるリスクを排除）」が追加されている -- OK
セクション2-3のディレクトリツリーからcontent/ブロックが削除され、blog/content/が追加されている -- OK
フェーズ6手順5でblog.tsのBLOG_DIRパスを'src/blog/content'に変更する計画がある -- OK
フェーズ6手順6-7でMarkdownファイルのgit mv移動とsrc/content/の削除が計画されている -- OK
フェーズ8に「src/content/ ディレクトリが完全に消滅していることを確認する」ステップが追加されている -- OK

1-2. 実コード照合によるパス変更の影響範囲確認

実コードを確認した結果:

blog.ts（L10）: const BLOG_DIR = path.join(process.cwd(), "src/content/blog") -- この1行のみがsrc/content/blogへのパス参照。計画通り、この1行の変更で完結する
他のファイルは全てblog.tsの関数経由でブログコンテンツにアクセスしており（getAllBlogPosts, getBlogPostBySlug等を経由）、パス変更の波及はblog.ts 1ファイルに閉じる -- OK
src/content/配下にはblog/のみが存在する（ls -la確認済み）-- OK、移動後にsrc/content/は空になるため削除可能
Markdownファイルは35本 -- v2.2差分の「約35ファイル」と一致

1-3. 配置ルール表の更新

旧「コンテンツ | src/content/ | 非コード資産」行が「コンテンツ | src/{feature}/content/ | フィーチャー固有のMarkdownコンテンツ」に修正されている -- OK
判断基準に項目7「Markdownコンテンツを持つフィーチャーは src/{feature-name}/content/ に配置する。src/content/ ディレクトリは使用しない」が追加されている -- OK

1-4. 将来のスケーラビリティ対応のカウント修正

フィーチャーディレクトリ数が8から7に修正され、「+ content」が削除されている -- OK

結論: ownerの根本課題（AIエージェントのAstro誤認）は本計画で完全に解決される。

2. 過去レビュー指摘の反映確認: 全件OK

2-1. v2レビュー指摘（Major 3件 + Minor 5件）

指摘ID	反映状況	実コード照合結果
M-1 (app/games/tests/ テストパス)	v2.1で反映済み	フェーズ1手順6に5ファイルを具体的に列挙。実際のファイル存在を確認: page.test.tsx(games), GameBoard.test.tsx, GuessInput.test.tsx, page.test.tsx(kanji-kanaru), page.test.tsx(yoji-kimeru) -- OK
M-2 (quiz/tests/ 具体化)	v2.1で反映済み	フェーズ4手順2に「2ファイル（registry.test.ts, scoring.test.ts）」と明記。実ファイル存在を確認 -- OK
M-3 (AP-2 Footer設計詳細)	v2.1で反映済み	セクション3 AP-2に選択肢A/Bの比較と採用根拠を詳細記載。実コードでFooter.tsxがallGameMetas, getGamePathをインポートしていることを確認。layout.tsx経由のprops方式は妥当 -- OK
N-1 (build-index.ts多段階更新)	v2.1で反映済み	セクション5にリスク8として追加。実コードでbuild-index.tsが7フィーチャーのregistryをインポートしていることを確認 -- OK
N-2 (cheatsheets tests/ 4ファイル)	v2.1で反映済み	フェーズ3手順1に4ファイル名を明記。実ファイル存在を確認: CheatsheetCard.test.tsx, CheatsheetLayout.test.tsx, CodeBlock.test.tsx, TableOfContents.test.tsx -- OK
N-3 (テストファイル内import)	v2.1で反映済み	フェーズ3手順2に「テストファイル内のインポートパスも含む」と追記 -- OK
N-4 (ADRにsearch共有層理由)	v2.1で反映済み	フェーズ8手順6のADR作成項目に具体的根拠を記載 -- OK
N-5 (webShare.test.ts移動)	v2.1で反映済み	フェーズ0手順1に移動先とパス更新を追記。実ファイル(src/lib/games/shared/tests/webShare.test.ts)の存在を確認 -- OK

2-2. v2.1再レビュー指摘（Critical 1件 + Minor 2件）

指摘ID	反映状況	確認結果
C-1 (src/content/ Astro誤認問題)	v2.2で反映済み	上記セクション1で詳細確認。完成の定義、ディレクトリツリー、配置ルール表、フェーズ6手順、フェーズ8検証、リスク9の全てに反映されている -- OK
N-1 (配置ルール表更新 + Markdown配置ルール)	v2.2で反映済み	配置ルール表修正 + 判断基準項目7追加 -- OK
N-2 (完成の定義にAstro誤認除去)	v2.2で反映済み	項目14として追加 -- OK

結論: 過去の全レビュー指摘（17件）が適切に反映されている。

3. 計画全体の整合性チェック: OK

3-1. ディレクトリツリーと配置ルール表の整合性

v2.2統合後の最終ディレクトリ構造:

src/blog/content/ にMarkdownファイルが配置される
src/content/ は存在しない
配置ルール表の「コンテンツ」行は src/{feature}/content/ パターンを指定

これらは相互に矛盾がなく整合している -- OK

3-2. フェーズ間の依存関係

フェーズ0 -> フェーズ1: webShare.tsのlib/への移動が先行する必要がある（フェーズ1でgamesのshared/を移動する際にwebShare.tsが既に移動済みであることを前提としている）-- OK
フェーズ6のblog移行: v2.2でMarkdownファイルの移動とBLOG_DIRパス変更が追加されたが、これは同一フェーズ内で完結する -- OK
フェーズ7のmemos移行: blog移行後にcross-links.tsのmemos部分を更新する（フェーズ6でblog部分を更新済み）-- OK
フェーズ8の最終検証: src/content/の不在確認が追加されており、フェーズ6でのMarkdown移動の完了を前提としている -- OK

3-3. 影響ファイル数の見積もり

v2.2差分でフェーズ6の影響ファイル数に「blog.ts パス変更1行 + Markdownファイル移動（約35ファイル）」が追加されている。実際のMarkdownファイル数は35本であり、見積もりと一致する -- OK

3-4. 完成の定義と最終検証の整合性

完成の定義14項目:

項目1-13: v2.1で定義済み。各項目はフェーズ8の最終検証ステップと対応している
項目14（src/content/不在）: v2.2で追加。フェーズ8の消滅確認ステップと対応している
全項目が検証可能かつ客観的に判定できる基準になっている -- OK

4. v2.2修正により新たに生じた問題の有無: なし

4-1. リスク9（blog Markdownファイル移動時のパス不整合）

v2.2で追加されたリスク9は適切にリスクを認識しており、対策（npm run buildでのブログページ生成確認 + src/content/の空確認）も妥当 -- OK

4-2. フェーズ6手順番号のシフト

v2.2差分で旧手順5,6を修正し、新手順7を追加した上で以降の番号を+1シフトしている。これは論理的に正しい -- OK

4-3. blog/content/ の_接頭辞なしの命名

v2.2ではblog/ 配下のサブディレクトリとして _components/, _lib/, content/ を配置する計画になっている。_components/ と _lib/ はNext.jsのプライベートフォルダ規約に従い_接頭辞を付けているが、content/ には_接頭辞がない。これについて:

content/ はMarkdownファイルの格納場所であり、Next.jsのルーティングには影響しない（blog/ はsrc/app/配下ではなくsrc/直下のため）
_接頭辞はNext.jsのapp/ディレクトリ内でのルーティング除外の慣習であり、src/直下のフィーチャーディレクトリでは必須ではない
content/ という名前は格納内容を明確に示しており、可読性が高い
問題なし -- OK

5. 設計アンチパターン修正方針の再検証: OK

5-1. AP-1（BlogListView -> page.module.css レイヤー逆転）

実コードを確認: BlogListView.tsx（L6）が import styles from "@/app/blog/page.module.css" でルーティング層のCSSに依存している。page.module.cssの内容（83行、10クラス: container, header, title, description, filters, filterPill, grid, empty + メディアクエリ）はほぼ全てBlogListView用であり、計画通りBlogListView.module.cssとして抽出するのは妥当 -- OK

5-2. AP-2（Footer -> games/registry 共有層逆転）

実コードを確認: Footer.tsx（L2）が import { allGameMetas, getGamePath } from "@/lib/games/registry" でgamesフィーチャーに依存している。選択肢A（layout.tsx経由のprops）の採用は適切 -- OK

5-3. AP-3（webShare.ts フィーチャー間依存）

実コードを確認: quiz/ShareButtons.tsx（L4）と games/shared/GameShareButtons.tsx（L8）がともにwebShare.tsを利用。lib/への移動は妥当 -- OK

5-4. AP-4（dictionary/index.ts 未使用）

実コードを確認: @/lib/dictionary でのインポートは0件（grep確認済み）。index.tsの削除は安全 -- OK

5-5. AP-5（seo.ts 型依存許容）

実コードを確認: seo.tsがToolMeta, CheatsheetMeta, QuizMetaの3つをimport typeで参照。ランタイム影響なし。許容判断は妥当 -- OK

6. constitution.mdとの整合性: OK

ルール1（法令遵守）: ディレクトリ構造のリファクタリングであり法令に関する問題なし
ルール2（有益・無害なコンテンツ）: コード品質の改善であり、訪問者への悪影響なし
ルール3（AI運営の告知）: 変更なし
ルール4（品質優先）: コード構造の品質を改善する施策であり、ルール4に積極的に合致。AIエージェントの作業精度向上を通じてコンテンツ品質の改善サイクルを加速する
ルール5（創造的な試み）: フィーチャーコロケーションという業界のベストプラクティスを適用する試み

7. 新規指摘事項

[Note] N-1: blog/tests/ のSeriesNav.test.tsxの移動

現在 src/components/blog/tests/SeriesNav.test.tsx が存在する。フェーズ6手順3で「src/components/blog/ を src/blog/_components/ に移動」とあり、tests/ もディレクトリごと移動されるため問題はない。ただし、v2.1でgames/quiz/cheatsheetsのテストファイルを個別に列挙した経緯に鑑み、blog/tests/SeriesNav.test.tsx も手順3に明記すると一貫性が高まる。対応は任意。

[Note] N-2: page.module.css の残余内容

AP-1修正でBlogListView.module.cssを抽出した後、app/blog/page.module.cssに残すべきスタイルがあるかどうかについて。実コードを確認したところ、page.module.cssの全10クラス（container, header, title, description, filters, filterPill, grid, empty + メディアクエリ2個）は全てBlogListView.tsx内で使用されている。app/blog/page.tsxやapp/blog/category/等のページコンポーネントはBlogListViewコンポーネントを呼び出すだけであり、page.module.cssを直接参照していない。したがってBlogListView.module.cssへの移動後、page.module.cssは空になり削除可能である。フェーズ6の実装時にこの点を確認し、page.module.cssが空になれば削除すべき。対応は任意だが、ビルダーへのヒントとして有用。

8. 結論

承認。

Critical/Major指摘はなし。Note 2件は対応任意。

計画v2.2は以下の全ての観点で十分な品質を満たしている:

ownerの根本課題の解決: src/content/ディレクトリの完全廃止により、AIエージェントのAstro誤認問題を解決する計画が明確に含まれている
過去レビュー指摘の完全反映: v1-v2.1の全17件の指摘が適切に反映されている
計画の内部整合性: ディレクトリツリー、配置ルール表、フェーズ手順、完成の定義が相互に矛盾なく整合している
実コードとの照合: 影響ファイル数、パス参照、依存関係が全て実コードと一致している
リスク対策: 9つのリスクが網羅的に識別され、適切な対策が講じられている
constitution.md準拠: ルール4（品質優先）に積極的に合致している

計画を確定し、builderへの作業依頼に進めてよい。

レビュー結果v3: B-119計画v2.2全体再レビュー