azure_devops: address bugbot review comments

- triggers/utils: match build.complete result case-insensitively, accept stopped/cancelled in addition to failed/canceled/partiallySucceeded so PascalCase and legacy Azure DevOps payloads aren't dropped
- get_work_items_batch: chunk comma-separated IDs into 200-batch loops with proper status checks (was failing or returning incomplete data on >200 IDs)
- Add tests for both behaviors

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Author: waleedlatif1

apps/sim/tools/azure_devops/azure-devops.test.ts

@@ -2,6 +2,7 @@
  * @vitest-environment node
  */
 import { afterEach, describe, expect, it, vi } from 'vitest'
+import { isAzureDevOpsEventMatch } from '@/triggers/azure_devops/utils'
 import { tools } from '../registry'
 import type { ToolConfig } from '../types'
 import { addCommentTool } from './add_comment'
@@ -572,9 +573,11 @@ describe('Azure DevOps response transforms', () => {
     })
 
     const batch = await getWorkItemsBatchTool.transformResponse!(
-      responseJson({ value: [rawWorkItem] })
+      responseJson({ value: [rawWorkItem] }),
+      { ...baseParams, ids: '101' } satisfies GetWorkItemsBatchParams
     )
     expect(batch.output.metadata.count).toBe(1)
+    expect(batch.output.metadata.totalRequested).toBe(1)
   })
 
   it('hydrates WIQL query results in chunks of 200 IDs', async () => {
@@ -602,6 +605,26 @@ describe('Azure DevOps response transforms', () => {
     expect(result.output.metadata.workItems).toHaveLength(2)
   })
 
+  it('chunks Get Work Items Batch requests larger than 200 IDs', async () => {
+    const fetchMock = vi
+      .fn()
+      .mockImplementation(() => Promise.resolve(responseJson({ value: [rawWorkItem] })))
+    globalThis.fetch = fetchMock as unknown as typeof fetch
+
+    const ids = Array.from({ length: 350 }, (_, i) => String(i + 1)).join(',')
+
+    const result = await getWorkItemsBatchTool.transformResponse!(
+      responseJson({ value: [rawWorkItem] }),
+      { ...baseParams, ids } satisfies GetWorkItemsBatchParams
+    )
+
+    expect(fetchMock).toHaveBeenCalledTimes(1)
+    const followupChunk = new URL(String(fetchMock.mock.calls[0][0]))
+    expect(followupChunk.searchParams.get('ids')?.split(',')).toHaveLength(150)
+    expect(result.output.metadata.totalRequested).toBe(350)
+    expect(result.output.metadata.workItems).toHaveLength(2)
+  })
+
   it('throws when WIQL hydration fetch returns a non-OK status', async () => {
     const fetchMock = vi
       .fn()
@@ -672,3 +695,57 @@ describe('Azure DevOps response transforms', () => {
     })
   })
 })
+
+describe('Azure DevOps trigger event matching', () => {
+  const baseBuild = { eventType: 'build.complete' }
+  const baseWorkItem = { eventType: 'workitem.created' }
+
+  it('matches build.complete results case-insensitively including stopped/Failed/Canceled', () => {
+    for (const result of [
+      'failed',
+      'Failed',
+      'FAILED',
+      'canceled',
+      'Canceled',
+      'cancelled',
+      'Cancelled',
+      'stopped',
+      'Stopped',
+      'partiallySucceeded',
+      'PartiallySucceeded',
+    ]) {
+      expect(
+        isAzureDevOpsEventMatch('azure_devops_build_failed', {
+          ...baseBuild,
+          resource: { result },
+        })
+      ).toBe(true)
+    }
+  })
+
+  it('does not match successful build.complete payloads', () => {
+    for (const result of ['succeeded', 'Succeeded', 'inProgress']) {
+      expect(
+        isAzureDevOpsEventMatch('azure_devops_build_failed', {
+          ...baseBuild,
+          resource: { result },
+        })
+      ).toBe(false)
+    }
+  })
+
+  it('ignores non-build event types when expecting build.complete', () => {
+    expect(
+      isAzureDevOpsEventMatch('azure_devops_build_failed', {
+        eventType: 'workitem.created',
+        resource: { result: 'failed' },
+      })
+    ).toBe(false)
+  })
+
+  it('matches workitem.created and passes through generic webhook', () => {
+    expect(isAzureDevOpsEventMatch('azure_devops_work_item_created', baseWorkItem)).toBe(true)
+    expect(isAzureDevOpsEventMatch('azure_devops_work_item_created', baseBuild)).toBe(false)
+    expect(isAzureDevOpsEventMatch('azure_devops_webhook', { eventType: 'anything' })).toBe(true)
+  })
+})

apps/sim/tools/azure_devops/get_work_items_batch.ts

@@ -12,7 +12,7 @@ export const getWorkItemsBatchTool: ToolConfig<GetWorkItemsBatchParams, GetWorkI
     id: 'azure_devops_get_work_items_batch',
     name: 'Azure DevOps Get Work Items Batch',
     description:
-      'Fetch full details for multiple work items by ID from Azure DevOps in a single call. Pass comma-separated IDs (e.g. "123,456,789"). Maximum 200 IDs per request.',
+      'Fetch full details for multiple work items by ID from Azure DevOps. Pass comma-separated IDs (e.g. "123,456,789"). Requests with more than 200 IDs are automatically split into chunks.',
     version: '1.0.0',
 
     params: {
@@ -33,7 +33,7 @@ export const getWorkItemsBatchTool: ToolConfig<GetWorkItemsBatchParams, GetWorkI
         required: true,
         visibility: 'user-or-llm',
         description:
-          'Comma-separated work item IDs to fetch (e.g. "123,456,789"). Maximum 200 IDs.',
+          'Comma-separated work item IDs to fetch (e.g. "123,456,789"). Lists longer than 200 IDs are chunked automatically.',
       },
       accessToken: {
         type: 'string',
@@ -45,10 +45,15 @@ export const getWorkItemsBatchTool: ToolConfig<GetWorkItemsBatchParams, GetWorkI
 
     request: {
       url: (params) => {
+        const allIds = params.ids
+          .split(',')
+          .map((id) => id.trim())
+          .filter(Boolean)
+        const firstChunk = allIds.slice(0, 200)
         const url = new URL(
           `https://dev.azure.com/${params.organization.trim()}/${params.project.trim()}/_apis/wit/workitems`
         )
-        url.searchParams.set('ids', params.ids)
+        url.searchParams.set('ids', firstChunk.join(','))
         url.searchParams.set('$expand', 'all')
         url.searchParams.set('api-version', '7.2-preview.3')
         return url.toString()
@@ -60,22 +65,61 @@ export const getWorkItemsBatchTool: ToolConfig<GetWorkItemsBatchParams, GetWorkI
       }),
     },
 
-    transformResponse: async (response) => {
-      const data = await response.json()
-      const workItems: AzureDevOpsWorkItem[] = (data.value ?? []).map(
+    transformResponse: async (response, params) => {
+      const firstData = await response.json()
+      const workItems: AzureDevOpsWorkItem[] = (firstData.value ?? []).map(
         (raw: AzureDevOpsRawWorkItem) => mapWorkItem(raw)
       )
 
+      const allIds = params!.ids
+        .split(',')
+        .map((id) => id.trim())
+        .filter(Boolean)
+
+      if (allIds.length > 200) {
+        const BATCH_SIZE = 200
+        const organization = params!.organization.trim()
+        const project = params!.project.trim()
+        const authHeader = `Basic ${btoa(`:${params!.accessToken}`)}`
+
+        for (let i = BATCH_SIZE; i < allIds.length; i += BATCH_SIZE) {
+          const chunk = allIds.slice(i, i + BATCH_SIZE)
+          const detailsUrl = new URL(
+            `https://dev.azure.com/${organization}/${project}/_apis/wit/workitems`
+          )
+          detailsUrl.searchParams.set('ids', chunk.join(','))
+          detailsUrl.searchParams.set('$expand', 'all')
+          detailsUrl.searchParams.set('api-version', '7.2-preview.3')
+
+          const chunkResponse = await fetch(detailsUrl.toString(), {
+            method: 'GET',
+            headers: { 'Content-Type': 'application/json', Authorization: authHeader },
+          })
+
+          if (!chunkResponse.ok) {
+            const errorBody = await chunkResponse.text().catch(() => '')
+            throw new Error(
+              `Failed to fetch work item batch chunk (${chunkResponse.status}): ${errorBody || chunkResponse.statusText}`
+            )
+          }
+
+          const chunkData = await chunkResponse.json()
+          for (const raw of chunkData.value ?? []) {
+            workItems.push(mapWorkItem(raw as AzureDevOpsRawWorkItem))
+          }
+        }
+      }
+
       const content =
         workItems.length === 0
           ? 'No work items found for the provided IDs.'
-          : `Found ${workItems.length} work item(s):\n\n${workItems.map(formatWorkItem).join('\n\n')}`
+          : `Found ${workItems.length} work item(s) (of ${allIds.length} requested):\n\n${workItems.map(formatWorkItem).join('\n\n')}`
 
       return {
         success: true,
         output: {
           content,
-          metadata: { count: workItems.length, workItems },
+          metadata: { count: workItems.length, totalRequested: allIds.length, workItems },
         },
       }
     },
@@ -90,6 +134,11 @@ export const getWorkItemsBatchTool: ToolConfig<GetWorkItemsBatchParams, GetWorkI
         description: 'Work items metadata',
         properties: {
           count: { type: 'number', description: 'Number of work items returned' },
+          totalRequested: {
+            type: 'number',
+            description: 'Total number of IDs requested (across all chunks)',
+            optional: true,
+          },
           workItems: {
             type: 'array',
             description: 'Array of work item details',

apps/sim/tools/azure_devops/types.ts

@@ -306,6 +306,7 @@ export interface GetWorkItemsBatchResponse extends ToolResponse {
     content: string
     metadata: {
       count: number
+      totalRequested?: number
       workItems: AzureDevOpsWorkItem[]
     }
   }

apps/sim/triggers/azure_devops/utils.ts

@@ -54,8 +54,14 @@ export function isAzureDevOpsEventMatch(triggerId: string, body: Record<string,
       return false
     }
     const resource = body.resource as Record<string, unknown> | undefined
-    const result = resource?.result as string | undefined
-    return result === 'failed' || result === 'canceled' || result === 'partiallySucceeded'
+    const result = (resource?.result as string | undefined)?.toLowerCase()
+    return (
+      result === 'failed' ||
+      result === 'canceled' ||
+      result === 'cancelled' ||
+      result === 'stopped' ||
+      result === 'partiallysucceeded'
+    )
   }
 
   if (triggerId === 'azure_devops_work_item_created') {