@posthog/agent 2.1.82 → 2.1.83

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@posthog/agent",
-  "version": "2.1.82",
+  "version": "2.1.83",
   "repository": "https://github.com/PostHog/twig",
   "description": "TypeScript agent framework wrapping Claude Agent SDK with Git-based task execution for PostHog",
   "exports": {

package/src/adapters/acp-connection.ts

@@ -170,16 +170,17 @@ function createClaudeConnection(config: AcpConnectionConfig): AcpConnection {
       });
     }
 
+    const taskRunId = config.taskRunId;
     agentWritable = createTappedWritableStream(streams.agent.writable, {
       onMessage: (line) => {
-        logWriter.appendRawLine(config.taskRunId!, line);
+        logWriter.appendRawLine(taskRunId, line);
       },
       logger,
     });
 
     clientWritable = createTappedWritableStream(streams.client.writable, {
       onMessage: (line) => {
-        logWriter.appendRawLine(config.taskRunId!, line);
+        logWriter.appendRawLine(taskRunId, line);
       },
       logger,
     });
@@ -425,8 +426,8 @@ function createCodexConnection(config: AcpConnectionConfig): AcpConnection {
 
   const shouldTapLogs = config.taskRunId && logWriter;
 
-  if (shouldTapLogs) {
-    const taskRunId = config.taskRunId!;
+  if (shouldTapLogs && config.taskRunId) {
+    const taskRunId = config.taskRunId;
     if (!logWriter.isRegistered(taskRunId)) {
       logWriter.register(taskRunId, {
         taskId: config.taskId ?? taskRunId,

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

@@ -181,7 +181,7 @@ export class ClaudeAcpAgent extends BaseAcpAgent {
 
     if (meta?.taskRunId) {
       await this.client.extNotification("_posthog/sdk_session", {
-        taskRunId: meta.taskRunId!,
+        taskRunId: meta.taskRunId,
         sessionId,
         adapter: "claude",
       });

package/src/adapters/claude/conversion/sdk-to-acp.ts

@@ -5,7 +5,10 @@ import type {
 } from "@agentclientprotocol/sdk";
 import { RequestError } from "@agentclientprotocol/sdk";
 import type {
+  SDKAssistantMessage,
+  SDKMessage,
   SDKPartialAssistantMessage,
+  SDKResultMessage,
   SDKUserMessage,
 } from "@anthropic-ai/claude-agent-sdk";
 import type { ContentBlockParam } from "@anthropic-ai/sdk/resources";
@@ -357,7 +360,7 @@ function streamEventToAcpNotifications(
 }
 
 export async function handleSystemMessage(
-  message: any,
+  message: Extract<SDKMessage, { type: "system" }>,
   context: MessageHandlerContext,
 ): Promise<void> {
   const { sessionId, client, logger } = context;
@@ -409,7 +412,7 @@ export async function handleSystemMessage(
 }
 
 export function handleResultMessage(
-  message: any,
+  message: SDKResultMessage,
   context: MessageHandlerContext,
 ): { shouldStop: boolean; stopReason?: string; error?: Error } {
   const { session } = context;
@@ -556,7 +559,7 @@ function filterMessageContent(
 }
 
 export async function handleUserAssistantMessage(
-  message: SDKUserMessage | { type: "assistant"; message: any },
+  message: SDKUserMessage | SDKAssistantMessage,
   context: MessageHandlerContext,
 ): Promise<{ shouldStop?: boolean; error?: Error }> {
   const { session, sessionId, client, toolUseCache, fileContentCache, logger } =

package/src/adapters/claude/mcp/tool-metadata.ts

@@ -1,4 +1,7 @@
-import type { McpServerConfig } from "@anthropic-ai/claude-agent-sdk";
+import type {
+  McpHttpServerConfig,
+  McpServerConfig,
+} from "@anthropic-ai/claude-agent-sdk";
 import { Client } from "@modelcontextprotocol/sdk/client/index.js";
 import { StreamableHTTPClientTransport } from "@modelcontextprotocol/sdk/client/streamableHttp.js";
 import type { Tool } from "@modelcontextprotocol/sdk/types.js";
@@ -16,17 +19,17 @@ function buildToolKey(serverName: string, toolName: string): string {
 
 function isHttpMcpServer(
   config: McpServerConfig,
-): config is McpServerConfig & { type: "http"; url: string } {
-  return config.type === "http" && typeof (config as any).url === "string";
+): config is McpHttpServerConfig {
+  return config.type === "http" && typeof config.url === "string";
 }
 
 async function fetchToolsFromHttpServer(
   _serverName: string,
-  config: McpServerConfig & { type: "http"; url: string },
+  config: McpHttpServerConfig,
 ): Promise<Tool[]> {
   const transport = new StreamableHTTPClientTransport(new URL(config.url), {
     requestInit: {
-      headers: (config as any).headers || {},
+      headers: config.headers ?? {},
     },
   });
 

package/src/adapters/claude/permissions/permission-handlers.ts

@@ -192,7 +192,7 @@ async function applyPlanApproval(
 async function handleEnterPlanModeTool(
   context: ToolHandlerContext,
 ): Promise<ToolPermissionResult> {
-  const { session, toolInput, logger } = context;
+  const { session, toolInput } = context;
 
   session.permissionMode = "plan";
   await session.query.setPermissionMode("plan");

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

@@ -152,9 +152,15 @@ function buildSpawnWrapper(
       });
     }
 
+    if (!child.stdin || !child.stdout) {
+      throw new Error(
+        `Failed to get stdio streams for spawned process (pid=${child.pid})`,
+      );
+    }
+
     return {
-      stdin: child.stdin!,
-      stdout: child.stdout!,
+      stdin: child.stdin,
+      stdout: child.stdout,
       get killed() {
         return child.killed;
       },
@@ -164,12 +170,15 @@ function buildSpawnWrapper(
       kill(signal: NodeJS.Signals) {
         return child.kill(signal);
       },
+      // biome-ignore lint/suspicious/noExplicitAny: ChildProcess event listener types require any[]
       on(event: "exit" | "error", listener: (...args: any[]) => void) {
         child.on(event, listener);
       },
+      // biome-ignore lint/suspicious/noExplicitAny: ChildProcess event listener types require any[]
       once(event: "exit" | "error", listener: (...args: any[]) => void) {
         child.once(event, listener);
       },
+      // biome-ignore lint/suspicious/noExplicitAny: ChildProcess event listener types require any[]
       off(event: "exit" | "error", listener: (...args: any[]) => void) {
         child.off(event, listener);
       },

package/src/agent.ts

@@ -7,7 +7,7 @@ import {
   DEFAULT_GATEWAY_MODEL,
   fetchArrayModels,
 } from "./gateway-models.js";
-import { PostHogAPIClient } from "./posthog-api.js";
+import { PostHogAPIClient, type TaskRunUpdate } from "./posthog-api.js";
 import { SessionLogWriter } from "./session-log-writer.js";
 import type { AgentConfig, TaskExecutionOptions } from "./types.js";
 import { Logger } from "./utils/logger.js";
@@ -142,7 +142,7 @@ export class Agent {
       throw error;
     }
 
-    const updates: any = {
+    const updates: TaskRunUpdate = {
       output: { pr_url: prUrl },
     };
     if (branchName) {

package/src/sagas/apply-snapshot-saga.ts

@@ -33,27 +33,30 @@ export class ApplySnapshotSaga extends Saga<
       throw new Error("Cannot apply snapshot: no archive URL");
     }
 
+    const archiveUrl = snapshot.archiveUrl;
+
     await this.step({
       name: "create_tmp_dir",
       execute: () => mkdir(tmpDir, { recursive: true }),
       rollback: async () => {},
     });
 
-    this.archivePath = join(tmpDir, `${snapshot.treeHash}.tar.gz`);
+    const archivePath = join(tmpDir, `${snapshot.treeHash}.tar.gz`);
+    this.archivePath = archivePath;
     await this.step({
       name: "download_archive",
       execute: async () => {
         const arrayBuffer = await apiClient.downloadArtifact(
           taskId,
           runId,
-          snapshot.archiveUrl!,
+          archiveUrl,
         );
         if (!arrayBuffer) {
           throw new Error("Failed to download archive");
         }
         const base64Content = Buffer.from(arrayBuffer).toString("utf-8");
         const binaryContent = Buffer.from(base64Content, "base64");
-        await writeFile(this.archivePath!, binaryContent);
+        await writeFile(archivePath, binaryContent);
       },
       rollback: async () => {
         if (this.archivePath) {

package/src/sagas/capture-tree-saga.test.ts

@@ -583,7 +583,7 @@ describe("CaptureTreeSaga", () => {
     });
 
     it("handles symlinks", async () => {
-      const { symlink, lstat } = await import("node:fs/promises");
+      const { symlink } = await import("node:fs/promises");
 
       await repo.writeFile("target.txt", "symlink target content");
       await symlink("target.txt", join(repo.path, "link.txt"));