fix(grafana): address PR review feedback

- Drop hardcoded orgID: 1 fallback; only send orgID when organizationId is
  provided, so token-scoped org context drives rule placement.
- Surface invalid JSON for notificationSettings/record on alert rule
  create/update instead of silently dropping the input.
- Fix execErrState description in update_alert_rule to include Error.

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

Author: waleedlatif1

apps/sim/tools/grafana/create_alert_rule.ts

@@ -163,12 +163,12 @@ export const createAlertRuleTool: ToolConfig<
       }
 
       const body: Record<string, unknown> = {
-        orgID: params.organizationId ? Number(params.organizationId) : 1,
         title: params.title,
         folderUID: params.folderUid,
         ruleGroup: params.ruleGroup,
         data: dataArray,
       }
+      if (params.organizationId) body.orgID = Number(params.organizationId)
 
       if (params.condition) body.condition = params.condition
       if (params.uid) body.uid = params.uid
@@ -201,15 +201,15 @@ export const createAlertRuleTool: ToolConfig<
         try {
           body.notification_settings = JSON.parse(params.notificationSettings)
         } catch {
-          // skip invalid notification settings JSON
+          throw new Error('Invalid JSON for notificationSettings parameter')
         }
       }
 
       if (params.record) {
         try {
           body.record = JSON.parse(params.record)
         } catch {
-          // skip invalid record JSON
+          throw new Error('Invalid JSON for record parameter')
         }
       }
 

apps/sim/tools/grafana/update_alert_rule.ts

@@ -80,7 +80,7 @@ export const updateAlertRuleTool: ToolConfig<GrafanaUpdateAlertRuleParams, ToolR
       type: 'string',
       required: false,
       visibility: 'user-only',
-      description: 'State on execution error (Alerting, OK)',
+      description: 'State on execution error (Error, Alerting, OK)',
     },
     annotations: {
       type: 'string',
@@ -195,15 +195,23 @@ export const updateAlertRuleTool: ToolConfig<GrafanaUpdateAlertRuleParams, ToolR
       try {
         updatedRule.notification_settings = JSON.parse(params.notificationSettings)
       } catch {
-        // keep existing notification_settings if parse fails
+        return {
+          success: false,
+          output: {},
+          error: 'Invalid JSON for notificationSettings parameter',
+        }
       }
     }
 
     if (params.record) {
       try {
         updatedRule.record = JSON.parse(params.record)
       } catch {
-        // keep existing record if parse fails
+        return {
+          success: false,
+          output: {},
+          error: 'Invalid JSON for record parameter',
+        }
       }
     }