hookified 1.12.2 → 1.14.0

package/dist/node/index.d.cts

@@ -6,7 +6,21 @@ type Logger = {
     error: (message: string, ...arguments_: unknown[]) => void;
     fatal: (message: string, ...arguments_: unknown[]) => void;
 };
-
+type EventEmitterOptions = {
+    /**
+     * Logger instance for logging errors.
+     */
+    logger?: Logger;
+    /**
+     * Whether to throw an error when emit 'error' and there are no listeners. Default is false and only emits an error event.
+     */
+    throwOnEmitError?: boolean;
+    /**
+     * Whether to throw on 'error' when there are no listeners. This is the standard functionality in EventEmitter
+     * @default false - in v2 this will be set to true by default
+     */
+    throwOnEmptyListeners?: boolean;
+};
 type IEventEmitter = {
     /**
      * Registers a listener for the specified event.
@@ -152,21 +166,54 @@ type IEventEmitter = {
     prependOnceListener(eventName: string | symbol, listener: (...arguments_: any[]) => void): IEventEmitter;
 };
 type EventListener = (...arguments_: any[]) => void;
-type EventEmitterOptions = {
+type Hook = (...arguments_: any[]) => Promise<void> | void;
+type HookEntry = {
     /**
-     * Logger instance for logging errors.
+     * The event name for the hook
      */
-    logger?: Logger;
+    event: string;
     /**
-     * Whether to throw an error when emit 'error' and there are no listeners. Default is false and only emits an error event.
+     * The handler function for the hook
      */
-    throwOnEmitError?: boolean;
+    handler: Hook;
 };
+type HookifiedOptions = {
+    /**
+     * Whether an error should be thrown when a hook throws an error. Default is false and only emits an error event.
+     * @deprecated - this will be deprecated in version 2. Please use throwOnHookError.
+     */
+    throwHookErrors?: boolean;
+    /**
+     * Whether an error should be thrown when a hook throws an error. Default is false and only emits an error event.
+     */
+    throwOnHookError?: boolean;
+    /**
+     * Whether to enforce that all hook names start with 'before' or 'after'. Default is false.
+     * @type {boolean}
+     * @default false
+     */
+    enforceBeforeAfter?: boolean;
+    /**
+     * Map of deprecated hook names to deprecation messages. When a deprecated hook is used, a warning will be emitted.
+     * @type {Map<string, string>}
+     * @default new Map()
+     */
+    deprecatedHooks?: Map<string, string>;
+    /**
+     * Whether to allow deprecated hooks to be registered and executed. Default is true.
+     * @type {boolean}
+     * @default true
+     */
+    allowDeprecated?: boolean;
+} & EventEmitterOptions;
+
 declare class Eventified implements IEventEmitter {
     private readonly _eventListeners;
     private _maxListeners;
     private _logger?;
     private _throwOnEmitError;
+    private _throwOnEmptyListeners;
+    private _errorEvent;
     constructor(options?: EventEmitterOptions);
     /**
      * Gets the logger
@@ -188,6 +235,16 @@ declare class Eventified implements IEventEmitter {
      * @param {boolean} value
      */
     set throwOnEmitError(value: boolean);
+    /**
+     * Gets whether an error should be thrown when emitting 'error' event with no listeners. Default is false.
+     * @returns {boolean}
+     */
+    get throwOnEmptyListeners(): boolean;
+    /**
+     * Sets whether an error should be thrown when emitting 'error' event with no listeners. Default is false.
+     * @param {boolean} value
+     */
+    set throwOnEmptyListeners(value: boolean);
     /**
      * Adds a handler function for a specific event that will run only once
      * @param {string | symbol} eventName
@@ -289,46 +346,17 @@ declare class Eventified implements IEventEmitter {
      * @returns {EventListener[]} An array of listeners
      */
     getAllListeners(): EventListener[];
-}
-
-type Hook = (...arguments_: any[]) => Promise<void> | void;
-type HookEntry = {
-    /**
-     * The event name for the hook
-     */
-    event: string;
     /**
-     * The handler function for the hook
+     * Sends a log message using the configured logger based on the event name
+     * @param {string | symbol} eventName - The event name that determines the log level
+     * @param {unknown} data - The data to log
      */
-    handler: Hook;
-};
-type HookifiedOptions = {
-    /**
-     * Whether an error should be thrown when a hook throws an error. Default is false and only emits an error event.
-     */
-    throwHookErrors?: boolean;
-    /**
-     * Whether to enforce that all hook names start with 'before' or 'after'. Default is false.
-     * @type {boolean}
-     * @default false
-     */
-    enforceBeforeAfter?: boolean;
-    /**
-     * Map of deprecated hook names to deprecation messages. When a deprecated hook is used, a warning will be emitted.
-     * @type {Map<string, string>}
-     * @default new Map()
-     */
-    deprecatedHooks?: Map<string, string>;
-    /**
-     * Whether to allow deprecated hooks to be registered and executed. Default is true.
-     * @type {boolean}
-     * @default true
-     */
-    allowDeprecated?: boolean;
-} & EventEmitterOptions;
+    private sendLog;
+}
+
 declare class Hookified extends Eventified {
     private readonly _hooks;
-    private _throwHookErrors;
+    private _throwOnHookError;
     private _enforceBeforeAfter;
     private _deprecatedHooks;
     private _allowDeprecated;
@@ -341,13 +369,25 @@ declare class Hookified extends Eventified {
     /**
      * Gets whether an error should be thrown when a hook throws an error. Default is false and only emits an error event.
      * @returns {boolean}
+     * @deprecated - this will be deprecated in version 2. Please use throwOnHookError.
      */
     get throwHookErrors(): boolean;
     /**
      * Sets whether an error should be thrown when a hook throws an error. Default is false and only emits an error event.
      * @param {boolean} value
+     * @deprecated - this will be deprecated in version 2. Please use throwOnHookError.
      */
     set throwHookErrors(value: boolean);
+    /**
+     * Gets whether an error should be thrown when a hook throws an error. Default is false and only emits an error event.
+     * @returns {boolean}
+     */
+    get throwOnHookError(): boolean;
+    /**
+     * Sets whether an error should be thrown when a hook throws an error. Default is false and only emits an error event.
+     * @param {boolean} value
+     */
+    set throwOnHookError(value: boolean);
     /**
      * Gets whether to enforce that all hook names start with 'before' or 'after'. Default is false.
      * @returns {boolean}
@@ -489,4 +529,4 @@ declare class Hookified extends Eventified {
     clearHooks(): void;
 }
 
-export { type EventListener, Eventified, type Hook, type HookEntry, Hookified, type HookifiedOptions, type Logger };
+export { type EventEmitterOptions, type EventListener, Eventified, type Hook, type HookEntry, Hookified, type HookifiedOptions, type IEventEmitter, type Logger };

package/dist/node/index.d.ts

@@ -6,7 +6,21 @@ type Logger = {
     error: (message: string, ...arguments_: unknown[]) => void;
     fatal: (message: string, ...arguments_: unknown[]) => void;
 };
-
+type EventEmitterOptions = {
+    /**
+     * Logger instance for logging errors.
+     */
+    logger?: Logger;
+    /**
+     * Whether to throw an error when emit 'error' and there are no listeners. Default is false and only emits an error event.
+     */
+    throwOnEmitError?: boolean;
+    /**
+     * Whether to throw on 'error' when there are no listeners. This is the standard functionality in EventEmitter
+     * @default false - in v2 this will be set to true by default
+     */
+    throwOnEmptyListeners?: boolean;
+};
 type IEventEmitter = {
     /**
      * Registers a listener for the specified event.
@@ -152,21 +166,54 @@ type IEventEmitter = {
     prependOnceListener(eventName: string | symbol, listener: (...arguments_: any[]) => void): IEventEmitter;
 };
 type EventListener = (...arguments_: any[]) => void;
-type EventEmitterOptions = {
+type Hook = (...arguments_: any[]) => Promise<void> | void;
+type HookEntry = {
     /**
-     * Logger instance for logging errors.
+     * The event name for the hook
      */
-    logger?: Logger;
+    event: string;
     /**
-     * Whether to throw an error when emit 'error' and there are no listeners. Default is false and only emits an error event.
+     * The handler function for the hook
      */
-    throwOnEmitError?: boolean;
+    handler: Hook;
 };
+type HookifiedOptions = {
+    /**
+     * Whether an error should be thrown when a hook throws an error. Default is false and only emits an error event.
+     * @deprecated - this will be deprecated in version 2. Please use throwOnHookError.
+     */
+    throwHookErrors?: boolean;
+    /**
+     * Whether an error should be thrown when a hook throws an error. Default is false and only emits an error event.
+     */
+    throwOnHookError?: boolean;
+    /**
+     * Whether to enforce that all hook names start with 'before' or 'after'. Default is false.
+     * @type {boolean}
+     * @default false
+     */
+    enforceBeforeAfter?: boolean;
+    /**
+     * Map of deprecated hook names to deprecation messages. When a deprecated hook is used, a warning will be emitted.
+     * @type {Map<string, string>}
+     * @default new Map()
+     */
+    deprecatedHooks?: Map<string, string>;
+    /**
+     * Whether to allow deprecated hooks to be registered and executed. Default is true.
+     * @type {boolean}
+     * @default true
+     */
+    allowDeprecated?: boolean;
+} & EventEmitterOptions;
+
 declare class Eventified implements IEventEmitter {
     private readonly _eventListeners;
     private _maxListeners;
     private _logger?;
     private _throwOnEmitError;
+    private _throwOnEmptyListeners;
+    private _errorEvent;
     constructor(options?: EventEmitterOptions);
     /**
      * Gets the logger
@@ -188,6 +235,16 @@ declare class Eventified implements IEventEmitter {
      * @param {boolean} value
      */
     set throwOnEmitError(value: boolean);
+    /**
+     * Gets whether an error should be thrown when emitting 'error' event with no listeners. Default is false.
+     * @returns {boolean}
+     */
+    get throwOnEmptyListeners(): boolean;
+    /**
+     * Sets whether an error should be thrown when emitting 'error' event with no listeners. Default is false.
+     * @param {boolean} value
+     */
+    set throwOnEmptyListeners(value: boolean);
     /**
      * Adds a handler function for a specific event that will run only once
      * @param {string | symbol} eventName
@@ -289,46 +346,17 @@ declare class Eventified implements IEventEmitter {
      * @returns {EventListener[]} An array of listeners
      */
     getAllListeners(): EventListener[];
-}
-
-type Hook = (...arguments_: any[]) => Promise<void> | void;
-type HookEntry = {
-    /**
-     * The event name for the hook
-     */
-    event: string;
     /**
-     * The handler function for the hook
+     * Sends a log message using the configured logger based on the event name
+     * @param {string | symbol} eventName - The event name that determines the log level
+     * @param {unknown} data - The data to log
      */
-    handler: Hook;
-};
-type HookifiedOptions = {
-    /**
-     * Whether an error should be thrown when a hook throws an error. Default is false and only emits an error event.
-     */
-    throwHookErrors?: boolean;
-    /**
-     * Whether to enforce that all hook names start with 'before' or 'after'. Default is false.
-     * @type {boolean}
-     * @default false
-     */
-    enforceBeforeAfter?: boolean;
-    /**
-     * Map of deprecated hook names to deprecation messages. When a deprecated hook is used, a warning will be emitted.
-     * @type {Map<string, string>}
-     * @default new Map()
-     */
-    deprecatedHooks?: Map<string, string>;
-    /**
-     * Whether to allow deprecated hooks to be registered and executed. Default is true.
-     * @type {boolean}
-     * @default true
-     */
-    allowDeprecated?: boolean;
-} & EventEmitterOptions;
+    private sendLog;
+}
+
 declare class Hookified extends Eventified {
     private readonly _hooks;
-    private _throwHookErrors;
+    private _throwOnHookError;
     private _enforceBeforeAfter;
     private _deprecatedHooks;
     private _allowDeprecated;
@@ -341,13 +369,25 @@ declare class Hookified extends Eventified {
     /**
      * Gets whether an error should be thrown when a hook throws an error. Default is false and only emits an error event.
      * @returns {boolean}
+     * @deprecated - this will be deprecated in version 2. Please use throwOnHookError.
      */
     get throwHookErrors(): boolean;
     /**
      * Sets whether an error should be thrown when a hook throws an error. Default is false and only emits an error event.
      * @param {boolean} value
+     * @deprecated - this will be deprecated in version 2. Please use throwOnHookError.
      */
     set throwHookErrors(value: boolean);
+    /**
+     * Gets whether an error should be thrown when a hook throws an error. Default is false and only emits an error event.
+     * @returns {boolean}
+     */
+    get throwOnHookError(): boolean;
+    /**
+     * Sets whether an error should be thrown when a hook throws an error. Default is false and only emits an error event.
+     * @param {boolean} value
+     */
+    set throwOnHookError(value: boolean);
     /**
      * Gets whether to enforce that all hook names start with 'before' or 'after'. Default is false.
      * @returns {boolean}
@@ -489,4 +529,4 @@ declare class Hookified extends Eventified {
     clearHooks(): void;
 }
 
-export { type EventListener, Eventified, type Hook, type HookEntry, Hookified, type HookifiedOptions, type Logger };
+export { type EventEmitterOptions, type EventListener, Eventified, type Hook, type HookEntry, Hookified, type HookifiedOptions, type IEventEmitter, type Logger };

package/dist/node/index.js

@@ -4,6 +4,8 @@ var Eventified = class {
   _maxListeners;
   _logger;
   _throwOnEmitError = false;
+  _throwOnEmptyListeners = false;
+  _errorEvent = "error";
   constructor(options) {
     this._eventListeners = /* @__PURE__ */ new Map();
     this._maxListeners = 100;
@@ -11,6 +13,9 @@ var Eventified = class {
     if (options?.throwOnEmitError !== void 0) {
       this._throwOnEmitError = options.throwOnEmitError;
     }
+    if (options?.throwOnEmptyListeners !== void 0) {
+      this._throwOnEmptyListeners = options.throwOnEmptyListeners;
+    }
   }
   /**
    * Gets the logger
@@ -40,6 +45,20 @@ var Eventified = class {
   set throwOnEmitError(value) {
     this._throwOnEmitError = value;
   }
+  /**
+   * Gets whether an error should be thrown when emitting 'error' event with no listeners. Default is false.
+   * @returns {boolean}
+   */
+  get throwOnEmptyListeners() {
+    return this._throwOnEmptyListeners;
+  }
+  /**
+   * Sets whether an error should be thrown when emitting 'error' event with no listeners. Default is false.
+   * @param {boolean} value
+   */
+  set throwOnEmptyListeners(value) {
+    this._throwOnEmptyListeners = value;
+  }
   /**
    * Adds a handler function for a specific event that will run only once
    * @param {string | symbol} eventName
@@ -190,12 +209,17 @@ var Eventified = class {
         result = true;
       }
     }
-    if (event === "error") {
+    if (event === this._errorEvent) {
       const error = arguments_[0] instanceof Error ? arguments_[0] : new Error(`${arguments_[0]}`);
       if (this._throwOnEmitError && !result) {
         throw error;
+      } else {
+        if (this.listeners(this._errorEvent).length === 0 && this._throwOnEmptyListeners === true) {
+          throw error;
+        }
       }
     }
+    this.sendLog(event, arguments_);
     return result;
   }
   /**
@@ -243,21 +267,75 @@ var Eventified = class {
     }
     return result;
   }
+  /**
+   * Sends a log message using the configured logger based on the event name
+   * @param {string | symbol} eventName - The event name that determines the log level
+   * @param {unknown} data - The data to log
+   */
+  sendLog(eventName, data) {
+    if (!this._logger) {
+      return;
+    }
+    let message;
+    if (typeof data === "string") {
+      message = data;
+    } else if (Array.isArray(data) && data.length > 0 && data[0] instanceof Error) {
+      message = data[0].message;
+    } else if (data instanceof Error) {
+      message = data.message;
+    } else if (Array.isArray(data) && data.length > 0 && typeof data[0]?.message === "string") {
+      message = data[0].message;
+    } else {
+      message = JSON.stringify(data);
+    }
+    switch (eventName) {
+      case "error": {
+        this._logger.error?.(message, { event: eventName, data });
+        break;
+      }
+      case "warn": {
+        this._logger.warn?.(message, { event: eventName, data });
+        break;
+      }
+      case "trace": {
+        this._logger.trace?.(message, { event: eventName, data });
+        break;
+      }
+      case "debug": {
+        this._logger.debug?.(message, { event: eventName, data });
+        break;
+      }
+      case "fatal": {
+        this._logger.fatal?.(message, { event: eventName, data });
+        break;
+      }
+      default: {
+        this._logger.info?.(message, { event: eventName, data });
+        break;
+      }
+    }
+  }
 };
 
 // src/index.ts
 var Hookified = class extends Eventified {
   _hooks;
-  _throwHookErrors = false;
+  _throwOnHookError = false;
   _enforceBeforeAfter = false;
   _deprecatedHooks;
   _allowDeprecated = true;
   constructor(options) {
-    super({ logger: options?.logger });
+    super({
+      logger: options?.logger,
+      throwOnEmitError: options?.throwOnEmitError,
+      throwOnEmptyListeners: options?.throwOnEmptyListeners
+    });
     this._hooks = /* @__PURE__ */ new Map();
     this._deprecatedHooks = options?.deprecatedHooks ? new Map(options.deprecatedHooks) : /* @__PURE__ */ new Map();
-    if (options?.throwHookErrors !== void 0) {
-      this._throwHookErrors = options.throwHookErrors;
+    if (options?.throwOnHookError !== void 0) {
+      this._throwOnHookError = options.throwOnHookError;
+    } else if (options?.throwHookErrors !== void 0) {
+      this._throwOnHookError = options.throwHookErrors;
     }
     if (options?.enforceBeforeAfter !== void 0) {
       this._enforceBeforeAfter = options.enforceBeforeAfter;
@@ -276,16 +354,32 @@ var Hookified = class extends Eventified {
   /**
    * Gets whether an error should be thrown when a hook throws an error. Default is false and only emits an error event.
    * @returns {boolean}
+   * @deprecated - this will be deprecated in version 2. Please use throwOnHookError.
    */
   get throwHookErrors() {
-    return this._throwHookErrors;
+    return this._throwOnHookError;
   }
   /**
    * Sets whether an error should be thrown when a hook throws an error. Default is false and only emits an error event.
    * @param {boolean} value
+   * @deprecated - this will be deprecated in version 2. Please use throwOnHookError.
    */
   set throwHookErrors(value) {
-    this._throwHookErrors = value;
+    this._throwOnHookError = value;
+  }
+  /**
+   * Gets whether an error should be thrown when a hook throws an error. Default is false and only emits an error event.
+   * @returns {boolean}
+   */
+  get throwOnHookError() {
+    return this._throwOnHookError;
+  }
+  /**
+   * Sets whether an error should be thrown when a hook throws an error. Default is false and only emits an error event.
+   * @param {boolean} value
+   */
+  set throwOnHookError(value) {
+    this._throwOnHookError = value;
   }
   /**
    * Gets whether to enforce that all hook names start with 'before' or 'after'. Default is false.
@@ -355,9 +449,6 @@ var Hookified = class extends Eventified {
       const message = this._deprecatedHooks.get(event);
       const warningMessage = `Hook "${event}" is deprecated${message ? `: ${message}` : ""}`;
       this.emit("warn", { hook: event, message: warningMessage });
-      if (this.logger?.warn) {
-        this.logger.warn(warningMessage);
-      }
       return this._allowDeprecated;
     }
     return true;
@@ -505,10 +596,7 @@ var Hookified = class extends Eventified {
         } catch (error) {
           const message = `${event}: ${error.message}`;
           this.emit("error", new Error(message));
-          if (this.logger) {
-            this.logger.error(message);
-          }
-          if (this._throwHookErrors) {
+          if (this._throwOnHookError) {
             throw new Error(message);
           }
         }
@@ -565,3 +653,4 @@ export {
   Eventified,
   Hookified
 };
+/* v8 ignore next -- @preserve */

package/package.json

@@ -1,21 +1,28 @@
 {
   "name": "hookified",
-  "version": "1.12.2",
+  "version": "1.14.0",
   "description": "Event Emitting and Middleware Hooks",
   "type": "module",
-  "main": "dist/node/index.cjs",
-  "module": "dist/node/index.js",
+  "main": "./dist/node/index.js",
+  "module": "./dist/node/index.js",
+  "types": "./dist/node/index.d.ts",
   "exports": {
     ".": {
-      "import": "./dist/node/index.js",
-      "require": "./dist/node/index.cjs"
+      "import": {
+        "types": "./dist/node/index.d.ts",
+        "default": "./dist/node/index.js"
+      },
+      "require": {
+        "types": "./dist/node/index.d.cts",
+        "default": "./dist/node/index.cjs"
+      },
+      "default": "./dist/node/index.js"
     },
     "./browser": {
       "import": "./dist/browser/index.js",
       "default": "./dist/browser/index.global.js"
     }
   },
-  "types": "dist/node/index.d.ts",
   "scripts": {
     "lint": "biome check --write --error-on-warnings",
     "test": "pnpm lint && vitest run --coverage",
@@ -61,29 +68,24 @@
   },
   "homepage": "https://github.com/jaredwray/hookified#readme",
   "devDependencies": {
-    "@biomejs/biome": "^2.2.6",
-    "@monstermann/tinybench-pretty-printer": "^0.2.0",
-    "@types/node": "^24.7.2",
-    "@vitest/coverage-v8": "^3.2.4",
-    "docula": "^0.30.0",
+    "@biomejs/biome": "^2.3.8",
+    "@monstermann/tinybench-pretty-printer": "^0.3.0",
+    "@types/node": "^24.10.1",
+    "@vitest/coverage-v8": "^4.0.15",
+    "docula": "^0.31.1",
     "emittery": "^1.2.0",
     "eventemitter3": "^5.0.1",
     "hookable": "^5.5.3",
-    "pino": "^10.0.0",
-    "rimraf": "^6.0.1",
-    "tinybench": "^5.0.1",
-    "tsup": "^8.5.0",
-    "tsx": "^4.20.6",
+    "pino": "^10.1.0",
+    "rimraf": "^6.1.2",
+    "tinybench": "^6.0.0",
+    "tsup": "^8.5.1",
+    "tsx": "^4.21.0",
     "typescript": "^5.9.3",
-    "vitest": "^3.2.4"
+    "vitest": "^4.0.15"
   },
   "files": [
     "dist",
     "LICENSE"
-  ],
-  "pnpm": {
-    "onlyBuiltDependencies": [
-      "unrs-resolver"
-    ]
-  }
+  ]
 }