B-147 タスク2: 四字キメル途中離脱バグ修正

背景

漢字カナールと同型のバグ。ゲーム進行中に status: "lost" プレースホルダーで保存しているため、途中離脱すると再読込時に「敗北」として復元され、入力不能の詰み状態になるバグを修正する。

実施計画

元の計画: メモ 19ca9b03b85（タスク2セクション） 修正版: メモ 19ca9b5162d（タスク2差分セクション）

上記2つのメモを読み、タスク2の修正を実施してください。

変更ファイル一覧

1. /mnt/data/yolo-web/src/games/yoji-kimeru/_lib/types.ts

• YojiGameHistory の status フィールドを "won" | "lost" → "won" | "lost" | "playing" に変更
• ファイル末尾に export const MAX_GUESSES = 6; を追加

2. /mnt/data/yolo-web/src/games/yoji-kimeru/_lib/storage.ts

• MAX_GUESSES を types.ts からインポート
• loadTodayGame 関数に旧データマイグレーション追加: status === "lost" かつ guessCount < MAX_GUESSES の場合、status を "playing" に補正して返す

3. /mnt/data/yolo-web/src/games/yoji-kimeru/_components/GameContainer.tsx

• L33の const MAX_GUESSES = 6; を削除し、types.ts からインポート
• L165-173: 進行中保存の status: "lost" を status: "playing" に変更
• L170の // Placeholder; overwritten on completion コメントを削除

4. /mnt/data/yolo-web/src/games/yoji-kimeru/_lib/__tests__/storage.test.ts

以下5つのテストケースを追加:

1. loadTodayGame が status: "playing" のデータをそのまま playing として返すこと
2. loadTodayGame が status: "lost" かつ guessCount < 6 のデータを playing に補正して返すこと（旧データマイグレーション）
3. loadTodayGame が status: "lost" かつ guessCount >= 6 のデータを lost のまま返すこと（本当の敗北）
4. loadTodayGame が status: "won" のデータを won のまま返すこと
5. saveTodayGame が status: "playing" で保存できること

注意事項

• 変更対象は src/games/yoji-kimeru/ 配下のファイルのみ。他のゲームのファイルは変更しないこと
• 既存テストが壊れないことを確認すること
• npm run lint && npm run typecheck が通ることを確認すること
• 作業完了後、変更内容と確認結果をメモで報告すること

B-147 タスク2: 四字キメル途中離脱バグ修正 完了報告

実施内容

計画メモ 19ca9b03b85（タスク2セクション）および修正版 19ca9b5162d の指示に従い、四字キメルの途中離脱バグを修正しました。

変更ファイル一覧（4ファイル）

1. src/games/yoji-kimeru/_lib/types.ts

• YojiGameHistory の status フィールドを "won" | "lost" から "won" | "lost" | "playing" に拡張
• ファイル末尾に export const MAX_GUESSES = 6; を追加（DRY原則に基づく一箇所定義）

2. src/games/yoji-kimeru/_lib/storage.ts

• MAX_GUESSES を types.ts からインポート（storage.ts に定数を重複定義しない）
• loadTodayGame 関数に旧データマイグレーションロジックを追加:
  • status === "lost" かつ guessCount < MAX_GUESSES の場合 → status を "playing" に補正して返す
  • それ以外のデータはそのまま返す（本当の敗北や勝利データに影響なし）
• JSDocコメントにマイグレーションの説明を追加

3. src/games/yoji-kimeru/_components/GameContainer.tsx

• L33の const MAX_GUESSES = 6; を削除し、types.ts からインポートに変更
• L165-173: 進行中保存の status: "lost" を status: "playing" に変更
• L170の // Placeholder; overwritten on completion コメントを削除

4. src/games/yoji-kimeru/_lib/__tests__/storage.test.ts

以下5つのテストケースを追加（計16テスト、元は11テスト）:

1. loadTodayGame が status: "playing" のデータをそのまま返すこと
2. loadTodayGame が旧 status: "lost" + guessCount < 6 を "playing" に補正すること
3. loadTodayGame が status: "lost" + guessCount >= 6 を "lost" のまま返すこと
4. loadTodayGame が status: "won" のデータを変更しないこと
5. saveTodayGame が status: "playing" で保存できること

検証結果

• lint（yoji-kimeru配下）: エラーなし
• typecheck: エラーなし（tsc --noEmit 通過）
• テスト: 全52テスト通過（5ファイル）
  • storage.test.ts: 16テスト通過（既存11 + 新規5）
  • 既存テストに影響なし

注意事項

• npm run lint を全体で実行するとイロドリ（irodori）の GameContainer.tsx で lint エラーが4件ありますが、これは別タスク（タスク3）の変更に起因するものであり、本タスクの変更対象外です。yoji-kimeru 配下の lint は問題ありません。