isaacscript-common 35.0.0 → 35.1.0

package/dist/index.rollup.d.ts

@@ -15160,8 +15160,11 @@ export declare function removeCollectibleFromAllPlayers(...collectibleTypes: Col
  * this function is not necessary when removing items from players. For example, when you remove a
  * collectible with the `EntityPlayer.RemoveCollectible` method, a proper message is sent to the log
  * the item tracker will automatically remove it.
+ *
+ * This function is variadic, meaning that you can pass as many collectible types as you want to
+ * remove.
  */
-export declare function removeCollectibleFromItemTracker(collectibleType: CollectibleType): void;
+export declare function removeCollectibleFromItemTracker(...collectibleTypes: CollectibleType[]): void;
 
 /**
  * Helper function to remove one or more collectibles from all item pools.

package/dist/isaacscript-common.lua

@@ -1,6 +1,6 @@
 --[[
 
-isaacscript-common 35.0.0
+isaacscript-common 35.1.0
 
 This is the "isaacscript-common" library, which was created with the IsaacScript tool.
 
@@ -30382,8 +30382,6 @@ local __TS__ClassExtends = ____lualib.__TS__ClassExtends
 local ____exports = {}
 local ____isaac_2Dtypescript_2Ddefinitions = require("lua_modules.isaac-typescript-definitions.dist.src.index")
 local ModCallback = ____isaac_2Dtypescript_2Ddefinitions.ModCallback
-local ____shouldFire = require("src.shouldFire")
-local shouldFirePickup = ____shouldFire.shouldFirePickup
 local ____CustomCallback = require("src.classes.private.CustomCallback")
 local CustomCallback = ____CustomCallback.CustomCallback
 ____exports.PostPickupSelectionFilter = __TS__Class()
@@ -30392,7 +30390,11 @@ PostPickupSelectionFilter.name = "PostPickupSelectionFilter"
 __TS__ClassExtends(PostPickupSelectionFilter, CustomCallback)
 function PostPickupSelectionFilter.prototype.____constructor(self)
     CustomCallback.prototype.____constructor(self)
-    self.shouldFire = shouldFirePickup
+    self.shouldFire = function(____, fireArgs, optionalArgs)
+        local _pickup, pickupVariant, subType = table.unpack(fireArgs)
+        local callbackPickupVariant, callbackPickupSubType = table.unpack(optionalArgs)
+        return (callbackPickupVariant == nil or callbackPickupVariant == pickupVariant) and (callbackPickupSubType == nil or callbackPickupSubType == subType)
+    end
     self.postPickupSelection = function(____, pickup, variant, subType) return self:fire(pickup, variant, subType) end
     self.callbacksUsed = {{ModCallback.POST_PICKUP_SELECTION, self.postPickupSelection}}
 end
@@ -35180,9 +35182,12 @@ local REBIRTH_ITEM_TRACKER_WRITE_TO_FILE_COMMAND = "REBIRTH_ITEM_TRACKER_WRITE_T
 function ____exports.rebirthItemTrackerWriteToFile(self, msg)
     Isaac.DebugString((REBIRTH_ITEM_TRACKER_WRITE_TO_FILE_COMMAND .. " ") .. msg)
 end
-function ____exports.removeCollectibleFromItemTracker(self, collectibleType)
-    local collectibleName = getCollectibleName(nil, collectibleType)
-    Isaac.DebugString(((("Removing collectible " .. tostring(collectibleType)) .. " (") .. collectibleName) .. ") on player 0 (Player)")
+function ____exports.removeCollectibleFromItemTracker(self, ...)
+    local collectibleTypes = {...}
+    for ____, collectibleType in ipairs(collectibleTypes) do
+        local collectibleName = getCollectibleName(nil, collectibleType)
+        Isaac.DebugString(((("Removing collectible " .. tostring(collectibleType)) .. " (") .. collectibleName) .. ") on player 0 (Player)")
+    end
 end
 return ____exports
  end,

package/dist/src/classes/callbacks/PostPickupSelectionFilter.d.ts

@@ -1,9 +1,16 @@
+import type { PickupVariant } from "isaac-typescript-definitions";
 import type { ModCallbackCustom } from "../../enums/ModCallbackCustom";
-import { shouldFirePickup } from "../../shouldFire";
 import { CustomCallback } from "../private/CustomCallback";
-export declare class PostPickupSelectionFilter extends CustomCallback<ModCallbackCustom.POST_PICKUP_SELECTION_FILTER> {
+type T = ModCallbackCustom.POST_PICKUP_SELECTION_FILTER;
+export declare class PostPickupSelectionFilter extends CustomCallback<T> {
     constructor();
-    protected shouldFire: typeof shouldFirePickup;
+    /**
+     * We cannot use the `shouldFirePickup` helper function because in the case of non-collectibles,
+     * `EntityPickup.Type` and `EntityPickup.Variant` and `EntityPickup.SubType` will all be set to 0
+     * in this callback.
+     */
+    protected shouldFire: (fireArgs: [pickup: EntityPickup, variant: PickupVariant, subType: int], optionalArgs: [pickupVariant?: PickupVariant | undefined, subType?: int | undefined]) => boolean;
     private readonly postPickupSelection;
 }
+export {};
 //# sourceMappingURL=PostPickupSelectionFilter.d.ts.map

package/dist/src/classes/callbacks/PostPickupSelectionFilter.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"PostPickupSelectionFilter.d.ts","sourceRoot":"","sources":["../../../../src/classes/callbacks/PostPickupSelectionFilter.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,+BAA+B,CAAC;AACvE,OAAO,EAAE,gBAAgB,EAAE,MAAM,kBAAkB,CAAC;AACpD,OAAO,EAAE,cAAc,EAAE,MAAM,2BAA2B,CAAC;AAE3D,qBAAa,yBAA0B,SAAQ,cAAc,CAAC,iBAAiB,CAAC,4BAA4B,CAAC;;IAU3G,UAAmB,UAAU,0BAAoB;IAGjD,OAAO,CAAC,QAAQ,CAAC,mBAAmB,CAIuC;CAC5E"}
+{"version":3,"file":"PostPickupSelectionFilter.d.ts","sourceRoot":"","sources":["../../../../src/classes/callbacks/PostPickupSelectionFilter.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,8BAA8B,CAAC;AAElE,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,+BAA+B,CAAC;AAEvE,OAAO,EAAE,cAAc,EAAE,MAAM,2BAA2B,CAAC;AAE3D,KAAK,CAAC,GAAG,iBAAiB,CAAC,4BAA4B,CAAC;AAExD,qBAAa,yBAA0B,SAAQ,cAAc,CAAC,CAAC,CAAC;;IAU9D;;;;OAIG;IACH,UAAmB,UAAU,oKAG1B,OAAO,CASR;IAGF,OAAO,CAAC,QAAQ,CAAC,mBAAmB,CAIuC;CAC5E"}

package/dist/src/classes/callbacks/PostPickupSelectionFilter.lua

@@ -4,8 +4,6 @@ local __TS__ClassExtends = ____lualib.__TS__ClassExtends
 local ____exports = {}
 local ____isaac_2Dtypescript_2Ddefinitions = require("isaac-typescript-definitions")
 local ModCallback = ____isaac_2Dtypescript_2Ddefinitions.ModCallback
-local ____shouldFire = require("src.shouldFire")
-local shouldFirePickup = ____shouldFire.shouldFirePickup
 local ____CustomCallback = require("src.classes.private.CustomCallback")
 local CustomCallback = ____CustomCallback.CustomCallback
 ____exports.PostPickupSelectionFilter = __TS__Class()
@@ -14,7 +12,11 @@ PostPickupSelectionFilter.name = "PostPickupSelectionFilter"
 __TS__ClassExtends(PostPickupSelectionFilter, CustomCallback)
 function PostPickupSelectionFilter.prototype.____constructor(self)
     CustomCallback.prototype.____constructor(self)
-    self.shouldFire = shouldFirePickup
+    self.shouldFire = function(____, fireArgs, optionalArgs)
+        local _pickup, pickupVariant, subType = table.unpack(fireArgs)
+        local callbackPickupVariant, callbackPickupSubType = table.unpack(optionalArgs)
+        return (callbackPickupVariant == nil or callbackPickupVariant == pickupVariant) and (callbackPickupSubType == nil or callbackPickupSubType == subType)
+    end
     self.postPickupSelection = function(____, pickup, variant, subType) return self:fire(pickup, variant, subType) end
     self.callbacksUsed = {{ModCallback.POST_PICKUP_SELECTION, self.postPickupSelection}}
 end

package/dist/src/functions/external.d.ts

@@ -24,6 +24,9 @@ export declare function rebirthItemTrackerWriteToFile(msg: string): void;
  * this function is not necessary when removing items from players. For example, when you remove a
  * collectible with the `EntityPlayer.RemoveCollectible` method, a proper message is sent to the log
  * the item tracker will automatically remove it.
+ *
+ * This function is variadic, meaning that you can pass as many collectible types as you want to
+ * remove.
  */
-export declare function removeCollectibleFromItemTracker(collectibleType: CollectibleType): void;
+export declare function removeCollectibleFromItemTracker(...collectibleTypes: CollectibleType[]): void;
 //# sourceMappingURL=external.d.ts.map

package/dist/src/functions/external.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"external.d.ts","sourceRoot":"","sources":["../../../src/functions/external.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAEH,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,8BAA8B,CAAC;AAMpE;;;;;;;GAOG;AACH,wBAAgB,6BAA6B,CAAC,GAAG,EAAE,MAAM,GAAG,IAAI,CAI/D;AAED;;;;;;;;;;;GAWG;AACH,wBAAgB,gCAAgC,CAC9C,eAAe,EAAE,eAAe,GAC/B,IAAI,CAQN"}
+{"version":3,"file":"external.d.ts","sourceRoot":"","sources":["../../../src/functions/external.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAEH,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,8BAA8B,CAAC;AAMpE;;;;;;;GAOG;AACH,wBAAgB,6BAA6B,CAAC,GAAG,EAAE,MAAM,GAAG,IAAI,CAI/D;AAED;;;;;;;;;;;;;;GAcG;AACH,wBAAgB,gCAAgC,CAC9C,GAAG,gBAAgB,EAAE,eAAe,EAAE,GACrC,IAAI,CAUN"}

package/dist/src/functions/external.lua

@@ -21,8 +21,14 @@ end
 -- this function is not necessary when removing items from players. For example, when you remove a
 -- collectible with the `EntityPlayer.RemoveCollectible` method, a proper message is sent to the log
 -- the item tracker will automatically remove it.
-function ____exports.removeCollectibleFromItemTracker(self, collectibleType)
-    local collectibleName = getCollectibleName(nil, collectibleType)
-    Isaac.DebugString(((("Removing collectible " .. tostring(collectibleType)) .. " (") .. collectibleName) .. ") on player 0 (Player)")
+-- 
+-- This function is variadic, meaning that you can pass as many collectible types as you want to
+-- remove.
+function ____exports.removeCollectibleFromItemTracker(self, ...)
+    local collectibleTypes = {...}
+    for ____, collectibleType in ipairs(collectibleTypes) do
+        local collectibleName = getCollectibleName(nil, collectibleType)
+        Isaac.DebugString(((("Removing collectible " .. tostring(collectibleType)) .. " (") .. collectibleName) .. ") on player 0 (Player)")
+    end
 end
 return ____exports

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "isaacscript-common",
-  "version": "35.0.0",
+  "version": "35.1.0",
   "description": "Helper functions and features for IsaacScript mods.",
   "keywords": [
     "isaac",

package/src/classes/callbacks/PostPickupSelectionFilter.ts

@@ -1,10 +1,12 @@
 import type { PickupVariant } from "isaac-typescript-definitions";
 import { ModCallback } from "isaac-typescript-definitions";
 import type { ModCallbackCustom } from "../../enums/ModCallbackCustom";
-import { shouldFirePickup } from "../../shouldFire";
+import type { FireArgs, OptionalArgs } from "../private/CustomCallback";
 import { CustomCallback } from "../private/CustomCallback";
 
-export class PostPickupSelectionFilter extends CustomCallback<ModCallbackCustom.POST_PICKUP_SELECTION_FILTER> {
+type T = ModCallbackCustom.POST_PICKUP_SELECTION_FILTER;
+
+export class PostPickupSelectionFilter extends CustomCallback<T> {
   constructor() {
     super();
 
@@ -14,7 +16,24 @@ export class PostPickupSelectionFilter extends CustomCallback<ModCallbackCustom.
     ];
   }
 
-  protected override shouldFire = shouldFirePickup;
+  /**
+   * We cannot use the `shouldFirePickup` helper function because in the case of non-collectibles,
+   * `EntityPickup.Type` and `EntityPickup.Variant` and `EntityPickup.SubType` will all be set to 0
+   * in this callback.
+   */
+  protected override shouldFire = (
+    fireArgs: FireArgs<T>,
+    optionalArgs: OptionalArgs<T>,
+  ): boolean => {
+    const [_pickup, pickupVariant, subType] = fireArgs;
+    const [callbackPickupVariant, callbackPickupSubType] = optionalArgs;
+
+    return (
+      (callbackPickupVariant === undefined ||
+        callbackPickupVariant === pickupVariant) &&
+      (callbackPickupSubType === undefined || callbackPickupSubType === subType)
+    );
+  };
 
   // ModCallback.POST_PICKUP_SELECTION (37)
   private readonly postPickupSelection = (

package/src/functions/external.ts

@@ -35,15 +35,20 @@ export function rebirthItemTrackerWriteToFile(msg: string): void {
  * this function is not necessary when removing items from players. For example, when you remove a
  * collectible with the `EntityPlayer.RemoveCollectible` method, a proper message is sent to the log
  * the item tracker will automatically remove it.
+ *
+ * This function is variadic, meaning that you can pass as many collectible types as you want to
+ * remove.
  */
 export function removeCollectibleFromItemTracker(
-  collectibleType: CollectibleType,
+  ...collectibleTypes: CollectibleType[]
 ): void {
-  const collectibleName = getCollectibleName(collectibleType);
+  for (const collectibleType of collectibleTypes) {
+    const collectibleName = getCollectibleName(collectibleType);
 
-  // This cannot use the "log" function since the prefix will prevent the Rebirth Item Tracker from
-  // recognizing the message.
-  Isaac.DebugString(
-    `Removing collectible ${collectibleType} (${collectibleName}) on player 0 (Player)`,
-  );
+    // This cannot use the "log" function since the prefix will prevent the Rebirth Item Tracker
+    // from recognizing the message.
+    Isaac.DebugString(
+      `Removing collectible ${collectibleType} (${collectibleName}) on player 0 (Player)`,
+    );
+  }
 }