@wooksjs/event-core 0.7.22 → 0.7.23

package/dist/index.cjs

@@ -3,7 +3,20 @@ let node_async_hooks = require("node:async_hooks");
 let node_crypto = require("node:crypto");
 
 //#region packages/event-core/src/key.ts
-let nextId = 0;
+const COUNTER_KEY = Symbol.for("wooks.core.keyCounter");
+/**
+* Slot ids are handed out from a counter living on `globalThis`.
+*
+* Two copies of `@wooksjs/event-core` loaded side by side share a single
+* `AsyncLocalStorage` (see `./storage`), and `EventContext.slots` is keyed by
+* the numeric `_id`. A module-scoped counter would restart at `0` in the
+* second copy, so its slots would collide with unrelated slots of the first
+* one and silently read/overwrite each other's values. A global counter keeps
+* ids unique across every loaded copy.
+*/
+const _g = globalThis;
+if (!_g[COUNTER_KEY]) _g[COUNTER_KEY] = { next: 0 };
+const counter = _g[COUNTER_KEY];
 /**
 * Creates a typed, writable context slot. Use `ctx.set(k, value)` to store
 * and `ctx.get(k)` to retrieve. Throws if read before being set.
@@ -19,7 +32,7 @@ let nextId = 0;
 */
 function key(name) {
 	return {
-		_id: nextId++,
+		_id: counter.next++,
 		_name: name
 	};
 }
@@ -38,9 +51,10 @@ function key(name) {
 * ```
 */
 function cached(fn) {
+	const id = counter.next++;
 	return {
-		_id: nextId++,
-		_name: `cached:${nextId}`,
+		_id: id,
+		_name: `cached:${id}`,
 		_fn: fn
 	};
 }
@@ -278,15 +292,40 @@ function resetContextInjector() {
 //#region packages/event-core/src/storage.ts
 const STORAGE_KEY = Symbol.for("wooks.core.asyncStorage");
 const VERSION_KEY = Symbol.for("wooks.core.asyncStorage.version");
-const CURRENT_VERSION = "0.7.21";
-const _g = globalThis;
-if (_g[STORAGE_KEY]) {
-	if (_g[VERSION_KEY] !== CURRENT_VERSION) throw new Error(`[wooks] Incompatible versions of @wooksjs/event-core detected: existing v${_g[VERSION_KEY]}, loading v${CURRENT_VERSION}. All packages must use the same @wooksjs/event-core version.`);
-} else {
-	_g[STORAGE_KEY] = new node_async_hooks.AsyncLocalStorage();
-	_g[VERSION_KEY] = CURRENT_VERSION;
+const PATH_KEY = Symbol.for("wooks.core.asyncStorage.path");
+const CURRENT_VERSION = "0.7.23";
+function describePath(path) {
+	return typeof path === "string" && path ? path : "unknown path";
+}
+/**
+* Registers this copy of `@wooksjs/event-core` on the global object and
+* returns the shared `AsyncLocalStorage`.
+*
+* A different version throws — the context layout is not compatible. The same
+* version is survivable (both copies then share one storage), but it still
+* means the bundler resolved the runtime twice, so we warn: the packages built
+* on top of event-core are duplicated too, and their slots do not cross over.
+*
+* @internal Exported for tests; invoked once at module scope.
+*/
+function registerCoreCopy(identity, globalObject = globalThis) {
+	const holder = globalObject;
+	const existing = holder[STORAGE_KEY];
+	if (!existing) {
+		const storage = new node_async_hooks.AsyncLocalStorage();
+		holder[STORAGE_KEY] = storage;
+		holder[VERSION_KEY] = identity.version;
+		holder[PATH_KEY] = identity.path;
+		return storage;
+	}
+	if (holder[VERSION_KEY] !== identity.version) throw new Error(`[wooks] Incompatible versions of @wooksjs/event-core detected: existing v${holder[VERSION_KEY]}, loading v${identity.version}. All packages must use the same @wooksjs/event-core version.`);
+	console.warn(`[wooks] A second copy of @wooksjs/event-core v${identity.version} was loaded (first: ${describePath(holder[PATH_KEY])}, now: ${describePath(identity.path)}). The copies share the event-context storage, but packages built on them (@wooksjs/event-http, moost, …) are most likely duplicated too, and a slot seeded by one copy is invisible to composables from the other — typical symptom: "Cannot read properties of undefined (reading 'headers')". Make your bundler resolve the whole wooks/moost runtime once: either bundle it entirely or keep it entirely external, together with every package that depends on it.`);
+	return existing;
 }
-const storage = _g[STORAGE_KEY];
+const storage = registerCoreCopy({
+	version: CURRENT_VERSION,
+	path: typeof __filename === "string" ? __filename : require("url").pathToFileURL(__filename).href
+});
 /**
 * Runs a callback with the given `EventContext` as the active context.
 * All composables and `current()` calls inside `fn` will resolve to `ctx`.

package/dist/index.mjs

@@ -2,7 +2,20 @@ import { AsyncLocalStorage } from "node:async_hooks";
 import { randomUUID } from "node:crypto";
 
 //#region packages/event-core/src/key.ts
-let nextId = 0;
+const COUNTER_KEY = Symbol.for("wooks.core.keyCounter");
+/**
+* Slot ids are handed out from a counter living on `globalThis`.
+*
+* Two copies of `@wooksjs/event-core` loaded side by side share a single
+* `AsyncLocalStorage` (see `./storage`), and `EventContext.slots` is keyed by
+* the numeric `_id`. A module-scoped counter would restart at `0` in the
+* second copy, so its slots would collide with unrelated slots of the first
+* one and silently read/overwrite each other's values. A global counter keeps
+* ids unique across every loaded copy.
+*/
+const _g = globalThis;
+if (!_g[COUNTER_KEY]) _g[COUNTER_KEY] = { next: 0 };
+const counter = _g[COUNTER_KEY];
 /**
 * Creates a typed, writable context slot. Use `ctx.set(k, value)` to store
 * and `ctx.get(k)` to retrieve. Throws if read before being set.
@@ -18,7 +31,7 @@ let nextId = 0;
 */
 function key(name) {
 	return {
-		_id: nextId++,
+		_id: counter.next++,
 		_name: name
 	};
 }
@@ -37,9 +50,10 @@ function key(name) {
 * ```
 */
 function cached(fn) {
+	const id = counter.next++;
 	return {
-		_id: nextId++,
-		_name: `cached:${nextId}`,
+		_id: id,
+		_name: `cached:${id}`,
 		_fn: fn
 	};
 }
@@ -277,15 +291,40 @@ function resetContextInjector() {
 //#region packages/event-core/src/storage.ts
 const STORAGE_KEY = Symbol.for("wooks.core.asyncStorage");
 const VERSION_KEY = Symbol.for("wooks.core.asyncStorage.version");
-const CURRENT_VERSION = "0.7.21";
-const _g = globalThis;
-if (_g[STORAGE_KEY]) {
-	if (_g[VERSION_KEY] !== CURRENT_VERSION) throw new Error(`[wooks] Incompatible versions of @wooksjs/event-core detected: existing v${_g[VERSION_KEY]}, loading v${CURRENT_VERSION}. All packages must use the same @wooksjs/event-core version.`);
-} else {
-	_g[STORAGE_KEY] = new AsyncLocalStorage();
-	_g[VERSION_KEY] = CURRENT_VERSION;
+const PATH_KEY = Symbol.for("wooks.core.asyncStorage.path");
+const CURRENT_VERSION = "0.7.23";
+function describePath(path) {
+	return typeof path === "string" && path ? path : "unknown path";
+}
+/**
+* Registers this copy of `@wooksjs/event-core` on the global object and
+* returns the shared `AsyncLocalStorage`.
+*
+* A different version throws — the context layout is not compatible. The same
+* version is survivable (both copies then share one storage), but it still
+* means the bundler resolved the runtime twice, so we warn: the packages built
+* on top of event-core are duplicated too, and their slots do not cross over.
+*
+* @internal Exported for tests; invoked once at module scope.
+*/
+function registerCoreCopy(identity, globalObject = globalThis) {
+	const holder = globalObject;
+	const existing = holder[STORAGE_KEY];
+	if (!existing) {
+		const storage = new AsyncLocalStorage();
+		holder[STORAGE_KEY] = storage;
+		holder[VERSION_KEY] = identity.version;
+		holder[PATH_KEY] = identity.path;
+		return storage;
+	}
+	if (holder[VERSION_KEY] !== identity.version) throw new Error(`[wooks] Incompatible versions of @wooksjs/event-core detected: existing v${holder[VERSION_KEY]}, loading v${identity.version}. All packages must use the same @wooksjs/event-core version.`);
+	console.warn(`[wooks] A second copy of @wooksjs/event-core v${identity.version} was loaded (first: ${describePath(holder[PATH_KEY])}, now: ${describePath(identity.path)}). The copies share the event-context storage, but packages built on them (@wooksjs/event-http, moost, …) are most likely duplicated too, and a slot seeded by one copy is invisible to composables from the other — typical symptom: "Cannot read properties of undefined (reading 'headers')". Make your bundler resolve the whole wooks/moost runtime once: either bundle it entirely or keep it entirely external, together with every package that depends on it.`);
+	return existing;
 }
-const storage = _g[STORAGE_KEY];
+const storage = registerCoreCopy({
+	version: CURRENT_VERSION,
+	path: typeof __filename === "string" ? __filename : import.meta.url
+});
 /**
 * Runs a callback with the given `EventContext` as the active context.
 * All composables and `current()` calls inside `fn` will resolve to `ctx`.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wooksjs/event-core",
-  "version": "0.7.22",
+  "version": "0.7.23",
   "description": "@wooksjs/event-core",
   "keywords": [
     "composables",