From cc3cbdcfffdaebc3c2e629069ad117b56bdd053f Mon Sep 17 00:00:00 2001
From: simpleqt <89645338+simpleqt@users.noreply.github.com>
Date: Tue, 8 Sep 2026 00:32:33 +0800
Subject: [PATCH] fix(brightdata): drop stray $ in f-string search URLs
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

get_search_url interpolated ${query} inside an f-string, producing
URLs like https://www.bing.com/search?q=$test — the literal $ is kept
and the query is interpolated as intended. The URL is passed straight
to the SERP request, so every search hit the wrong query string.
---
 .../crewai_tools/tools/brightdata_tool/brightdata_serp.py   | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/lib/crewai-tools/src/crewai_tools/tools/brightdata_tool/brightdata_serp.py b/lib/crewai-tools/src/crewai_tools/tools/brightdata_tool/brightdata_serp.py
index 83b5c4c30e..81deac5d4e 100644
--- a/lib/crewai-tools/src/crewai_tools/tools/brightdata_tool/brightdata_serp.py
+++ b/lib/crewai-tools/src/crewai_tools/tools/brightdata_tool/brightdata_serp.py
@@ -131,10 +131,10 @@ def __init__(
 
     def get_search_url(self, engine: str, query: str) -> str:
         if engine == "yandex":
-            return f"https://yandex.com/search/?text=${query}"
+            return f"https://yandex.com/search/?text={query}"
         if engine == "bing":
-            return f"https://www.bing.com/search?q=${query}"
-        return f"https://www.google.com/search?q=${query}"
+            return f"https://www.bing.com/search?q={query}"
+        return f"https://www.google.com/search?q={query}"
 
     def _run(
         self,