fix(brightdata): use params for echo-back fields in transformResponse

transformResponse receives params as its second argument. Use it to
return the original url, query, snapshotId, and searchEngine values
instead of hardcoding null or extracting from response data that may
not contain them.

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

Author: waleedlatif1

apps/sim/tools/brightdata/cancel_snapshot.ts

@@ -38,17 +38,17 @@ export const brightDataCancelSnapshotTool: ToolConfig<
     }),
   },
 
-  transformResponse: async (response: Response) => {
+  transformResponse: async (response: Response, params) => {
     if (!response.ok) {
       const errorText = await response.text()
       throw new Error(errorText || `Cancel snapshot failed with status ${response.status}`)
     }
 
-    const data = (await response.json().catch(() => null)) as Record<string, unknown> | null
+    await response.json().catch(() => null)
     return {
       success: true,
       output: {
-        snapshotId: (data?.snapshot_id as string) ?? null,
+        snapshotId: params?.snapshotId ?? null,
         cancelled: true,
       },
     }

apps/sim/tools/brightdata/discover.ts

@@ -84,7 +84,7 @@ export const brightDataDiscoverTool: ToolConfig<
     },
   },
 
-  transformResponse: async (response: Response) => {
+  transformResponse: async (response: Response, params) => {
     if (!response.ok) {
       const errorText = await response.text()
       throw new Error(errorText || `Discover request failed with status ${response.status}`)
@@ -117,7 +117,7 @@ export const brightDataDiscoverTool: ToolConfig<
       success: true,
       output: {
         results,
-        query: null,
+        query: params?.query ?? null,
         totalResults: results.length,
       },
     }

apps/sim/tools/brightdata/download_snapshot.ts

@@ -56,7 +56,7 @@ export const brightDataDownloadSnapshotTool: ToolConfig<
     }),
   },
 
-  transformResponse: async (response: Response) => {
+  transformResponse: async (response: Response, params) => {
     if (response.status === 409) {
       throw new Error(
         'Snapshot is not ready for download. Check the snapshot status first and wait until it is "ready".'
@@ -89,7 +89,7 @@ export const brightDataDownloadSnapshotTool: ToolConfig<
       output: {
         data,
         format: contentType,
-        snapshotId: (data[0]?.snapshot_id as string) ?? null,
+        snapshotId: params?.snapshotId ?? null,
       },
     }
   },

apps/sim/tools/brightdata/scrape_url.ts

@@ -66,7 +66,7 @@ export const brightDataScrapeUrlTool: ToolConfig<
     },
   },
 
-  transformResponse: async (response: Response) => {
+  transformResponse: async (response: Response, params) => {
     const contentType = response.headers.get('content-type') || ''
 
     if (!response.ok) {
@@ -86,7 +86,7 @@ export const brightDataScrapeUrlTool: ToolConfig<
       success: true,
       output: {
         content,
-        url: null,
+        url: params?.url ?? null,
         statusCode: response.status,
       },
     }

apps/sim/tools/brightdata/serp_search.ts

@@ -129,7 +129,7 @@ export const brightDataSerpSearchTool: ToolConfig<
     },
   },
 
-  transformResponse: async (response: Response) => {
+  transformResponse: async (response: Response, params) => {
     if (!response.ok) {
       const errorText = await response.text()
       throw new Error(errorText || `SERP request failed with status ${response.status}`)
@@ -178,9 +178,14 @@ export const brightDataSerpSearchTool: ToolConfig<
       success: true,
       output: {
         results,
-        query: ((data?.general as Record<string, unknown> | undefined)?.query as string) ?? null,
+        query:
+          ((data?.general as Record<string, unknown> | undefined)?.query as string) ??
+          params?.query ??
+          null,
         searchEngine:
-          ((data?.general as Record<string, unknown> | undefined)?.search_engine as string) ?? null,
+          ((data?.general as Record<string, unknown> | undefined)?.search_engine as string) ??
+          params?.searchEngine ??
+          null,
       },
     }
   },