@posthog/agent 2.3.556 → 2.3.619

package/dist/server/bin.cjs

@@ -3951,6 +3951,7 @@ var import_sdk5 = require("@agentclientprotocol/sdk");
 var import_node_server = require("@hono/node-server");
 
 // ../git/dist/queries.js
+var import_node_fs = require("fs");
 var fs3 = __toESM(require("fs/promises"), 1);
 var path2 = __toESM(require("path"), 1);
 
@@ -8502,13 +8503,20 @@ init_git_response_error();
 var simpleGit = gitInstanceFactory;
 
 // ../git/dist/client.js
+var PERFORMANCE_CONFIG = [
+  "core.untrackedCache=true",
+  "core.fsmonitor=true",
+  "core.preloadIndex=true"
+];
 function createGitClient(baseDir, options) {
-  const { abortSignal: signal, ...rest } = options ?? {};
+  const { abortSignal: signal, config: callerConfig, ...rest } = options ?? {};
+  const config = callerConfig ? [...PERFORMANCE_CONFIG, ...callerConfig] : PERFORMANCE_CONFIG;
   return simpleGit({
     baseDir,
     maxConcurrentProcesses: 6,
     trimmed: true,
     abort: signal,
+    config,
     ...rest
   });
 }
@@ -8661,14 +8669,18 @@ var GitOperationManagerImpl = class _GitOperationManagerImpl {
   }
   async executeRead(repoPath, operation, options) {
     const state = this.getRepoState(repoPath);
+    const env = {
+      ...getCleanEnv(),
+      GIT_OPTIONAL_LOCKS: "0",
+      ...options?.env
+    };
     if (options?.signal) {
       const scopedGit = createGitClient(repoPath, {
         abortSignal: options.signal
       });
-      return operation(scopedGit.env({ ...getCleanEnv(), GIT_OPTIONAL_LOCKS: "0" }));
+      return operation(scopedGit.env(env));
     }
-    const git = state.client.env({ ...getCleanEnv(), GIT_OPTIONAL_LOCKS: "0" });
-    return operation(git);
+    return operation(state.client.env(env));
   }
   async executeWrite(repoPath, operation, options) {
     const state = this.getRepoState(repoPath);
@@ -8678,15 +8690,16 @@ var GitOperationManagerImpl = class _GitOperationManagerImpl {
         throw new Error(`Git repository is locked: ${repoPath}`);
       }
     }
+    const env = { ...getCleanEnv(), ...options?.env };
     await state.lock.acquireWrite();
     try {
       if (options?.signal) {
         const scopedGit = createGitClient(repoPath, {
           abortSignal: options.signal
         });
-        return await operation(scopedGit.env(getCleanEnv()));
+        return await operation(scopedGit.env(env));
       }
-      return await operation(state.client.env(getCleanEnv()));
+      return await operation(state.client.env(env));
     } catch (error) {
       if (options?.signal?.aborted) {
         await removeLock(repoPath).catch(() => {
@@ -8713,6 +8726,9 @@ function getGitOperationManager() {
   return instance2;
 }
 
+// ../git/dist/status-stream.js
+var import_node_child_process2 = require("child_process");
+
 // ../git/dist/queries.js
 async function getCurrentBranch(baseDir, options) {
   const manager = getGitOperationManager();
@@ -8747,6 +8763,44 @@ async function listWorktrees(baseDir, options) {
     return worktrees;
   }, { signal: options?.abortSignal });
 }
+async function inspectGitBusyState(git) {
+  const toplevel = (await git.raw(["rev-parse", "--show-toplevel"])).trim();
+  const resolveGitPath = async (gitPath) => {
+    const relative = (await git.raw(["rev-parse", "--git-path", gitPath])).trim();
+    return path2.isAbsolute(relative) ? relative : path2.resolve(toplevel, relative);
+  };
+  const pathExists = async (gitPath) => {
+    const resolved = await resolveGitPath(gitPath);
+    try {
+      await fs3.access(resolved);
+      return true;
+    } catch {
+      return false;
+    }
+  };
+  const dirExists = async (gitPath) => {
+    const resolved = await resolveGitPath(gitPath);
+    try {
+      const stat4 = await fs3.stat(resolved);
+      return stat4.isDirectory();
+    } catch {
+      return false;
+    }
+  };
+  if (await dirExists("rebase-merge") || await dirExists("rebase-apply")) {
+    return { busy: true, operation: "rebase" };
+  }
+  if (await pathExists("MERGE_HEAD")) {
+    return { busy: true, operation: "merge" };
+  }
+  if (await pathExists("CHERRY_PICK_HEAD")) {
+    return { busy: true, operation: "cherry-pick" };
+  }
+  if (await pathExists("REVERT_HEAD")) {
+    return { busy: true, operation: "revert" };
+  }
+  return { busy: false };
+}
 
 // src/server/agent-server.ts
 var import_hono = require("hono");
@@ -8755,7 +8809,7 @@ var import_zod3 = require("zod");
 // package.json
 var package_default = {
   name: "@posthog/agent",
-  version: "2.3.556",
+  version: "2.3.619",
   repository: "https://github.com/PostHog/code",
   description: "TypeScript agent framework wrapping Claude Agent SDK with Git-based task execution for PostHog",
   exports: {
@@ -11311,8 +11365,7 @@ var ParserManager = class {
       }
       this.queryCache.set(cacheKey, query2);
       return query2;
-    } catch (err2) {
-      warn("Query compilation failed", err2);
+    } catch {
       this.failedQueries.add(cacheKey);
       return null;
     }
@@ -13698,6 +13751,23 @@ function tryParsePartialJson(s) {
   return null;
 }
 
+// src/adapters/error-classification.ts
+var UPSTREAM_PROVIDER_ERROR_STATUS_PATTERN = /API Error:\s*(?:429|5\d\d)\b/i;
+function classifyAgentError(result) {
+  if (!result) return "agent_error";
+  const text2 = result.trim();
+  if (/API Error:\s*terminated\b/i.test(text2)) {
+    return "upstream_stream_terminated";
+  }
+  if (/API Error:\s*Connection error\b/i.test(text2)) {
+    return "upstream_connection_error";
+  }
+  if (UPSTREAM_PROVIDER_ERROR_STATUS_PATTERN.test(text2)) {
+    return "upstream_provider_failure";
+  }
+  return "agent_error";
+}
+
 // src/adapters/claude/permissions/posthog-exec-gate.ts
 var POSTHOG_EXEC_TOOL_RE = /^mcp__posthog(?:_[^_]+)*__exec$/;
 var POSTHOG_CALL_COMMAND_RE = /^\s*call\s+(?:--json\s+)?([a-zA-Z0-9_-]+)/;
@@ -13894,7 +13964,7 @@ var createPreToolUseHook = (settingsManager, logger) => async (input, _toolUseID
 };
 
 // src/adapters/claude/conversion/tool-use-to-acp.ts
-var import_node_fs = __toESM(require("fs"), 1);
+var import_node_fs2 = __toESM(require("fs"), 1);
 var import_node_path3 = __toESM(require("path"), 1);
 
 // src/adapters/claude/mcp/tool-metadata.ts
@@ -14128,7 +14198,7 @@ function toolInfoFromToolUse(toolUse, options) {
           oldContent = options.cachedFileContent[writeFilePath];
         } else {
           try {
-            oldContent = import_node_fs.default.readFileSync(writeFilePath, "utf-8");
+            oldContent = import_node_fs2.default.readFileSync(writeFilePath, "utf-8");
           } catch {
           }
         }
@@ -14542,7 +14612,7 @@ function resolveFileContent(filePath, oldText, cachedFileContent) {
     }
   }
   try {
-    const content = import_node_fs.default.readFileSync(filePath, "utf-8");
+    const content = import_node_fs2.default.readFileSync(filePath, "utf-8");
     if (content.includes(oldText)) {
       return content;
     }
@@ -15014,17 +15084,6 @@ async function handleSystemMessage(message, context) {
       break;
   }
 }
-function classifyAgentError(result) {
-  if (!result) return "agent_error";
-  const text2 = result.trim();
-  if (/API Error:\s*terminated\b/i.test(text2)) {
-    return "upstream_stream_terminated";
-  }
-  if (/API Error:\s*Connection error\b/i.test(text2)) {
-    return "upstream_connection_error";
-  }
-  return "agent_error";
-}
 function handleResultMessage(message) {
   const usage = extractUsageFromResult(message);
   switch (message.subtype) {
@@ -16086,7 +16145,7 @@ function parseMcpServers(params) {
 }
 
 // src/adapters/claude/session/options.ts
-var import_node_child_process2 = require("child_process");
+var import_node_child_process3 = require("child_process");
 var fs7 = __toESM(require("fs"), 1);
 var os3 = __toESM(require("os"), 1);
 var path9 = __toESM(require("path"), 1);
@@ -16234,7 +16293,7 @@ function getAbortController(userProvidedController) {
 }
 function buildSpawnWrapper(sessionId, onProcessSpawned, onProcessExited, logger) {
   return (spawnOpts) => {
-    const child = (0, import_node_child_process2.spawn)(spawnOpts.command, spawnOpts.args, {
+    const child = (0, import_node_child_process3.spawn)(spawnOpts.command, spawnOpts.args, {
       cwd: spawnOpts.cwd,
       env: spawnOpts.env,
       stdio: ["pipe", "pipe", "pipe"]
@@ -17907,7 +17966,7 @@ var ClaudeAcpAgent = class extends BaseAcpAgent {
 };
 
 // src/adapters/codex/codex-agent.ts
-var import_node_fs3 = require("fs");
+var import_node_fs4 = require("fs");
 var import_node_path5 = require("path");
 var import_sdk3 = require("@agentclientprotocol/sdk");
 
@@ -18192,8 +18251,8 @@ function parseCodexToml(content, cwd) {
 }
 
 // src/adapters/codex/spawn.ts
-var import_node_child_process3 = require("child_process");
-var import_node_fs2 = require("fs");
+var import_node_child_process4 = require("child_process");
+var import_node_fs3 = require("fs");
 var import_node_path4 = require("path");
 function buildConfigArgs(options) {
   const args2 = [];
@@ -18225,7 +18284,7 @@ function buildConfigArgs(options) {
 }
 function findCodexBinary(options) {
   const configArgs = buildConfigArgs(options);
-  if (options.binaryPath && (0, import_node_fs2.existsSync)(options.binaryPath)) {
+  if (options.binaryPath && (0, import_node_fs3.existsSync)(options.binaryPath)) {
     return { command: options.binaryPath, args: configArgs };
   }
   if (options.binaryPath) {
@@ -18244,7 +18303,7 @@ function spawnCodexProcess(options) {
     env.POSTHOG_GATEWAY_API_KEY = options.apiKey;
   }
   const { command, args: args2 } = findCodexBinary(options);
-  if (options.binaryPath && (0, import_node_fs2.existsSync)(options.binaryPath)) {
+  if (options.binaryPath && (0, import_node_fs3.existsSync)(options.binaryPath)) {
     const binDir = (0, import_node_path4.dirname)(options.binaryPath);
     env.PATH = `${binDir}${import_node_path4.delimiter}${env.PATH ?? ""}`;
   }
@@ -18256,7 +18315,7 @@ function spawnCodexProcess(options) {
     hasApiKey: !!options.apiKey,
     binaryPath: options.binaryPath
   });
-  const child = (0, import_node_child_process3.spawn)(command, args2, {
+  const child = (0, import_node_child_process4.spawn)(command, args2, {
     cwd: options.cwd,
     env,
     stdio: ["pipe", "pipe", "pipe"],
@@ -18315,6 +18374,17 @@ function prependPrContext(params) {
     prompt: [{ type: "text", text: prContext }, ...params.prompt]
   };
 }
+function classifyPromptError(error) {
+  const message = error instanceof Error ? error.message : String(error ?? "");
+  const classification = classifyAgentError(message);
+  if (classification === "agent_error") {
+    return error;
+  }
+  return import_sdk3.RequestError.internalError(
+    { classification, result: message },
+    message
+  );
+}
 var CODEX_NATIVE_MODE = {
   auto: "auto",
   default: "auto",
@@ -18345,7 +18415,7 @@ function resolveStructuredOutputMcpScript() {
   let dir = import_meta2.dirname ?? __dirname;
   for (let i2 = 0; i2 < 5; i2++) {
     const candidate = (0, import_node_path5.resolve)(dir, rel);
-    if ((0, import_node_fs3.existsSync)(candidate)) return candidate;
+    if ((0, import_node_fs4.existsSync)(candidate)) return candidate;
     dir = (0, import_node_path5.resolve)(dir, "..");
   }
   throw new Error(
@@ -18644,6 +18714,8 @@ var CodexAcpAgent = class extends BaseAcpAgent {
     let response;
     try {
       response = await this.codexConnection.prompt(prependPrContext(params));
+    } catch (error) {
+      throw classifyPromptError(error);
     } finally {
       this.session.promptRunning = false;
     }
@@ -18979,7 +19051,7 @@ var import_node_os2 = require("os");
 var import_node_path7 = require("path");
 
 // ../git/dist/handoff.js
-var import_node_child_process4 = require("child_process");
+var import_node_child_process5 = require("child_process");
 var import_promises2 = require("fs/promises");
 var import_node_os = require("os");
 var import_node_path6 = __toESM(require("path"), 1);
@@ -19145,7 +19217,7 @@ var GitSaga = class extends Saga {
     return manager.executeWrite(input.baseDir, async (git) => {
       this._git = git;
       return this.executeGitOperations(input);
-    }, { signal: input.signal });
+    }, { signal: input.signal, env: input.env });
   }
 };
 
@@ -19257,42 +19329,7 @@ async function hasUnmergedEntries(git) {
   return output.trim().length > 0;
 }
 async function getGitBusyState(git) {
-  const toplevel = (await git.raw(["rev-parse", "--show-toplevel"])).trim();
-  const resolveGitPath = async (gitPath) => {
-    const relative = (await git.raw(["rev-parse", "--git-path", gitPath])).trim();
-    return path13.isAbsolute(relative) ? relative : path13.resolve(toplevel, relative);
-  };
-  const pathExists = async (gitPath) => {
-    const resolved = await resolveGitPath(gitPath);
-    try {
-      await fs11.access(resolved);
-      return true;
-    } catch {
-      return false;
-    }
-  };
-  const dirExists = async (gitPath) => {
-    const resolved = await resolveGitPath(gitPath);
-    try {
-      const stat4 = await fs11.stat(resolved);
-      return stat4.isDirectory();
-    } catch {
-      return false;
-    }
-  };
-  if (await dirExists("rebase-merge") || await dirExists("rebase-apply")) {
-    return { busy: true, operation: "rebase" };
-  }
-  if (await pathExists("MERGE_HEAD")) {
-    return { busy: true, operation: "merge" };
-  }
-  if (await pathExists("CHERRY_PICK_HEAD")) {
-    return { busy: true, operation: "cherry-pick" };
-  }
-  if (await pathExists("REVERT_HEAD")) {
-    return { busy: true, operation: "revert" };
-  }
-  return { busy: false };
+  return inspectGitBusyState(git);
 }
 var MAX_WORKTREE_FILE_BYTES = 1024 * 1024;
 async function createWorktreeTree(git, baseDir, head) {
@@ -19794,7 +19831,7 @@ var GitHandoffTracker = class {
   }
   async runGitProcessAllowingFailure(args2) {
     return new Promise((resolve7, reject) => {
-      const child = (0, import_node_child_process4.spawn)("git", args2, {
+      const child = (0, import_node_child_process5.spawn)("git", args2, {
         cwd: this.repositoryPath,
         stdio: ["ignore", "ignore", "pipe"]
       });
@@ -19818,7 +19855,7 @@ var GitHandoffTracker = class {
   }
   async runGitWithEnv(env, args2) {
     return new Promise((resolve7, reject) => {
-      const child = (0, import_node_child_process4.spawn)("git", args2, {
+      const child = (0, import_node_child_process5.spawn)("git", args2, {
         cwd: this.repositoryPath,
         stdio: ["ignore", "pipe", "pipe"],
         env
@@ -19843,7 +19880,7 @@ var GitHandoffTracker = class {
   }
   runGitProcess(args2, input) {
     return new Promise((resolve7, reject) => {
-      const child = (0, import_node_child_process4.spawn)("git", args2, {
+      const child = (0, import_node_child_process5.spawn)("git", args2, {
         cwd: this.repositoryPath,
         stdio: "pipe"
       });
@@ -20695,7 +20732,7 @@ ${toolSummary}`);
 }
 
 // src/session-log-writer.ts
-var import_node_fs4 = __toESM(require("fs"), 1);
+var import_node_fs5 = __toESM(require("fs"), 1);
 var import_promises4 = __toESM(require("fs/promises"), 1);
 var import_node_path8 = __toESM(require("path"), 1);
 var SessionLogWriter = class _SessionLogWriter {
@@ -20739,7 +20776,7 @@ var SessionLogWriter = class _SessionLogWriter {
         context.runId
       );
       try {
-        import_node_fs4.default.mkdirSync(sessionDir, { recursive: true });
+        import_node_fs5.default.mkdirSync(sessionDir, { recursive: true });
       } catch (error) {
         this.logger.warn("Failed to create local cache directory", {
           sessionDir,
@@ -20998,7 +21035,7 @@ var SessionLogWriter = class _SessionLogWriter {
       "logs.ndjson"
     );
     try {
-      import_node_fs4.default.appendFileSync(logPath, `${JSON.stringify(entry)}
+      import_node_fs5.default.appendFileSync(logPath, `${JSON.stringify(entry)}
 `);
     } catch (error) {
       this.logger.warn("Failed to write to local cache", {
@@ -21033,11 +21070,11 @@ var SessionLogWriter = class _SessionLogWriter {
 };
 
 // src/server/agentsh-runtime.ts
-var import_node_child_process5 = require("child_process");
+var import_node_child_process6 = require("child_process");
 var import_promises5 = require("fs/promises");
 var import_node_util2 = require("util");
 var AGENTSH_SESSION_ID_FILE = "/tmp/agentsh-session-id";
-var execFileAsync2 = (0, import_node_util2.promisify)(import_node_child_process5.execFile);
+var execFileAsync2 = (0, import_node_util2.promisify)(import_node_child_process6.execFile);
 function errorMessage(error) {
   return error instanceof Error ? error.message : String(error);
 }
@@ -21257,8 +21294,15 @@ function validateCommandParams(method, params) {
 var agentErrorClassificationSchema = import_zod3.z.enum([
   "upstream_stream_terminated",
   "upstream_connection_error",
+  "upstream_provider_failure",
   "agent_error"
 ]);
+var UPSTREAM_PROVIDER_FAILURE_MESSAGE = "The upstream AI provider failed to process the request. Please retry the task in a few minutes.";
+var upstreamProviderFailureClassifications = /* @__PURE__ */ new Set([
+  "upstream_stream_terminated",
+  "upstream_connection_error",
+  "upstream_provider_failure"
+]);
 var errorWithClassificationSchema = import_zod3.z.object({
   data: import_zod3.z.object({ classification: agentErrorClassificationSchema })
 });
@@ -22002,7 +22046,9 @@ var AgentServer = class {
   }
   classifyAndSignalFailure(payload, phase, error) {
     const { classification, message } = this.extractErrorClassification(error);
-    const errorMessage2 = classification === "upstream_stream_terminated" ? "Upstream LLM stream terminated" : classification === "upstream_connection_error" ? "Upstream LLM connection error" : message || "Agent error";
+    const errorMessage2 = upstreamProviderFailureClassifications.has(
+      classification
+    ) ? UPSTREAM_PROVIDER_FAILURE_MESSAGE : message || "Agent error";
     this.logger.error(`send_${phase}_task_message_failed`, {
       classification,
       message
@@ -22440,9 +22486,14 @@ After completing the requested changes:
 1. Check out the existing PR branch with \`gh pr checkout ${prUrl}\`
 2. Stage and commit all changes with a clear commit message
 3. Push to the existing PR branch
+4. For every PR review comment or review thread you addressed, treat the thread as done only after BOTH of these:
+   - Reply on the thread with a short note describing what changed (reference the commit SHA when useful) using \`gh api -X POST /repos/{owner}/{repo}/pulls/{n}/comments/{id}/replies -f body='...'\`.
+   - Resolve the thread via the \`resolveReviewThread\` GraphQL mutation: \`gh api graphql -f query='mutation($id:ID!){resolveReviewThread(input:{threadId:$id}){thread{isResolved}}}' -f id="<thread-node-id>"\`.
+   List unresolved threads first with \`gh api graphql -f query='{repository(owner:"<owner>",name:"<repo>"){pullRequest(number:<n>){reviewThreads(first:100){nodes{id isResolved comments(first:1){nodes{body}}}}}}}'\` so you can resolve each one you fixed.
 
 Important:
 - Do NOT create a new branch or a new pull request.
+- Do NOT push fixes for review comments without replying to and resolving each related thread.
 ${attributionInstructions}
 `;
     }
@@ -22454,7 +22505,7 @@ When the user asks for code changes:
 When the user explicitly asks to clone or work in a GitHub repository:
 - Clone the repository into /tmp/workspace/repos/<owner>/<repo> using \`gh repo clone <owner>/<repo> /tmp/workspace/repos/<owner>/<repo>\`
 - Work from inside that cloned repository for follow-up code changes
-- If the user explicitly asks you to open or update a pull request, create a branch, commit the requested changes, push it, and open a draft pull request from inside the clone
+- If the user explicitly asks you to open or update a pull request, create a branch, commit the requested changes, push it, and open a draft pull request from inside the clone. Before opening the PR, check the cloned repo for a PR template at \`.github/pull_request_template.md\` (or variants; fall back to the org's \`.github\` repo via \`gh api\`) and use it as the body structure, and search for matching open issues with \`gh issue list --search\` to include \`Closes #<n>\` / \`Refs #<n>\` links.
 - Do NOT create branches, commits, push changes, or open pull requests unless the user explicitly asks for that`;
       return `
 # Cloud Task Execution \u2014 No Repository Mode
@@ -22494,7 +22545,11 @@ After completing the requested changes:
 1. Create a new branch prefixed with \`posthog-code/\` (e.g. \`posthog-code/fix-login-redirect\`) based on the work done
 2. Stage and commit all changes with a clear commit message
 3. Push the branch to origin
-4. Create a draft pull request using \`gh pr create --draft${this.config.baseBranch ? ` --base ${this.config.baseBranch}` : ""}\` with a descriptive title and body. Add the following footer at the end of the PR description:
+4. Before opening the PR, prepare the body:
+   - Check the repo for a PR template at \`.github/pull_request_template.md\` (also try \`.github/PULL_REQUEST_TEMPLATE.md\`, \`docs/pull_request_template.md\`, and root variants). If one exists, use its exact section headings as the PR body \u2014 do NOT fall back to a generic Summary/Test plan format.
+   - If no repo-level template exists, check the org's \`.github\` repo via \`gh api /repos/<owner>/.github/contents/.github/pull_request_template.md\` (and other common paths) and use that as a fallback.
+   - Search for matching open issues with \`gh issue list --state open --search '<keywords>'\` (derive keywords from the branch name, commits, and changed files; \`gh issue view <n>\` to confirm relevance). For every issue this PR would resolve, include a \`Closes #<n>\` line in the body so GitHub auto-links and auto-closes it on merge. For issues that are related but not fully resolved, use \`Refs #<n>\` instead.
+5. Create a draft pull request using \`gh pr create --draft${this.config.baseBranch ? ` --base ${this.config.baseBranch}` : ""}\` with a descriptive title and the body prepared above. Add the following footer at the end of the PR description:
 \`\`\`
 ---
 *Created with [PostHog Code](https://posthog.com/code?ref=pr)*