From 98fb3c5fcd6d21e30c6c1000e54e1b0ee0ea0226 Mon Sep 17 00:00:00 2001 From: gamer147 Date: Sun, 31 May 2026 11:41:57 -0400 Subject: [PATCH] fix(svc): default MaxBattleCount=7 with warn-log on empty reward catalog --- .../Services/ArenaTwoPickService.cs | 14 +++++++++++++- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/SVSim.EmulatedEntrypoint/Services/ArenaTwoPickService.cs b/SVSim.EmulatedEntrypoint/Services/ArenaTwoPickService.cs index 261fa24..abe1178 100644 --- a/SVSim.EmulatedEntrypoint/Services/ArenaTwoPickService.cs +++ b/SVSim.EmulatedEntrypoint/Services/ArenaTwoPickService.cs @@ -80,7 +80,19 @@ public class ArenaTwoPickService : IArenaTwoPickService } var aCfg = _config.Get(); - var maxWins = Math.Max(1, await _rewards.GetMaxWinCountAsync()); + var rawMaxWins = await _rewards.GetMaxWinCountAsync(); + int maxWins; + if (rawMaxWins == 0) + { + // Reward catalog not seeded — bootstrap hasn't run. Fall back to the spec's + // documented default (7) and log a warning so misconfigured deployments are visible. + Console.Error.WriteLine("[ArenaTwoPickService] ArenaTwoPickRewards catalog empty; defaulting MaxBattleCount=7. Run SVSim.Bootstrap to seed."); + maxWins = 7; + } + else + { + maxWins = rawMaxWins; + } var candidates = SampleCandidateClasses(aCfg.AllowedClassIds, _rng); var run = new ViewerArenaTwoPickRun