fix(posthog): remove dead response.ok checks, fix real schema/endpoint bugs

The prior commit added if (!response.ok) branches inside transformResponse
across ~40 tool files. This is dead code — tools/index.ts already throws
on non-2xx (and error-payload) responses before transformResponse is ever
invoked, so transformResponse only ever receives already-successful
responses. Reverted to the idiomatic no-error-branch pattern used
elsewhere in the codebase (e.g. hunter/email_verifier.ts), and added a
posthog-errors errorExtractor so the framework's own error throw surfaces
PostHog's real {type, code, detail, attr} error shape instead of a
generic status message.

Verified against PostHog's live OpenAPI schema, fixed real bugs:
- delete_feature_flag: DELETE always returns 405 (hard delete not
  allowed); switched to PATCH with deleted: true
- delete_person: no single-person DELETE endpoint exists; switched to
  POST .../persons/bulk_delete/ with ids: [personId]
- create_annotation: insight_short_id is read-only on create; the
  writable field is dashboard_id
- removed nonexistent fields (experiment variants, insight
  filters/saved) not present in the current schema
- added missing trailing slashes on feature-flag/experiment URLs

Author: waleedlatif1

apps/sim/blocks/blocks/posthog.ts

@@ -717,8 +717,6 @@ Return ONLY the description text.`,
       condition: {
         field: 'operation',
         value: [
-          'posthog_create_insight',
-          'posthog_update_insight',
           'posthog_create_feature_flag',
           'posthog_update_feature_flag',
           'posthog_create_cohort',
@@ -930,13 +928,6 @@ Return ONLY the timestamp string - no explanations, no quotes, no extra text.`,
         value: ['posthog_create_experiment', 'posthog_update_experiment'],
       },
     },
-    {
-      id: 'variants',
-      title: 'Variants (JSON)',
-      type: 'long-input',
-      placeholder: '{"control": 50, "test": 50}',
-      condition: { field: 'operation', value: 'posthog_create_experiment' },
-    },
     {
       id: 'archived',
       title: 'Archived',

apps/sim/tools/error-extractors.ts

@@ -243,6 +243,17 @@ const ERROR_EXTRACTORS: ErrorExtractorConfig[] = [
       return undefined
     },
   },
+  {
+    id: 'posthog-errors',
+    description: 'PostHog API error format with type/code/detail/attr fields',
+    examples: ['PostHog API'],
+    extract: (errorInfo) => {
+      const detail = errorInfo?.data?.detail
+      if (typeof detail !== 'string' || !detail.trim()) return undefined
+      const attr = errorInfo?.data?.attr
+      return typeof attr === 'string' && attr ? `${detail} (${attr})` : detail
+    },
+  },
   {
     id: 'plain-text-data',
     description: 'Plain text error response',
@@ -320,6 +331,7 @@ export const ErrorExtractorId = {
   SOAP_FAULT: 'soap-fault',
   OAUTH_ERROR_DESCRIPTION: 'oauth-error-description',
   NESTED_ERROR_OBJECT: 'nested-error-object',
+  POSTHOG_ERRORS: 'posthog-errors',
   PLAIN_TEXT_DATA: 'plain-text-data',
   HTTP_STATUS_TEXT: 'http-status-text',
 } as const

apps/sim/tools/posthog/batch_events.ts

@@ -78,31 +78,19 @@ export const batchEventsTool: ToolConfig<PostHogBatchEventsParams, PostHogBatchE
   },
 
   transformResponse: async (response: Response, params) => {
-    if (response.ok) {
-      const data = await response.json()
-      let eventsProcessed = 0
-      try {
-        eventsProcessed = params ? JSON.parse(params.batch).length : 0
-      } catch {
-        eventsProcessed = 0
-      }
-      return {
-        success: true,
-        output: {
-          status: 'Batch events captured successfully',
-          events_processed: data.status === 1 ? eventsProcessed : 0,
-        },
-      }
+    const data = await response.json()
+    let eventsProcessed = 0
+    try {
+      eventsProcessed = params ? JSON.parse(params.batch).length : 0
+    } catch {
+      eventsProcessed = 0
     }
-
-    const error = await response.text()
     return {
-      success: false,
+      success: true,
       output: {
-        status: 'Failed to capture batch events',
-        events_processed: 0,
+        status: 'Batch events captured successfully',
+        events_processed: data.status === 1 ? eventsProcessed : 0,
       },
-      error: error || 'Unknown error occurred',
     }
   },
 

apps/sim/tools/posthog/capture_event.ts

@@ -109,22 +109,11 @@ export const captureEventTool: ToolConfig<PostHogCaptureEventParams, PostHogCapt
     },
 
     transformResponse: async (response: Response) => {
-      if (response.ok) {
-        return {
-          success: true,
-          output: {
-            status: 'Event captured successfully',
-          },
-        }
-      }
-
-      const error = await response.text()
       return {
-        success: false,
+        success: true,
         output: {
-          status: 'Failed to capture event',
+          status: 'Event captured successfully',
         },
-        error: error || 'Unknown error occurred',
       }
     },
 

apps/sim/tools/posthog/create_annotation.ts

@@ -10,7 +10,7 @@ interface PostHogCreateAnnotationParams {
   date_marker: string
   scope?: string
   dashboard_item?: string
-  insight_short_id?: string
+  dashboard_id?: string
 }
 
 interface PostHogCreateAnnotationResponse {
@@ -23,6 +23,7 @@ interface PostHogCreateAnnotationResponse {
     updated_at: string
     created_by: Record<string, any> | null
     dashboard_item: number | null
+    dashboard_id: number | null
     insight_short_id: string | null
     insight_name: string | null
     scope: string
@@ -39,6 +40,7 @@ export const createAnnotationTool: ToolConfig<
   description:
     'Create a new annotation in PostHog. Mark important events on your graphs with date and description.',
   version: '1.0.0',
+  errorExtractor: 'posthog-errors',
 
   params: {
     apiKey: {
@@ -84,19 +86,22 @@ export const createAnnotationTool: ToolConfig<
       type: 'string',
       required: false,
       visibility: 'user-or-llm',
-      description: 'Scope of the annotation: "project" or "dashboard_item" (default: "project")',
+      description:
+        'Scope of the annotation: "project", "organization", "dashboard", or "dashboard_item" (default: "project")',
     },
     dashboard_item: {
       type: 'string',
       required: false,
       visibility: 'user-or-llm',
-      description: 'ID of dashboard item to attach this annotation to',
+      description:
+        'ID of the dashboard tile (insight) to attach this annotation to (used when scope is "dashboard_item")',
     },
-    insight_short_id: {
+    dashboard_id: {
       type: 'string',
       required: false,
       visibility: 'user-or-llm',
-      description: 'Short ID of the insight to attach this annotation to',
+      description:
+        'ID of the dashboard to attach this annotation to (used when scope is "dashboard")',
     },
   },
 
@@ -124,36 +129,15 @@ export const createAnnotationTool: ToolConfig<
         body.dashboard_item = Number(params.dashboard_item)
       }
 
-      if (params.insight_short_id) {
-        body.insight_short_id = params.insight_short_id
+      if (params.dashboard_id) {
+        body.dashboard_id = Number(params.dashboard_id)
       }
 
       return body
     },
   },
 
   transformResponse: async (response: Response) => {
-    if (!response.ok) {
-      const error = await response.text()
-      return {
-        success: false,
-        output: {
-          id: 0,
-          content: '',
-          date_marker: '',
-          created_at: '',
-          updated_at: '',
-          created_by: null,
-          dashboard_item: null,
-          insight_short_id: null,
-          insight_name: null,
-          scope: '',
-          deleted: false,
-        },
-        error: error || 'Failed to create annotation',
-      }
-    }
-
     const data = await response.json()
 
     return {
@@ -166,6 +150,7 @@ export const createAnnotationTool: ToolConfig<
         updated_at: data.updated_at,
         created_by: data.created_by || null,
         dashboard_item: data.dashboard_item || null,
+        dashboard_id: data.dashboard_id || null,
         insight_short_id: data.insight_short_id || null,
         insight_name: data.insight_name || null,
         scope: data.scope || '',
@@ -205,6 +190,11 @@ export const createAnnotationTool: ToolConfig<
       description: 'ID of dashboard item this annotation is attached to',
       optional: true,
     },
+    dashboard_id: {
+      type: 'number',
+      description: 'ID of the dashboard this annotation is attached to',
+      optional: true,
+    },
     insight_short_id: {
       type: 'string',
       description: 'Short ID of the insight this annotation is attached to',
@@ -217,7 +207,7 @@ export const createAnnotationTool: ToolConfig<
     },
     scope: {
       type: 'string',
-      description: 'Scope of the annotation (project or dashboard_item)',
+      description: 'Scope of the annotation (project, organization, dashboard, or dashboard_item)',
     },
     deleted: {
       type: 'boolean',

apps/sim/tools/posthog/create_cohort.ts

@@ -40,6 +40,7 @@ export const createCohortTool: ToolConfig<PostHogCreateCohortParams, PostHogCrea
     description:
       'Create a new cohort in PostHog. Requires cohort name and filter or query configuration.',
     version: '1.0.0',
+    errorExtractor: 'posthog-errors',
 
     params: {
       apiKey: {
@@ -160,29 +161,6 @@ export const createCohortTool: ToolConfig<PostHogCreateCohortParams, PostHogCrea
     },
 
     transformResponse: async (response: Response) => {
-      if (!response.ok) {
-        const error = await response.text()
-        return {
-          success: false,
-          output: {
-            id: 0,
-            name: '',
-            description: '',
-            groups: [],
-            deleted: false,
-            filters: {},
-            query: null,
-            created_at: '',
-            created_by: null,
-            is_calculating: false,
-            count: 0,
-            is_static: false,
-            version: 0,
-          },
-          error: error || 'Failed to create cohort',
-        }
-      }
-
       const data = await response.json()
 
       return {

apps/sim/tools/posthog/create_dashboard.ts

@@ -36,6 +36,7 @@ export const createDashboardTool: ToolConfig<
   description:
     'Create a new dashboard in PostHog. Optionally seed it from a built-in template, then attach insights to it afterward.',
   version: '1.0.0',
+  errorExtractor: 'posthog-errors',
 
   params: {
     apiKey: {
@@ -128,24 +129,6 @@ export const createDashboardTool: ToolConfig<
   },
 
   transformResponse: async (response: Response) => {
-    if (!response.ok) {
-      const error = await response.text()
-      return {
-        success: false,
-        output: {
-          id: 0,
-          name: '',
-          description: '',
-          pinned: false,
-          created_at: '',
-          tiles: [],
-          filters: {},
-          tags: [],
-        },
-        error: error || 'Failed to create dashboard',
-      }
-    }
-
     const data = await response.json()
 
     return {

apps/sim/tools/posthog/create_experiment.ts

@@ -11,7 +11,6 @@ interface CreateExperimentParams {
   featureFlagKey: string
   parameters?: string
   filters?: string
-  variants?: string
   startDate?: string
   endDate?: string
 }
@@ -24,7 +23,6 @@ interface Experiment {
   feature_flag: Record<string, any>
   parameters: Record<string, any>
   filters: Record<string, any>
-  variants: Record<string, any>
   start_date: string | null
   end_date: string | null
   created_at: string
@@ -41,6 +39,7 @@ export const createExperimentTool: ToolConfig<CreateExperimentParams, CreateExpe
   name: 'PostHog Create Experiment',
   description: 'Create a new experiment in PostHog',
   version: '1.0.0',
+  errorExtractor: 'posthog-errors',
 
   params: {
     projectId: {
@@ -70,9 +69,9 @@ export const createExperimentTool: ToolConfig<CreateExperimentParams, CreateExpe
     },
     name: {
       type: 'string',
-      required: false,
+      required: true,
       visibility: 'user-or-llm',
-      description: 'Experiment name (optional)',
+      description: 'Experiment name',
     },
     description: {
       type: 'string',
@@ -98,12 +97,6 @@ export const createExperimentTool: ToolConfig<CreateExperimentParams, CreateExpe
       visibility: 'user-or-llm',
       description: 'Experiment filters as JSON string',
     },
-    variants: {
-      type: 'string',
-      required: false,
-      visibility: 'user-or-llm',
-      description: 'Experiment variants as JSON string',
-    },
     startDate: {
       type: 'string',
       required: false,
@@ -154,14 +147,6 @@ export const createExperimentTool: ToolConfig<CreateExperimentParams, CreateExpe
         }
       }
 
-      if (params.variants) {
-        try {
-          body.variants = JSON.parse(params.variants)
-        } catch {
-          body.variants = {}
-        }
-      }
-
       if (params.startDate !== undefined) {
         body.start_date = params.startDate
       }
@@ -175,11 +160,6 @@ export const createExperimentTool: ToolConfig<CreateExperimentParams, CreateExpe
   },
 
   transformResponse: async (response: Response) => {
-    if (!response.ok) {
-      const error = await response.text()
-      throw new Error(error || 'Failed to create experiment')
-    }
-
     const data = await response.json()
 
     return {
@@ -199,7 +179,6 @@ export const createExperimentTool: ToolConfig<CreateExperimentParams, CreateExpe
         feature_flag: { type: 'object', description: 'Feature flag details' },
         parameters: { type: 'object', description: 'Experiment parameters' },
         filters: { type: 'object', description: 'Experiment filters' },
-        variants: { type: 'object', description: 'Experiment variants' },
         start_date: { type: 'string', description: 'Start date' },
         end_date: { type: 'string', description: 'End date' },
         created_at: { type: 'string', description: 'Creation timestamp' },