Skip to content

chore: harden uv dependency resolution#3014

Merged
seratch merged 1 commit into
mainfrom
dev/codex/package-manager-safety-dry-run
Apr 23, 2026
Merged

chore: harden uv dependency resolution#3014
seratch merged 1 commit into
mainfrom
dev/codex/package-manager-safety-dry-run

Conversation

@mcgrew-oai
Copy link
Copy Markdown
Contributor

This pull request hardens package-manager dependency resolution for this public repository. It adds uv age-gate and safe index policy, pins GitHub Actions setup for uv to an immutable action release, and explicitly installs uv 0.11.7 in CI so the relative exclude-newer = "7 days" policy is supported.

What changed

  • Add [tool.uv] and [tool.uv.pip] policy with exclude-newer = "7 days" and index-strategy = "first-index".
  • Refresh uv.lock so the lock records the resolved cutoff for the configured 7-day policy.
  • Pin astral-sh/setup-uv to an immutable setup-uv v8.1.0 commit and document the action and uv package versions inline.
  • Update the sandbox tutorial Docker image to copy a digest-pinned uv binary and install Python dependencies with uv pip install --system using age-gate and safe index controls.

Why

This reduces package-compromise and dependency-confusion risk for CI, local lock resolution, and the tutorial container build without changing the SDK public API.

Verification

  • Package-manager audit completed with no unsuppressed follow-up items.
  • uv build
  • Docker build for examples/sandbox/tutorials/Dockerfile
  • make typecheck
  • make tests

Note: the full helper script that runs typecheck and tests concurrently reproduced an existing timing-sensitive test failure on the unchanged baseline worktree. Running the project checks sequentially passed.

@mcgrew-oai mcgrew-oai marked this pull request as ready for review April 23, 2026 17:18
@mcgrew-oai mcgrew-oai requested review from sdcoffey and seratch April 23, 2026 17:19
@seratch seratch merged commit c2cb031 into main Apr 23, 2026
11 checks passed
@seratch seratch deleted the dev/codex/package-manager-safety-dry-run branch April 23, 2026 23:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants