isaacscript-common 87.5.0 → 87.6.0

package/src/functions/deepCopyTests.ts

@@ -203,21 +203,21 @@ function copiedObjectHasNoReferencesForArray() {
     error("The copied object does not have an equal array.");
   }
 
-  oldObject.abc[0]++;
+  oldObject.abc[0]!++; // eslint-disable-line @typescript-eslint/no-non-null-assertion
   if (arrayEquals(oldObject.abc, newObject.abc)) {
     error(
       "The copied object has an equal array after a modification to the old array.",
     );
   }
-  oldObject.abc[0]--;
+  oldObject.abc[0]!--; // eslint-disable-line @typescript-eslint/no-non-null-assertion
 
-  newObject.abc[0]++;
+  newObject.abc[0]!++; // eslint-disable-line @typescript-eslint/no-non-null-assertion
   if (arrayEquals(oldObject.abc, newObject.abc)) {
     error(
       "The copied object has an equal array after a modification to the new array.",
     );
   }
-  newObject.abc[0]--;
+  newObject.abc[0]!--; // eslint-disable-line @typescript-eslint/no-non-null-assertion
 }
 
 function copiedObjectHasChildObject() {

package/src/functions/doors.ts

@@ -326,9 +326,9 @@ export function getUnusedDoorSlots(): readonly DoorSlot[] {
   return DOOR_SLOT_VALUES.filter(
     (doorSlot) =>
       // We need to filter out the -1 value to prevent crashes.
-      doorSlot !== DoorSlot.NO_DOOR_SLOT &&
-      room.IsDoorSlotAllowed(doorSlot) &&
-      room.GetDoor(doorSlot) === undefined,
+      doorSlot !== DoorSlot.NO_DOOR_SLOT
+      && room.IsDoorSlotAllowed(doorSlot)
+      && room.GetDoor(doorSlot) === undefined,
   );
 }
 
@@ -377,6 +377,7 @@ export function isAngelRoomDoor(door: GridEntityDoor): boolean {
  * Room.)
  */
 export function isBlueWombDoor(door: GridEntityDoor): boolean {
+  // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
   return door.TargetRoomIndex === GridRoom.BLUE_WOMB;
 }
 
@@ -385,6 +386,7 @@ export function isBlueWombDoor(door: GridEntityDoor): boolean {
  * vanilla, the door will only appear in the Boss Room of the sixth floor.)
  */
 export function isBossRushDoor(door: GridEntityDoor): boolean {
+  // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
   return door.TargetRoomIndex === GridRoom.BOSS_RUSH;
 }
 
@@ -493,6 +495,7 @@ export function isHiddenSecretRoomDoor(door: GridEntityDoor): boolean {
  * (In vanilla, the door will only appear in the starting room of The Chest / Dark Room.)
  */
 export function isMegaSatanDoor(door: GridEntityDoor): boolean {
+  // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
   return door.TargetRoomIndex === GridRoom.MEGA_SATAN;
 }
 
@@ -501,6 +504,7 @@ export function isMegaSatanDoor(door: GridEntityDoor): boolean {
  * you to the Repentance floor.
  */
 export function isRepentanceDoor(door: GridEntityDoor): boolean {
+  // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
   return door.TargetRoomIndex === GridRoom.SECRET_EXIT;
 }
 
@@ -525,6 +529,7 @@ export function isSecretRoomDoor(door: GridEntityDoor): boolean {
  * contains the portal to The Void. (In vanilla, the door will only appear in the Hush Boss Room.)
  */
 export function isVoidDoor(door: GridEntityDoor): boolean {
+  // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
   return door.TargetRoomIndex === GridRoom.VOID;
 }
 

package/src/functions/easing.ts

@@ -158,8 +158,8 @@ export function easeInOutElastic(time: number): number {
       ? 1
       : time < 0.5
         ? -(2 ** (20 * time - 10) * Math.sin((20 * time - 11.125) * c5)) / 2
-        : (2 ** (-20 * time + 10) * Math.sin((20 * time - 11.125) * c5)) / 2 +
-          1;
+        : (2 ** (-20 * time + 10) * Math.sin((20 * time - 11.125) * c5)) / 2
+          + 1;
 }
 
 /**

package/src/functions/emptyRoom.ts

@@ -44,9 +44,9 @@ function emptyRoomEntities() {
     }
 
     if (
-      entity.HasEntityFlags(EntityFlag.CHARM) ||
-      entity.HasEntityFlags(EntityFlag.FRIENDLY) ||
-      entity.HasEntityFlags(EntityFlag.PERSISTENT)
+      entity.HasEntityFlags(EntityFlag.CHARM)
+      || entity.HasEntityFlags(EntityFlag.FRIENDLY)
+      || entity.HasEntityFlags(EntityFlag.PERSISTENT)
     ) {
       continue;
     }
@@ -74,8 +74,8 @@ export function emptyRoomGridEntities(): void {
     // We cannot simply check if the grid entity type is equal to a wall because other mods use
     // walls as a base for custom grid entities.
     if (
-      gridEntityType === GridEntityType.WALL &&
-      isVanillaWallGridIndex(gridIndex)
+      gridEntityType === GridEntityType.WALL
+      && isVanillaWallGridIndex(gridIndex)
     ) {
       continue;
     }

package/src/functions/entities.ts

@@ -149,8 +149,8 @@ export function getClosestEntityTo<T extends AnyEntity>(
     const distance = referenceEntity.Position.Distance(entity.Position);
 
     if (
-      distance < closestDistance &&
-      (filterFunc === undefined || filterFunc(entity))
+      distance < closestDistance
+      && (filterFunc === undefined || filterFunc(entity))
     ) {
       closestEntity = entity;
       closestDistance = distance;
@@ -425,6 +425,7 @@ export function isActiveEnemy(entity: Entity): boolean {
 
       // 912
       case EntityType.MOTHER: {
+        // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
         if (entity.Variant === MotherVariant.MOTHER_1) {
           const npc = entity.ToNPC();
           if (npc !== undefined && npc.State === NPCState.SPECIAL) {
@@ -441,7 +442,7 @@ export function isActiveEnemy(entity: Entity): boolean {
     }
   }
 
-  // eslint-disable-next-line deprecation/deprecation
+  // eslint-disable-next-line @typescript-eslint/no-deprecated
   return entity.IsActiveEnemy(false);
 }
 
@@ -478,9 +479,9 @@ export function parseEntityID(
   const [entityTypeString, variantString, subTypeString] = entityIDArray;
 
   if (
-    entityTypeString === undefined ||
-    variantString === undefined ||
-    subTypeString === undefined
+    entityTypeString === undefined
+    || variantString === undefined
+    || subTypeString === undefined
   ) {
     return undefined;
   }
@@ -490,9 +491,9 @@ export function parseEntityID(
   const subType = parseIntSafe(subTypeString);
 
   if (
-    entityType === undefined ||
-    variant === undefined ||
-    subType === undefined
+    entityType === undefined
+    || variant === undefined
+    || subType === undefined
   ) {
     return undefined;
   }
@@ -679,9 +680,7 @@ export function spawn(
     ? positionOrGridIndex
     : room.GetGridPosition(positionOrGridIndex);
 
-  if (seedOrRNG === undefined) {
-    seedOrRNG = newRNG();
-  }
+  seedOrRNG ??= newRNG();
 
   const seed = isRNG(seedOrRNG) ? seedOrRNG.Next() : seedOrRNG;
   return game.Spawn(

package/src/functions/enums.ts

@@ -230,7 +230,7 @@ export function getRandomEnumValue<T extends TranspiledEnum>(
  * `satisfies` operator with the `Record` type instead.
  */
 export function interfaceSatisfiesEnum<
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
+  // eslint-disable-next-line @typescript-eslint/no-unused-vars, @typescript-eslint/no-unnecessary-type-parameters
   T extends Record<Enum, unknown>,
   Enum extends string | number,
 >(): void {} // eslint-disable-line @typescript-eslint/no-empty-function
@@ -277,9 +277,9 @@ export function validateCustomEnum(
  *
  * This is useful to automate checking large enums for typos.
  */
-export function validateEnumContiguous<T extends TranspiledEnum>(
+export function validateEnumContiguous(
   transpiledEnumName: string,
-  transpiledEnum: T,
+  transpiledEnum: TranspiledEnum,
 ): void {
   const values = getEnumValues(transpiledEnum);
   const lastValue = values.at(-1);
@@ -296,7 +296,7 @@ export function validateEnumContiguous<T extends TranspiledEnum>(
 
   const valuesSet = new ReadonlySet(values);
   for (const value of iRange(lastValue)) {
-    if (!valuesSet.has(value as unknown as T[keyof T])) {
+    if (!valuesSet.has(value)) {
       error(
         `Failed to find a custom enum value of ${value} for: ${transpiledEnumName}`,
       );

package/src/functions/familiars.ts

@@ -146,8 +146,8 @@ export function getSirenHelper(familiar: EntityFamiliar): Entity | undefined {
   const sirenHelpers = getEntities(EntityType.SIREN_HELPER);
   return sirenHelpers.find(
     (sirenHelper) =>
-      sirenHelper.Target !== undefined &&
-      GetPtrHash(sirenHelper.Target) === familiarPtrHash,
+      sirenHelper.Target !== undefined
+      && GetPtrHash(sirenHelper.Target) === familiarPtrHash,
   );
 }
 

package/src/functions/flag.ts

@@ -115,7 +115,7 @@ export function hasFlag<T extends BitFlag | BitFlag128>(
  * (This is equivalent to checking if the flag is equal to 0, but this is not possible without
  * casting the flag to a number.)
  */
-export function isEmptyFlag<T extends BitFlag | BitFlag128>(flag: T): boolean {
+export function isEmptyFlag(flag: BitFlag | BitFlag128): boolean {
   return flag === 0;
 }
 
@@ -129,9 +129,9 @@ export function isSelfDamage(
 ): boolean {
   return (
     // Exclude self-damage from e.g. Curse Room door spikes.
-    hasFlag(damageFlags, DamageFlag.NO_PENALTIES) ||
+    hasFlag(damageFlags, DamageFlag.NO_PENALTIES)
     // Exclude self-damage from e.g. Razor.
-    hasFlag(damageFlags, DamageFlag.RED_HEARTS)
+    || hasFlag(damageFlags, DamageFlag.RED_HEARTS)
   );
 }
 

package/src/functions/globals.ts

@@ -220,8 +220,7 @@ export function getNewGlobals(): ReadonlyArray<[AnyNotNil, unknown]> {
   const newGlobals: Array<[AnyNotNil, unknown]> = [];
   for (const [key, value] of pairs(_G)) {
     if (!defaultGlobals.has(key)) {
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
-      const keyValueTuple: [AnyNotNil, unknown] = [key, value as any];
+      const keyValueTuple: [AnyNotNil, unknown] = [key, value];
       newGlobals.push(keyValueTuple);
     }
   }
@@ -256,7 +255,7 @@ export function setLogFunctionsGlobal(): void {
   const globals = _G as Record<string, unknown>;
 
   for (const exports of [logExports, logMiscExports, logEntitiesExports]) {
-    // eslint-disable-next-line isaacscript/no-object-any
+    // eslint-disable-next-line complete/no-object-any
     for (const [logFuncName, logFunc] of Object.entries(exports)) {
       globals[logFuncName] = logFunc;
     }

package/src/functions/gridEntities.ts

@@ -110,8 +110,8 @@ export function doesGridEntityExist(
     const thisGridEntityType = gridEntity.GetType();
     const thisVariant = gridEntity.GetVariant();
     return (
-      gridEntityType === thisGridEntityType &&
-      (variant === -1 || variant === thisVariant)
+      gridEntityType === thisGridEntityType
+      && (variant === -1 || variant === thisVariant)
     );
   });
 }
@@ -150,8 +150,8 @@ export function getCollidingEntitiesWithGridEntity(
 
   return closeEntities.filter(
     (entity) =>
-      entity.CollidesWithGrid() &&
-      isCircleIntersectingRectangle(
+      entity.CollidesWithGrid()
+      && isCircleIntersectingRectangle(
         entity.Position,
         // We arbitrarily add 0.1 to account for entities that are already pushed back by the time
         // the `POST_UPDATE` callback fires.
@@ -791,8 +791,8 @@ export function isGridEntityBreakableByExplosion(
   const gridEntityTypeVariant = `${gridEntityType}.${variant}`;
 
   return (
-    BREAKABLE_GRID_ENTITY_TYPES_BY_EXPLOSIONS.has(gridEntityType) ||
-    BREAKABLE_GRID_ENTITY_TYPES_VARIANTS_BY_EXPLOSIONS.has(
+    BREAKABLE_GRID_ENTITY_TYPES_BY_EXPLOSIONS.has(gridEntityType)
+    || BREAKABLE_GRID_ENTITY_TYPES_VARIANTS_BY_EXPLOSIONS.has(
       gridEntityTypeVariant,
     )
   );
@@ -816,7 +816,7 @@ export function isGridEntityBroken(gridEntity: GridEntity): boolean {
  * the `PRE_ROOM_ENTITY_SPAWN` callback for narrowing the type of the first argument.
  */
 export function isGridEntityXMLType(num: number): num is GridEntityXMLType {
-  return GRID_ENTITY_XML_TYPES_SET.has(num); // eslint-disable-line isaacscript/strict-enums
+  return GRID_ENTITY_XML_TYPES_SET.has(num); // eslint-disable-line complete/strict-enums
 }
 
 /**
@@ -860,9 +860,10 @@ export function isPostBossVoidPortal(gridEntity: GridEntity): boolean {
   const saveState = gridEntity.GetSaveState();
 
   return (
-    saveState.Type === GridEntityType.TRAPDOOR &&
-    saveState.Variant === TrapdoorVariant.VOID_PORTAL &&
-    saveState.VarData === 1
+    saveState.Type === GridEntityType.TRAPDOOR
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
+    && saveState.Variant === TrapdoorVariant.VOID_PORTAL
+    && saveState.VarData === 1
   );
 }
 
@@ -953,8 +954,8 @@ export function removeEntitiesSpawnedFromGridEntity(
 ): void {
   const entitiesFromGridEntity = entities.filter(
     (entity) =>
-      entity.FrameCount === 0 &&
-      vectorEquals(entity.Position, gridEntity.Position),
+      entity.FrameCount === 0
+      && vectorEquals(entity.Position, gridEntity.Position),
   );
   removeEntities(entitiesFromGridEntity);
 }
@@ -1042,6 +1043,7 @@ export function removeGridEntity(
   // corresponding effect.
   if (gridEntityType === GridEntityType.STATUE) {
     const effectVariant =
+      // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
       variant === StatueVariant.DEVIL
         ? EffectVariant.DEVIL
         : EffectVariant.ANGEL;
@@ -1145,18 +1147,22 @@ export function spawnGiantPoop(topLeftGridIndex: int): boolean {
   );
 
   return (
-    topLeft !== undefined &&
-    topLeft.GetType() === GridEntityType.POOP &&
-    topLeft.GetVariant() === PoopGridEntityVariant.GIANT_TOP_LEFT &&
-    topRight !== undefined &&
-    topRight.GetType() === GridEntityType.POOP &&
-    topRight.GetVariant() === PoopGridEntityVariant.GIANT_TOP_RIGHT &&
-    bottomLeft !== undefined &&
-    bottomLeft.GetType() === GridEntityType.POOP &&
-    bottomLeft.GetVariant() === PoopGridEntityVariant.GIANT_BOTTOM_LEFT &&
-    bottomRight !== undefined &&
-    bottomRight.GetType() === GridEntityType.POOP &&
-    bottomRight.GetVariant() === PoopGridEntityVariant.GIANT_BOTTOM_RIGHT
+    topLeft !== undefined
+    && topLeft.GetType() === GridEntityType.POOP
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
+    && topLeft.GetVariant() === PoopGridEntityVariant.GIANT_TOP_LEFT
+    && topRight !== undefined
+    && topRight.GetType() === GridEntityType.POOP
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
+    && topRight.GetVariant() === PoopGridEntityVariant.GIANT_TOP_RIGHT
+    && bottomLeft !== undefined
+    && bottomLeft.GetType() === GridEntityType.POOP
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
+    && bottomLeft.GetVariant() === PoopGridEntityVariant.GIANT_BOTTOM_LEFT
+    && bottomRight !== undefined
+    && bottomRight.GetType() === GridEntityType.POOP
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
+    && bottomRight.GetVariant() === PoopGridEntityVariant.GIANT_BOTTOM_RIGHT
   );
 }
 

package/src/functions/gridEntitiesSpecific.ts

@@ -97,8 +97,8 @@ export function getPressurePlates(
     if (pressurePlate !== undefined) {
       const thisPressurePlateVariant = pressurePlate.GetVariant();
       if (
-        pressurePlateVariant === -1 ||
-        pressurePlateVariant === thisPressurePlateVariant
+        pressurePlateVariant === -1
+        || pressurePlateVariant === thisPressurePlateVariant
       ) {
         pressurePlates.push(pressurePlate);
       }

package/src/functions/isaacAPIClass.ts

@@ -83,8 +83,8 @@ export function isIsaacAPIClassOfType(
 ): boolean {
   const isaacAPIClassType = getIsaacAPIClassName(object);
   return (
-    isaacAPIClassType === classType ||
-    isaacAPIClassType === `const ${classType}`
+    isaacAPIClassType === classType
+    || isaacAPIClassType === `const ${classType}`
   );
 }
 

package/src/functions/jsonHelpers.ts

@@ -13,19 +13,21 @@ function tryEncode(this: void, luaTable: unknown) {
  * Converts a JSON string to a Lua table.
  *
  * In most cases, this function will be used for reading data from a "save#.dat" file. If decoding
- * fails, it will return a blank Lua table instead of throwing an error. (This allows execution to
- * continue in cases where users have no current save data or have manually removed their existing
- * save data.)
+ * fails, it will return undefined instead of throwing an error. (This allows execution to continue
+ * in cases where users have no current save data or have manually removed their existing save
+ * data.)
  *
  * Under the hood, this uses a custom JSON parser that was measured to be 11.8 times faster than the
  * vanilla JSON parser.
  */
-export function jsonDecode(jsonString: string): LuaMap<AnyNotNil, unknown> {
+export function jsonDecode(
+  jsonString: string,
+): LuaMap<AnyNotNil, unknown> | undefined {
   const [ok, luaTableOrErrMsg] = pcall(tryDecode, jsonString);
   if (!ok) {
     // Instead of throwing an error, continue execution of the callback.
     logError(`Failed to convert the JSON string to a Lua table: ${jsonString}`);
-    return new LuaMap();
+    return undefined;
   }
 
   return luaTableOrErrMsg;

package/src/functions/level.ts

@@ -44,9 +44,9 @@ export function getLevelBossIDs(): readonly BossID[] {
   const roomsInsideGrid = getRoomsInsideGrid();
 
   return filterMap(roomsInsideGrid, (roomDescriptor) =>
-    roomDescriptor.Data !== undefined &&
-    roomDescriptor.Data.Type === RoomType.BOSS &&
-    roomDescriptor.Data.StageID === StageID.SPECIAL_ROOMS
+    roomDescriptor.Data !== undefined
+    && roomDescriptor.Data.Type === RoomType.BOSS
+    && roomDescriptor.Data.StageID === StageID.SPECIAL_ROOMS
       ? roomDescriptor.Data.Subtype
       : undefined,
   );

package/src/functions/levelGrid.ts

@@ -247,10 +247,10 @@ export function getNewRoomCandidatesForLevel(
   // However, we want to filter out special rooms because they are supposed to be dead ends.
   const normalRooms = roomsInsideGrid.filter(
     (room) =>
-      room.Data !== undefined &&
-      room.Data.Type === RoomType.DEFAULT &&
-      !isMirrorRoom(room.Data) && // Mirror rooms do not count as special rooms.
-      !isMineShaft(room.Data), // Mineshaft rooms do not count as special rooms.
+      room.Data !== undefined
+      && room.Data.Type === RoomType.DEFAULT
+      && !isMirrorRoom(room.Data) // Mirror rooms do not count as special rooms.
+      && !isMineShaft(room.Data), // Mineshaft rooms do not count as special rooms.
   );
 
   const roomsToLookThrough = ensureDeadEnd ? normalRooms : roomsInsideGrid;
@@ -325,8 +325,8 @@ export function getRoomDescriptorsForType(
   const roomsInsideGrid = getRoomsInsideGrid();
   return roomsInsideGrid.filter(
     (roomDescriptor) =>
-      roomDescriptor.Data !== undefined &&
-      roomTypesSet.has(roomDescriptor.Data.Type),
+      roomDescriptor.Data !== undefined
+      && roomTypesSet.has(roomDescriptor.Data.Type),
   );
 }
 
@@ -531,9 +531,7 @@ export function isRedKeyRoom(roomGridIndex: int): boolean {
  * @param roomGridIndex Optional. Default is the current room index.
  */
 export function isRoomInsideGrid(roomGridIndex?: int): boolean {
-  if (roomGridIndex === undefined) {
-    roomGridIndex = getRoomGridIndex();
-  }
+  roomGridIndex ??= getRoomGridIndex();
 
   return roomGridIndex >= 0 && roomGridIndex <= MAX_LEVEL_GRID_INDEX;
 }
@@ -567,9 +565,8 @@ export function newRoom(
 ): int | undefined {
   const level = game.GetLevel();
 
-  if (seedOrRNG === undefined) {
-    seedOrRNG = level.GetDungeonPlacementSeed();
-  }
+  seedOrRNG ??= level.GetDungeonPlacementSeed();
+
   const rng = isRNG(seedOrRNG) ? seedOrRNG : newRNG(seedOrRNG);
 
   const newRoomCandidate = getNewRoomCandidate(rng, ensureDeadEnd);

package/src/functions/logEntities.ts

@@ -56,9 +56,9 @@ export function logAllEntities(
 
     const effect = entity.ToEffect();
     if (
-      !includeBackgroundEffects &&
-      effect !== undefined &&
-      IGNORE_EFFECT_VARIANTS.has(effect.Variant)
+      !includeBackgroundEffects
+      && effect !== undefined
+      && IGNORE_EFFECT_VARIANTS.has(effect.Variant)
     ) {
       continue;
     }
@@ -109,16 +109,16 @@ export function logAllGridEntities(
 
     // If a filter was specified, exclude all entities outside of the filter.
     if (
-      gridEntityTypeFilter !== undefined &&
-      gridEntityType !== gridEntityTypeFilter
+      gridEntityTypeFilter !== undefined
+      && gridEntityType !== gridEntityTypeFilter
     ) {
       continue;
     }
 
     if (
-      !includeWalls &&
-      gridEntityType === GridEntityType.WALL &&
-      gridEntityTypeFilter !== GridEntityType.WALL
+      !includeWalls
+      && gridEntityType === GridEntityType.WALL
+      && gridEntityTypeFilter !== GridEntityType.WALL
     ) {
       continue;
     }
@@ -222,8 +222,11 @@ function getEntityLogLine(this: void, entity: Entity, num?: int): string {
   msg += `  - Position: (${entity.Position.X}, ${entity.Position.Y})\n`;
   msg += `  - Velocity: (${entity.Velocity.X}, ${entity.Velocity.Y})\n`;
   msg += `  - HP: ${entity.HitPoints} / ${entity.MaxHitPoints}\n`;
+  // eslint-disable-next-line @typescript-eslint/no-base-to-string
   msg += `  - Parent: ${entity.Parent}\n`;
+  // eslint-disable-next-line @typescript-eslint/no-base-to-string
   msg += `  - Child: ${entity.Child}\n`;
+  // eslint-disable-next-line @typescript-eslint/no-base-to-string
   msg += `  - SpawnerEntity: ${entity.SpawnerEntity}\n`;
   msg += `  - SpawnerType / SpawnerVariant: ${entity.SpawnerType}.${entity.SpawnerVariant}\n`;
   msg += `  - FrameCount: ${entity.FrameCount}\n`;

package/src/functions/logMisc.ts

@@ -51,11 +51,13 @@ import { vectorToString } from "./vector";
  * @param array The array to log.
  * @param name Optional. The name of the array, which will be logged before the elements.
  */
-export function logArray<T>(
+export function logArray(
   this: void,
-  array: readonly T[],
+  array: readonly unknown[],
   name?: string,
 ): void {
+  name ??= "array";
+
   // We do not assume the given array has contiguous values in order to be more permissive about the
   // kinds of arrays that will successfully log without a run-time error.
   if (!isArray(array, false)) {
@@ -64,9 +66,6 @@ export function logArray<T>(
   }
 
   const arrayString = arrayToString(array);
-  if (name === undefined) {
-    name = "array";
-  }
   log(`Logging ${name}: ${arrayString}`);
 }
 
@@ -81,9 +80,7 @@ export function logCollectibleTypes(
   collectibleTypes: readonly CollectibleType[],
   name?: string,
 ): void {
-  if (name === undefined) {
-    name = "collectibles";
-  }
+  name ??= "collectibles";
 
   log(`Logging ${name}:`);
 
@@ -102,9 +99,7 @@ export function logCollectibleTypes(
  * @param name Optional. The name of the object, which will be logged before the properties.
  */
 export function logColor(this: void, color: Color, name?: string): void {
-  if (name === undefined) {
-    name = "color";
-  }
+  name ??= "color";
 
   log(
     `Logging ${name}: R${color.R}, G${color.G}, B${color.B}, A${color.A}, RO${color.RO}, BO${color.BO}, GO${color.GO}`,
@@ -199,9 +194,7 @@ export function logItemPoolTypes(
   itemPoolTypes: readonly ItemPoolType[],
   name?: string,
 ): void {
-  if (name === undefined) {
-    name = "item pool types";
-  }
+  name ??= "item pool types";
 
   log(`Logging ${name}:`);
 
@@ -220,9 +213,7 @@ export function logItemPoolTypes(
  * @param name Optional. The name of the object, which will be logged before the properties.
  */
 export function logKColor(this: void, kColor: KColor, name?: string): void {
-  if (name === undefined) {
-    name = "KColor";
-  }
+  name ??= "KColor";
 
   log(
     `Logging ${name}: R${kColor.Red}, G${kColor.Green}, B${kColor.Blue}, A${kColor.Alpha}`,
@@ -647,9 +638,7 @@ export function logVector(
   name?: string,
   round = false,
 ): void {
-  if (name === undefined) {
-    name = "vector";
-  }
+  name ??= "vector";
 
   const vectorString = vectorToString(vector, round);
   log(`Logging ${name}: ${vectorString}`);

package/src/functions/map.ts

@@ -2,7 +2,7 @@ import type { DefaultMap } from "../classes/DefaultMap";
 import { sumArray } from "./array";
 
 /** Helper function to copy a map. (You can also use a Map constructor to accomplish this task.) */
-// eslint-disable-next-line isaacscript/no-mutable-return
+// eslint-disable-next-line complete/no-mutable-return
 export function copyMap<K, V>(oldMap: ReadonlyMap<K, V>): Map<K, V> {
   const newMap = new Map<K, V>();
   for (const [key, value] of oldMap) {
@@ -33,7 +33,7 @@ export function defaultMapGetHash<V, A extends unknown[]>(
  * `mapSetHash` helper function.
  */
 export function defaultMapSetHash<V>(
-  // eslint-disable-next-line isaacscript/prefer-readonly-parameter-types
+  // eslint-disable-next-line complete/prefer-readonly-parameter-types
   map: Map<PtrHash, V>,
   entity: Entity,
   value: V,
@@ -79,7 +79,7 @@ export function getReversedMap<K, V>(
  * the map uses `PtrHash` as an index.
  */
 export function mapSetHash<V>(
-  // eslint-disable-next-line isaacscript/prefer-readonly-parameter-types
+  // eslint-disable-next-line complete/prefer-readonly-parameter-types
   map: Map<PtrHash, V>,
   entity: Entity,
   value: V,
@@ -101,7 +101,7 @@ export function mapSetHash<V>(
  *
  * Also see the `objectToReadonlyMap` function.
  */
-// eslint-disable-next-line isaacscript/no-mutable-return
+// eslint-disable-next-line complete/no-mutable-return
 export function objectToMap<K extends string | number | symbol, V>(
   object: Record<K, V>,
 ): Map<K, V> {

package/src/functions/math.ts

@@ -62,10 +62,10 @@ export function inRectangle(
   bottomRight: Vector,
 ): boolean {
   return (
-    position.X >= topLeft.X &&
-    position.X <= bottomRight.X &&
-    position.Y >= topLeft.Y &&
-    position.Y <= bottomRight.Y
+    position.X >= topLeft.X
+    && position.X <= bottomRight.X
+    && position.Y >= topLeft.Y
+    && position.Y <= bottomRight.Y
   );
 }