@b9g/node-webworker 0.1.3 → 0.2.0

package/README.md

@@ -6,7 +6,7 @@ Minimal Web Worker shim for Node.js until native support arrives.
 
 Node.js lacks native Web Worker support, despite being a web standard since 2009. This package provides a minimal, reliable shim using Node.js `worker_threads` until native support is added.
 
-**🔗 Canonical Issue:** https://github.com/nodejs/node/issues/43583
+**Canonical Issue:** https://github.com/nodejs/node/issues/43583
 **Please 👍 and comment** on the issue to show demand for native Web Worker support!
 
 ## Installation
@@ -18,7 +18,7 @@ npm install @b9g/node-webworker
 ## Usage
 
 ```typescript
-import { Worker } from '@b9g/node-webworker';
+import {Worker} from '@b9g/node-webworker';
 
 // Create a worker (same API as Web Workers)
 const worker = new Worker('./worker.js', { type: 'module' });
@@ -32,7 +32,7 @@ worker.addEventListener('message', (event) => {
 worker.postMessage({ hello: 'world' });
 
 // Terminate when done
-await worker.terminate();
+worker.terminate();
 ```
 
 ## Exports
@@ -40,11 +40,8 @@ await worker.terminate();
 ### Classes
 
 - `Worker` - Web Worker implementation using Node.js worker_threads
-
-### Types
-
-- `MessageEvent` - Event type for worker messages
-- `ErrorEvent` - Event type for worker errors
+- `MessageEvent` - Event class for worker messages
+- `ErrorEvent` - Event class for worker errors
 
 ### Default Export
 
@@ -60,9 +57,9 @@ await worker.terminate();
 
 ## Limitations
 
-- **Transferable objects** - Limited support (logs warning)
 - **Node.js only** - Don't use this in browsers (they have native Web Workers)
-- **Basic API** - Only core Worker features, not full spec
+- **Module workers only** - Classic workers with `importScripts()` not supported
+- **Serialization differences** - Uses Node.js structured clone, not web's algorithm
 
 ## Deprecation Notice
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@b9g/node-webworker",
-  "version": "0.1.3",
+  "version": "0.2.0",
   "description": "Minimal Web Worker shim for Node.js until native support arrives",
   "keywords": [
     "worker",
@@ -11,8 +11,7 @@
   ],
   "dependencies": {},
   "devDependencies": {
-    "@b9g/libuild": "^0.1.11",
-    "bun-types": "latest",
+    "@b9g/libuild": "^0.1.18",
     "@types/node": "^18.0.0"
   },
   "engines": {
@@ -34,14 +33,6 @@
     "./index.js": {
       "types": "./src/index.d.ts",
       "import": "./src/index.js"
-    },
-    "./worker-wrapper": {
-      "types": "./src/undefined.d.ts",
-      "import": "./src/worker-wrapper.js"
-    },
-    "./worker-wrapper.js": {
-      "types": "./src/undefined.d.ts",
-      "import": "./src/worker-wrapper.js"
     }
   }
 }

package/src/index.d.ts

@@ -6,20 +6,27 @@
  *
  * @see https://github.com/nodejs/node/issues/43583
  */
-import { Worker as NodeWorker } from "worker_threads";
 /**
- * Event-like object for message events
+ * Message event for worker communication
  */
-export interface MessageEvent {
+export declare class MessageEvent extends Event {
     readonly data: any;
-    readonly type: "message";
+    constructor(data: any);
 }
 /**
- * Error event object
+ * Error event for worker errors
  */
-export interface ErrorEvent {
+export declare class ErrorEvent extends Event {
     readonly error: Error;
-    readonly type: "error";
+    constructor(error: Error);
+}
+/**
+ * Close event for worker termination
+ * Includes exit code for crash detection
+ */
+export declare class CloseEvent extends Event {
+    readonly code: number;
+    constructor(code: number);
 }
 /**
  * Web Worker API implementation using Node.js worker_threads
@@ -31,8 +38,11 @@ export declare class Worker {
     #private;
     onmessage: ((event: MessageEvent) => void) | null;
     onerror: ((event: ErrorEvent) => void) | null;
+    onmessageerror: ((event: MessageEvent) => void) | null;
+    onclose: ((event: CloseEvent) => void) | null;
     constructor(scriptURL: string | URL, _options?: {
         type?: "classic" | "module";
+        env?: Record<string, string>;
     });
     /**
      * Send a message to the worker
@@ -43,18 +53,18 @@ export declare class Worker {
      */
     addEventListener(type: "message", listener: (event: MessageEvent) => void): void;
     addEventListener(type: "error", listener: (event: ErrorEvent) => void): void;
+    addEventListener(type: "messageerror", listener: (event: MessageEvent) => void): void;
+    addEventListener(type: "close", listener: (event: CloseEvent) => void): void;
     /**
      * Remove an event listener
      */
     removeEventListener(type: "message", listener: (event: MessageEvent) => void): void;
     removeEventListener(type: "error", listener: (event: ErrorEvent) => void): void;
+    removeEventListener(type: "messageerror", listener: (event: MessageEvent) => void): void;
+    removeEventListener(type: "close", listener: (event: CloseEvent) => void): void;
     /**
      * Terminate the worker (Web Worker standard - returns void, not a promise)
      */
     terminate(): void;
-    /**
-     * Get the underlying Node.js Worker (for advanced usage)
-     */
-    get nodeWorker_(): NodeWorker;
 }
 export default Worker;

package/src/index.js

@@ -1,28 +1,28 @@
 /// <reference types="./index.d.ts" />
 // src/index.ts
 import { Worker as NodeWorker } from "worker_threads";
-var WORKER_WRAPPER_CODE = `
-import {parentPort} from "worker_threads";
-
-// Provide Web Worker globals
-globalThis.onmessage = null;
-globalThis.postMessage = (data) => parentPort.postMessage(data);
-
-// Set up message forwarding
-parentPort.on("message", (data) => {
-	if (globalThis.onmessage) {
-		globalThis.onmessage({data, type: "message"});
-	}
-});
-
-// Import the actual worker script URL from environment variable
-const WORKER_SCRIPT_URL = process.env.WORKER_SCRIPT_URL;
-if (WORKER_SCRIPT_URL) {
-	await import(WORKER_SCRIPT_URL);
-} else {
-	throw new Error("WORKER_SCRIPT_URL environment variable not set");
-}
-`;
+var MessageEvent = class extends Event {
+  data;
+  constructor(data) {
+    super("message");
+    this.data = data;
+  }
+};
+var ErrorEvent = class extends Event {
+  error;
+  constructor(error) {
+    super("error");
+    this.error = error;
+  }
+};
+var CloseEvent = class extends Event {
+  code;
+  constructor(code) {
+    super("close");
+    this.code = code;
+  }
+};
+var WORKER_WRAPPER_CODE = `import{parentPort as p}from"worker_threads";const l=new Set();globalThis.onmessage=null;globalThis.onmessageerror=null;globalThis.postMessage=(d,t)=>t?.length?p.postMessage(d,t):p.postMessage(d);globalThis.self=globalThis;globalThis.addEventListener=(t,f)=>t==="message"&&l.add(f);globalThis.removeEventListener=(t,f)=>t==="message"&&l.delete(f);p.on("message",d=>{const e={data:d,type:"message"};globalThis.onmessage?.(e);l.forEach(f=>f(e))});const u=process.env.WORKER_SCRIPT_URL;if(u)await import(u);else throw Error("WORKER_SCRIPT_URL not set");`;
 var WORKER_WRAPPER_DATA_URL = new URL(
   `data:text/javascript,${encodeURIComponent(WORKER_WRAPPER_CODE)}`
 );
@@ -30,14 +30,22 @@ var Worker = class {
   #nodeWorker;
   #messageListeners;
   #errorListeners;
+  #messageerrorListeners;
+  #closeListeners;
   // Web Worker standard properties
-  // eslint-disable-next-line no-restricted-syntax
-  onmessage = null;
-  // eslint-disable-next-line no-restricted-syntax
-  onerror = null;
+  onmessage;
+  onerror;
+  onmessageerror;
+  onclose;
   constructor(scriptURL, _options) {
     this.#messageListeners = /* @__PURE__ */ new Set();
     this.#errorListeners = /* @__PURE__ */ new Set();
+    this.#messageerrorListeners = /* @__PURE__ */ new Set();
+    this.#closeListeners = /* @__PURE__ */ new Set();
+    this.onmessage = null;
+    this.onerror = null;
+    this.onmessageerror = null;
+    this.onclose = null;
     const scriptURLString = scriptURL.toString();
     let workerScriptURL = scriptURLString;
     if (!scriptURLString.startsWith("file://") && !scriptURLString.startsWith("data:")) {
@@ -51,17 +59,31 @@ var Worker = class {
     this.#nodeWorker = new NodeWorker(WORKER_WRAPPER_DATA_URL, {
       ...{ type: "module" },
       env: {
+        // eslint-disable-next-line no-restricted-properties -- Workers inherit parent env
         ...process.env,
+        ..._options?.env,
         WORKER_SCRIPT_URL: workerScriptURL
       }
     });
     this.#setupEventForwarding();
   }
+  /**
+   * Report a close event when the worker exits
+   */
+  #reportClose(code) {
+    const event = new CloseEvent(code);
+    if (this.onclose) {
+      this.onclose(event);
+    }
+    this.#closeListeners.forEach((listener) => {
+      listener(event);
+    });
+  }
   /**
    * Report an error through the error event mechanism
    */
   #reportError(error) {
-    const event = { error, type: "error" };
+    const event = new ErrorEvent(error);
     if (this.onerror) {
       this.onerror(event);
     }
@@ -74,7 +96,7 @@ var Worker = class {
    */
   #setupEventForwarding() {
     this.#nodeWorker.on("message", (data) => {
-      const event = { data, type: "message" };
+      const event = new MessageEvent(data);
       if (this.onmessage) {
         try {
           this.onmessage(event);
@@ -93,6 +115,18 @@ var Worker = class {
     this.#nodeWorker.on("error", (error) => {
       this.#reportError(error);
     });
+    this.#nodeWorker.on("messageerror", (data) => {
+      const event = new MessageEvent(data);
+      if (this.onmessageerror) {
+        this.onmessageerror(event);
+      }
+      this.#messageerrorListeners.forEach((listener) => {
+        listener(event);
+      });
+    });
+    this.#nodeWorker.on("exit", (code) => {
+      this.#reportClose(code);
+    });
   }
   /**
    * Send a message to the worker
@@ -109,6 +143,12 @@ var Worker = class {
       this.#messageListeners.add(listener);
     } else if (type === "error") {
       this.#errorListeners.add(listener);
+    } else if (type === "messageerror") {
+      this.#messageerrorListeners.add(
+        listener
+      );
+    } else if (type === "close") {
+      this.#closeListeners.add(listener);
     }
   }
   removeEventListener(type, listener) {
@@ -116,6 +156,12 @@ var Worker = class {
       this.#messageListeners.delete(listener);
     } else if (type === "error") {
       this.#errorListeners.delete(listener);
+    } else if (type === "messageerror") {
+      this.#messageerrorListeners.delete(
+        listener
+      );
+    } else if (type === "close") {
+      this.#closeListeners.delete(listener);
     }
   }
   /**
@@ -126,18 +172,19 @@ var Worker = class {
     });
     this.#messageListeners.clear();
     this.#errorListeners.clear();
+    this.#messageerrorListeners.clear();
+    this.#closeListeners.clear();
     this.onmessage = null;
     this.onerror = null;
-  }
-  /**
-   * Get the underlying Node.js Worker (for advanced usage)
-   */
-  get nodeWorker_() {
-    return this.#nodeWorker;
+    this.onmessageerror = null;
+    this.onclose = null;
   }
 };
 var src_default = Worker;
 export {
+  CloseEvent,
+  ErrorEvent,
+  MessageEvent,
   Worker,
   src_default as default
 };

package/src/worker-wrapper.js

@@ -1,15 +0,0 @@
-// src/worker-wrapper.js
-import { parentPort } from "worker_threads";
-globalThis.onmessage = null;
-globalThis.postMessage = (data) => parentPort.postMessage(data);
-parentPort.on("message", (data) => {
-  if (globalThis.onmessage) {
-    globalThis.onmessage({ data, type: "message" });
-  }
-});
-var WORKER_SCRIPT_URL = process.env.WORKER_SCRIPT_URL;
-if (WORKER_SCRIPT_URL) {
-  await import(WORKER_SCRIPT_URL);
-} else {
-  throw new Error("WORKER_SCRIPT_URL environment variable not set");
-}