Add `uv workspace dir` command by mikaylathompson · Pull Request #16678 · astral-sh/uv

mikaylathompson · 2025-11-11T00:30:12Z

Addresses #13636

Prints the path to the workspace root by default, and any of the child packages if requested.

It has a preview flag like workspace metadata, called workspace-dir.

Summary

─> uv workspace dir
/Users/mikayla/code/uv/dev-envs

─> uv workspace dir --package foo-proj
/Users/mikayla/code/uv/dev-envs/foo-proj

─> uv workspace dir --package bar-proj
error: Package `bar-proj` not found in workspace.

Test Plan

Unit tests added.

Signed-off-by: Mikayla Thompson <[email protected]>

crates/uv/src/commands/workspace/dir.rs

zanieb · 2025-11-11T16:50:52Z

crates/uv/src/commands/workspace/dir.rs

+    preview: Preview,
+    printer: Printer,
+) -> Result<ExitStatus> {
+    if preview.is_enabled(PreviewFeatures::WORKSPACE_METADATA) {


I think this should be a separate preview flag so we can stabilize things separately

crates/uv/src/lib.rs

crates/uv/tests/it/workspace_dir.rs

crates/uv-cli/src/lib.rs

zanieb · 2025-11-11T16:57:17Z

crates/uv-cli/src/lib.rs

@@ -6853,11 +6853,20 @@ pub enum WorkspaceCommand {
    /// Display package metadata.


(My own doc here is bad, I should fix that)

crates/uv/src/commands/workspace/dir.rs

Co-authored-by: Zanie Blue <[email protected]>

codspeed-hq · 2025-11-11T17:36:14Z

CodSpeed Performance Report

Merging #16678 will not alter performance

_{Comparing mikayla/workspace-dir (e249d84) with main (63ab247)}

Summary

✅ 6 untouched

Signed-off-by: Mikayla Thompson <[email protected]>

konstin · 2025-11-11T18:34:11Z

crates/uv/src/commands/workspace/dir.rs

+    let dir: &Path = match package_name {
+        None => workspace.install_path().as_path(),
+        Some(package) => {
+            if let Some(p) = workspace.packages().get(&package) {
+                p.root().as_path()
+            } else {
+                bail!("Package `{package}` not found in workspace.")
+            }
+        }
+    };


This code compiles without the type cast:

Suggested change

let dir: &Path = match package_name {

None => workspace.install_path().as_path(),

Some(package) => {

if let Some(p) = workspace.packages().get(&package) {

p.root().as_path()

} else {

bail!("Package `{package}` not found in workspace.")

}

}

};

let dir = match package_name {

None => workspace.install_path(),

Some(package) => {

if let Some(p) = workspace.packages().get(&package) {

p.root()

} else {

bail!("Package `{package}` not found in workspace.")

}

}

};

konstin · 2025-11-11T18:35:48Z

crates/uv/tests/it/workspace_dir.rs

+    );
+}
+
+// Workspace dir output when run with `--package`


nit:

Suggested change

// Workspace dir output when run with `--package`

/// Workspace dir output when run with `--package`.

Signed-off-by: Mikayla Thompson <[email protected]>

…ollowups (#16690) ## Summary 1. Discussed in review of #16678 that println should be replaced by using `printer`. The `println` pattern was pretty consistent across all the `dir` commands, so I've updated all of them in this PR (there are some usages of `println` outside of `uv/src/commands` that I didn't touch -- the use cases there seemed more complex and nuanced). 2. I missed two comments in the previous PR before merging, so updates from those are in here as well. ## Test Plan No behavior changes, existing tests for all commands pass. --------- Signed-off-by: Mikayla Thompson <[email protected]>

This MR contains the following updates: | Package | Update | Change | |---|---|---| | [astral-sh/uv](https://github.com/astral-sh/uv) | patch | `0.9.8` -> `0.9.9` | MR created with the help of [el-capitano/tools/renovate-bot](https://gitlab.com/el-capitano/tools/renovate-bot). **Proposed changes to behavior should be submitted there as MRs.** --- ### Release Notes <details> <summary>astral-sh/uv (astral-sh/uv)</summary> ### [`v0.9.9`](https://github.com/astral-sh/uv/blob/HEAD/CHANGELOG.md#099) [Compare Source](astral-sh/uv@0.9.8...0.9.9) Released on 2025-11-12. ##### Deprecations - Deprecate use of `--project` in `uv init` ([#16674](astral-sh/uv#16674)) ##### Enhancements - Add iOS support to Python interpreter discovery ([#16686](astral-sh/uv#16686)) - Reject ambiguously parsed URLs ([#16622](astral-sh/uv#16622)) - Allow explicit values in `uv version --bump` ([#16555](astral-sh/uv#16555)) - Warn on use of managed pre-release Python versions when a stable version is available ([#16619](astral-sh/uv#16619)) - Allow signing trampolines on Windows by using `.rcdata` to store metadata ([#15068](astral-sh/uv#15068)) - Add `--only-emit-workspace` and similar variants to `uv export` ([#16681](astral-sh/uv#16681)) ##### Preview features - Add `uv workspace dir` command ([#16678](astral-sh/uv#16678)) - Add `uv workspace metadata` command ([#16516](astral-sh/uv#16516)) ##### Configuration - Add `UV_NO_DEFAULT_GROUPS` environment variable ([#16645](astral-sh/uv#16645)) ##### Bug fixes - Remove `torch-model-archiver` and `torch-tb-profiler` from PyTorch backend ([#16655](astral-sh/uv#16655)) - Fix Pixi environment detection ([#16585](astral-sh/uv#16585)) ##### Documentation - Fix `CMD` path in FastAPI Dockerfile ([#16701](astral-sh/uv#16701)) </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever MR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this MR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this MR, check this box --- This MR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).

mikaylathompson changed the base branch from main to zb/metadata November 11, 2025 00:30

zanieb force-pushed the zb/metadata branch 2 times, most recently from 83d8f8a to 8658f99 Compare November 11, 2025 15:34

Base automatically changed from zb/metadata to main November 11, 2025 15:46

Initial implementation of workspace dir

aaca6e5

Signed-off-by: Mikayla Thompson <[email protected]>

mikaylathompson force-pushed the mikayla/workspace-dir branch from f602976 to aaca6e5 Compare November 11, 2025 15:49

mikaylathompson temporarily deployed to uv-test-registries November 11, 2025 15:51 — with GitHub Actions Inactive

mikaylathompson had a problem deploying to uv-test-registries November 11, 2025 16:38 — with GitHub Actions Error

mikaylathompson force-pushed the mikayla/workspace-dir branch from 311c99d to 98bcecf Compare November 11, 2025 16:38

Add tests, clean up, doc string

8401729

Signed-off-by: Mikayla Thompson <[email protected]>

mikaylathompson force-pushed the mikayla/workspace-dir branch from 98bcecf to 8401729 Compare November 11, 2025 16:39

mikaylathompson marked this pull request as ready for review November 11, 2025 16:39

mikaylathompson temporarily deployed to uv-test-registries November 11, 2025 16:41 — with GitHub Actions Inactive

Formatting fixes

e2437c7

Signed-off-by: Mikayla Thompson <[email protected]>

mikaylathompson temporarily deployed to uv-test-registries November 11, 2025 16:46 — with GitHub Actions Inactive