@posthog/agent 1.25.0 → 1.27.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@posthog/agent",
-  "version": "1.25.0",
+  "version": "1.27.0",
   "repository": "https://github.com/PostHog/array",
   "description": "TypeScript agent framework wrapping Claude Agent SDK with Git-based task execution for PostHog",
   "main": "./dist/index.js",
@@ -8,9 +8,9 @@
   "types": "./dist/index.d.ts",
   "exports": {
     ".": {
+      "types": "./dist/index.d.ts",
       "import": "./dist/index.js",
-      "require": "./dist/index.js",
-      "types": "./dist/index.d.ts"
+      "require": "./dist/index.js"
     }
   },
   "type": "module",
@@ -29,15 +29,9 @@
   },
   "devDependencies": {
     "@changesets/cli": "^2.27.8",
-    "@rollup/plugin-commonjs": "^25.0.7",
-    "@rollup/plugin-json": "^6.1.0",
-    "@rollup/plugin-node-resolve": "^15.2.3",
     "@types/bun": "latest",
     "minimatch": "^10.0.3",
-    "rimraf": "^6.0.1",
-    "rollup": "^4.24.0",
-    "rollup-plugin-copy": "^3.5.0",
-    "rollup-plugin-typescript2": "^0.36.0",
+    "tsup": "^8.5.1",
     "tsx": "^4.20.6",
     "typescript": "^5.5.0"
   },
@@ -62,8 +56,8 @@
     "access": "public"
   },
   "scripts": {
-    "build": "pnpm exec rimraf dist && pnpm exec rollup -c && pnpm exec tsc --project tsconfig.build.json --emitDeclarationOnly",
-    "dev": "pnpm exec rollup -c --watch",
+    "build": "tsup",
+    "dev": "tsup --watch",
     "typecheck": "pnpm exec tsc --noEmit",
     "example": "tsx example.ts"
   }

package/src/adapters/claude/claude.ts

@@ -171,7 +171,7 @@ export class ClaudeAcpAgent implements Agent {
   fileContentCache: { [key: string]: string };
   backgroundTerminals: { [key: string]: BackgroundTerminal } = {};
   clientCapabilities?: ClientCapabilities;
-  logger: Logger = new Logger({ debug: false, prefix: "[ClaudeAcpAgent]" });
+  logger: Logger = new Logger({ debug: true, prefix: "[ClaudeAcpAgent]" });
   sessionStore?: SessionStore;
 
   constructor(client: AgentSideConnection, sessionStore?: SessionStore) {
@@ -341,9 +341,13 @@ export class ClaudeAcpAgent implements Agent {
       allowDangerouslySkipPermissions: !IS_ROOT,
       permissionMode,
       canUseTool: this.canUseTool(sessionId),
-      // note: although not documented by the types, passing an absolute path
-      // here works to find zed's managed node version.
-      executable: process.execPath as any,
+      // Use "node" to resolve via PATH where a symlink to Electron exists.
+      // This avoids launching the Electron binary directly from the app bundle,
+      // which can cause dock icons to appear on macOS even with ELECTRON_RUN_AS_NODE.
+      executable: "node",
+      // Prevent spawned Electron processes from showing in dock/tray.
+      // Must merge with process.env since SDK replaces rather than merges.
+      env: { ...process.env, ELECTRON_RUN_AS_NODE: "1" },
       ...(process.env.CLAUDE_CODE_EXECUTABLE && {
         pathToClaudeCodeExecutable: process.env.CLAUDE_CODE_EXECUTABLE,
       }),
@@ -930,6 +934,7 @@ export class ClaudeAcpAgent implements Agent {
   async resumeSession(
     params: LoadSessionRequest,
   ): Promise<LoadSessionResponse> {
+    this.logger.info("[RESUME] Resuming session", { params });
     const { sessionId } = params;
 
     // Extract persistence config and SDK session ID from _meta
@@ -976,6 +981,11 @@ export class ClaudeAcpAgent implements Agent {
 
       const permissionMode = "default";
 
+      this.logger.info("Resuming session", {
+        cwd: params.cwd,
+        sdkSessionId,
+        persistence,
+      });
       const options: Options = {
         cwd: params.cwd,
         includePartialMessages: true,
@@ -986,7 +996,13 @@ export class ClaudeAcpAgent implements Agent {
         permissionMode,
         canUseTool: this.canUseTool(sessionId),
         stderr: (err) => this.logger.error(err),
-        executable: process.execPath as any,
+        // Use "node" to resolve via PATH where a symlink to Electron exists.
+        // This avoids launching the Electron binary directly from the app bundle,
+        // which can cause dock icons to appear on macOS even with ELECTRON_RUN_AS_NODE.
+        executable: "node",
+        // Prevent spawned Electron processes from showing in dock/tray.
+        // Must merge with process.env since SDK replaces rather than merges.
+        env: { ...process.env, ELECTRON_RUN_AS_NODE: "1" },
         ...(process.env.CLAUDE_CODE_EXECUTABLE && {
           pathToClaudeCodeExecutable: process.env.CLAUDE_CODE_EXECUTABLE,
         }),

package/src/adapters/claude/tools.ts

@@ -153,7 +153,14 @@ export function toolInfoFromToolUse(
         title: "Read File",
         kind: "read",
         content: [],
-        locations: [{ path: input.file_path, line: input.offset ?? 0 }],
+        locations: input.file_path
+          ? [
+              {
+                path: input.file_path,
+                line: input.offset ?? 0,
+              },
+            ]
+          : [],
       };
 
     case "LS":

package/src/agent.ts

@@ -1,3 +1,10 @@
+import {
+  type Client,
+  ClientSideConnection,
+  type ContentBlock,
+  ndJsonStream,
+  PROTOCOL_VERSION,
+} from "@agentclientprotocol/sdk";
 import { POSTHOG_NOTIFICATIONS } from "./acp-extensions.js";
 import {
   createAcpConnection,
@@ -10,7 +17,13 @@ import { PromptBuilder } from "./prompt-builder.js";
 import { SessionStore } from "./session-store.js";
 import { TaskManager } from "./task-manager.js";
 import { TemplateManager } from "./template-manager.js";
-import type { AgentConfig, CanUseTool, Task } from "./types.js";
+import type {
+  AgentConfig,
+  CanUseTool,
+  StoredNotification,
+  Task,
+  TaskExecutionOptions,
+} from "./types.js";
 import { Logger } from "./utils/logger.js";
 import { TASK_WORKFLOW } from "./workflow/config.js";
 import type { SendNotification, WorkflowRuntime } from "./workflow/types.js";
@@ -312,6 +325,29 @@ export class Agent {
     return this.posthogAPI;
   }
 
+  /**
+   * Send a notification to a cloud task run's S3 log.
+   * The cloud runner will pick up new notifications via interrupt polling.
+   */
+  async sendNotification(
+    taskId: string,
+    runId: string,
+    notification: StoredNotification,
+  ): Promise<void> {
+    if (!this.posthogAPI) {
+      throw new Error(
+        "PostHog API not configured. Cannot send notification to cloud task.",
+      );
+    }
+
+    await this.posthogAPI.appendTaskRunLog(taskId, runId, [notification]);
+    this.logger.debug("Notification sent to cloud task", {
+      taskId,
+      runId,
+      method: notification.notification.method,
+    });
+  }
+
   async getTaskFiles(taskId: string): Promise<any[]> {
     this.logger.debug("Getting task files", { taskId });
     const files = await this.fileManager.getTaskFiles(taskId);
@@ -493,6 +529,219 @@ Generated by PostHog Agent`;
     }
   }
 
+  async runTaskCloud(
+    taskId: string,
+    taskRunId: string,
+    options: TaskExecutionOptions = {},
+  ): Promise<void> {
+    await this._configureLlmGateway();
+
+    const task = await this.fetchTask(taskId);
+    const cwd = options.repositoryPath || this.workingDirectory;
+    const taskSlug = (task as any).slug || task.id;
+
+    this.currentRunId = taskRunId;
+
+    this.logger.info("Starting cloud task execution", {
+      taskId: task.id,
+      taskSlug,
+      taskRunId,
+      cwd,
+    });
+
+    if (!this.sessionStore) {
+      throw new Error(
+        "SessionStore required for cloud mode. Ensure PostHog API credentials are configured.",
+      );
+    }
+
+    // Start session in SessionStore (updates task run status to in_progress)
+    const taskRun = await this.sessionStore.start(taskRunId, taskId, taskRunId);
+    this.logger.debug("Session started", {
+      taskRunId,
+      logUrl: taskRun?.log_url,
+    });
+
+    // Create internal ACP connection with S3 persistence
+    const acpConnection = createAcpConnection({
+      sessionStore: this.sessionStore,
+      sessionId: taskRunId,
+      taskId: task.id,
+    });
+
+    // Create client connection using the client-side streams
+    const clientStream = ndJsonStream(
+      acpConnection.clientStreams.writable as WritableStream<Uint8Array>,
+      acpConnection.clientStreams.readable as ReadableStream<Uint8Array>,
+    );
+
+    // Create auto-approving client for headless cloud mode
+    const cloudClient: Client = {
+      async requestPermission(params) {
+        const allowOption = params.options.find(
+          (o) => o.kind === "allow_once" || o.kind === "allow_always",
+        );
+        return {
+          outcome: {
+            outcome: "selected",
+            optionId: allowOption?.optionId ?? params.options[0].optionId,
+          },
+        };
+      },
+      async sessionUpdate(_params) {
+        // Notifications are already being persisted to S3 via tapped streams
+      },
+    };
+
+    const clientConnection = new ClientSideConnection(
+      (_agent) => cloudClient,
+      clientStream,
+    );
+
+    try {
+      // Initialize the connection
+      await clientConnection.initialize({
+        protocolVersion: PROTOCOL_VERSION,
+        clientCapabilities: {},
+      });
+
+      // Create new session
+      await clientConnection.newSession({
+        cwd,
+        mcpServers: [],
+        _meta: { sessionId: taskRunId },
+      });
+
+      // Prepare git branch if not skipped
+      if (!options.skipGitBranch) {
+        const sendNotification: SendNotification = async (method, params) => {
+          this.logger.debug(`Notification: ${method}`, params);
+          await acpConnection.agentConnection.extNotification?.(method, params);
+        };
+        await this.prepareTaskBranch(taskSlug, true, sendNotification);
+      }
+
+      // Build initial prompt from task description
+      const initialPrompt: ContentBlock[] = [
+        {
+          type: "text",
+          text: `# Task: ${task.title}\n\n${task.description}`,
+        },
+      ];
+
+      // Track the last known log entry count for interrupt polling
+      let lastKnownEntryCount = 0;
+      let isPolling = true;
+
+      // Start interrupt polling in background
+      const pollForInterrupts = async () => {
+        while (isPolling) {
+          await new Promise((resolve) => setTimeout(resolve, 2000)); // Poll every 2 seconds
+          if (!isPolling) break;
+
+          try {
+            const newEntries = await this.sessionStore?.pollForNewEntries(
+              taskRunId,
+              lastKnownEntryCount,
+            );
+
+            for (const entry of newEntries ?? []) {
+              lastKnownEntryCount++;
+              // Look for user_message notifications
+              if (
+                entry.notification?.method === "sessionUpdate" &&
+                (entry.notification?.params as any)?.sessionUpdate ===
+                  "user_message"
+              ) {
+                const content = (entry.notification?.params as any)?.content;
+                if (content) {
+                  this.logger.info("Processing user interrupt", { content });
+                  // Send as new prompt - will be processed after current prompt completes
+                  await clientConnection.prompt({
+                    sessionId: taskRunId,
+                    prompt: Array.isArray(content) ? content : [content],
+                  });
+                }
+              }
+            }
+          } catch (err) {
+            this.logger.warn("Interrupt polling error", { error: err });
+          }
+        }
+      };
+
+      // Start polling in background (don't await)
+      const pollingPromise = pollForInterrupts();
+
+      // Send initial prompt and wait for completion
+      this.logger.info("Sending initial prompt to agent");
+      const result = await clientConnection.prompt({
+        sessionId: taskRunId,
+        prompt: initialPrompt,
+      });
+
+      // Stop interrupt polling
+      isPolling = false;
+      await pollingPromise;
+
+      this.logger.info("Task execution complete", {
+        taskId: task.id,
+        stopReason: result.stopReason,
+      });
+
+      const branchName = await this.gitManager.getCurrentBranch();
+      const hasChanges = await this.gitManager.hasChanges();
+      const shouldCreatePR = options.createPR ?? false;
+
+      if (hasChanges) {
+        this.logger.info("Committing uncommitted changes", { taskId: task.id });
+        await this.gitManager.commitImplementation(
+          task.id,
+          task.title,
+          task.description ?? undefined,
+        );
+      }
+
+      const defaultBranch = await this.gitManager.getDefaultBranch();
+      if (branchName !== defaultBranch) {
+        this.logger.info("Pushing branch", { branchName, taskId: task.id });
+        await this.gitManager.pushBranch(branchName);
+
+        if (shouldCreatePR) {
+          this.logger.info("Creating PR", { branchName, taskId: task.id });
+
+          const prUrl = await this.createPullRequest(
+            task.id,
+            branchName,
+            task.title,
+            task.description ?? "",
+          );
+
+          this.logger.info("PR created", { prUrl, taskId: task.id });
+
+          try {
+            await this.attachPullRequestToTask(task.id, prUrl, branchName);
+          } catch (err) {
+            this.logger.warn("Could not attach PR to task", {
+              error: err instanceof Error ? err.message : String(err),
+            });
+          }
+        }
+      }
+
+      await this.sessionStore.complete(taskRunId);
+    } catch (error) {
+      const errorMessage =
+        error instanceof Error ? error.message : String(error);
+      this.logger.error("Cloud task execution failed", {
+        taskId: task.id,
+        error: errorMessage,
+      });
+      await this.sessionStore.fail(taskRunId, errorMessage);
+      throw error;
+    }
+  }
+
   private async ensurePullRequest(
     task: Task,
     stepResults: Record<string, any>,

package/src/session-store.ts

@@ -90,7 +90,7 @@ export class SessionStore {
   }
 
   /** Load raw JSON-RPC messages from S3 */
-  async load(logUrl: string): Promise<unknown[]> {
+  async load(logUrl: string): Promise<StoredNotification[]> {
     const response = await fetch(logUrl);
 
     if (!response.ok) {
@@ -105,12 +105,27 @@ export class SessionStore {
       .split("\n")
       .map((line) => {
         try {
-          return JSON.parse(line);
+          return JSON.parse(line) as StoredNotification;
         } catch {
           return null;
         }
       })
-      .filter((entry): entry is unknown => entry !== null);
+      .filter((entry): entry is StoredNotification => entry !== null);
+  }
+
+  /**
+   * Poll S3 for new entries since last check.
+   * Used for interrupt handling in cloud mode.
+   */
+  async pollForNewEntries(
+    sessionId: string,
+    lastKnownCount: number,
+  ): Promise<StoredNotification[]> {
+    const config = this.configs.get(sessionId);
+    if (!config?.logUrl) return [];
+
+    const entries = await this.load(config.logUrl);
+    return entries.slice(lastKnownCount);
   }
 
   /** Force flush pending entries */

package/src/types.ts

@@ -134,7 +134,7 @@ export interface TaskExecutionOptions {
   repositoryPath?: string;
   permissionMode?: PermissionMode;
   isCloudMode?: boolean; // Determines local vs cloud behavior (local pauses after each phase)
-  createPR?: boolean; // Whether to create PR after build (defaults to false if local. This setting has no effect if isCloudMode is true.)
+  createPR?: boolean; // Whether to create PR after build (defaults to false)
   autoProgress?: boolean;
   queryOverrides?: Record<string, unknown>;
   // Fine-grained permission control (only applied to build phase)

package/src/worktree-manager.ts

@@ -643,7 +643,7 @@ export class WorktreeManager {
     // Generate unique worktree name
     const worktreeName = await this.generateUniqueWorktreeName();
     const worktreePath = this.getWorktreePath(worktreeName);
-    const branchName = `posthog/${worktreeName}`;
+    const branchName = `array/${worktreeName}`;
     const baseBranch = await this.getDefaultBranch();
 
     this.logger.info("Creating worktree", {
@@ -769,13 +769,10 @@ export class WorktreeManager {
       // 1. Are in our worktree folder (external or in-repo)
       // 2. Have a posthog/ branch prefix (our naming convention)
       const isInWorktreeFolder = worktreePath?.startsWith(worktreeFolderPath);
-      const isPosthogBranch = branchName?.startsWith("posthog/");
+      const isArrayBranch =
+        branchName?.startsWith("array/") || branchName?.startsWith("posthog/");
 
-      if (
-        worktreePath &&
-        branchName &&
-        (isInWorktreeFolder || isPosthogBranch)
-      ) {
+      if (worktreePath && branchName && (isInWorktreeFolder || isArrayBranch)) {
         const worktreeName = path.basename(worktreePath);
         worktrees.push({
           worktreePath,

package/dist/_virtual/_commonjsHelpers.js

@@ -1,6 +0,0 @@
-function getDefaultExportFromCjs (x) {
-	return x && x.__esModule && Object.prototype.hasOwnProperty.call(x, 'default') ? x['default'] : x;
-}
-
-export { getDefaultExportFromCjs };
-//# sourceMappingURL=_commonjsHelpers.js.map

package/dist/_virtual/_commonjsHelpers.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"_commonjsHelpers.js","sources":[],"sourcesContent":[],"names":[],"mappings":";;;;;;"}

package/dist/_virtual/additionalItems.js

@@ -1,4 +0,0 @@
-var additionalItems = {};
-
-export { additionalItems as __exports };
-//# sourceMappingURL=additionalItems.js.map

package/dist/_virtual/additionalItems.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"additionalItems.js","sources":[],"sourcesContent":[],"names":[],"mappings":";;;;"}

package/dist/_virtual/additionalProperties.js

@@ -1,4 +0,0 @@
-var additionalProperties = {};
-
-export { additionalProperties as __exports };
-//# sourceMappingURL=additionalProperties.js.map

package/dist/_virtual/additionalProperties.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"additionalProperties.js","sources":[],"sourcesContent":[],"names":[],"mappings":";;;;"}

package/dist/_virtual/ajv.js

@@ -1,4 +0,0 @@
-var ajv = {exports: {}};
-
-export { ajv as __module };
-//# sourceMappingURL=ajv.js.map

package/dist/_virtual/ajv.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"ajv.js","sources":[],"sourcesContent":[],"names":[],"mappings":";;;;"}

package/dist/_virtual/allOf.js

@@ -1,4 +0,0 @@
-var allOf = {};
-
-export { allOf as __exports };
-//# sourceMappingURL=allOf.js.map

package/dist/_virtual/allOf.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"allOf.js","sources":[],"sourcesContent":[],"names":[],"mappings":";;;;"}

package/dist/_virtual/anyOf.js

@@ -1,4 +0,0 @@
-var anyOf = {};
-
-export { anyOf as __exports };
-//# sourceMappingURL=anyOf.js.map

package/dist/_virtual/anyOf.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"anyOf.js","sources":[],"sourcesContent":[],"names":[],"mappings":";;;;"}

package/dist/_virtual/applicability.js

@@ -1,4 +0,0 @@
-var applicability = {};
-
-export { applicability as __exports };
-//# sourceMappingURL=applicability.js.map

package/dist/_virtual/applicability.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"applicability.js","sources":[],"sourcesContent":[],"names":[],"mappings":";;;;"}

package/dist/_virtual/boolSchema.js

@@ -1,4 +0,0 @@
-var boolSchema = {};
-
-export { boolSchema as __exports };
-//# sourceMappingURL=boolSchema.js.map

package/dist/_virtual/boolSchema.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"boolSchema.js","sources":[],"sourcesContent":[],"names":[],"mappings":";;;;"}

package/dist/_virtual/code.js

@@ -1,4 +0,0 @@
-var code = {};
-
-export { code as __exports };
-//# sourceMappingURL=code.js.map

package/dist/_virtual/code.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"code.js","sources":[],"sourcesContent":[],"names":[],"mappings":";;;;"}

package/dist/_virtual/code2.js

@@ -1,4 +0,0 @@
-var code = {};
-
-export { code as __exports };
-//# sourceMappingURL=code2.js.map

package/dist/_virtual/code2.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"code2.js","sources":[],"sourcesContent":[],"names":[],"mappings":";;;;"}

package/dist/_virtual/const.js

@@ -1,4 +0,0 @@
-var _const = {};
-
-export { _const as __exports };
-//# sourceMappingURL=const.js.map

package/dist/_virtual/const.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"const.js","sources":[],"sourcesContent":[],"names":[],"mappings":";;;;"}

package/dist/_virtual/contains.js

@@ -1,4 +0,0 @@
-var contains = {};
-
-export { contains as __exports };
-//# sourceMappingURL=contains.js.map

package/dist/_virtual/contains.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"contains.js","sources":[],"sourcesContent":[],"names":[],"mappings":";;;;"}

package/dist/_virtual/core.js

@@ -1,4 +0,0 @@
-var core = {};
-
-export { core as __exports };
-//# sourceMappingURL=core.js.map

package/dist/_virtual/core.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"core.js","sources":[],"sourcesContent":[],"names":[],"mappings":";;;;"}

package/dist/_virtual/dataType.js

@@ -1,4 +0,0 @@
-var dataType = {};
-
-export { dataType as __exports };
-//# sourceMappingURL=dataType.js.map

package/dist/_virtual/dataType.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"dataType.js","sources":[],"sourcesContent":[],"names":[],"mappings":";;;;"}

package/dist/_virtual/defaults.js

@@ -1,4 +0,0 @@
-var defaults = {};
-
-export { defaults as __exports };
-//# sourceMappingURL=defaults.js.map

package/dist/_virtual/defaults.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"defaults.js","sources":[],"sourcesContent":[],"names":[],"mappings":";;;;"}

package/dist/_virtual/dependencies.js

@@ -1,4 +0,0 @@
-var dependencies = {};
-
-export { dependencies as __exports };
-//# sourceMappingURL=dependencies.js.map

package/dist/_virtual/dependencies.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"dependencies.js","sources":[],"sourcesContent":[],"names":[],"mappings":";;;;"}

package/dist/_virtual/draft7.js

@@ -1,4 +0,0 @@
-var draft7 = {};
-
-export { draft7 as __exports };
-//# sourceMappingURL=draft7.js.map

package/dist/_virtual/draft7.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"draft7.js","sources":[],"sourcesContent":[],"names":[],"mappings":";;;;"}

package/dist/_virtual/enum.js

@@ -1,4 +0,0 @@
-var _enum = {};
-
-export { _enum as __exports };
-//# sourceMappingURL=enum.js.map

package/dist/_virtual/enum.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"enum.js","sources":[],"sourcesContent":[],"names":[],"mappings":";;;;"}

package/dist/_virtual/equal.js

@@ -1,4 +0,0 @@
-var equal = {};
-
-export { equal as __exports };
-//# sourceMappingURL=equal.js.map

package/dist/_virtual/equal.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"equal.js","sources":[],"sourcesContent":[],"names":[],"mappings":";;;;"}