fix: only process worker message events when necessary

[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 a0adf80595a46d1ef2b4ba7772d735a6640c7248..ab32a464fabbcf3ca3ee2e20a7c48d685945f327 100644 (file)
--- a/src/charging-station/ChargingStationWorker.ts
+++ b/src/charging-station/ChargingStationWorker.ts
@@ -1,71 +1,72 @@
-// 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 { BaseError } from '../exception';
-import type { ChargingStationWorkerData } from '../types';
-import { Configuration } from '../utils';
-import { type WorkerData, 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 - data sent to worker
- */
-const startChargingStation = (data?: ChargingStationWorkerData): void => {
-  new ChargingStation(data!.index, data!.templateFile).start();
-};
-
-class ChargingStationWorker<Data extends WorkerData> extends AsyncResource {
-  constructor() {
-    super(moduleName);
-    // Add message listener to create and start charging station from the main thread
-    parentPort?.on('message', (message: WorkerMessage<Data>) => {
-      switch (message.event) {
-        case WorkerMessageEvents.startWorkerElement:
-          try {
-            this.runInAsyncScope(
-              startChargingStation.bind(this) as (data?: Data) => 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 BaseError(
-            `Unknown worker event: '${message.event}' received with data: '${JSON.stringify(
-              message.data,
-              undefined,
-              2,
-            )}'`,
-          );
-      }
-    });
-  }
-}
-
-export let chargingStationWorker:
-  | ChargingStationWorker<ChargingStationWorkerData>
-  | ThreadWorker<ChargingStationWorkerData>;
+export let chargingStationWorker: object
 if (Configuration.workerPoolInUse()) {
-  chargingStationWorker = new ThreadWorker<ChargingStationWorkerData>(startChargingStation);
+  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<ChargingStationWorkerData>();
+  // 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
+        // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
+        if (uuid != null) {
+          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 message event: '${event}' received with data: '${JSON.stringify(
+                  data,
+                  undefined,
+                  2
+                )}'`
+              )
+          }
+        }
+      })
+    }
+  }
+  chargingStationWorker = new ChargingStationWorker<ChargingStationWorkerData>()
 }