fix(rootly): add id to PATCH body and unchanged option to update status dropdown

waleedlatif1 · waleedlatif1 · commit d3491dbfda44 · 2026-04-02T10:52:27.000-07:00
- Include incident id in JSON:API PATCH body per spec requirement
- Add 'Unchanged' empty option to updateStatus dropdown to avoid accidental overwrites
diff --git a/apps/sim/blocks/blocks/rootly.ts b/apps/sim/blocks/blocks/rootly.ts
@@ -189,6 +189,7 @@ export const RootlyBlock: BlockConfig<RootlyResponse> = {
       title: 'Status',
       type: 'dropdown',
       options: [
+        { label: 'Unchanged', id: '' },
         { label: 'In Triage', id: 'in_triage' },
         { label: 'Started', id: 'started' },
         { label: 'Detected', id: 'detected' },
diff --git a/apps/sim/tools/rootly/update_incident.ts b/apps/sim/tools/rootly/update_incident.ts
@@ -104,7 +104,7 @@ export const rootlyUpdateIncidentTool: ToolConfig<
       if (params.groupIds) {
         attributes.group_ids = params.groupIds.split(',').map((s: string) => s.trim())
       }
-      return { data: { type: 'incidents', attributes } }
+      return { data: { type: 'incidents', id: params.incidentId.trim(), attributes } }
     },
   },
 

Original file line number	Diff line number	Diff line change
`@@ -104,7 +104,7 @@ export const rootlyUpdateIncidentTool: ToolConfig<`
`104`	`104`	`if (params.groupIds) {`
`105`	`105`	`attributes.group_ids = params.groupIds.split(',').map((s: string) => s.trim())`
`106`	`106`	`}`
`107`		`- return { data: { type: 'incidents', attributes } }`
	`107`	`+ return { data: { type: 'incidents', id: params.incidentId.trim(), attributes } }`
`108`	`108`	`},`
`109`	`109`	`},`
`110`	`110`