@posthog/agent 2.1.35 → 2.1.45

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@posthog/agent",
-  "version": "2.1.35",
+  "version": "2.1.45",
   "repository": "https://github.com/PostHog/twig",
   "description": "TypeScript agent framework wrapping Claude Agent SDK with Git-based task execution for PostHog",
   "exports": {
@@ -72,8 +72,8 @@
     "tsx": "^4.20.6",
     "typescript": "^5.5.0",
     "vitest": "^2.1.8",
-    "@posthog/shared": "1.0.0",
-    "@twig/git": "1.0.0"
+    "@twig/git": "1.0.0",
+    "@posthog/shared": "1.0.0"
   },
   "dependencies": {
     "@agentclientprotocol/sdk": "^0.14.0",

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

@@ -145,7 +145,10 @@ export class ClaudeAcpAgent extends BaseAcpAgent {
         : "default";
 
     const mcpServers = parseMcpServers(params);
-    await fetchMcpToolMetadata(mcpServers, this.logger);
+
+    // Fire off MCP metadata fetch early — it populates a module-level cache
+    // used later during permission checks, not needed by buildSessionOptions or query()
+    const mcpMetadataPromise = fetchMcpToolMetadata(mcpServers, this.logger);
 
     const options = buildSessionOptions({
       cwd: params.cwd,
@@ -184,14 +187,17 @@ export class ClaudeAcpAgent extends BaseAcpAgent {
       });
     }
 
-    const modelOptions = await this.getModelConfigOptions();
+    // Run model config, slash commands, and MCP metadata fetch in parallel
+    const [modelOptions, slashCommands] = await Promise.all([
+      this.getModelConfigOptions(),
+      getAvailableSlashCommands(q),
+      mcpMetadataPromise,
+    ]);
+
     session.modelId = modelOptions.currentModelId;
     await this.trySetModel(q, modelOptions.currentModelId);
 
-    this.sendAvailableCommandsUpdate(
-      sessionId,
-      await getAvailableSlashCommands(q),
-    );
+    this.sendAvailableCommandsUpdate(sessionId, slashCommands);
 
     return {
       sessionId,
@@ -216,7 +222,9 @@ export class ClaudeAcpAgent extends BaseAcpAgent {
     }
 
     const mcpServers = parseMcpServers(params);
-    await fetchMcpToolMetadata(mcpServers, this.logger);
+
+    // Fire off MCP metadata fetch early — populates cache for permission checks
+    const mcpMetadataPromise = fetchMcpToolMetadata(mcpServers, this.logger);
 
     const permissionMode: TwigExecutionMode =
       meta?.permissionMode &&
@@ -238,10 +246,14 @@ export class ClaudeAcpAgent extends BaseAcpAgent {
     session.taskRunId = meta?.taskRunId;
 
     this.registerPersistence(sessionId, meta as Record<string, unknown>);
-    this.sendAvailableCommandsUpdate(
-      sessionId,
-      await getAvailableSlashCommands(q),
-    );
+
+    // Run slash commands fetch and MCP metadata in parallel
+    const [slashCommands] = await Promise.all([
+      getAvailableSlashCommands(q),
+      mcpMetadataPromise,
+    ]);
+
+    this.sendAvailableCommandsUpdate(sessionId, slashCommands);
 
     return {
       configOptions: await this.buildConfigOptions(),

package/src/adapters/claude/session/options.ts

@@ -245,11 +245,7 @@ function clearStatsigCache(): void {
     process.env.CLAUDE_CONFIG_DIR || path.join(os.homedir(), ".claude"),
     "statsig",
   );
-  try {
-    if (fs.existsSync(statsigPath)) {
-      fs.rmSync(statsigPath, { recursive: true, force: true });
-    }
-  } catch {
-    // Ignore errors - cache clearing is best-effort
-  }
+  fs.rm(statsigPath, { recursive: true, force: true }, () => {
+    // Best-effort, ignore errors
+  });
 }

package/src/gateway-models.ts

@@ -26,6 +26,14 @@ type ArrayModelsResponse =
     }
   | Array<{ id?: string; owned_by?: string }>;
 
+const CACHE_TTL = 10 * 60 * 1000; // 10 minutes
+
+let gatewayModelsCache: {
+  models: GatewayModel[];
+  expiry: number;
+  url: string;
+} | null = null;
+
 export async function fetchGatewayModels(
   options?: FetchGatewayModelsOptions,
 ): Promise<GatewayModel[]> {
@@ -34,6 +42,14 @@ export async function fetchGatewayModels(
     return [];
   }
 
+  if (
+    gatewayModelsCache &&
+    gatewayModelsCache.url === gatewayUrl &&
+    Date.now() < gatewayModelsCache.expiry
+  ) {
+    return gatewayModelsCache.models;
+  }
+
   const modelsUrl = `${gatewayUrl}/v1/models`;
 
   try {
@@ -44,8 +60,13 @@ export async function fetchGatewayModels(
     }
 
     const data = (await response.json()) as GatewayModelsResponse;
-    const models = data.data ?? [];
-    return models.filter((m) => !BLOCKED_MODELS.has(m.id));
+    const models = (data.data ?? []).filter((m) => !BLOCKED_MODELS.has(m.id));
+    gatewayModelsCache = {
+      models,
+      expiry: Date.now() + CACHE_TTL,
+      url: gatewayUrl,
+    };
+    return models;
   } catch {
     return [];
   }
@@ -70,6 +91,12 @@ export interface ArrayModelInfo {
   owned_by?: string;
 }
 
+let arrayModelsCache: {
+  models: ArrayModelInfo[];
+  expiry: number;
+  url: string;
+} | null = null;
+
 export async function fetchArrayModels(
   options?: FetchGatewayModelsOptions,
 ): Promise<ArrayModelInfo[]> {
@@ -78,6 +105,14 @@ export async function fetchArrayModels(
     return [];
   }
 
+  if (
+    arrayModelsCache &&
+    arrayModelsCache.url === gatewayUrl &&
+    Date.now() < arrayModelsCache.expiry
+  ) {
+    return arrayModelsCache.models;
+  }
+
   try {
     const base = new URL(gatewayUrl);
     base.pathname = "/array/v1/models";
@@ -97,6 +132,11 @@ export async function fetchArrayModels(
       if (!id) continue;
       results.push({ id, owned_by: model?.owned_by });
     }
+    arrayModelsCache = {
+      models: results,
+      expiry: Date.now() + CACHE_TTL,
+      url: gatewayUrl,
+    };
     return results;
   } catch {
     return [];