@posthog/agent 1.26.0 → 1.28.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@posthog/agent",
-  "version": "1.26.0",
+  "version": "1.28.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/agent.ts

@@ -4,9 +4,6 @@ import {
   type ContentBlock,
   ndJsonStream,
   PROTOCOL_VERSION,
-  type RequestPermissionRequest,
-  type RequestPermissionResponse,
-  type SessionNotification,
 } from "@agentclientprotocol/sdk";
 import { POSTHOG_NOTIFICATIONS } from "./acp-extensions.js";
 import {
@@ -643,12 +640,12 @@ Generated by PostHog Agent`;
           if (!isPolling) break;
 
           try {
-            const newEntries = await this.sessionStore!.pollForNewEntries(
+            const newEntries = await this.sessionStore?.pollForNewEntries(
               taskRunId,
               lastKnownEntryCount,
             );
 
-            for (const entry of newEntries) {
+            for (const entry of newEntries ?? []) {
               lastKnownEntryCount++;
               // Look for user_message notifications
               if (
@@ -692,6 +689,46 @@ Generated by PostHog Agent`;
         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 =

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

@@ -686,6 +686,51 @@ export class WorktreeManager {
   }
 
   async deleteWorktree(worktreePath: string): Promise<void> {
+    const resolvedWorktreePath = path.resolve(worktreePath);
+    const resolvedMainRepoPath = path.resolve(this.mainRepoPath);
+
+    // Safety check 1: Never delete the main repo path
+    if (resolvedWorktreePath === resolvedMainRepoPath) {
+      const error = new Error(
+        "Cannot delete worktree: path matches main repo path",
+      );
+      this.logger.error("Safety check failed", { worktreePath, error });
+      throw error;
+    }
+
+    // Safety check 2: Never delete a parent of the main repo path
+    if (
+      resolvedMainRepoPath.startsWith(resolvedWorktreePath) &&
+      resolvedMainRepoPath !== resolvedWorktreePath
+    ) {
+      const error = new Error(
+        "Cannot delete worktree: path is a parent of main repo path",
+      );
+      this.logger.error("Safety check failed", { worktreePath, error });
+      throw error;
+    }
+
+    // Safety check 3: Check for .git directory (indicates main repo)
+    try {
+      const gitPath = path.join(resolvedWorktreePath, ".git");
+      const stat = await fs.stat(gitPath);
+      if (stat.isDirectory()) {
+        const error = new Error(
+          "Cannot delete worktree: path appears to be a main repository (contains .git directory)",
+        );
+        this.logger.error("Safety check failed", { worktreePath, error });
+        throw error;
+      }
+    } catch (error) {
+      // If .git doesn't exist or we can't read it, proceed (unless it was the directory check above)
+      if (
+        error instanceof Error &&
+        error.message.includes("Cannot delete worktree")
+      ) {
+        throw error;
+      }
+    }
+
     this.logger.info("Deleting worktree", { worktreePath });
 
     try {

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/_tslib.js

@@ -1,74 +0,0 @@
-/******************************************************************************
-Copyright (c) Microsoft Corporation.
-
-Permission to use, copy, modify, and/or distribute this software for any
-purpose with or without fee is hereby granted.
-
-THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES WITH
-REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF MERCHANTABILITY
-AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR ANY SPECIAL, DIRECT,
-INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES WHATSOEVER RESULTING FROM
-LOSS OF USE, DATA OR PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR
-OTHER TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR
-PERFORMANCE OF THIS SOFTWARE.
-***************************************************************************** */
-/* global Reflect, Promise, SuppressedError, Symbol, Iterator */
-
-
-function __addDisposableResource(env, value, async) {
-    if (value !== null && value !== void 0) {
-        if (typeof value !== "object" && typeof value !== "function") throw new TypeError("Object expected.");
-        var dispose, inner;
-        if (async) {
-            if (!Symbol.asyncDispose) throw new TypeError("Symbol.asyncDispose is not defined.");
-            dispose = value[Symbol.asyncDispose];
-        }
-        if (dispose === void 0) {
-            if (!Symbol.dispose) throw new TypeError("Symbol.dispose is not defined.");
-            dispose = value[Symbol.dispose];
-            if (async) inner = dispose;
-        }
-        if (typeof dispose !== "function") throw new TypeError("Object not disposable.");
-        if (inner) dispose = function() { try { inner.call(this); } catch (e) { return Promise.reject(e); } };
-        env.stack.push({ value: value, dispose: dispose, async: async });
-    }
-    else if (async) {
-        env.stack.push({ async: true });
-    }
-    return value;
-
-}
-
-var _SuppressedError = typeof SuppressedError === "function" ? SuppressedError : function (error, suppressed, message) {
-    var e = new Error(message);
-    return e.name = "SuppressedError", e.error = error, e.suppressed = suppressed, e;
-};
-
-function __disposeResources(env) {
-    function fail(e) {
-        env.error = env.hasError ? new _SuppressedError(e, env.error, "An error was suppressed during disposal.") : e;
-        env.hasError = true;
-    }
-    var r, s = 0;
-    function next() {
-        while (r = env.stack.pop()) {
-            try {
-                if (!r.async && s === 1) return s = 0, env.stack.push(r), Promise.resolve().then(next);
-                if (r.dispose) {
-                    var result = r.dispose.call(r.value);
-                    if (r.async) return s |= 2, Promise.resolve(result).then(next, function(e) { fail(e); return next(); });
-                }
-                else s |= 1;
-            }
-            catch (e) {
-                fail(e);
-            }
-        }
-        if (s === 1) return env.hasError ? Promise.reject(env.error) : Promise.resolve();
-        if (env.hasError) throw env.error;
-    }
-    return next();
-}
-
-export { __addDisposableResource, __disposeResources };
-//# sourceMappingURL=_tslib.js.map

package/dist/_virtual/_tslib.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"_tslib.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":";;;;"}

package/dist/_virtual/errors.js

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

package/dist/_virtual/errors.js.map

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

package/dist/_virtual/format.js

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

package/dist/_virtual/format.js.map

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

package/dist/_virtual/formats.js

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

package/dist/_virtual/formats.js.map

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

package/dist/_virtual/id.js

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

package/dist/_virtual/id.js.map

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

package/dist/_virtual/if.js

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

package/dist/_virtual/if.js.map

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

package/dist/_virtual/index.js

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

package/dist/_virtual/index.js.map

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

package/dist/_virtual/index10.js

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

package/dist/_virtual/index10.js.map

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

package/dist/_virtual/index11.js

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

package/dist/_virtual/index11.js.map

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

package/dist/_virtual/index2.js

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

package/dist/_virtual/index2.js.map

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

package/dist/_virtual/index3.js

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

package/dist/_virtual/index3.js.map

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

package/dist/_virtual/index4.js

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

package/dist/_virtual/index4.js.map

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

package/dist/_virtual/index5.js

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

package/dist/_virtual/index5.js.map

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

package/dist/_virtual/index6.js

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

package/dist/_virtual/index6.js.map

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

package/dist/_virtual/index7.js

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

package/dist/_virtual/index7.js.map

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

package/dist/_virtual/index8.js

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

package/dist/_virtual/index8.js.map

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

package/dist/_virtual/index9.js

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

package/dist/_virtual/index9.js.map

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

package/dist/_virtual/items.js

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

package/dist/_virtual/items.js.map

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

package/dist/_virtual/items2020.js

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

package/dist/_virtual/items2020.js.map

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

package/dist/_virtual/keyword.js

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

package/dist/_virtual/keyword.js.map

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