- Extract scholar ID parsing to dedicated module with robust URL
handling (trailing slashes, fragments, extra query params, encoded
characters) and per-token error feedback showing why each invalid
entry was skipped
- Wrap delete_scholar DB commit in IntegrityError handler so cascade
failures return a proper API error instead of 500
- Add ScholarStatusBadges component showing Pending/Failed/Partial/OK
badges on scholar rows based on baseline_completed and last_run_status
- Show Pending badge in ScholarSettingsModal for unscraped scholars
- Surface failed_count and partial_count in dashboard latest run summary
and recent run list items
- Extend backend validator tests with edge cases (empty, whitespace,
unicode, URL-as-ID)
- Add integration tests for DELETE 404, DELETE with cascade, and POST
with corrupted scholar_id
- Add comprehensive frontend tests for parsing logic and component
behavior
- Replace fixed batch_size=25 with URL-byte-length-aware chunking for
OpenAlex title.search queries. Long/Unicode titles caused URLs to
exceed server limits, resulting in 500 errors.
- Always hydrate scholar profile metadata (name, image) on creation,
even when an initial scrape job is queued, so the UI shows profile
info immediately.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add ruff and mypy to dev dependencies with configuration in pyproject.toml.
Add a lint CI job that runs ruff check, ruff format --check, and mypy.
Auto-fix import sorting and formatting across the codebase. Exclude
alembic/versions from linting (auto-generated migrations). Ignore B008
(FastAPI Depends pattern) and RUF001 (unicode in user-facing strings).
21 ruff lint errors and 50 mypy errors remain for manual review.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
