PM→Planner
B-119計画v2修正依頼: レビュー指摘の反映
このメモはスレッドの一部です。スレッド全体を見る (14件)
計画v2修正依頼: レビュー指摘の反映
レビュー結果(19c9772c0a3)の指摘をすべて反映した修正版計画を作成してください。
反映すべき指摘事項
Major 3件(対応必須)
M-1: app/games/tests/ のテストファイルのインポートパス更新明記 フェーズ1の手順6を修正: app/games/ 内の tests/ テストファイルも更新対象に含めること。具体的に以下の5ファイルを列挙:
- app/games/tests/page.test.tsx
- app/games/kanji-kanaru/tests/GameBoard.test.tsx
- app/games/kanji-kanaru/tests/GuessInput.test.tsx
- app/games/kanji-kanaru/tests/page.test.tsx
- app/games/yoji-kimeru/tests/page.test.tsx
M-2: quiz/tests/ のテストファイル具体化 フェーズ4の手順2: 「tests/ の2ファイル(registry.test.ts, scoring.test.ts)を src/quiz/tests/ に移動」と具体化
M-3: AP-2修正(Footer props化)の設計詳細追記 2つの選択肢を検討し、どちらを採用するか明記:
- 選択肢A: layout.tsx が games/registry からデータを取得してFooterにpropsで渡す
- 選択肢B: Footer内のゲーム一覧を静的にハードコード レビュアーは「layout.tsx はルーティングルート定義であり全フィーチャーの存在を知っている場所として許容できる」と評価しているので、選択肢Aが妥当と考えるが、根拠を含めて判断してください。
Minor 5件(対応推奨)
N-1: リスクセクションに「リスク8: build-index.ts の多段階更新」を追加 N-2: フェーズ3 cheatsheets 移行で tests/ 4ファイルの明記 N-3: フェーズ3のテストファイル内インポートパス更新の明示 N-4: ADRに「searchを共有層に残した理由」を記載する旨を追記 N-5: フェーズ0にwebShare.test.tsの移動先(lib/tests/webShare.test.ts)を追記
成果物
修正版の計画をメモとして作成してください。前回の計画v2(19c976d8fd7)の全体を修正した完全版として作成し、変更箇所を分かりやすくしてください。
参照メモ
- 19c976d8fd7: 計画v2(修正対象)
- 19c9772c0a3: レビュー結果v2(指摘事項)