@leo000001/codex-mcp 0.2.0 → 0.2.1

package/README.md

@@ -57,18 +57,23 @@ Add to your MCP client config (e.g. Claude Desktop, Cursor, etc.):
 }
 ```
 
-### OpenAI Codex CLI
+### Claude Code
 
 ```bash
-codex mcp add codex-mcp -- npx -y @leo000001/codex-mcp
+claude mcp add codex-mcp -- npx -y @leo000001/codex-mcp
 ```
 
-Or add to `~/.codex/config.toml`:
+Or add to `~/.claude/settings.json`:
 
-```toml
-[mcp_servers.codex-mcp]
-command = "npx"
-args = ["-y", "@leo000001/codex-mcp"]
+```json
+{
+  "mcpServers": {
+    "codex-mcp": {
+      "command": "npx",
+      "args": ["-y", "@leo000001/codex-mcp"]
+    }
+  }
+}
 ```
 
 ## STDIO Guard Modes
@@ -95,16 +100,16 @@ $env:CODEX_MCP_STDIO_MODE = "strict"; npx -y @leo000001/codex-mcp
 
 Start a Codex agent session asynchronously. Returns immediately with `sessionId`.
 
-| Parameter        | Type   | Required | Description                                                                                                            |
-| ---------------- | ------ | -------- | ---------------------------------------------------------------------------------------------------------------------- |
-| `prompt`         | string | Yes      | Task or question for the Codex agent                                                                                   |
-| `approvalPolicy` | string | Yes      | Approval policy: `untrusted`, `on-failure`, `on-request`, `never` — caller must set based on its own permission level  |
-| `sandbox`        | string | Yes      | Sandbox mode: `read-only`, `workspace-write`, `danger-full-access` — caller must set based on its own permission level |
+| Parameter        | Type   | Required | Description                                                                                                                       |
+| ---------------- | ------ | -------- | --------------------------------------------------------------------------------------------------------------------------------- |
+| `prompt`         | string | Yes      | Task or question for the Codex agent                                                                                              |
+| `approvalPolicy` | string | Yes      | Approval policy: `untrusted`, `on-failure`, `on-request`, `never` — caller must set based on its own permission level             |
+| `sandbox`        | string | Yes      | Sandbox mode: `read-only`, `workspace-write`, `danger-full-access` — caller must set based on its own permission level            |
 | `effort`         | string | No       | Reasoning effort: `none`, `minimal`, `low`, `medium`, `high`, `xhigh`. Default: `low`; increase/decrease based on task complexity |
-| `cwd`            | string | No       | Working directory. Default: server cwd                                                                                 |
-| `model`          | string | No       | Model override. Default: from `~/.codex/config.toml`                                                                   |
-| `profile`        | string | No       | `config.toml` profile name (passed as `codex app-server -p`)                                                           |
-| `advanced`       | object | No       | Low-frequency options (see below)                                                                                      |
+| `cwd`            | string | No       | Working directory. Default: server cwd                                                                                            |
+| `model`          | string | No       | Model override. Default: from `~/.codex/config.toml`                                                                              |
+| `profile`        | string | No       | `config.toml` profile name (passed as `codex app-server -p`)                                                                      |
+| `advanced`       | object | No       | Low-frequency options (see below)                                                                                                 |
 
 <details>
 <summary><code>advanced</code> object parameters (9 low-frequency parameters)</summary>
@@ -203,12 +208,12 @@ Query a running session for events, respond to approval requests, or answer user
 
 | Parameter             | Type     | Required                          | Description                                                                                                                                                                                      |
 | --------------------- | -------- | --------------------------------- | ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------ |
-| `action`              | string   | Yes                               | `"poll"`, `"respond_permission"`, `"respond_approval"` (deprecated alias), or `"respond_user_input"`                                                                                            |
+| `action`              | string   | Yes                               | `"poll"`, `"respond_permission"`, or `"respond_user_input"`                                                                                                                                    |
 | `sessionId`           | string   | Yes                               | Target session ID                                                                                                                                                                                |
-| `cursor`              | number   | No                                | Event cursor for incremental polling (`action="poll"`). For `respond_*`, codex-mcp applies monotonic cursor progression: `max(cursor, sessionLastCursor)`.                                   |
-| `maxEvents`           | number   | No                                | Keep this small. `poll` default: `1` (minimum `1`; increase only for catch-up). `respond_*` default: `0` (recommended; compact ACK, no event replay).                                           |
-| `responseMode`        | string   | No                                | Response shaping mode: `minimal` (default), `delta_compact`, `full`                                                                                                                             |
-| `pollOptions`         | object   | No                                | Optional controls: `includeEvents` (default `true`), `includeActions` (default `true`), `includeResult` (default `true`), `maxBytes` (default unlimited)                                       |
+| `cursor`              | number   | No                                | Event cursor for incremental polling (`action="poll"`). For `respond_*`, codex-mcp applies monotonic cursor progression: `max(cursor, sessionLastCursor)`.                                       |
+| `maxEvents`           | number   | No                                | Keep this small. `poll` default: `1` (minimum `1`; increase only for catch-up). `respond_*` default: `0` (recommended; compact ACK, no event replay).                                            |
+| `responseMode`        | string   | No                                | Response shaping mode: `minimal` (default), `delta_compact`, `full`                                                                                                                              |
+| `pollOptions`         | object   | No                                | Optional controls: `includeEvents` (default `true`), `includeActions` (default `true`), `includeResult` (default `true`), `maxBytes` (default unlimited)                                         |
 | `requestId`           | string   | For respond_permission/user_input | Request ID from `actions[]`                                                                                                                                                                      |
 | `decision`            | string   | For respond_permission            | For command approvals: `"accept"`, `"acceptForSession"`, `"acceptWithExecpolicyAmendment"`, `"decline"`, `"cancel"`; for file changes: `"accept"`, `"acceptForSession"`, `"decline"`, `"cancel"` |
 | `execpolicyAmendment` | string[] | For acceptWithExecpolicyAmendment | Exec policy amendment list (required when `decision="acceptWithExecpolicyAmendment"`)                                                                                                            |
@@ -260,7 +265,6 @@ When the agent requests approval or user input, `poll` includes an `actions[]` l
 
 - `respond_permission`: `decision` is one of `accept`, `acceptForSession`, `decline`, `cancel`.
   - For command approvals, `acceptWithExecpolicyAmendment` is supported and requires `execpolicyAmendment`.
-- `respond_approval` is still accepted as a deprecated alias for `respond_permission`.
 - `respond_user_input`: send `answers` keyed by `questionId`.
 
 Pending approvals auto-decline after `advanced.approvalTimeoutMs`.

package/dist/index.js

@@ -165,12 +165,7 @@ var PERSONALITIES = ["none", "friendly", "pragmatic"];
 var EFFORT_LEVELS = ["none", "minimal", "low", "medium", "high", "xhigh"];
 var SUMMARY_MODES = ["auto", "concise", "detailed", "none"];
 var SESSION_ACTIONS = ["list", "get", "cancel", "interrupt", "fork"];
-var CHECK_ACTIONS = [
-  "poll",
-  "respond_permission",
-  "respond_approval",
-  "respond_user_input"
-];
+var CHECK_ACTIONS = ["poll", "respond_permission", "respond_user_input"];
 var RESPONSE_MODES = ["minimal", "delta_compact", "full"];
 var COMMAND_DECISIONS = [
   "accept",
@@ -218,7 +213,7 @@ var DEFAULT_TERMINAL_CLEANUP_MS = 5 * 60 * 1e3;
 var CLEANUP_INTERVAL_MS = 6e4;
 
 // src/app-server/client.ts
-var CLIENT_VERSION = true ? "0.2.0" : "0.0.0-dev";
+var CLIENT_VERSION = true ? "0.2.1" : "0.0.0-dev";
 var DEFAULT_REQUEST_TIMEOUT = 3e4;
 var STARTUP_REQUEST_TIMEOUT = 9e4;
 var MAX_WRITE_QUEUE_BYTES = 5 * 1024 * 1024;
@@ -1084,13 +1079,6 @@ var SessionManager = class {
       actions: actions.length > 0 ? actions : void 0,
       result: includeResult && (session.status === "idle" || session.status === "error" || session.status === "cancelled") ? session.lastResult : void 0
     };
-    if (pollOptions?.includeTools === true) {
-      addCompatWarningWithinBudget(
-        result,
-        "pollOptions.includeTools is not yet supported by codex-mcp; returning no tool metadata.",
-        maxBytes
-      );
-    }
     if (typeof maxBytes === "number") {
       const normalizedMaxBytes = Math.max(1, Math.floor(maxBytes));
       const hasAnyPayload = result.events.length > 0 || typeof result.actions !== "undefined" || typeof result.result !== "undefined";
@@ -2126,11 +2114,10 @@ function executeCodexCheck(args, sessionManager) {
         pollOptions
       });
     }
-    case "respond_permission":
-    case "respond_approval": {
+    case "respond_permission": {
       if (!args.requestId || !args.decision) {
         return {
-          error: `Error [${"INVALID_ARGUMENT" /* INVALID_ARGUMENT */}]: requestId and decision required for respond_permission/respond_approval`,
+          error: `Error [${"INVALID_ARGUMENT" /* INVALID_ARGUMENT */}]: requestId and decision required for respond_permission`,
           isError: true
         };
       }
@@ -2144,18 +2131,10 @@ function executeCodexCheck(args, sessionManager) {
         return { error: message, isError: true };
       }
       const maxEvents = args.maxEvents ?? RESPOND_DEFAULT_MAX_EVENTS;
-      const result = sessionManager.pollEventsMonotonic(args.sessionId, args.cursor, maxEvents, {
+      return sessionManager.pollEventsMonotonic(args.sessionId, args.cursor, maxEvents, {
         responseMode,
         pollOptions
       });
-      if (args.action === "respond_approval") {
-        return addWarning(
-          result,
-          "Action 'respond_approval' is deprecated, use 'respond_permission'.",
-          pollOptions?.maxBytes
-        );
-      }
-      return result;
     }
     case "respond_user_input": {
       if (!args.requestId || !args.answers) {
@@ -2183,23 +2162,6 @@ function executeCodexCheck(args, sessionManager) {
       };
   }
 }
-function addWarning(result, warning, maxBytes) {
-  if (!result.compatWarnings) result.compatWarnings = [];
-  result.compatWarnings.push(warning);
-  if (typeof maxBytes !== "number") {
-    return result;
-  }
-  const normalizedMaxBytes = Math.max(1, Math.floor(maxBytes));
-  if (Buffer.byteLength(JSON.stringify(result), "utf8") <= normalizedMaxBytes) {
-    return result;
-  }
-  if (result.compatWarnings.length > 1) {
-    result.compatWarnings.pop();
-  } else {
-    result.compatWarnings = void 0;
-  }
-  return result;
-}
 
 // src/resources/register-resources.ts
 import { spawnSync } from "child_process";
@@ -2465,7 +2427,6 @@ function buildGotchasText() {
     `- Poll default is \`maxEvents=${POLL_DEFAULT_MAX_EVENTS}\` (authoritative: tool schema / constants).`,
     `- Poll enforces minimum \`maxEvents=${POLL_MIN_MAX_EVENTS}\`; sending \`0\` is normalized to \`${POLL_MIN_MAX_EVENTS}\`.`,
     `- \`respond_permission\` and \`respond_user_input\` default to compact ACK with \`maxEvents=${RESPOND_DEFAULT_MAX_EVENTS}\`.`,
-    "- `respond_approval` is a deprecated alias for `respond_permission`.",
     "- Default response mode is `minimal`; use `full` if you need full raw event payloads.",
     "- respond_* uses monotonic cursor handling: `max(cursor, sessionLastCursor)`.",
     "- If `cursorResetTo` is present, your cursor is stale (old events were evicted); restart from that value.",
@@ -2771,7 +2732,7 @@ function registerResources(server, deps) {
 }
 
 // src/server.ts
-var SERVER_VERSION = true ? "0.2.0" : "0.0.0-dev";
+var SERVER_VERSION = true ? "0.2.1" : "0.0.0-dev";
 function formatErrorMessage(err) {
   const message = err instanceof Error ? err.message : String(err);
   const m = /^Error \[([A-Z_]+)\]:\s*(.*)$/.exec(message);
@@ -3000,8 +2961,6 @@ poll: events since cursor. Default maxEvents=${POLL_DEFAULT_MAX_EVENTS}.
 
 respond_permission: approval decision. Default maxEvents=${RESPOND_DEFAULT_MAX_EVENTS} (compact ACK).
 
-respond_approval: deprecated alias for respond_permission.
-
 respond_user_input: user-input answers. Default maxEvents=${RESPOND_DEFAULT_MAX_EVENTS} (compact ACK).
 
 events[].type is coarse-grained; details are in events[].data.method.
@@ -3018,13 +2977,12 @@ cursor omitted => use session last cursor. cursorResetTo => reset and continue.`
           includeEvents: z.boolean().optional().describe("Default: true. Include events[] in response."),
           includeActions: z.boolean().optional().describe("Default: true. Include actions[] in response."),
           includeResult: z.boolean().optional().describe("Default: true. Include result in response."),
-          maxBytes: z.number().int().positive().optional().describe("Default: unlimited. Best-effort response payload cap in bytes."),
-          includeTools: z.boolean().optional().describe("Default: false. Reserved for future dynamic tool metadata support.")
+          maxBytes: z.number().int().positive().optional().describe("Default: unlimited. Best-effort response payload cap in bytes.")
         }).optional().describe("Optional poll shaping controls."),
-        // respond_permission/respond_approval
+        // respond_permission
         requestId: z.string().optional().describe("Request ID from actions[]"),
         decision: z.enum(ALL_DECISIONS).optional().describe(
-          "Approval decision for respond_permission/respond_approval. acceptWithExecpolicyAmendment requires execpolicyAmendment."
+          "Approval decision for respond_permission. acceptWithExecpolicyAmendment requires execpolicyAmendment."
         ),
         execpolicyAmendment: z.array(z.string()).optional().describe("For acceptWithExecpolicyAmendment only"),
         denyMessage: z.string().optional().describe("Deny reason (not sent to agent)"),