chore: lint with oxfmt and oxlint

pi0 · pi0 · commit 56c67fbfee5b · 2026-03-15T00:19:22.000+01:00
diff --git a/.oxfmtrc.json b/.oxfmtrc.json
@@ -0,0 +1,4 @@
+{
+  "$schema": "https://unpkg.com/oxfmt/configuration_schema.json",
+  "ignorePatterns": ["CHANGELOG.md"]
+}
diff --git a/.oxlintrc.json b/.oxlintrc.json
@@ -0,0 +1,9 @@
+{
+  "$schema": "https://unpkg.com/oxlint/configuration_schema.json",
+  "plugins": [
+    "unicorn",
+    "typescript",
+    "oxc"
+  ],
+  "rules": {}
+}
diff --git a/.prettierrc b/.prettierrc
diff --git a/CHANGELOG.md b/CHANGELOG.md
diff --git a/eslint.config.mjs b/eslint.config.mjs
diff --git a/renovate.json b/renovate.json
@@ -1,5 +1,3 @@
 {
-  "extends": [
-    "github>unjs/renovate-config"
-  ]
+  "extends": ["github>unjs/renovate-config"]
 }
diff --git a/shims.d.ts b/shims.d.ts
@@ -1,7 +1,7 @@
 declare global {
   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> }
+    createTask(name: string): { run: <T extends () => any>(function_: T) => ReturnType<T> };
   }
 }
 
diff --git a/src/debugger.ts b/src/debugger.ts
@@ -41,14 +41,10 @@ export function createDebugger(
   } satisfies CreateDebuggerOptions;
 
   const _filter = options.filter;
-  const filter =
-    typeof _filter === "string"
-      ? (name: string) => name.startsWith(_filter)
-      : _filter;
+  const filter = typeof _filter === "string" ? (name: string) => name.startsWith(_filter) : _filter;
 
   const _tag = options.tag ? `[${options.tag}] ` : "";
-  const logPrefix = (event: any) =>
-    _tag + event.name + "".padEnd(event._id, "\0");
+  const logPrefix = (event: any) => _tag + event.name + "".padEnd(event._id, "\0");
 
   const _idCtr: Record<string, number> = {};
 
diff --git a/src/hookable.ts b/src/hookable.ts
@@ -6,16 +6,9 @@ import {
   callHooks,
 } from "./utils.ts";
 
-import type {
-  DeprecatedHook,
-  NestedHooks,
-  HookCallback,
-  HookKeys,
-} from "./types.ts";
-
-type InferCallback<HT, HN extends keyof HT> = HT[HN] extends HookCallback
-  ? HT[HN]
-  : never;
+import type { DeprecatedHook, NestedHooks, HookCallback, HookKeys } from "./types.ts";
+
+type InferCallback<HT, HN extends keyof HT> = HT[HN] extends HookCallback ? HT[HN] : never;
 type InferSpyEvent<HT extends Record<string, any>> = {
   [key in keyof HT]: {
     name: key;
@@ -66,8 +59,7 @@ export class Hookable<
       let message = dep.message;
       if (!message) {
         message =
-          `${originalName} hook has been deprecated` +
-          (dep.to ? `, please use ${dep.to}` : "");
+          `${originalName} hook has been deprecated` + (dep.to ? `, please use ${dep.to}` : "");
       }
       if (!this._deprecatedMessages) {
         this._deprecatedMessages = new Set();
@@ -107,9 +99,7 @@ export class Hookable<
     function_: InferCallback<HooksT, NameT>,
   ): () => void {
     let _unreg: (() => void) | undefined;
-    let _function: ((...arguments_: any) => any) | undefined = (
-      ...arguments_: any
-    ) => {
+    let _function: ((...arguments_: any) => any) | undefined = (...arguments_: any) => {
       if (typeof _unreg === "function") {
         _unreg();
       }
@@ -121,10 +111,7 @@ export class Hookable<
     return _unreg;
   }
 
-  removeHook<NameT extends HookNameT>(
-    name: NameT,
-    function_: InferCallback<HooksT, NameT>,
-  ): void {
+  removeHook<NameT extends HookNameT>(name: NameT, function_: InferCallback<HooksT, NameT>): void {
     const hooks = this._hooks[name];
 
     if (hooks) {
@@ -148,18 +135,15 @@ export class Hookable<
     name: NameT,
     deprecated: HookKeys<HooksT> | DeprecatedHook<HooksT>,
   ): void {
-    this._deprecatedHooks[name] =
-      typeof deprecated === "string" ? { to: deprecated } : deprecated;
+    this._deprecatedHooks[name] = typeof deprecated === "string" ? { to: deprecated } : deprecated;
     const _hooks = this._hooks[name] || [];
     this._hooks[name] = undefined;
     for (const hook of _hooks) {
       this.hook(name, hook as any);
     }
   }
 
-  deprecateHooks(
-    deprecatedHooks: Partial<Record<HookNameT, DeprecatedHook<HooksT>>>,
-  ): void {
+  deprecateHooks(deprecatedHooks: Partial<Record<HookNameT, DeprecatedHook<HooksT>>>): void {
     for (const name in deprecatedHooks) {
       this.deprecateHook(name, deprecatedHooks[name] as DeprecatedHook<HooksT>);
     }
@@ -168,9 +152,7 @@ export class Hookable<
   addHooks(configHooks: NestedHooks<HooksT>): () => void {
     const hooks = flatHooks<HooksT>(configHooks);
     // @ts-ignore
-    const removeFns = Object.keys(hooks).map((key) =>
-      this.hook(key as HookNameT, hooks[key]),
-    );
+    const removeFns = Object.keys(hooks).map((key) => this.hook(key as HookNameT, hooks[key]));
 
     return () => {
       for (const unreg of removeFns) {
@@ -221,8 +203,7 @@ export class Hookable<
     name: NameT,
     args: Parameters<InferCallback<HooksT, NameT>>,
   ): ReturnType<CallFunction> {
-    const event =
-      this._before || this._after ? { name, args, context: {} } : undefined;
+    const event = this._before || this._after ? { name, args, context: {} } : undefined;
     if (this._before) {
       callEachWith(this._before, event);
     }
@@ -285,10 +266,7 @@ export class HookableCore<
     this._hooks = {};
   }
 
-  hook<NameT extends HookNameT>(
-    name: NameT,
-    fn: InferCallback<HooksT, NameT>,
-  ): () => void {
+  hook<NameT extends HookNameT>(name: NameT, fn: InferCallback<HooksT, NameT>): () => void {
     if (!name || typeof fn !== "function") {
       return () => {};
     }
@@ -303,10 +281,7 @@ export class HookableCore<
     };
   }
 
-  removeHook<NameT extends HookNameT>(
-    name: NameT,
-    function_: InferCallback<HooksT, NameT>,
-  ): void {
+  removeHook<NameT extends HookNameT>(name: NameT, function_: InferCallback<HooksT, NameT>): void {
     const hooks = this._hooks[name];
 
     if (hooks) {
diff --git a/src/index.ts b/src/index.ts
@@ -1,12 +1,6 @@
 export { Hookable, HookableCore, createHooks } from "./hookable.ts";
 
-export {
-  flatHooks,
-  mergeHooks,
-  parallelCaller,
-  serial,
-  serialCaller,
-} from "./utils.ts";
+export { flatHooks, mergeHooks, parallelCaller, serial, serialCaller } from "./utils.ts";
 
 export * from "./debugger.ts";
 
diff --git a/src/types.ts b/src/types.ts
@@ -10,45 +10,28 @@ export type DeprecatedHooks<T> = { [name in HookKeys<T>]: DeprecatedHook<T> };
 type ValueOf<C> = C extends Record<any, any> ? C[keyof C] : never;
 type Strings<T> = Exclude<keyof T, number | symbol>;
 type KnownKeys<T> = keyof {
-  [K in keyof T as string extends K
-    ? never
-    : number extends K
-      ? never
-      : K]: never;
+  [K in keyof T as string extends K ? never : number extends K ? never : K]: never;
 };
-type StripGeneric<T> = Pick<
-  T,
-  KnownKeys<T> extends keyof T ? KnownKeys<T> : never
->;
-type OnlyGeneric<T> = Omit<
-  T,
-  KnownKeys<T> extends keyof T ? KnownKeys<T> : never
->;
+type StripGeneric<T> = Pick<T, KnownKeys<T> extends keyof T ? KnownKeys<T> : never>;
+type OnlyGeneric<T> = Omit<T, KnownKeys<T> extends keyof T ? KnownKeys<T> : never>;
 
 // Unwrapping utilities
 type Namespaces<T> = ValueOf<{
-  [key in Strings<T>]: key extends `${infer Namespace}:${string}`
-    ? Namespace
-    : never;
+  [key in Strings<T>]: key extends `${infer Namespace}:${string}` ? Namespace : never;
 }>;
 type BareHooks<T> = ValueOf<{
   [key in Strings<T>]: key extends `${string}:${string}` ? never : key;
 }>;
 type HooksInNamespace<T, Namespace extends string> = ValueOf<{
-  [key in Strings<T>]: key extends `${Namespace}:${infer HookName}`
-    ? HookName
-    : never;
+  [key in Strings<T>]: key extends `${Namespace}:${infer HookName}` ? HookName : never;
 }>;
 type WithoutNamespace<T, Namespace extends string> = {
   [key in HooksInNamespace<T, Namespace>]: `${Namespace}:${key}` extends keyof T
     ? T[`${Namespace}:${key}`]
     : never;
 };
 
-export type NestedHooks<T> = (
-  | Partial<StripGeneric<T>>
-  | Partial<OnlyGeneric<T>>
-) &
+export type NestedHooks<T> = (Partial<StripGeneric<T>> | Partial<OnlyGeneric<T>>) &
   Partial<{
     [key in Namespaces<StripGeneric<T>>]: NestedHooks<WithoutNamespace<T, key>>;
   }> &
diff --git a/src/utils.ts b/src/utils.ts
@@ -46,15 +46,9 @@ export function mergeHooks<T>(...hooks: NestedHooks<T>[]): T {
   return finalHooks as any;
 }
 
-export function serial<T>(
-  tasks: T[],
-  function_: (task: T) => Promise<any> | any,
-): Promise<any> {
+export function serial<T>(tasks: T[], function_: (task: T) => Promise<any> | any): Promise<any> {
   // eslint-disable-next-line unicorn/no-array-reduce
-  return tasks.reduce(
-    (promise, task) => promise.then(() => function_(task)),
-    Promise.resolve(),
-  );
+  return tasks.reduce((promise, task) => promise.then(() => function_(task)), Promise.resolve());
 }
 
 // https://developer.chrome.com/blog/devtools-modern-web-debugging/#linked-stack-traces
@@ -85,9 +79,7 @@ export function callHooks(
 ): Promise<any> | void {
   for (let i = startIndex; i < hooks.length; i += 1) {
     try {
-      const result = task
-        ? task.run(() => hooks[i](...args))
-        : hooks[i](...args);
+      const result = task ? task.run(() => hooks[i](...args)) : hooks[i](...args);
       if (result instanceof Promise) {
         return result.then(() => callHooks(hooks, args, i + 1, task));
       }
@@ -119,30 +111,20 @@ export function parallelTaskCaller(
 }
 
 /** @deprecated */
-export function serialCaller(
-  hooks: HookCallback[],
-  arguments_?: any[],
-): Promise<any> {
+export function serialCaller(hooks: HookCallback[], arguments_?: any[]): Promise<any> {
   // eslint-disable-next-line unicorn/no-array-reduce
   return hooks.reduce(
-    (promise, hookFunction) =>
-      promise.then(() => hookFunction(...(arguments_ || []))),
+    (promise, hookFunction) => promise.then(() => hookFunction(...(arguments_ || []))),
     Promise.resolve(),
   );
 }
 
 /** @deprecated */
-export function parallelCaller(
-  hooks: HookCallback[],
-  args?: any[],
-): Promise<any> {
+export function parallelCaller(hooks: HookCallback[], args?: any[]): Promise<any> {
   return Promise.all(hooks.map((hook) => hook(...(args || []))));
 }
 
-export function callEachWith(
-  callbacks: Array<(arg0: any) => any>,
-  arg0?: any,
-): void {
+export function callEachWith(callbacks: Array<(arg0: any) => any>, arg0?: any): void {
   // eslint-disable-next-line unicorn/no-useless-spread
   for (const callback of [...callbacks]) {
     callback(arg0);
diff --git a/test/bench.ts b/test/bench.ts
@@ -16,15 +16,9 @@ for (const instance of Object.values(instances)) {
 
 summary(() => {
   compact(() => {
-    bench("Hookable", () =>
-      do_not_optimize(instances.hookable.callHook("test", {})),
-    );
-    bench("HookableCore", () =>
-      do_not_optimize(instances.hookableCore.callHook("test", {})),
-    );
-    bench("HookablePrev", () =>
-      do_not_optimize(instances.hookablePrev.callHook("test", {})),
-    );
+    bench("Hookable", () => do_not_optimize(instances.hookable.callHook("test", {})));
+    bench("HookableCore", () => do_not_optimize(instances.hookableCore.callHook("test", {})));
+    bench("HookablePrev", () => do_not_optimize(instances.hookablePrev.callHook("test", {})));
   });
 });
 
diff --git a/test/debuger.test.ts b/test/debuger.test.ts
@@ -1,21 +1,7 @@
-import {
-  afterAll,
-  beforeAll,
-  describe,
-  it,
-  beforeEach,
-  expect,
-  vi,
-} from "vitest";
+import { afterAll, beforeAll, describe, it, beforeEach, expect, vi } from "vitest";
 import { createDebugger, Hookable } from "../src/index.ts";
 
-const consoleMethods = [
-  "time",
-  "timeEnd",
-  "timeLog",
-  "groupCollapsed",
-  "groupEnd",
-] as const;
+const consoleMethods = ["time", "timeEnd", "timeLog", "groupCollapsed", "groupEnd"] as const;
 
 describe("debugger", () => {
   let hooks: Hookable;
@@ -37,9 +23,7 @@ describe("debugger", () => {
     createDebugger(hooks, { tag: "tag" });
     await hooks.callHook("hook");
     expect(console.time).toBeCalledWith(expect.stringContaining("[tag] hook"));
-    expect(console.timeEnd).toBeCalledWith(
-      expect.stringContaining("[tag] hook"),
-    );
+    expect(console.timeEnd).toBeCalledWith(expect.stringContaining("[tag] hook"));
   });
   it("should respect `inspect` option", async () => {
     createDebugger(hooks, { inspect: true });
diff --git a/test/hookable.test.ts b/test/hookable.test.ts

-Original file line number
+Diff line change
@@ @@ -0,0 +1,4 @@ @@
 +{
 +  "$schema": "https://unpkg.com/oxfmt/configuration_schema.json",
 +  "ignorePatterns": ["CHANGELOG.md"]
 +}
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,3 @@`
`1`	`1`	`{`
`2`		`- "extends": [`
`3`		`- "github>unjs/renovate-config"`
`4`		`- ]`
	`2`	`+ "extends": ["github>unjs/renovate-config"]`
`5`	`3`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,7 @@`
`1`	`1`	`declare global {`
`2`	`2`	`interface Console {`
`3`	`3`	`// https://developer.chrome.com/blog/devtools-modern-web-debugging/#linked-stack-traces`
`4`		`- createTask(name: string): { run: <T extends () => any>(function_: T) => ReturnType<T> }`
	`4`	`+ createTask(name: string): { run: <T extends () => any>(function_: T) => ReturnType<T> };`
`5`	`5`	`}`
`6`	`6`	`}`
`7`	`7`