fix(rootly): send empty body on resolve_alert, guard assignedToUserId parse

resolve_alert now sends { data: {} } instead of undefined when no
optional params are provided, matching the acknowledge_alert fix.
create_action_item now validates assignedToUserId is numeric before
parseInt to avoid silent NaN coercion.

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

Author: waleedlatif1

apps/sim/tools/rootly/create_action_item.ts

@@ -85,7 +85,10 @@ export const rootlyCreateActionItemTool: ToolConfig<
       if (params.kind) attributes.kind = params.kind
       if (params.priority) attributes.priority = params.priority
       if (params.status) attributes.status = params.status
-      if (params.assignedToUserId) attributes.assigned_to_user_id = Number(params.assignedToUserId)
+      if (params.assignedToUserId) {
+        const numericId = Number.parseInt(params.assignedToUserId, 10)
+        if (!Number.isNaN(numericId)) attributes.assigned_to_user_id = numericId
+      }
       if (params.dueDate) attributes.due_date = params.dueDate
       return { data: { type: 'incident_action_items', attributes } }
     },

apps/sim/tools/rootly/resolve_alert.ts

@@ -49,7 +49,7 @@ export const rootlyResolveAlertTool: ToolConfig<
       if (params.resolutionMessage) attributes.resolution_message = params.resolutionMessage
       if (params.resolveRelatedIncidents !== undefined)
         attributes.resolve_related_incidents = params.resolveRelatedIncidents
-      if (Object.keys(attributes).length === 0) return undefined
+      if (Object.keys(attributes).length === 0) return { data: {} }
       return { data: { type: 'alerts', attributes } }
     },
   },