@salesforce/sfdx-agent-sdk 0.6.0 → 0.7.0

package/dist/agent-manager.js

@@ -157,7 +157,7 @@ export class DefaultAgentManager {
         const runtime = await this.agentConnectivityResolver.resolve(projectRoot, config);
         await this.harness.createAgent(agentId, projectRoot, runtime.llmGatewayClient, toHarnessConfig(config, runtime.orgJwt), options.abortSignal !== undefined ? { abortSignal: options.abortSignal } : undefined);
         const agentSlice = this.router.registerAgent(agentId);
-        const agent = new DefaultAgent(this.harness, agentId, projectRoot, config, runtime.llmGatewayClient, runtime.orgConnection, runtime.orgJwt, this.agentConnectivityResolver, this.router, agentSlice, { telemetry: this.telemetryBus, log: this.logBus }, this.clock, this.agentIdGenerator);
+        const agent = new DefaultAgent(this.harness, agentId, projectRoot, config, runtime.llmGatewayClient, runtime.orgConnection, runtime.orgJwt, this.agentConnectivityResolver, this.identityStore, this.router, agentSlice, { telemetry: this.telemetryBus, log: this.logBus }, this.clock, this.agentIdGenerator);
         this.agents.set(agentId, agent);
         this.telemetryBus.emit({
             type: 'agent-created',

package/dist/agent.d.ts

@@ -5,6 +5,7 @@ import { type ChatSession } from './chat-session.js';
 import type { McpServerInfo } from './mcp-config.js';
 import { type JSONWebToken, type LLMGatewayClient } from '@salesforce/llm-gateway-sdk';
 import type { AgentConnectivityResolver } from './agent-connectivity-resolver.js';
+import type { AgentIdentityStore } from './internal/agent-identity-store.js';
 import type { TelemetryRouter, TelemetrySlice } from './internal/telemetry-router.js';
 import type { TelemetryBus, TelemetryEventCallback } from './types/telemetry-events.js';
 /**
@@ -110,6 +111,7 @@ export declare class DefaultAgent implements Agent {
     private orgConnection;
     private orgJwt;
     private readonly agentConnectivityResolver;
+    private readonly identityStore;
     private readonly sessions;
     private readonly sessionSliceUnregisters;
     private readonly router;
@@ -129,11 +131,13 @@ export declare class DefaultAgent implements Agent {
      * @param orgConnection - Authenticated org connection carrying identity and env inference.
      * @param orgJwt - Self-refreshing JWT for the resolved org (used for MCP auth injection).
      * @param agentConnectivityResolver - Used to re-resolve org connectivity when the org or model changes.
+     * @param identityStore - SDK-owned persistence for the `{ agentId, projectRoot, AgentConfig }` triple. The agent
+     *     calls `write()` on a successful `updateAgentConfig` so disk state and in-memory state stay in lockstep.
      * @param router - Telemetry router used to obtain session slices when sessions are created.
      * @param inbound - Router slice delivering harness events routed to this agent (non-session-scoped).
      * @param parent - Manager's bus pair; this agent forwards its events upward into them.
      */
-    constructor(harness: AgentHarness, agentId: string, projectRoot: string, config: AgentConfig, llmGatewayClient: LLMGatewayClient, orgConnection: OrgConnection, orgJwt: JSONWebToken, agentConnectivityResolver: AgentConnectivityResolver, router: TelemetryRouter, inbound: TelemetrySlice, parent: AgentParentBuses, clock?: Clock, idGenerator?: UniqueIDGenerator);
+    constructor(harness: AgentHarness, agentId: string, projectRoot: string, config: AgentConfig, llmGatewayClient: LLMGatewayClient, orgConnection: OrgConnection, orgJwt: JSONWebToken, agentConnectivityResolver: AgentConnectivityResolver, identityStore: AgentIdentityStore, router: TelemetryRouter, inbound: TelemetrySlice, parent: AgentParentBuses, clock?: Clock, idGenerator?: UniqueIDGenerator);
     /**
      * @requirements
      * - MUST return the agent's ID.
@@ -154,7 +158,9 @@ export declare class DefaultAgent implements Agent {
      * - MUST guarantee that the `agentId` remains unchanged during the merge.
      * - MUST destroy the existing agent in the harness by delegating to `this.harness.destroyAgent(this.getId())`.
      * - MUST recreate the agent in the harness with the newly merged configuration by delegating to `this.harness.createAgent(...)`.
-     * - MUST preserve the previous in-memory config state if recreation fails.
+     * - MUST persist the merged config via `this.identityStore.write(...)` after the harness recreate succeeds and
+     *   before the in-memory swaps, so a write failure rolls back through the same catch path as a recreate failure.
+     * - MUST preserve the previous in-memory config state if recreation or persistence fails.
      */
     updateAgentConfig(config?: AgentConfig, options?: {
         abortSignal?: AbortSignal;

package/dist/agent.js

@@ -20,6 +20,7 @@ export class DefaultAgent {
     orgConnection;
     orgJwt;
     agentConnectivityResolver;
+    identityStore;
     sessions = new Map();
     sessionSliceUnregisters = new Map();
     router;
@@ -39,11 +40,13 @@ export class DefaultAgent {
      * @param orgConnection - Authenticated org connection carrying identity and env inference.
      * @param orgJwt - Self-refreshing JWT for the resolved org (used for MCP auth injection).
      * @param agentConnectivityResolver - Used to re-resolve org connectivity when the org or model changes.
+     * @param identityStore - SDK-owned persistence for the `{ agentId, projectRoot, AgentConfig }` triple. The agent
+     *     calls `write()` on a successful `updateAgentConfig` so disk state and in-memory state stay in lockstep.
      * @param router - Telemetry router used to obtain session slices when sessions are created.
      * @param inbound - Router slice delivering harness events routed to this agent (non-session-scoped).
      * @param parent - Manager's bus pair; this agent forwards its events upward into them.
      */
-    constructor(harness, agentId, projectRoot, config, llmGatewayClient, orgConnection, orgJwt, agentConnectivityResolver, router, inbound, parent, clock = new RealClock(), idGenerator = new UUIDGenerator()) {
+    constructor(harness, agentId, projectRoot, config, llmGatewayClient, orgConnection, orgJwt, agentConnectivityResolver, identityStore, router, inbound, parent, clock = new RealClock(), idGenerator = new UUIDGenerator()) {
         this.harness = harness;
         this.agentId = agentId;
         this.projectRoot = projectRoot;
@@ -52,6 +55,7 @@ export class DefaultAgent {
         this.orgConnection = orgConnection;
         this.orgJwt = orgJwt;
         this.agentConnectivityResolver = agentConnectivityResolver;
+        this.identityStore = identityStore;
         this.router = router;
         this.clock = clock;
         this.idGenerator = idGenerator;
@@ -93,7 +97,9 @@ export class DefaultAgent {
      * - MUST guarantee that the `agentId` remains unchanged during the merge.
      * - MUST destroy the existing agent in the harness by delegating to `this.harness.destroyAgent(this.getId())`.
      * - MUST recreate the agent in the harness with the newly merged configuration by delegating to `this.harness.createAgent(...)`.
-     * - MUST preserve the previous in-memory config state if recreation fails.
+     * - MUST persist the merged config via `this.identityStore.write(...)` after the harness recreate succeeds and
+     *   before the in-memory swaps, so a write failure rolls back through the same catch path as a recreate failure.
+     * - MUST preserve the previous in-memory config state if recreation or persistence fails.
      */
     async updateAgentConfig(config = {}, options) {
         this.assertNotDisposed();
@@ -121,6 +127,10 @@ export class DefaultAgent {
         await this.harness.destroyAgent(this.agentId);
         try {
             await this.harness.createAgent(this.agentId, this.projectRoot, nextClient, toHarnessConfig(nextConfig, nextOrgJwt), options);
+            // Persist before the in-memory swaps so a write failure flows through the same
+            // catch block as a recreate failure: the rollback restores the harness with
+            // previousConfig and disk state remains the pre-update record.
+            await this.identityStore.write(this.agentId, this.projectRoot, nextConfig);
             this.config = nextConfig;
             this.llmGatewayClient = nextClient;
             this.orgConnection = nextConnection;
@@ -138,6 +148,12 @@ export class DefaultAgent {
                 if (nextClient === previousClient) {
                     previousClient.setModel(Models.getByName(previousModelName));
                 }
+                // Clear any nextConfig registration left behind by a successful harness recreate
+                // before the rollback createAgent runs. On the harness-recreate-failure path this
+                // is a no-op (the agent was never registered with nextConfig); on the
+                // identityStore.write-failure path it removes the live nextConfig so the rollback
+                // doesn't trip the harness's duplicate-registration guard.
+                await this.harness.destroyAgent(this.agentId);
                 await this.harness.createAgent(this.agentId, this.projectRoot, previousClient, toHarnessConfig(previousConfig, previousOrgJwt));
             }
             catch {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@salesforce/sfdx-agent-sdk",
-  "version": "0.6.0",
+  "version": "0.7.0",
   "description": "Harness-agnostic agentic infrastructure for Salesforce developer experience tooling",
   "type": "module",
   "main": "dist/index.js",