ultravisor 1.3.1 → 1.3.2

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "ultravisor",
-    "version": "1.3.1",
+    "version": "1.3.2",
     "description": "Cyclic process execution with ai integration.",
     "main": "source/Ultravisor.cjs",
     "bin": {
@@ -57,7 +57,7 @@
         "cron": "^4.4.0",
         "meadow": "^2.0.37",
         "meadow-connection-sqlite": "^1.0.19",
-        "meadow-migrationmanager": "^0.0.14",
+        "meadow-migrationmanager": "^0.0.15",
         "orator": "^6.1.1",
         "orator-authentication": "^1.0.1",
         "orator-serviceserver-restify": "^2.0.10",
@@ -71,7 +71,7 @@
     "devDependencies": {
         "pict-docuserve": "^0.1.5",
         "puppeteer": "^24.40.0",
-        "quackage": "^1.2.0"
+        "quackage": "^1.2.2"
     },
     "mocha": {
         "diff": true,

package/source/services/Ultravisor-Beacon-Coordinator.cjs

@@ -1154,29 +1154,51 @@ class UltravisorBeaconCoordinator extends libPictService
 			tmpDefaults.applyToWorkItem(tmpWorkItem, tmpSettings);
 		}
 
-		// Check for affinity binding — pre-assign to a specific Beacon
+		// Routing for AffinityKey:
+		//   1) Name resolution — if AffinityKey matches a registered
+		//      beacon's Name, route directly to that beacon. This is
+		//      the explicit "send to this specific beacon" path the
+		//      data-platform initiative needs (many beacons of the
+		//      same capability, callers must designate which one).
+		//   2) Sticky binding fallback — if no name match, behave like
+		//      a session-affinity key: the first item picks any
+		//      beacon, subsequent items with the same key go to the
+		//      same beacon (existing behavior).
 		if (tmpWorkItem.AffinityKey)
 		{
-			let tmpBinding = this._AffinityBindings[tmpWorkItem.AffinityKey];
-
-			if (tmpBinding && this._Beacons[tmpBinding.BeaconID])
+			let tmpNamedBeacon = this.findBeaconByName(tmpWorkItem.AffinityKey);
+			if (tmpNamedBeacon)
 			{
-				// Check if the binding has expired
-				if (new Date(tmpBinding.ExpiresAt) > new Date())
+				tmpWorkItem.AssignedBeaconID = tmpNamedBeacon.BeaconID;
+				tmpWorkItem.Status = 'Assigned';
+				tmpWorkItem.ClaimedAt = new Date().toISOString();
+				if (!tmpNamedBeacon.CurrentWorkItems) tmpNamedBeacon.CurrentWorkItems = [];
+				tmpNamedBeacon.CurrentWorkItems.push(tmpWorkItemHash);
+				this.log.info(`BeaconCoordinator: work item [${tmpWorkItemHash}] routed by name to beacon [${tmpNamedBeacon.BeaconID}] (Name=${tmpNamedBeacon.Name}) via AffinityKey [${tmpWorkItem.AffinityKey}].`);
+			}
+			else
+			{
+				let tmpBinding = this._AffinityBindings[tmpWorkItem.AffinityKey];
+
+				if (tmpBinding && this._Beacons[tmpBinding.BeaconID])
 				{
-					tmpWorkItem.AssignedBeaconID = tmpBinding.BeaconID;
-					tmpWorkItem.Status = 'Assigned';
-					tmpWorkItem.ClaimedAt = new Date().toISOString();
+					// Check if the binding has expired
+					if (new Date(tmpBinding.ExpiresAt) > new Date())
+					{
+						tmpWorkItem.AssignedBeaconID = tmpBinding.BeaconID;
+						tmpWorkItem.Status = 'Assigned';
+						tmpWorkItem.ClaimedAt = new Date().toISOString();
 
-					let tmpBeacon = this._Beacons[tmpBinding.BeaconID];
-					tmpBeacon.CurrentWorkItems.push(tmpWorkItemHash);
+						let tmpBeacon = this._Beacons[tmpBinding.BeaconID];
+						tmpBeacon.CurrentWorkItems.push(tmpWorkItemHash);
 
-					this.log.info(`BeaconCoordinator: work item [${tmpWorkItemHash}] pre-assigned to beacon [${tmpBinding.BeaconID}] via affinity [${tmpWorkItem.AffinityKey}].`);
-				}
-				else
-				{
-					// Binding expired, clean it up
-					delete this._AffinityBindings[tmpWorkItem.AffinityKey];
+						this.log.info(`BeaconCoordinator: work item [${tmpWorkItemHash}] pre-assigned to beacon [${tmpBinding.BeaconID}] via affinity [${tmpWorkItem.AffinityKey}].`);
+					}
+					else
+					{
+						// Binding expired, clean it up
+						delete this._AffinityBindings[tmpWorkItem.AffinityKey];
+					}
 				}
 			}
 		}

package/source/services/Ultravisor-Beacon-Scheduler.cjs

@@ -227,6 +227,19 @@ class UltravisorBeaconScheduler extends libPictService
 			return null;
 		}
 
+		// Name-based routing: AffinityKey may designate a specific
+		// beacon by Name. Mirrors the resolution Coordinator does at
+		// enqueue time; we repeat it here so paths that bypass
+		// enqueueWorkItem (e.g. requeue) still honor it.
+		if (pItem.AffinityKey && typeof pCoordinator.findBeaconByName === 'function')
+		{
+			let tmpNamed = pCoordinator.findBeaconByName(pItem.AffinityKey);
+			if (tmpNamed)
+			{
+				return this._beaconCanTake(tmpNamed, pItem) ? tmpNamed : null;
+			}
+		}
+
 		let tmpBestBeacon = null;
 		let tmpBestLoad = Infinity;
 		let tmpBeacons = pCoordinator._Beacons || {};

package/source/services/Ultravisor-ExecutionEngine.cjs

@@ -1108,26 +1108,35 @@ class UltravisorExecutionEngine extends libPictService
 			fProcessNextDownstream();
 		};
 
-		// Execute the task
+		// Execute the task. Wrap the synchronous portion of the call in
+		// try/catch — a misbehaving executor that throws synchronously
+		// (e.g. simple-get's ERR_INVALID_ARG_TYPE on a non-string body)
+		// would otherwise tear down the entire UV process. Treating the
+		// throw as a task error fires the Error edge and lets the rest
+		// of the operation graph clean up gracefully.
 		this.log.info(`[Engine] executeTask: running task type="${tmpNode.Type}" node=${pNodeHash}`);
-		tmpTaskInstance.execute(tmpResolvedSettings, tmpTaskContext, (pError, pResult) =>
+		let fHandleTaskError = (pError) =>
 		{
-			if (pError)
+			this.log.warn(`[Engine] executeTask: TASK ERROR node=${pNodeHash}: ${pError.message}`);
+			this._log(pContext, `Task [${pNodeHash}] error: ${pError.message}`, 'error');
+			if (tmpManifestService)
 			{
-				this.log.warn(`[Engine] executeTask: TASK ERROR node=${pNodeHash}: ${pError.message}`);
-				this._log(pContext, `Task [${pNodeHash}] error: ${pError.message}`, 'error');
-				if (tmpManifestService)
+				tmpManifestService.recordTaskError(pContext, pNodeHash, pError);
+				tmpManifestService.recordEvent(pContext, pNodeHash, 'TaskError',
+					`Error in [${pNodeHash}]: ${pError.message}`, 0);
+			}
+			this._enqueueDownstreamEvents(pNodeHash, 'Error', pContext);
+			return fCallback(null);
+		};
+		try
+		{
+			tmpTaskInstance.execute(tmpResolvedSettings, tmpTaskContext, (pError, pResult) =>
+			{
+				if (pError)
 				{
-					tmpManifestService.recordTaskError(pContext, pNodeHash, pError);
-					tmpManifestService.recordEvent(pContext, pNodeHash, 'TaskError',
-						`Error in [${pNodeHash}]: ${pError.message}`, 0);
+					return fHandleTaskError(pError);
 				}
 
-				// Fire error event if the task has one
-				this._enqueueDownstreamEvents(pNodeHash, 'Error', pContext);
-				return fCallback(null);
-			}
-
 			if (!pResult)
 			{
 				this._log(pContext, `Task [${pNodeHash}] returned no result.`, 'warn');
@@ -1257,6 +1266,11 @@ class UltravisorExecutionEngine extends libPictService
 			return fCallback(null);
 		},
 		fFireIntermediateEvent);
+		}
+		catch (pSyncError)
+		{
+			return fHandleTaskError(pSyncError);
+		}
 	}
 
 	// ====================================================================

package/source/services/tasks/http-client/Ultravisor-TaskConfigs-HttpClient.cjs

@@ -305,16 +305,24 @@ module.exports =
 
 			let tmpRequestOptions = { url: tmpURL, method: tmpMethod, headers: tmpHeaders, timeout: pResolvedSettings.TimeoutMs || 30000 };
 
-			// Parse body for non-GET methods
+			// Body for non-GET methods. The HTTP client's `body` option
+			// expects a string (or Buffer / stream); object bodies cause
+			// `Buffer.byteLength(object)` to throw ERR_INVALID_ARG_TYPE
+			// inside simple-get and crash the engine. Pass the configured
+			// Body through verbatim — Content-Type tells the server how
+			// to interpret it. (Validating-parse the JSON for an early
+			// clearer error, but always serialize back to a string.)
 			if (tmpMethod !== 'GET' && pResolvedSettings.Body)
 			{
-				try
+				if (typeof pResolvedSettings.Body === 'string')
 				{
-					tmpRequestOptions.body = JSON.parse(pResolvedSettings.Body);
+					try { JSON.parse(pResolvedSettings.Body); }
+					catch (pParseError) { /* ok — non-JSON bodies (e.g. form-encoded) pass through too */ }
+					tmpRequestOptions.body = pResolvedSettings.Body;
 				}
-				catch (pParseError)
+				else
 				{
-					tmpRequestOptions.body = pResolvedSettings.Body;
+					tmpRequestOptions.body = JSON.stringify(pResolvedSettings.Body);
 				}
 			}