Conversation
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
| - name: Install uv | ||
| uses: astral-sh/setup-uv@v5 | ||
| with: | ||
| python-version: ${{ env.PYTHON_VERSION }} |
There was a problem hiding this comment.
Smoke test fails: bare pytest call after venv activation removed
High Severity
The old CI workflows ran source .venv/bin/activate before make docker-test, putting pytest on PATH. This PR removes that activation and uses uv sync instead, but docker-smoke-test.sh (line 83) still calls bare pytest — not uv run pytest. Since uv sync creates a .venv without activating it, and setup-uv only adds uv to PATH, pytest won't be found. Every other pytest invocation in the Makefile was correctly updated to use uv run, but this script was missed.


Note
Medium Risk
Broad build/CI/Docker dependency-management migration (tooling, lockfile, image build steps) could break installs or release pipelines if
uv.lock/workflow assumptions are off, but runtime application logic is largely unchanged.Overview
Migrates the project from
pip/requirements/*.txttouvby introducing a fullpyproject.toml(including test extras anduvconstraints) and switching installs/tests/linting in theMakefile, Docker image, and GitHub Actions touv sync/uv runwithuv.lockas the reproducible source of truth.CI/release automation is updated accordingly: Dependabot now targets the repo root, the library-bump workflow upgrades via
uv lock --upgrade, caching is based onuv.lock, Docker publish reads version fromprepline_general/api/__version__.py, and legacy pinnedrequirements/*,setup.cfg,preprocessing-pipeline-family.yaml, the Rocky Linux Dockerfile, and the pandoc install script are removed. Tests are tweaked for stability (more tolerant OCR assertions) and the version is bumped to0.1.0with a changelog entry.Written by Cursor Bugbot for commit 1ebbd36. This will update automatically on new commits. Configure here.