volleyballsimtypes 0.0.404 → 0.0.407

package/dist/cjs/src/data/transformers/court-position.js

@@ -21,6 +21,8 @@ function transformToType(position) {
             return 'MIDDLE_BACK';
         case service_1.CourtPosition.RIGHT_BACK:
             return 'RIGHT_BACK';
+        case service_1.CourtPosition.RESERVE_LIBERO:
+            return 'RESERVE_LIBERO';
         default:
             throw new Error('UNKNOWN_ROLE');
     }

package/dist/cjs/src/data/transformers/match-set.d.ts

@@ -1,6 +1,6 @@
 import { MatchSetAttributes, MatchSetModel } from '../models';
 import { MatchSet } from '../../service';
-export type CourtPositionType = 'NOT_IN_COURT' | 'LIBERO_ZONE' | 'LEFT_FRONT' | 'MIDDLE_FRONT' | 'RIGHT_FRONT' | 'LEFT_BACK' | 'MIDDLE_BACK' | 'RIGHT_BACK';
+export type CourtPositionType = 'NOT_IN_COURT' | 'LIBERO_ZONE' | 'LEFT_FRONT' | 'MIDDLE_FRONT' | 'RIGHT_FRONT' | 'LEFT_BACK' | 'MIDDLE_BACK' | 'RIGHT_BACK' | 'RESERVE_LIBERO';
 export type Side = 'HOME' | 'AWAY';
 export interface PlayerPositionAttributes {
     player_id: string;

package/dist/cjs/src/data/transformers/tactics.js

@@ -88,15 +88,27 @@ function buildPinchServerSubs(pinchServerSubs, roster) {
     return map;
 }
 function transformToObject(model, roster) {
+    const lineup = transformToLineup(model.lineup, roster);
+    // A second libero only makes sense alongside a starting libero (LIBERO_ZONE). Edge/legacy data can carry a
+    // second_libero (and its swap config) with no starting libero -- e.g. the starting libero was removed
+    // without clearing it -- which fails validation on load and would crash the whole sim batch. Drop the
+    // orphaned pair so the team still loads (it just plays without liberos); the next tactics save rewrites it.
+    // Drop the second libero when there's no starting libero, OR when the L2 player is no longer on the roster
+    // (e.g. retired/dismissed) -- both leave it orphaned. Use roster.find (not findPlayer) so a gone L2 yields
+    // undefined instead of throwing.
+    const hasStartingLibero = lineup[service_1.CourtPosition.LIBERO_ZONE] != null;
+    const secondLibero = (hasStartingLibero && model.second_libero != null)
+        ? roster.find((p) => p.id === model.second_libero)
+        : undefined;
     return service_1.Tactics.create({
         substitutionTolerance: model.substitution_tolerance,
         receiveRotationOffset: model.receive_rotation_offset,
-        lineup: transformToLineup(model.lineup, roster),
+        lineup,
         liberoReplacements: model.libero_replacements.map((lr) => findPlayer(lr, roster)),
         pinchServerSubs: buildPinchServerSubs(model.pinch_server_subs, roster),
         substitutionBand: model.substitution_band,
-        secondLibero: model.second_libero != null ? findPlayer(model.second_libero, roster) : undefined,
-        liberoSub: model.libero_sub ?? undefined,
+        secondLibero,
+        liberoSub: secondLibero != null ? (model.libero_sub ?? undefined) : undefined,
         liberoReplacementSets: model.libero_replacement_sets?.map(set => set.map((id) => findPlayer(id, roster))),
         designatedSubs: (model.designated_subs ?? []).map(d => ({
             starter: findPlayer(d.starterId, roster),

package/dist/esm/src/data/transformers/court-position.js

@@ -17,6 +17,8 @@ function transformToType(position) {
             return 'MIDDLE_BACK';
         case CourtPosition.RIGHT_BACK:
             return 'RIGHT_BACK';
+        case CourtPosition.RESERVE_LIBERO:
+            return 'RESERVE_LIBERO';
         default:
             throw new Error('UNKNOWN_ROLE');
     }

package/dist/esm/src/data/transformers/match-set.d.ts

@@ -1,6 +1,6 @@
 import { MatchSetAttributes, MatchSetModel } from '../models';
 import { MatchSet } from '../../service';
-export type CourtPositionType = 'NOT_IN_COURT' | 'LIBERO_ZONE' | 'LEFT_FRONT' | 'MIDDLE_FRONT' | 'RIGHT_FRONT' | 'LEFT_BACK' | 'MIDDLE_BACK' | 'RIGHT_BACK';
+export type CourtPositionType = 'NOT_IN_COURT' | 'LIBERO_ZONE' | 'LEFT_FRONT' | 'MIDDLE_FRONT' | 'RIGHT_FRONT' | 'LEFT_BACK' | 'MIDDLE_BACK' | 'RIGHT_BACK' | 'RESERVE_LIBERO';
 export type Side = 'HOME' | 'AWAY';
 export interface PlayerPositionAttributes {
     player_id: string;

package/dist/esm/src/data/transformers/tactics.js

@@ -84,15 +84,27 @@ function buildPinchServerSubs(pinchServerSubs, roster) {
     return map;
 }
 function transformToObject(model, roster) {
+    const lineup = transformToLineup(model.lineup, roster);
+    // A second libero only makes sense alongside a starting libero (LIBERO_ZONE). Edge/legacy data can carry a
+    // second_libero (and its swap config) with no starting libero -- e.g. the starting libero was removed
+    // without clearing it -- which fails validation on load and would crash the whole sim batch. Drop the
+    // orphaned pair so the team still loads (it just plays without liberos); the next tactics save rewrites it.
+    // Drop the second libero when there's no starting libero, OR when the L2 player is no longer on the roster
+    // (e.g. retired/dismissed) -- both leave it orphaned. Use roster.find (not findPlayer) so a gone L2 yields
+    // undefined instead of throwing.
+    const hasStartingLibero = lineup[CourtPosition.LIBERO_ZONE] != null;
+    const secondLibero = (hasStartingLibero && model.second_libero != null)
+        ? roster.find((p) => p.id === model.second_libero)
+        : undefined;
     return Tactics.create({
         substitutionTolerance: model.substitution_tolerance,
         receiveRotationOffset: model.receive_rotation_offset,
-        lineup: transformToLineup(model.lineup, roster),
+        lineup,
         liberoReplacements: model.libero_replacements.map((lr) => findPlayer(lr, roster)),
         pinchServerSubs: buildPinchServerSubs(model.pinch_server_subs, roster),
         substitutionBand: model.substitution_band,
-        secondLibero: model.second_libero != null ? findPlayer(model.second_libero, roster) : undefined,
-        liberoSub: model.libero_sub ?? undefined,
+        secondLibero,
+        liberoSub: secondLibero != null ? (model.libero_sub ?? undefined) : undefined,
         liberoReplacementSets: model.libero_replacement_sets?.map(set => set.map((id) => findPlayer(id, roster))),
         designatedSubs: (model.designated_subs ?? []).map(d => ({
             starter: findPlayer(d.starterId, roster),

package/package.json

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