volleyballsimtypes 0.0.403 → 0.0.406

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/cjs/src/service/match/court-position.d.ts

@@ -10,7 +10,8 @@ export declare enum CourtPosition {
     RIGHT_FRONT = 2,
     LEFT_BACK = 5,
     MIDDLE_BACK = 6,
-    RIGHT_BACK = 1
+    RIGHT_BACK = 1,
+    RESERVE_LIBERO = 8
 }
 export declare enum CourtTarget {
     NO_TARGET = 0,

package/dist/cjs/src/service/match/court-position.js

@@ -23,6 +23,10 @@ var CourtPosition;
     CourtPosition[CourtPosition["LEFT_BACK"] = 5] = "LEFT_BACK";
     CourtPosition[CourtPosition["MIDDLE_BACK"] = 6] = "MIDDLE_BACK";
     CourtPosition[CourtPosition["RIGHT_BACK"] = 1] = "RIGHT_BACK";
+    // The off-court member of the libero pair: the second libero held in reserve, or the starting libero
+    // once the second libero is swapped in for a set. Distinct from NOT_IN_COURT so it is never counted as
+    // one of the (max 6) bench subs.
+    CourtPosition[CourtPosition["RESERVE_LIBERO"] = 8] = "RESERVE_LIBERO";
 })(CourtPosition || (exports.CourtPosition = CourtPosition = {}));
 var CourtTarget;
 (function (CourtTarget) {

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/dist/esm/src/service/match/court-position.d.ts

@@ -10,7 +10,8 @@ export declare enum CourtPosition {
     RIGHT_FRONT = 2,
     LEFT_BACK = 5,
     MIDDLE_BACK = 6,
-    RIGHT_BACK = 1
+    RIGHT_BACK = 1,
+    RESERVE_LIBERO = 8
 }
 export declare enum CourtTarget {
     NO_TARGET = 0,

package/dist/esm/src/service/match/court-position.js

@@ -14,6 +14,10 @@ export var CourtPosition;
     CourtPosition[CourtPosition["LEFT_BACK"] = 5] = "LEFT_BACK";
     CourtPosition[CourtPosition["MIDDLE_BACK"] = 6] = "MIDDLE_BACK";
     CourtPosition[CourtPosition["RIGHT_BACK"] = 1] = "RIGHT_BACK";
+    // The off-court member of the libero pair: the second libero held in reserve, or the starting libero
+    // once the second libero is swapped in for a set. Distinct from NOT_IN_COURT so it is never counted as
+    // one of the (max 6) bench subs.
+    CourtPosition[CourtPosition["RESERVE_LIBERO"] = 8] = "RESERVE_LIBERO";
 })(CourtPosition || (CourtPosition = {}));
 export var CourtTarget;
 (function (CourtTarget) {

package/package.json

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