user2595/codex-py
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(sources): resolve arXiv IDs via arXiv DOI (OpenAlex /works/arxiv: 404s) #5

Merged
user2595 merged 1 commits from fix/openalex-arxiv-doi into main 2026-06-14 12:31:34 +00:00
Conversation 0 Commits 1 Files Changed 2 +23 -9
user2595 commented 2026-06-14 12:31:25 +00:00
Owner
Copy Link

Summary

  • _resolve_id() was building arxiv:<id> paths that 404 on the OpenAlex /works/ endpoint
  • Fix: arXiv IDs (bare and arxiv:-prefixed, modern and legacy) now resolve to doi:10.48550/arXiv.<id>, which OpenAlex accepts reliably
  • 3 new tests cover modern ID, legacy math/0603097, and explicit arxiv: prefix stripping

Review-Gate

Opus APPROVE — 3 WARNs (all non-blocking: case-sensitivity edge case, no double-prefix guard test, minor docstring drift in fetch_paper).

Test plan

  • uv run pytest tests/sources/test_openalex.py — 19 tests green
  • uv run ruff check . && uv run mypy codex/ — clean

🤖 Generated with Claude Code

## Summary - `_resolve_id()` was building `arxiv:<id>` paths that 404 on the OpenAlex `/works/` endpoint - Fix: arXiv IDs (bare and `arxiv:`-prefixed, modern and legacy) now resolve to `doi:10.48550/arXiv.<id>`, which OpenAlex accepts reliably - 3 new tests cover modern ID, legacy `math/0603097`, and explicit `arxiv:` prefix stripping ## Review-Gate Opus APPROVE — 3 WARNs (all non-blocking: case-sensitivity edge case, no double-prefix guard test, minor docstring drift in `fetch_paper`). ## Test plan - [ ] `uv run pytest tests/sources/test_openalex.py` — 19 tests green - [ ] `uv run ruff check . && uv run mypy codex/` — clean 🤖 Generated with [Claude Code](https://claude.com/claude-code)
user2595 added 1 commit 2026-06-14 12:31:26 +00:00
fix(sources): resolve arXiv IDs via arXiv DOI (OpenAlex /works/arxiv: 404s) c8b127bae0 
OpenAlex' /works/{id}-Pfad akzeptiert die arxiv:-Namespace-Form nicht (404).
_resolve_id löst arXiv-IDs jetzt auf die arXiv-DOI 10.48550/arXiv.<id> auf —
empirisch verifiziert für modern (1005.2698) und legacy (math/0603097) IDs.
Behebt D-03 (Erstingest: alle Metadaten leer, weil fetch_paper 404 gab).

- _resolve_id: arxiv:-Präfix strippen, arXiv-DOI bauen
- Tests: arxiv (modern/legacy/prefixed) erwarten jetzt doi:10.48550/arXiv.*
- 19 Tests grün, ruff + mypy sauber

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
user2595 merged commit 1d1203b225 into main 2026-06-14 12:31:34 +00:00
user2595 deleted branch fix/openalex-arxiv-doi 2026-06-14 12:31:34 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
user2595
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: user2595/codex-py#5
No description provided.
Delete Branch "fix/openalex-arxiv-doi"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?