azure_devops: address more bugbot comments

- Webhook provider extractIdempotencyId returns null when subscriptionId or notificationId is missing/empty, preventing the literal "azure_devops:undefined:undefined" key from collapsing unrelated deliveries into duplicates
- Get Work Items Batch validates that at least one non-empty ID is supplied before issuing the API request, throwing a clear error instead of hitting an empty ids= query
- Tests cover both behaviors

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

Author: waleedlatif1

apps/sim/lib/webhooks/providers/azure-devops.ts

@@ -48,7 +48,14 @@ export const azureDevOpsHandler: WebhookProviderHandler = {
   extractIdempotencyId(body: unknown): string | null {
     const obj = body as Record<string, unknown> | null
     if (!obj) return null
-    return `azure_devops:${obj.subscriptionId}:${obj.notificationId}`
+    const subscriptionId =
+      typeof obj.subscriptionId === 'string' && obj.subscriptionId ? obj.subscriptionId : null
+    const notificationId =
+      typeof obj.notificationId === 'number' || typeof obj.notificationId === 'string'
+        ? String(obj.notificationId)
+        : null
+    if (!subscriptionId || !notificationId) return null
+    return `azure_devops:${subscriptionId}:${notificationId}`
   },
 
   async formatInput({ body, webhook, requestId }: FormatInputContext): Promise<FormatInputResult> {

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

@@ -606,6 +606,15 @@ describe('Azure DevOps response transforms', () => {
     expect(result.output.metadata.workItems).toHaveLength(2)
   })
 
+  it('throws when Get Work Items Batch is invoked with no valid IDs', () => {
+    expect(() =>
+      buildUrl(getWorkItemsBatchTool, {
+        ...baseParams,
+        ids: ' , , ',
+      } satisfies GetWorkItemsBatchParams)
+    ).toThrow(/requires at least one work item ID/)
+  })
+
   it('chunks Get Work Items Batch requests larger than 200 IDs', async () => {
     const fetchMock = vi
       .fn()
@@ -765,4 +774,15 @@ describe('Azure DevOps trigger event matching', () => {
     expect(isAzureDevOpsEventMatch('azure_devops_work_item_created', baseBuild)).toBe(false)
     expect(isAzureDevOpsEventMatch('azure_devops_webhook', { eventType: 'anything' })).toBe(true)
   })
+
+  it('extractIdempotencyId returns null when subscriptionId or notificationId is missing', async () => {
+    const { azureDevOpsHandler } = await import('@/lib/webhooks/providers/azure-devops')
+    expect(azureDevOpsHandler.extractIdempotencyId!({})).toBeNull()
+    expect(azureDevOpsHandler.extractIdempotencyId!({ subscriptionId: 'sub-1' })).toBeNull()
+    expect(azureDevOpsHandler.extractIdempotencyId!({ notificationId: 42 })).toBeNull()
+    expect(
+      azureDevOpsHandler.extractIdempotencyId!({ subscriptionId: 'sub-1', notificationId: 42 })
+    ).toBe('azure_devops:sub-1:42')
+    expect(azureDevOpsHandler.extractIdempotencyId!(null)).toBeNull()
+  })
 })

apps/sim/tools/azure_devops/get_work_items_batch.ts

@@ -49,6 +49,9 @@ export const getWorkItemsBatchTool: ToolConfig<GetWorkItemsBatchParams, GetWorkI
           .split(',')
           .map((id) => id.trim())
           .filter(Boolean)
+        if (allIds.length === 0) {
+          throw new Error('Get Work Items Batch requires at least one work item ID.')
+        }
         const firstChunk = allIds.slice(0, 200)
         const url = new URL(
           `https://dev.azure.com/${params.organization.trim()}/${params.project.trim()}/_apis/wit/workitems`