isaacscript-common 1.2.250 → 1.2.253

package/dist/features/saveDataManager/merge.lua

@@ -36,7 +36,9 @@ function ____exports.merge(self, oldObject, newTable, traversalDescription)
     if newTableType ~= "table" then
         error("The second argument given to the merge function is not a table.")
     end
-    if mergeArray(nil, oldObject, newTable) then
+    if isArray(nil, oldObject) and isArray(nil, newTable) then
+        local oldTable = oldObject
+        mergeArray(nil, oldTable, newTable)
         return
     end
     if __TS__InstanceOf(oldObject, Map) or __TS__InstanceOf(oldObject, Set) then
@@ -45,16 +47,11 @@ function ____exports.merge(self, oldObject, newTable, traversalDescription)
         mergeTable(nil, oldObject, newTable, traversalDescription)
     end
 end
-function mergeArray(self, oldObject, newTable)
-    local oldArray = oldObject
-    if not isArray(nil, oldArray) or not isArray(nil, newTable) then
-        return false
-    end
+function mergeArray(self, oldArray, newArray)
     clearTable(nil, oldArray)
-    for key, value in pairs(newTable) do
+    for key, value in pairs(newArray) do
         oldArray[key] = value
     end
-    return true
 end
 function mergeTSTLObject(self, oldObject, newTable, traversalDescription)
     oldObject:clear()
@@ -62,13 +59,13 @@ function mergeTSTLObject(self, oldObject, newTable, traversalDescription)
     for key, value in pairs(newTable) do
         do
             if isSerializationBrand(nil, key) then
-                goto __continue13
+                goto __continue12
             end
             local keyToUse = key
             if convertStringKeysToNumbers then
                 local numberKey = tonumber(key)
                 if numberKey == nil then
-                    goto __continue13
+                    goto __continue12
                 end
                 keyToUse = numberKey
             end
@@ -85,13 +82,10 @@ function mergeTSTLObject(self, oldObject, newTable, traversalDescription)
                 oldObject:add(keyToUse)
             end
         end
-        ::__continue13::
+        ::__continue12::
     end
 end
 function mergeTable(self, oldTable, newTable, traversalDescription)
-    if SAVE_DATA_MANAGER_DEBUG then
-        log("merge is operating on a table. Iterating through the keys...")
-    end
     for key, value in pairs(newTable) do
         do
             if SAVE_DATA_MANAGER_DEBUG then
@@ -99,7 +93,7 @@ function mergeTable(self, oldTable, newTable, traversalDescription)
                 log((("merge is merging: " .. traversalDescription) .. " --> ") .. valueToPrint)
             end
             if isSerializationBrand(nil, key) then
-                goto __continue23
+                goto __continue21
             end
             if isSerializedIsaacAPIClass(nil, value) then
                 if SAVE_DATA_MANAGER_DEBUG then
@@ -107,7 +101,7 @@ function mergeTable(self, oldTable, newTable, traversalDescription)
                 end
                 local deserializedObject = deserializeIsaacAPIClass(nil, value)
                 oldTable[key] = deserializedObject
-                goto __continue23
+                goto __continue21
             end
             local valueType = type(value)
             if valueType == "table" then
@@ -123,7 +117,7 @@ function mergeTable(self, oldTable, newTable, traversalDescription)
                 oldTable[key] = value
             end
         end
-        ::__continue23::
+        ::__continue21::
     end
 end
 return ____exports

package/dist/functions/array.d.ts

@@ -82,7 +82,7 @@ export declare function initArray<T>(defaultValue: T, size: int): T[];
  * - the table contains all numerical indexes that are contiguous, starting at 1
  * - the table has no keys (i.e. an "empty" table)
  */
-export declare function isArray(thing: unknown): boolean;
+export declare function isArray(object: unknown): object is unknown[];
 /** Checks if an array is in the provided 2-dimensional array. */
 export declare function isArrayInArray<T>(arrayToMatch: T[] | readonly T[], parentArray: Array<T[] | readonly T[]>): boolean;
 /**

package/dist/functions/array.lua

@@ -155,17 +155,17 @@ function ____exports.initArray(self, defaultValue, size)
     )
     return array
 end
-function ____exports.isArray(self, thing)
-    if type(thing) ~= "table" then
+function ____exports.isArray(self, object)
+    if type(object) ~= "table" then
         return false
     end
-    local thingTable = thing
-    local metatable = getmetatable(thingTable)
+    local ____table = object
+    local metatable = getmetatable(____table)
     if metatable ~= nil then
         return false
     end
     local numEntries = 0
-    for key in pairs(thingTable) do
+    for key in pairs(____table) do
         numEntries = numEntries + 1
         if type(key) ~= "number" then
             return false
@@ -177,7 +177,7 @@ function ____exports.isArray(self, thing)
     do
         local i = 1
         while i <= numEntries do
-            local element = thingTable[i]
+            local element = ____table[i]
             if element == nil then
                 return false
             end

package/dist/functions/deepCopy.d.ts

@@ -18,7 +18,7 @@ import { SerializationType } from "../enums/SerializationType";
  *
  * It does not support:
  * - objects with values of `null` (since that transpiles to `nil`)
- * - other Isaac API objects (that have a type of "userdata")
+ * - other Isaac API objects such as `EntityPtr` (that have a type of "userdata")
  *
  * @param value The primitive or object to copy.
  * @param serializationType Has 3 possible values. Can leave objects as-is, or can serialize objects

package/dist/functions/deepCopy.lua

@@ -9,7 +9,7 @@ local __TS__ArrayPush = ____lualib.__TS__ArrayPush
 local __TS__Iterator = ____lualib.__TS__Iterator
 local __TS__ArraySome = ____lualib.__TS__ArraySome
 local ____exports = {}
-local deepCopyTable, deepCopyDefaultMap, deepCopyMap, deepCopySet, deepCopyTSTLClass, getCopiedEntries, checkMetatable, deepCopyUserdata, COPYABLE_ISAAC_API_CLASS_TYPES_SET
+local deepCopyTable, deepCopyDefaultMap, deepCopyMap, deepCopySet, deepCopyTSTLClass, deepCopyArray, deepCopyNormalLuaTable, getCopiedEntries, checkMetatable, deepCopyUserdata, COPYABLE_ISAAC_API_CLASS_TYPES_SET
 local ____DefaultMap = require("classes.DefaultMap")
 local DefaultMap = ____DefaultMap.DefaultMap
 local ____CopyableIsaacAPIClassType = require("enums.private.CopyableIsaacAPIClassType")
@@ -21,6 +21,8 @@ local ____SerializationType = require("enums.SerializationType")
 local SerializationType = ____SerializationType.SerializationType
 local ____constants = require("features.saveDataManager.constants")
 local SAVE_DATA_MANAGER_DEBUG = ____constants.SAVE_DATA_MANAGER_DEBUG
+local ____array = require("functions.array")
+local isArray = ____array.isArray
 local ____isaacAPIClass = require("functions.isaacAPIClass")
 local getIsaacAPIClassType = ____isaacAPIClass.getIsaacAPIClassType
 local ____log = require("functions.log")
@@ -102,10 +104,10 @@ function deepCopyTable(self, ____table, serializationType, traversalDescription)
         return deepCopySet(nil, ____table, serializationType, traversalDescription)
     end
     if __TS__InstanceOf(____table, WeakMap) then
-        error("The deep copy function does not support copying the \"WeakMap\" class: " .. traversalDescription)
+        error("The deep copy function does not support copying the \"WeakMap\" class for: " .. traversalDescription)
     end
     if __TS__InstanceOf(____table, WeakSet) then
-        error("The deep copy function does not support copying the \"WeakSet\" class: " .. traversalDescription)
+        error("The deep copy function does not support copying the \"WeakSet\" class for: " .. traversalDescription)
     end
     if isUserDefinedTSTLClass(nil, ____table) then
         return deepCopyTSTLClass(nil, ____table, serializationType, traversalDescription)
@@ -114,40 +116,31 @@ function deepCopyTable(self, ____table, serializationType, traversalDescription)
     if isSerializedIsaacAPIClass(nil, ____table) and serializationType == SerializationType.DESERIALIZE then
         return deserializeIsaacAPIClass(nil, ____table)
     end
-    local newTable = {}
-    local ____getCopiedEntries_result_0 = getCopiedEntries(nil, ____table, serializationType, traversalDescription)
-    local entries = ____getCopiedEntries_result_0.entries
-    local convertedNumberKeysToStrings = ____getCopiedEntries_result_0.convertedNumberKeysToStrings
-    if convertedNumberKeysToStrings then
-        newTable[SerializationBrand.OBJECT_WITH_NUMBER_KEYS] = ""
-    end
-    for ____, ____value in ipairs(entries) do
-        local key = ____value[1]
-        local value = ____value[2]
-        newTable[key] = value
+    if isArray(nil, ____table) then
+        return deepCopyArray(nil, ____table, serializationType, traversalDescription)
     end
-    return newTable
+    return deepCopyNormalLuaTable(nil, ____table, serializationType, traversalDescription)
 end
 function deepCopyDefaultMap(self, defaultMap, serializationType, traversalDescription)
-    local ____temp_1
+    local ____temp_0
     if __TS__InstanceOf(defaultMap, DefaultMap) then
-        ____temp_1 = defaultMap:getConstructorArg()
+        ____temp_0 = defaultMap:getConstructorArg()
     else
-        ____temp_1 = nil
+        ____temp_0 = nil
     end
-    local constructorArg = ____temp_1
+    local constructorArg = ____temp_0
     local newDefaultMap
     repeat
-        local ____switch25 = serializationType
-        local ____cond25 = ____switch25 == SerializationType.NONE
-        if ____cond25 then
+        local ____switch23 = serializationType
+        local ____cond23 = ____switch23 == SerializationType.NONE
+        if ____cond23 then
             do
                 newDefaultMap = __TS__New(DefaultMap, constructorArg)
                 break
             end
         end
-        ____cond25 = ____cond25 or ____switch25 == SerializationType.SERIALIZE
-        if ____cond25 then
+        ____cond23 = ____cond23 or ____switch23 == SerializationType.SERIALIZE
+        if ____cond23 then
             do
                 if type(constructorArg) ~= "boolean" and type(constructorArg) ~= "number" and type(constructorArg) ~= "string" then
                     return deepCopyMap(nil, defaultMap, serializationType, traversalDescription)
@@ -158,8 +151,8 @@ function deepCopyDefaultMap(self, defaultMap, serializationType, traversalDescri
                 break
             end
         end
-        ____cond25 = ____cond25 or ____switch25 == SerializationType.DESERIALIZE
-        if ____cond25 then
+        ____cond23 = ____cond23 or ____switch23 == SerializationType.DESERIALIZE
+        if ____cond23 then
             do
                 if __TS__InstanceOf(defaultMap, DefaultMap) then
                     error(("The deep copy function failed to deserialize a default map of \"" .. traversalDescription) .. "\", since it was not a Lua table.")
@@ -178,9 +171,9 @@ function deepCopyDefaultMap(self, defaultMap, serializationType, traversalDescri
             end
         end
     until true
-    local ____getCopiedEntries_result_2 = getCopiedEntries(nil, defaultMap, serializationType, traversalDescription)
-    local entries = ____getCopiedEntries_result_2.entries
-    local convertedNumberKeysToStrings = ____getCopiedEntries_result_2.convertedNumberKeysToStrings
+    local ____getCopiedEntries_result_1 = getCopiedEntries(nil, defaultMap, serializationType, traversalDescription)
+    local entries = ____getCopiedEntries_result_1.entries
+    local convertedNumberKeysToStrings = ____getCopiedEntries_result_1.convertedNumberKeysToStrings
     if convertedNumberKeysToStrings then
         if __TS__InstanceOf(newDefaultMap, DefaultMap) then
             newDefaultMap:set(SerializationBrand.OBJECT_WITH_NUMBER_KEYS, "")
@@ -207,9 +200,9 @@ function deepCopyMap(self, map, serializationType, traversalDescription)
     else
         newMap = __TS__New(Map)
     end
-    local ____getCopiedEntries_result_3 = getCopiedEntries(nil, map, serializationType, traversalDescription)
-    local entries = ____getCopiedEntries_result_3.entries
-    local convertedNumberKeysToStrings = ____getCopiedEntries_result_3.convertedNumberKeysToStrings
+    local ____getCopiedEntries_result_2 = getCopiedEntries(nil, map, serializationType, traversalDescription)
+    local entries = ____getCopiedEntries_result_2.entries
+    local convertedNumberKeysToStrings = ____getCopiedEntries_result_2.convertedNumberKeysToStrings
     if convertedNumberKeysToStrings then
         if __TS__InstanceOf(newMap, Map) then
             newMap:set(SerializationBrand.OBJECT_WITH_NUMBER_KEYS, "")
@@ -236,9 +229,9 @@ function deepCopySet(self, set, serializationType, traversalDescription)
     else
         newSet = __TS__New(Set)
     end
-    local ____getCopiedEntries_result_4 = getCopiedEntries(nil, set, serializationType, traversalDescription)
-    local entries = ____getCopiedEntries_result_4.entries
-    local convertedNumberKeysToStrings = ____getCopiedEntries_result_4.convertedNumberKeysToStrings
+    local ____getCopiedEntries_result_3 = getCopiedEntries(nil, set, serializationType, traversalDescription)
+    local entries = ____getCopiedEntries_result_3.entries
+    local convertedNumberKeysToStrings = ____getCopiedEntries_result_3.convertedNumberKeysToStrings
     if convertedNumberKeysToStrings then
         if __TS__InstanceOf(newSet, Set) then
             error("The deep copy function cannot convert number keys to strings for a Set.")
@@ -263,9 +256,9 @@ function deepCopyTSTLClass(self, tstlClass, serializationType, traversalDescript
     else
         newClass = newTSTLClass(nil, tstlClass)
     end
-    local ____getCopiedEntries_result_5 = getCopiedEntries(nil, tstlClass, serializationType, traversalDescription)
-    local entries = ____getCopiedEntries_result_5.entries
-    local convertedNumberKeysToStrings = ____getCopiedEntries_result_5.convertedNumberKeysToStrings
+    local ____getCopiedEntries_result_4 = getCopiedEntries(nil, tstlClass, serializationType, traversalDescription)
+    local entries = ____getCopiedEntries_result_4.entries
+    local convertedNumberKeysToStrings = ____getCopiedEntries_result_4.convertedNumberKeysToStrings
     if convertedNumberKeysToStrings then
         newClass[SerializationBrand.OBJECT_WITH_NUMBER_KEYS] = ""
     end
@@ -276,6 +269,29 @@ function deepCopyTSTLClass(self, tstlClass, serializationType, traversalDescript
     end
     return newClass
 end
+function deepCopyArray(self, array, serializationType, traversalDescription)
+    local newArray = {}
+    for ____, value in ipairs(array) do
+        local newValue = ____exports.deepCopy(nil, value, serializationType, traversalDescription)
+        __TS__ArrayPush(newArray, newValue)
+    end
+    return newArray
+end
+function deepCopyNormalLuaTable(self, ____table, serializationType, traversalDescription)
+    local newTable = {}
+    local ____getCopiedEntries_result_5 = getCopiedEntries(nil, ____table, serializationType, traversalDescription)
+    local entries = ____getCopiedEntries_result_5.entries
+    local convertedNumberKeysToStrings = ____getCopiedEntries_result_5.convertedNumberKeysToStrings
+    if convertedNumberKeysToStrings then
+        newTable[SerializationBrand.OBJECT_WITH_NUMBER_KEYS] = ""
+    end
+    for ____, ____value in ipairs(entries) do
+        local key = ____value[1]
+        local value = ____value[2]
+        newTable[key] = value
+    end
+    return newTable
+end
 function getCopiedEntries(self, object, serializationType, traversalDescription)
     local entries = {}
     if __TS__InstanceOf(object, Map) or __TS__InstanceOf(object, Set) then
@@ -304,14 +320,14 @@ function getCopiedEntries(self, object, serializationType, traversalDescription)
         local value = ____value[2]
         do
             if isSerializationBrand(nil, key) then
-                goto __continue73
+                goto __continue78
             end
             traversalDescription = getTraversalDescription(nil, key, traversalDescription)
             local newValue = ____exports.deepCopy(nil, value, serializationType, traversalDescription)
             local keyToUse = convertNumberKeysToStrings and tostring(key) or key
             __TS__ArrayPush(copiedEntries, {keyToUse, newValue})
         end
-        ::__continue73::
+        ::__continue78::
     end
     return {entries = copiedEntries, convertedNumberKeysToStrings = convertNumberKeysToStrings}
 end

package/dist/functions/familiars.d.ts

@@ -1,33 +1,40 @@
 /// <reference types="isaac-typescript-definitions" />
 /**
- * Helper function to add and remove familiars based on a target amount that you specify. Use this
- * instead of the `EntityPlayer.CheckFamiliar` method so that the InitSeed of the spawned familiar
- * will be set properly. Note that when using this function, you need to manually account for Box of
- * Friends and Monster Manual (by looking at the number of collectible effects for the familiar).
+ * Helper function to add and remove familiars based on a target amount that you specify.
+ *
+ * This is a convenience wrapper around the `EntityPlayer.CheckFamiliar` method. Use this helper
+ * function instead so that you do not have to retrieve the `ItemConfigItem` and so that you do not
+ * specify an incorrect RNG object. (The vanilla method is bugged in that it does not increment the
+ * RNG object; see the documentation of the method for more details.)
  *
  * This function is meant to be called in the EvaluateCache callback (when the cache flag is equal
  * to `CacheFlag.CACHE_FAMILIARS`).
  *
- * Note that this function is only meant to be used in special circumstances. For general purpose
- * usage, please use the `checkFamiliarFromCollectibles` helper function instead.
+ * Note that this function is only meant to be used in special circumstances where the familiar
+ * count is completely custom and does not correspond to the amount of collectibles. For the general
+ * case, use the `checkFamiliarFromCollectibles` helper function instead.
+ *
+ * Note that this will spawn familiars with a completely random `InitSeed`. When calculating random
+ * events for this familiar, you should use a data structure that maps familiar `InitSeed` to RNG
+ * objects that are initialized based on the seed from
+ * `EntityPlayer.GetCollectibleRNG(collectibleType)`.
  *
  * @param player The player that owns the familiars.
  * @param collectibleType The collectible type of the collectible associated with this familiar.
- * @param numTargetFamiliars The number of familiars that should exist. This function will add or
- * remove familiars until it matches the target count.
+ * @param targetCount The number of familiars that should exist. This function will add or remove
+ * familiars until it matches the target count.
  * @param familiarVariant The variant of the familiar to spawn or remove.
  * @param familiarSubType Optional. The sub-type of the familiar to spawn or remove. If not
  * specified, it will search for existing familiars of all sub-types, and spawn new familiars with a
  * sub-type of 0.
- * @returns The amount of familiars that were added or removed. For example, the player has 0
- * collectibles and there were 2 familiars, this function would remove the 2 familiars and return
- * -2.
  */
-export declare function checkFamiliar(player: EntityPlayer, collectibleType: int, numTargetFamiliars: int, familiarVariant: int, familiarSubType?: int): int;
+export declare function checkFamiliar(player: EntityPlayer, collectibleType: int, targetCount: int, familiarVariant: int, familiarSubType?: int): void;
 /**
  * Helper function to add and remove familiars based on the amount of associated collectibles that a
- * player has. Use this instead of the `EntityPlayer.CheckFamiliar` method so that the InitSeed of
- * the spawned familiar will be set properly and Box of Friends is handled automatically.
+ * player has.
+ *
+ * Use this helper function instead of invoking the `EntityPlayer.CheckFamiliar` method directly so
+ * that the target count is handled automatically.
  *
  * This function is meant to be called in the EvaluateCache callback (when the cache flag is equal
  * to `CacheFlag.CACHE_FAMILIARS`).
@@ -37,15 +44,17 @@ export declare function checkFamiliar(player: EntityPlayer, collectibleType: int
  * Manual). If you instead need to have a custom amount of familiars, use the `checkFamiliars`
  * function instead.
  *
+ * Note that this will spawn familiars with a completely random `InitSeed`. When calculating random
+ * events for this familiar, you should use a data structure that maps familiar `InitSeed` to RNG
+ * objects that are initialized based on the seed from
+ * `EntityPlayer.GetCollectibleRNG(collectibleType)`.
+ *
  * @param player The player that owns the familiars and collectibles.
  * @param collectibleType The collectible type of the collectible associated with this familiar.
  * @param familiarVariant The variant of the familiar to spawn or remove.
  * @param familiarSubType Optional. The sub-type of the familiar to spawn or remove. If not
  * specified, it will search for existing familiars of all sub-types, and spawn new familiars with a
  * sub-type of 0.
- * @returns The amount of familiars that were added or removed. For example, the player has 0
- * collectibles and there were 2 familiars, this function would remove the 2 familiars and return
- * -2.
  */
-export declare function checkFamiliarFromCollectibles(player: EntityPlayer, collectibleType: int, familiarVariant: int, familiarSubType?: int): int;
+export declare function checkFamiliarFromCollectibles(player: EntityPlayer, collectibleType: int, familiarVariant: int, familiarSubType?: int): void;
 export declare function isFamiliarThatShootsPlayerTears(familiar: EntityFamiliar): boolean;

package/dist/functions/familiars.lua

@@ -1,64 +1,29 @@
-local ____lualib = require("lualib_bundle")
-local __TS__ArrayFilter = ____lualib.__TS__ArrayFilter
+--[[ Generated with https://github.com/TypeScriptToLua/TypeScriptToLua ]]
 local ____exports = {}
+local ____cachedClasses = require("cachedClasses")
+local itemConfig = ____cachedClasses.itemConfig
 local ____familiarsThatShootPlayerTearsSet = require("sets.familiarsThatShootPlayerTearsSet")
 local FAMILIARS_THAT_SHOOT_PLAYER_TEARS_SET = ____familiarsThatShootPlayerTearsSet.FAMILIARS_THAT_SHOOT_PLAYER_TEARS_SET
-local ____array = require("functions.array")
-local copyArray = ____array.copyArray
-local ____entity = require("functions.entity")
-local removeEntities = ____entity.removeEntities
-local ____entitySpecific = require("functions.entitySpecific")
-local getFamiliars = ____entitySpecific.getFamiliars
-local spawnFamiliarWithSeed = ____entitySpecific.spawnFamiliarWithSeed
-local ____utils = require("functions.utils")
-local ____repeat = ____utils["repeat"]
-function ____exports.checkFamiliar(self, player, collectibleType, numTargetFamiliars, familiarVariant, familiarSubType)
-    local familiarSubTypeToSearchFor = familiarSubType == nil and -1 or familiarSubType
-    local playerPtrHash = GetPtrHash(player)
-    local familiars = getFamiliars(nil, familiarVariant, familiarSubTypeToSearchFor)
-    local familiarsForThisPlayer = __TS__ArrayFilter(
-        familiars,
-        function(____, familiar) return GetPtrHash(familiar.Player) == playerPtrHash end
-    )
-    if #familiarsForThisPlayer == numTargetFamiliars then
-        return 0
-    end
-    if #familiarsForThisPlayer > numTargetFamiliars then
-        local numFamiliarsToRemove = #familiarsForThisPlayer - numTargetFamiliars
-        local familiarsToRemove = copyArray(nil, familiarsForThisPlayer, numFamiliarsToRemove)
-        removeEntities(nil, familiarsToRemove)
-        return numFamiliarsToRemove * -1
-    end
-    local numFamiliarsToSpawn = numTargetFamiliars - #familiarsForThisPlayer
-    local collectibleRNG = player:GetCollectibleRNG(collectibleType)
-    local familiarSubTypeToUse = familiarSubType == nil and 0 or familiarSubType
-    ____repeat(
-        nil,
-        numFamiliarsToSpawn,
-        function()
-            local seed = collectibleRNG:Next()
-            local familiar = spawnFamiliarWithSeed(
-                nil,
-                familiarVariant,
-                familiarSubTypeToUse,
-                player.Position,
-                seed
-            )
-            familiar.Player = player
-        end
+function ____exports.checkFamiliar(self, player, collectibleType, targetCount, familiarVariant, familiarSubType)
+    local itemConfigItem = itemConfig:GetCollectible(collectibleType)
+    player:CheckFamiliar(
+        familiarVariant,
+        targetCount,
+        RNG(),
+        itemConfigItem,
+        familiarSubType
     )
-    return numFamiliarsToSpawn
 end
 function ____exports.checkFamiliarFromCollectibles(self, player, collectibleType, familiarVariant, familiarSubType)
     local numCollectibles = player:GetCollectibleNum(collectibleType)
     local effects = player:GetEffects()
     local numCollectibleEffects = effects:GetCollectibleEffectNum(collectibleType)
-    local numTargetFamiliars = numCollectibles + numCollectibleEffects
-    return ____exports.checkFamiliar(
+    local targetCount = numCollectibles + numCollectibleEffects
+    ____exports.checkFamiliar(
         nil,
         player,
         collectibleType,
-        numTargetFamiliars,
+        targetCount,
         familiarVariant,
         familiarSubType
     )

package/dist/functions/serialization.lua

@@ -31,7 +31,7 @@ function ____exports.copyIsaacAPIClass(self, isaacAPIClass, serializationType)
     end
     local isaacAPIClassType = getIsaacAPIClassType(nil, isaacAPIClass)
     if isaacAPIClassType == nil then
-        error("Failed to copy an Isaac API class due to it not having a class type.")
+        error("Failed to copy an Isaac API class since it does not have a class type.")
     end
     local copyableIsaacAPIClassType = isaacAPIClassType
     local copyFunction = ISAAC_API_CLASS_TYPE_TO_COPY_FUNCTION[copyableIsaacAPIClassType]

package/dist/functions/utils.d.ts

@@ -92,6 +92,11 @@ export declare function hexToKColor(hexString: string, alpha: float): KColor;
  * is enabled or not.
  */
 export declare function isLuaDebugEnabled(): boolean;
+/**
+ * Since this is a UI element, we do not want to draw it in water reflections. `renderOffset` will
+ * be a non-zero value in reflections.
+ */
+export declare function isReflectionRender(renderOffset: Vector): boolean;
 /**
  * Helper function to print something to the in-game console. Use this instead of invoking the
  * `Isaac.ConsoleOutput` method directly because it will automatically insert a newline at the end

package/dist/functions/utils.lua

@@ -66,6 +66,11 @@ end
 function ____exports.isLuaDebugEnabled(self)
     return package ~= nil
 end
+function ____exports.isReflectionRender(self, renderOffset)
+    local x = math.floor(renderOffset.X + 0.5)
+    local y = math.floor(renderOffset.Y + 0.5)
+    return x ~= 0 or y ~= 0
+end
 function ____exports.printConsole(self, msg)
     Isaac.ConsoleOutput(msg .. "\n")
 end

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "isaacscript-common",
-  "version": "1.2.250",
+  "version": "1.2.253",
   "description": "Helper functions for IsaacScript mods",
   "keywords": [
     "isaac",
@@ -25,7 +25,7 @@
     "dist/**/*.d.ts"
   ],
   "devDependencies": {
-    "isaac-typescript-definitions": "^1.0.388",
+    "isaac-typescript-definitions": "^1.0.389",
     "isaacscript-lint": "^1.0.99",
     "isaacscript-tsconfig": "^1.1.8",
     "typedoc": "^0.22.13",