Skip to content

Rename --uv-with to --build-with; reject constraints on unsupported ecosystems - #6116

Merged
JAORMX merged 1 commit into
mainfrom
build-with-rename
Jul 28, 2026
Merged

Rename --uv-with to --build-with; reject constraints on unsupported ecosystems#6116
JAORMX merged 1 commit into
mainfrom
build-with-rename

Conversation

@JAORMX

@JAORMX JAORMX commented Jul 28, 2026

Copy link
Copy Markdown
Collaborator

Follow-up to #6111, while the flag is hours old and unreleased.

The build-time dependency-constraint concept is per-ecosystem, not uv-specific, so the flag becomes --build-with on thv run and thv build, documented as "interpreted per package ecosystem" — for uvx:// that means PEP 508 specifiers passed to uv tool install --with (e.g. --build-with 'mcp<2'). Future ecosystems can map it to their native mechanism (npm overrides, Go module directives) if/when their builders can support it. RuntimeConfig.UVWithBuildWith (wire tag build_with); swagger and CLI docs regenerated.

This also fixes a wart from #6111: a non-empty constraint list on npx:// or go:// builds was silently ignored. createTemplateData now rejects it with a clear error, so nobody is left believing an unsupported constraint applied — covered by a new test across both unsupported transports, plus the uvx acceptance case.

Lint clean, all touched package tests green, arxiv e2e updated to the new flag name.

🤖 Generated with Claude Code

…cosystems

The constraint concept is per-ecosystem, not uv-specific: --build-with
now names the generic surface, documented as interpreted per package
ecosystem (uvx://: PEP 508 specifiers via 'uv tool install --with').
RuntimeConfig.UVWith becomes BuildWith (wire tag build_with).

Also fixes a wart from #6111: a non-empty constraint list on npx:// or
go:// builds was silently ignored; createTemplateData now rejects it
so users are not left believing an unsupported constraint applied.

Flag shipped hours ago and is unreleased, so the rename is free.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@github-actions github-actions Bot added the size/S Small PR: 100-299 lines changed label Jul 28, 2026
@codecov

codecov Bot commented Jul 28, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.41%. Comparing base (ac16484) to head (73941e7).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #6116   +/-   ##
=======================================
  Coverage   72.40%   72.41%           
=======================================
  Files         733      733           
  Lines       75845    75849    +4     
=======================================
+ Hits        54918    54927    +9     
+ Misses      17034    17031    -3     
+ Partials     3893     3891    -2     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@JAORMX
JAORMX merged commit 333d71d into main Jul 28, 2026
48 checks passed
@JAORMX
JAORMX deleted the build-with-rename branch July 28, 2026 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/S Small PR: 100-299 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants