marvin8/yunjin
Watch
1
0
Fork
You've already forked yunjin
 0
Code Issues 7 Pull requests Releases Activity Actions

Migrate HTTP client from httpx to httpx2 #15

Manually merged
marvin8 merged 2 commits from feat/issue-10-httpx2-migration into main 2026-09-11 20:28:17 +00:00
Conversation 2 Commits 2 Files changed 9 +899 -748
agent-pi commented 2026-09-11 19:52:30 +00:00
Collaborator
Copy link

Swaps the HTTP client dependency from httpx~=0.28.1 to httpx2~=2.12.0 (pydantic fork) per project convention, and migrates all imports:

  • src/yunjin/services/fetcher.py — httpx2.get()
  • src/yunjin/services/media_extractor.py — httpx2.get()/httpx2.head(), exception types
  • src/yunjin/services/llm_aggregator.py — httpx2.post(), exception types (in scope per issue goal: all HTTP client usage; file postdates the issue body)
  • Test patch targets updated in test_fetcher.py/test_aggregator.py
  • Roadmap.md dependency summary updated

API compatibility verified against the installed httpx2 2.12.0 source: get/post/head signatures, TimeoutException/RequestError/HTTPStatusError, and Response attributes are identical to the httpx call sites. Only yunjin depended on httpx in uv.lock, so it drops out cleanly.

Closes #10

Swaps the HTTP client dependency from `httpx~=0.28.1` to `httpx2~=2.12.0` (pydantic fork) per project convention, and migrates all imports: - `src/yunjin/services/fetcher.py` — `httpx2.get()` - `src/yunjin/services/media_extractor.py` — `httpx2.get()`/`httpx2.head()`, exception types - `src/yunjin/services/llm_aggregator.py` — `httpx2.post()`, exception types (in scope per issue goal: all HTTP client usage; file postdates the issue body) - Test patch targets updated in `test_fetcher.py`/`test_aggregator.py` - `Roadmap.md` dependency summary updated API compatibility verified against the installed httpx2 2.12.0 source: `get`/`post`/`head` signatures, `TimeoutException`/`RequestError`/`HTTPStatusError`, and `Response` attributes are identical to the httpx call sites. Only yunjin depended on httpx in `uv.lock`, so it drops out cleanly. Closes #10
📝 Update dependency summary in Roadmap.md
All checks were successful
/ pr-review (pull_request) Successful in 10s
Details
/ gitleaks (pull_request) Successful in 29s
Details
/ checks (pull_request) Successful in 3m57s
Details
c07eef8fbb
forgejo-actions commented 2026-09-11 19:52:40 +00:00
Copy link

This PR is too large to review automatically (2485 lines; limit is 2000). Please consider splitting it into smaller PRs.

This PR is too large to review automatically (2485 lines; limit is 2000). Please consider splitting it into smaller PRs.
marvin8 approved these changes 2026-09-11 20:27:12 +00:00
marvin8 manually merged commit 7201df4548 into main 2026-09-11 20:28:17 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
marvin8
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Assignees
Clear assignees
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Reference
marvin8/yunjin!15
No description provided.