@algosuite/vo-mcp 0.2.0-beta.66 → 0.2.0-beta.67

package/dist/runner-supervisor.js

@@ -2440,8 +2440,6 @@ var DEFAULT_DRAIN_CHECK_MS = 3e4;
 var MAX_DRAIN_CAP_MS = 6 * 60 * 60 * 1e3;
 var MIN_DRAIN_CHECK_MS = 1e3;
 var MAX_DRAIN_CHECK_MS = 5 * 60 * 1e3;
-var UPDATE_RESTART_ERROR_MESSAGE = "runner update restart after drain timeout";
-var UPDATE_RESTART_RESULT = "runner_update_restart";
 var sleepMs = (ms) => new Promise((resolve5) => setTimeout(resolve5, ms));
 function positiveNumber(raw) {
   if (raw === void 0 || raw === null || String(raw).trim() === "") return null;
@@ -2489,34 +2487,13 @@ function describeBusy(active) {
   const ids = active.ids.length > 0 ? ` [${active.ids.join(", ")}]` : " [ids unavailable]";
   return `${active.count} active task(s)${ids}`;
 }
-async function markCappedTasks({ active, failInFlightTask, log }) {
-  if (active.ids.length === 0) {
-    if (active.count > 0 || !active.known) {
-      log(`update drain cap reached with ${describeBusy(active)} \u2014 cannot annotate tasks the runner did not name`);
-    }
-    return [];
-  }
-  const marked = [];
-  for (const taskId of active.ids) {
-    try {
-      await failInFlightTask(taskId, {
-        message: UPDATE_RESTART_ERROR_MESSAGE,
-        result: UPDATE_RESTART_RESULT,
-        runnerId: active.runnerId,
-        runnerInstanceId: active.runnerInstanceId
-      });
-      marked.push(taskId);
-    } catch (error) {
-      log(`could not mark task ${taskId} before update restart: ${error instanceof Error ? error.message : String(error)}`);
-    }
-  }
-  return marked;
+function describeDeferral(active) {
+  if (!active.known) return "status unreadable from a running child (assumed busy)";
+  return `${active.count} active task(s)`;
 }
 async function awaitRunnerUpdateDrain({
   readStatus,
   isChildRunning = () => true,
-  failInFlightTask = async () => {
-  },
   drainable = false,
   env = {},
   now = Date.now,
@@ -2565,15 +2542,18 @@ async function awaitRunnerUpdateDrain({
       return { proceed: true, detail: "runner idle", capped: false, waitedMs: elapsedMs, checks, markedTaskIds: [] };
     }
     if (step.action === "cap") {
-      log(`update drain cap reached after ${Math.round(elapsedMs / 1e3)}s (cap ${Math.round(capMs / 1e3)}s) \u2014 ${describeBusy(active)}; marking task(s) before restart`);
-      const markedTaskIds = await markCappedTasks({ active, failInFlightTask, log });
+      log(`update drain cap reached after ${Math.round(elapsedMs / 1e3)}s (cap ${Math.round(capMs / 1e3)}s) \u2014 ${describeBusy(active)}; deferring update until idle`);
       return {
-        proceed: true,
+        proceed: false,
         capped: true,
-        detail: `update restart applied after drain cap; marked ${markedTaskIds.length} task(s) as "${UPDATE_RESTART_ERROR_MESSAGE}"`,
+        // This reaches the runner-control `detail` field; keep it bounded and
+        // reserve task IDs for the local log above.
+        detail: `deferred safely: update drain cap reached with ${describeDeferral(active)}; retry when the runner is idle`,
         waitedMs: elapsedMs,
         checks,
-        markedTaskIds
+        // Kept for result-shape compatibility; a safe deferral never writes
+        // a terminal task outcome or stops the child.
+        markedTaskIds: []
       };
     }
     const fingerprint = `${step.busy}:${active.ids.join(",")}`;
@@ -3694,7 +3674,6 @@ async function main() {
         drainable: bundledAction,
         env: process.env,
         isChildRunning: () => supervisorChildIsRunning(child),
-        failInFlightTask: (taskId, patch) => client.postProgress(taskId, { status: "failed", message: patch.message, result: patch.result, ...patch.runnerId ? { runner_id: patch.runnerId } : {}, ...patch.runnerInstanceId ? { runner_instance_id: patch.runnerInstanceId } : {} }),
         log: (message) => console.warn(`[vo-runner supervisor] ${message}`)
       });
       if (!drain.proceed) {