fix(pagerduty): reject empty merge sources, enforce dedupKey, require integer snooze duration

- merge_incidents: throw if source_incidents ends up empty after filtering blanks
- send_event: require dedupKey for acknowledge/resolve to match block's required condition
- snooze_incident: require an integer duration, not just a finite number

Author: waleedlatif1

apps/sim/tools/pagerduty/merge_incidents.ts

@@ -50,16 +50,23 @@ export const mergeIncidentsTool: ToolConfig<
       'Content-Type': 'application/json',
       From: params.fromEmail,
     }),
-    body: (params) => ({
-      source_incidents: params.sourceIncidentIds
+    body: (params) => {
+      const sourceIds = params.sourceIncidentIds
         .split(',')
         .map((id) => id.trim())
         .filter((id) => id.length > 0)
-        .map((id) => ({
+
+      if (sourceIds.length === 0) {
+        throw new Error('sourceIncidentIds must contain at least one non-empty incident ID')
+      }
+
+      return {
+        source_incidents: sourceIds.map((id) => ({
           id,
           type: 'incident_reference',
         })),
-    }),
+      }
+    },
   },
 
   transformResponse: async (response: Response) => {

apps/sim/tools/pagerduty/send_event.ts

@@ -80,6 +80,12 @@ export const sendEventTool: ToolConfig<PagerDutySendEventParams, PagerDutySendEv
         event_action: params.eventAction,
       }
 
+      if (params.eventAction === 'acknowledge' || params.eventAction === 'resolve') {
+        if (!params.dedupKey) {
+          throw new Error('dedupKey is required when eventAction is acknowledge or resolve')
+        }
+      }
+
       if (params.dedupKey) body.dedup_key = params.dedupKey
 
       if (params.eventAction === 'trigger') {

apps/sim/tools/pagerduty/snooze_incident.ts

@@ -52,7 +52,7 @@ export const snoozeIncidentTool: ToolConfig<
     }),
     body: (params) => {
       const duration = Number(params.duration)
-      if (!Number.isFinite(duration) || duration < 1 || duration > 604800) {
+      if (!Number.isInteger(duration) || duration < 1 || duration > 604800) {
         throw new Error('duration must be a whole number of seconds between 1 and 604800')
       }
       return { duration }