@zudojs/events 1.0.0 → 1.0.1

package/dist/eventBus/eventBus.core.js

@@ -5,6 +5,7 @@ import { isEvent } from "../eventTypes/eventDefinition.type.js";
 import { EventEmitter } from "../eventEmitter/eventEmitter.core.js";
 import { EventErrorMode } from "../eventEmitter/eventEmitter.type.js";
 import { EventRegistry } from "../eventRegistry/eventRegistry.store.js";
+import { normalizeRegistryEventType } from "../eventRegistry/eventRegistry.registration.js";
 import { EventBusDisposedError, EventBusStoppedError, EventError, toEventError, } from "../eventErrors/eventError.base.js";
 import { EventBusState } from "./eventBus.type.js";
 export { EventBusState } from "./eventBus.type.js";
@@ -135,9 +136,13 @@ export class EventBus {
         this.ensureNotDisposed();
         const removed = this.registry.unregister(eventType);
         if (options.removeHandlers) {
-            const definition = this.registry.getHandlersForType(eventType);
-            for (const handler of definition) {
-                if (handler.eventType !== "*" && !handler.eventType.endsWith(".*")) {
+            /**
+             * Compare patterns on the full handler list: the matching
+             * query skips disabled handlers, which must be dropped too.
+             */
+            const type = normalizeRegistryEventType(eventType);
+            for (const handler of this.registry.getHandlers()) {
+                if (handler.eventType === type) {
                     this.registry.unregisterHandler(handler.id);
                 }
             }

package/dist/eventBus/eventBus.publish.js

@@ -42,25 +42,31 @@ export async function busPublish(event, options, deps) {
         signal: options.signal,
         metadata: options.metadata,
     });
+    /**
+     * The emit result is captured here, at the terminal, rather
+     * than read back from the pipeline's return value: a middleware
+     * that awaits next() and returns nothing (or something else)
+     * has still dispatched the handlers, and their outcome must not
+     * be reported as a short-circuit.
+     */
+    let emitResult;
     const terminal = async () => {
-        return deps.emitter.emit(event, {
+        const result = await deps.emitter.emit(event, {
             mode: options.mode,
             errorMode: options.errorMode,
             signal: options.signal,
             metadata: options.metadata,
         });
+        emitResult = result;
+        return result;
     };
-    let emitResult;
     let middlewareExecutions;
     if (allMiddleware.length > 0) {
         const pipelineResult = await executeEventMiddlewarePipeline(allMiddleware, middlewareContext, terminal);
         middlewareExecutions = pipelineResult.executions;
-        if (isEventEmitResult(pipelineResult.result)) {
-            emitResult = pipelineResult.result;
-        }
     }
     else {
-        emitResult = await terminal();
+        await terminal();
     }
     deps.notify({
         type: "published",

package/dist/eventEmitter/eventEmitter.core.js

@@ -138,7 +138,15 @@ export class EventEmitter {
         if (this.disposed) {
             return;
         }
-        this.removeAllListeners();
+        /**
+         * A shared store (the bus registry) may already have been
+         * disposed, in which case its handlers are gone and querying
+         * it would throw; disposing the emitter must still succeed.
+         */
+        const store = this.store;
+        if (typeof store.isDisposed !== "function" || !store.isDisposed()) {
+            this.removeAllListeners();
+        }
         this.disposed = true;
     }
     isDisposed() {

package/dist/eventTypes/eventPayload.type.js

@@ -151,13 +151,12 @@ function freezeRecursively(value, visited) {
  * Removes undefined properties from an object payload.
  */
 export function stripUndefinedValues(payload) {
-    const result = {};
-    for (const [key, value] of Object.entries(payload)) {
-        if (value !== undefined) {
-            result[key] = value;
-        }
-    }
-    return result;
+    /**
+     * Object.fromEntries defines every entry as an own data
+     * property, so a key such as "__proto__" (e.g. from JSON.parse)
+     * is copied as data instead of replacing the result's prototype.
+     */
+    return Object.fromEntries(Object.entries(payload).filter(([, value]) => value !== undefined));
 }
 /**
  * Merges two object payloads.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zudojs/events",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "description": "Event-driven architecture with event bus, emitter, middleware, and registry for decoupled communication.",
   "type": "module",
   "main": "./dist/index.js",
@@ -20,8 +20,8 @@
   ],
   "sideEffects": false,
   "dependencies": {
-    "@zudojs/errors": "1.0.0",
-    "@zudojs/constants": "1.0.0"
+    "@zudojs/errors": "1.0.1",
+    "@zudojs/constants": "1.0.1"
   },
   "engines": {
     "node": ">=24.0.0"
@@ -31,6 +31,10 @@
     "typescript": "7.0.2"
   },
   "license": "MIT",
+  "author": {
+    "name": "Oluwayemi Oyinlola",
+    "url": "https://github.com/oyinlola-tech"
+  },
   "publishConfig": {
     "access": "public"
   },