@skaile/workspaces 4.28.2 → 4.29.0

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 4.29.0
+
+### Minor Changes
+
+- [#925](https://github.com/skaile-ai/workspaces/pull/925) [`5579aeb`](https://github.com/skaile-ai/workspaces/commit/5579aeb0db740725b6651b6ebc17fcb63288f655) Thanks [@peteralbert](https://github.com/peteralbert)! - The runner now sends `status: idle` at the end of a turn only when no other prompt, reply or flow turn is queued behind it. Before, every turn end reported idle, so a host that clears per-turn state on idle (the platform's turn-principal ledger) lost the starters of the turns still queued, and those turns read as `unknown`. An idle held back this way is sent once the queue drains, even when the driver ends several queued prompts with one `agent_end` or the last turn fails. The compaction-end idle is unchanged.
+
 ## 4.28.2
 
 ### Patch Changes

package/dist/{chunk-RPZ6ZE7S.js → chunk-3XMGAODZ.js}

@@ -4,7 +4,7 @@ import { findAiAssetsRoot } from './chunk-7IH3RKKK.js';
 import { WebSocketServerTransport } from './chunk-5LKOAX7I.js';
 import { assembleSystemPrompt, buildCapabilitiesPromptSection } from './chunk-W3UDISS2.js';
 import { prepareNativePolicy, codexInstanceEnvironment, parseCodexCredential, parseCodexRefreshInput, parseCodexRefreshResult } from './chunk-PE3A6FFW.js';
-import { resolveFlowPath } from './chunk-YU2CVP3M.js';
+import { resolveFlowPath } from './chunk-G6NTA3BZ.js';
 import { findAiAssetsRoot as findAiAssetsRoot$1, deployCatalogEntry, undeployCatalogEntry } from './chunk-Q6NDQ6BH.js';
 import { registerBuiltinConnectors, buildConnectorPromptSection, findMissingPackages, installNpmPackages, ConnectorManager, buildSdkConnectorTools, buildLazySdkFlowTools, describeFlowOperations, buildConnectorToolDefs, buildFlowToolDefs, SessionNodeExecutor, SessionSubpromptRunner, ClassifierSubpromptRunner, SessionInlineSubFlowRunner, driveOrchestratorTurn, buildSdkFlowTools } from './chunk-FUKJ4IU6.js';
 import { loadYamlConnectorDeclarations, withMaterializedConnectorDeclarations, SecretProviderChain, ConnectorStartupError, PreMintedSecretProvider, InMemorySecretProvider, resolvedConfigToDeclarations, isKnownSecretRef, deriveSingleMaterializedConnectorDeclaration } from './chunk-2JANXYTJ.js';
@@ -4800,6 +4800,43 @@ var DriverSwapGate = class {
   }
 };
 
+// runner/src/pending-turns.ts
+var PendingTurns = class {
+  constructor(emitIdle) {
+    this.emitIdle = emitIdle;
+  }
+  emitIdle;
+  count = 0;
+  idleOwed = false;
+  /** Count a turn as taken on. Returns an idempotent `end`; call it in a `finally`. */
+  begin() {
+    this.count += 1;
+    let ended = false;
+    return () => {
+      if (ended) return;
+      ended = true;
+      this.count -= 1;
+      if (this.count === 0 && this.idleOwed) {
+        this.idleOwed = false;
+        this.emitIdle();
+      }
+    };
+  }
+  /** Call at a turn's end, while that turn is still counted: whether to report idle now. */
+  shouldReportIdle() {
+    if (this.count > 1) {
+      this.idleOwed = true;
+      return false;
+    }
+    this.idleOwed = false;
+    return true;
+  }
+  /** Turns taken on and not finished; for diagnostics and tests. */
+  get size() {
+    return this.count;
+  }
+};
+
 // runner/src/serial-lock.ts
 function createSerialLock() {
   let tail = Promise.resolve();
@@ -8822,6 +8859,9 @@ async function startAgentServer(opts) {
   let driverStartInFlight = null;
   const driverSwapGate = new DriverSwapGate();
   const compactionGate = new CompactionGate();
+  const pendingTurns = new PendingTurns(() => {
+    if (!compacting) sendEvent({ type: "status", phase: "idle" });
+  });
   let driverRestartChain = Promise.resolve();
   let driver = secretsMode === "env" ? agentSession.driver : void 0;
   let secretsProvisioned = secretsMode === "env";
@@ -8932,7 +8972,13 @@ async function startAgentServer(opts) {
           if (driver !== starting) return;
           starting.on("agent-event", (event) => {
             const normalized = normalizer.normalize(event);
-            for (const e of normalized) forwardNormalizedEvent(e);
+            const turnEnded = event.type === "agent_end";
+            for (const e of normalized) {
+              if (turnEnded && !compacting && e.type === "status" && e.phase === "idle" && !pendingTurns.shouldReportIdle()) {
+                continue;
+              }
+              forwardNormalizedEvent(e);
+            }
           });
           starting.on("error", (err) => {
             log(`[serve] driver error: ${err.message}`);
@@ -8984,18 +9030,28 @@ async function startAgentServer(opts) {
     return stimulusBus;
   }
   function awaitFlowAgentEnd(prompt, timeoutMs) {
-    return dispatchFlowTurn(
-      {
-        gate: driverSwapGate,
-        awaitCompactionIdle: () => awaitCompactionIdle("flow turn"),
-        isCompacting: () => !compactionGate.isIdle(),
-        ensureDriverStarted,
-        getDriver: () => driver
-      },
-      prompt,
-      timeoutMs
+    return countPendingTurn(
+      () => dispatchFlowTurn(
+        {
+          gate: driverSwapGate,
+          awaitCompactionIdle: () => awaitCompactionIdle("flow turn"),
+          isCompacting: () => !compactionGate.isIdle(),
+          ensureDriverStarted,
+          getDriver: () => driver
+        },
+        prompt,
+        timeoutMs
+      )
     );
   }
+  async function countPendingTurn(run) {
+    const end = pendingTurns.begin();
+    try {
+      return await run();
+    } finally {
+      end();
+    }
+  }
   function ensureResourceManagerForFlow() {
     if (!resourceManager) {
       registerBuiltinConnectors();
@@ -10041,7 +10097,7 @@ async function startAgentServer(opts) {
         log(`[serve] flow user-message turn failed: ${errMsg}`);
         sendEvent({ type: "error", message: `Flow turn failed: ${errMsg}`, fatal: false });
       } finally {
-        sendEvent({ type: "status", phase: "idle" });
+        if (pendingTurns.shouldReportIdle()) sendEvent({ type: "status", phase: "idle" });
       }
       return;
     }
@@ -10366,10 +10422,10 @@ async function startAgentServer(opts) {
         await handleSessionInit(cmd);
         break;
       case "prompt":
-        await handlePrompt(cmd);
+        await countPendingTurn(() => handlePrompt(cmd));
         break;
       case "reply":
-        await handleReply(cmd);
+        await countPendingTurn(() => handleReply(cmd));
         break;
       case "cancel":
         await handleCancel();
@@ -10679,5 +10735,5 @@ function touchSession(state) {
 }
 
 export { CLAUDE_CODE_CREDENTIALS_KEY, COMPILE_MANIFEST_FILENAME, CapabilityRegistry, DEFAULT_CAPABILITY_CALL_TIMEOUT_MS, DEFAULT_COALESCE_MS, DEFAULT_FLOW_MUTATE_CONNECTOR_WAIT_MS, MOUNT_LIST_MAX_DEPTH, MOUNT_LIST_MAX_ENTRIES, MOUNT_READ_DEADLINE_FACTOR, MOUNT_STAT_BUDGET_MS, MOUNT_STAT_MAX_ENTRIES, MOUNT_TREE_OP_DEADLINE_FACTOR, MarkdownStreamer, PreInitRingSink, ResourceOperationError, SESSION_ROOT_WRITABLE_DIRS, SESSION_ROOT_WRITABLE_FILES, WS_AUTH_TOKEN_ENV, WS_AUTH_TOKEN_FILE_ENV, agentDefinitionExists, bootstrapCapabilityRegistry, bootstrapRunnerLogStore, buildAgentResources, buildClientCapabilityHandler, buildConnectorTokenMediator, buildContextSection, buildEnvironmentSection, buildResourcesAvailablePayload, buildStoreSnapshotReplay, builtinCapabilities, capabilityLogInstance, clearPreInitRingSink, clearSession, compileComposition, computeCapabilitySignature, computeDirectoryTotals, connectorRefreshKind, createAgentSession, createFlowConnectorReadiness, createManagedCodexSubpromptDriver, createSessionStimulusBus, defineCapability, deleteSession, dispatchResourceRequest, dispatchRunnerCapabilityInvocation, emitSystemPromptComposed, ensureClaudeSettingsDisablesConnectors, ensureGitConfigInclude, executeConnectorMutate, extractClaudeAiOauthExpiresAt, flowRunDrivesTurns, flowStartNeedsDriverRestart, getPreInitRingSink, handleMountResourceRequest, handleResourceRequest, handleSessionRootResourceRequest, hasNonTerminalFlow, hostOpResumeStimulus, installPreInitRingSink, isSessionRootPathAllowed, isTerminalFlowStatus, listSessions, loadAgentManifest, loadCompileManifest, loadCompileManifestFromDir, loadManagedCodexHostConfig, loadSession, loadSessionById, mcpAuthSecretKey, newSession, nulTailPaddingLength, pickChatFlowTarget, pickFlowToolTarget, pickRepointedManager, planFlowMutate, prepareCodexTools, reconcilePendingDecisions, recreateKeepingLiveFlows, registerActiveFlow, registerCompositionCapabilities, rejectCapabilityOnApprovalDeny, rejectPendingCapabilityCallsOnDisconnect, resetRunnerLogStore, resolveAgentComposition, resolveAgentMixins, resolveBinding, resolveCapabilityCallTimeoutMs, resolveCapabilityResult, resolveComposition, resolveFlowMutateConnectorWaitMs, resolveMixin, restoredRunHasWorkToResume, runAgentChat, saveSession, setCurrentSession, setUpFlowConnector, startAgentServer, stimulusFromMetas, supportsLiveToolUpdates, takeRunnerWsAuthToken, toActiveFlowTarget, touchSession, validateInputResponse, writeClaudeCodeCredentialsFile };
-//# sourceMappingURL=chunk-RPZ6ZE7S.js.map
-//# sourceMappingURL=chunk-RPZ6ZE7S.js.map
+//# sourceMappingURL=chunk-3XMGAODZ.js.map
+//# sourceMappingURL=chunk-3XMGAODZ.js.map