@agentconnect.md/codex-acp 1.7.0-agentconnect.1 → 1.7.0-agentconnect.2

package/README.md

@@ -9,7 +9,7 @@ Use [OpenAI Codex](https://github.com/openai/codex) from [Agent Client Protocol]
 ## Features
 
 - ChatGPT, API key, and client-provided custom gateway authentication.
-- Model, reasoning effort, fast mode, approval reviewer, and approval/sandbox mode configuration.
+- Model, reasoning effort, fast mode, approval, and sandbox mode configuration.
 - Text prompts, embedded context, images, resource links, and additional workspace directories.
 - Shell command, file change, [permission request](docs/permission-extension.md), MCP tool call, terminal output, reasoning, plan, web search, image generation, image view, token usage, and review events.
 - [Native ACP subagent sessions](docs/subagent-sessions.md) (after capability negotiation) with separate child histories and root-routed permissions; a legacy tool-call fallback otherwise.

package/dist/index.js

@@ -27235,7 +27235,7 @@ var package_default = {
   publishConfig: {
     access: "public"
   },
-  version: "1.7.0-agentconnect.1",
+  version: "1.7.0-agentconnect.2",
   description: "",
   main: "dist/index.js",
   bin: {
@@ -27797,41 +27797,6 @@ var CodexSubagentSubscriptions = class {
   }
 };
 
-// src/ApprovalsReviewerConfig.ts
-var APPROVALS_REVIEWER_CONFIG_ID = "approvals_reviewer";
-var USER_APPROVALS_REVIEWER = "user";
-var AUTO_APPROVALS_REVIEWER = "auto_review";
-function createApprovalsReviewerConfigOption(currentValue) {
-  return {
-    id: APPROVALS_REVIEWER_CONFIG_ID,
-    name: "Approval reviewer",
-    description: "Who reviews eligible approval requests",
-    category: "_approvals_reviewer",
-    type: "select",
-    currentValue,
-    options: [
-      {
-        value: USER_APPROVALS_REVIEWER,
-        name: "User",
-        description: "Ask the user to approve or deny requests"
-      },
-      {
-        value: AUTO_APPROVALS_REVIEWER,
-        name: "Auto-review",
-        description: "Use a reviewer agent to assess eligible requests"
-      }
-    ]
-  };
-}
-function parseApprovalsReviewer(value) {
-  if (value === USER_APPROVALS_REVIEWER) return USER_APPROVALS_REVIEWER;
-  if (value === AUTO_APPROVALS_REVIEWER) return AUTO_APPROVALS_REVIEWER;
-  return null;
-}
-function normalizeApprovalsReviewer(value) {
-  return value === AUTO_APPROVALS_REVIEWER || value === "guardian_subagent" ? AUTO_APPROVALS_REVIEWER : USER_APPROVALS_REVIEWER;
-}
-
 // src/PermissionProfileConfig.ts
 var PERMISSION_PROFILE_CONFIG_ENV = "CODEX_ACP_PERMISSION_PROFILE_CONFIG";
 var permissionProfileConfigSchema = external_exports.object({
@@ -28200,7 +28165,6 @@ var CodexAcpClient = class {
       currentModelId,
       models: codexModels,
       collaborationMode: this.getCollaborationMode(response.thread.id),
-      approvalsReviewer: normalizeApprovalsReviewer(response.approvalsReviewer),
       modelProvider: response.modelProvider,
       currentServiceTier: response.serviceTier ?? null,
       additionalDirectories
@@ -28229,7 +28193,6 @@ var CodexAcpClient = class {
       currentModelId,
       models: codexModels,
       collaborationMode: this.getCollaborationMode(response.thread.id),
-      approvalsReviewer: normalizeApprovalsReviewer(response.approvalsReviewer),
       modelProvider: response.modelProvider,
       currentServiceTier: response.serviceTier ?? null,
       thread: historyResponse.thread,
@@ -28262,7 +28225,6 @@ var CodexAcpClient = class {
       currentModelId,
       models: codexModels,
       collaborationMode: this.getCollaborationMode(response.thread.id),
-      approvalsReviewer: normalizeApprovalsReviewer(response.approvalsReviewer),
       modelProvider: response.modelProvider,
       currentServiceTier: response.serviceTier ?? null,
       additionalDirectories
@@ -28500,7 +28462,7 @@ var CodexAcpClient = class {
       }
     });
   }
-  async sendPrompt(request, agentMode, approvalsReviewer, modelId, serviceTier, disableSummary, cwd, additionalDirectories, onTurnStarted, shouldCancel) {
+  async sendPrompt(request, agentMode, modelId, serviceTier, disableSummary, cwd, additionalDirectories, onTurnStarted, shouldCancel) {
     const input = buildPromptItems(request.prompt);
     const effort = modelId.effort;
     await this.refreshSkills(cwd, additionalDirectories);
@@ -28516,7 +28478,7 @@ var CodexAcpClient = class {
       threadId: request.sessionId,
       input,
       approvalPolicy: agentMode.approvalPolicy,
-      approvalsReviewer,
+      approvalsReviewer: agentMode.approvalsReviewer,
       ...sandboxSelection,
       summary: disableSummary ? "none" : "auto",
       effort,
@@ -31716,7 +31678,6 @@ Check ${configPath} and project .codex directories, especially their config.toml
       supportedReasoningEfforts: currentModel?.supportedReasoningEfforts ?? [],
       supportedInputModalities: currentModel?.inputModalities ?? ["text", "image"],
       agentMode: AgentMode.getInitialAgentMode(),
-      approvalsReviewer: sessionMetadata.approvalsReviewer,
       collaborationMode: sessionMetadata.collaborationMode,
       currentTurnId: null,
       lastTokenUsage: null,
@@ -32105,9 +32066,6 @@ Check ${configPath} and project .codex directories, especially their config.toml
       case MODE_CONFIG_ID:
         await this.applyModeChange(sessionState, this.stringConfigValue(params));
         break;
-      case APPROVALS_REVIEWER_CONFIG_ID:
-        this.applyApprovalsReviewerChange(sessionState, this.stringConfigValue(params));
-        break;
       case COLLABORATION_MODE_CONFIG_ID:
         await this.applyCollaborationModeChange(sessionState, this.stringConfigValue(params));
         break;
@@ -32149,13 +32107,6 @@ Check ${configPath} and project .codex directories, especially their config.toml
     );
     sessionState.agentMode = newMode;
   }
-  applyApprovalsReviewerChange(sessionState, value) {
-    const approvalsReviewer = parseApprovalsReviewer(value);
-    if (approvalsReviewer === null) {
-      throw RequestError.invalidParams();
-    }
-    sessionState.approvalsReviewer = approvalsReviewer;
-  }
   async applyCollaborationModeChange(sessionState, value) {
     const mode = parseCollaborationMode(value);
     if (mode === null) {
@@ -32443,7 +32394,6 @@ Check ${configPath} and project .codex directories, especially their config.toml
     const currentModelId = ModelId.fromString(sessionState.currentModelId);
     const configOptions = [
       sessionState.agentMode.toConfigOption(),
-      createApprovalsReviewerConfigOption(sessionState.approvalsReviewer),
       createCollaborationModeConfigOption(sessionState.collaborationMode),
       createModelConfigOption(sessionState.availableModels, currentModelId.model)
     ];
@@ -32583,7 +32533,6 @@ Check ${configPath} and project .codex directories, especially their config.toml
       supportedReasoningEfforts: currentModel?.supportedReasoningEfforts ?? [],
       supportedInputModalities: currentModel?.inputModalities ?? ["text", "image"],
       agentMode: AgentMode.getInitialAgentMode(),
-      approvalsReviewer: sessionMetadata.approvalsReviewer,
       collaborationMode: sessionMetadata.collaborationMode,
       currentTurnId: null,
       lastTokenUsage: null,
@@ -33401,7 +33350,6 @@ Check ${configPath} and project .codex directories, especially their config.toml
         () => this.codexAcpClient.sendPrompt(
           effectiveParams,
           agentMode,
-          sessionState.approvalsReviewer,
           modelId,
           serviceTier,
           disableSummary,
@@ -33492,7 +33440,6 @@ Check ${configPath} and project .codex directories, especially their config.toml
             () => this.codexAcpClient.sendPrompt(
               implementationRequest,
               agentMode,
-              sessionState.approvalsReviewer,
               modelId,
               serviceTier,
               disableSummary,

package/package.json

@@ -3,7 +3,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "1.7.0-agentconnect.1",
+  "version": "1.7.0-agentconnect.2",
   "description": "",
   "main": "dist/index.js",
   "bin": {