Bug: the type of the beforeBulkDestroy hander's options attribute is BulkCreateOptions instead of BulkDestroyOptions

[kf6kjg]

In my use case I needed to access the where attribute of the options property that's passed to the callback/handler passed to beforeBulkDestroy. However since that's described as an BulkCreateOptions not a BulkDestroyOptions, that isn't possible when using the inferred type in TypeScript.

Here is the patch that solved my problem:

diff --git a/node_modules/sequelize/types/model.d.ts b/node_modules/sequelize/types/model.d.ts
index 7ab36db..a0665d4 100644
--- a/node_modules/sequelize/types/model.d.ts
+++ b/node_modules/sequelize/types/model.d.ts
@@ -1091,6 +1091,13 @@ export interface BulkCreateOptions<TAttributes = any> extends Logging, Transacti
   conflictAttributes?: Array<keyof TAttributes>;
 }
 
+/**
+ * Options for Model.bulkDestroy method
+ */
+export interface BulkDestroyOptions<TAttributes = any> extends BulkCreateOptions<TAttributes> {
+  where?: WhereOptions;
+}
+
 /**
  * The options passed to Model.destroy in addition to truncate
  */
@@ -2770,10 +2777,10 @@ export abstract class Model<TModelAttributes extends {} = any, TCreationAttribut
    */
   public static beforeBulkDestroy<M extends Model>(
     this: ModelStatic<M>,
-    name: string, fn: (options: BulkCreateOptions<Attributes<M>>) => HookReturn): void;
+    name: string, fn: (options: BulkDestroyOptions<Attributes<M>>) => HookReturn): void;
   public static beforeBulkDestroy<M extends Model>(
     this: ModelStatic<M>,
-    fn: (options: BulkCreateOptions<Attributes<M>>) => HookReturn
+    fn: (options: BulkDestroyOptions<Attributes<M>>) => HookReturn
   ): void;
 
   /**

However I didn't dig to see if there's a better implementation, or if there's more to change, this just minimally solved my problem.