@effect/platform-bun 0.16.1 → 0.16.3

package/Worker/dist/effect-platform-bun-Worker.cjs.dev.js

@@ -45,12 +45,15 @@ const platformWorkerImpl = /*#__PURE__*/Worker__namespace.PlatformWorker.of({
           signal
         });
         port.postMessage([1]);
-      }), Effect__namespace.timeout(5000), Effect__namespace.tap(Option__namespace.match({
+      }),
+      // TODO: make configurable
+      // sometimes bun doesn't fire the close event
+      Effect__namespace.timeout(1000), Effect__namespace.tap(Option__namespace.match({
         onNone: () => Effect__namespace.sync(() => port.terminate()),
         onSome: _ => Effect__namespace.unit
       })))));
       const queue = yield* _(Queue__namespace.unbounded());
-      const run = Effect__namespace.async((resume, signal) => {
+      const fiber = yield* _(Effect__namespace.async((resume, signal) => {
         port.addEventListener("message", function (event) {
           queue.unsafeOffer(event.data);
         }, {
@@ -61,10 +64,10 @@ const platformWorkerImpl = /*#__PURE__*/Worker__namespace.PlatformWorker.of({
         }, {
           signal
         });
-      });
+      }), Effect__namespace.interruptible, Effect__namespace.forkScoped);
       const send = (message, transfers) => Effect__namespace.sync(() => port.postMessage([0, message], transfers));
       return {
-        run,
+        fiber,
         queue,
         send
       };

package/Worker/dist/effect-platform-bun-Worker.cjs.prod.js

@@ -45,12 +45,15 @@ const platformWorkerImpl = /*#__PURE__*/Worker__namespace.PlatformWorker.of({
           signal
         });
         port.postMessage([1]);
-      }), Effect__namespace.timeout(5000), Effect__namespace.tap(Option__namespace.match({
+      }),
+      // TODO: make configurable
+      // sometimes bun doesn't fire the close event
+      Effect__namespace.timeout(1000), Effect__namespace.tap(Option__namespace.match({
         onNone: () => Effect__namespace.sync(() => port.terminate()),
         onSome: _ => Effect__namespace.unit
       })))));
       const queue = yield* _(Queue__namespace.unbounded());
-      const run = Effect__namespace.async((resume, signal) => {
+      const fiber = yield* _(Effect__namespace.async((resume, signal) => {
         port.addEventListener("message", function (event) {
           queue.unsafeOffer(event.data);
         }, {
@@ -61,10 +64,10 @@ const platformWorkerImpl = /*#__PURE__*/Worker__namespace.PlatformWorker.of({
         }, {
           signal
         });
-      });
+      }), Effect__namespace.interruptible, Effect__namespace.forkScoped);
       const send = (message, transfers) => Effect__namespace.sync(() => port.postMessage([0, message], transfers));
       return {
-        run,
+        fiber,
         queue,
         send
       };

package/Worker/dist/effect-platform-bun-Worker.esm.js

@@ -18,12 +18,15 @@ const platformWorkerImpl = /*#__PURE__*/Worker.PlatformWorker.of({
           signal
         });
         port.postMessage([1]);
-      }), Effect.timeout(5000), Effect.tap(Option.match({
+      }),
+      // TODO: make configurable
+      // sometimes bun doesn't fire the close event
+      Effect.timeout(1000), Effect.tap(Option.match({
         onNone: () => Effect.sync(() => port.terminate()),
         onSome: _ => Effect.unit
       })))));
       const queue = yield* _(Queue.unbounded());
-      const run = Effect.async((resume, signal) => {
+      const fiber = yield* _(Effect.async((resume, signal) => {
         port.addEventListener("message", function (event) {
           queue.unsafeOffer(event.data);
         }, {
@@ -34,10 +37,10 @@ const platformWorkerImpl = /*#__PURE__*/Worker.PlatformWorker.of({
         }, {
           signal
         });
-      });
+      }), Effect.interruptible, Effect.forkScoped);
       const send = (message, transfers) => Effect.sync(() => port.postMessage([0, message], transfers));
       return {
-        run,
+        fiber,
         queue,
         send
       };

package/WorkerRunner/dist/effect-platform-bun-WorkerRunner.cjs.dev.js

@@ -40,7 +40,6 @@ const platformRunnerImpl = /*#__PURE__*/Runner__namespace.PlatformRunner.of({
       }
       const port = self;
       const queue = yield* _(Queue__namespace.unbounded());
-      const fiberId = yield* _(Effect__namespace.fiberId);
       const fiber = yield* _(Effect__namespace.async((resume, signal) => {
         port.addEventListener("message", function (event) {
           const message = event.data;
@@ -57,8 +56,7 @@ const platformRunnerImpl = /*#__PURE__*/Runner__namespace.PlatformRunner.of({
         }, {
           signal
         });
-      }), Effect__namespace.forkDaemon);
-      yield* _(Effect__namespace.addFinalizer(() => fiber.interruptAsFork(fiberId)));
+      }), Effect__namespace.forkScoped);
       const send = (message, transfer) => Effect__namespace.sync(() => port.postMessage([1, message], {
         transfer: transfer
       }));

package/WorkerRunner/dist/effect-platform-bun-WorkerRunner.cjs.prod.js

@@ -40,7 +40,6 @@ const platformRunnerImpl = /*#__PURE__*/Runner__namespace.PlatformRunner.of({
       }
       const port = self;
       const queue = yield* _(Queue__namespace.unbounded());
-      const fiberId = yield* _(Effect__namespace.fiberId);
       const fiber = yield* _(Effect__namespace.async((resume, signal) => {
         port.addEventListener("message", function (event) {
           const message = event.data;
@@ -57,8 +56,7 @@ const platformRunnerImpl = /*#__PURE__*/Runner__namespace.PlatformRunner.of({
         }, {
           signal
         });
-      }), Effect__namespace.forkDaemon);
-      yield* _(Effect__namespace.addFinalizer(() => fiber.interruptAsFork(fiberId)));
+      }), Effect__namespace.forkScoped);
       const send = (message, transfer) => Effect__namespace.sync(() => port.postMessage([1, message], {
         transfer: transfer
       }));

package/WorkerRunner/dist/effect-platform-bun-WorkerRunner.esm.js

@@ -14,7 +14,6 @@ const platformRunnerImpl = /*#__PURE__*/Runner.PlatformRunner.of({
       }
       const port = self;
       const queue = yield* _(Queue.unbounded());
-      const fiberId = yield* _(Effect.fiberId);
       const fiber = yield* _(Effect.async((resume, signal) => {
         port.addEventListener("message", function (event) {
           const message = event.data;
@@ -31,8 +30,7 @@ const platformRunnerImpl = /*#__PURE__*/Runner.PlatformRunner.of({
         }, {
           signal
         });
-      }), Effect.forkDaemon);
-      yield* _(Effect.addFinalizer(() => fiber.interruptAsFork(fiberId)));
+      }), Effect.forkScoped);
       const send = (message, transfer) => Effect.sync(() => port.postMessage([1, message], {
         transfer: transfer
       }));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@effect/platform-bun",
-  "version": "0.16.1",
+  "version": "0.16.3",
   "description": "Unified interfaces for common platform-specific services",
   "main": "dist/effect-platform-bun.cjs.js",
   "module": "dist/effect-platform-bun.esm.js",
@@ -30,8 +30,8 @@
   "license": "MIT",
   "sideEffects": false,
   "dependencies": {
-    "@effect/platform": "^0.27.1",
-    "@effect/platform-node": "^0.28.1"
+    "@effect/platform": "^0.27.3",
+    "@effect/platform-node": "^0.28.3"
   },
   "devDependencies": {
     "@effect/schema": "^0.47.1",
@@ -179,7 +179,7 @@
     }
   },
   "scripts": {
-    "example": "ts-node --project tsconfig.examples.json",
+    "example": "node -r ts-node/register",
     "lint": "eslint src/**/*.ts test/**/*.ts",
     "tc": "tsc -b",
     "test": "vitest",

package/src/internal/worker.ts

@@ -18,7 +18,9 @@ const platformWorkerImpl = Worker.PlatformWorker.of({
             port.addEventListener("close", () => resume(Effect.unit), { once: true, signal })
             port.postMessage([1])
           }),
-          Effect.timeout(5000),
+          // TODO: make configurable
+          // sometimes bun doesn't fire the close event
+          Effect.timeout(1000),
           Effect.tap(Option.match({
             onNone: () => Effect.sync(() => port.terminate()),
             onSome: (_) => Effect.unit
@@ -28,19 +30,23 @@ const platformWorkerImpl = Worker.PlatformWorker.of({
 
       const queue = yield* _(Queue.unbounded<Worker.BackingWorker.Message<O>>())
 
-      const run = Effect.async<never, WorkerError, never>((resume, signal) => {
-        port.addEventListener("message", function(event) {
-          queue.unsafeOffer(event.data)
-        }, { signal })
-        port.addEventListener("error", function(event) {
-          resume(Effect.fail(WorkerError("unknown", event.message)))
-        }, { signal })
-      })
+      const fiber = yield* _(
+        Effect.async<never, WorkerError, never>((resume, signal) => {
+          port.addEventListener("message", function(event) {
+            queue.unsafeOffer(event.data)
+          }, { signal })
+          port.addEventListener("error", function(event) {
+            resume(Effect.fail(WorkerError("unknown", event.message)))
+          }, { signal })
+        }),
+        Effect.interruptible,
+        Effect.forkScoped
+      )
 
       const send = (message: I, transfers?: ReadonlyArray<unknown>) =>
         Effect.sync(() => port.postMessage([0, message], transfers as any))
 
-      return { run, queue, send }
+      return { fiber, queue, send }
     })
   }
 })

package/src/internal/workerRunner.ts

@@ -17,7 +17,6 @@ const platformRunnerImpl = Runner.PlatformRunner.of({
       }
       const port = self
       const queue = yield* _(Queue.unbounded<I>())
-      const fiberId = yield* _(Effect.fiberId)
       const fiber = yield* _(
         Effect.async<never, WorkerError, never>((resume, signal) => {
           port.addEventListener("message", function(event) {
@@ -32,9 +31,8 @@ const platformRunnerImpl = Runner.PlatformRunner.of({
             resume(Effect.fail(WorkerError("unknown", error.message)))
           }, { signal })
         }),
-        Effect.forkDaemon
+        Effect.forkScoped
       )
-      yield* _(Effect.addFinalizer(() => fiber.interruptAsFork(fiberId)))
       const send = (message: O, transfer?: ReadonlyArray<unknown>) =>
         Effect.sync(() =>
           port.postMessage([1, message], {