Replay: correct winner, per-bankruptcy auctions, revocations, fill terms (Closes #125) #134

Merged
robertismo merged 4 commits from 125-replay into master 2026-10-01 15:52:53 +00:00
Owner
  • replayOutcome writes the highest-equity standing bank (g.Winner, as endGame decided) instead of the last alive seat — verified against the console FINAL RANKING.
  • Auctions are a list: every bankruptcy in a finance phase gets its own record, including the no-bidder case (the last bank standing fails), which was previously absent from the replay entirely.
  • Revocations are recorded (seat + returned cash) via a replayRevocation hook in revokeOffers.
  • ReplayFill carries turns_left so accruals can be recomputed.
  • Replay version bumps to 0.6.0 (schema change).

Full suite + vet clean.

- replayOutcome writes the highest-equity standing bank (g.Winner, as endGame decided) instead of the last alive seat — verified against the console FINAL RANKING. - Auctions are a list: every bankruptcy in a finance phase gets its own record, including the no-bidder case (the last bank standing fails), which was previously absent from the replay entirely. - Revocations are recorded (seat + returned cash) via a replayRevocation hook in revokeOffers. - ReplayFill carries turns_left so accruals can be recomputed. - Replay version bumps to 0.6.0 (schema change). Full suite + vet clean.
- replayOutcome now writes the highest-equity standing bank (g.Winner,
  as endGame decided) instead of the last alive seat.
- Auctions are a list: every bankruptcy in a finance phase gets its own
  record, including the no-bidder case (the last bank standing fails),
  which was previously absent from the replay entirely.
- Revocations are recorded (seat + returned cash) via a
  replayRevocation hook in revokeOffers.
- ReplayFill carries turns_left so accruals can be recomputed.
- The replay version bumps to 0.6.0 (schema change).
Merge remote-tracking branch 'origin/master' into 125-replay
All checks were successful
build / build (push) Successful in 31s
025f8f2a73
# Conflicts:
#	game.go
First-time contributor

@Winnie merge conflict

@Winnie merge conflict
Merge remote-tracking branch 'origin/master' into 125-replay
Some checks failed
build / build (push) Has been cancelled
cc29a55122
# Conflicts:
#	game.go
Merge branch 'master' into 125-replay
All checks were successful
build / build (push) Successful in 31s
74203b4e0c
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
Winnie/bankwars!134
No description provided.