Skip to content

fix: add HTTP status code check in BraveSearchProvider#1024

Merged
afjcjsbx merged 1 commit intosipeed:mainfrom
wangyanfu2:fix-TavilySearch-response
Mar 3, 2026
Merged

fix: add HTTP status code check in BraveSearchProvider#1024
afjcjsbx merged 1 commit intosipeed:mainfrom
wangyanfu2:fix-TavilySearch-response

Conversation

@wangyanfu2
Copy link
Contributor

  • Add status code validation after reading response body, consistent with TavilySearchProvider and PerplexitySearchProvider

📝 Description

🗣️ Type of Change

  • 🐞 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 📖 Documentation update
  • ⚡ Code refactoring (no functional changes, no api changes)

🤖 AI Code Generation

  • 🤖 Fully AI-generated (100% AI, 0% Human)
  • 🛠️ Mostly AI-generated (AI draft, Human verified/modified)
  • 👨‍💻 Mostly Human-written (Human lead, AI assisted or none)

🔗 Related Issue

📚 Technical Context (Skip for Docs)

  • Reference URL:
  • Reasoning:

🧪 Test Environment

  • Hardware:
  • OS:
  • Model/Provider:
  • Channels:

📸 Evidence (Optional)

Click to view Logs/Screenshots

☑️ Checklist

  • My code/docs follow the style of this project.
  • I have performed a self-review of my own changes.
  • I have updated the documentation accordingly.

- Add status code validation after reading response body, consistent
  with TavilySearchProvider and PerplexitySearchProvider
@afjcjsbx afjcjsbx merged commit 4a7605e into sipeed:main Mar 3, 2026
2 checks passed
@afjcjsbx
Copy link
Collaborator

afjcjsbx commented Mar 3, 2026

@wangyanfu2 thanks for your PR!

@Orgmar
Copy link
Contributor

Orgmar commented Mar 4, 2026

@wangyanfu2 BraveSearchProvider之前缺少HTTP状态码校验确实容易踩坑,和Tavily、Perplexity那边对齐之后整体一致性好多了,这种细节修复很有价值。

我们在Discord上组建了 PicoClaw Dev Group,方便贡献者们交流技术和协作开发。如果你有兴趣加入,发一封邮件到 [email protected],主题写 [Join PicoClaw Dev Group] + 你的GitHub账号,我们会把邀请链接发给你!

hyperwd pushed a commit to hyperwd/picoclaw that referenced this pull request Mar 5, 2026
fix: add HTTP status code check in BraveSearchProvider
Pluckypan pushed a commit to Pluckypan/picoclaw that referenced this pull request Mar 6, 2026
fix: add HTTP status code check in BraveSearchProvider
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants