Use PathBuf types in Source enum (#6708)

crates/uv-distribution/src/metadata/lowering.rs

@@ -257,6 +257,8 @@ pub enum LoweringError {
     EditableFile(String),
     #[error(transparent)]
     ParsedUrl(#[from] ParsedUrlError),
+    #[error("Path must be UTF-8: `{0}`")]
+    NonUtf8Path(PathBuf),
     #[error(transparent)] // Function attaches the context
     RelativeTo(io::Error),
 }
@@ -264,7 +266,7 @@ pub enum LoweringError {
 /// Convert a Git source into a [`RequirementSource`].
 fn git_source(
     git: &Url,
-    subdirectory: Option<String>,
+    subdirectory: Option<PathBuf>,
     rev: Option<String>,
     tag: Option<String>,
     branch: Option<String>,
@@ -282,7 +284,10 @@ fn git_source(
     if let Some(rev) = reference.as_str() {
         url.set_path(&format!("{}@{}", url.path(), rev));
     }
-    if let Some(subdirectory) = &subdirectory {
+    if let Some(subdirectory) = subdirectory.as_ref() {
+        let subdirectory = subdirectory
+            .to_str()
+            .ok_or_else(|| LoweringError::NonUtf8Path(subdirectory.clone()))?;
         url.set_fragment(Some(&format!("subdirectory={subdirectory}")));
     }
     let url = VerbatimUrl::from_url(url);
@@ -294,17 +299,20 @@ fn git_source(
         repository,
         reference,
         precise: None,
-        subdirectory: subdirectory.map(PathBuf::from),
+        subdirectory,
     })
 }
 
 /// Convert a URL source into a [`RequirementSource`].
-fn url_source(url: Url, subdirectory: Option<String>) -> Result<RequirementSource, LoweringError> {
+fn url_source(url: Url, subdirectory: Option<PathBuf>) -> Result<RequirementSource, LoweringError> {
     let mut verbatim_url = url.clone();
     if verbatim_url.fragment().is_some() {
         return Err(LoweringError::ForbiddenFragment(url));
     }
-    if let Some(subdirectory) = &subdirectory {
+    if let Some(subdirectory) = subdirectory.as_ref() {
+        let subdirectory = subdirectory
+            .to_str()
+            .ok_or_else(|| LoweringError::NonUtf8Path(subdirectory.clone()))?;
         verbatim_url.set_fragment(Some(subdirectory));
     }
 

crates/uv-workspace/src/pyproject.rs

@@ -314,7 +314,7 @@ pub enum Source {
         /// The repository URL (without the `git+` prefix).
         git: Url,
         /// The path to the directory with the `pyproject.toml`, if it's not in the archive root.
-        subdirectory: Option<String>,
+        subdirectory: Option<PathBuf>,
         // Only one of the three may be used; we'll validate this later and emit a custom error.
         rev: Option<String>,
         tag: Option<String>,
@@ -331,13 +331,13 @@ pub enum Source {
         url: Url,
         /// For source distributions, the path to the directory with the `pyproject.toml`, if it's
         /// not in the archive root.
-        subdirectory: Option<String>,
+        subdirectory: Option<PathBuf>,
     },
     /// The path to a dependency, either a wheel (a `.whl` file), source distribution (a `.zip` or
     /// `.tar.gz` file), or source tree (i.e., a directory containing a `pyproject.toml` or
     /// `setup.py` file in the root).
     Path {
-        path: String,
+        path: PathBuf,
         /// `false` by default.
         editable: Option<bool>,
     },
@@ -355,12 +355,12 @@ pub enum Source {
     /// A catch-all variant used to emit precise error messages when deserializing.
     CatchAll {
         git: String,
-        subdirectory: Option<String>,
+        subdirectory: Option<PathBuf>,
         rev: Option<String>,
         tag: Option<String>,
         branch: Option<String>,
         url: String,
-        patch: String,
+        path: PathBuf,
         index: String,
         workspace: bool,
     },
@@ -437,16 +437,13 @@ impl Source {
                 editable,
                 path: relative_to(&install_path, root)
                     .or_else(|_| std::path::absolute(&install_path))
-                    .map_err(SourceError::Absolute)?
-                    .to_str()
-                    .ok_or_else(|| SourceError::NonUtf8Path(install_path))?
-                    .to_string(),
+                    .map_err(SourceError::Absolute)?,
             },
             RequirementSource::Url {
                 subdirectory, url, ..
             } => Source::Url {
                 url: url.to_url(),
-                subdirectory: subdirectory.map(|path| path.to_string_lossy().into_owned()),
+                subdirectory,
             },
             RequirementSource::Git {
                 repository,
@@ -470,7 +467,7 @@ impl Source {
                         tag,
                         branch,
                         git: repository,
-                        subdirectory: subdirectory.map(|path| path.to_string_lossy().into_owned()),
+                        subdirectory,
                     }
                 } else {
                     Source::Git {
@@ -478,7 +475,7 @@ impl Source {
                         tag,
                         branch,
                         git: repository,
-                        subdirectory: subdirectory.map(|path| path.to_string_lossy().into_owned()),
+                        subdirectory,
                     }
                 }
             }

uv.schema.json

@@ -1252,7 +1252,7 @@
           "required": [
             "git",
             "index",
-            "patch",
+            "path",
             "url",
             "workspace"
           ],
@@ -1269,7 +1269,7 @@
             "index": {
               "type": "string"
             },
-            "patch": {
+            "path": {
               "type": "string"
             },
             "rev": {