@nsshunt/stsrunnerframework 2.0.11 → 2.0.12

package/README.md

@@ -1097,8 +1097,6 @@ stateDiagram
 import { IRunnerInstance, ITestRunnerTelemetryPayload, eIWMessageCommands, WorkerInstance } from './../index'
 import { TestCase01 } from './testCase01'
 
-import isNode from 'detect-node'
-
 import { parentPort } from 'worker_threads';
 
 export class WorkerTestCases extends WorkerInstance {
@@ -1186,7 +1184,6 @@ import { TestCase01 } from './testCase01'
 
 import { MessagePort } from 'worker_threads';
 
-import isNode from 'detect-node'
 import { JSONObject } from '@nsshunt/stsutils';
 
 export class WorkerTestCases extends WorkerInstance {

package/dist/index.cjs

@@ -22,8 +22,6 @@ var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__ge
 	enumerable: true
 }) : target, mod));
 //#endregion
-let detect_node = require("detect-node");
-detect_node = __toESM(detect_node);
 let _nsshunt_stsutils = require("@nsshunt/stsutils");
 let _nsshunt_stsobservability = require("@nsshunt/stsobservability");
 //#region src/commonTypes.ts
@@ -574,7 +572,7 @@ var AbstractRunnerExecutionWorker = class {
 				this.#debug(chalk.grey(`WorkerInstance:#processLoopExecutor(): Removing runners from collection: [${removeList}]`));
 				for (let i = 0; i < removeList.length; i++) {
 					this.#runners[removeList[i]].archived = true;
-					if (this.#archiveDeleteTimeout !== 0) if (detect_node.default) setTimeout(() => delete this.#runners[removeList[i]], this.#archiveDeleteTimeout * 1e3).unref();
+					if (this.#archiveDeleteTimeout !== 0) if (_nsshunt_stsutils.isNode) setTimeout(() => delete this.#runners[removeList[i]], this.#archiveDeleteTimeout * 1e3).unref();
 					else setTimeout(() => delete this.#runners[removeList[i]], this.#archiveDeleteTimeout * 1e3);
 				}
 				const message = `WorkerInstance:#processLoopExecutor(): Remaining Runner Count: [${Object.keys(this.#runners).length}]`;
@@ -647,7 +645,7 @@ var AbstractRunnerExecutionWorker = class {
 	#SetMessagePort = (workerMessagePort) => {
 		try {
 			this.#collectorCollectorPort = workerMessagePort.port;
-			if (detect_node.default) this.#collectorCollectorPort.on("message", (data) => {
+			if (_nsshunt_stsutils.isNode) this.#collectorCollectorPort.on("message", (data) => {
 				this.#silly(`collectorCollectorPort on('message'): ${JSON.stringify(data)}`);
 			});
 			else this.#collectorCollectorPort.addEventListener("message", (data) => {
@@ -1484,6 +1482,32 @@ var AbstractRunnerExecutionWorker = class {
 };
 //#endregion
 //#region src/messageBroker.ts
+/**
+* STSMessageBroker
+* ----------------
+* Central message broker for worker <-> manager communications.
+*
+* Responsibilities:
+* - Create and manage callback-correlated request/response message flows
+* - Route inbound port messages as either:
+*   - solicited responses (request/response messages with `messageId`)
+*   - unsolicited push/event messages (telemetry, state changes, etc.)
+* - Initialise message ports between the manager and the worker
+* - Provide helper methods for sending worker-level and runner-level commands
+* - Manage response timeouts for outstanding requests
+*
+* High-level design:
+* - Every solicited request gets a generated `messageId`
+* - The broker stores a pending promise resolver keyed by `messageId`
+* - When a matching inbound response arrives, the broker resolves that promise
+* - If no response arrives before the configured timeout, the request is rejected
+*
+* Notes:
+* - This broker supports both Node and browser worker environments
+* - Environment-specific message port listener wiring is handled internally
+* - The broker itself does not interpret business meaning of unsolicited messages;
+*   those are delegated to a caller-supplied callback
+*/
 var import_lodash_merge = /* @__PURE__ */ __toESM((/* @__PURE__ */ __commonJSMin(((exports, module) => {
 	/**
 	* Lodash (Custom Build) <https://lodash.com/>
@@ -3126,7 +3150,7 @@ var STSMessageBroker = class {
 	* @param cb Callback used to handle unsolicited inbound messages.
 	*/
 	SetupMessagePortListener = (stsWorkerEx, cb) => {
-		if (detect_node.default) stsWorkerEx.messagePort.on("message", (data) => {
+		if (_nsshunt_stsutils.isNode) stsWorkerEx.messagePort.on("message", (data) => {
 			this._processInboundMessage(data, cb);
 		});
 		else stsWorkerEx.messagePort.addEventListener("message", (data) => {
@@ -3357,7 +3381,7 @@ var STSMessageBroker = class {
 	*/
 	GetPorts = () => {
 		const { port1, port2 } = new MessageChannel();
-		if (!detect_node.default) port1.start();
+		if (!_nsshunt_stsutils.isNode) port1.start();
 		return {
 			port1,
 			port2
@@ -3981,7 +4005,7 @@ var ArchiveManager = class {
 										}
 										delete this.archiveDelTimeout[delTimeoutId];
 									}, this.archiveDeleteTimeout * 1e3);
-									if (detect_node.default) delTimeout.unref();
+									if (_nsshunt_stsutils.isNode) delTimeout.unref();
 									this.archiveDelTimeout[delTimeoutId] = {
 										workerId: workerEx.id,
 										runnerId: removeList[i].runnerId,
@@ -4002,15 +4026,15 @@ var ArchiveManager = class {
 					* Schedule the next archive scan.
 					*/
 					this.timeout = setTimeout(() => this.ProcessLoopExecutor(), 1e3);
-					if (detect_node.default) this.timeout.unref();
+					if (_nsshunt_stsutils.isNode) this.timeout.unref();
 				}, 100);
-				if (detect_node.default) startTimeout.unref();
+				if (_nsshunt_stsutils.isNode) startTimeout.unref();
 			} else {
 				/**
 				* No archive candidates found; just schedule the next scan.
 				*/
 				this.timeout = setTimeout(() => this.ProcessLoopExecutor(), 1e3);
-				if (detect_node.default) this.timeout.unref();
+				if (_nsshunt_stsutils.isNode) this.timeout.unref();
 			}
 		} catch (error) {
 			this.options.logger.error(`_processLoopExecutor(): Error: [${error}]`);
@@ -4065,7 +4089,7 @@ var ArchiveManager = class {
 	*/
 	Terminate = () => {
 		this.options.logger.debug(`ArchiveManager:Terminate()`);
-		this.options.logger.debug(`Runtime context: isNode = [${detect_node.default}]`);
+		this.options.logger.debug(`Runtime context: isNode = [${_nsshunt_stsutils.isNode}]`);
 		if (this.timeout) {
 			this.options.logger.debug(`ArchiveManager:Terminate() - this.timeout cleared ...`);
 			clearTimeout(this.timeout);
@@ -4641,7 +4665,7 @@ var WorkerInstanceManager = class {
 	* @param stsWorkerEx Manager-side worker instance whose underlying actual worker should be monitored.
 	*/
 	SetupWorkerSystemEvents = (stsWorkerEx) => {
-		if (detect_node.default) {
+		if (_nsshunt_stsutils.isNode) {
 			/**
 			* Node.js worker thread exit event.
 			*