@salesforce/sfdx-agent-harness-mastra 0.86.0 → 0.88.0

package/CHANGELOG.md

@@ -3,6 +3,16 @@
 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.88.0] - 2026-09-18
+
+### Fixes
+- defer previous-workspace destroy while a coordinator is parked so a remembered-approval resume doesn't crash @W-24233008@ ([#834](https://github.com/forcedotcom/agentic-dx/pull/834))
+
+## [0.87.0] - 2026-09-17
+
+### Chores
+- **deps**: upgrade all dependencies to latest @W-24127072@ ([#831](https://github.com/forcedotcom/agentic-dx/pull/831))
+
 ## [0.86.0] - 2026-09-17
 
 ### Fixes

package/dist/mastra-approval-coordinator.js

@@ -242,7 +242,10 @@ export class MastraApprovalCoordinator {
         });
     }
     onTeardown(callback) {
-        if (this.aborted || this.disposed) {
+        // If already terminal, fire now: a callback added after teardown ran
+        // would land in an already-cleared set and never fire. `isSettled()`
+        // covers natural-finish/timeout/pump-error settles (no abort/dispose flag).
+        if (this.aborted || this.disposed || this.isSettled()) {
             queueMicrotask(callback);
             return;
         }

package/dist/mastra-harness.js

@@ -1257,8 +1257,45 @@ export class MastraHarness {
             state.toolSearchProcessor = toolSearchProcessor;
             state.skillSearchProcessor = skillSearchProcessor;
             state.hooks = options?.hooks ?? {};
-            previousSkillSearchProcessor?.dispose();
-            await previousWorkspace?.destroy().catch(() => undefined);
+            // Defer tearing down the previous Agent's workspace (+ its skill-search
+            // processor) until every coordinator still bound to that Agent settles;
+            // do NOT destroy synchronously. A coordinator parked on a tool-approval
+            // resumes against the old Agent and reads `workspace.skills` (in
+            // resumeStream, and via the skill-search processor's getter) — if the
+            // workspace is already destroyed, @mastra/core throws
+            // WorkspaceNotReadyError. Common trigger: an "Allow always" approval,
+            // whose rule-persist calls updateAgentConfig (→ this method) before the
+            // approval settles.
+            //
+            // Two things must BOTH stay for this to be correct: the filter +
+            // `onTeardown` registration run in one synchronous tick (so no
+            // coordinator settles between them and is missed), and `onTeardown`'s
+            // own `isSettled()` arm handles an already-settled coordinator. Bounded
+            // because `destroyAgent()`/`shutdown()` force-dispose coordinators, so
+            // the deferred teardown always eventually runs.
+            const disposePreviousAgentResources = async () => {
+                previousSkillSearchProcessor?.dispose();
+                await previousWorkspace?.destroy();
+            };
+            const onTeardownFailed = (error) => {
+                // Guarded: a deferred teardown can reject after shutdown() disposed
+                // the buses; logging there must not become an unhandled rejection.
+                try {
+                    this.buses.getLogBus()?.debug('Deferred previous-Agent resource teardown failed', { error });
+                }
+                catch {
+                    /* buses already disposed */
+                }
+            };
+            const unsettledCoordinators = [...state.coordinators.values()].filter((c) => !c.isSettled());
+            if (unsettledCoordinators.length === 0) {
+                await disposePreviousAgentResources().catch(onTeardownFailed);
+            }
+            else {
+                void Promise.all(unsettledCoordinators.map((c) => new Promise((resolve) => c.onTeardown(resolve))))
+                    .then(() => disposePreviousAgentResources())
+                    .catch(onTeardownFailed);
+            }
         }
         catch (error) {
             // Phase 2 failed after Phase 1 mutated state.mcpServers. We don't

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@salesforce/sfdx-agent-harness-mastra",
-  "version": "0.86.0",
+  "version": "0.88.0",
   "description": "Mastra-backed AgentHarness implementation for @salesforce/sfdx-agent-sdk",
   "type": "module",
   "main": "dist/index.js",
@@ -29,27 +29,27 @@
     "LICENSE.txt"
   ],
   "dependencies": {
-    "@ai-sdk/provider": "^4.0.10",
-    "@anthropic-ai/bedrock-sdk": "^0.32.0",
-    "@anthropic-ai/sdk": "^0.122.0",
-    "@mastra/core": "^1.64.0",
-    "@mastra/libsql": "^1.22.3",
-    "@mastra/mcp": "^1.17.3",
-    "@mastra/memory": "^1.28.2",
-    "@mastra/schema-compat": "^1.3.7",
-    "@salesforce/agentic-common": "0.20.0",
+    "@ai-sdk/provider": "4.0.4",
+    "@anthropic-ai/bedrock-sdk": "^0.33.6",
+    "@anthropic-ai/sdk": "^0.126.0",
+    "@mastra/core": "^1.67.0",
+    "@mastra/libsql": "^1.23.0",
+    "@mastra/mcp": "^1.18.0",
+    "@mastra/memory": "^1.30.0",
+    "@mastra/schema-compat": "^1.3.10",
+    "@salesforce/agentic-common": "0.21.0",
     "@smithy/eventstream-codec": "^4.5.2",
-    "openai": "^7.0.0"
+    "openai": "^7.17.0"
   },
   "peerDependencies": {
-    "@salesforce/sfdx-agent-sdk": "0.87.0"
+    "@salesforce/sfdx-agent-sdk": "0.89.0"
   },
   "devDependencies": {
     "@eslint/js": "^10.0.1",
     "@salesforce/harness-conformance": "0.0.1",
-    "@salesforce/sfdx-agent-sdk": "0.87.0",
+    "@salesforce/sfdx-agent-sdk": "0.89.0",
     "@types/node": "^22.20.1",
-    "@vitest/coverage-istanbul": "^4.1.11",
+    "@vitest/coverage-istanbul": "^5.0.1",
     "@vitest/eslint-plugin": "^1.6.27",
     "eslint": "^10.10.0",
     "eslint-config-prettier": "^10.1.8",
@@ -57,13 +57,13 @@
     "eslint-plugin-import": "^2.32.0",
     "eslint-plugin-n": "^18.3.0",
     "globals": "^17.12.0",
-    "lint-staged": "^17.5.0",
-    "prettier": "^3.9.6",
+    "lint-staged": "^17.5.1",
+    "prettier": "^3.9.7",
     "rimraf": "^6.1.3",
     "tsx": "^4.23.13",
     "typescript": "^7.0.2",
     "typescript-eslint": "^8.70.0",
-    "vitest": "^4.1.11"
+    "vitest": "^5.0.1"
   },
   "engines": {
     "node": ">=22.22.1"