hookable 5.5.1 → 5.5.3

package/README.md

@@ -194,12 +194,16 @@ hookable.removeHooks({
 })
 ```
 
+### `removeAllHooks`
+
+Remove all hook handlers.
+
 ### `beforeEach (syncCallback)`
 
 Registers a (sync) callback to be called before each hook is being called.
 
 ```js
-hookable.beforeEach((event) => { console.log(`${event.name} hook is being called with ${event.args}`)})
+hookable.beforeEach((event) => { console.log(`${event.name} hook is being called with ${event.args}`)}`)
 hookable.hook('test', () => { console.log('running test hook') })
 
 // test hook is being called with []
@@ -212,7 +216,7 @@ await hookable.callHook('test')
 Registers a (sync) callback to be called after each hook is being called.
 
 ```js
-hookable.afterEach((event) => { console.log(`${event.name} hook called with ${event.args}`)})
+hookable.afterEach((event) => { console.log(`${event.name} hook called with ${event.args}`)}`)
 hookable.hook('test', () => { console.log('running test hook') })
 
 // running test hook

package/dist/index.cjs

@@ -58,15 +58,15 @@ function parallelTaskCaller(hooks, args) {
 }
 function serialCaller(hooks, arguments_) {
   return hooks.reduce(
-    (promise, hookFunction) => promise.then(() => hookFunction(...arguments_)),
+    (promise, hookFunction) => promise.then(() => hookFunction(...arguments_ || [])),
     Promise.resolve()
   );
 }
 function parallelCaller(hooks, args) {
-  return Promise.all(hooks.map((hook) => hook(...args)));
+  return Promise.all(hooks.map((hook) => hook(...args || [])));
 }
 function callEachWith(callbacks, arg0) {
-  for (const callback of callbacks) {
+  for (const callback of [...callbacks]) {
     callback(arg0);
   }
 }
@@ -151,7 +151,7 @@ class Hookable {
   deprecateHook(name, deprecated) {
     this._deprecatedHooks[name] = typeof deprecated === "string" ? { to: deprecated } : deprecated;
     const _hooks = this._hooks[name] || [];
-    this._hooks[name] = void 0;
+    delete this._hooks[name];
     for (const hook of _hooks) {
       this.hook(name, hook);
     }
@@ -197,7 +197,10 @@ class Hookable {
     if (this._before) {
       callEachWith(this._before, event);
     }
-    const result = caller(this._hooks[name] || [], arguments_);
+    const result = caller(
+      name in this._hooks ? [...this._hooks[name]] : [],
+      arguments_
+    );
     if (result instanceof Promise) {
       return result.finally(() => {
         if (this._after && event) {
@@ -214,9 +217,11 @@ class Hookable {
     this._before = this._before || [];
     this._before.push(function_);
     return () => {
-      const index = this._before.indexOf(function_);
-      if (index !== -1) {
-        this._before.splice(index, 1);
+      if (this._before !== void 0) {
+        const index = this._before.indexOf(function_);
+        if (index !== -1) {
+          this._before.splice(index, 1);
+        }
       }
     };
   }
@@ -224,9 +229,11 @@ class Hookable {
     this._after = this._after || [];
     this._after.push(function_);
     return () => {
-      const index = this._after.indexOf(function_);
-      if (index !== -1) {
-        this._after.splice(index, 1);
+      if (this._after !== void 0) {
+        const index = this._after.indexOf(function_);
+        if (index !== -1) {
+          this._after.splice(index, 1);
+        }
       }
     };
   }
@@ -249,7 +256,7 @@ function createDebugger(hooks, _options = {}) {
   const logPrefix = (event) => _tag + event.name + "".padEnd(event._id, "\0");
   const _idCtr = {};
   const unsubscribeBefore = hooks.beforeEach((event) => {
-    if (!filter(event.name)) {
+    if (filter !== void 0 && !filter(event.name)) {
       return;
     }
     _idCtr[event.name] = _idCtr[event.name] || 0;
@@ -257,7 +264,7 @@ function createDebugger(hooks, _options = {}) {
     console.time(logPrefix(event));
   });
   const unsubscribeAfter = hooks.afterEach((event) => {
-    if (!filter(event.name)) {
+    if (filter !== void 0 && !filter(event.name)) {
       return;
     }
     if (options.group) {

package/dist/index.d.ts

@@ -43,12 +43,12 @@ type InferSpyEvent<HT extends Record<string, any>> = {
         context: Record<string, any>;
     };
 }[keyof HT];
-declare class Hookable<HooksT = Record<string, HookCallback>, HookNameT extends HookKeys<HooksT> = HookKeys<HooksT>> {
+declare class Hookable<HooksT extends Record<string, any> = Record<string, HookCallback>, HookNameT extends HookKeys<HooksT> = HookKeys<HooksT>> {
     private _hooks;
-    private _before;
-    private _after;
+    private _before?;
+    private _after?;
     private _deprecatedHooks;
-    private _deprecatedMessages;
+    private _deprecatedMessages?;
     constructor();
     hook<NameT extends HookNameT>(name: NameT, function_: InferCallback<HooksT, NameT>, options?: {
         allowDeprecated?: boolean;
@@ -66,7 +66,7 @@ declare class Hookable<HooksT = Record<string, HookCallback>, HookNameT extends
     beforeEach(function_: (event: InferSpyEvent<HooksT>) => void): () => void;
     afterEach(function_: (event: InferSpyEvent<HooksT>) => void): () => void;
 }
-declare function createHooks<T>(): Hookable<T>;
+declare function createHooks<T extends Record<string, any>>(): Hookable<T>;
 
 declare function flatHooks<T>(configHooks: NestedHooks<T>, hooks?: T, parentName?: string): T;
 declare function mergeHooks<T>(...hooks: NestedHooks<T>[]): T;

package/dist/index.mjs

@@ -56,15 +56,15 @@ function parallelTaskCaller(hooks, args) {
 }
 function serialCaller(hooks, arguments_) {
   return hooks.reduce(
-    (promise, hookFunction) => promise.then(() => hookFunction(...arguments_)),
+    (promise, hookFunction) => promise.then(() => hookFunction(...arguments_ || [])),
     Promise.resolve()
   );
 }
 function parallelCaller(hooks, args) {
-  return Promise.all(hooks.map((hook) => hook(...args)));
+  return Promise.all(hooks.map((hook) => hook(...args || [])));
 }
 function callEachWith(callbacks, arg0) {
-  for (const callback of callbacks) {
+  for (const callback of [...callbacks]) {
     callback(arg0);
   }
 }
@@ -149,7 +149,7 @@ class Hookable {
   deprecateHook(name, deprecated) {
     this._deprecatedHooks[name] = typeof deprecated === "string" ? { to: deprecated } : deprecated;
     const _hooks = this._hooks[name] || [];
-    this._hooks[name] = void 0;
+    delete this._hooks[name];
     for (const hook of _hooks) {
       this.hook(name, hook);
     }
@@ -195,7 +195,10 @@ class Hookable {
     if (this._before) {
       callEachWith(this._before, event);
     }
-    const result = caller(this._hooks[name] || [], arguments_);
+    const result = caller(
+      name in this._hooks ? [...this._hooks[name]] : [],
+      arguments_
+    );
     if (result instanceof Promise) {
       return result.finally(() => {
         if (this._after && event) {
@@ -212,9 +215,11 @@ class Hookable {
     this._before = this._before || [];
     this._before.push(function_);
     return () => {
-      const index = this._before.indexOf(function_);
-      if (index !== -1) {
-        this._before.splice(index, 1);
+      if (this._before !== void 0) {
+        const index = this._before.indexOf(function_);
+        if (index !== -1) {
+          this._before.splice(index, 1);
+        }
       }
     };
   }
@@ -222,9 +227,11 @@ class Hookable {
     this._after = this._after || [];
     this._after.push(function_);
     return () => {
-      const index = this._after.indexOf(function_);
-      if (index !== -1) {
-        this._after.splice(index, 1);
+      if (this._after !== void 0) {
+        const index = this._after.indexOf(function_);
+        if (index !== -1) {
+          this._after.splice(index, 1);
+        }
       }
     };
   }
@@ -247,7 +254,7 @@ function createDebugger(hooks, _options = {}) {
   const logPrefix = (event) => _tag + event.name + "".padEnd(event._id, "\0");
   const _idCtr = {};
   const unsubscribeBefore = hooks.beforeEach((event) => {
-    if (!filter(event.name)) {
+    if (filter !== void 0 && !filter(event.name)) {
       return;
     }
     _idCtr[event.name] = _idCtr[event.name] || 0;
@@ -255,7 +262,7 @@ function createDebugger(hooks, _options = {}) {
     console.time(logPrefix(event));
   });
   const unsubscribeAfter = hooks.afterEach((event) => {
-    if (!filter(event.name)) {
+    if (filter !== void 0 && !filter(event.name)) {
       return;
     }
     if (options.group) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hookable",
-  "version": "5.5.1",
+  "version": "5.5.3",
   "description": "Awaitable hook system",
   "keywords": [
     "hook",
@@ -23,18 +23,19 @@
     "dist"
   ],
   "devDependencies": {
-    "@types/node": "^18.15.0",
-    "@vitest/coverage-c8": "^0.29.2",
-    "changelogen": "^0.5.1",
-    "eslint": "^8.35.0",
+    "@types/node": "^18.15.11",
+    "@vitest/coverage-c8": "^0.29.8",
+    "changelogen": "^0.5.2",
+    "eslint": "^8.37.0",
     "eslint-config-unjs": "^0.1.0",
     "expect-type": "^0.15.0",
-    "prettier": "^2.8.4",
-    "typescript": "^4.9.5",
+    "prettier": "^2.8.7",
+    "typescript": "^5.0.2",
     "unbuild": "^1.1.2",
-    "vitest": "^0.29.2"
+    "vite": "^4.2.1",
+    "vitest": "^0.29.8"
   },
-  "packageManager": "pnpm@7.29.1",
+  "packageManager": "pnpm@8.0.0",
   "scripts": {
     "build": "unbuild",
     "dev": "vitest",