chore: lint

Author: pi0

README.md

@@ -203,7 +203,7 @@ Remove all hook handlers.
 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 []
@@ -216,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.json

@@ -34,7 +34,7 @@
   },
   "devDependencies": {
     "@types/node": "^22.14.0",
-    "@vitest/coverage-c8": "^0.33.0",
+    "@vitest/coverage-v8": "3.1.1",
     "changelogen": "^0.6.1",
     "eslint": "^9.23.0",
     "eslint-config-unjs": "^0.4.2",
@@ -46,4 +46,4 @@
     "vitest": "^3.1.1"
   },
   "packageManager": "pnpm@10.7.1"
-}
+}

pnpm-lock.yaml

@@ -1,5 +1,4 @@
 declare global {
-  // eslint-disable-next-line no-unused-vars
   interface Console {
     // https://developer.chrome.com/blog/devtools-modern-web-debugging/#linked-stack-traces
     createTask(name: string): { run: <T extends () => any>(function_: T) => ReturnType<T> }

shims.d.ts

@@ -22,12 +22,13 @@ export interface CreateDebuggerOptions {
   filter?: string | ((event: string) => boolean);
 }
 
+// eslint-disable-next-line unicorn/prefer-global-this
 const isBrowser = typeof window !== "undefined";
 
 /** Start debugging hook names and timing in console */
 export function createDebugger(
   hooks: Hookable<any>,
-  _options: CreateDebuggerOptions = {}
+  _options: CreateDebuggerOptions = {},
 ) {
   const options = <CreateDebuggerOptions>{
     inspect: isBrowser,

src/debugger.ts

@@ -24,7 +24,7 @@ type InferSpyEvent<HT extends Record<string, any>> = {
 
 export class Hookable<
   HooksT extends Record<string, any> = Record<string, HookCallback>,
-  HookNameT extends HookKeys<HooksT> = HookKeys<HooksT>
+  HookNameT extends HookKeys<HooksT> = HookKeys<HooksT>,
 > {
   private _hooks: { [key: string]: HookCallback[] };
   private _before?: HookCallback[];
@@ -48,7 +48,7 @@ export class Hookable<
   hook<NameT extends HookNameT>(
     name: NameT,
     function_: InferCallback<HooksT, NameT>,
-    options: { allowDeprecated?: boolean } = {}
+    options: { allowDeprecated?: boolean } = {},
   ) {
     if (!name || typeof function_ !== "function") {
       return () => {};
@@ -83,7 +83,9 @@ export class Hookable<
           get: () => "_" + name.replace(/\W+/g, "_") + "_hook_cb",
           configurable: true,
         });
-      } catch {}
+      } catch {
+        // ignore
+      }
     }
 
     this._hooks[name] = this._hooks[name] || [];
@@ -100,7 +102,7 @@ export class Hookable<
 
   hookOnce<NameT extends HookNameT>(
     name: NameT,
-    function_: InferCallback<HooksT, NameT>
+    function_: InferCallback<HooksT, NameT>,
   ) {
     let _unreg: (() => void) | undefined;
     let _function: ((...arguments_: any) => any) | undefined = (
@@ -119,7 +121,7 @@ export class Hookable<
 
   removeHook<NameT extends HookNameT>(
     name: NameT,
-    function_: InferCallback<HooksT, NameT>
+    function_: InferCallback<HooksT, NameT>,
   ) {
     if (this._hooks[name]) {
       const index = this._hooks[name].indexOf(function_);
@@ -136,7 +138,7 @@ export class Hookable<
 
   deprecateHook<NameT extends HookNameT>(
     name: NameT,
-    deprecated: HookKeys<HooksT> | DeprecatedHook<HooksT>
+    deprecated: HookKeys<HooksT> | DeprecatedHook<HooksT>,
   ) {
     this._deprecatedHooks[name] =
       typeof deprecated === "string" ? { to: deprecated } : deprecated;
@@ -148,7 +150,7 @@ export class Hookable<
   }
 
   deprecateHooks(
-    deprecatedHooks: Partial<Record<HookNameT, DeprecatedHook<HooksT>>>
+    deprecatedHooks: Partial<Record<HookNameT, DeprecatedHook<HooksT>>>,
   ) {
     Object.assign(this._deprecatedHooks, deprecatedHooks);
     for (const name in deprecatedHooks) {
@@ -160,7 +162,7 @@ export class Hookable<
     const hooks = flatHooks<HooksT>(configHooks);
     // @ts-ignore
     const removeFns = Object.keys(hooks).map((key) =>
-      this.hook(key as HookNameT, hooks[key])
+      this.hook(key as HookNameT, hooks[key]),
     );
 
     return () => {
@@ -206,8 +208,8 @@ export class Hookable<
     NameT extends HookNameT,
     CallFunction extends (
       hooks: HookCallback[],
-      arguments_: Parameters<InferCallback<HooksT, NameT>>
-    ) => any
+      arguments_: Parameters<InferCallback<HooksT, NameT>>,
+    ) => any,
   >(
     caller: CallFunction,
     name: NameT,
@@ -222,7 +224,7 @@ export class Hookable<
     }
     const result = caller(
       name in this._hooks ? [...this._hooks[name]] : [],
-      arguments_
+      arguments_,
     );
     if ((result as any) instanceof Promise) {
       return result.finally(() => {

src/hookable.ts

@@ -4,7 +4,6 @@ export interface Hooks {
 }
 export type HookKeys<T> = keyof T & string;
 export type DeprecatedHook<T> = { message?: string; to: HookKeys<T> };
-// eslint-disable-next-line no-unused-vars
 export type DeprecatedHooks<T> = { [name in HookKeys<T>]: DeprecatedHook<T> };
 
 // Utilities
@@ -14,8 +13,8 @@ type KnownKeys<T> = keyof {
   [K in keyof T as string extends K
     ? never
     : number extends K
-    ? never
-    : K]: never;
+      ? never
+      : K]: never;
 };
 type StripGeneric<T> = Pick<
   T,

src/types.ts

@@ -3,7 +3,7 @@ import type { NestedHooks, HookCallback } from "./types";
 export function flatHooks<T>(
   configHooks: NestedHooks<T>,
   hooks: T = {} as T,
-  parentName?: string
+  parentName?: string,
 ): T {
   for (const key in configHooks) {
     // @ts-ignore
@@ -48,12 +48,12 @@ export function mergeHooks<T>(...hooks: NestedHooks<T>[]): T {
 
 export function serial<T>(
   tasks: T[],
-  function_: (task: T) => Promise<any> | any
+  function_: (task: T) => Promise<any> | any,
 ) {
   // eslint-disable-next-line unicorn/no-array-reduce
   return tasks.reduce(
     (promise, task) => promise.then(() => function_(task)),
-    Promise.resolve()
+    Promise.resolve(),
   );
 }
 
@@ -62,7 +62,7 @@ type CreateTask = typeof console.createTask;
 const defaultTask: ReturnType<CreateTask> = { run: (function_) => function_() };
 const _createTask: CreateTask = () => defaultTask;
 const createTask =
-  typeof console.createTask !== "undefined" ? console.createTask : _createTask;
+  console.createTask === undefined ? _createTask : console.createTask;
 
 export function serialTaskCaller(hooks: HookCallback[], args: any[]) {
   const name = args.shift();
@@ -71,7 +71,7 @@ export function serialTaskCaller(hooks: HookCallback[], args: any[]) {
   return hooks.reduce(
     (promise, hookFunction) =>
       promise.then(() => task.run(() => hookFunction(...args))),
-    Promise.resolve()
+    Promise.resolve(),
   );
 }
 
@@ -87,7 +87,7 @@ export function serialCaller(hooks: HookCallback[], arguments_?: any[]) {
   return hooks.reduce(
     (promise, hookFunction) =>
       promise.then(() => hookFunction(...(arguments_ || []))),
-    Promise.resolve()
+    Promise.resolve(),
   );
 }
 

src/utils.ts

@@ -38,7 +38,7 @@ describe("debugger", () => {
     await hooks.callHook("hook");
     expect(console.time).toBeCalledWith(expect.stringContaining("[tag] hook"));
     expect(console.timeEnd).toBeCalledWith(
-      expect.stringContaining("[tag] hook")
+      expect.stringContaining("[tag] hook"),
     );
   });
   it("should respect `inspect` option", async () => {

test/debuger.test.ts

@@ -1,4 +1,3 @@
-/* eslint-disable unicorn/consistent-function-scoping */
 // @ts-nocheck
 import { describe, test, beforeEach, expect, vi } from "vitest";
 import { createHooks, flatHooks, mergeHooks } from "../src/index";
@@ -64,10 +63,10 @@ describe("core: hookable", () => {
     hook.hook("x", () => {});
 
     expect(console.warn).toBeCalledWith(
-      "a hook has been deprecated, please use c"
+      "a hook has been deprecated, please use c",
     );
     expect(console.warn).toBeCalledWith(
-      "b hook has been deprecated, please use c"
+      "b hook has been deprecated, please use c",
     );
     expect(console.warn).toBeCalledWith("Custom");
     expect(console.warn).toBeCalledTimes(3);
@@ -94,7 +93,7 @@ describe("core: hookable", () => {
     hook.hook("b", () => {});
     hook.deprecateHook("a", "b");
     expect(console.warn).toBeCalledWith(
-      "a hook has been deprecated, please use b"
+      "a hook has been deprecated, please use b",
     );
     expect(hook._hooks.a).toBeUndefined();
     expect(hook._hooks.b).toEqual([expect.any(Function), expect.any(Function)]);
@@ -109,7 +108,7 @@ describe("core: hookable", () => {
     hook.hook("test:hook", () => {});
 
     expect(console.warn).toBeCalledWith(
-      "test:hook hook has been deprecated, please use test:before"
+      "test:hook hook has been deprecated, please use test:before",
     );
     expect(hook._hooks["test:hook"]).toBeUndefined();
     expect(hook._hooks["test:before"]).toEqual([expect.any(Function)]);
@@ -349,16 +348,22 @@ describe("core: hookable", () => {
     const unreg = hook.beforeEach((event) => {
       unreg();
       expect(event.context.count).toBeUndefined();
-      event.name === "test" && x++;
+      if (event.name === "test") {
+        x++;
+      }
       event.context.count = x;
     });
     hook.beforeEach((event) => {
-      event.name === "test" && x++;
+      if (event.name === "test") {
+        x++;
+      }
       event.context.count = x;
     });
     hook.afterEach((event) => {
       expect(event.context.count).toEqual(x);
-      event.name === "test" && x++;
+      if (event.name === "test") {
+        x++;
+      }
     });
 
     await hook.callHook("test");