isaacscript-common 1.0.482 → 1.0.486

package/dist/features/deployJSONRoom.lua

@@ -201,6 +201,8 @@ function spawnAllEntities(self, jsonRoom, seed, verbose)
             log("Setting the room to be uncleared since there were one or more battle NPCs spawned.")
         end
         setRoomUncleared(nil)
+    elseif verbose then
+        log("Leaving the room cleared since there were no battle NPCs spawned.")
     end
     return seed
 end

package/dist/functions/entity.d.ts

@@ -60,6 +60,25 @@ export declare function getEffects(matchingVariant?: number, matchingSubType?: n
  * returned. False by default. Will only be taken into account if `matchingEntityType` is specified.
  */
 export declare function getEntities(matchingEntityType?: EntityType | int, matchingVariant?: number, matchingSubType?: number, ignoreFriendly?: boolean): Entity[];
+/**
+ * Helper function to get a map containing the positions of every entity in the current room.
+ *
+ * This is useful for rewinding entity positions at a later time. Also see `setEntityPositions()`.
+ *
+ * @param entities Optional. If provided, will only get the positions of the provided entities. This
+ * can be used to cache the entities to avoid invoking `Isaac.GetRoomEntities()` multiple times.
+ */
+export declare function getEntityPositions(entities?: Entity[]): Map<PtrHash, Vector>;
+/**
+ * Helper function to get a map containing the velocities of every entity in the current room.
+ *
+ * This is useful for rewinding entity velocities at a later time. Also see `setEntityVelocities()`.
+ *
+ * @param entities Optional. If provided, will only get the velocities of the provided entities.
+ * This can be used to cache the entities to avoid invoking `Isaac.GetRoomEntities()` multiple
+ * times.
+ */
+export declare function getEntityVelocities(entities?: Entity[]): Map<PtrHash, Vector>;
 /**
  * Helper function to get all of the familiars in the room.
  *
@@ -144,4 +163,31 @@ export declare function removeAllMatchingEntities(entityType: int, entityVariant
 export declare function removeAllPickups(): void;
 export declare function removeAllProjectiles(): void;
 export declare function removeAllTears(): void;
+/**
+ * Helper function to set the position of every entity in the room based on a map of positions. If
+ * an entity is found that does not have matching element in the provided map, then that entity will
+ * be skipped.
+ *
+ * This function is useful for rewinding entity positions at a later time. Also see
+ * `getEntityPositions()`.
+ *
+ * @param entityPositions The map providing the positions for every entity.
+ * @param entities Optional. If provided, will only set the positions of the provided entities. This
+ * can be used to cache the entities to avoid invoking `Isaac.GetRoomEntities()` multiple times.
+ */
+export declare function setEntityPositions(entityPositions: Map<PtrHash, Vector>, entities?: Entity[]): void;
 export declare function setEntityRandomColor(entity: Entity): void;
+/**
+ * Helper function to set the velocity of every entity in the room based on a map of velocities. If
+ * an entity is found that does not have matching element in the provided map, then that entity will
+ * be skipped.
+ *
+ * This function is useful for rewinding entity velocities at a later time. Also see
+ * `getEntityVelocities()`.
+ *
+ * @param entityVelocities The map providing the velocities for every entity.
+ * @param entities Optional. If provided, will only set the velocities of the provided entities.
+ * This can be used to cache the entities to avoid invoking `Isaac.GetRoomEntities()` multiple
+ * times.
+ */
+export declare function setEntityVelocities(entityVelocities: Map<PtrHash, Vector>, entities?: Entity[]): void;

package/dist/functions/entity.lua

@@ -76,6 +76,28 @@ function ____exports.getEntities(self, matchingEntityType, matchingVariant, matc
     end
     return Isaac.FindByType(matchingEntityType, matchingVariant, matchingSubType, ignoreFriendly)
 end
+function ____exports.getEntityPositions(self, entities)
+    if entities == nil then
+        entities = ____exports.getEntities(nil)
+    end
+    local entityPositions = __TS__New(Map)
+    for ____, entity in ipairs(entities) do
+        local ptrHash = GetPtrHash(entity)
+        entityPositions:set(ptrHash, entity.Position)
+    end
+    return entityPositions
+end
+function ____exports.getEntityVelocities(self, entities)
+    if entities == nil then
+        entities = ____exports.getEntities(nil)
+    end
+    local entityVelocities = __TS__New(Map)
+    for ____, entity in ipairs(entities) do
+        local ptrHash = GetPtrHash(entity)
+        entityVelocities:set(ptrHash, entity.Velocity)
+    end
+    return entityVelocities
+end
 function ____exports.getFamiliars(self, matchingVariant, matchingSubType)
     if matchingVariant == nil then
         matchingVariant = -1
@@ -251,6 +273,18 @@ function ____exports.removeAllTears(self)
         tear:Remove()
     end
 end
+function ____exports.setEntityPositions(self, entityPositions, entities)
+    if entities == nil then
+        entities = ____exports.getEntities(nil)
+    end
+    for ____, entity in ipairs(entities) do
+        local ptrHash = GetPtrHash(entity)
+        local entityPosition = entityPositions:get(ptrHash)
+        if entityPosition ~= nil then
+            entity.Position = entityPosition
+        end
+    end
+end
 function ____exports.setEntityRandomColor(self, entity)
     local colorValues = {}
     local seed = entity.InitSeed
@@ -266,4 +300,16 @@ function ____exports.setEntityRandomColor(self, entity)
     local color = Color(colorValues[1], colorValues[2], colorValues[3])
     entity:SetColor(color, 100000, 100000, false, false)
 end
+function ____exports.setEntityVelocities(self, entityVelocities, entities)
+    if entities == nil then
+        entities = ____exports.getEntities(nil)
+    end
+    for ____, entity in ipairs(entities) do
+        local ptrHash = GetPtrHash(entity)
+        local entityVelocity = entityVelocities:get(ptrHash)
+        if entityVelocity ~= nil then
+            entity.Velocity = entityVelocity
+        end
+    end
+end
 return ____exports

package/dist/functions/gridEntity.d.ts

@@ -38,9 +38,30 @@ export declare function isAllPressurePlatesPushed(): boolean;
  * at the VarData of the entity.)
  */
 export declare function isPostBossVoidPortal(gridEntity: GridEntity): boolean;
+/**
+ * Helper function to remove most grid entities in the room.
+ *
+ * Example:
+ * ```
+ * removeAllGridEntitiesExceptFor(
+ *   GridEntityType.GRID_WALL,
+ *   GridEntityType.GRID_DOOR,
+ * );
+ * ```
+ */
 export declare function removeAllGridEntitiesExceptFor(...gridEntityTypes: GridEntityType[]): void;
 export declare function removeAllMatchingGridEntities(gridEntityType: GridEntityType): void;
-export declare function removeGridEntity(gridEntity: GridEntity): void;
+/**
+ * Helper function to remove a grid entity simply by providing the grid entity object.
+ *
+ * @param gridEntity The grid entity to remove.
+ * @param updateRoom Optional. Whether or not to update the room after the grid entity is removed.
+ * True by default. This is generally a good idea because if the room is not updated, you will be
+ * unable to spawn another grid entity on the same tile until a frame has passed. However, doing
+ * this is expensive, since it involves a call to `Isaac.GetRoomEntities()`, so set it to false if
+ * you need to invoke this function multiple times.
+ */
+export declare function removeGridEntity(gridEntity: GridEntity, updateRoom?: boolean): void;
 /**
  * Helper function to make a grid entity invisible. This is accomplished by setting its sprite to
  * "gfx/none.png" (a non-existent PNG file).

package/dist/functions/gridEntity.lua

@@ -3,12 +3,19 @@ require("lualib_bundle");
 local ____exports = {}
 local ____constants = require("constants")
 local GRID_ENTITY_XML_MAP = ____constants.GRID_ENTITY_XML_MAP
-function ____exports.removeGridEntity(self, gridEntity)
+local ____rooms = require("functions.rooms")
+local roomUpdateSafe = ____rooms.roomUpdateSafe
+function ____exports.removeGridEntity(self, gridEntity, updateRoom)
+    if updateRoom == nil then
+        updateRoom = true
+    end
     local game = Game()
     local room = game:GetRoom()
     local gridIndex = gridEntity:GetGridIndex()
     room:RemoveGridEntity(gridIndex, 0, false)
-    room:Update()
+    if updateRoom then
+        roomUpdateSafe(nil)
+    end
 end
 function ____exports.spawnGridEntityWithVariant(self, gridEntityType, variant, gridIndex)
     local game = Game()
@@ -113,15 +120,17 @@ function ____exports.removeAllGridEntitiesExceptFor(self, ...)
     for ____, gridEntity in ipairs(gridEntities) do
         local gridEntityType = gridEntity:GetType()
         if not gridEntityTypeExceptions:has(gridEntityType) then
-            ____exports.removeGridEntity(nil, gridEntity)
+            ____exports.removeGridEntity(nil, gridEntity, false)
         end
     end
+    roomUpdateSafe(nil)
 end
 function ____exports.removeAllMatchingGridEntities(self, gridEntityType)
     local gridEntities = ____exports.getGridEntities(nil, gridEntityType)
     for ____, gridEntity in ipairs(gridEntities) do
-        ____exports.removeGridEntity(nil, gridEntity)
+        ____exports.removeGridEntity(nil, gridEntity, false)
     end
+    roomUpdateSafe(nil)
 end
 function ____exports.setGridEntityInvisible(self, gridEntity)
     local sprite = gridEntity:GetSprite()

package/dist/functions/log.d.ts

@@ -5,31 +5,28 @@
  */
 export declare function getDebugPrependString(msg: string, numParentFunctions?: number): string;
 /**
- * Helper function to avoid typing out `Isaac.DebugString()`.
- * If you have the --luadebug launch flag turned on or the Racing+ sandbox enabled,
- * then this function will also prepend the function name and the line number before the string.
+ * Helper function to avoid typing out `Isaac.DebugString()`. If you have the --luadebug launch flag
+ * turned on or the Racing+ sandbox enabled, then this function will also prepend the function name
+ * and the line number before the string.
  */
 export declare function log(this: void, msg: string): void;
-/**
- * Helper function for printing out every damage flag that is turned on. Helpful when debugging.
- */
+/** Helper function for printing out every damage flag that is turned on. Helpful when debugging. */
 export declare function logAllDamageFlags(this: void, flags: int): void;
-/**
- * Helper function for printing out every entity flag that is turned on. Helpful when debugging.
- */
+/** Helper function for printing out every entity flag that is turned on. Helpful when debugging. */
 export declare function logAllEntityFlags(this: void, flags: int): void;
+/** Helper function for printing out every flag that is turned on. Helpful when debugging. */
+export declare function logAllFlags(this: void, flags: int, flagEnum: LuaTable, description?: string): void;
 /**
- * Helper function for printing out every flag that is turned on. Helpful when debugging.
+ * Helper function for printing out every game state flag that is turned on. Helpful when debugging.
  */
-export declare function logAllFlags(this: void, flags: int, flagEnum: LuaTable, description?: string): void;
 export declare function logAllGameStateFlags(this: void): void;
 /**
  * Helper function for printing out every projectile flag that is turned on. Helpful when debugging.
  */
 export declare function logAllProjectileFlags(this: void, flags: int): void;
-/**
- * Helper function for printing out every use flag that is turned on. Helpful when debugging.
- */
+/** Helper function for printing out every use flag that is turned on. Helpful when debugging. */
+export declare function logAllTearFlags(this: void, flags: int): void;
+/** Helper function for printing out every use flag that is turned on. Helpful when debugging. */
 export declare function logAllUseFlags(this: void, flags: int): void;
 export declare function logArray<T>(this: void, array: T[]): void;
 export declare function logColor(this: void, color: Color): void;

package/dist/functions/log.lua

@@ -68,6 +68,9 @@ end
 function ____exports.logAllProjectileFlags(flags)
     ____exports.logAllFlags(flags, ProjectileFlags, "projectile")
 end
+function ____exports.logAllTearFlags(flags)
+    ____exports.logAllFlags(flags, TearFlags, "tear")
+end
 function ____exports.logAllUseFlags(flags)
     ____exports.logAllFlags(flags, UseFlag, "use")
 end

package/dist/functions/player.d.ts

@@ -1,6 +1,5 @@
 /// <reference types="isaac-typescript-definitions" />
 import { HealthType } from "../types/HealthType";
-import { PocketItemDescription } from "../types/PocketItemDescription";
 /**
  * PlayerIndex is a specific type of string; see the documentation for the [[`getPlayerIndex`]]
  * function. Mods can signify that data structures handle EntityPlayers by using this type:
@@ -47,20 +46,6 @@ export declare function getLastHeart(player: EntityPlayer): HealthType;
  */
 export declare function getNewestPlayer(): EntityPlayer;
 export declare function getFinalPlayer(): EntityPlayer;
-/**
- * Returns the slot number corresponding to where a trinket can be safely inserted.
- *
- * Example:
- * ```
- * const player = Isaac.GetPlayer();
- * const trinketSlot = getOpenTrinketSlotNum(player);
- * if (trinketSlot !== undefined) {
- *   // They have one or more open trinket slots
- *   player.AddTrinket(TrinketType.TRINKET_SWALLOWED_PENNY);
- * }
- * ```
- */
-export declare function getOpenTrinketSlot(player: EntityPlayer): int | undefined;
 /**
  * Iterates over all players and checks if any are close enough to the specified position.
  *
@@ -115,17 +100,6 @@ export declare function getPlayerIndexVanilla(playerToFind: EntityPlayer): int |
  * This is equivalent to the number of hits that the player can currently take.
  */
 export declare function getPlayerNumAllHearts(player: EntityPlayer): int;
-/**
- * Use this helper function as a workaround for `EntityPlayer.GetPocketItem()` not working
- * correctly.
- *
- * Note that due to API limitations, there is no way to determine the location of a Dice Bag trinket
- * dice. Furthermore, when the player has a Dice Bag trinket dice and a pocket active at the same
- * time, there is no way to determine the location of the pocket active item. If this function
- * cannot determine the identity of a particular slot, it will mark the type of the slot as
- * `PocketItemType.UNDETERMINABLE`.
- */
-export declare function getPocketItems(player: EntityPlayer): PocketItemDescription[];
 /**
  * Helper function to return the active charge and the battery charge combined. This is useful
  * because you are not able to set the battery charge directly.
@@ -146,23 +120,6 @@ export declare function hasLostCurse(player: EntityPlayer): boolean;
  * items. (Only Tainted Forgotten can pick up items.)
  */
 export declare function hasOpenActiveItemSlot(player: EntityPlayer): boolean;
-/**
- * Returns whether or not the player can hold an additional pocket item, beyond what they are
- * currently carrying. This takes into account items that modify the max number of pocket items,
- * like Starter Deck.
- *
- * If the player is the Tainted Soul, this always returns false, since that character cannot pick up
- * items. (Only Tainted Forgotten can pick up items.)
- */
-export declare function hasOpenPocketItemSlot(player: EntityPlayer): boolean;
-/**
- * Returns whether or not the player can hold an additional trinket, beyond what they are currently
- * carrying. This takes into account items that modify the max number of trinkets, like Mom's Purse.
- *
- * If the player is the Tainted Soul, this always returns false, since that character cannot pick up
- * items. (Only Tainted Forgotten can pick up items.)
- */
-export declare function hasOpenTrinketSlot(player: EntityPlayer): boolean;
 /**
  * Helper function for detecting when a player is Bethany or Tainted Bethany. This is useful if you
  * need to adjust UI elements to account for Bethany's soul charges or Tainted Bethany's blood
@@ -198,6 +155,22 @@ export declare function removeCollectibleCostume(player: EntityPlayer, collectib
  */
 export declare function removeDeadEyeMultiplier(player: EntityPlayer): void;
 export declare function removeTrinketCostume(player: EntityPlayer, trinketType: TrinketType | int): void;
+/**
+ * Helper function to set an active collectible to a particular slot. This has different behavior
+ * than calling `player.AddCollectible()` with the `activeSlot` argument, because this function will
+ * not shift existing items into the Schoolbag and it handles `ActiveSlot.SLOT_POCKET2`.
+ *
+ * Note that if an item is set to `ActiveSlot.SLOT_POCKET2`, it will disappear after being used and
+ * will be automatically removed upon entering a new room.
+ *
+ * @param player The player to give the item to.
+ * @param collectibleType The collectible type of the item to give.
+ * @param activeSlot The slot to set.
+ * @param charge Optional. The argument of charges to set. If not specified, the item will be set
+ * with maximum charges.
+ * @param keepInPools Optional. Whether or not to remove the item from pools. False by default.
+ */
+export declare function setActiveItem(player: EntityPlayer, collectibleType: CollectibleType, activeSlot: ActiveSlot, charge?: int, keepInPools?: boolean): void;
 /**
  * If you want to stop the player from shooting without the blindfold costume, then simply call
  * `player.TryRemoveNullCostume(NullItemID.ID_BLINDFOLD)` after invoking this function.

package/dist/functions/player.lua

@@ -6,13 +6,15 @@ local ____constants = require("constants")
 local LOST_STYLE_PLAYER_TYPES = ____constants.LOST_STYLE_PLAYER_TYPES
 local ____HealthType = require("types.HealthType")
 local HealthType = ____HealthType.HealthType
-local ____PocketItemType = require("types.PocketItemType")
-local PocketItemType = ____PocketItemType.PocketItemType
 local ____bitwise = require("functions.bitwise")
 local getKBitOfN = ____bitwise.getKBitOfN
 local getNumBitsOfN = ____bitwise.getNumBitsOfN
+local ____collectibles = require("functions.collectibles")
+local getCollectibleMaxCharges = ____collectibles.getCollectibleMaxCharges
 local ____collectibleSet = require("functions.collectibleSet")
 local getCollectibleSet = ____collectibleSet.getCollectibleSet
+local ____util = require("functions.util")
+local ensureAllCases = ____util.ensureAllCases
 function ____exports.getPlayers(self, performExclusions)
     if performExclusions == nil then
         performExclusions = false
@@ -25,18 +27,18 @@ function ____exports.getPlayers(self, performExclusions)
             do
                 local player = Isaac.GetPlayer(i)
                 if player == nil then
-                    goto __continue55
+                    goto __continue51
                 end
                 if ____exports.isChildPlayer(nil, player) then
-                    goto __continue55
+                    goto __continue51
                 end
                 local character = player:GetPlayerType()
                 if performExclusions and EXCLUDED_CHARACTERS:has(character) then
-                    goto __continue55
+                    goto __continue51
                 end
                 __TS__ArrayPush(players, player)
             end
-            ::__continue55::
+            ::__continue51::
             i = i + 1
         end
     end
@@ -206,24 +208,6 @@ function ____exports.getFinalPlayer(self)
     local players = ____exports.getPlayers(nil)
     return players[#players]
 end
-function ____exports.getOpenTrinketSlot(self, player)
-    local maxTrinkets = player:GetMaxTrinkets()
-    local trinket0 = player:GetTrinket(0)
-    local trinket1 = player:GetTrinket(1)
-    if maxTrinkets == 1 then
-        return ((trinket0 == TrinketType.TRINKET_NULL) and 0) or nil
-    end
-    if maxTrinkets == 2 then
-        if trinket0 == TrinketType.TRINKET_NULL then
-            return 0
-        end
-        return ((trinket1 == TrinketType.TRINKET_NULL) and 1) or nil
-    end
-    error(
-        "The player has an unknown number of trinket slots: " .. tostring(maxTrinkets)
-    )
-    return nil
-end
 function ____exports.getPlayerCloserThan(self, position, distance)
     for ____, player in ipairs(
         ____exports.getPlayers(nil)
@@ -279,14 +263,14 @@ function ____exports.getPlayerIndexVanilla(self, playerToFind)
             do
                 local player = Isaac.GetPlayer(i)
                 if player == nil then
-                    goto __continue64
+                    goto __continue60
                 end
                 local playerHash = GetPtrHash(player)
                 if playerHash == playerToFindHash then
                     return i
                 end
             end
-            ::__continue64::
+            ::__continue60::
             i = i + 1
         end
     end
@@ -299,43 +283,6 @@ function ____exports.getPlayerNumAllHearts(self, player)
     local eternalHearts = player:GetEternalHearts()
     return ((hearts + soulHearts) + boneHearts) + eternalHearts
 end
-function ____exports.getPocketItems(self, player)
-    local pocketItem = player:GetActiveItem(ActiveSlot.SLOT_POCKET)
-    local hasPocketItem = pocketItem ~= CollectibleType.COLLECTIBLE_NULL
-    local pocketItem2 = player:GetActiveItem(ActiveSlot.SLOT_POCKET2)
-    local hasPocketItem2 = pocketItem2 ~= CollectibleType.COLLECTIBLE_NULL
-    local maxPocketItems = player:GetMaxPocketItems()
-    local pocketItems = {}
-    local pocketItemIdentified = false
-    local pocketItem2Identified = false
-    do
-        local slot = 0
-        while slot < 4 do
-            local card = player:GetCard(slot)
-            local pill = player:GetPill(slot)
-            if card ~= Card.CARD_NULL then
-                __TS__ArrayPush(pocketItems, {type = PocketItemType.CARD, id = card})
-            elseif pill ~= PillColor.PILL_NULL then
-                __TS__ArrayPush(pocketItems, {type = PocketItemType.PILL, id = pill})
-            elseif (hasPocketItem and (not hasPocketItem2)) and (not pocketItemIdentified) then
-                pocketItemIdentified = true
-                __TS__ArrayPush(pocketItems, {type = PocketItemType.ACTIVE_ITEM, id = pocketItem})
-            elseif ((not hasPocketItem) and hasPocketItem2) and (not pocketItem2Identified) then
-                pocketItem2Identified = true
-                __TS__ArrayPush(pocketItems, {type = PocketItemType.DICE_BAG_DICE, id = pocketItem2})
-            elseif hasPocketItem and hasPocketItem2 then
-                __TS__ArrayPush(pocketItems, {type = PocketItemType.UNDETERMINABLE, id = 0})
-            else
-                __TS__ArrayPush(pocketItems, {type = PocketItemType.EMPTY, id = 0})
-            end
-            if (slot + 1) == maxPocketItems then
-                break
-            end
-            slot = slot + 1
-        end
-    end
-    return pocketItems
-end
 function ____exports.getTotalCharge(self, player, activeSlot)
     local activeCharge = player:GetActiveCharge(activeSlot)
     local batteryCharge = player:GetBatteryCharge(activeSlot)
@@ -367,27 +314,6 @@ function ____exports.hasOpenActiveItemSlot(self, player)
     end
     return activeItemPrimary == CollectibleType.COLLECTIBLE_NULL
 end
-function ____exports.hasOpenPocketItemSlot(self, player)
-    local character = player:GetPlayerType()
-    if character == PlayerType.PLAYER_THESOUL_B then
-        return false
-    end
-    local pocketItems = ____exports.getPocketItems(nil, player)
-    for ____, pocketItem in ipairs(pocketItems) do
-        if pocketItem.type == PocketItemType.EMPTY then
-            return true
-        end
-    end
-    return false
-end
-function ____exports.hasOpenTrinketSlot(self, player)
-    local character = player:GetPlayerType()
-    if character == PlayerType.PLAYER_THESOUL_B then
-        return false
-    end
-    local openTrinketSlot = ____exports.getOpenTrinketSlot(nil, player)
-    return openTrinketSlot ~= nil
-end
 function ____exports.isBethany(self, player)
     local character = player:GetPlayerType()
     return (character == PlayerType.PLAYER_BETHANY) or (character == PlayerType.PLAYER_BETHANY_B)
@@ -436,6 +362,71 @@ function ____exports.removeTrinketCostume(self, player, trinketType)
     end
     player:RemoveCostume(itemConfigTrinket)
 end
+function ____exports.setActiveItem(self, player, collectibleType, activeSlot, charge, keepInPools)
+    if keepInPools == nil then
+        keepInPools = false
+    end
+    local game = Game()
+    local itemPool = game:GetItemPool()
+    local primaryCollectibleType = player:GetActiveItem(ActiveSlot.SLOT_PRIMARY)
+    local primaryCharge = player:GetActiveCharge(ActiveSlot.SLOT_PRIMARY)
+    local secondaryCollectibleType = player:GetActiveItem(ActiveSlot.SLOT_SECONDARY)
+    if charge == nil then
+        charge = getCollectibleMaxCharges(nil, collectibleType)
+    end
+    if not keepInPools then
+        itemPool:RemoveCollectible(collectibleType)
+    end
+    repeat
+        local ____switch87 = activeSlot
+        local ____cond87 = ____switch87 == ActiveSlot.SLOT_PRIMARY
+        if ____cond87 then
+            do
+                if primaryCollectibleType ~= CollectibleType.COLLECTIBLE_NULL then
+                    player:RemoveCollectible(primaryCollectibleType)
+                end
+                player:AddCollectible(collectibleType, charge, false)
+                break
+            end
+        end
+        ____cond87 = ____cond87 or (____switch87 == ActiveSlot.SLOT_SECONDARY)
+        if ____cond87 then
+            do
+                if primaryCollectibleType ~= CollectibleType.COLLECTIBLE_NULL then
+                    player:RemoveCollectible(primaryCollectibleType)
+                end
+                if secondaryCollectibleType ~= CollectibleType.COLLECTIBLE_NULL then
+                    player:RemoveCollectible(secondaryCollectibleType)
+                end
+                player:AddCollectible(secondaryCollectibleType, charge, false)
+                if primaryCollectibleType ~= CollectibleType.COLLECTIBLE_NULL then
+                    player:AddCollectible(primaryCollectibleType, primaryCharge, false)
+                end
+                break
+            end
+        end
+        ____cond87 = ____cond87 or (____switch87 == ActiveSlot.SLOT_POCKET)
+        if ____cond87 then
+            do
+                player:SetPocketActiveItem(collectibleType, activeSlot, keepInPools)
+                player:SetActiveCharge(charge, activeSlot)
+                break
+            end
+        end
+        ____cond87 = ____cond87 or (____switch87 == ActiveSlot.SLOT_POCKET2)
+        if ____cond87 then
+            do
+                player:SetPocketActiveItem(collectibleType, activeSlot, keepInPools)
+                break
+            end
+        end
+        do
+            do
+                ensureAllCases(nil, activeSlot)
+            end
+        end
+    until true
+end
 function ____exports.setBlindfold(self, player, enabled)
     local game = Game()
     local character = player:GetPlayerType()

package/dist/functions/pocketItems.d.ts

@@ -1,21 +1,23 @@
 /// <reference types="isaac-typescript-definitions" />
+import { PocketItemDescription } from "../types/PocketItemDescription";
+export declare function getFirstCardOrPill(player: EntityPlayer): PocketItemDescription | undefined;
 /**
- * This is a helper function to get a card name from a Card.
+ * Use this helper function as a workaround for `EntityPlayer.GetPocketItem()` not working
+ * correctly.
  *
- * Example:
- * ```
- * const card = Card.CARD_FOOL;
- * const cardName = getCardName(card); // cardName is "0 - The Fool"
- * ```
+ * Note that due to API limitations, there is no way to determine the location of a Dice Bag trinket
+ * dice. Furthermore, when the player has a Dice Bag trinket dice and a pocket active at the same
+ * time, there is no way to determine the location of the pocket active item. If this function
+ * cannot determine the identity of a particular slot, it will mark the type of the slot as
+ * `PocketItemType.UNDETERMINABLE`.
  */
-export declare function getCardName(card: Card | int): string;
+export declare function getPocketItems(player: EntityPlayer): PocketItemDescription[];
 /**
- * This is a helper function to get a pill effect name from a PillEffect.
+ * Returns whether or not the player can hold an additional pocket item, beyond what they are
+ * currently carrying. This takes into account items that modify the max number of pocket items,
+ * like Starter Deck.
  *
- * Example:
- * ```
- * const pillEffect = PillEffect.PILLEFFECT_BAD_GAS;
- * const pillEffectName = getPillEffectName(pillEffect); // trinketName is "Bad Gas"
- * ```
+ * If the player is the Tainted Soul, this always returns false, since that character cannot pick up
+ * items. (Only Tainted Forgotten can pick up items.)
  */
-export declare function getPillEffectName(pillEffect: PillEffect | int): string;
+export declare function hasOpenPocketItemSlot(player: EntityPlayer): boolean;

package/dist/functions/pocketItems.lua

@@ -1,39 +1,65 @@
 --[[ Generated with https://github.com/TypeScriptToLua/TypeScriptToLua ]]
+require("lualib_bundle");
 local ____exports = {}
-local ____cardNameMap = require("maps.cardNameMap")
-local CARD_NAME_MAP = ____cardNameMap.CARD_NAME_MAP
-local ____pillEffectNameMap = require("maps.pillEffectNameMap")
-local PILL_EFFECT_NAME_MAP = ____pillEffectNameMap.PILL_EFFECT_NAME_MAP
-function ____exports.getCardName(self, card)
-    local itemConfig = Isaac.GetItemConfig()
-    local defaultName = "Unknown"
-    if type(card) ~= "number" then
-        return defaultName
+local ____PocketItemType = require("types.PocketItemType")
+local PocketItemType = ____PocketItemType.PocketItemType
+function ____exports.getPocketItems(self, player)
+    local pocketItem = player:GetActiveItem(ActiveSlot.SLOT_POCKET)
+    local hasPocketItem = pocketItem ~= CollectibleType.COLLECTIBLE_NULL
+    local pocketItem2 = player:GetActiveItem(ActiveSlot.SLOT_POCKET2)
+    local hasPocketItem2 = pocketItem2 ~= CollectibleType.COLLECTIBLE_NULL
+    local maxPocketItems = player:GetMaxPocketItems()
+    local pocketItems = {}
+    local pocketItemIdentified = false
+    local pocketItem2Identified = false
+    do
+        local slot = 0
+        while slot < 4 do
+            local card = player:GetCard(slot)
+            local pill = player:GetPill(slot)
+            if card ~= Card.CARD_NULL then
+                __TS__ArrayPush(pocketItems, {type = PocketItemType.CARD, id = card})
+            elseif pill ~= PillColor.PILL_NULL then
+                __TS__ArrayPush(pocketItems, {type = PocketItemType.PILL, id = pill})
+            elseif (hasPocketItem and (not hasPocketItem2)) and (not pocketItemIdentified) then
+                pocketItemIdentified = true
+                __TS__ArrayPush(pocketItems, {type = PocketItemType.ACTIVE_ITEM, id = pocketItem})
+            elseif ((not hasPocketItem) and hasPocketItem2) and (not pocketItem2Identified) then
+                pocketItem2Identified = true
+                __TS__ArrayPush(pocketItems, {type = PocketItemType.DICE_BAG_DICE, id = pocketItem2})
+            elseif hasPocketItem and hasPocketItem2 then
+                __TS__ArrayPush(pocketItems, {type = PocketItemType.UNDETERMINABLE, id = 0})
+            else
+                __TS__ArrayPush(pocketItems, {type = PocketItemType.EMPTY, id = 0})
+            end
+            if (slot + 1) == maxPocketItems then
+                break
+            end
+            slot = slot + 1
+        end
     end
-    local cardName = CARD_NAME_MAP:get(card)
-    if cardName ~= nil then
-        return cardName
-    end
-    local itemConfigCard = itemConfig:GetCard(card)
-    if itemConfigCard == nil then
-        return defaultName
-    end
-    return itemConfigCard.Name
+    return pocketItems
 end
-function ____exports.getPillEffectName(self, pillEffect)
-    local itemConfig = Isaac.GetItemConfig()
-    local defaultName = "Unknown"
-    if type(pillEffect) ~= "number" then
-        return defaultName
+function ____exports.getFirstCardOrPill(self, player)
+    local pocketItems = ____exports.getPocketItems(nil, player)
+    for ____, pocketItem in ipairs(pocketItems) do
+        if (pocketItem.type == PocketItemType.CARD) or (pocketItem.type == PocketItemType.PILL) then
+            return pocketItem
+        end
     end
-    local pillEffectName = PILL_EFFECT_NAME_MAP:get(pillEffect)
-    if pillEffectName ~= nil then
-        return pillEffectName
+    return nil
+end
+function ____exports.hasOpenPocketItemSlot(self, player)
+    local character = player:GetPlayerType()
+    if character == PlayerType.PLAYER_THESOUL_B then
+        return false
     end
-    local itemConfigPillEffect = itemConfig:GetPillEffect(pillEffect)
-    if itemConfigPillEffect == nil then
-        return defaultName
+    local pocketItems = ____exports.getPocketItems(nil, player)
+    for ____, pocketItem in ipairs(pocketItems) do
+        if pocketItem.type == PocketItemType.EMPTY then
+            return true
+        end
     end
-    return itemConfigPillEffect.Name
+    return false
 end
 return ____exports

package/dist/functions/rooms.d.ts

@@ -108,6 +108,13 @@ export declare function inDimension(dimension: Dimension): boolean;
 export declare function inLRoom(): boolean;
 export declare function inGenesisRoom(): boolean;
 export declare function inStartingRoom(): boolean;
+/**
+ * If `Room.Update()` is called in a PostNewRoom callback, then some entities will slide around
+ * (such as the player). Since those entity velocities are already at zero, setting them to zero
+ * will have no effect. Thus, a generic solution is to record all of the entity positions/velocities
+ * before updating the room, and then restore those positions/velocities.
+ */
+export declare function roomUpdateSafe(): void;
 /**
  * Helper function to convert an uncleared room to a cleared room in the PostNewRoom callback. This
  * is useful because if enemies are removed in this callback, a room drop will be awarded and the

package/dist/functions/rooms.lua

@@ -11,6 +11,12 @@ local ____doors = require("functions.doors")
 local closeAllDoors = ____doors.closeAllDoors
 local getDoors = ____doors.getDoors
 local isHiddenSecretRoomDoor = ____doors.isHiddenSecretRoomDoor
+local ____entity = require("functions.entity")
+local getEntities = ____entity.getEntities
+local getEntityPositions = ____entity.getEntityPositions
+local getEntityVelocities = ____entity.getEntityVelocities
+local setEntityPositions = ____entity.setEntityPositions
+local setEntityVelocities = ____entity.setEntityVelocities
 local ____flag = require("functions.flag")
 local hasFlag = ____flag.hasFlag
 function ____exports.getRoomIndex(self)
@@ -196,6 +202,16 @@ function ____exports.inStartingRoom(self)
     local roomIndex = ____exports.getRoomIndex(nil)
     return roomIndex == startingRoomIndex
 end
+function ____exports.roomUpdateSafe(self)
+    local game = Game()
+    local room = game:GetRoom()
+    local entities = getEntities(nil)
+    local entityPositions = getEntityPositions(nil, entities)
+    local entityVelocities = getEntityVelocities(nil, entities)
+    room:Update()
+    setEntityPositions(nil, entityPositions, entities)
+    setEntityVelocities(nil, entityVelocities, entities)
+end
 function ____exports.setRoomCleared(self)
     local game = Game()
     local room = game:GetRoom()
@@ -210,14 +226,14 @@ function ____exports.setRoomCleared(self)
     ) do
         do
             if isHiddenSecretRoomDoor(nil, door) then
-                goto __continue38
+                goto __continue39
             end
             door.State = DoorState.STATE_OPEN
             local sprite = door:GetSprite()
             sprite:Play("Opened", true)
             door.ExtraVisible = false
         end
-        ::__continue38::
+        ::__continue39::
     end
     sfx:Stop(SoundEffect.SOUND_DOOR_HEAVY_OPEN)
     game:ShakeScreen(0)

package/dist/functions/trinkets.d.ts

@@ -1,6 +1,20 @@
 /// <reference types="isaac-typescript-definitions" />
 export declare function getMaxTrinketID(): int;
-/** Helper function to get all of the collectible entities in the room. */
+/**
+ * Returns the slot number corresponding to where a trinket can be safely inserted.
+ *
+ * Example:
+ * ```
+ * const player = Isaac.GetPlayer();
+ * const trinketSlot = getOpenTrinketSlotNum(player);
+ * if (trinketSlot !== undefined) {
+ *   // They have one or more open trinket slots
+ *   player.AddTrinket(TrinketType.TRINKET_SWALLOWED_PENNY);
+ * }
+ * ```
+ */
+export declare function getOpenTrinketSlot(player: EntityPlayer): int | undefined;
+/** Helper function to get all of the trinket entities in the room. */
 export declare function getTrinkets(matchingSubType?: number): EntityPickup[];
 /** This is a helper function to get a trinket description from a TrinketType. */
 export declare function getTrinketDescription(trinketType: TrinketType | int): string;
@@ -14,4 +28,12 @@ export declare function getTrinketDescription(trinketType: TrinketType | int): s
  * ```
  */
 export declare function getTrinketName(trinketType: TrinketType | int): string;
+/**
+ * Returns whether or not the player can hold an additional trinket, beyond what they are currently
+ * carrying. This takes into account items that modify the max number of trinkets, like Mom's Purse.
+ *
+ * If the player is the Tainted Soul, this always returns false, since that character cannot pick up
+ * items. (Only Tainted Forgotten can pick up items.)
+ */
+export declare function hasOpenTrinketSlot(player: EntityPlayer): boolean;
 export declare function isGoldenTrinket(trinketType: TrinketType | int): boolean;

package/dist/functions/trinkets.lua

@@ -11,6 +11,24 @@ function ____exports.getMaxTrinketID(self)
     local itemConfig = Isaac.GetItemConfig()
     return itemConfig:GetTrinkets().Size - 1
 end
+function ____exports.getOpenTrinketSlot(self, player)
+    local maxTrinkets = player:GetMaxTrinkets()
+    local trinket0 = player:GetTrinket(0)
+    local trinket1 = player:GetTrinket(1)
+    if maxTrinkets == 1 then
+        return ((trinket0 == TrinketType.TRINKET_NULL) and 0) or nil
+    end
+    if maxTrinkets == 2 then
+        if trinket0 == TrinketType.TRINKET_NULL then
+            return 0
+        end
+        return ((trinket1 == TrinketType.TRINKET_NULL) and 1) or nil
+    end
+    error(
+        "The player has an unknown number of trinket slots: " .. tostring(maxTrinkets)
+    )
+    return nil
+end
 function ____exports.getTrinkets(self, matchingSubType)
     if matchingSubType == nil then
         matchingSubType = -1
@@ -57,6 +75,14 @@ function ____exports.getTrinketName(self, trinketType)
     end
     return itemConfigItem.Name
 end
+function ____exports.hasOpenTrinketSlot(self, player)
+    local character = player:GetPlayerType()
+    if character == PlayerType.PLAYER_THESOUL_B then
+        return false
+    end
+    local openTrinketSlot = ____exports.getOpenTrinketSlot(nil, player)
+    return openTrinketSlot ~= nil
+end
 function ____exports.isGoldenTrinket(self, trinketType)
     return trinketType > GOLDEN_TRINKET_SHIFT
 end

package/dist/index.d.ts

@@ -32,6 +32,7 @@ export * from "./functions/pickups";
 export * from "./functions/pills";
 export * from "./functions/player";
 export * from "./functions/playerHealth";
+export * from "./functions/pocketItems";
 export * from "./functions/position";
 export * from "./functions/random";
 export * from "./functions/revive";

package/dist/index.lua

@@ -262,6 +262,14 @@ do
         end
     end
 end
+do
+    local ____export = require("functions.pocketItems")
+    for ____exportKey, ____exportValue in pairs(____export) do
+        if ____exportKey ~= "default" then
+            ____exports[____exportKey] = ____exportValue
+        end
+    end
+end
 do
     local ____export = require("functions.position")
     for ____exportKey, ____exportValue in pairs(____export) do

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "isaacscript-common",
-  "version": "1.0.482",
+  "version": "1.0.486",
   "description": "Helper functions for IsaacScript mods",
   "keywords": [
     "isaac",
@@ -25,7 +25,7 @@
     "dist/**/*.d.ts"
   ],
   "devDependencies": {
-    "isaac-typescript-definitions": "^1.0.282",
+    "isaac-typescript-definitions": "^1.0.284",
     "isaacscript-lint": "^1.0.67",
     "typedoc": "^0.22.10",
     "typescript": "4.4.4",