fix: untangle worker set message from application message

[e-mobility-charging-stations-simulator.git] / src / charging-station / ChargingStationWorker.ts

diff --git a/src/charging-station/ChargingStationWorker.ts b/src/charging-station/ChargingStationWorker.ts
index db7ae2cb4462ff79f50359dc989b50536a73edd6..779c638a49fab5ec30119e184682335afa01b20d 100644 (file)
--- a/src/charging-station/ChargingStationWorker.ts
+++ b/src/charging-station/ChargingStationWorker.ts
@@ -1,70 +1,69 @@
-// Partial Copyright Jerome Benoit. 2021-2023. All Rights Reserved.
+// Partial Copyright Jerome Benoit. 2021-2024. All Rights Reserved.
 
-import { AsyncResource } from 'node:async_hooks';
-import { parentPort } from 'node:worker_threads';
+import { parentPort } from 'node:worker_threads'
 
-import { ThreadWorker } from 'poolifier';
+import { ThreadWorker } from 'poolifier'
 
-import { ChargingStation } from './ChargingStation';
-import type { ChargingStationWorkerData } from '../types';
-import { Configuration } from '../utils';
-import { WorkerConstants, type WorkerMessage, WorkerMessageEvents } from '../worker';
+import { BaseError } from '../exception/index.js'
+import type { ChargingStationInfo, ChargingStationWorkerData } from '../types/index.js'
+import { Configuration } from '../utils/index.js'
+import { type WorkerDataError, type WorkerMessage, WorkerMessageEvents } from '../worker/index.js'
+import { ChargingStation } from './ChargingStation.js'
 
-const moduleName = 'ChargingStationWorker';
-
-/**
- * Creates and starts a charging station instance
- *
- * @param data - workerData
- */
-const startChargingStation = (data?: ChargingStationWorkerData): void => {
-  new ChargingStation(data!.index, data!.templateFile).start();
-};
-
-class ChargingStationWorker extends AsyncResource {
-  constructor() {
-    super(moduleName);
-    // Add message listener to create and start charging station from the main thread
-    parentPort?.on('message', (message: WorkerMessage<ChargingStationWorkerData>) => {
-      switch (message.event) {
-        case WorkerMessageEvents.startWorkerElement:
-          try {
-            this.runInAsyncScope(
-              startChargingStation.bind(this) as (data?: ChargingStationWorkerData) => void,
-              this,
-              message.data,
-            );
-            parentPort?.postMessage({
-              event: WorkerMessageEvents.startedWorkerElement,
-            });
-          } catch (error) {
-            parentPort?.postMessage({
-              event: WorkerMessageEvents.startWorkerElementError,
-              data: {
-                message: (error as Error).message,
-                stack: (error as Error).stack,
-              },
-            });
-          }
-          break;
-        default:
-          throw new Error(
-            `Unknown worker event: '${message.event}' received with data: '${JSON.stringify(
-              message.data,
-              null,
-              2,
-            )}'`,
-          );
-      }
-    });
-  }
-}
-
-export let chargingStationWorker: ChargingStationWorker | ThreadWorker<ChargingStationWorkerData>;
+export let chargingStationWorker: object
 if (Configuration.workerPoolInUse()) {
-  chargingStationWorker = new ThreadWorker<ChargingStationWorkerData>(startChargingStation, {
-    maxInactiveTime: WorkerConstants.POOL_MAX_INACTIVE_TIME,
-  });
+  chargingStationWorker = new ThreadWorker<
+  ChargingStationWorkerData,
+  ChargingStationInfo | undefined
+  >((data?: ChargingStationWorkerData): ChargingStationInfo | undefined => {
+    // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+    const { index, templateFile, options } = data!
+    return new ChargingStation(index, templateFile, options).stationInfo
+  })
 } else {
-  chargingStationWorker = new ChargingStationWorker();
+  // eslint-disable-next-line @typescript-eslint/no-extraneous-class
+  class ChargingStationWorker<Data extends ChargingStationWorkerData> {
+    constructor () {
+      parentPort?.on('message', (message: WorkerMessage<Data>) => {
+        const { uuid, event, data } = message
+        switch (event) {
+          case WorkerMessageEvents.addWorkerElement:
+            try {
+              const chargingStation = new ChargingStation(
+                data.index,
+                data.templateFile,
+                data.options
+              )
+              parentPort?.postMessage({
+                uuid,
+                event: WorkerMessageEvents.addedWorkerElement,
+                // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+                data: chargingStation.stationInfo!
+              } satisfies WorkerMessage<ChargingStationInfo>)
+            } catch (error) {
+              parentPort?.postMessage({
+                uuid,
+                event: WorkerMessageEvents.workerElementError,
+                data: {
+                  event,
+                  name: (error as Error).name,
+                  message: (error as Error).message,
+                  stack: (error as Error).stack
+                }
+              } satisfies WorkerMessage<WorkerDataError>)
+            }
+            break
+          default:
+            throw new BaseError(
+              `Unknown worker event: '${event}' received with data: '${JSON.stringify(
+                data,
+                undefined,
+                2
+              )}'`
+            )
+        }
+      })
+    }
+  }
+  chargingStationWorker = new ChargingStationWorker<ChargingStationWorkerData>()
 }