@posthog/agent 2.1.45 → 2.1.47

package/dist/{agent-DcBmoTR4.d.ts → agent-BJ7Uacyp.d.ts}

@@ -94,6 +94,8 @@ type AcpConnectionConfig = {
     /** Deployment environment - "local" for desktop, "cloud" for cloud sandbox */
     deviceType?: "local" | "cloud";
     logger?: Logger;
+    /** Enable dev-only instrumentation (timing, verbose logging) */
+    debug?: boolean;
     processCallbacks?: ProcessSpawnedCallback;
     codexOptions?: CodexProcessOptions;
     allowedModelIds?: Set<string>;

package/dist/agent.d.ts

@@ -1,4 +1,4 @@
-export { b as Agent } from './agent-DcBmoTR4.js';
+export { b as Agent } from './agent-BJ7Uacyp.js';
 import './types.js';
 import '@agentclientprotocol/sdk';
 import './logger-DDBiMOOD.js';

package/dist/agent.js

@@ -271,12 +271,49 @@ import {
 import {
   query
 } from "@anthropic-ai/claude-agent-sdk";
+
+// ../shared/dist/index.js
+var NOOP_COLLECTOR = {
+  time: (_label, fn) => fn(),
+  timeSync: (_label, fn) => fn(),
+  record: () => {
+  },
+  summarize: () => {
+  }
+};
+function createTimingCollector(enabled, log) {
+  if (!enabled) return NOOP_COLLECTOR;
+  const steps = {};
+  return {
+    async time(label, fn) {
+      const start = Date.now();
+      const result = await fn();
+      steps[label] = Date.now() - start;
+      return result;
+    },
+    timeSync(label, fn) {
+      const start = Date.now();
+      const result = fn();
+      steps[label] = Date.now() - start;
+      return result;
+    },
+    record(label, ms) {
+      steps[label] = ms;
+    },
+    summarize(label) {
+      const total = Object.values(steps).reduce((a, b) => a + b, 0);
+      log(`[timing] ${label}: ${total}ms`, steps);
+    }
+  };
+}
+
+// src/adapters/claude/claude-agent.ts
 import { v7 as uuidv7 } from "uuid";
 
 // package.json
 var package_default = {
   name: "@posthog/agent",
-  version: "2.1.45",
+  version: "2.1.47",
   repository: "https://github.com/PostHog/twig",
   description: "TypeScript agent framework wrapping Claude Agent SDK with Git-based task execution for PostHog",
   exports: {
@@ -2385,12 +2422,14 @@ var ClaudeAcpAgent = class extends BaseAcpAgent {
   backgroundTerminals = {};
   clientCapabilities;
   logWriter;
-  processCallbacks;
+  options;
   lastSentConfigOptions;
-  constructor(client, logWriter, processCallbacks) {
+  debug;
+  constructor(client, logWriter, options) {
     super(client);
     this.logWriter = logWriter;
-    this.processCallbacks = processCallbacks;
+    this.options = options;
+    this.debug = options?.debug ?? false;
     this.toolUseCache = {};
     this.logger = new Logger({ debug: true, prefix: "[ClaudeAcpAgent]" });
   }
@@ -2433,27 +2472,36 @@ var ClaudeAcpAgent = class extends BaseAcpAgent {
   }
   async newSession(params) {
     this.checkAuthStatus();
+    const tc = createTimingCollector(
+      this.debug,
+      (msg, data) => this.logger.info(msg, data)
+    );
     const meta = params._meta;
     const sessionId = uuidv7();
     const permissionMode = meta?.permissionMode && TWIG_EXECUTION_MODES.includes(meta.permissionMode) ? meta.permissionMode : "default";
-    const mcpServers = parseMcpServers(params);
-    const mcpMetadataPromise = fetchMcpToolMetadata(mcpServers, this.logger);
-    const options = buildSessionOptions({
-      cwd: params.cwd,
-      mcpServers,
-      permissionMode,
-      canUseTool: this.createCanUseTool(sessionId),
-      logger: this.logger,
-      systemPrompt: buildSystemPrompt(meta?.systemPrompt),
-      userProvidedOptions: meta?.claudeCode?.options,
-      sessionId,
-      isResume: false,
-      onModeChange: this.createOnModeChange(sessionId),
-      onProcessSpawned: this.processCallbacks?.onProcessSpawned,
-      onProcessExited: this.processCallbacks?.onProcessExited
-    });
+    const mcpServers = tc.timeSync(
+      "parseMcpServers",
+      () => parseMcpServers(params)
+    );
+    const options = tc.timeSync(
+      "buildSessionOptions",
+      () => buildSessionOptions({
+        cwd: params.cwd,
+        mcpServers,
+        permissionMode,
+        canUseTool: this.createCanUseTool(sessionId),
+        logger: this.logger,
+        systemPrompt: buildSystemPrompt(meta?.systemPrompt),
+        userProvidedOptions: meta?.claudeCode?.options,
+        sessionId,
+        isResume: false,
+        onModeChange: this.createOnModeChange(sessionId),
+        onProcessSpawned: this.options?.onProcessSpawned,
+        onProcessExited: this.options?.onProcessExited
+      })
+    );
     const input = new Pushable();
-    const q = query({ prompt: input, options });
+    const q = tc.timeSync("sdkQuery", () => query({ prompt: input, options }));
     const session = this.createSession(
       sessionId,
       q,
@@ -2465,29 +2513,40 @@ var ClaudeAcpAgent = class extends BaseAcpAgent {
     session.taskRunId = meta?.taskRunId;
     this.registerPersistence(sessionId, meta);
     if (meta?.taskRunId) {
-      await this.client.extNotification("_posthog/sdk_session", {
-        taskRunId: meta.taskRunId,
-        sessionId,
-        adapter: "claude"
-      });
+      await tc.time(
+        "extNotification",
+        () => this.client.extNotification("_posthog/sdk_session", {
+          taskRunId: meta.taskRunId,
+          sessionId,
+          adapter: "claude"
+        })
+      );
     }
-    const [modelOptions, slashCommands] = await Promise.all([
-      this.getModelConfigOptions(),
-      getAvailableSlashCommands(q),
-      mcpMetadataPromise
-    ]);
+    const modelOptions = await tc.time(
+      "fetchModels",
+      () => this.getModelConfigOptions()
+    );
+    this.deferBackgroundFetches(tc, q, sessionId, mcpServers);
     session.modelId = modelOptions.currentModelId;
     await this.trySetModel(q, modelOptions.currentModelId);
-    this.sendAvailableCommandsUpdate(sessionId, slashCommands);
+    const configOptions = await tc.time(
+      "buildConfigOptions",
+      () => this.buildConfigOptions(modelOptions)
+    );
+    tc.summarize("newSession");
     return {
       sessionId,
-      configOptions: await this.buildConfigOptions(modelOptions)
+      configOptions
     };
   }
   async loadSession(params) {
     return this.resumeSession(params);
   }
   async resumeSession(params) {
+    const tc = createTimingCollector(
+      this.debug,
+      (msg, data) => this.logger.info(msg, data)
+    );
     const meta = params._meta;
     const sessionId = meta?.sessionId;
     if (!sessionId) {
@@ -2496,29 +2555,33 @@ var ClaudeAcpAgent = class extends BaseAcpAgent {
     if (this.sessionId === sessionId) {
       return {};
     }
-    const mcpServers = parseMcpServers(params);
-    const mcpMetadataPromise = fetchMcpToolMetadata(mcpServers, this.logger);
+    const mcpServers = tc.timeSync(
+      "parseMcpServers",
+      () => parseMcpServers(params)
+    );
     const permissionMode = meta?.permissionMode && TWIG_EXECUTION_MODES.includes(meta.permissionMode) ? meta.permissionMode : "default";
-    const { query: q, session } = await this.initializeQuery({
-      cwd: params.cwd,
-      permissionMode,
-      mcpServers,
-      systemPrompt: buildSystemPrompt(meta?.systemPrompt),
-      userProvidedOptions: meta?.claudeCode?.options,
-      sessionId,
-      isResume: true,
-      additionalDirectories: meta?.claudeCode?.options?.additionalDirectories
-    });
+    const { query: q, session } = await tc.time(
+      "initializeQuery",
+      () => this.initializeQuery({
+        cwd: params.cwd,
+        permissionMode,
+        mcpServers,
+        systemPrompt: buildSystemPrompt(meta?.systemPrompt),
+        userProvidedOptions: meta?.claudeCode?.options,
+        sessionId,
+        isResume: true,
+        additionalDirectories: meta?.claudeCode?.options?.additionalDirectories
+      })
+    );
     session.taskRunId = meta?.taskRunId;
     this.registerPersistence(sessionId, meta);
-    const [slashCommands] = await Promise.all([
-      getAvailableSlashCommands(q),
-      mcpMetadataPromise
-    ]);
-    this.sendAvailableCommandsUpdate(sessionId, slashCommands);
-    return {
-      configOptions: await this.buildConfigOptions()
-    };
+    this.deferBackgroundFetches(tc, q, sessionId, mcpServers);
+    const configOptions = await tc.time(
+      "buildConfigOptions",
+      () => this.buildConfigOptions()
+    );
+    tc.summarize("resumeSession");
+    return { configOptions };
   }
   async prompt(params) {
     this.session.cancelled = false;
@@ -2590,8 +2653,8 @@ var ClaudeAcpAgent = class extends BaseAcpAgent {
       isResume: config.isResume,
       additionalDirectories: config.additionalDirectories,
       onModeChange: this.createOnModeChange(config.sessionId),
-      onProcessSpawned: this.processCallbacks?.onProcessSpawned,
-      onProcessExited: this.processCallbacks?.onProcessExited
+      onProcessSpawned: this.options?.onProcessSpawned,
+      onProcessExited: this.options?.onProcessExited
     });
     const q = query({ prompt: input, options });
     const abortController = options.abortController;
@@ -2701,6 +2764,23 @@ var ClaudeAcpAgent = class extends BaseAcpAgent {
       await q.setModel(fallback);
     }
   }
+  /**
+   * Fire-and-forget: fetch slash commands and MCP tool metadata in parallel.
+   * Both populate caches used later — neither is needed to return configOptions.
+   */
+  deferBackgroundFetches(tc, q, sessionId, mcpServers) {
+    Promise.all([
+      tc.time("slashCommands", () => getAvailableSlashCommands(q)),
+      tc.time(
+        "mcpMetadata",
+        () => fetchMcpToolMetadata(mcpServers, this.logger)
+      )
+    ]).then(([slashCommands]) => {
+      this.sendAvailableCommandsUpdate(sessionId, slashCommands);
+    }).catch((err) => {
+      this.logger.warn("Failed to fetch deferred session data", { err });
+    });
+  }
   registerPersistence(sessionId, meta) {
     const persistence = meta?.persistence;
     if (persistence && this.logWriter) {
@@ -2988,7 +3068,10 @@ function createClaudeConnection(config) {
   const agentStream = ndJsonStream(agentWritable, streams.agent.readable);
   let agent = null;
   const agentConnection = new AgentSideConnection((client) => {
-    agent = new ClaudeAcpAgent(client, logWriter, config.processCallbacks);
+    agent = new ClaudeAcpAgent(client, logWriter, {
+      ...config.processCallbacks,
+      debug: config.debug
+    });
     logger.info(`Created ${agent.adapterName} agent`);
     return agent;
   }, agentStream);
@@ -3688,6 +3771,7 @@ var Agent = class {
       taskId,
       deviceType: "local",
       logger: this.logger,
+      debug: this.debug,
       processCallbacks: options.processCallbacks,
       allowedModelIds,
       codexOptions: options.adapter === "codex" && gatewayConfig ? {