fix(ketch): add response.ok guards and fix registry ordering

Author: waleedlatif1

apps/sim/tools/ketch/get_consent.ts

@@ -70,6 +70,23 @@ export const getConsentTool: ToolConfig<KetchGetConsentParams, KetchGetConsentRe
   },
 
   transformResponse: async (response: Response) => {
+    if (!response.ok) {
+      let errorMessage = `Request failed with status ${response.status}`
+      try {
+        const data = await response.json()
+        errorMessage = data.message ?? data.error ?? errorMessage
+      } catch {
+        // No JSON body in error response
+      }
+      return {
+        success: false,
+        output: {
+          purposes: {},
+          vendors: null,
+        },
+      }
+    }
+
     const data = await response.json()
     return {
       success: true,

apps/sim/tools/ketch/get_subscriptions.ts

@@ -58,6 +58,23 @@ export const getSubscriptionsTool: ToolConfig<
   },
 
   transformResponse: async (response: Response) => {
+    if (!response.ok) {
+      let errorMessage = `Request failed with status ${response.status}`
+      try {
+        const data = await response.json()
+        errorMessage = data.message ?? data.error ?? errorMessage
+      } catch {
+        // No JSON body in error response
+      }
+      return {
+        success: false,
+        output: {
+          topics: {},
+          controls: {},
+        },
+      }
+    }
+
     const data = await response.json()
     return {
       success: true,

apps/sim/tools/ketch/set_consent.ts

@@ -78,6 +78,22 @@ export const setConsentTool: ToolConfig<KetchSetConsentParams, KetchSetConsentRe
   },
 
   transformResponse: async (response: Response) => {
+    if (!response.ok) {
+      let errorMessage = `Request failed with status ${response.status}`
+      try {
+        const data = await response.json()
+        errorMessage = data.message ?? data.error ?? errorMessage
+      } catch {
+        // No JSON body in error response
+      }
+      return {
+        success: false,
+        output: {
+          purposes: {},
+        },
+      }
+    }
+
     if (response.status === 204) {
       return {
         success: true,

apps/sim/tools/ketch/set_subscriptions.ts

@@ -77,21 +77,22 @@ export const setSubscriptionsTool: ToolConfig<
   },
 
   transformResponse: async (response: Response) => {
-    if (response.status === 204) {
+    if (!response.ok) {
+      let errorMessage = `Request failed with status ${response.status}`
+      try {
+        const data = await response.json()
+        errorMessage = data.message ?? data.error ?? errorMessage
+      } catch {
+        // No JSON body in error response
+      }
       return {
-        success: true,
+        success: false,
         output: {
-          success: true,
+          success: false,
         },
       }
     }
 
-    try {
-      await response.json()
-    } catch {
-      // 204-like response with no body
-    }
-
     return {
       success: true,
       output: {

apps/sim/tools/registry.ts

@@ -2685,12 +2685,12 @@ export const tools: Record<string, ToolConfig> = {
   hex_run_project: hexRunProjectTool,
   hex_update_project: hexUpdateProjectTool,
   jina_read_url: jinaReadUrlTool,
+  jina_search: jinaSearchTool,
   ketch_get_consent: ketchGetConsentTool,
   ketch_get_subscriptions: ketchGetSubscriptionsTool,
   ketch_invoke_right: ketchInvokeRightTool,
   ketch_set_consent: ketchSetConsentTool,
   ketch_set_subscriptions: ketchSetSubscriptionsTool,
-  jina_search: jinaSearchTool,
   linkup_search: linkupSearchTool,
   loops_create_contact: loopsCreateContactTool,
   loops_create_contact_property: loopsCreateContactPropertyTool,