@posthog/agent 2.3.167 → 2.3.169

package/src/adapters/codex/codex-agent.ts

@@ -0,0 +1,355 @@
+/**
+ * In-process ACP proxy agent for Codex.
+ *
+ * Implements the ACP Agent interface and delegates to the codex-acp binary
+ * via a ClientSideConnection. This gives us interception points for:
+ * - PostHog-specific notifications (sdk_session, usage_update, turn_complete)
+ * - Session resume/fork (not natively supported by codex-acp)
+ * - Usage accumulation
+ * - System prompt injection
+ */
+
+import {
+  type AgentSideConnection,
+  type AuthenticateRequest,
+  type CancelNotification,
+  ClientSideConnection,
+  type ForkSessionRequest,
+  type ForkSessionResponse,
+  type InitializeRequest,
+  type InitializeResponse,
+  type ListSessionsRequest,
+  type ListSessionsResponse,
+  type LoadSessionRequest,
+  type LoadSessionResponse,
+  type NewSessionRequest,
+  type NewSessionResponse,
+  ndJsonStream,
+  type PromptRequest,
+  type PromptResponse,
+  type ResumeSessionRequest,
+  type ResumeSessionResponse,
+  type SetSessionConfigOptionRequest,
+  type SetSessionConfigOptionResponse,
+  type SetSessionModeRequest,
+  type SetSessionModeResponse,
+} from "@agentclientprotocol/sdk";
+import packageJson from "../../../package.json" with { type: "json" };
+import { POSTHOG_NOTIFICATIONS } from "../../acp-extensions";
+import type { ProcessSpawnedCallback } from "../../types";
+import { Logger } from "../../utils/logger";
+import {
+  nodeReadableToWebReadable,
+  nodeWritableToWebWritable,
+} from "../../utils/streams";
+import { BaseAcpAgent, type BaseSession } from "../base-acp-agent";
+import { createCodexClient } from "./codex-client";
+import {
+  type CodexSessionState,
+  createSessionState,
+  resetUsage,
+} from "./session-state";
+import { CodexSettingsManager } from "./settings";
+import {
+  type CodexProcess,
+  type CodexProcessOptions,
+  spawnCodexProcess,
+} from "./spawn";
+
+interface NewSessionMeta {
+  taskRunId?: string;
+  taskId?: string;
+  systemPrompt?: string;
+  permissionMode?: string;
+  model?: string;
+  persistence?: { taskId?: string; runId?: string; logUrl?: string };
+  claudeCode?: {
+    options?: Record<string, unknown>;
+  };
+  additionalRoots?: string[];
+  disableBuiltInTools?: boolean;
+  allowedDomains?: string[];
+}
+
+export interface CodexAcpAgentOptions {
+  codexProcessOptions: CodexProcessOptions;
+  processCallbacks?: ProcessSpawnedCallback;
+}
+
+type CodexSession = BaseSession & {
+  settingsManager: CodexSettingsManager;
+};
+
+export class CodexAcpAgent extends BaseAcpAgent {
+  readonly adapterName = "codex";
+  declare session: CodexSession;
+  private codexProcess: CodexProcess;
+  private codexConnection!: ClientSideConnection;
+  private sessionState!: CodexSessionState;
+
+  constructor(client: AgentSideConnection, options: CodexAcpAgentOptions) {
+    super(client);
+    this.logger = new Logger({ debug: true, prefix: "[CodexAcpAgent]" });
+
+    // Spawn the codex-acp subprocess
+    this.codexProcess = spawnCodexProcess({
+      ...options.codexProcessOptions,
+      logger: this.logger,
+      processCallbacks: options.processCallbacks,
+    });
+
+    // Create ACP connection to codex-acp over stdin/stdout
+    const codexReadable = nodeReadableToWebReadable(this.codexProcess.stdout);
+    const codexWritable = nodeWritableToWebWritable(this.codexProcess.stdin);
+    const codexStream = ndJsonStream(codexWritable, codexReadable);
+
+    // Set up session with CodexSettingsManager
+    const cwd = options.codexProcessOptions.cwd ?? process.cwd();
+    const settingsManager = new CodexSettingsManager(cwd);
+    const abortController = new AbortController();
+    this.session = {
+      abortController,
+      settingsManager,
+      notificationHistory: [],
+      cancelled: false,
+    };
+
+    // Create the ClientSideConnection to codex-acp.
+    // The Client handler delegates all requests from codex-acp to the upstream
+    // PostHog Code client via our AgentSideConnection.
+    this.codexConnection = new ClientSideConnection(
+      (_agent) =>
+        createCodexClient(
+          this.client,
+          this.logger,
+          this.sessionState ?? {
+            sessionId: "",
+            cwd: "",
+            modeId: "default",
+            configOptions: [],
+            accumulatedUsage: {
+              inputTokens: 0,
+              outputTokens: 0,
+              cachedReadTokens: 0,
+              cachedWriteTokens: 0,
+            },
+            cancelled: false,
+          },
+        ),
+      codexStream,
+    );
+  }
+
+  async initialize(request: InitializeRequest): Promise<InitializeResponse> {
+    // Initialize settings
+    await this.session.settingsManager.initialize();
+
+    // Forward to codex-acp
+    const response = await this.codexConnection.initialize(request);
+
+    // Merge our enhanced capabilities
+    return {
+      ...response,
+      agentCapabilities: {
+        ...response.agentCapabilities,
+        sessionCapabilities: {
+          ...response.agentCapabilities?.sessionCapabilities,
+          resume: {},
+          fork: {},
+        },
+        _meta: {
+          posthog: {
+            resumeSession: true,
+          },
+        },
+      },
+      agentInfo: {
+        name: packageJson.name,
+        title: "Codex Agent",
+        version: packageJson.version,
+      },
+    };
+  }
+
+  async newSession(params: NewSessionRequest): Promise<NewSessionResponse> {
+    const meta = params._meta as NewSessionMeta | undefined;
+
+    const response = await this.codexConnection.newSession(params);
+
+    // Initialize session state
+    this.sessionState = createSessionState(response.sessionId, params.cwd, {
+      taskRunId: meta?.taskRunId,
+      taskId: meta?.taskId ?? meta?.persistence?.taskId,
+      modeId: response.modes?.currentModeId ?? "default",
+      modelId: response.models?.currentModelId,
+    });
+    this.sessionId = response.sessionId;
+    this.sessionState.configOptions = response.configOptions ?? [];
+
+    // Emit _posthog/sdk_session so the app can track the session
+    if (meta?.taskRunId) {
+      await this.client.extNotification(POSTHOG_NOTIFICATIONS.SDK_SESSION, {
+        taskRunId: meta.taskRunId,
+        sessionId: response.sessionId,
+        adapter: "codex",
+      });
+    }
+
+    this.logger.info("Codex session created", {
+      sessionId: response.sessionId,
+      taskRunId: meta?.taskRunId,
+    });
+
+    return response;
+  }
+
+  async loadSession(params: LoadSessionRequest): Promise<LoadSessionResponse> {
+    const response = await this.codexConnection.loadSession(params);
+
+    // Update session state
+    this.sessionState = createSessionState(params.sessionId, params.cwd);
+    this.sessionId = params.sessionId;
+    this.sessionState.configOptions = response.configOptions ?? [];
+
+    return response;
+  }
+
+  async unstable_resumeSession(
+    params: ResumeSessionRequest,
+  ): Promise<ResumeSessionResponse> {
+    // codex-acp doesn't support resume natively, use loadSession instead
+    const loadResponse = await this.codexConnection.loadSession({
+      sessionId: params.sessionId,
+      cwd: params.cwd,
+      mcpServers: params.mcpServers ?? [],
+    });
+
+    this.sessionState = createSessionState(params.sessionId, params.cwd);
+    this.sessionId = params.sessionId;
+    this.sessionState.configOptions = loadResponse.configOptions ?? [];
+
+    const meta = params._meta as NewSessionMeta | undefined;
+    if (meta?.taskRunId) {
+      await this.client.extNotification(POSTHOG_NOTIFICATIONS.SDK_SESSION, {
+        taskRunId: meta.taskRunId,
+        sessionId: params.sessionId,
+        adapter: "codex",
+      });
+    }
+
+    return {
+      modes: loadResponse.modes,
+      models: loadResponse.models,
+      configOptions: loadResponse.configOptions,
+    };
+  }
+
+  async unstable_forkSession(
+    params: ForkSessionRequest,
+  ): Promise<ForkSessionResponse> {
+    // Create a new session via codex-acp (fork isn't natively supported)
+    const newResponse = await this.codexConnection.newSession({
+      cwd: params.cwd,
+      mcpServers: params.mcpServers ?? [],
+      _meta: params._meta,
+    });
+
+    this.sessionState = createSessionState(newResponse.sessionId, params.cwd);
+    this.sessionId = newResponse.sessionId;
+    this.sessionState.configOptions = newResponse.configOptions ?? [];
+
+    return newResponse;
+  }
+
+  async listSessions(
+    params: ListSessionsRequest,
+  ): Promise<ListSessionsResponse> {
+    return this.codexConnection.listSessions(params);
+  }
+
+  async unstable_listSessions(
+    params: ListSessionsRequest,
+  ): Promise<ListSessionsResponse> {
+    return this.codexConnection.listSessions(params);
+  }
+
+  async prompt(params: PromptRequest): Promise<PromptResponse> {
+    if (this.sessionState) {
+      this.sessionState.cancelled = false;
+      this.sessionState.interruptReason = undefined;
+      resetUsage(this.sessionState);
+    }
+
+    const response = await this.codexConnection.prompt(params);
+
+    // Emit PostHog usage notification
+    if (this.sessionState?.taskRunId && response.usage) {
+      await this.client.extNotification("_posthog/usage_update", {
+        sessionId: params.sessionId,
+        used: {
+          inputTokens: response.usage.inputTokens ?? 0,
+          outputTokens: response.usage.outputTokens ?? 0,
+          cachedReadTokens: response.usage.cachedReadTokens ?? 0,
+          cachedWriteTokens: response.usage.cachedWriteTokens ?? 0,
+        },
+        cost: null,
+      });
+    }
+
+    return response;
+  }
+
+  protected async interrupt(): Promise<void> {
+    if (this.sessionState) {
+      this.sessionState.cancelled = true;
+    }
+    await this.codexConnection.cancel({
+      sessionId: this.sessionId,
+    });
+  }
+
+  async cancel(params: CancelNotification): Promise<void> {
+    if (this.sessionState) {
+      this.sessionState.cancelled = true;
+      const meta = params._meta as { interruptReason?: string } | undefined;
+      if (meta?.interruptReason) {
+        this.sessionState.interruptReason = meta.interruptReason;
+      }
+    }
+    await this.codexConnection.cancel(params);
+  }
+
+  async setSessionMode(
+    params: SetSessionModeRequest,
+  ): Promise<SetSessionModeResponse> {
+    const response = await this.codexConnection.setSessionMode(params);
+    if (this.sessionState) {
+      this.sessionState.modeId = params.modeId;
+    }
+    return response ?? {};
+  }
+
+  async setSessionConfigOption(
+    params: SetSessionConfigOptionRequest,
+  ): Promise<SetSessionConfigOptionResponse> {
+    const response = await this.codexConnection.setSessionConfigOption(params);
+    if (this.sessionState && response.configOptions) {
+      this.sessionState.configOptions = response.configOptions;
+    }
+    return response;
+  }
+
+  async authenticate(_params: AuthenticateRequest): Promise<void> {
+    // Auth handled externally
+  }
+
+  async closeSession(): Promise<void> {
+    this.logger.info("Closing Codex session", { sessionId: this.sessionId });
+    this.session.settingsManager.dispose();
+    try {
+      this.codexProcess.kill();
+    } catch (err) {
+      this.logger.warn("Failed to kill codex-acp process", { error: err });
+    }
+  }
+}

package/src/adapters/codex/codex-client.ts

@@ -0,0 +1,151 @@
+/**
+ * ACP Client implementation for communicating with codex-acp subprocess.
+ *
+ * This acts as the "client" from codex-acp's perspective: it receives
+ * permission requests, session updates, file I/O, and terminal operations
+ * from codex-acp and delegates them to the upstream PostHog Code client.
+ */
+
+import type {
+  AgentSideConnection,
+  Client,
+  CreateTerminalRequest,
+  CreateTerminalResponse,
+  KillTerminalRequest,
+  KillTerminalResponse,
+  ReadTextFileRequest,
+  ReadTextFileResponse,
+  ReleaseTerminalRequest,
+  ReleaseTerminalResponse,
+  RequestPermissionRequest,
+  RequestPermissionResponse,
+  SessionNotification,
+  TerminalHandle,
+  TerminalOutputRequest,
+  TerminalOutputResponse,
+  WaitForTerminalExitRequest,
+  WaitForTerminalExitResponse,
+  WriteTextFileRequest,
+  WriteTextFileResponse,
+} from "@agentclientprotocol/sdk";
+import type { Logger } from "../../utils/logger";
+import type { CodexSessionState } from "./session-state";
+
+export interface CodexClientCallbacks {
+  /** Called when a usage_update session notification is received */
+  onUsageUpdate?: (update: Record<string, unknown>) => void;
+}
+
+/**
+ * Creates an ACP Client that delegates all requests from codex-acp
+ * to the upstream PostHog Code client (via AgentSideConnection).
+ */
+export function createCodexClient(
+  upstreamClient: AgentSideConnection,
+  logger: Logger,
+  sessionState: CodexSessionState,
+  callbacks?: CodexClientCallbacks,
+): Client {
+  // Track terminal handles for delegation
+  const terminalHandles = new Map<string, TerminalHandle>();
+
+  return {
+    async requestPermission(
+      params: RequestPermissionRequest,
+    ): Promise<RequestPermissionResponse> {
+      logger.debug("Relaying permission request to upstream", {
+        sessionId: params.sessionId,
+      });
+      return upstreamClient.requestPermission(params);
+    },
+
+    async sessionUpdate(params: SessionNotification): Promise<void> {
+      // Parse usage data from session updates
+      const update = params.update as Record<string, unknown> | undefined;
+      if (update?.sessionUpdate === "usage_update") {
+        const used = update.used as number | undefined;
+        const size = update.size as number | undefined;
+        if (used !== undefined) sessionState.contextUsed = used;
+        if (size !== undefined) sessionState.contextSize = size;
+        callbacks?.onUsageUpdate?.(update);
+      }
+
+      // Forward to upstream client
+      await upstreamClient.sessionUpdate(params);
+    },
+
+    async readTextFile(
+      params: ReadTextFileRequest,
+    ): Promise<ReadTextFileResponse> {
+      return upstreamClient.readTextFile(params);
+    },
+
+    async writeTextFile(
+      params: WriteTextFileRequest,
+    ): Promise<WriteTextFileResponse> {
+      return upstreamClient.writeTextFile(params);
+    },
+
+    async createTerminal(
+      params: CreateTerminalRequest,
+    ): Promise<CreateTerminalResponse> {
+      const handle = await upstreamClient.createTerminal(params);
+      terminalHandles.set(handle.id, handle);
+      return { terminalId: handle.id };
+    },
+
+    async terminalOutput(
+      params: TerminalOutputRequest,
+    ): Promise<TerminalOutputResponse> {
+      const handle = terminalHandles.get(params.terminalId);
+      if (!handle) {
+        return { output: "", truncated: false };
+      }
+      return handle.currentOutput();
+    },
+
+    async releaseTerminal(
+      params: ReleaseTerminalRequest,
+    ): Promise<ReleaseTerminalResponse | undefined> {
+      const handle = terminalHandles.get(params.terminalId);
+      if (handle) {
+        terminalHandles.delete(params.terminalId);
+        const result = await handle.release();
+        return result ?? undefined;
+      }
+    },
+
+    async waitForTerminalExit(
+      params: WaitForTerminalExitRequest,
+    ): Promise<WaitForTerminalExitResponse> {
+      const handle = terminalHandles.get(params.terminalId);
+      if (!handle) {
+        return { exitCode: 1 };
+      }
+      return handle.waitForExit();
+    },
+
+    async killTerminal(
+      params: KillTerminalRequest,
+    ): Promise<KillTerminalResponse | undefined> {
+      const handle = terminalHandles.get(params.terminalId);
+      if (handle) {
+        return handle.kill();
+      }
+    },
+
+    async extMethod(
+      method: string,
+      params: Record<string, unknown>,
+    ): Promise<Record<string, unknown>> {
+      return upstreamClient.extMethod(method, params);
+    },
+
+    async extNotification(
+      method: string,
+      params: Record<string, unknown>,
+    ): Promise<void> {
+      return upstreamClient.extNotification(method, params);
+    },
+  };
+}

package/src/adapters/codex/session-state.ts

@@ -0,0 +1,65 @@
+/**
+ * Session state tracking for Codex proxy agent.
+ * Tracks usage accumulation, model/mode state, and config options.
+ */
+
+import type { SessionConfigOption } from "@agentclientprotocol/sdk";
+
+export interface CodexUsage {
+  inputTokens: number;
+  outputTokens: number;
+  cachedReadTokens: number;
+  cachedWriteTokens: number;
+}
+
+export interface CodexSessionState {
+  sessionId: string;
+  cwd: string;
+  modelId?: string;
+  modeId: string;
+  configOptions: SessionConfigOption[];
+  accumulatedUsage: CodexUsage;
+  contextSize?: number;
+  contextUsed?: number;
+  cancelled: boolean;
+  interruptReason?: string;
+  taskRunId?: string;
+  taskId?: string;
+}
+
+export function createSessionState(
+  sessionId: string,
+  cwd: string,
+  opts?: {
+    taskRunId?: string;
+    taskId?: string;
+    modeId?: string;
+    modelId?: string;
+  },
+): CodexSessionState {
+  return {
+    sessionId,
+    cwd,
+    modeId: opts?.modeId ?? "default",
+    modelId: opts?.modelId,
+    configOptions: [],
+    accumulatedUsage: {
+      inputTokens: 0,
+      outputTokens: 0,
+      cachedReadTokens: 0,
+      cachedWriteTokens: 0,
+    },
+    cancelled: false,
+    taskRunId: opts?.taskRunId,
+    taskId: opts?.taskId,
+  };
+}
+
+export function resetUsage(state: CodexSessionState): void {
+  state.accumulatedUsage = {
+    inputTokens: 0,
+    outputTokens: 0,
+    cachedReadTokens: 0,
+    cachedWriteTokens: 0,
+  };
+}

package/src/adapters/codex/settings.ts

@@ -0,0 +1,127 @@
+import * as fs from "node:fs";
+import * as os from "node:os";
+import * as path from "node:path";
+
+/**
+ * Codex settings parsed from ~/.codex/config.toml and project-level config.
+ *
+ * Mirrors the shape of ClaudeCodeSettings so both adapters have a
+ * consistent settings interface.
+ */
+export interface CodexSettings {
+  model?: string;
+  personality?: string;
+  modelReasoningEffort?: string;
+  trustLevel?: string;
+}
+
+/**
+ * SettingsManager for Codex sessions.
+ *
+ * Reads from ~/.codex/config.toml (user-level) and respects
+ * per-project trust configuration. Has the same public interface
+ * as Claude's SettingsManager so both can satisfy BaseSession.
+ */
+export class CodexSettingsManager {
+  private cwd: string;
+  private settings: CodexSettings = {};
+  private initialized = false;
+
+  constructor(cwd: string) {
+    this.cwd = cwd;
+  }
+
+  async initialize(): Promise<void> {
+    if (this.initialized) {
+      return;
+    }
+    await this.loadSettings();
+    this.initialized = true;
+  }
+
+  private getConfigPath(): string {
+    return path.join(os.homedir(), ".codex", "config.toml");
+  }
+
+  private async loadSettings(): Promise<void> {
+    const configPath = this.getConfigPath();
+    try {
+      const content = await fs.promises.readFile(configPath, "utf-8");
+      this.settings = parseCodexToml(content, this.cwd);
+    } catch {
+      this.settings = {};
+    }
+  }
+
+  getSettings(): CodexSettings {
+    return this.settings;
+  }
+
+  getCwd(): string {
+    return this.cwd;
+  }
+
+  async setCwd(cwd: string): Promise<void> {
+    if (this.cwd === cwd) {
+      return;
+    }
+    this.dispose();
+    this.cwd = cwd;
+    this.initialized = false;
+    await this.initialize();
+  }
+
+  dispose(): void {
+    this.initialized = false;
+  }
+}
+
+/**
+ * Minimal TOML parser for codex config.toml.
+ * Handles flat key=value pairs and [projects."path"] sections.
+ * Does NOT handle full TOML spec — only what codex config uses.
+ */
+function parseCodexToml(content: string, cwd: string): CodexSettings {
+  const settings: CodexSettings = {};
+  let currentSection = "";
+
+  for (const line of content.split("\n")) {
+    const trimmed = line.trim();
+    if (!trimmed || trimmed.startsWith("#")) continue;
+
+    // Section header: [projects."/some/path"] or [section]
+    const sectionMatch = trimmed.match(/^\[(.+)\]$/);
+    if (sectionMatch) {
+      currentSection = sectionMatch[1] ?? "";
+      continue;
+    }
+
+    // Key = value
+    const kvMatch = trimmed.match(/^(\w+)\s*=\s*(.+)$/);
+    if (!kvMatch) continue;
+
+    const key = kvMatch[1];
+    let value = kvMatch[2]?.trim() ?? "";
+
+    // Strip quotes
+    if (
+      (value.startsWith('"') && value.endsWith('"')) ||
+      (value.startsWith("'") && value.endsWith("'"))
+    ) {
+      value = value.slice(1, -1);
+    }
+
+    if (!currentSection) {
+      // Top-level keys
+      if (key === "model") settings.model = value;
+      if (key === "personality") settings.personality = value;
+      if (key === "model_reasoning_effort")
+        settings.modelReasoningEffort = value;
+    } else if (currentSection === `projects."${cwd}"`) {
+      // Project-specific keys
+      if (key === "trust_level") settings.trustLevel = value;
+    }
+  }
+
+  return settings;
+}

package/src/agent.ts

@@ -4,6 +4,7 @@ import {
 } from "./adapters/acp-connection";
 import {
   BLOCKED_MODELS,
+  DEFAULT_CODEX_MODEL,
   DEFAULT_GATEWAY_MODEL,
   fetchModelsList,
 } from "./gateway-models";
@@ -104,7 +105,9 @@ export class Agent {
       }
 
       if (!sanitizedModel || !allowedModelIds?.has(sanitizedModel)) {
-        sanitizedModel = codexModelIds[0];
+        sanitizedModel = codexModelIds.includes(DEFAULT_CODEX_MODEL)
+          ? DEFAULT_CODEX_MODEL
+          : codexModelIds[0];
       }
     }
     if (!sanitizedModel && options.adapter !== "codex") {