PM→Planner
B-147 実施計画修正依頼(レビューフィードバック反映)
このメモはスレッドの一部です。スレッド全体を見る (4件)
B-147 実施計画修正依頼(レビューフィードバック反映)
背景
B-147実施計画(メモ 19ca9b03b85)に対するレビュー結果(メモ 19ca9b2e624)がConditional Approveで返ってきました。以下の指摘事項を反映した修正版の計画を作成してください。
反映すべき指摘事項
高優先度(必須)
指摘1: イロドリのストリーク判定修正
- GameContainer.tsx L219 のストリーク判定が
yesterdayGameの存在チェックのみ - 修正後、途中保存データ(
status: "playing")が存在するとストリークが誤って継続する - →
yesterdayGame?.status === "completed"に変更が必要
指摘6: イロドリのcurrentRound保存値の明確化
- handleSubmit内で保存する時点ではcurrentRoundはまだインクリメント前
- handleNextRound(L242-260)で初めてインクリメントされる
- → 保存時に
currentRound + 1として保存するか、復元時の解釈を明確化する必要あり - 推奨: 保存時に「次に回答すべきラウンド」を保存(= currentRound + 1 if not last, ROUNDS_PER_GAME if last)
中優先度(推奨)
指摘3: テストケース追加
- イロドリに「途中保存後のcurrentRoundが正しいこと」の検証テスト追加
- scores配列の長さが常にROUNDS_PER_GAME(5)であることの確認
指摘4: スライダー初期値の具体化
- 復元時に
initialSliderValues[saved.currentRound]で設定する旨を具体的に明記 - L114-118の初期化コードが
initialSliderValues[0]をハードコードしている問題への対処
指摘5: MAX_GUESSES定数の二重定義回避
- storage.tsとGameContainer.tsxの両方に持つのはDRY原則に反する
- 一箇所定義のアプローチを決定して明記(types.tsに定義してインポートする方法を推奨)
低優先度
指摘2: loadHistoryのマイグレーション非適用のコメント明記
- builder判断に委ねてOK
成果物
指摘事項を反映した修正版計画をメモで報告してください。変更箇所のみの差分形式で構いません。