@agentclientprotocol/codex-acp 2.5.1-preview.8 → 2.5.1-preview.9

package/dist/index.js

@@ -41064,7 +41064,7 @@ var package_default = {
   publishConfig: {
     access: "public"
   },
-  version: "2.5.1-preview.8",
+  version: "2.5.1-preview.9",
   description: "",
   main: "dist/index.js",
   bin: {
@@ -51788,7 +51788,11 @@ You have been logged out. Please try again.`);
       modeId: _params.modeId
     });
     const sessionState = this.getSessionState(_params.sessionId);
-    this.applyModeChange(sessionState, _params.modeId);
+    await this.applySessionConfigOption(sessionState, {
+      sessionId: _params.sessionId,
+      configId: MODE_CONFIG_ID,
+      value: _params.modeId
+    });
     return {};
   }
   async setSessionConfigOption(params) {
@@ -51800,7 +51804,7 @@ You have been logged out. Please try again.`);
     const sessionState = this.getSessionState(params.sessionId);
     await this.applySessionConfigOption(sessionState, params);
     return {
-      configOptions: this.createSessionConfigOptions(sessionState)
+      configOptions: this.createSessionConfigOptions(this.getSessionState(params.sessionId))
     };
   }
   async setSessionConfigOptionV2(params) {
@@ -51812,24 +51816,37 @@ You have been logged out. Please try again.`);
     return { configOptions: toV2ConfigOptions(response.configOptions) };
   }
   async applySessionConfigOption(sessionState, params) {
-    switch (params.configId) {
-      case FAST_MODE_CONFIG_ID:
-        this.applyFastModeChange(sessionState, params);
-        break;
-      case MODE_CONFIG_ID:
-        this.applyModeChange(sessionState, this.stringConfigValue(params));
-        break;
-      case COLLABORATION_MODE_CONFIG_ID:
-        await this.applyCollaborationModeChange(sessionState, this.stringConfigValue(params));
-        break;
-      case MODEL_CONFIG_ID:
-        this.applyModelChange(sessionState, this.stringConfigValue(params));
-        break;
-      case REASONING_EFFORT_CONFIG_ID:
-        this.applyReasoningEffortChange(sessionState, this.stringConfigValue(params));
-        break;
-      default:
-        throw RequestError.invalidParams();
+    await this.updateSessionConfig(sessionState, async () => {
+      switch (params.configId) {
+        case FAST_MODE_CONFIG_ID:
+          this.applyFastModeChange(sessionState, params);
+          break;
+        case MODE_CONFIG_ID:
+          this.applyModeChange(sessionState, this.stringConfigValue(params));
+          break;
+        case COLLABORATION_MODE_CONFIG_ID:
+          await this.applyCollaborationModeChange(sessionState, this.stringConfigValue(params));
+          break;
+        case MODEL_CONFIG_ID:
+          this.applyModelChange(sessionState, this.stringConfigValue(params));
+          break;
+        case REASONING_EFFORT_CONFIG_ID:
+          this.applyReasoningEffortChange(sessionState, this.stringConfigValue(params));
+          break;
+        default:
+          throw RequestError.invalidParams();
+      }
+    });
+  }
+  async updateSessionConfig(sessionState, change) {
+    const previous = JSON.stringify(this.createSessionConfigOptions(sessionState));
+    await change();
+    const configOptions = this.createSessionConfigOptions(sessionState);
+    if (this.isSessionConfigEnabled() && this.sessions.get(sessionState.sessionId) === sessionState && JSON.stringify(configOptions) !== previous) {
+      await new ACPSessionConnection(this.connection, sessionState.sessionId).update({
+        sessionUpdate: "config_option_update",
+        configOptions
+      });
     }
   }
   applyFastModeChange(sessionState, params) {
@@ -51894,20 +51911,15 @@ You have been logged out. Please try again.`);
    * The session keeps its model when it can, the way a resumed thread keeps its own; see `createModelId`.
    */
   async applyResumedModels(sessionState, client, resumed) {
-    const configOptions = JSON.stringify(this.createSessionConfigOptions(sessionState));
-    const { model, effort } = ModelId.fromString(sessionState.currentModelId);
-    sessionState.availableModels = resumed.models;
-    sessionState.currentModelId = client.createModelId(resumed.models, model, effort).toString();
-    const currentModel = this.findCurrentModel(resumed.models, sessionState.currentModelId);
-    sessionState.supportedReasoningEfforts = currentModel?.supportedReasoningEfforts ?? [];
-    sessionState.supportedInputModalities = currentModel?.inputModalities ?? ["text", "image"];
-    sessionState.currentModelSupportsFast = modelSupportsFast(currentModel);
-    if (JSON.stringify(this.createSessionConfigOptions(sessionState)) !== configOptions) {
-      await new ACPSessionConnection(this.connection, sessionState.sessionId).update({
-        sessionUpdate: "config_option_update",
-        configOptions: this.createSessionConfigOptions(sessionState)
-      });
-    }
+    await this.updateSessionConfig(sessionState, () => {
+      const { model, effort } = ModelId.fromString(sessionState.currentModelId);
+      sessionState.availableModels = resumed.models;
+      sessionState.currentModelId = client.createModelId(resumed.models, model, effort).toString();
+      const currentModel = this.findCurrentModel(resumed.models, sessionState.currentModelId);
+      sessionState.supportedReasoningEfforts = currentModel?.supportedReasoningEfforts ?? [];
+      sessionState.supportedInputModalities = currentModel?.inputModalities ?? ["text", "image"];
+      sessionState.currentModelSupportsFast = modelSupportsFast(currentModel);
+    });
   }
   applyModelAndEffort(sessionState, model, effort) {
     sessionState.currentModelId = ModelId.fromComponents(model, effort).toString();
@@ -51938,8 +51950,10 @@ You have been logged out. Please try again.`);
     } else {
       reasoningEffort = model.defaultReasoningEffort;
     }
-    sessionState.availableModels = models;
-    this.applyModelAndEffort(sessionState, model, reasoningEffort);
+    await this.updateSessionConfig(sessionState, () => {
+      sessionState.availableModels = models;
+      this.applyModelAndEffort(sessionState, model, reasoningEffort);
+    });
     return {};
   }
   parseLegacySetSessionModelParams(params) {
@@ -53722,11 +53736,6 @@ You have been logged out. Please try again.`);
             configId,
             value
           });
-          const session = new ACPSessionConnection(this.connection, sessionState.sessionId);
-          await session.update({
-            sessionUpdate: "config_option_update",
-            configOptions: this.createSessionConfigOptions(sessionState)
-          });
         }
       });
       void commandPromise.catch((err) => {
@@ -53895,11 +53904,10 @@ You have been logged out. Please try again.`);
           return cancelledPromptResponse();
         }
         if (approved && !this.promptShouldStop(params.sessionId, activePrompt)) {
-          await this.applyCollaborationModeChange(sessionState, DEFAULT_COLLABORATION_MODE);
-          const session = new ACPSessionConnection(this.connection, sessionState.sessionId);
-          await session.update({
-            sessionUpdate: "config_option_update",
-            configOptions: this.createSessionConfigOptions(sessionState)
+          await this.applySessionConfigOption(sessionState, {
+            sessionId: sessionState.sessionId,
+            configId: COLLABORATION_MODE_CONFIG_ID,
+            value: DEFAULT_COLLABORATION_MODE
           });
           const implementationRequest = {
             sessionId: params.sessionId,

package/package.json

@@ -3,7 +3,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "2.5.1-preview.8",
+  "version": "2.5.1-preview.9",
   "description": "",
   "main": "dist/index.js",
   "bin": {