@atolljs/node 0.1.1 → 0.1.2

package/README.md

@@ -4,7 +4,7 @@
 and `connectWorker` run on Node's `Worker` (an `EventEmitter`) instead of the
 DOM surface they expect. No framework required.
 
-**[Atoll — package documentation](https://jwhenry3.github.io/atolljs/consumer/#/quickstart)**
+**[Atoll — package documentation](https://jwhenry3.github.io/atolljs/consumer/quickstart/)**
 
 ## Install
 
@@ -93,8 +93,8 @@ export const counterWorker = defineWorker({
 
 ## Documentation
 
-- [Atoll — package documentation](https://jwhenry3.github.io/atolljs/consumer/#/quickstart)
-- [Node backends — Express, Fastify, Hono, Koa](https://jwhenry3.github.io/atolljs/consumer/#/fw-node)
-- [Worker HTTP servers — clustering & gateway](https://jwhenry3.github.io/atolljs/consumer/#/fw-node/clustering)
-- [Worker pool & tasks](https://jwhenry3.github.io/atolljs/consumer/#/tasks)
-- [NestJS guide](https://jwhenry3.github.io/atolljs/consumer/#/fw-nestjs)
+- [Atoll — package documentation](https://jwhenry3.github.io/atolljs/consumer/quickstart/)
+- [Node backends — Express, Fastify, Hono, Koa](https://jwhenry3.github.io/atolljs/consumer/fw-node/)
+- [Worker HTTP servers — clustering & gateway](https://jwhenry3.github.io/atolljs/consumer/fw-node/clustering/)
+- [Worker pool & tasks](https://jwhenry3.github.io/atolljs/consumer/tasks/)
+- [NestJS guide](https://jwhenry3.github.io/atolljs/consumer/fw-nestjs/)

package/package.json

@@ -1,8 +1,8 @@
 {
   "name": "@atolljs/node",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "description": "Node runtime adapter — node:worker_threads pools, no framework",
-  "homepage": "https://jwhenry3.github.io/atolljs/consumer/#/quickstart",
+  "homepage": "https://jwhenry3.github.io/atolljs/consumer/quickstart/",
   "type": "module",
   "exports": {
     ".": "./src/index.ts",
@@ -18,7 +18,7 @@
     "registry": "https://registry.npmjs.org"
   },
   "peerDependencies": {
-    "@atolljs/core": "0.1.1"
+    "@atolljs/core": "0.1.2"
   },
   "repository": {
     "type": "git",

package/src/redis.ts

@@ -205,6 +205,9 @@ export function persistSharedMemory<S extends SharedSpec>(
             }
           });
         }
+        // stop() may have run before ready resolved — don't start a poll
+        // loop that nothing will ever clear.
+        if (stopped) return;
         timer = setInterval(() => {
           flushing = flushing.then(flush).catch((e) => {
             log.warn('flush failed', e);
@@ -228,9 +231,13 @@ export function persistSharedMemory<S extends SharedSpec>(
       await subscriber?.unsubscribe?.(channel);
       await flushing.catch(() => 0);
       // One last flush — durability on shutdown — bypass the stopped guard.
+      // A Redis outage here must not reject stop(): teardown is done either
+      // way, the flush is best-effort persistence.
       stopped = false;
       try {
         await flush();
+      } catch (e) {
+        log.warn('final flush failed', e);
       } finally {
         stopped = true;
       }

package/src/sharedBuffer.ts

@@ -58,19 +58,22 @@ export function bindSharedBuffer(timeoutMs = 10_000): Promise<SharedArrayBuffer>
   }
   const port = parentPort;
   return new Promise((resolve, reject) => {
-    const timer = setTimeout(
-      () => reject(new Error(`bindSharedBuffer(): no shared buffer arrived within ${timeoutMs}ms`)),
-      timeoutMs,
-    );
-    port.on('message', (msg: { type?: string; buffer?: SharedArrayBuffer }) => {
+    const onMessage = (msg: { type?: string; buffer?: SharedArrayBuffer }) => {
       if (msg?.type !== SHARED_BUFFER) return;
       clearTimeout(timer);
+      port.off('message', onMessage);
       if (!msg.buffer) {
         reject(new Error('bindSharedBuffer(): the producing side sent an empty buffer'));
         return;
       }
       bindSharedMemories(msg.buffer);
       resolve(msg.buffer);
-    });
+    };
+    const timer = setTimeout(() => {
+      // Detach — a late SHARED_BUFFER must not bind after the rejection.
+      port.off('message', onMessage);
+      reject(new Error(`bindSharedBuffer(): no shared buffer arrived within ${timeoutMs}ms`));
+    }, timeoutMs);
+    port.on('message', onMessage);
   });
 }