@travetto/worker 5.0.15 → 5.0.16

package/LICENSE

@@ -1,6 +1,6 @@
 MIT License
 
-Copyright (c) 2023 ArcSine Technologies
+Copyright (c) 2020 ArcSine Technologies
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@travetto/worker",
-  "version": "5.0.15",
+  "version": "5.0.16",
   "description": "Process management utilities, with a focus on inter-process communication",
   "keywords": [
     "exec",
@@ -25,7 +25,7 @@
     "directory": "module/worker"
   },
   "dependencies": {
-    "@travetto/runtime": "^5.0.13",
+    "@travetto/runtime": "^5.0.14",
     "generic-pool": "^3.9.0"
   },
   "travetto": {

package/src/ipc.ts

@@ -87,8 +87,6 @@ export class IpcChannel<V = unknown> {
           const complete = new Promise<void>(r => this.proc.on('close', r));
           this.proc.kill();
           await Promise.race([complete, Util.nonBlockingTimeout(1000)]);
-        } else {
-          this.proc.disconnect();
         }
       } catch { }
     }

package/src/pool.ts

@@ -4,27 +4,29 @@ import { Options, Pool, createPool } from 'generic-pool';
 import { Env, Util, AsyncQueue } from '@travetto/runtime';
 
 type ItrSource<I> = Iterable<I> | AsyncIterable<I>;
+type WorkerExecutor<I, O> = (input: I, idx: number) => Promise<O>;
 
 /**
  * Worker definition
  */
 export interface Worker<I, O = unknown> {
-  active?: boolean;
-  id?: unknown;
+  active: boolean;
+  id: unknown;
   init?(): Promise<unknown>;
-  execute(input: I, idx: number): Promise<O>;
+  execute: WorkerExecutor<I, O>;
   destroy?(): Promise<void>;
   release?(): unknown;
 }
 
-type WorkerInput<I, O> = (() => Worker<I, O>) | ((input: I, inputIdx: number) => Promise<O>);
+type WorkerFactoryInput<I, O = unknown> = Partial<Worker<I, O>> & { execute: WorkerExecutor<I, O> };
+type WorkerInput<I, O> = (() => WorkerFactoryInput<I, O>) | WorkerExecutor<I, O>;
 type WorkPoolConfig<I, O> = Options & {
   onComplete?: (output: O, input: I, finishIdx: number) => void;
   onError?(ev: Error, input: I, finishIdx: number): (unknown | Promise<unknown>);
   shutdown?: AbortSignal;
 };
 
-const isWorkerFactory = <I, O>(o: WorkerInput<I, O>): o is (() => Worker<I, O>) => o.length === 0;
+const isWorkerFactory = <I, O>(o: WorkerInput<I, O>): o is (() => WorkerFactoryInput<I, O>) => o.length === 0;
 
 /**
  * Work pool support
@@ -45,13 +47,12 @@ export class WorkPool {
       async create() {
         try {
           pendingAcquires += 1;
-          const res = isWorkerFactory(worker) ? await worker() : { execute: worker };
-          res.id ??= Util.uuid();
-
-          if (res.init) {
-            await res.init();
-          }
-
+          const res: Worker<I, O> = {
+            id: Util.uuid(),
+            active: true,
+            ...isWorkerFactory(worker) ? await worker() : { execute: worker }
+          };
+          await res.init?.();
           return res;
         } finally {
           pendingAcquires -= 1;
@@ -63,7 +64,7 @@ export class WorkPool {
         }
         return x.destroy?.();
       },
-      validate: async (x: Worker<I, O>) => x.active ?? true
+      validate: async (x: Worker<I, O>) => x.active
     }, {
       evictionRunIntervalMillis: 5000,
       ...(opts ?? {}),
@@ -115,7 +116,7 @@ export class WorkPool {
             console.debug('Releasing', { pid: process.pid, worker: worker.id });
           }
           try {
-            if (worker.active ?? true) {
+            if (worker.active) {
               try {
                 await worker.release?.();
               } catch { }