PYTHON-6091 Pin a consistent uv binary version in CI and locally by blink1073 · Pull Request #3053 · mongodb/mongo-python-driver

blink1073 · 2026-09-11T21:47:08Z

PYTHON-6091

Changes in this PR

Enforces a consistent version of uv in Evergreen CI, GitHub Actions, and local development.

Pins the uv binary version using [tool.uv] required-version in pyproject.toml, enforced locally and honored by astral-sh/setup-uv.
Bumps the uv version to one that can install Python 3.15 binaries.
Bootstrapping uv is now much more robust, using ensure-uv.sh from drivers-tools.

Test Plan

Tested on a spawn host and on a variety of hosts with patch build.

Checklist

Checklist for Author

Did you update the changelog (if necessary)? N/A
Is there test coverage? Yes
Is any followup work tracked in a JIRA ticket? If so, add link(s). PYTHON-6084 builds on this work to clean up Python selection and installation, including pre-release versions that are not yet in the toolchain.

Checklist for Reviewer

Does the title of the PR reference a JIRA Ticket?
Do you fully understand the implementation? (Would you be comfortable explaining how this code works to someone else?)
Is all relevant documentation (README or docstring) updated?

…ired-version)

…comment

…it is absent

…ng uv

Copilot

🟡 Changes recommended

Windows self-updates and PATH handling can leave setup failing or the pinned tools unavailable.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

Pins one uv version across local development, GitHub Actions, and Evergreen.

Changes:

Adds the central uv version pin and documentation.
Installs pinned tooling into dedicated binary directories.
Updates CI and spawn-host PATH configuration.

File summaries

File	Description
`pyproject.toml`	Defines the required `uv` version.
`CONTRIBUTING.md`	Documents version management and installation.
`.evergreen/scripts/setup-system.sh`	Configures non-CI shell PATH persistence.
`.evergreen/scripts/setup-dev-env.sh`	Reorders dependency and Python setup.
`.evergreen/scripts/install-dependencies.sh`	Discovers and installs pinned tooling.
`.evergreen/scripts/configure-env.sh`	Establishes task-local binary paths.

Review details

Files reviewed: 6/6 changed files
Comments generated: 3
Review effort level: Balanced

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

…id Windows uv self-overwrite

Copilot

🟡 Changes recommended

Version enforcement can block bootstrap installation, and the CI fallback directory is shared rather than task-local.

Get a fresh assessment by requesting another Copilot review.

Review details

Suppressed comments (3)

Previously missed (1) — in code that hasn't changed since the last review.

.evergreen/scripts/configure-env.sh:26

The /tmp/pymongo-bin fallback is host-global, not task-local as documented. Concurrent tasks or later tasks under another user can race on, overwrite, or be unable to update the same executables; use the checkout's existing task-local .local directory instead.

.evergreen/scripts/install-dependencies.sh:84

An unpinned uv found on the toolchain or PATH is invoked from the project root here, so the newly added required-version check rejects it before it can install that required version. Bootstrap with project config disabled for this command.

      uv tool install -q --force --from "uv${_uv_pin}" uv

.evergreen/scripts/install-dependencies.sh:102

After an external bootstrap uv installs the pinned binary, PATH still resolves uv to that external (potentially mismatched) version. If just is absent, this command therefore fails the project's required-version check; invoke the newly installed pinned binary instead.

    uv tool install rust-just

Files reviewed: 6/6 changed files
Comments generated: 2
Review effort level: Balanced

…prepend bin dir

Copilot

🟡 Changes recommended

Shared CI paths and stale spawn-host environment files can cause races or make installed tools unavailable.

Get a fresh assessment by requesting another Copilot review.

Review details

Files reviewed: 6/6 changed files
Comments generated: 2
Review effort level: Balanced

Copilot

🔵 Needs a closer look

The revised bootstrap does not provision a pinned uvx executable as intended.

Review details

Suppressed comments (1)

Previously missed (1) — in code that hasn't changed since the last review.

.evergreen/scripts/install-dependencies.sh:84

The new bootstrap no longer provisions uvx. The pinned PyPI uv package builds only the uv executable, while the removed toolchain bootstrap explicitly linked both uv and uvx; therefore a fresh toolchain-based setup leaves no uvx, and the astral fallback can leave an unpinned uvx beside the pinned uv. This also conflicts with the PR's stated intent to discover/install uv/uvx/just. Please create a platform-appropriate uvx alias/copy from the pinned binary after this install (including when the existing target uv already matches).

Files reviewed: 6/7 changed files
Comments generated: 0 new
Review effort level: Balanced

codecov · 2026-09-12T00:44:22Z

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

…into PYTHON-6091

Copilot

Copilot review overview

🟡 Changes recommended

The bootstrap currently mishandles local PATH persistence and may expose or execute values from credential-bearing uv environment variables.

Get a fresh assessment by requesting another Copilot review.

Review effort: Balanced
Findings: 2 · 2

Open (4)

Avoid persisting credential-bearing UV_* environment variables · New
Shell-escape values before writing them to env.sh · New
Add uv's install directory to PATH before running uv sync · New
Select the shell rc file based on $SHELL · New

…THON-6091

…into PYTHON-6091

…SHELL, shared toolchain helper

…v.sh

… python3

Copilot

Copilot review overview

🟡 Changes recommended

The bootstrap loses task-local Python storage configuration and performs costly, side-effecting dependency synchronization as a version check.

Get a fresh assessment by requesting another Copilot review.

Review effort: Balanced
Findings: 2

Open (2)

Avoid full uv sync during version probing · New
Persist UV_PYTHON_INSTALL_DIR for the parent environment · New

Resolved since last review (4)

Shell-escape values before writing them to env.sh
Avoid persisting credential-bearing UV_* environment variables
Select the shell rc file based on $SHELL
Add uv's install directory to PATH before running uv sync

…N_INSTALL_DIR

Copilot

Copilot review overview

🔵 Needs a closer look

The cross-platform bootstrap and persistent environment changes require final human validation across supported Evergreen and local host configurations.

Review effort: Balanced
Findings: None

Resolved since last review (2)

Persist UV_PYTHON_INSTALL_DIR for the parent environment
Avoid full uv sync during version probing

sleepyStick · 2026-09-24T00:35:05Z

    DRIVERS_TOOLS_BINARIES=$(cygpath -m "$DRIVERS_TOOLS_BINARIES")
    MONGODB_BINARIES=$(cygpath -m "$MONGODB_BINARIES")
-    PYMONGO_BIN_DIR=$(cygpath -m "$PYMONGO_BIN_DIR")
+    PYMONGO_BIN_DIR=$(cygpath -u "$PYMONGO_BIN_DIR")


wait why did this change do -u?

It made a bunch of the other consumers simpler, like comparing paths. But I see your point about being consistent here, so I added PYMONGO_BIN_DIR_POSIX as well. Sound good?

The variable stays in native (Windows) form on cygwin, uniform with the other paths in configure-env.sh, for consumers like uv; the derived PYMONGO_BIN_DIR_POSIX is used for bash PATH contexts (PATH_EXT, the dedupe/prepend steps, and the shell rc writer).

blink1073 added 8 commits September 11, 2026 06:45

PYTHON-6091 Pin a consistent uv binary version in CI and locally

93b8645

PYTHON-6091 Drop unneeded test-python action bump (uv auto-reads requ…

10fc7a2

…ired-version)

PYTHON-6091 Address review: always install pinned uv and fix bin dir/…

239c968

…comment

PYTHON-6091 Fix shellcheck SC2155 in Windows uv path handling

097964a

PYTHON-6091 Source PATH from env.sh; only prepend local bin dir when …

e484fd1

…it is absent

PYTHON-6091 Use env.sh on CI and ~/.local/bin on spawn/local hosts

17d616b

PYTHON-6091 Note non-CI hosts include VMs in bin dir comment

81abd31

PYTHON-6091 Fix uv pin on no-toolchain hosts; avoid overwriting runni…

04c1ef2

…ng uv

blink1073 requested a balanced review from Copilot September 11, 2026 21:47

Copilot started reviewing on behalf of blink1073 September 11, 2026 21:48 View session

Copilot AI reviewed Sep 11, 2026

View reviewed changes

Comment thread .evergreen/scripts/install-dependencies.sh Outdated

Comment thread .evergreen/scripts/setup-dev-env.sh Outdated

Comment thread .evergreen/scripts/setup-system.sh Outdated

PYTHON-6091 Address Copilot: prepend PATH, persist .bashrc entry, avo…

15760ac

…id Windows uv self-overwrite

blink1073 requested a balanced review from Copilot September 11, 2026 21:57

Copilot started reviewing on behalf of blink1073 September 11, 2026 21:57 View session

Copilot AI reviewed Sep 11, 2026

View reviewed changes

Comment thread .evergreen/scripts/install-dependencies.sh Outdated

Comment thread .evergreen/scripts/setup-dev-env.sh Outdated

PYTHON-6091 Address Copilot: no-config bootstrap installs and always …

c394180

…prepend bin dir

blink1073 requested a balanced review from Copilot September 11, 2026 23:16

Copilot started reviewing on behalf of blink1073 September 11, 2026 23:17 View session

Copilot AI reviewed Sep 11, 2026

View reviewed changes

Comment thread .evergreen/scripts/configure-env.sh Outdated

Comment thread .evergreen/scripts/setup-system.sh Outdated

PYTHON-6091 Use checkout-local bin dir and persist actual install dir

50b804f

blink1073 requested a balanced review from Copilot September 11, 2026 23:26

Copilot started reviewing on behalf of blink1073 September 11, 2026 23:27 View session

Copilot AI reviewed Sep 11, 2026

View reviewed changes

blink1073 added 6 commits September 14, 2026 05:21

Merge branch 'PYTHON-6091' of github.com:blink1073/mongo-python-driver …

54a4229

…into PYTHON-6091

more cleanup

10ff9d6

PYTHON-6091 Use toolchain python3 for setup-uv.py

2a2212d

PYTHON-6091 Convert setup-uv.py path for Windows python3

22a9b9a

PYTHON-6091 Bypass required-version via uv run --no-config

c3daaf9

PYTHON-6091 Add toolchain python to PATH and fix Windows script path

905d3bf

blink1073 added 2 commits September 22, 2026 06:42

PYTHON-6091 Address review: export bin dir PATH, py3.6 compat, docs

a1cc070

Merge branch 'main' into PYTHON-6091

3f30df5

blink1073 requested a balanced review from Copilot September 22, 2026 11:57

Copilot started reviewing on behalf of blink1073 September 22, 2026 11:58 View session

Copilot AI reviewed Sep 22, 2026

View reviewed changes

Comment thread .evergreen/scripts/setup-uv.py Outdated

Comment thread .evergreen/scripts/setup-uv.py Outdated

Comment thread .evergreen/scripts/setup-dev-env.sh

Comment thread .evergreen/scripts/setup-dev-env.sh Outdated

blink1073 added 6 commits September 22, 2026 10:05

Merge branch 'main' of github.com:mongodb/mongo-python-driver into PY…

e1aedbb

…THON-6091

Merge branch 'PYTHON-6091' of github.com:blink1073/mongo-python-driver …

b60d904

…into PYTHON-6091

PYTHON-6091 Prepend toolchain python dir to PATH before setup-uv.py

784b545

PYTHON-6091 Address copilot review: credential-safe env.sh, rc from $…

95d8c07

…SHELL, shared toolchain helper

PYTHON-6091 Revert toolchain PATH re-derivation: rely on configure-en…

26da029

…v.sh

PYTHON-6091 Drop py3.6 compat from setup-uv.py: hosts have known-good…

9d0d31c

… python3

blink1073 requested a balanced review from Copilot September 22, 2026 15:31

Copilot started reviewing on behalf of blink1073 September 22, 2026 15:31 View session

Copilot AI reviewed Sep 22, 2026

View reviewed changes

Comment thread .evergreen/scripts/install-dependencies.sh Outdated

Comment thread .evergreen/scripts/setup-uv.py Outdated

PYTHON-6091 Probe pinned uv via setup-uv.py --check; persist UV_PYTHO…

d960278

…N_INSTALL_DIR

blink1073 requested a balanced review from Copilot September 22, 2026 15:42

Copilot started reviewing on behalf of blink1073 September 22, 2026 15:43 View session

Copilot AI reviewed Sep 22, 2026

View reviewed changes

blink1073 marked this pull request as ready for review September 23, 2026 00:09

blink1073 requested a review from a team as a code owner September 23, 2026 00:09

blink1073 requested a review from sleepyStick September 23, 2026 00:09

sleepyStick reviewed Sep 24, 2026

View reviewed changes

blink1073 force-pushed the PYTHON-6091 branch from 482cb5a to 3481a36 Compare September 24, 2026 01:45

blink1073 force-pushed the PYTHON-6091 branch from 3481a36 to 37553ac Compare September 24, 2026 01:48

blink1073 requested a review from sleepyStick September 24, 2026 01:50

sleepyStick approved these changes Sep 24, 2026

View reviewed changes

blink1073 merged commit e23bd66 into mongodb:main Sep 24, 2026
105 of 108 checks passed

This was referenced Sep 24, 2026

Ignore uv configuration when installing ecosystem dependencies astral-sh/ecosystem-analyzer#205

Merged

Ignore uv configuration when installing dependencies hauntsaninja/mypy_primer#262

Merged

Conversation

blink1073 commented Sep 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Changes in this PR

Test Plan

Checklist

Checklist for Author

Checklist for Reviewer

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🟡 Changes recommended

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🟡 Changes recommended

Suppressed comments (3)

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🟡 Changes recommended

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🔵 Needs a closer look

Suppressed comments (1)

Uh oh!

codecov Bot commented Sep 12, 2026

Codecov Report

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Copilot review overview

🟡 Changes recommended

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Copilot review overview

🟡 Changes recommended

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Copilot review overview

🔵 Needs a closer look

Uh oh!

sleepyStick Sep 24, 2026

Choose a reason for hiding this comment

Uh oh!

blink1073 Sep 24, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

blink1073 commented Sep 11, 2026 •

edited

Loading