fix(grafana): surface route failures in transformResponse instead of masking them

The grafana update tools' transformResponse hardcoded success: true and dropped
the route's error, so an upstream/validation failure (HTTP 200 with
{ success: false, error }) was reported to the workflow as a success. Forward
data.success and data.error (matching the agiloft tools) so failures propagate
as before the move to internal routes.

Author: waleedlatif1

apps/sim/tools/grafana/update_alert_rule.ts

@@ -161,8 +161,9 @@ export const updateAlertRuleTool: ToolConfig<GrafanaUpdateAlertRuleParams, ToolR
   transformResponse: async (response: Response) => {
     const data = await response.json()
     return {
-      success: true,
-      output: data.output,
+      success: data.success ?? true,
+      output: data.output ?? {},
+      ...(data.error ? { error: data.error } : {}),
     }
   },
 

apps/sim/tools/grafana/update_dashboard.ts

@@ -107,8 +107,9 @@ export const updateDashboardTool: ToolConfig<GrafanaUpdateDashboardParams, ToolR
   transformResponse: async (response: Response) => {
     const data = await response.json()
     return {
-      success: true,
-      output: data.output,
+      success: data.success ?? true,
+      output: data.output ?? {},
+      ...(data.error ? { error: data.error } : {}),
     }
   },
 

apps/sim/tools/grafana/update_folder.ts

@@ -56,8 +56,9 @@ export const updateFolderTool: ToolConfig<GrafanaUpdateFolderParams, ToolRespons
   transformResponse: async (response: Response) => {
     const data = await response.json()
     return {
-      success: true,
-      output: data.output,
+      success: data.success ?? true,
+      output: data.output ?? {},
+      ...(data.error ? { error: data.error } : {}),
     }
   },