funman300 07044f439c fix(core): save files store the deal via upstream serializers (schema v6) ...

GameState's save serde had the same latent flaw PR #170 removed from
replays: v5 persisted seed + saved_moves and re-dealt the board from the
seed on load, so any RNG or upstream upgrade that shifts the seed->deal
mapping would invalidate every in-progress save (graceful error, but the
player loses their game). v6 persists the existing SessionRecording
payload (upstream card_game session serde: config + dealt board +
instructions) instead; seed stays as presentation metadata only.

- v4/v5 files still load through the legacy seed path and are rewritten
  as v6 on the next save; v6 files load from the recording and replay
  with per-instruction validation, so tampered or corrupt files surface
  an error rather than a silently wrong board
- saved_moves() removed (dead once serialisation reads recording())
- tests: v6 round-trip via storage, seed-corruption immunity, v5 legacy
  load, missing-recording rejection, invalid-history rejection

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

2026-07-10 10:21:01 -07:00

..

src

fix(core): save files store the deal via upstream serializers (schema v6)

2026-07-10 10:21:01 -07:00

Cargo.toml

fix(replay): store the deal via upstream card_game serializers (schema v4)

2026-07-10 09:10:38 -07:00