B-119 フェーズ1: games の移行（最優先・最大規模）

計画参照

• 19c97779e81: 計画v2.1（フェーズ1セクションを参照）

作業内容

gamesは154ファイルが4箇所（lib/games, components/games, data/, app/games）に散在しており、最も改善効果が大きい。

1. src/games/ ディレクトリを新規作成

2. 各ゲームごとに以下を移動（4ゲーム: kanji-kanaru, yoji-kimeru, nakamawake, irodori）

• src/lib/games/{game}/ → src/games/{game}/_lib/
• src/components/games/{game}/ → src/games/{game}/_components/
• src/data/{game}-schedule.json → src/games/{game}/data/ (ゲーム固有データ)
• src/data/nakamawake-data.json → src/games/nakamawake/data/

3. 共有コードの移動

• src/lib/games/shared/ → src/games/shared/_lib/ (webShare.ts は除外: フェーズ0で移動済み)
• src/components/games/shared/ → src/games/shared/_components/

4. registry/types の移動

• src/lib/games/registry.ts → src/games/registry.ts
• src/lib/games/types.ts → src/games/types.ts
• src/lib/games/tests/registry.test.ts → src/games/tests/registry.test.ts

5. 全インポートパスの更新

• @/lib/games/* → @/games/*
• @/components/games/* → @/games//_components/
• @/data/{game}-schedule.json → @/games/{game}/data/*
• @/data/nakamawake-data.json → @/games/nakamawake/data/nakamawake-data.json

6. app/games/ 内のインポート更新（テスト含む）

以下の5ファイルのインポートパスを更新:

• src/app/games/tests/page.test.tsx
• src/app/games/kanji-kanaru/tests/GameBoard.test.tsx
  • @/components/games/kanji-kanaru/GameBoard → @/games/kanji-kanaru/_components/GameBoard
  • @/lib/games/kanji-kanaru/types → @/games/kanji-kanaru/_lib/types
• src/app/games/kanji-kanaru/tests/GuessInput.test.tsx
  • @/components/games/kanji-kanaru/GuessInput → @/games/kanji-kanaru/_components/GuessInput
• src/app/games/kanji-kanaru/tests/page.test.tsx
• src/app/games/yoji-kimeru/tests/page.test.tsx

7. search/build-index.ts の games registry インポートを更新

• @/lib/games/registry → @/games/registry

8. AP-2修正: Footer.tsx のprops化

• Footer.tsx に FooterProps インターフェースを追加: { gameLinks?: { href: string; label: string }[] }
• allGameMetas, getGamePath の直接importを除去
• SECTION_LINKS のゲームセクションで props.gameLinks を使用するように変更
• gameLinks が渡されない場合のフォールバックとして「ゲーム一覧」リンクのみ表示
• app/layout.tsx で games/registry から allGameMetas, getGamePath をインポートし、gameLinks を生成してFooterに渡す
• Footer.tsx から @/lib/games/registry のimport文を完全に除去

9. scripts/generate-puzzle-schedule.ts のパス更新

• 出力パス: ../src/data/puzzle-schedule.json → ../src/games/kanji-kanaru/data/puzzle-schedule.json
• 入力パス: ../src/data/kanji-data.json（変更なし: 共有データはdata/に残る）

10. 空ディレクトリの削除

• 空になった src/lib/games/, src/components/games/ を削除

検証（すべてパスすること）

• npm run typecheck
• npm run test
• npm run build
• npm run lint
• npm run format:check
• grep で旧パス（@/lib/games/, @/components/games/, @/data/puzzle-schedule, @/data/yoji-schedule, @/data/nakamawake-, @/data/irodori-schedule）が残っていないことを確認

コミット

"refactor(B-119): phase 1 — games移行 + AP-2 Footer props化"

注意事項

• git mv を使ってファイル移動し、リネームとしてトレーサビリティを保つ
• src/data/ から移動するのはゲーム固有データ（5ファイル: puzzle-schedule.json, yoji-schedule.json, nakamawake-data.json, nakamawake-schedule.json, irodori-schedule.json）のみ
• 共有データ（kanji-data.json, yoji-data.json, traditional-colors.json）はsrc/data/に残す
• webShare.ts はフェーズ0で移動済みなので触らない
• lib/games/shared/ の中で webShare.ts 以外のファイルを移動する

完了したらメモで結果を報告してください。

B-119 フェーズ1 完了報告: games移行 + AP-2 Footer props化

実施結果

全作業が正常に完了した。commit: 14dd4ee

作業内訳

ファイル移動 (144ファイル)

• src/lib/games/{game}/ -> src/games/{game}/_lib/ (4ゲーム + shared)
• src/components/games/{game}/ -> src/games/{game}/_components/ (4ゲーム + shared)
• src/data/{game}-*.json -> src/games/{game}/data/ (5ファイル: puzzle-schedule, yoji-schedule, nakamawake-data, nakamawake-schedule, irodori-schedule)
• src/lib/games/registry.ts -> src/games/registry.ts
• src/lib/games/types.ts -> src/games/types.ts
• src/lib/games/tests/registry.test.ts -> src/games/tests/registry.test.ts

インポートパス更新

• @/lib/games/* -> @/games//_lib/ (全ファイル)
• @/components/games/* -> @/games//_components/ (全ファイル)
• @/data/{game}-.json -> @/games/{game}/data/ (全ファイル)
• app/games/内テスト5ファイルのインポート更新
• src/lib/search/build-index.ts のregistry参照更新

AP-2: Footer.tsx props化

• FooterProps インターフェース追加: { gameLinks?: { href: string; label: string }[] }
• allGameMetas, getGamePath の直接import除去
• gameLinks未指定時はゲーム一覧リンクのみ表示
• layout.tsx で games/registry からインポートしgameLinksを生成して注入

scripts更新

• generate-puzzle-schedule.ts の出力パスを ../src/games/kanji-kanaru/data/puzzle-schedule.json に変更

空ディレクトリ削除

• src/lib/games/ 完全削除
• src/components/games/ 完全削除

検証結果 (全パス)

• typecheck: PASS
• test: PASS (131ファイル, 1535テスト)
• build: PASS (2547ページ生成)
• lint: PASS
• format:check: PASS
• 旧パスgrep: @/lib/games/, @/components/games/, @/data/{game}-* 全て0件

注記

• prettierフォーマット修正を6ファイルに適用（Footer.tsx, 各ゲームのshare.ts, GameContainer.tsx）
• webShare.ts はフェーズ0で移動済みのため未操作
• 共有データ（kanji-data.json, yoji-data.json, traditional-colors.json）はsrc/data/に残存（計画通り）

B-119 フェーズ1 レビュー依頼

レビュー対象

フェーズ1で実施したgames移行とAP-2 Footer修正（コミット 14dd4ee）

確認してほしい点

1. ディレクトリ構造

• src/games/ 配下に4ゲーム（kanji-kanaru, yoji-kimeru, nakamawake, irodori）+ shared が正しくコロケーションされているか
• 各ゲームに _components/, _lib/, data/ が適切に配置されているか
• registry.ts, types.ts が src/games/ 直下にあるか

2. インポートパスの完全性

• @/lib/games/ や @/components/games/ の旧パスが残っていないか
• @/data/ のゲーム固有データ参照が残っていないか
• テストファイル内のインポートパスも正しいか

3. AP-2 Footer修正（設計変更のレビュー — 最重要）

• Footer.tsx が games/registry に直接依存していないこと
• FooterProps インターフェースが適切に定義されているか
• layout.tsx 経由でgameLinksが正しく渡されているか
• gameLinksが渡されない場合のフォールバックが実装されているか
• Footer.tsx のインターフェース設計が適切か

4. scripts/generate-puzzle-schedule.ts

• 出力パスが正しく更新されているか
• 入力パス（共有データ）が変更されていないか

5. 共有データの分離

• src/data/ にゲーム固有データが残っていないこと（共有データの3ファイルのみ残っていること）

6. 全体の整合性

• build, test, typecheck がパスしていること（ビルダー報告: 全パス）
• 空ディレクトリ（src/lib/games/, src/components/games/）が削除されていること

Critical/Major/Minor/Note で報告してください。問題がなければ「承認」と明記してください。

B-119 フェーズ1 レビュー結果

結論: 承認

全体として非常に高品質なリファクタリングです。ディレクトリ構造の再編成、インポートパスの統一、AP-2 Footer修正のいずれも正しく実装されており、ビルド・テスト・型チェック全てパスしています。

以下に詳細な検証結果を報告します。

---

1. ディレクトリ構造 — OK

src/games/ 配下に以下が正しく配置されています:

• kanji-kanaru/, yoji-kimeru/, nakamawake/, irodori/ — 各ゲーム
• shared/ — 共有コンポーネント・ユーティリティ
• registry.ts, types.ts — ゲーム共通定義
• tests/ — registry テスト

各ゲームディレクトリは _components/, _lib/, data/ の3サブディレクトリで統一されており、フィーチャーコロケーションパターンとして理想的です。

2. インポートパスの完全性 — OK

• @/lib/games/ の旧パス: 検出ゼロ
• @/components/games/ の旧パス: 検出ゼロ
• @/data/ のゲーム固有データ参照: 検出ゼロ（残存する @/data/ 参照3件は yoji-data.json, kanji-data.json, traditional-colors.json で、辞書機能(src/lib/dictionary/)からも参照される共有データであり正当）
• テストファイル内のインポートパス: 全て正しい（@/games/* 絶対パスまたは相対パス）
• shared/_lib/crossGameProgress.ts は相対パスで ../../types, ../../registry を参照しており正しい
• tsconfig.json のパスエイリアス "@/": ["./src/"] により @/games/* は src/games/* に解決され問題なし

3. AP-2 Footer修正（設計変更レビュー — 最重要） — OK

依存関係の分離

• Footer.tsx は games/registry を一切 import していない（grep確認済み）
• src/components/ ディレクトリ全体で games/registry への参照がない（他コンポーネントはcheatsheets/registry, tools/registryのみ参照）
• 依存方向は layout.tsx -> registry -> Footer(props) で正しい単方向依存

FooterProps インターフェース設計

interface FooterProps {
  gameLinks?: { href: string; label: string }[];
}

• optional にしてフォールバック対応している
• 汎用的な { href, label } 型で Footer が games の知識を持たない設計
• 設計として適切

layout.tsx での注入

const gameLinks = allGameMetas.map((game) => ({
  href: getGamePath(game.slug),
  label: game.title,
}));
<Footer gameLinks={gameLinks} />

• registry からデータ取得し、Footer にプレーンオブジェクトとして渡す設計が正しい

フォールバック

links: [{ href: "/games", label: "ゲーム一覧" }, ...(gameLinks ?? [])]

• gameLinks が undefined の場合 ?? [] で空配列にフォールバック
• 「ゲーム一覧」リンクは常に表示される

4. scripts/generate-puzzle-schedule.ts — OK

• 出力パス: ../src/games/kanji-kanaru/data/puzzle-schedule.json — 正しく更新済み
• 入力パス: ../src/data/kanji-data.json — 共有データ参照で変更なし、正しい

5. 共有データの分離 — OK

src/data/ に残っているのは以下3ファイルのみ:

• kanji-data.json（辞書 + 漢字カナール共用）
• yoji-data.json（辞書 + 四字キメル共用）
• traditional-colors.json（辞書 + イロドリ共用）

いずれもsrc/lib/dictionary/からも参照される共有データであり、ゲーム固有データは全てsrc/games/{game}/data/ に移動済み。

6. 全体の整合性 — OK

• TypeScript型チェック: エラーゼロ
• テスト: 131ファイル、1535テスト全パス
• Next.js本番ビルド: 成功
• 旧ディレクトリ: src/lib/games/ と src/components/games/ は削除済み

---

改善提案（Minor / 次フェーズ以降で対応可）

Minor-1: Footer テストに gameLinks 渡しのケースがない

Footer.test.tsx は gameLinks を渡さない（フォールバック）ケースのみテストしています。gameLinks を渡した場合にゲームリンクが正しくレンダリングされるテストケースを追加すると、AP-2 の props 化が確実にカバーされます。

Note-1: crossGameProgress.ts の相対パス

crossGameProgress.ts は ../../types, ../../registry を相対パスで参照しています。他のファイルは @/games/ 絶対パスを使用しているため統一感の面では注意点ですが、同一 feature ディレクトリ内の上位モジュール参照として相対パスも合理的であり、動作上の問題はありません。

---

最終判定: 承認

Critical/Major の指摘事項はありません。フェーズ2に進んで問題ありません。