isaacscript-common 87.5.0 → 87.5.1

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/merge.ts

@@ -131,7 +131,7 @@ function mergeSerializedArray(
 }
 
 function mergeSerializedTSTLObject(
-  // eslint-disable-next-line isaacscript/prefer-readonly-parameter-types
+  // eslint-disable-next-line complete/prefer-readonly-parameter-types
   oldObject: Map<AnyNotNil, unknown> | Set<AnyNotNil>,
   newTable: LuaMap<AnyNotNil, unknown>,
   traversalDescription: string,

package/src/functions/minimap.ts

@@ -91,9 +91,7 @@ export function getRoomDisplayFlags(
   roomGridIndex?: int,
   minimapAPI = true,
 ): BitFlags<DisplayFlag> {
-  if (roomGridIndex === undefined) {
-    roomGridIndex = getRoomGridIndex();
-  }
+  roomGridIndex ??= getRoomGridIndex();
 
   if (MinimapAPI === undefined || !minimapAPI) {
     const roomDescriptor = getRoomDescriptor(roomGridIndex);
@@ -217,9 +215,7 @@ export function setRoomDisplayFlags(
   displayFlags: BitFlags<DisplayFlag>,
   updateVisibility = true,
 ): void {
-  if (roomGridIndex === undefined) {
-    roomGridIndex = getRoomGridIndex();
-  }
+  roomGridIndex ??= getRoomGridIndex();
 
   if (MinimapAPI === undefined) {
     const roomDescriptor = getRoomDescriptor(roomGridIndex);

package/src/functions/newArray.ts

@@ -16,7 +16,7 @@ import { repeat } from "./utils";
  * const arrayWithArrays = newArray([0], 20); // Has 20 elements of an array with a 0 in it.
  * ```
  */
-// eslint-disable-next-line isaacscript/no-mutable-return
+// eslint-disable-next-line complete/no-mutable-return
 export function newArray<T>(defaultValue: T, size: int): T[] {
   const array: T[] = [];
   repeat(size, () => {

package/src/functions/nextStage.ts

@@ -41,16 +41,19 @@ export function getNextStage(): LevelStage {
   // Second, handle the special case of being in a specific off-grid room.
   switch (roomGridIndex) {
     // -8
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
     case GridRoom.BLUE_WOMB: {
       return LevelStage.BLUE_WOMB;
     }
 
     // -9
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
     case GridRoom.VOID: {
       return LevelStage.VOID;
     }
 
     // -10
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
     case GridRoom.SECRET_EXIT: {
       if (repentanceStage) {
         // e.g. From Downpour 2 to Mines 1, etc.
@@ -149,6 +152,7 @@ export function getNextStageType(upwards = false): StageType {
   }
 
   // Second, handle the special case of being in a specific off-grid room.
+  // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
   if (roomGridIndex === GridRoom.SECRET_EXIT) {
     return calculateStageTypeRepentance(nextStage);
   }

package/src/functions/npcDataStructures.ts

@@ -41,7 +41,7 @@ export function defaultMapGetNPC<V, Args extends unknown[]>(
  * `mapSetNPC` helper function.
  */
 export function defaultMapSetNPC<V>(
-  // eslint-disable-next-line isaacscript/prefer-readonly-parameter-types
+  // eslint-disable-next-line complete/prefer-readonly-parameter-types
   map: Map<PtrHash, V>,
   npc: EntityNPC,
   value: V,
@@ -54,7 +54,7 @@ export function defaultMapSetNPC<V>(
  * `Map.delete` method if you have a set of this type.
  */
 export function mapDeleteNPC(
-  // eslint-disable-next-line isaacscript/prefer-readonly-parameter-types
+  // eslint-disable-next-line complete/prefer-readonly-parameter-types
   map: Map<PtrHash, unknown>,
   npc: EntityNPC,
 ): boolean {
@@ -123,7 +123,7 @@ export function mapHasNPC<V>(
  * ```
  */
 export function mapSetNPC<V>(
-  // eslint-disable-next-line isaacscript/prefer-readonly-parameter-types
+  // eslint-disable-next-line complete/prefer-readonly-parameter-types
   map: Map<PtrHash, V>,
   npc: EntityNPC,
   value: V,
@@ -136,7 +136,7 @@ export function mapSetNPC<V>(
  * Helper function to make using sets with an type of `PtrHash` easier. Use this instead of the
  * `Set.add` method if you have a set of this type.
  */
-// eslint-disable-next-line isaacscript/prefer-readonly-parameter-types
+// eslint-disable-next-line complete/prefer-readonly-parameter-types
 export function setAddNPC(set: Set<PtrHash>, npc: EntityNPC): void {
   const ptrHash = GetPtrHash(npc);
   set.add(ptrHash);
@@ -146,7 +146,7 @@ export function setAddNPC(set: Set<PtrHash>, npc: EntityNPC): void {
  * Helper function to make using sets with an type of `PtrHash` easier. Use this instead of the
  * `Set.delete` method if you have a set of this type.
  */
-// eslint-disable-next-line isaacscript/prefer-readonly-parameter-types
+// eslint-disable-next-line complete/prefer-readonly-parameter-types
 export function setDeleteNPC(set: Set<PtrHash>, npc: EntityNPC): boolean {
   const ptrHash = GetPtrHash(npc);
   return set.delete(ptrHash);

package/src/functions/npcs.ts

@@ -132,6 +132,7 @@ export function isDaddyLongLegsChildStompEntity(npc: EntityNPC): boolean {
 export function isDyingDump(npc: EntityNPC): boolean {
   return (
     npc.Type === EntityType.DUMP &&
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
     npc.Variant === DumpVariant.DUMP &&
     npc.State === NPCState.SPECIAL
   );
@@ -145,6 +146,7 @@ export function isDyingDump(npc: EntityNPC): boolean {
 export function isDyingEggyWithNoSpidersLeft(npc: EntityNPC): boolean {
   return (
     npc.Type === EntityType.HOPPER &&
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
     npc.Variant === HopperVariant.EGGY &&
     npc.State === NPCState.SUICIDE &&
     npc.StateFrame >= EGGY_STATE_FRAME_OF_FINAL_SPIDER
@@ -160,6 +162,7 @@ export function isDyingEggyWithNoSpidersLeft(npc: EntityNPC): boolean {
 export function isRaglingDeathPatch(npc: EntityNPC): boolean {
   return (
     npc.Type === EntityType.RAGLING &&
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
     npc.Variant === RaglingVariant.RAG_MANS_RAGLING &&
     // They go to `STATE_SPECIAL` when they are patches on the ground.
     npc.State === NPCState.SPECIAL

package/src/functions/playerCollectibles.ts

@@ -315,9 +315,7 @@ export function setActiveItem(
   const primaryCharge = player.GetActiveCharge(ActiveSlot.PRIMARY);
   const secondaryCollectibleType = player.GetActiveItem(ActiveSlot.SECONDARY);
 
-  if (charge === undefined) {
-    charge = getCollectibleMaxCharges(collectibleType);
-  }
+  charge ??= getCollectibleMaxCharges(collectibleType);
 
   if (!keepInPools) {
     itemPool.RemoveCollectible(collectibleType);

package/src/functions/playerDataStructures.ts

@@ -39,7 +39,7 @@ export function defaultMapGetPlayer<V, Args extends unknown[]>(
  * `mapSetPlayer` helper function.
  */
 export function defaultMapSetPlayer<V>(
-  // eslint-disable-next-line isaacscript/prefer-readonly-parameter-types
+  // eslint-disable-next-line complete/prefer-readonly-parameter-types
   map: Map<PlayerIndex, V>,
   player: EntityPlayer,
   value: V,
@@ -52,7 +52,7 @@ export function defaultMapSetPlayer<V>(
  * `Map.delete` method if you have a set of this type.
  */
 export function mapDeletePlayer(
-  // eslint-disable-next-line isaacscript/prefer-readonly-parameter-types
+  // eslint-disable-next-line complete/prefer-readonly-parameter-types
   map: Map<PlayerIndex, unknown>,
   player: EntityPlayer,
 ): boolean {
@@ -121,7 +121,7 @@ export function mapHasPlayer<V>(
  * ```
  */
 export function mapSetPlayer<V>(
-  // eslint-disable-next-line isaacscript/prefer-readonly-parameter-types
+  // eslint-disable-next-line complete/prefer-readonly-parameter-types
   map: Map<PlayerIndex, V>,
   player: EntityPlayer,
   value: V,
@@ -135,7 +135,7 @@ export function mapSetPlayer<V>(
  * `Set.add` method if you have a set of this type.
  */
 export function setAddPlayer(
-  // eslint-disable-next-line isaacscript/prefer-readonly-parameter-types
+  // eslint-disable-next-line complete/prefer-readonly-parameter-types
   set: Set<PlayerIndex>,
   player: EntityPlayer,
 ): void {
@@ -148,7 +148,7 @@ export function setAddPlayer(
  * `Set.delete` method if you have a set of this type.
  */
 export function setDeletePlayer(
-  // eslint-disable-next-line isaacscript/prefer-readonly-parameter-types
+  // eslint-disable-next-line complete/prefer-readonly-parameter-types
   set: Set<PlayerIndex>,
   player: EntityPlayer,
 ): boolean {

package/src/functions/positionVelocity.ts

@@ -112,9 +112,7 @@ export function findFreePosition(
 export function getEntityPositions(
   entities?: readonly Entity[],
 ): ReadonlyMap<PtrHash, Vector> {
-  if (entities === undefined) {
-    entities = getEntities();
-  }
+  entities ??= getEntities();
 
   const entityPositions = new Map<PtrHash, Vector>();
   for (const entity of entities) {
@@ -137,9 +135,7 @@ export function getEntityPositions(
 export function getEntityVelocities(
   entities?: readonly Entity[],
 ): ReadonlyMap<PtrHash, Vector> {
-  if (entities === undefined) {
-    entities = getEntities();
-  }
+  entities ??= getEntities();
 
   const entityVelocities = new Map<PtrHash, Vector>();
   for (const entity of entities) {
@@ -167,9 +163,7 @@ export function setEntityPositions(
   entityPositions: ReadonlyMap<PtrHash, Vector>,
   entities?: readonly Entity[],
 ): void {
-  if (entities === undefined) {
-    entities = getEntities();
-  }
+  entities ??= getEntities();
 
   for (const entity of entities) {
     const ptrHash = GetPtrHash(entity);
@@ -197,9 +191,7 @@ export function setEntityVelocities(
   entityVelocities: ReadonlyMap<PtrHash, Vector>,
   entities?: readonly Entity[],
 ): void {
-  if (entities === undefined) {
-    entities = getEntities();
-  }
+  entities ??= getEntities();
 
   for (const entity of entities) {
     const ptrHash = GetPtrHash(entity);

package/src/functions/rng.ts

@@ -17,7 +17,8 @@ export type SerializedRNG = LuaMap<string, unknown> & {
 /**
  * This is the ShiftIdx that Blade recommended after having reviewing the game's internal functions.
  * Any value between 0 and 80 should work equally well.
- * https://www.jstatsoft.org/article/view/v008i14/xorshift.pdf
+ *
+ * @see https://www.jstatsoft.org/article/view/v008i14/xorshift.pdf
  */
 const RECOMMENDED_SHIFT_IDX = 35;
 
@@ -63,7 +64,7 @@ export function deserializeRNG(rng: SerializedRNG): RNG {
  * the game.
  */
 export function getRandomSeed(): Seed {
-  const randomNumber = Random(); // eslint-disable-line deprecation/deprecation
+  const randomNumber = Random(); // eslint-disable-line @typescript-eslint/no-deprecated
   const safeRandomNumber = randomNumber === 0 ? 1 : randomNumber;
   return safeRandomNumber as Seed;
 }
@@ -171,6 +172,6 @@ export function setSeed(rng: RNG, seed: Seed): void {
   }
 
   // The game expects seeds in the range of 1 to 4294967295 (1^32 - 1).
-  // eslint-disable-next-line deprecation/deprecation
+  // eslint-disable-next-line @typescript-eslint/no-deprecated
   rng.SetSeed(seed, RECOMMENDED_SHIFT_IDX);
 }

package/src/functions/roomData.ts

@@ -65,9 +65,7 @@ export function getRoomData(roomGridIndex?: int): RoomConfig | undefined {
 export function getRoomDescriptor(roomGridIndex?: int): RoomDescriptor {
   const level = game.GetLevel();
 
-  if (roomGridIndex === undefined) {
-    roomGridIndex = getRoomGridIndex();
-  }
+  roomGridIndex ??= getRoomGridIndex();
 
   return level.GetRoomByIdx(roomGridIndex);
 }

package/src/functions/rooms.ts

@@ -600,6 +600,7 @@ export function isAllRoomsClear(
 export function isAngelShop(roomData: RoomConfig): boolean {
   return (
     roomData.Type === RoomType.ANGEL &&
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
     roomData.Subtype === AngelRoomSubType.SHOP
   );
 }
@@ -615,6 +616,7 @@ export function isAngelShop(roomData: RoomConfig): boolean {
 export function isBeastRoom(roomData: RoomConfig): boolean {
   return (
     roomData.Type === RoomType.DUNGEON &&
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
     roomData.Subtype === DungeonSubType.BEAST_ROOM
   );
 }
@@ -635,6 +637,7 @@ export function isBossRoomOf(roomData: RoomConfig, bossID: BossID): boolean {
   return (
     roomData.Type === RoomType.BOSS &&
     roomData.StageID === StageID.SPECIAL_ROOMS &&
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
     roomData.Subtype === bossID
   );
 }
@@ -647,6 +650,7 @@ export function isBossRoomOf(roomData: RoomConfig, bossID: BossID): boolean {
 export function isCrawlSpace(roomData: RoomConfig): boolean {
   return (
     roomData.Type === RoomType.DUNGEON &&
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
     roomData.Subtype === DungeonSubType.NORMAL
   );
 }
@@ -669,7 +673,9 @@ export function isCrawlSpaceWithBlackMarketEntrance(
 export function isDeathCertificateArea(roomData: RoomConfig): boolean {
   return (
     roomData.StageID === StageID.HOME &&
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
     (roomData.Subtype === HomeRoomSubType.DEATH_CERTIFICATE_ENTRANCE ||
+      // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
       roomData.Subtype === HomeRoomSubType.DEATH_CERTIFICATE_ITEMS)
   );
 }
@@ -704,6 +710,7 @@ export function isDogmaRoom(roomData: RoomConfig): boolean {
     roomData.StageID === StageID.HOME &&
     roomData.Type === RoomType.DEFAULT &&
     roomData.Variant === 1000 &&
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
     roomData.Subtype === HomeRoomSubType.LIVING_ROOM
   );
 }
@@ -729,6 +736,7 @@ export function isDoubleTrouble(roomData: RoomConfig): boolean {
  * Helper function to determine if the index of the provided room is equal to `GridRoom.GENESIS`.
  */
 export function isGenesisRoom(roomGridIndex: int): boolean {
+  // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
   return roomGridIndex === GridRoom.GENESIS;
 }
 
@@ -741,7 +749,9 @@ export function isGenesisRoom(roomGridIndex: int): boolean {
 export function isHomeCloset(roomData: RoomConfig): boolean {
   return (
     roomData.StageID === StageID.HOME &&
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
     (roomData.Subtype === HomeRoomSubType.CLOSET_LEFT ||
+      // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
       roomData.Subtype === HomeRoomSubType.CLOSET_RIGHT)
   );
 }
@@ -755,6 +765,7 @@ export function isLRoom(roomData: RoomConfig): boolean {
  * Helper function to determine if the index of the provided room is equal to `GridRoom.MEGA_SATAN`.
  */
 export function isMegaSatanRoom(roomGridIndex: int): boolean {
+  // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
   return roomGridIndex === GridRoom.MEGA_SATAN;
 }
 
@@ -766,7 +777,7 @@ export function isMineShaft(roomData: RoomConfig): boolean {
   return (
     (roomData.StageID === StageID.MINES ||
       roomData.StageID === StageID.ASHPIT) &&
-    // eslint-disable-next-line isaacscript/strict-enums
+    // eslint-disable-next-line complete/strict-enums
     MINE_SHAFT_ROOM_SUB_TYPE_SET.has(roomData.Subtype)
   );
 }
@@ -782,6 +793,7 @@ export function isMinibossRoomOf(
   return (
     roomData.Type === RoomType.MINI_BOSS &&
     roomData.StageID === StageID.SPECIAL_ROOMS &&
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
     roomData.Subtype === minibossID
   );
 }
@@ -795,6 +807,7 @@ export function isMirrorRoom(roomData: RoomConfig): boolean {
     roomData.Type === RoomType.DEFAULT &&
     (roomData.StageID === StageID.DOWNPOUR ||
       roomData.StageID === StageID.DROSS) &&
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
     roomData.Subtype === DownpourRoomSubType.MIRROR
   );
 }
@@ -828,6 +841,7 @@ export function isRoomType(
  * floor.
  */
 export function isSecretExit(roomGridIndex: int): boolean {
+  // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
   return roomGridIndex === GridRoom.SECRET_EXIT;
 }
 
@@ -848,6 +862,7 @@ export function isSecretRoomType(roomType: RoomType): boolean {
  * the only way to detect them is by using the grid index.
  */
 export function isSecretShop(roomGridIndex: int): boolean {
+  // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
   return roomGridIndex === GridRoom.SECRET_SHOP;
 }
 

package/src/functions/set.ts

@@ -10,7 +10,7 @@ import { isPrimitive } from "./types";
  * This function is variadic, meaning that you can specify N sets to add to the first set.
  */
 export function addSetsToSet<T>(
-  // eslint-disable-next-line isaacscript/prefer-readonly-parameter-types
+  // eslint-disable-next-line complete/prefer-readonly-parameter-types
   mainSet: Set<T>,
   ...setsToAdd: ReadonlyArray<ReadonlySet<T>>
 ): void {
@@ -26,7 +26,7 @@ export function addSetsToSet<T>(
  *
  * This function is variadic, meaning that you can specify N sets.
  */
-// eslint-disable-next-line isaacscript/no-mutable-return
+// eslint-disable-next-line complete/no-mutable-return
 export function combineSets<T>(...sets: ReadonlyArray<ReadonlySet<T>>): Set<T> {
   const newSet = new Set<T>();
   for (const set of sets) {
@@ -39,7 +39,7 @@ export function combineSets<T>(...sets: ReadonlyArray<ReadonlySet<T>>): Set<T> {
 }
 
 /** Helper function to copy a set. (You can also use a Set constructor to accomplish this task.) */
-// eslint-disable-next-line isaacscript/no-mutable-return
+// eslint-disable-next-line complete/no-mutable-return
 export function copySet<T>(oldSet: ReadonlySet<T>): Set<T> {
   const newSet = new Set<T>();
   for (const value of oldSet) {
@@ -56,7 +56,7 @@ export function copySet<T>(oldSet: ReadonlySet<T>): Set<T> {
  * This function is variadic, meaning that you can specify N sets to remove from the first set.
  */
 export function deleteSetsFromSet<T>(
-  // eslint-disable-next-line isaacscript/prefer-readonly-parameter-types
+  // eslint-disable-next-line complete/prefer-readonly-parameter-types
   mainSet: Set<T>,
   ...setsToRemove: ReadonlyArray<ReadonlySet<T>>
 ): void {
@@ -123,7 +123,7 @@ export function getSetCombinations<T extends number | string>(
  * Normally, set values are returned in insertion order, so use this function when the ordering of
  * the contents is important.
  */
-// eslint-disable-next-line isaacscript/no-mutable-return
+// eslint-disable-next-line complete/no-mutable-return
 export function getSortedSetValues<T extends number | string>(
   set: ReadonlySet<T>,
 ): T[] {
@@ -167,7 +167,7 @@ export function objectKeysToReadonlySet<K extends string | number | symbol, V>(
  *
  * Also see the `objectKeysToReadonlySet` function.
  */
-// eslint-disable-next-line isaacscript/no-mutable-return
+// eslint-disable-next-line complete/no-mutable-return
 export function objectKeysToSet<K extends string | number | symbol, V>(
   object: Record<K, V>,
 ): Set<K> {
@@ -203,7 +203,7 @@ export function objectValuesToReadonlySet<
  *
  * Also see the `objectValuesToReadonlySet` function.
  */
-// eslint-disable-next-line isaacscript/no-mutable-return
+// eslint-disable-next-line complete/no-mutable-return
 export function objectValuesToSet<K extends string | number | symbol, V>(
   object: Record<K, V>,
 ): Set<V> {
@@ -222,7 +222,7 @@ export function objectValuesToSet<K extends string | number | symbol, V>(
  *
  * This function is variadic, meaning that you can pass as many things as you want to add.
  */
-// eslint-disable-next-line isaacscript/prefer-readonly-parameter-types
+// eslint-disable-next-line complete/prefer-readonly-parameter-types
 export function setAdd<T>(set: Set<T>, ...elements: readonly T[]): void {
   for (const element of elements) {
     set.add(element);

package/src/functions/sort.ts

@@ -149,9 +149,9 @@ export function sortTwoDimensionalArray<T>(
  *
  * Under the hood, this uses the merge sort algorithm.
  */
-// eslint-disable-next-line isaacscript/no-mutable-return
+// eslint-disable-next-line complete/no-mutable-return
 export function stableSort<T>(
-  // eslint-disable-next-line isaacscript/prefer-readonly-parameter-types
+  // eslint-disable-next-line complete/prefer-readonly-parameter-types
   array: T[],
   sortFunc: (a: T, b: T) => -1 | 0 | 1 = sortNormal,
 ): T[] {

package/src/functions/sound.ts

@@ -24,13 +24,8 @@ export function getMusicForStage(
 ): Music {
   const level = game.GetLevel();
 
-  if (stage === undefined) {
-    stage = level.GetStage();
-  }
-
-  if (stageType === undefined) {
-    stageType = level.GetStageType();
-  }
+  stage ??= level.GetStage();
+  stageType ??= level.GetStageType();
 
   const stageTypeToMusic = STAGE_TO_MUSIC[stage];
   return stageTypeToMusic[stageType];

package/src/functions/spawnCollectible.ts

@@ -44,9 +44,7 @@ export function spawnCollectible(
   forceFreeItem = false,
   spawner?: Entity,
 ): EntityPickupCollectible {
-  if (seedOrRNG === undefined) {
-    seedOrRNG = getRandomSeed();
-  }
+  seedOrRNG ??= getRandomSeed();
 
   const seed = isRNG(seedOrRNG) ? seedOrRNG.Next() : seedOrRNG;
   const collectible = spawnPickupWithSeed(

package/src/functions/stage.ts

@@ -133,13 +133,8 @@ export function getLevelName(
 ): string {
   const level = game.GetLevel();
 
-  if (stage === undefined) {
-    stage = level.GetStage();
-  }
-
-  if (stageType === undefined) {
-    stageType = level.GetStageType();
-  }
+  stage ??= level.GetStage();
+  stageType ??= level.GetStageType();
 
   const stageID = getStageID(stage, stageType);
   const stageIDName = getStageIDName(stageID);
@@ -195,13 +190,8 @@ export function getStage(): LevelStage {
 export function getStageID(stage?: LevelStage, stageType?: StageType): StageID {
   const level = game.GetLevel();
 
-  if (stage === undefined) {
-    stage = level.GetStage();
-  }
-
-  if (stageType === undefined) {
-    stageType = level.GetStageType();
-  }
+  stage ??= level.GetStage();
+  stageType ??= level.GetStageType();
 
   if (game.IsGreedMode()) {
     const stageTypeToStageID = STAGE_TO_STAGE_ID_GREED_MODE.get(stage);

package/src/functions/trinkets.ts

@@ -69,9 +69,7 @@ export function getMysteriousPaperEffectForFrame(
   player: EntityPlayer,
   frameCount?: int,
 ): MysteriousPaperEffect | undefined {
-  if (frameCount === undefined) {
-    frameCount = player.FrameCount;
-  }
+  frameCount ??= player.FrameCount;
 
   if (!player.HasTrinket(TrinketType.MYSTERIOUS_PAPER)) {
     return undefined;

package/src/functions/tstlClass.ts

@@ -45,10 +45,10 @@ export function getTSTLClassName(object: unknown): string | undefined {
 }
 
 /**
- * Helper function to determine if a given object is a TypeScriptToLua `Map`.
+ * Helper function to determine if a given object is a `DefaultMap` from `isaacscript-common`.
  *
  * It is not reliable to use the `instanceof` operator to determine this because each Lua module has
- * their own copies of the entire lualib and thus their own instantiated version of a `Map`.
+ * their own copies of the entire lualib and thus their own instantiated version of a `DefaultMap`.
  */
 export function isDefaultMap(
   object: unknown,