hookable 5.5.0 → 5.5.2

package/README.md

@@ -2,11 +2,11 @@
 
 [![npm version][npm-version-src]][npm-version-href]
 [![npm downloads][npm-downloads-src]][npm-downloads-href]
-[![packagephobia][packagephobia-src]][packagephobia-href]
-[![Github Actions CI][github-actions-ci-src]][github-actions-ci-href]
+[![bundle][bundle-src]][bundle-href]
 [![Codecov][codecov-src]][codecov-href]
+[![License][license-src]][license-href]
 
-> Awaitable hook system
+Awaitable hooks system.
 
 ## Install
 
@@ -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
@@ -255,17 +259,13 @@ Thanks to [Joe Paice](https://github.com/RGBboy) for donating [hookable](https:/
 MIT - Made with 💖
 
 <!-- Badges -->
-[npm-version-src]: https://flat.badgen.net/npm/dt/hookable
+[npm-version-src]: https://img.shields.io/npm/v/hookable?style=flat&colorA=18181B&colorB=F0DB4F
 [npm-version-href]: https://npmjs.com/package/hookable
-
-[npm-downloads-src]: https://flat.badgen.net/npm/v/hookable
+[npm-downloads-src]: https://img.shields.io/npm/dm/hookable?style=flat&colorA=18181B&colorB=F0DB4F
 [npm-downloads-href]: https://npmjs.com/package/hookable
-
-[github-actions-ci-src]: https://flat.badgen.net/github/checks/unjs/hookable/main
-[github-actions-ci-href]: https://github.com/unjs/hookable/actions
-
-[codecov-src]: https://flat.badgen.net/codecov/c/github/unjs/hookable
-[codecov-href]: https://codecov.io/gh/unjs/hookable
-
-[packagephobia-src]: https://flat.badgen.net/packagephobia/install/hookable
-[packagephobia-href]: https://packagephobia.now.sh/result?p=hookable
+[codecov-src]: https://img.shields.io/codecov/c/gh/unjs/hookable/main?style=flat&colorA=18181B&colorB=F0DB4F
+[codecov-href]: https://codecov.io/gh/unjs/h3
+[bundle-src]: https://img.shields.io/bundlephobia/minzip/hookable?style=flat&colorA=18181B&colorB=F0DB4F
+[bundle-href]: https://bundlephobia.com/result?p=hookable
+[license-src]: https://img.shields.io/github/license/unjs/hookable.svg?style=flat&colorA=18181B&colorB=F0DB4F
+[license-href]: https://github.com/unjs/hookable/blob/main/LICENSE

package/dist/index.cjs

@@ -43,25 +43,27 @@ function serial(tasks, function_) {
 const defaultTask = { run: (function_) => function_() };
 const _createTask = () => defaultTask;
 const createTask = typeof console.createTask !== "undefined" ? console.createTask : _createTask;
-function serialTaskCaller(hooks, name, ...args) {
+function serialTaskCaller(hooks, args) {
+  const name = args.shift();
   const task = createTask(name);
   return hooks.reduce(
     (promise, hookFunction) => promise.then(() => task.run(() => hookFunction(...args))),
     Promise.resolve()
   );
 }
-function parallelTaskCaller(hooks, name, ...args) {
+function parallelTaskCaller(hooks, args) {
+  const name = args.shift();
   const task = createTask(name);
   return Promise.all(hooks.map((hook) => task.run(() => hook(...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) {
@@ -149,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);
     }
@@ -183,9 +185,11 @@ class Hookable {
     }
   }
   callHook(name, ...arguments_) {
+    arguments_.unshift(name);
     return this.callHookWith(serialTaskCaller, name, ...arguments_);
   }
   callHookParallel(name, ...arguments_) {
+    arguments_.unshift(name);
     return this.callHookWith(parallelTaskCaller, name, ...arguments_);
   }
   callHookWith(caller, name, ...arguments_) {
@@ -210,9 +214,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);
+        }
       }
     };
   }
@@ -220,9 +226,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);
+        }
       }
     };
   }
@@ -245,7 +253,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;
@@ -253,7 +261,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

@@ -41,25 +41,27 @@ function serial(tasks, function_) {
 const defaultTask = { run: (function_) => function_() };
 const _createTask = () => defaultTask;
 const createTask = typeof console.createTask !== "undefined" ? console.createTask : _createTask;
-function serialTaskCaller(hooks, name, ...args) {
+function serialTaskCaller(hooks, args) {
+  const name = args.shift();
   const task = createTask(name);
   return hooks.reduce(
     (promise, hookFunction) => promise.then(() => task.run(() => hookFunction(...args))),
     Promise.resolve()
   );
 }
-function parallelTaskCaller(hooks, name, ...args) {
+function parallelTaskCaller(hooks, args) {
+  const name = args.shift();
   const task = createTask(name);
   return Promise.all(hooks.map((hook) => task.run(() => hook(...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) {
@@ -147,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);
     }
@@ -181,9 +183,11 @@ class Hookable {
     }
   }
   callHook(name, ...arguments_) {
+    arguments_.unshift(name);
     return this.callHookWith(serialTaskCaller, name, ...arguments_);
   }
   callHookParallel(name, ...arguments_) {
+    arguments_.unshift(name);
     return this.callHookWith(parallelTaskCaller, name, ...arguments_);
   }
   callHookWith(caller, name, ...arguments_) {
@@ -208,9 +212,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);
+        }
       }
     };
   }
@@ -218,9 +224,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);
+        }
       }
     };
   }
@@ -243,7 +251,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;
@@ -251,7 +259,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.0",
+  "version": "5.5.2",
   "description": "Awaitable hook system",
   "keywords": [
     "hook",
@@ -23,18 +23,19 @@
     "dist"
   ],
   "devDependencies": {
-    "@types/node": "^18.15.0",
+    "@types/node": "^18.15.3",
     "@vitest/coverage-c8": "^0.29.2",
     "changelogen": "^0.5.1",
-    "eslint": "^8.35.0",
+    "eslint": "^8.36.0",
     "eslint-config-unjs": "^0.1.0",
     "expect-type": "^0.15.0",
     "prettier": "^2.8.4",
     "typescript": "^4.9.5",
     "unbuild": "^1.1.2",
+    "vite": "^4.2.1",
     "vitest": "^0.29.2"
   },
-  "packageManager": "pnpm@7.29.1",
+  "packageManager": "pnpm@7.29.2",
   "scripts": {
     "build": "unbuild",
     "dev": "vitest",