Migrating httpx.AsyncClient to httpx_aiohttp.HttpxAiohttpClient

[jamesbraza]

Aligning with openai/openai-python#1596 (comment)

[dosubot]

Related Documentation

Checked 1 published document(s). No updates required.

^{How did I do? Any feedback?}  

[Copilot]

Pull Request Overview

This PR migrates the HTTP client from httpx.AsyncClient to httpx_aiohttp.HttpxAiohttpClient throughout the codebase to align with OpenAI Python library requirements.

Key changes:

• Replace all instances of httpx.AsyncClient with httpx_aiohttp.HttpxAiohttpClient
• Add httpx-aiohttp as a dependency in pyproject.toml
• Update test fixtures and mocks to use the new client type

Reviewed Changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
pyproject.toml	Add httpx-aiohttp dependency
src/paperqa/clients/init.py	Replace httpx.AsyncClient with HttpxAiohttpClient in DocMetadataClient
src/paperqa/clients/crossref.py	Update retraction dataset download to use new client
src/paperqa/clients/journal_quality.py	Replace client in download function
src/paperqa/sources/clinical_trials.py	Update client initialization for clinical trials
src/paperqa/contrib/openreview_paper_helper.py	Replace client in PDF download method
tests/test_clients.py	Update all test cases to use new client type
tests/test_clinical_trials.py	Update mock fixture to use HttpxAiohttpClient spec
.pre-commit-config.yaml	Add uv.lock to exclude pattern

Comments suppressed due to low confidence (1)

src/paperqa/clients/journal_quality.py:110

• The function parameter type annotation should be updated from httpx.AsyncClient to match the new client type being used (httpx_aiohttp.HttpxAiohttpClient).

    async def download(client_: httpx.AsyncClient) -> None:

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[whitead]

LGTM, pending vcr fixes

[jamesbraza]

LGTM, pending vcr fixes

They are done now 👌 merging

[dosubot]

Documentation Updates

Checked 1 published document(s). No updates required.

^{How did I do? Any feedback?}