volleyballsimtypes 0.0.483 → 0.0.486

package/dist/cjs/src/service/competition/season.d.ts

@@ -10,12 +10,11 @@ export declare class Season {
     readonly iteration: Iteration;
     readonly divisionId: string;
     readonly standings: Standing[];
-    private _status;
-    private _champion?;
+    private readonly _status;
+    private readonly _champion?;
     static create(input: unknown): Season;
     private constructor();
     calculateStandings(): Standing[];
-    updateStandings(): void;
     get champion(): Team | undefined;
     get status(): Status;
     static createSeason(iteration: Iteration, teams: Team[], divisionId: string): Season;

package/dist/cjs/src/service/competition/season.js

@@ -40,17 +40,6 @@ class Season {
             return standing;
         });
     }
-    updateStandings() {
-        if (this.matches == null || this.matches.length < 1)
-            throw new Error('NO_MATCHES');
-        const updated = this.calculateStandings();
-        this.standings.splice(0, this.standings.length, ...updated);
-        const isOver = this.matches.every((match) => match.isOver());
-        if (isOver) {
-            this._status = data_1.StatusEnum.COMPLETE;
-            this._champion = this.teams.find((team) => team.id === this.standings[0].teamId);
-        }
-    }
     get champion() {
         if (this._champion != null)
             return this._champion;

package/dist/cjs/src/service/event/block.d.ts

@@ -21,5 +21,4 @@ export declare class Block extends InPlayEvent<BlockFailure, BlockType> {
     readonly blockers: string[];
     static create(input: unknown): Block;
     private constructor();
-    toString(): string;
 }

package/dist/cjs/src/service/event/block.js

@@ -40,11 +40,5 @@ class Block extends in_play_event_1.InPlayEvent {
         this.failure = failure;
         this.blockers = blockers;
     }
-    toString() {
-        const failure = `"failure":${this.failure}`;
-        const type = `"type":${this.type}`;
-        const blockers = `"blockers":[${this.blockers.map(b => `"${b}"`).join(',')}]`;
-        return `{${this._toString()},${failure},${type},${blockers}}`;
-    }
 }
 exports.Block = Block;

package/dist/cjs/src/service/event/free-ball.d.ts

@@ -13,5 +13,4 @@ export declare class FreeBall extends InPlayEvent<FreeBallFailure, FreeBallType>
     readonly type: FreeBallType;
     static create(input: unknown): FreeBall;
     private constructor();
-    toString(): string;
 }

package/dist/cjs/src/service/event/free-ball.js

@@ -38,10 +38,5 @@ class FreeBall extends in_play_event_1.InPlayEvent {
         this.type = type;
         this.failure = failure;
     }
-    toString() {
-        const failure = `"failure":${this.failure}`;
-        const type = `"type":${this.type}`;
-        return `{${this._toString()},${failure},${type}}`;
-    }
 }
 exports.FreeBall = FreeBall;

package/dist/cjs/src/service/event/in-play-event.d.ts

@@ -20,5 +20,4 @@ export declare abstract class InPlayEvent<F, T> extends RallyEvent {
     readonly activeTraits: Trait[];
     readonly incident?: EventIncident;
     protected constructor({ eventType, playerId, target, score, activeTraits, incident }: InPlayEventOpts);
-    protected _toString(): string;
 }

package/dist/cjs/src/service/event/in-play-event.js

@@ -10,13 +10,5 @@ class InPlayEvent extends rally_event_1.RallyEvent {
         this.activeTraits = activeTraits;
         this.incident = incident;
     }
-    _toString() {
-        const score = `"score":${this.score}`;
-        const playerId = `"playerId":"${this.playerId}"`;
-        const target = `"target":${this.target}`;
-        const eventType = `"eventType":${this.eventType}`;
-        const traits = `"activeTraits":[${this.activeTraits.map(t => `"${t}"`).join(',')}]`;
-        return `${score},${playerId},${target},${eventType},${traits}`;
-    }
 }
 exports.InPlayEvent = InPlayEvent;

package/dist/cjs/src/service/event/libero-replacement.d.ts

@@ -9,5 +9,4 @@ export declare class LiberoReplacement extends RallyEvent {
     readonly libero: string;
     static create(input: unknown): LiberoReplacement;
     private constructor();
-    toString(): string;
 }

package/dist/cjs/src/service/event/libero-replacement.js

@@ -27,12 +27,5 @@ class LiberoReplacement extends rally_event_1.RallyEvent {
         this.type = type;
         this.libero = libero;
     }
-    toString() {
-        const playerId = `"playerId":"${this.playerId}"`;
-        const type = `"type":${this.type}`;
-        const eventType = `"eventType":${this.eventType}`;
-        const libero = `"libero":"${this.libero}"`;
-        return `{${playerId},${type},${eventType},${libero}}`;
-    }
 }
 exports.LiberoReplacement = LiberoReplacement;

package/dist/cjs/src/service/event/rally-event.d.ts

@@ -17,5 +17,4 @@ export declare abstract class RallyEvent {
     readonly playerId: string;
     readonly eventType: EventType;
     protected constructor({ eventType, playerId }: RallyEventOpts);
-    abstract toString(): string;
 }

package/dist/cjs/src/service/event/reception.d.ts

@@ -20,5 +20,4 @@ export declare class Reception extends InPlayEvent<ReceptionFailure, ReceptionTy
     readonly type: ReceptionType;
     static create(input: unknown): Reception;
     private constructor();
-    toString(): string;
 }

package/dist/cjs/src/service/event/reception.js

@@ -41,10 +41,5 @@ class Reception extends in_play_event_1.InPlayEvent {
         this.type = type;
         this.failure = failure;
     }
-    toString() {
-        const failure = `"failure":${this.failure}`;
-        const type = `"type":${this.type}`;
-        return `{${this._toString()},${failure},${type}}`;
-    }
 }
 exports.Reception = Reception;

package/dist/cjs/src/service/event/serve.d.ts

@@ -19,5 +19,4 @@ export declare class Serve extends InPlayEvent<ServeFailure, ServeType> {
     readonly type: ServeType;
     static create(input: unknown): Serve;
     private constructor();
-    toString(): string;
 }

package/dist/cjs/src/service/event/serve.js

@@ -38,10 +38,5 @@ class Serve extends in_play_event_1.InPlayEvent {
         this.type = type;
         this.failure = failure;
     }
-    toString() {
-        const failure = `"failure":${this.failure}`;
-        const type = `"type":${this.type}`;
-        return `{${this._toString()},${failure},${type}}`;
-    }
 }
 exports.Serve = Serve;

package/dist/cjs/src/service/event/set.d.ts

@@ -24,5 +24,4 @@ export declare class Set extends InPlayEvent<SetFailure, SetType> {
     readonly tempo?: SetTempo;
     static create(input: unknown): Set;
     private constructor();
-    toString(): string;
 }

package/dist/cjs/src/service/event/set.js

@@ -47,11 +47,5 @@ class Set extends in_play_event_1.InPlayEvent {
         this.failure = failure;
         this.tempo = tempo;
     }
-    toString() {
-        const failure = `"failure":${this.failure}`;
-        const type = `"type":${this.type}`;
-        const tempo = this.tempo !== undefined ? `,"tempo":${this.tempo}` : '';
-        return `{${this._toString()},${failure},${type}${tempo}}`;
-    }
 }
 exports.Set = Set;

package/dist/cjs/src/service/event/spike.d.ts

@@ -21,5 +21,4 @@ export declare class Spike extends InPlayEvent<SpikeFailure, SpikeType> {
     readonly type: SpikeType;
     static create(input: unknown): Spike;
     private constructor();
-    toString(): string;
 }

package/dist/cjs/src/service/event/spike.js

@@ -40,10 +40,5 @@ class Spike extends in_play_event_1.InPlayEvent {
         this.type = type;
         this.failure = failure;
     }
-    toString() {
-        const failure = `"failure":${this.failure}`;
-        const type = `"type":${this.type}`;
-        return `{${this._toString()},${failure},${type}}`;
-    }
 }
 exports.Spike = Spike;

package/dist/cjs/src/service/event/substitution.d.ts

@@ -4,5 +4,4 @@ export declare class Substitution extends RallyEvent {
     readonly injury?: boolean;
     static create(input: unknown): Substitution;
     private constructor();
-    toString(): string;
 }

package/dist/cjs/src/service/event/substitution.js

@@ -22,12 +22,5 @@ class Substitution extends rally_event_1.RallyEvent {
         this.playerOut = playerOut;
         this.injury = injury;
     }
-    toString() {
-        const playerId = `"playerId":"${this.playerId}"`;
-        const playerOut = `"playerOut":"${this.playerOut}"`;
-        const eventType = `"eventType":${this.eventType}`;
-        const injury = this.injury === true ? ',"injury":true' : '';
-        return `{${playerId},${playerOut},${eventType}${injury}}`;
-    }
 }
 exports.Substitution = Substitution;

package/dist/cjs/src/service/player/improvement.d.ts

@@ -3,7 +3,6 @@ import { Stats } from './stats';
 import { DeclineProfile } from './decline';
 export type TrainingFocus = Stats;
 export declare function getStatCap(rarity: Rarity): number;
-export declare function getRarityMultiplier(rarity: Rarity): number;
 export declare function getImprovementFloor(rarity: Rarity): number;
 export declare function getImprovementThreshold(statTotal: number, rarity: Rarity): number;
 export declare function getCoachMultiplier(rarity: Rarity): number;

package/dist/cjs/src/service/player/improvement.js

@@ -1,7 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.getStatCap = getStatCap;
-exports.getRarityMultiplier = getRarityMultiplier;
 exports.getImprovementFloor = getImprovementFloor;
 exports.getImprovementThreshold = getImprovementThreshold;
 exports.getCoachMultiplier = getCoachMultiplier;
@@ -40,14 +39,6 @@ function getStatCap(rarity) {
         throw new Error(`UNKNOWN_RARITY: ${rarity}`);
     return entry.cap;
 }
-// Retained for config/reference. No longer used by the improvement curve -- superseded by IMPROVEMENT_FLOOR,
-// which is the per-rarity adjustment now.
-function getRarityMultiplier(rarity) {
-    const entry = statCaps.find(e => e.rarity === rarity);
-    if (entry == null)
-        throw new Error(`UNKNOWN_RARITY: ${rarity}`);
-    return entry.rarityMultiplier;
-}
 function getImprovementFloor(rarity) {
     const floor = IMPROVEMENT_FLOOR[rarity];
     if (floor == null)

package/dist/cjs/src/service/player/improvement.test.js

@@ -20,21 +20,6 @@ const decline_1 = require("./decline");
         (0, globals_1.expect)(() => (0, improvement_1.getStatCap)('DIVINE')).toThrow('UNKNOWN_RARITY');
     });
 });
-// ─── getRarityMultiplier ──────────────────────────────────────────────────────
-(0, globals_1.describe)('getRarityMultiplier()', () => {
-    globals_1.it.each([
-        [rarity_1.RarityEnum.COMMON, 1.0],
-        [rarity_1.RarityEnum.RARE, 1.15],
-        [rarity_1.RarityEnum.LEGENDARY, 1.3],
-        [rarity_1.RarityEnum.MYTHIC, 1.5],
-        [rarity_1.RarityEnum.SPECIAL, 1.75]
-    ])('%s → multiplier %f', (rarity, expected) => {
-        (0, globals_1.expect)((0, improvement_1.getRarityMultiplier)(rarity)).toBeCloseTo(expected, 5);
-    });
-    (0, globals_1.it)('throws UNKNOWN_RARITY for an unrecognised rarity string', () => {
-        (0, globals_1.expect)(() => (0, improvement_1.getRarityMultiplier)('DIVINE')).toThrow('UNKNOWN_RARITY');
-    });
-});
 // ─── getImprovementThreshold ──────────────────────────────────────────────────
 (0, globals_1.describe)('getImprovementThreshold()', () => {
     (0, globals_1.it)('at or below the floor the threshold is CURVE_BASE (180) — the fastest pace', () => {

package/dist/cjs/src/service/team/energy-band.d.ts

@@ -6,5 +6,4 @@ export declare enum EnergyBand {
 }
 export type SubBand = EnergyBand | 'NEVER';
 export type SubMode = 'FATIGUE' | 'PINCH' | 'NEVER';
-export declare const SUB_MODES: readonly SubMode[];
 export declare const ENERGY_BAND_ORDER: readonly EnergyBand[];

package/dist/cjs/src/service/team/energy-band.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.ENERGY_BAND_ORDER = exports.SUB_MODES = exports.EnergyBand = void 0;
+exports.ENERGY_BAND_ORDER = exports.EnergyBand = void 0;
 // The named fatigue bands a player's energy falls into. The numeric cutoffs and per-band performance
 // multipliers live in the simulator (VolleyballSim court-player.ts ENERGY_BANDS); this enum is the shared
 // vocabulary used by tactics configuration (substitution thresholds) and, later, the UI.
@@ -11,7 +11,6 @@ var EnergyBand;
     EnergyBand["TIRED"] = "TIRED";
     EnergyBand["EXHAUSTED"] = "EXHAUSTED";
 })(EnergyBand || (exports.EnergyBand = EnergyBand = {}));
-exports.SUB_MODES = ['FATIGUE', 'PINCH', 'NEVER'];
 // Ordered best -> worst. Used to compare "is this player at-or-below the configured sub band".
 exports.ENERGY_BAND_ORDER = [
     EnergyBand.ENERGETIC,

package/dist/cjs/src/service/team/formation.d.ts

@@ -25,6 +25,5 @@ export declare class Formation {
     readonly liberoSubPriority: SubPriority;
     static create(input: unknown): Formation;
     private constructor();
-    static getFormations(): Formation[];
 }
 export {};

package/dist/cjs/src/service/team/formation.js

@@ -23,9 +23,6 @@ class Formation {
         this.base = base;
         this.liberoSubPriority = liberoSubPriority;
     }
-    static getFormations() {
-        return [this['4-2'], this['5-1'], this['6-2']];
-    }
 }
 exports.Formation = Formation;
 Formation['5-1'] = Formation.create({

package/dist/cjs/src/service/team/sub-config-convert.js

@@ -23,12 +23,17 @@ function fatigueCondition(band) {
 function convertLegacySetSubConfig(raw) {
     if (!isRecord(raw))
         return raw;
-    // Derive the mode for the oldest (pre-mode) generation; a unified shape has none of these keys.
+    // Derive the mode for the oldest (pre-mode) generation; a unified shape has none of these keys. Test the
+    // isPinchServer VALUE, not key presence: the sim read path (transformToObject) spreads `isPinchServer:
+    // d.isPinchServer` onto every row, so a unified row carries an `isPinchServer: undefined` key. `'isPinchServer'
+    // in raw` would then misread that unified row as a legacy FATIGUE sub and overwrite its real conditions /
+    // pinchServer flag. A genuine legacy isPinchServer-era row always has a boolean flag (true handled above), so
+    // the only remaining legacy case is an explicit `=== false`.
     const mode = raw.mode ?? (raw.isPinchServer === true
         ? 'PINCH'
         : raw.fatigueBand === 'NEVER'
             ? 'NEVER'
-            : raw.fatigueBand != null || 'isPinchServer' in raw ? 'FATIGUE' : null);
+            : raw.fatigueBand != null || raw.isPinchServer === false ? 'FATIGUE' : null);
     if (mode == null)
         return raw;
     const { mode: _m, fatigueBand, isPinchServer: _p, ...rest } = raw;

package/dist/cjs/src/service/team/sub-config-convert.test.js

@@ -12,6 +12,25 @@ const sub_config_convert_1 = require("./sub-config-convert");
         const unified = { bench: 'X', conditions: [{ type: pinch_condition_1.PinchConditionType.ASAP }], pinchServer: true };
         (0, globals_1.expect)((0, sub_config_convert_1.convertLegacySetSubConfig)(unified)).toBe(unified);
     });
+    (0, globals_1.it)('keeps a unified pinch server intact when a caller injects isPinchServer: undefined (sim read path)', () => {
+        // The sim read path (transformToObject) spreads `isPinchServer: d.isPinchServer` onto every row, so a
+        // unified row arrives carrying an `isPinchServer: undefined` key. Detecting the legacy shape by key
+        // PRESENCE misread it as a FATIGUE sub and wiped its real conditions + pinchServer flag; the value check
+        // (=== false) keeps the unified shape. Regression for the "pinch server never serves" prod bug.
+        const out = (0, sub_config_convert_1.convertLegacySetSubConfig)({
+            bench: 'X',
+            conditions: [{ type: pinch_condition_1.PinchConditionType.AFTER_ROTATIONS, rotations: 1 }],
+            conditionLogic: 'ANY',
+            pinchServer: true,
+            subBackMode: 'FATIGUE',
+            isPinchServer: undefined,
+            mode: undefined,
+            fatigueBand: undefined
+        });
+        (0, globals_1.expect)(out.pinchServer).toBe(true);
+        (0, globals_1.expect)(out.conditions).toEqual([{ type: pinch_condition_1.PinchConditionType.AFTER_ROTATIONS, rotations: 1 }]);
+        (0, globals_1.expect)(out.conditionLogic).toBe('ANY');
+    });
     (0, globals_1.it)('converts mode NEVER to an explicit empty condition list, dropping every other field', () => {
         (0, globals_1.expect)((0, sub_config_convert_1.convertLegacySetSubConfig)({ mode: 'NEVER', bench: 'X' })).toEqual({ conditions: [] });
     });

package/dist/cjs/src/service/utils/object-utils.d.ts

@@ -1,4 +1,3 @@
-export declare const getKeys: <T extends object>(obj: T) => Array<keyof T>;
 export type DataProps<T> = {
     [K in keyof T as T[K] extends (...args: any[]) => any ? never : K]: T[K];
 };

package/dist/cjs/src/service/utils/object-utils.js

@@ -1,4 +1,2 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.getKeys = void 0;
-exports.getKeys = Object.keys;

package/dist/esm/src/service/competition/season.d.ts

@@ -10,12 +10,11 @@ export declare class Season {
     readonly iteration: Iteration;
     readonly divisionId: string;
     readonly standings: Standing[];
-    private _status;
-    private _champion?;
+    private readonly _status;
+    private readonly _champion?;
     static create(input: unknown): Season;
     private constructor();
     calculateStandings(): Standing[];
-    updateStandings(): void;
     get champion(): Team | undefined;
     get status(): Status;
     static createSeason(iteration: Iteration, teams: Team[], divisionId: string): Season;

package/dist/esm/src/service/competition/season.js

@@ -34,17 +34,6 @@ export class Season {
             return standing;
         });
     }
-    updateStandings() {
-        if (this.matches == null || this.matches.length < 1)
-            throw new Error('NO_MATCHES');
-        const updated = this.calculateStandings();
-        this.standings.splice(0, this.standings.length, ...updated);
-        const isOver = this.matches.every((match) => match.isOver());
-        if (isOver) {
-            this._status = StatusEnum.COMPLETE;
-            this._champion = this.teams.find((team) => team.id === this.standings[0].teamId);
-        }
-    }
     get champion() {
         if (this._champion != null)
             return this._champion;

package/dist/esm/src/service/event/block.d.ts

@@ -21,5 +21,4 @@ export declare class Block extends InPlayEvent<BlockFailure, BlockType> {
     readonly blockers: string[];
     static create(input: unknown): Block;
     private constructor();
-    toString(): string;
 }

package/dist/esm/src/service/event/block.js

@@ -37,10 +37,4 @@ export class Block extends InPlayEvent {
         this.failure = failure;
         this.blockers = blockers;
     }
-    toString() {
-        const failure = `"failure":${this.failure}`;
-        const type = `"type":${this.type}`;
-        const blockers = `"blockers":[${this.blockers.map(b => `"${b}"`).join(',')}]`;
-        return `{${this._toString()},${failure},${type},${blockers}}`;
-    }
 }

package/dist/esm/src/service/event/free-ball.d.ts

@@ -13,5 +13,4 @@ export declare class FreeBall extends InPlayEvent<FreeBallFailure, FreeBallType>
     readonly type: FreeBallType;
     static create(input: unknown): FreeBall;
     private constructor();
-    toString(): string;
 }

package/dist/esm/src/service/event/free-ball.js

@@ -35,9 +35,4 @@ export class FreeBall extends InPlayEvent {
         this.type = type;
         this.failure = failure;
     }
-    toString() {
-        const failure = `"failure":${this.failure}`;
-        const type = `"type":${this.type}`;
-        return `{${this._toString()},${failure},${type}}`;
-    }
 }

package/dist/esm/src/service/event/in-play-event.d.ts

@@ -20,5 +20,4 @@ export declare abstract class InPlayEvent<F, T> extends RallyEvent {
     readonly activeTraits: Trait[];
     readonly incident?: EventIncident;
     protected constructor({ eventType, playerId, target, score, activeTraits, incident }: InPlayEventOpts);
-    protected _toString(): string;
 }

package/dist/esm/src/service/event/in-play-event.js

@@ -7,12 +7,4 @@ export class InPlayEvent extends RallyEvent {
         this.activeTraits = activeTraits;
         this.incident = incident;
     }
-    _toString() {
-        const score = `"score":${this.score}`;
-        const playerId = `"playerId":"${this.playerId}"`;
-        const target = `"target":${this.target}`;
-        const eventType = `"eventType":${this.eventType}`;
-        const traits = `"activeTraits":[${this.activeTraits.map(t => `"${t}"`).join(',')}]`;
-        return `${score},${playerId},${target},${eventType},${traits}`;
-    }
 }

package/dist/esm/src/service/event/libero-replacement.d.ts

@@ -9,5 +9,4 @@ export declare class LiberoReplacement extends RallyEvent {
     readonly libero: string;
     static create(input: unknown): LiberoReplacement;
     private constructor();
-    toString(): string;
 }

package/dist/esm/src/service/event/libero-replacement.js

@@ -24,11 +24,4 @@ export class LiberoReplacement extends RallyEvent {
         this.type = type;
         this.libero = libero;
     }
-    toString() {
-        const playerId = `"playerId":"${this.playerId}"`;
-        const type = `"type":${this.type}`;
-        const eventType = `"eventType":${this.eventType}`;
-        const libero = `"libero":"${this.libero}"`;
-        return `{${playerId},${type},${eventType},${libero}}`;
-    }
 }

package/dist/esm/src/service/event/rally-event.d.ts

@@ -17,5 +17,4 @@ export declare abstract class RallyEvent {
     readonly playerId: string;
     readonly eventType: EventType;
     protected constructor({ eventType, playerId }: RallyEventOpts);
-    abstract toString(): string;
 }

package/dist/esm/src/service/event/reception.d.ts

@@ -20,5 +20,4 @@ export declare class Reception extends InPlayEvent<ReceptionFailure, ReceptionTy
     readonly type: ReceptionType;
     static create(input: unknown): Reception;
     private constructor();
-    toString(): string;
 }

package/dist/esm/src/service/event/reception.js

@@ -38,9 +38,4 @@ export class Reception extends InPlayEvent {
         this.type = type;
         this.failure = failure;
     }
-    toString() {
-        const failure = `"failure":${this.failure}`;
-        const type = `"type":${this.type}`;
-        return `{${this._toString()},${failure},${type}}`;
-    }
 }

package/dist/esm/src/service/event/serve.d.ts

@@ -19,5 +19,4 @@ export declare class Serve extends InPlayEvent<ServeFailure, ServeType> {
     readonly type: ServeType;
     static create(input: unknown): Serve;
     private constructor();
-    toString(): string;
 }

package/dist/esm/src/service/event/serve.js

@@ -35,9 +35,4 @@ export class Serve extends InPlayEvent {
         this.type = type;
         this.failure = failure;
     }
-    toString() {
-        const failure = `"failure":${this.failure}`;
-        const type = `"type":${this.type}`;
-        return `{${this._toString()},${failure},${type}}`;
-    }
 }

package/dist/esm/src/service/event/set.d.ts

@@ -24,5 +24,4 @@ export declare class Set extends InPlayEvent<SetFailure, SetType> {
     readonly tempo?: SetTempo;
     static create(input: unknown): Set;
     private constructor();
-    toString(): string;
 }

package/dist/esm/src/service/event/set.js

@@ -44,10 +44,4 @@ export class Set extends InPlayEvent {
         this.failure = failure;
         this.tempo = tempo;
     }
-    toString() {
-        const failure = `"failure":${this.failure}`;
-        const type = `"type":${this.type}`;
-        const tempo = this.tempo !== undefined ? `,"tempo":${this.tempo}` : '';
-        return `{${this._toString()},${failure},${type}${tempo}}`;
-    }
 }

package/dist/esm/src/service/event/spike.d.ts

@@ -21,5 +21,4 @@ export declare class Spike extends InPlayEvent<SpikeFailure, SpikeType> {
     readonly type: SpikeType;
     static create(input: unknown): Spike;
     private constructor();
-    toString(): string;
 }

package/dist/esm/src/service/event/spike.js

@@ -37,9 +37,4 @@ export class Spike extends InPlayEvent {
         this.type = type;
         this.failure = failure;
     }
-    toString() {
-        const failure = `"failure":${this.failure}`;
-        const type = `"type":${this.type}`;
-        return `{${this._toString()},${failure},${type}}`;
-    }
 }

package/dist/esm/src/service/event/substitution.d.ts

@@ -4,5 +4,4 @@ export declare class Substitution extends RallyEvent {
     readonly injury?: boolean;
     static create(input: unknown): Substitution;
     private constructor();
-    toString(): string;
 }

package/dist/esm/src/service/event/substitution.js

@@ -19,11 +19,4 @@ export class Substitution extends RallyEvent {
         this.playerOut = playerOut;
         this.injury = injury;
     }
-    toString() {
-        const playerId = `"playerId":"${this.playerId}"`;
-        const playerOut = `"playerOut":"${this.playerOut}"`;
-        const eventType = `"eventType":${this.eventType}`;
-        const injury = this.injury === true ? ',"injury":true' : '';
-        return `{${playerId},${playerOut},${eventType}${injury}}`;
-    }
 }

package/dist/esm/src/service/player/improvement.d.ts

@@ -3,7 +3,6 @@ import { Stats } from './stats';
 import { DeclineProfile } from './decline';
 export type TrainingFocus = Stats;
 export declare function getStatCap(rarity: Rarity): number;
-export declare function getRarityMultiplier(rarity: Rarity): number;
 export declare function getImprovementFloor(rarity: Rarity): number;
 export declare function getImprovementThreshold(statTotal: number, rarity: Rarity): number;
 export declare function getCoachMultiplier(rarity: Rarity): number;

package/dist/esm/src/service/player/improvement.js

@@ -30,14 +30,6 @@ export function getStatCap(rarity) {
         throw new Error(`UNKNOWN_RARITY: ${rarity}`);
     return entry.cap;
 }
-// Retained for config/reference. No longer used by the improvement curve -- superseded by IMPROVEMENT_FLOOR,
-// which is the per-rarity adjustment now.
-export function getRarityMultiplier(rarity) {
-    const entry = statCaps.find(e => e.rarity === rarity);
-    if (entry == null)
-        throw new Error(`UNKNOWN_RARITY: ${rarity}`);
-    return entry.rarityMultiplier;
-}
 export function getImprovementFloor(rarity) {
     const floor = IMPROVEMENT_FLOOR[rarity];
     if (floor == null)

package/dist/esm/src/service/player/improvement.test.js

@@ -1,6 +1,6 @@
 import { describe, it, expect } from '@jest/globals';
 import { RarityEnum } from './rarity';
-import { getStatCap, getRarityMultiplier, getImprovementFloor, getImprovementThreshold, getCoachMultiplier, selectImprovableStat, matchExperience, experienceGained } from './improvement';
+import { getStatCap, getImprovementFloor, getImprovementThreshold, getCoachMultiplier, selectImprovableStat, matchExperience, experienceGained } from './improvement';
 import { StatsEnum } from './stats';
 import { DeclineProfileEnum } from './decline';
 // ─── getStatCap ───────────────────────────────────────────────────────────────
@@ -18,21 +18,6 @@ describe('getStatCap()', () => {
         expect(() => getStatCap('DIVINE')).toThrow('UNKNOWN_RARITY');
     });
 });
-// ─── getRarityMultiplier ──────────────────────────────────────────────────────
-describe('getRarityMultiplier()', () => {
-    it.each([
-        [RarityEnum.COMMON, 1.0],
-        [RarityEnum.RARE, 1.15],
-        [RarityEnum.LEGENDARY, 1.3],
-        [RarityEnum.MYTHIC, 1.5],
-        [RarityEnum.SPECIAL, 1.75]
-    ])('%s → multiplier %f', (rarity, expected) => {
-        expect(getRarityMultiplier(rarity)).toBeCloseTo(expected, 5);
-    });
-    it('throws UNKNOWN_RARITY for an unrecognised rarity string', () => {
-        expect(() => getRarityMultiplier('DIVINE')).toThrow('UNKNOWN_RARITY');
-    });
-});
 // ─── getImprovementThreshold ──────────────────────────────────────────────────
 describe('getImprovementThreshold()', () => {
     it('at or below the floor the threshold is CURVE_BASE (180) — the fastest pace', () => {

package/dist/esm/src/service/team/energy-band.d.ts

@@ -6,5 +6,4 @@ export declare enum EnergyBand {
 }
 export type SubBand = EnergyBand | 'NEVER';
 export type SubMode = 'FATIGUE' | 'PINCH' | 'NEVER';
-export declare const SUB_MODES: readonly SubMode[];
 export declare const ENERGY_BAND_ORDER: readonly EnergyBand[];

package/dist/esm/src/service/team/energy-band.js

@@ -8,7 +8,6 @@ export var EnergyBand;
     EnergyBand["TIRED"] = "TIRED";
     EnergyBand["EXHAUSTED"] = "EXHAUSTED";
 })(EnergyBand || (EnergyBand = {}));
-export const SUB_MODES = ['FATIGUE', 'PINCH', 'NEVER'];
 // Ordered best -> worst. Used to compare "is this player at-or-below the configured sub band".
 export const ENERGY_BAND_ORDER = [
     EnergyBand.ENERGETIC,

package/dist/esm/src/service/team/formation.d.ts

@@ -25,6 +25,5 @@ export declare class Formation {
     readonly liberoSubPriority: SubPriority;
     static create(input: unknown): Formation;
     private constructor();
-    static getFormations(): Formation[];
 }
 export {};

package/dist/esm/src/service/team/formation.js

@@ -20,9 +20,6 @@ export class Formation {
         this.base = base;
         this.liberoSubPriority = liberoSubPriority;
     }
-    static getFormations() {
-        return [this['4-2'], this['5-1'], this['6-2']];
-    }
 }
 Formation['5-1'] = Formation.create({
     name: '5-1',

package/dist/esm/src/service/team/sub-config-convert.js

@@ -17,12 +17,17 @@ function fatigueCondition(band) {
 export function convertLegacySetSubConfig(raw) {
     if (!isRecord(raw))
         return raw;
-    // Derive the mode for the oldest (pre-mode) generation; a unified shape has none of these keys.
+    // Derive the mode for the oldest (pre-mode) generation; a unified shape has none of these keys. Test the
+    // isPinchServer VALUE, not key presence: the sim read path (transformToObject) spreads `isPinchServer:
+    // d.isPinchServer` onto every row, so a unified row carries an `isPinchServer: undefined` key. `'isPinchServer'
+    // in raw` would then misread that unified row as a legacy FATIGUE sub and overwrite its real conditions /
+    // pinchServer flag. A genuine legacy isPinchServer-era row always has a boolean flag (true handled above), so
+    // the only remaining legacy case is an explicit `=== false`.
     const mode = raw.mode ?? (raw.isPinchServer === true
         ? 'PINCH'
         : raw.fatigueBand === 'NEVER'
             ? 'NEVER'
-            : raw.fatigueBand != null || 'isPinchServer' in raw ? 'FATIGUE' : null);
+            : raw.fatigueBand != null || raw.isPinchServer === false ? 'FATIGUE' : null);
     if (mode == null)
         return raw;
     const { mode: _m, fatigueBand, isPinchServer: _p, ...rest } = raw;

package/dist/esm/src/service/team/sub-config-convert.test.js

@@ -10,6 +10,25 @@ describe('convertLegacySetSubConfig', () => {
         const unified = { bench: 'X', conditions: [{ type: PinchConditionType.ASAP }], pinchServer: true };
         expect(convertLegacySetSubConfig(unified)).toBe(unified);
     });
+    it('keeps a unified pinch server intact when a caller injects isPinchServer: undefined (sim read path)', () => {
+        // The sim read path (transformToObject) spreads `isPinchServer: d.isPinchServer` onto every row, so a
+        // unified row arrives carrying an `isPinchServer: undefined` key. Detecting the legacy shape by key
+        // PRESENCE misread it as a FATIGUE sub and wiped its real conditions + pinchServer flag; the value check
+        // (=== false) keeps the unified shape. Regression for the "pinch server never serves" prod bug.
+        const out = convertLegacySetSubConfig({
+            bench: 'X',
+            conditions: [{ type: PinchConditionType.AFTER_ROTATIONS, rotations: 1 }],
+            conditionLogic: 'ANY',
+            pinchServer: true,
+            subBackMode: 'FATIGUE',
+            isPinchServer: undefined,
+            mode: undefined,
+            fatigueBand: undefined
+        });
+        expect(out.pinchServer).toBe(true);
+        expect(out.conditions).toEqual([{ type: PinchConditionType.AFTER_ROTATIONS, rotations: 1 }]);
+        expect(out.conditionLogic).toBe('ANY');
+    });
     it('converts mode NEVER to an explicit empty condition list, dropping every other field', () => {
         expect(convertLegacySetSubConfig({ mode: 'NEVER', bench: 'X' })).toEqual({ conditions: [] });
     });

package/dist/esm/src/service/utils/object-utils.d.ts

@@ -1,4 +1,3 @@
-export declare const getKeys: <T extends object>(obj: T) => Array<keyof T>;
 export type DataProps<T> = {
     [K in keyof T as T[K] extends (...args: any[]) => any ? never : K]: T[K];
 };

package/dist/esm/src/service/utils/object-utils.js

@@ -1 +1 @@
-export const getKeys = Object.keys;
+export {};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "volleyballsimtypes",
-  "version": "0.0.483",
+  "version": "0.0.486",
   "description": "vbsim types",
   "main": "./dist/cjs/src/index.js",
   "module": "./dist/esm/src/index.js",