@salesforce/sfdx-agent-harness-mastra 0.62.0 → 0.63.0

package/CHANGELOG.md

@@ -3,6 +3,11 @@
 All notable changes to `@salesforce/sfdx-agent-harness-mastra` are documented in this file.
 Format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
+## [0.63.0] - 2026-09-02
+
+### Features
+- **agent-sdk,harness-mastra,harness-claude,harness-openai**: reject concurrent turns with a typed SESSION_BUSY error @W-24044572@ ([#783](https://github.com/forcedotcom/agentic-dx/pull/783))
+
 ## [0.62.0] - 2026-09-02
 
 ### Fixes

package/dist/mastra-approval-coordinator.d.ts

@@ -261,6 +261,18 @@ export declare class MastraApprovalCoordinator {
     private timedOut;
     constructor(opts: MastraApprovalCoordinatorOptions);
     initialSink(): GenSink<ChatEvent>;
+    /**
+     * True once this turn has reached a terminal event — a natural `finish`
+     * pushed by the pump, or the synthetic `error` + `finish('error')` pair a
+     * `teardown` (abort / dispose / timeout / pump-error) emits. Both end the
+     * single per-turn sink, so `initial.isEnded()` is the authoritative "this
+     * turn is over" signal. A coordinator still pumping — including one PARKED on
+     * a `tool-approval-request` or a consumer-tool `submitToolResult` — has not
+     * ended its sink and is therefore NOT settled. `MastraHarness.stream()` reads
+     * this to reject a concurrent turn with `SESSION_BUSY` (a still-active
+     * coordinator) versus safely replace a settled one.
+     */
+    isSettled(): boolean;
     /**
      * Mastra's current run id. The harness's MCP-progress dispatcher uses
      * this to filter incoming `notifications/progress` to the coordinator

package/dist/mastra-approval-coordinator.js

@@ -184,6 +184,20 @@ export class MastraApprovalCoordinator {
     initialSink() {
         return this.initial;
     }
+    /**
+     * True once this turn has reached a terminal event — a natural `finish`
+     * pushed by the pump, or the synthetic `error` + `finish('error')` pair a
+     * `teardown` (abort / dispose / timeout / pump-error) emits. Both end the
+     * single per-turn sink, so `initial.isEnded()` is the authoritative "this
+     * turn is over" signal. A coordinator still pumping — including one PARKED on
+     * a `tool-approval-request` or a consumer-tool `submitToolResult` — has not
+     * ended its sink and is therefore NOT settled. `MastraHarness.stream()` reads
+     * this to reject a concurrent turn with `SESSION_BUSY` (a still-active
+     * coordinator) versus safely replace a settled one.
+     */
+    isSettled() {
+        return this.initial.isEnded();
+    }
     /**
      * Mastra's current run id. The harness's MCP-progress dispatcher uses
      * this to filter incoming `notifications/progress` to the coordinator

package/dist/mastra-harness.js

@@ -1618,12 +1618,20 @@ export class MastraHarness {
         // round-trip per tool call.
         const { lookupPolicy, batchApprovals } = this.buildPolicyGate(agentState, options);
         const gatingActive = lookupPolicy !== undefined;
-        // Dispose any coordinator the prior turn on this thread allocated.
-        // Without this a stale coordinator from the prior turn would still
-        // route settle calls and accumulate listeners. Mode-flip safe: the
-        // new coordinator is constructed fresh below regardless of the new
-        // turn's mode.
+        // Concurrency guard (spike agent-offline-failure-modes #10, W-24044572):
+        // reject a second turn on a thread whose prior turn is still ACTIVE with a
+        // typed `SESSION_BUSY` error, instead of silently disposing the first
+        // (which surfaced to the first consumer as a spurious `DISPOSED` — a
+        // "silent lost turn"). A coordinator is active until its sink ends (natural
+        // finish or teardown), so one still pumping — including one parked on an
+        // approval / consumer-tool result — blocks a new turn. An already-settled
+        // coordinator (finished or errored) is safe to dispose-and-replace, so the
+        // idempotent settling of an already-terminal turn is preserved.
         const priorCoordinator = agentState.coordinators.get(threadId);
+        if (priorCoordinator && !priorCoordinator.isSettled()) {
+            throw new AgentSDKError(`A turn is already in progress for thread "${threadId}" on agent "${agentId}". ` +
+                'Only one turn may run at a time per thread; await the in-flight turn before starting another.', AgentSDKErrorType.SESSION_BUSY);
+        }
         if (priorCoordinator) {
             priorCoordinator.dispose();
             agentState.coordinators.delete(threadId);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@salesforce/sfdx-agent-harness-mastra",
-  "version": "0.62.0",
+  "version": "0.63.0",
   "description": "Mastra-backed AgentHarness implementation for @salesforce/sfdx-agent-sdk",
   "type": "module",
   "main": "dist/index.js",
@@ -42,12 +42,12 @@
     "openai": "^7.0.0"
   },
   "peerDependencies": {
-    "@salesforce/sfdx-agent-sdk": "0.63.0"
+    "@salesforce/sfdx-agent-sdk": "0.64.0"
   },
   "devDependencies": {
     "@eslint/js": "^10.0.1",
     "@salesforce/harness-conformance": "0.0.1",
-    "@salesforce/sfdx-agent-sdk": "0.63.0",
+    "@salesforce/sfdx-agent-sdk": "0.64.0",
     "@types/node": "^22.20.1",
     "@vitest/coverage-istanbul": "^4.1.10",
     "@vitest/eslint-plugin": "^1.6.27",