fix(incidentio): stop the alert filter sentinel reaching the API

The has_notes and include_maintenance_window dropdowns default to the
string "any", meaning "do not filter". The params transform skipped the
key in that case, but the executor merges its output over the raw inputs
(`{ ...inputs, ...transformedParams }`), so the sentinel survived and the
tool sent has_notes[is]=any, which incident.io rejects.

The transform now always assigns the key, mapping "any" to undefined so
it overwrites the sentinel instead of leaving it in place. The tool also
only serializes these filters when it actually has a boolean.

Adds tests covering the sentinel, both real boolean values, and the
documented bracket-operator filter syntax.

Author: waleedlatif1

apps/sim/blocks/blocks/incidentio.ts

@@ -13,6 +13,19 @@ const OVERRIDE_USER_FIELD = ['user_id', 'user_email', 'user_slack_id'] as const
 /** An escalation pages either a path or an explicit user list, never both. */
 const ESCALATION_TARGET_FIELD = ['escalation_path_id', 'user_ids'] as const
 
+/**
+ * Maps an optional-filter dropdown onto a real boolean, or `undefined` for the "Any" sentinel.
+ *
+ * The executor merges the transform output over the raw inputs, so a key this returns
+ * `undefined` for still overwrites the sentinel string rather than letting it reach the tool.
+ * Skipping the assignment instead would leak `has_notes[is]=any` to the API.
+ */
+function toTriState(value: unknown): boolean | undefined {
+  if (value === true || value === 'true') return true
+  if (value === false || value === 'false') return false
+  return undefined
+}
+
 export const IncidentioBlock: BlockConfig<IncidentioResponse> = {
   type: 'incidentio',
   name: 'incident.io',
@@ -1791,12 +1804,8 @@ Return ONLY the description - no explanations.`,
         }
         if (params.operation === 'incidentio_alerts_list') {
           if (params.alert_status) result.status = params.alert_status
-          if (params.has_notes && params.has_notes !== 'any') {
-            result.has_notes = params.has_notes === 'true'
-          }
-          if (params.include_maintenance_window && params.include_maintenance_window !== 'any') {
-            result.include_maintenance_window = params.include_maintenance_window === 'true'
-          }
+          result.has_notes = toTriState(params.has_notes)
+          result.include_maintenance_window = toTriState(params.include_maintenance_window)
         }
         if (params.operation === 'incidentio_alert_events_create') {
           if (params.alert_status) result.status = params.alert_status

apps/sim/tools/incidentio/alerts_list.test.ts

@@ -0,0 +1,81 @@
+/**
+ * @vitest-environment node
+ */
+import { describe, expect, it } from 'vitest'
+import { IncidentioBlock } from '@/blocks/blocks/incidentio'
+import { alertsListTool } from '@/tools/incidentio/alerts_list'
+
+/**
+ * The executor merges the block's param transform over the raw inputs
+ * (`{ ...inputs, ...transformedParams }`), so any subblock the transform leaves alone keeps its
+ * raw editor value. The tri-state filter dropdowns default to the string 'any', which the
+ * incident.io API rejects — these cover the whole path from editor value to query string.
+ */
+function resolveInputs(inputs: Record<string, unknown>): Record<string, unknown> {
+  const transform = IncidentioBlock.tools.config!.params!
+  return { ...inputs, ...transform(inputs as never) }
+}
+
+function buildUrl(params: Record<string, unknown>): URL {
+  const url = alertsListTool.request.url
+  return new URL(typeof url === 'function' ? url(params as never) : url)
+}
+
+describe('incidentio_alerts_list filters', () => {
+  it('omits the tri-state filters when left on "Any"', () => {
+    const resolved = resolveInputs({
+      operation: 'incidentio_alerts_list',
+      apiKey: 'k',
+      has_notes: 'any',
+      include_maintenance_window: 'any',
+    })
+
+    expect(resolved.has_notes).toBeUndefined()
+    expect(resolved.include_maintenance_window).toBeUndefined()
+
+    const url = buildUrl(resolved)
+    expect(url.searchParams.has('has_notes[is]')).toBe(false)
+    expect(url.searchParams.has('include_maintenance_window[is]')).toBe(false)
+  })
+
+  it('sends the tri-state filters as booleans when set', () => {
+    const url = buildUrl(
+      resolveInputs({
+        operation: 'incidentio_alerts_list',
+        apiKey: 'k',
+        has_notes: 'true',
+        include_maintenance_window: 'false',
+      })
+    )
+
+    expect(url.searchParams.get('has_notes[is]')).toBe('true')
+    expect(url.searchParams.get('include_maintenance_window[is]')).toBe('false')
+  })
+
+  it('never forwards a non-boolean filter value to the query string', () => {
+    const url = buildUrl({ apiKey: 'k', has_notes: 'any', include_maintenance_window: 'any' })
+
+    expect(url.searchParams.has('has_notes[is]')).toBe(false)
+    expect(url.searchParams.has('include_maintenance_window[is]')).toBe(false)
+  })
+
+  it('applies the documented bracket-operator filter syntax', () => {
+    const url = buildUrl(
+      resolveInputs({
+        operation: 'incidentio_alerts_list',
+        apiKey: 'k',
+        alert_status: 'firing',
+        status_operator: 'not_in',
+        alert_source_id: '01GBSQF3FHF7FWZQNWGHAVQ804',
+        deduplication_key: 'ABC',
+        created_at_gte: '2025-01-01',
+      })
+    )
+
+    expect(url.searchParams.get('status[not_in]')).toBe('firing')
+    expect(url.searchParams.get('alert_source[one_of]')).toBe('01GBSQF3FHF7FWZQNWGHAVQ804')
+    expect(url.searchParams.get('deduplication_key[is]')).toBe('ABC')
+    expect(url.searchParams.get('created_at[gte]')).toBe('2025-01-01')
+    expect(url.searchParams.get('page_size')).toBe('25')
+  })
+})

apps/sim/tools/incidentio/alerts_list.ts

@@ -127,10 +127,10 @@ export const alertsListTool: ToolConfig<IncidentioAlertsListParams, IncidentioAl
         if (params.created_at_lte) {
           url.searchParams.set('created_at[lte]', params.created_at_lte.trim())
         }
-        if (params.has_notes !== undefined) {
+        if (typeof params.has_notes === 'boolean') {
           url.searchParams.set('has_notes[is]', String(params.has_notes))
         }
-        if (params.include_maintenance_window !== undefined) {
+        if (typeof params.include_maintenance_window === 'boolean') {
           url.searchParams.set(
             'include_maintenance_window[is]',
             String(params.include_maintenance_window)