SVSim/SVSimServer
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

refactor(battle-node): retire hardcoded BattleSeed + ReadyIdxChangeSeed

Browse Source 
Both now derive per-battle from the master seed via BattleSeeds; only
animation/UI constants (ReadySpin, rank/battlePoint placeholders) remain in
BattleFrameDefaults.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
gamer147
2026-06-04 18:21:28 -04:00
parent 3f5d97cb2f
commit b229885259
1 changed files with 6 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
SVSim.BattleNode/Lifecycle/BattleFrameDefaults.cs
View File

@@ -4,24 +4,20 @@ namespace SVSim.BattleNode.Lifecycle;
/// Default frame constants templated from TK2 prod captures, shared by the /// Default frame constants templated from TK2 prod captures, shared by the
/// server-authored battle-frame builders. Every value here originated in a real prod /// server-authored battle-frame builders. Every value here originated in a real prod
/// frame in <c>data_dumps/captures/battle-traffic_tk2_regular.ndjson</c>; pulling them /// frame in <c>data_dumps/captures/battle-traffic_tk2_regular.ndjson</c>; pulling them
/// out of <see cref="ServerBattleFrames"/> makes the magic numerics navigable and gives /// out of <see cref="ServerBattleFrames"/> makes the magic numerics navigable. The shared effect
/// the seed a single source of truth instead of two duplicated literals. /// seed and the deck-shuffle/idxChangeSeed are now derived per-battle from a master seed (see
/// <see cref="BattleSeeds"/>) — only animation/UI constants remain here.
/// </summary> /// </summary>
internal static class BattleFrameDefaults internal static class BattleFrameDefaults
{ {
// Shared per the spec — selfInfo.seed and oppoInfo.seed always agree.
// From frame[2] (Matched).
public const long BattleSeed = 17_548_138L;
// From frame[5] (BattleStart). Hardcoded; see spec §Deferred plumbing — sourcing these // From frame[5] (BattleStart). Hardcoded; see spec §Deferred plumbing — sourcing these
// from real per-viewer state needs a TK2 rank/battle-point tracker. // from real per-viewer state needs a TK2 rank/battle-point tracker.
public const string PlayerRank = "10"; public const string PlayerRank = "10";
public const string PlayerBattlePoint = "6270"; public const string PlayerBattlePoint = "6270";
// From frame[8] (Ready). Provenance is "what prod sent"; the client // From frame[8] (Ready). Provenance is "what prod sent"; the client doesn't validate. This is
// doesn't validate, but echoing matches the capture protects against // an animation crank value (shared-RNG spin), NOT gameplay randomness — both clients crank it
// a regression on a future tightening. // identically and stay synced, so it stays a constant. See the spin-rng audit.
public const int ReadyIdxChangeSeed = 771_335_280;
public const int ReadySpin = 243; public const int ReadySpin = 243;
/// <summary> /// <summary>