Skip to content

Conversation

@maxdswain
Copy link

Related Issues

Proposed Changes:

  • Add run_async methods to SearchApiWebSearch and SerperDevWebSearch
  • Switch from requests to httpx to allow consistent HTTP request library to be used across run and run_async methods (requests doesn't support async)
  • Add 9 new unit tests and 2 new integration tests

How did you test it?

New units and integration tests mirroring the existing unit/integration tests for the respective run methods.

Notes for the reviewer

Checklist

  • I have read the contributors guidelines and the code of conduct.
  • I have updated the related issue with new insights and changes.
  • I have added unit tests and updated the docstrings.
  • I've used one of the conventional commit types for my PR title: fix:, feat:, build:, chore:, ci:, docs:, style:, refactor:, perf:, test: and added ! in case the PR includes breaking changes.
  • I have documented my code.
  • I have added a release note file, following the contributors guidelines.
  • I have run pre-commit hooks and fixed any issue.

@maxdswain maxdswain requested a review from a team as a code owner January 31, 2026 16:13
@maxdswain maxdswain requested review from sjrl and removed request for a team January 31, 2026 16:13
@vercel
Copy link

vercel bot commented Jan 31, 2026

@maxdswain is attempting to deploy a commit to the deepset Team on Vercel.

A member of the Team first needs to authorize it.

@CLAassistant
Copy link

CLAassistant commented Jan 31, 2026

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions bot added topic:tests type:documentation Improvements on the docs labels Jan 31, 2026
@maxdswain maxdswain changed the title Feat: Add run_async methods to SearchApiWebSearch and SerperDevWebSearch feat: Add run_async methods to SearchApiWebSearch and SerperDevWebSearch Jan 31, 2026
@coveralls
Copy link
Collaborator

Pull Request Test Coverage Report for Build 21547228321

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 21 unchanged lines in 1 file lost coverage.
  • Overall coverage increased (+0.01%) to 92.45%

Files with Coverage Reduction New Missed Lines %
components/websearch/serper_dev.py 21 79.0%
Totals Coverage Status
Change from base Build 21517269097: 0.01%
Covered Lines: 14793
Relevant Lines: 16001

💛 - Coveralls

@sjrl
Copy link
Contributor

sjrl commented Feb 2, 2026

@maxdswain thanks for the changes!

Could you also add a release note following our instructions here: https://github.com/deepset-ai/haystack/blob/main/CONTRIBUTING.md#release-notes

Copy link
Contributor

@sjrl sjrl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good @maxdswain! Just need the release note and we can merge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

topic:tests type:documentation Improvements on the docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add run_async for all Websearch Pipeline components

4 participants