@posthog/agent 2.3.159 → 2.3.167

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@posthog/agent",
-  "version": "2.3.159",
+  "version": "2.3.167",
   "repository": "https://github.com/PostHog/code",
   "description": "TypeScript agent framework wrapping Claude Agent SDK with Git-based task execution for PostHog",
   "exports": {
@@ -103,7 +103,7 @@
     "tar": "^7.5.0",
     "uuid": "13.0.0",
     "yoga-wasm-web": "^0.3.3",
-    "zod": "^3.24.1"
+    "zod": "^4.2.0"
   },
   "files": [
     "dist/**/*",

package/src/adapters/claude/hooks.ts

@@ -71,6 +71,56 @@ export const createPostToolUseHook =
     return { continue: true };
   };
 
+/**
+ * Rewrites Agent tool calls targeting built-in subagent types to use our custom
+ * definitions instead. This works around a Claude Agent SDK bug where
+ * `options.agents` cannot override built-in agent definitions because the
+ * built-ins appear first in the agents array and `Array.find()` returns the
+ * first match.
+ *
+ * By giving our custom agent a different name (e.g. "ph-explore") and rewriting
+ * the subagent_type in the tool input, we sidestep the collision entirely.
+ *
+ * https://github.com/anthropics/claude-agent-sdk-typescript/issues/267
+ */
+const SUBAGENT_REWRITES: Record<string, string> = {
+  Explore: "ph-explore",
+};
+
+export const createSubagentRewriteHook =
+  (logger: Logger): HookCallback =>
+  async (input: HookInput, _toolUseID: string | undefined) => {
+    if (input.hook_event_name !== "PreToolUse") {
+      return { continue: true };
+    }
+
+    if (input.tool_name !== "Agent") {
+      return { continue: true };
+    }
+
+    const toolInput = input.tool_input as Record<string, unknown> | undefined;
+    const subagentType = toolInput?.subagent_type;
+    if (typeof subagentType !== "string" || !SUBAGENT_REWRITES[subagentType]) {
+      return { continue: true };
+    }
+
+    const target = SUBAGENT_REWRITES[subagentType];
+    logger.info(
+      `[SubagentRewriteHook] Rewriting subagent_type: ${subagentType} → ${target}`,
+    );
+
+    return {
+      continue: true,
+      hookSpecificOutput: {
+        hookEventName: "PreToolUse" as const,
+        updatedInput: {
+          ...toolInput,
+          subagent_type: target,
+        },
+      },
+    };
+  };
+
 export const createPreToolUseHook =
   (settingsManager: SettingsManager, logger: Logger): HookCallback =>
   async (input: HookInput, _toolUseID: string | undefined) => {

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

@@ -14,6 +14,7 @@ import type { Logger } from "../../../utils/logger";
 import {
   createPostToolUseHook,
   createPreToolUseHook,
+  createSubagentRewriteHook,
   type OnModeChange,
 } from "../hooks";
 import type { CodeExecutionMode } from "../tools";
@@ -117,7 +118,10 @@ function buildHooks(
     PreToolUse: [
       ...(userHooks?.PreToolUse || []),
       {
-        hooks: [createPreToolUseHook(settingsManager, logger)],
+        hooks: [
+          createPreToolUseHook(settingsManager, logger),
+          createSubagentRewriteHook(logger),
+        ],
       },
     ],
   };

package/src/server/bin.ts

@@ -7,25 +7,21 @@ import { claudeCodeConfigSchema, mcpServersSchema } from "./schemas";
 const envSchema = z.object({
   JWT_PUBLIC_KEY: z
     .string({
-      required_error:
-        "JWT_PUBLIC_KEY is required for authenticating client connections",
+      error: "JWT_PUBLIC_KEY is required for authenticating client connections",
     })
     .min(1, "JWT_PUBLIC_KEY cannot be empty"),
-  POSTHOG_API_URL: z
-    .string({
-      required_error:
-        "POSTHOG_API_URL is required for LLM gateway communication",
-    })
-    .url("POSTHOG_API_URL must be a valid URL"),
+  POSTHOG_API_URL: z.url({
+    error: "POSTHOG_API_URL is required for LLM gateway communication",
+  }),
   POSTHOG_PERSONAL_API_KEY: z
     .string({
-      required_error:
+      error:
         "POSTHOG_PERSONAL_API_KEY is required for authenticating with PostHog services",
     })
     .min(1, "POSTHOG_PERSONAL_API_KEY cannot be empty"),
   POSTHOG_PROJECT_ID: z
     .string({
-      required_error:
+      error:
         "POSTHOG_PROJECT_ID is required for routing requests to the correct project",
     })
     .regex(/^\d+$/, "POSTHOG_PROJECT_ID must be a numeric string")
@@ -34,7 +30,7 @@ const envSchema = z.object({
 
 const program = new Command();
 
-function parseJsonOption<S extends z.ZodTypeAny>(
+function parseJsonOption<S extends z.ZodType>(
   raw: string | undefined,
   schema: S,
   flag: string,

package/src/server/schemas.ts

@@ -8,7 +8,7 @@ const httpHeaderSchema = z.object({
 const remoteMcpServerSchema = z.object({
   type: z.enum(["http", "sse"]),
   name: z.string().min(1, "MCP server name is required"),
-  url: z.string().url("MCP server url must be a valid URL"),
+  url: z.url({ error: "MCP server url must be a valid URL" }),
   headers: z.array(httpHeaderSchema).default([]),
 });
 
@@ -35,7 +35,7 @@ export const claudeCodeConfigSchema = z.object({
 export const jsonRpcRequestSchema = z.object({
   jsonrpc: z.literal("2.0"),
   method: z.string(),
-  params: z.record(z.unknown()).optional(),
+  params: z.record(z.string(), z.unknown()).optional(),
   id: z.union([z.string(), z.number()]).optional(),
 });