Skip to content

Split parser and service install extras - #3

Merged
JSv4 merged 2 commits into
mainfrom
feature/optional-install-extras
Jul 6, 2026
Merged

Split parser and service install extras#3
JSv4 merged 2 commits into
mainfrom
feature/optional-install-extras

Conversation

@JSv4

@JSv4 JSv4 commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Move FastAPI, uvicorn, and multipart dependencies out of the base package install and into an explicit service extra.
  • Add documented install profiles: parser, service, ocr, and all; keep the benchmark-only tables extra separate.
  • Add a clear service-runtime install hint when service entrypoints are used without warp-ingest[service] or warp-ingest[all].
  • Update README, Docker builds, requirements export, uv.lock, and add metadata tests guarding the dependency boundary.

Impact

Parser consumers can install warp-ingest or warp-ingest[parser] without microservice runtime dependencies. Service deployments can use warp-ingest[service] or warp-ingest[all] for the OCR-enabled service image/profile.

Because the package was just released today, this intentionally treats the bare-install behavior change as a packaging correction before broader adoption.

Validation

  • env UV_CACHE_DIR=/tmp/uv-cache uv run pytest tests/test_package_metadata.py tests/test_service.py -q
  • env UV_CACHE_DIR=/tmp/uv-cache uv run black --check warp_ingest/ingestion_daemon/service_dependencies.py warp_ingest/ingestion_daemon/__main__.py warp_ingest/ingestion_daemon/app.py warp_ingest/ingestion_daemon/auth.py tests/test_package_metadata.py
  • env UV_CACHE_DIR=/tmp/uv-cache uv run isort --check-only warp_ingest/ingestion_daemon/service_dependencies.py warp_ingest/ingestion_daemon/__main__.py warp_ingest/ingestion_daemon/app.py warp_ingest/ingestion_daemon/auth.py tests/test_package_metadata.py
  • env UV_CACHE_DIR=/tmp/uv-cache uv build --wheel --out-dir /tmp/warp-ingest-dist

@JSv4
JSv4 marked this pull request as ready for review July 6, 2026 06:32
@JSv4 JSv4 changed the title [codex] Split parser and service install extras Split parser and service install extras Jul 6, 2026
@JSv4
JSv4 marked this pull request as draft July 6, 2026 06:40
@JSv4
JSv4 marked this pull request as ready for review July 6, 2026 06:42
@JSv4
JSv4 merged commit 946c672 into main Jul 6, 2026
10 checks passed
@JSv4
JSv4 deleted the feature/optional-install-extras branch July 6, 2026 06:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant