chore: merge main into MCP transport extraction

Preserve the shared MCP boundary while adopting main's schema-validated automation transport and reasoning coverage.

Generated-By: PostHog Code
Task-Id: c1bbe3cf-742b-4b24-bf96-d11a18b4cf22

Author: richardsolomou

packages/api-client/src/posthog-client.ts

@@ -4,22 +4,30 @@ import type {
   CloudMcpServerImport,
   CloudMcpServerRelayDesignation,
   CloudRunSource,
+  CreateTaskAutomationOptions,
   ExecutionMode,
   PrAuthorshipMode,
   SourceProduct,
   SourceType,
   StoredLogEntry,
+  TaskAutomation,
   TaskRunArtifactMetadata,
+  UpdateTaskAutomationOptions,
 } from "@posthog/shared";
 import {
   buildCloudTaskConfigOptions,
   type CloudTaskConfigOption,
+  createTaskAutomationSchema,
   DISMISSAL_REASON_OPTIONS,
   type DismissalReasonOptionValue,
   getCloudTaskGatewayUrl,
   isSupportedReasoningEffort,
   normalizeGatewayModelsResponse,
   resolveCloudInitialPermissionMode,
+  taskAutomationListSchema,
+  taskAutomationSchema,
+  taskAutomationValidationErrorSchema,
+  updateTaskAutomationSchema,
 } from "@posthog/shared";
 import type {
   AgentAnalyticsData,
@@ -202,22 +210,11 @@ export class TaskAutomationValidationError extends Error {
 
 function rethrowTaskAutomationError(error: unknown): never {
   if (error instanceof ApiRequestError && error.status === 400) {
-    const body = error.body;
-    if (
-      typeof body === "object" &&
-      body !== null &&
-      "detail" in body &&
-      typeof body.detail === "string"
-    ) {
-      throw new TaskAutomationValidationError({
-        detail: body.detail,
-        code:
-          "code" in body && typeof body.code === "string"
-            ? body.code
-            : "invalid_input",
-        attr:
-          "attr" in body && typeof body.attr === "string" ? body.attr : null,
-      });
+    const validationError = taskAutomationValidationErrorSchema.safeParse(
+      error.body,
+    );
+    if (validationError.success) {
+      throw new TaskAutomationValidationError(validationError.data);
     }
   }
 
@@ -253,41 +250,6 @@ export type {
 
 export type Evaluation = Schemas.Evaluation;
 
-export type TaskAutomation = Omit<
-  Schemas.TaskAutomation,
-  "github_integration" | "timezone" | "template_id" | "enabled"
-> & {
-  github_integration: number | null;
-  timezone: string | null;
-  template_id: string | null;
-  enabled: boolean;
-};
-
-export type CreateTaskAutomationOptions = Pick<
-  Schemas.TaskAutomation,
-  "name" | "prompt" | "repository" | "cron_expression"
-> &
-  Partial<
-    Pick<
-      Schemas.TaskAutomation,
-      "github_integration" | "template_id" | "enabled"
-    >
-  > & { timezone: string };
-
-export type UpdateTaskAutomationOptions = Partial<CreateTaskAutomationOptions>;
-
-function normalizeTaskAutomation(
-  automation: Schemas.TaskAutomation,
-): TaskAutomation {
-  return {
-    ...automation,
-    github_integration: automation.github_integration ?? null,
-    timezone: automation.timezone ?? null,
-    template_id: automation.template_id ?? null,
-    enabled: automation.enabled ?? true,
-  };
-}
-
 export interface UserGitHubIntegration {
   id: string;
   kind: "github";
@@ -2524,7 +2486,7 @@ export class PostHogAPIClient {
       },
     );
 
-    return data.results.map(normalizeTaskAutomation);
+    return taskAutomationListSchema.parse(data).results;
   }
 
   async getTaskAutomation(automationId: string): Promise<TaskAutomation> {
@@ -2536,22 +2498,24 @@ export class PostHogAPIClient {
       },
     );
 
-    return normalizeTaskAutomation(data);
+    return taskAutomationSchema.parse(data);
   }
 
   async createTaskAutomation(
     options: CreateTaskAutomationOptions,
   ): Promise<TaskAutomation> {
     const teamId = await this.getTeamId();
+    const body = createTaskAutomationSchema.parse(options);
+
     try {
       const data = await this.api.post(
         `/api/projects/{project_id}/task_automations/`,
         {
           path: { project_id: teamId.toString() },
-          body: options as Schemas.TaskAutomation,
+          body: body as Schemas.TaskAutomation,
         },
       );
-      return normalizeTaskAutomation(data);
+      return taskAutomationSchema.parse(data);
     } catch (error) {
       rethrowTaskAutomationError(error);
     }
@@ -2562,15 +2526,17 @@ export class PostHogAPIClient {
     updates: UpdateTaskAutomationOptions,
   ): Promise<TaskAutomation> {
     const teamId = await this.getTeamId();
+    const body = updateTaskAutomationSchema.parse(updates);
+
     try {
       const data = await this.api.patch(
         `/api/projects/{project_id}/task_automations/{id}/`,
         {
           path: { project_id: teamId.toString(), id: automationId },
-          body: updates,
+          body,
         },
       );
-      return normalizeTaskAutomation(data);
+      return taskAutomationSchema.parse(data);
     } catch (error) {
       rethrowTaskAutomationError(error);
     }
@@ -2593,9 +2559,7 @@ export class PostHogAPIClient {
         path,
         url: new URL(`${this.api.baseUrl}${path}`),
       });
-      return normalizeTaskAutomation(
-        (await response.json()) as Schemas.TaskAutomation,
-      );
+      return taskAutomationSchema.parse(await response.json());
     } catch (error) {
       rethrowTaskAutomationError(error);
     }
@@ -2640,13 +2604,16 @@ export class PostHogAPIClient {
     return normalizeTaskResponse(data, { teamId });
   }
 
-  async updateTask(taskId: string, updates: Partial<Task>): Promise<Task> {
+  async updateTask(
+    taskId: string,
+    updates: Partial<Schemas.Task>,
+  ): Promise<Task> {
     const teamId = await this.getTeamId();
     const data = await this.api.patch(
       `/api/projects/{project_id}/tasks/{id}/`,
       {
         path: { project_id: teamId.toString(), id: taskId },
-        body: updates as unknown as Partial<Schemas.Task>,
+        body: updates,
       },
     );
 

packages/core/src/automations/automationTemplatePresentation.ts

@@ -1,4 +1,4 @@
-import type { TaskAutomation } from "@posthog/api-client/posthog-client";
+import type { TaskAutomation } from "@posthog/shared";
 
 export const SKILL_TEMPLATE_ID_PREFIX = "llm-skill:";
 

packages/core/src/context-menu/context-menu.test.ts

@@ -142,6 +142,17 @@ describe("ContextMenuService.showTaskContextMenu", () => {
     );
   });
 
+  it("can hide Archive prior tasks for task lists without that action", async () => {
+    const menu = new FakeContextMenu();
+    makeService(menu).showTaskContextMenu({
+      ...baseTask,
+      showArchivePrior: false,
+    });
+    await menu.shown;
+    expect(labels(menu.lastItems)).not.toContain("Archive prior tasks");
+    expect(labels(menu.lastItems)).toContain("Archive");
+  });
+
   it("resolves to null when the menu is dismissed", async () => {
     const menu = new FakeContextMenu();
     const result = makeService(menu).showTaskContextMenu(baseTask);

packages/core/src/context-menu/context-menu.ts

@@ -117,6 +117,7 @@ export class ContextMenuService {
       canStop,
       isInCommandCenter,
       hasEmptyCommandCenterCell,
+      showArchivePrior = true,
       channels,
     } = input;
     const { apps, lastUsedAppId } = await this.getExternalAppsData();
@@ -162,7 +163,7 @@ export class ContextMenuService {
       ...(!isInCommandCenter
         ? [
             this.separator(),
-            this.item(
+            this.item<TaskAction>(
               "Add to Command Center",
               { type: "add-to-command-center" as const },
               { enabled: hasEmptyCommandCenterCell ?? true },
@@ -172,19 +173,23 @@ export class ContextMenuService {
       ...fileToItems,
       this.separator(),
       this.item("Archive", { type: "archive" }),
-      this.item(
-        "Archive prior tasks",
-        { type: "archive-prior" },
-        {
-          confirm: {
-            title: "Archive Prior Tasks",
-            message: "Archive all tasks older than this one?",
-            detail:
-              "This will archive every task created before this one. You can unarchive them later.",
-            confirmLabel: "Archive",
-          },
-        },
-      ),
+      ...(showArchivePrior
+        ? [
+            this.item<TaskAction>(
+              "Archive prior tasks",
+              { type: "archive-prior" },
+              {
+                confirm: {
+                  title: "Archive Prior Tasks",
+                  message: "Archive all tasks older than this one?",
+                  detail:
+                    "This will archive every task created before this one. You can unarchive them later.",
+                  confirmLabel: "Archive",
+                },
+              },
+            ),
+          ]
+        : []),
     ]);
   }
 

packages/core/src/context-menu/schemas.ts

@@ -9,6 +9,7 @@ export const taskContextMenuInput = z.object({
   canStop: z.boolean().optional(),
   isInCommandCenter: z.boolean().optional(),
   hasEmptyCommandCenterCell: z.boolean().optional(),
+  showArchivePrior: z.boolean().optional(),
   // Top-level desktop_file_system channels available as "File to…" targets.
   // Omit (or pass empty) to hide the submenu entirely.
   channels: z.array(z.object({ id: z.string(), name: z.string() })).optional(),

packages/shared/src/reasoning-effort.test.ts

@@ -7,6 +7,8 @@ describe("isSupportedReasoningEffort", () => {
     ["codex", "gpt-5.6-sol", "max", true],
     ["codex", "gpt-5.4", "max", false],
     ["claude", "claude-opus-4-8", "xhigh", true],
+    ["claude", "claude-opus-5", "high", true],
+    ["claude", "claude-opus-5", "max", true],
     ["claude", "claude-sonnet-4-6", "xhigh", false],
     ["claude", "@cf/zai-org/glm-5.2", "high", true],
     ["claude", "@cf/zai-org/glm-5.2", "max", true],

packages/shared/src/reasoning-effort.ts

@@ -29,6 +29,7 @@ const CLAUDE_MODEL_EFFORTS: Readonly<
   "claude-sonnet-5": ["low", "medium", "high", "xhigh", "max"],
   "claude-fable-5": ["low", "medium", "high", "xhigh", "max"],
   "@cf/zai-org/glm-5.2": ["high", "max"],
+  "claude-opus-5": ["low", "medium", "high", "xhigh", "max"],
 };
 
 const EFFORT_NAMES: Record<SupportedReasoningEffort, string> = {

packages/ui/src/features/canvas/components/ActivityPanel.tsx

@@ -3,7 +3,7 @@ import {
   CaretRightIcon,
   XIcon,
 } from "@phosphor-icons/react";
-import { Button, cn, Tabs, TabsList, TabsTrigger } from "@posthog/quill";
+import { Button, Tabs, TabsList, TabsTrigger } from "@posthog/quill";
 import { ANALYTICS_EVENTS } from "@posthog/shared/analytics-events";
 import type { Task } from "@posthog/shared/domain-types";
 import { ActivityTimeline } from "@posthog/ui/features/canvas/components/ActivityTimeline";
@@ -35,9 +35,6 @@ const TABS_WITH_COMPOSER: ReadonlySet<ActivityTab> = new Set([
   "comments",
 ]);
 
-const TIMESTAMP_END_CLASS =
-  "[&_[data-slot=thread-item-timestamp]]:ml-auto [&_[data-slot=thread-item-timestamp]]:shrink-0 [&_[data-slot=thread-item-timestamp]]:pl-2";
-
 /** The 32px row this panel leads with: the tabs are the header, so the strip
  *  lines up with the tab bar of the pane on its left (TabbedPanel) and the
  *  review toolbar, which are the same fixed height and border. */
@@ -119,13 +116,15 @@ function ActivityConversation({
   onToggleCollapsed,
   onOpenFull,
   showTaskSummary,
+  canOpenInPlace,
 }: {
   task: Task;
   channelId: string;
   onClose?: () => void;
   onToggleCollapsed?: () => void;
   onOpenFull?: () => void;
   showTaskSummary: boolean;
+  canOpenInPlace?: boolean;
 }) {
   const taskId = task.id;
   const {
@@ -183,7 +182,13 @@ function ActivityConversation({
 
   const body = () => {
     if (tab === "artifacts") {
-      return <TaskArtifactsList task={task} timeline={timeline} />;
+      return (
+        <TaskArtifactsList
+          task={task}
+          timeline={timeline}
+          canOpenInPlace={canOpenInPlace}
+        />
+      );
     }
     if (tab === "comments") {
       return (
@@ -209,19 +214,15 @@ function ActivityConversation({
         currentUserEmail={currentUser?.email}
         isTaskAuthor={isTaskAuthor}
         canForward={canForward}
+        canOpenInPlace={canOpenInPlace}
         onSendToAgent={sendMessageToAgent}
         onDelete={deleteMessage}
       />
     );
   };
 
   return (
-    <div
-      className={cn(
-        "flex h-full min-w-0 flex-col bg-gray-1",
-        TIMESTAMP_END_CLASS,
-      )}
-    >
+    <div className="flex h-full min-w-0 flex-col bg-gray-1">
       <ActivityHeader
         tab={tab}
         onTabChange={handleTabChange}
@@ -269,6 +270,7 @@ export function ActivityPanel({
   onToggleCollapsed,
   onOpenFull,
   showTaskSummary = true,
+  canOpenInPlace,
 }: {
   taskId: string;
   channelId: string;
@@ -278,6 +280,7 @@ export function ActivityPanel({
   onToggleCollapsed?: () => void;
   onOpenFull?: () => void;
   showTaskSummary?: boolean;
+  canOpenInPlace?: boolean;
 }) {
   const { data: fetchedTask } = useQuery({
     ...taskDetailQuery(taskId),
@@ -315,6 +318,7 @@ export function ActivityPanel({
       onToggleCollapsed={onToggleCollapsed}
       onOpenFull={onOpenFull}
       showTaskSummary={showTaskSummary}
+      canOpenInPlace={canOpenInPlace}
     />
   );
 }