PR #5271: fix: remove erroneous $ in BrightData SERP search URL f-strings

• Repository: crewAIInc/crewAI
• Author: MilesQLi
• State: open | merged: False
• Link: https://github.com/crewAIInc/crewAI/pull/5271

Description (problem / solution / changelog)

Summary

Fixes #5269

The get_search_url() method in BrightDataSearchTool was using JavaScript template literal syntax (${query}) instead of Python f-string syntax ({query}). This caused every search URL to contain a literal $ before the query string, e.g. ?q=$test instead of ?q=test.

All three search engine paths (Google, Bing, Yandex) had the same issue.

Changes

• Removed the $ prefix from all three f-string interpolations in get_search_url() (lines 134, 136, 137 of brightdata_serp.py)

That's it — three characters removed. No behavioral changes beyond fixing the URLs.

Test plan

• Verified no other ${...} patterns exist in the BrightData tool files
• Confirmed the fix produces correct URLs by inspection:

  >>> query = "AI+news"
  >>> f"https://www.google.com/search?q={query}"
  'https://www.google.com/search?q=AI+news'

• Existing CI should pass (no test changes)

Changed files

• lib/crewai-tools/src/crewai_tools/tools/brightdata_tool/brightdata_serp.py (modified, +3/-3)

---

PR #5273: fix: remove $ prefix from f-strings in BrightDataSearchTool.get_search_url

• Repository: crewAIInc/crewAI
• Author: devin-ai-integration[bot]
• State: open | merged: False
• Link: https://github.com/crewAIInc/crewAI/pull/5273

Description (problem / solution / changelog)

Summary

Fixes #5269

The get_search_url method in BrightDataSearchTool used JavaScript template literal syntax (${query}) instead of Python f-string syntax ({query}) when building search URLs. This caused every search query to be prepended with a literal $ character (e.g. ?q=$AI%20news instead of ?q=AI%20news). All three search engines (Google, Bing, Yandex) were affected.

The fix removes the erroneous $ from all three f-strings.

Added 4 tests to cover the fix:

• Unit tests for get_search_url on each engine verifying correct URL format
• Integration test through _run verifying the URL sent to the API has no $ prefix

Review & Testing Checklist for Human

• Verify the three f-string changes in brightdata_serp.py lines 133-137 look correct (removed $ only, nothing else changed)
• Optionally run BrightDataSearchTool with a real API key to confirm queries work end-to-end without the $ prefix

Link to Devin session: https://app.devin.ai/sessions/0f820d69f9f74596a25ecab76c3fed67

<!-- CURSOR_SUMMARY -->

---

[!NOTE] Low Risk Low risk: fixes a small URL formatting bug and adds focused tests; behavioral change is limited to removing an unintended $ character in outgoing search queries.

Overview Fixes BrightDataSearchTool.get_search_url to use proper Python f-string interpolation ({query}) for Google/Bing/Yandex, removing the accidental literal $ prefix that was being sent in search queries.

Adds unit tests for each engine’s URL formatting and an _run flow test to assert the request payload URL is encoded correctly and contains no $ in the query.

^{Reviewed by Cursor Bugbot for commit 1386a4325ba52e178af0d28d1a8bf8e43723157a. Bugbot is set up for automated code reviews on this repo. Configure here.}

<!-- /CURSOR_SUMMARY -->

Changed files

• lib/crewai-tools/src/crewai_tools/tools/brightdata_tool/brightdata_serp.py (modified, +3/-3)
• lib/crewai-tools/tests/tools/brightdata_serp_tool_test.py (modified, +54/-0)