isaacscript-common 31.5.0 → 31.6.0

package/dist/src/sets/bossSets.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"bossSets.d.ts","sourceRoot":"","sources":["../../../src/sets/bossSets.ts"],"names":[],"mappings":"AAmbA,eAAO,MAAM,mCAAmC,+DAe9C,CAAC;AAEH,eAAO,MAAM,8BAA8B,0CAezC,CAAC;AAEH,eAAO,MAAM,cAAc,qBAQzB,CAAC;AAEH,eAAO,MAAM,qCAAqC,qBACX,CAAC"}
+{"version":3,"file":"bossSets.d.ts","sourceRoot":"","sources":["../../../src/sets/bossSets.ts"],"names":[],"mappings":"AAobA,eAAO,MAAM,mCAAmC,+DAe9C,CAAC;AAEH,eAAO,MAAM,8BAA8B,0CAezC,CAAC;AAEH,eAAO,MAAM,cAAc,qBAQzB,CAAC;AAEH,eAAO,MAAM,qCAAqC,qBACX,CAAC"}

package/dist/src/sets/bossSets.lua

@@ -41,6 +41,8 @@ local ____entities = require("src.functions.entities")
 local parseEntityTypeVariantString = ____entities.parseEntityTypeVariantString
 local ____set = require("src.functions.set")
 local copySet = ____set.copySet
+local ____utils = require("src.functions.utils")
+local assertDefined = ____utils.assertDefined
 local ____ReadonlyMap = require("src.types.ReadonlyMap")
 local ReadonlyMap = ____ReadonlyMap.ReadonlyMap
 local ____ReadonlySet = require("src.types.ReadonlySet")
@@ -52,9 +54,7 @@ function getAllBossesExcludingStoryBossesSet(self)
     local allBosses = {__TS__Spread(____exports.ALL_BOSSES_SET:values())}
     for ____, entityTypeVariantString in ipairs(allBosses) do
         local tuple = parseEntityTypeVariantString(nil, entityTypeVariantString)
-        if tuple == nil then
-            error("Failed to parse a boss tuple when constructing the story boss set.")
-        end
+        assertDefined(nil, tuple, "Failed to parse a boss tuple when constructing the story boss set.")
         local entityType, _variant = table.unpack(tuple)
         if STORY_BOSSES_SET:has(entityType) then
             allBossesExcludingStoryBossesSet:delete(entityTypeVariantString)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "isaacscript-common",
-  "version": "31.5.0",
+  "version": "31.6.0",
   "description": "Helper functions and features for IsaacScript mods.",
   "keywords": [
     "isaac",
@@ -25,6 +25,6 @@
   "main": "dist/src/index",
   "types": "dist/index.rollup.d.ts",
   "dependencies": {
-    "isaac-typescript-definitions": "^13.0.26"
+    "isaac-typescript-definitions": "^13.0.27"
   }
 }

package/src/classes/ModFeature.ts

@@ -7,6 +7,7 @@ import {
   getTSTLClassName,
 } from "../functions/tstlClass";
 import { isFunction, isNumber, isTable } from "../functions/types";
+import { assertDefined } from "../functions/utils";
 import type { TSTLClassMetatable } from "../interfaces/TSTLClassMetatable";
 import type { AnyFunction } from "../types/AnyFunction";
 import type { ModUpgraded } from "./ModUpgraded";
@@ -124,14 +125,16 @@ export class ModFeature {
     this.initialized = init;
 
     const constructor = getTSTLClassConstructor(this);
-    if (constructor === undefined) {
-      error("Failed to get the TSTL class constructor for a mod feature.");
-    }
+    assertDefined(
+      constructor,
+      "Failed to get the TSTL class constructor for a mod feature.",
+    );
 
     const tstlClassName = getTSTLClassName(this);
-    if (tstlClassName === undefined) {
-      error("Failed to get the TSTL class name for a mod feature.");
-    }
+    assertDefined(
+      tstlClassName,
+      "Failed to get the TSTL class name for a mod feature.",
+    );
 
     initDecoratedCallbacks(this, constructor, tstlClassName, true, init);
     initDecoratedCallbacks(this, constructor, tstlClassName, false, init);
@@ -360,12 +363,13 @@ function initSaveDataManager(
   const saveDataManagerMethodName = init
     ? "saveDataManager"
     : "saveDataManagerRemove";
-  const saveDataManagerMethod = mod[saveDataManagerMethodName];
-  if (saveDataManagerMethod === undefined) {
-    error(
-      'Failed to initialize a mod feature class due to having a "v" object and not having the save data manager initialized. You must pass "ISCFeature.SAVE_DATA_MANAGER" to the "upgradeMod" function.',
-    );
-  }
+  const saveDataManagerMethod = mod[saveDataManagerMethodName] as
+    | AnyFunction
+    | undefined;
+  assertDefined(
+    saveDataManagerMethod,
+    'Failed to initialize a mod feature class due to having a "v" object and not having the save data manager initialized. You must pass "ISCFeature.SAVE_DATA_MANAGER" to the "upgradeMod" function.',
+  );
 
   if (typeof saveDataManagerMethod !== "function") {
     error(

package/src/classes/ModUpgraded.ts

@@ -11,6 +11,7 @@ import {
   getTSTLClassConstructor,
   getTSTLClassName,
 } from "../functions/tstlClass";
+import { assertDefined } from "../functions/utils";
 import type { AddCallbackParametersCustom } from "../interfaces/private/AddCallbackParametersCustom";
 import type { ModUpgradedInterface } from "../interfaces/private/ModUpgradedInterface";
 import type { AnyFunction } from "../types/AnyFunction";
@@ -234,11 +235,10 @@ export class ModUpgraded implements Mod {
       }
 
       const modCallbackCustom = tonumber(modCallbackCustomString);
-      if (modCallbackCustom === undefined) {
-        error(
-          "Failed to convert an index on the callbacks object to a number.",
-        );
-      }
+      assertDefined(
+        modCallbackCustom,
+        "Failed to convert an index on the callbacks object to a number.",
+      );
 
       log(
         `- ModCallbackCustom.${ModCallbackCustom[modCallbackCustom]} (${modCallbackCustom})`,
@@ -254,9 +254,10 @@ export class ModUpgraded implements Mod {
       }
 
       const iscFeature = tonumber(iscFeatureString);
-      if (iscFeature === undefined) {
-        error("Failed to convert an index on the features object to a number.");
-      }
+      assertDefined(
+        iscFeature,
+        "Failed to convert an index on the features object to a number.",
+      );
 
       log(`- ISCFeature.${ISCFeature[iscFeature]} (${iscFeature})`);
     }
@@ -324,9 +325,8 @@ export class ModUpgraded implements Mod {
 
     if (feature.v !== undefined) {
       const className = getTSTLClassName(feature);
-      if (className === undefined) {
-        error("Failed to get the name of a feature.");
-      }
+      assertDefined(className, "Failed to get the name of a feature.");
+
       const saveDataManagerClass = this.features[ISCFeature.SAVE_DATA_MANAGER];
       saveDataManagerClass.saveDataManager(
         className,
@@ -387,9 +387,8 @@ export class ModUpgraded implements Mod {
 
     if (feature.v !== undefined) {
       const className = getTSTLClassName(feature);
-      if (className === undefined) {
-        error("Failed to get the name of a feature.");
-      }
+      assertDefined(className, "Failed to get the name of a feature.");
+
       const saveDataManagerClass = this.features[ISCFeature.SAVE_DATA_MANAGER];
       saveDataManagerClass.saveDataManagerRemove(className);
     }
@@ -430,6 +429,9 @@ function getExportedMethodsFromFeature(featureClass: unknown): FunctionTuple[] {
 
   return exportedMethodNames.map((name) => {
     const featureClassRecord = featureClass as Record<string, AnyFunction>;
+
+    // We cannot split out the method to a separate variable or else the "self" parameter will not
+    // be properly passed to the method.
     if (featureClassRecord[name] === undefined) {
       error(`Failed to find a decorated exported method: ${name}`);
     }
@@ -437,8 +439,8 @@ function getExportedMethodsFromFeature(featureClass: unknown): FunctionTuple[] {
     // In order for "this" to work properly in the method, we have to wrap the method invocation in
     // an arrow function.
     const wrappedMethod = (...args: unknown[]) =>
-      // We cannot split out the method to a separate variable or else the "self" parameter will not
-      // be properly passed to the method.
+      // We use a non-null assertion since we have already validated that the function exists. (See
+      // the above comment.)
       // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
       featureClassRecord[name]!(...args);
 

package/src/classes/callbacks/PostNewRoomEarly.ts

@@ -68,7 +68,7 @@ export class PostNewRoomEarly extends CustomCallback<ModCallbackCustom.POST_NEW_
       topLeftWall = spawnGridEntity(GridEntityType.WALL, topLeftWallGridIndex);
       if (topLeftWall === undefined) {
         logError(
-          "Failed to spawn a new wall (1) for the POST_NEW_ROOM_EARLY callback.",
+          "Failed to spawn a new wall for the POST_NEW_ROOM_EARLY callback (on the first try).",
         );
         return false;
       }
@@ -83,7 +83,7 @@ export class PostNewRoomEarly extends CustomCallback<ModCallbackCustom.POST_NEW_
       );
       if (topLeftWall2 === undefined) {
         logError(
-          "Failed to spawn a new wall (2) for the POST_NEW_ROOM_EARLY callback.",
+          "Failed to spawn a new wall for the POST_NEW_ROOM_EARLY callback (on the second try).",
         );
         return false;
       }

package/src/classes/features/callbackLogic/CustomGridEntities.ts

@@ -16,6 +16,7 @@ import {
 import { getPlayerFromPtr } from "../../../functions/players";
 import { getRoomListIndex } from "../../../functions/roomData";
 import { isNumber } from "../../../functions/types";
+import { assertDefined } from "../../../functions/utils";
 import { isVector } from "../../../functions/vector";
 import type { GridEntityCustomData } from "../../../interfaces/GridEntityCustomData";
 import { DefaultMap } from "../../DefaultMap";
@@ -245,9 +246,7 @@ export class CustomGridEntities extends Feature {
       baseGridEntityVariant,
       gridIndexOrPosition,
     );
-    if (customGridEntity === undefined) {
-      error("Failed to spawn a custom grid entity.");
-    }
+    assertDefined(customGridEntity, "Failed to spawn a custom grid entity.");
 
     if (gridCollisionClass !== undefined) {
       customGridEntity.CollisionClass = gridCollisionClass;

package/src/classes/features/other/CharacterHealthConversion.ts

@@ -70,7 +70,7 @@ export class CharacterHealthConversion extends Feature {
     const conversionHeartSubType =
       this.characterHealthReplacementMap.get(character);
     if (conversionHeartSubType === undefined) {
-      return;
+      return undefined;
     }
 
     convertRedHeartContainers(player, conversionHeartSubType);

package/src/classes/features/other/CustomItemPools.ts

@@ -5,6 +5,7 @@ import { ModCallbackCustom } from "../../../enums/ModCallbackCustom";
 import { arrayRemoveIndexInPlace } from "../../../functions/array";
 import { copyMap } from "../../../functions/map";
 import { getRandomSeed } from "../../../functions/rng";
+import { assertDefined } from "../../../functions/utils";
 import { getRandomIndexFromWeightedArray } from "../../../functions/weighted";
 import type { WeightedArray } from "../../../types/WeightedArray";
 import { Feature } from "../../private/Feature";
@@ -116,9 +117,10 @@ export class CustomItemPools extends Feature {
     defaultItem = CollectibleType.NULL,
   ): CollectibleType {
     const customItemPool = v.run.customItemPools.get(itemPoolTypeCustom);
-    if (customItemPool === undefined) {
-      error(`Failed to find the custom item pool of: ${itemPoolTypeCustom}`);
-    }
+    assertDefined(
+      customItemPool,
+      `Failed to find the custom item pool of: ${itemPoolTypeCustom}`,
+    );
 
     if (customItemPool.length === 0) {
       return defaultItem;
@@ -129,11 +131,10 @@ export class CustomItemPools extends Feature {
       seedOrRNG,
     );
     const tuple = customItemPool[randomIndex];
-    if (tuple === undefined) {
-      error(
-        `Failed to get an element from a custom item pool using a random index of: ${randomIndex}`,
-      );
-    }
+    assertDefined(
+      tuple,
+      `Failed to get an element from a custom item pool using a random index of: ${randomIndex}`,
+    );
 
     if (decrease) {
       arrayRemoveIndexInPlace(customItemPool, randomIndex);

package/src/classes/features/other/CustomStages.ts

@@ -31,6 +31,7 @@ import {
 import { getMusicForStage } from "../../../functions/sound";
 import { setStage } from "../../../functions/stage";
 import { asNumber } from "../../../functions/types";
+import { assertDefined } from "../../../functions/utils";
 import type {
   CustomStageLua,
   CustomStageRoomMetadata,
@@ -201,11 +202,10 @@ export class CustomStages extends Feature {
     destinationStage: LevelStage,
     _destinationStageType: StageType,
   ) => {
-    if (destinationName === undefined) {
-      error(
-        "Failed to go to a custom stage since the custom trapdoors feature did not pass a destination name to the logic function.",
-      );
-    }
+    assertDefined(
+      destinationName,
+      "Failed to go to a custom stage since the custom trapdoors feature did not pass a destination name to the logic function.",
+    );
 
     const firstFloor = destinationStage === LevelStage.BASEMENT_1;
     this.setCustomStage(destinationName, firstFloor);
@@ -498,11 +498,10 @@ export class CustomStages extends Feature {
     verbose = false,
   ): void {
     const customStage = this.customStagesMap.get(name);
-    if (customStage === undefined) {
-      error(
-        `Failed to set the custom stage of "${name}" because it was not found in the custom stages map. (Try restarting IsaacScript / recompiling the mod / restarting the game, and try again. If that does not work, you probably forgot to define it in your "tsconfig.json" file.) See the website for more details on how to set up custom stages.`,
-      );
-    }
+    assertDefined(
+      customStage,
+      `Failed to set the custom stage of "${name}" because it was not found in the custom stages map. (Try restarting IsaacScript / recompiling the mod / restarting the game, and try again. If that does not work, you probably forgot to define it in your "tsconfig.json" file.) See the website for more details on how to set up custom stages.`,
+    );
 
     const level = game.GetLevel();
     const stage = level.GetStage();

package/src/classes/features/other/CustomTrapdoors.ts

@@ -37,6 +37,7 @@ import {
 import { teleport } from "../../../functions/roomTransition";
 import { setStage } from "../../../functions/stage";
 import { getTSTLClassName } from "../../../functions/tstlClass";
+import { assertDefined } from "../../../functions/utils";
 import { isVector } from "../../../functions/vector";
 import type { CustomTrapdoorDescription } from "../../../interfaces/private/CustomTrapdoorDescription";
 import { ReadonlySet } from "../../../types/ReadonlySet";
@@ -365,11 +366,10 @@ export class CustomTrapdoors extends Feature {
     this.logStateChanged();
 
     const tstlClassName = getTSTLClassName(this);
-    if (tstlClassName === undefined) {
-      error(
-        "Failed to find get the class name for the custom trapdoor feature.",
-      );
-    }
+    assertDefined(
+      tstlClassName,
+      "Failed to find get the class name for the custom trapdoor feature.",
+    );
 
     this.disableInputs.enableAllInputs(tstlClassName);
   }
@@ -506,11 +506,10 @@ export class CustomTrapdoors extends Feature {
     this.logStateChanged();
 
     const tstlClassName = getTSTLClassName(this);
-    if (tstlClassName === undefined) {
-      error(
-        "Failed to find get the class name for the custom trapdoor feature.",
-      );
-    }
+    assertDefined(
+      tstlClassName,
+      "Failed to find get the class name for the custom trapdoor feature.",
+    );
 
     // We don't want to allow pausing, since that will allow render frames to pass without advancing
     // the stage traveling logic. (We track how many render frames have passed to know when to move

package/src/classes/features/other/DeployJSONRoom.ts

@@ -28,6 +28,7 @@ import { getRandomSeed, isRNG, newRNG } from "../../../functions/rng";
 import { gridCoordinatesToWorldPosition } from "../../../functions/roomGrid";
 import { setRoomCleared, setRoomUncleared } from "../../../functions/rooms";
 import { asCollectibleType, asNumber } from "../../../functions/types";
+import { assertDefined } from "../../../functions/utils";
 import type { JSONRoom } from "../../../interfaces/JSONRoomsFile";
 import { ReadonlySet } from "../../../types/ReadonlySet";
 import { Feature } from "../../private/Feature";
@@ -68,41 +69,40 @@ export class DeployJSONRoom extends Feature {
     for (const jsonSpawn of jsonRoom.spawn) {
       const xString = jsonSpawn.$.x;
       const x = tonumber(xString);
-      if (x === undefined) {
-        error(
-          `Failed to convert the following x coordinate to a number (for a spawn): ${xString}`,
-        );
-      }
+      assertDefined(
+        x,
+        `Failed to convert the following x coordinate to a number (for a spawn): ${xString}`,
+      );
 
       const yString = jsonSpawn.$.y;
       const y = tonumber(yString);
-      if (y === undefined) {
-        error(
-          `Failed to convert the following y coordinate to a number (for a spawn): ${yString}`,
-        );
-      }
+      assertDefined(
+        y,
+        `Failed to convert the following y coordinate to a number (for a spawn): ${yString}`,
+      );
 
       const jsonEntity = getRandomJSONEntity(jsonSpawn.entity);
 
       const entityTypeString = jsonEntity.$.type;
       const entityTypeNumber = tonumber(entityTypeString);
-      if (entityTypeNumber === undefined) {
-        error(
-          `Failed to convert the entity type to a number: ${entityTypeString}`,
-        );
-      }
+      assertDefined(
+        entityTypeNumber,
+        `Failed to convert the entity type to a number: ${entityTypeString}`,
+      );
 
       const variantString = jsonEntity.$.variant;
       const variant = tonumber(variantString);
-      if (variant === undefined) {
-        error(`Failed to convert the entity variant to a number: ${variant}`);
-      }
+      assertDefined(
+        variant,
+        `Failed to convert the entity variant to a number: ${variant}`,
+      );
 
       const subTypeString = jsonEntity.$.subtype;
       const subType = tonumber(subTypeString);
-      if (subType === undefined) {
-        error(`Failed to convert the entity sub-type to a number: ${subType}`);
-      }
+      assertDefined(
+        subType,
+        `Failed to convert the entity sub-type to a number: ${subType}`,
+      );
 
       const isGridEntity = GRID_ENTITY_XML_TYPE_SET.has(
         entityTypeNumber as GridEntityXMLType,

package/src/classes/features/other/ModdedElementSets.ts

@@ -40,7 +40,7 @@ import {
   getVanillaTrinketTypeRange,
   trinketHasCacheFlag,
 } from "../../../functions/trinkets";
-import { repeat } from "../../../functions/utils";
+import { assertDefined, repeat } from "../../../functions/utils";
 import { ITEM_CONFIG_CARD_TYPES_FOR_CARDS_SET } from "../../../sets/itemConfigCardTypesForCardsSet";
 import { ReadonlyMap } from "../../../types/ReadonlyMap";
 import { ReadonlySet } from "../../../types/ReadonlySet";
@@ -440,11 +440,11 @@ export class ModdedElementSets extends Feature {
       if (itemConfigCardType !== undefined) {
         const cardTypeSet =
           this.itemConfigCardTypeToCardTypeMap.get(itemConfigCardType);
-        if (cardTypeSet === undefined) {
-          error(
-            `Failed to get the card set for item config card type: ${itemConfigCardType}`,
-          );
-        }
+        assertDefined(
+          cardTypeSet,
+          `Failed to get the card set for item config card type: ${itemConfigCardType}`,
+        );
+
         cardTypeSet.add(cardType);
 
         if (ITEM_CONFIG_CARD_TYPES_FOR_CARDS_SET.has(itemConfigCardType)) {
@@ -515,11 +515,10 @@ export class ModdedElementSets extends Feature {
     for (const itemConfigCardType of itemConfigCardTypes) {
       const cardTypeSet =
         this.itemConfigCardTypeToCardTypeMap.get(itemConfigCardType);
-      if (cardTypeSet === undefined) {
-        error(
-          `Failed to get the card type set for item config type: ${itemConfigCardType}`,
-        );
-      }
+      assertDefined(
+        cardTypeSet,
+        `Failed to get the card type set for item config type: ${itemConfigCardType}`,
+      );
 
       for (const cardType of cardTypeSet) {
         matchingCardTypes.add(cardType);
@@ -588,11 +587,10 @@ export class ModdedElementSets extends Feature {
     playerForm: PlayerForm,
   ): ReadonlySet<CollectibleType> {
     const itemConfigTag = TRANSFORMATION_TO_TAG_MAP.get(playerForm);
-    if (itemConfigTag === undefined) {
-      error(
-        `Failed to get the collectible types for the transformation of ${playerForm} because that transformation is not based on collectibles.`,
-      );
-    }
+    assertDefined(
+      itemConfigTag,
+      `Failed to get the collectible types for the transformation of ${playerForm} because that transformation is not based on collectibles.`,
+    );
 
     return this.getCollectiblesWithTag(itemConfigTag);
   }
@@ -648,11 +646,10 @@ export class ModdedElementSets extends Feature {
     this.lazyInitTagToCollectibleTypesMap();
 
     const collectibleTypes = this.tagToCollectibleTypesMap.get(itemConfigTag);
-    if (collectibleTypes === undefined) {
-      error(
-        `The item config tag of ${itemConfigTag} is not a valid value of the "ItemConfigTag" enum.`,
-      );
-    }
+    assertDefined(
+      collectibleTypes,
+      `The item config tag of ${itemConfigTag} is not a valid value of the "ItemConfigTag" enum.`,
+    );
 
     return collectibleTypes;
   }

package/src/classes/features/other/Pause.ts

@@ -17,6 +17,7 @@ import { logError } from "../../../functions/logMisc";
 import { getAllPlayers } from "../../../functions/playerIndex";
 import { useActiveItemTemp } from "../../../functions/players";
 import { getTSTLClassName } from "../../../functions/tstlClass";
+import { assertDefined } from "../../../functions/utils";
 import { ReadonlySet } from "../../../types/ReadonlySet";
 import { Feature } from "../../private/Feature";
 import type { DisableInputs } from "./DisableInputs";
@@ -170,9 +171,10 @@ export class Pause extends Feature {
     useActiveItemTemp(firstPlayer, CollectibleType.PAUSE);
 
     const tstlClassName = getTSTLClassName(this);
-    if (tstlClassName === undefined) {
-      error("Failed to get the class name for the pause feature.");
-    }
+    assertDefined(
+      tstlClassName,
+      "Failed to get the class name for the pause feature.",
+    );
 
     const whitelist = new ReadonlySet([
       ButtonAction.MENU_CONFIRM,
@@ -210,9 +212,10 @@ export class Pause extends Feature {
     v.run.shouldUnpause = true;
 
     const tstlClassName = getTSTLClassName(this);
-    if (tstlClassName === undefined) {
-      error("Failed to find get the class name for the pause feature.");
-    }
+    assertDefined(
+      tstlClassName,
+      "Failed to find get the class name for the pause feature.",
+    );
 
     this.disableInputs.enableAllInputs(tstlClassName);
     for (const player of getAllPlayers()) {

package/src/classes/features/other/SaveDataManager.ts

@@ -10,6 +10,7 @@ import { log } from "../../../functions/log";
 import { onFirstFloor } from "../../../functions/stage";
 import { getTSTLClassName, isTSTLClass } from "../../../functions/tstlClass";
 import { isString, isTable } from "../../../functions/types";
+import { assertDefined } from "../../../functions/utils";
 import type { SaveData } from "../../../interfaces/SaveData";
 import type { AnyClass } from "../../../types/AnyClass";
 import { ReadonlySet } from "../../../types/ReadonlySet";
@@ -332,11 +333,10 @@ export class SaveDataManager extends Feature {
   ): void {
     if (isTSTLClass(key)) {
       const className = getTSTLClassName(key);
-      if (className === undefined) {
-        error(
-          'Failed to get the class name for the submitted class (as part of the "key" parameter) when registering new data with the save data manager.',
-        );
-      }
+      assertDefined(
+        className,
+        'Failed to get the class name for the submitted class (as part of the "key" parameter) when registering new data with the save data manager.',
+      );
 
       key = className;
     }
@@ -459,12 +459,11 @@ export class SaveDataManager extends Feature {
   public saveDataManagerRegisterClass(...tstlClasses: AnyClass[]): void {
     for (const tstlClass of tstlClasses) {
       const { name } = tstlClass;
-      // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
-      if (name === undefined) {
-        error(
-          "Failed to register a class with the save data manager due to not being able to derive the name of the class.",
-        );
-      }
+      assertDefined(
+        // Since we are accepting untrusted user input, this might nto be a real TSTL class.
+        name as string | undefined,
+        "Failed to register a class with the save data manager due to not being able to derive the name of the class.",
+      );
 
       this.classConstructors.set(name, tstlClass);
     }
@@ -528,11 +527,10 @@ export class SaveDataManager extends Feature {
     }
 
     const saveData = this.saveDataMap.get(key);
-    if (saveData === undefined) {
-      error(
-        `The save data manager is not managing save data for a key of: ${key}`,
-      );
-    }
+    assertDefined(
+      saveData,
+      `The save data manager is not managing save data for a key of: ${key}`,
+    );
 
     restoreDefaultForFeatureKey(
       this.saveDataDefaultsMap,

package/src/classes/features/other/customStages/backdrop.ts

@@ -19,7 +19,7 @@ import {
   removeCharactersBefore,
   trimPrefix,
 } from "../../../../functions/string";
-import { eRange, iRange } from "../../../../functions/utils";
+import { assertDefined, eRange, iRange } from "../../../../functions/utils";
 import type { CustomStage } from "../../../../interfaces/private/CustomStage";
 import { ReadonlySet } from "../../../../types/ReadonlySet";
 import { ISAACSCRIPT_CUSTOM_STAGE_GFX_PATH } from "./constants";
@@ -149,11 +149,10 @@ function spawnWallEntity(
     ? ROOM_SHAPE_WALL_EXTRA_ANM2_LAYERS
     : ROOM_SHAPE_WALL_ANM2_LAYERS;
   const numWallLayers = wallLayersArray[roomShape];
-  if (numWallLayers === undefined) {
-    error(
-      `Failed to get the layers when creating the backdrop for custom stage: ${customStage.name}`,
-    );
-  }
+  assertDefined(
+    numWallLayers,
+    `Failed to get the layers when creating the backdrop for custom stage: ${customStage.name}`,
+  );
 
   if (isLRoomShape(roomShape)) {
     const cornerPNGPath = getBackdropPNGPath(

package/src/classes/features/other/saveDataManager/restoreDefaults.ts

@@ -80,7 +80,7 @@ export function restoreDefaultForFeatureKey(
   const childTableDefaults = saveDataDefaults[saveDataKey];
   if (childTableDefaults === undefined) {
     logError(
-      `Failed to find the default copy of the child table "${saveDataKey}" for subscriber "${subscriberName}". This error usually means that your save data is out of date. You can try purging all of your save data by deleting the following directory: C:\\Program Files (x86)\\Steam\\steamapps\\common\\The Binding of Isaac Rebirth\\data`,
+      `Failed to find the default copy of the child table "${saveDataKey}" for subscriber "${subscriberName}". This error usually means that your mod-specific save data is out of date. You can try purging all of your mod-specific save data by deleting the following directory: C:\\Program Files (x86)\\Steam\\steamapps\\common\\The Binding of Isaac Rebirth\\data`,
     );
     return;
   }