Avoid using editable tag in lockfile for non-package dependencies

crates/distribution-types/src/cached.rs

@@ -34,6 +34,7 @@ pub struct CachedDirectUrlDist {
     pub url: VerbatimUrl,
     pub path: PathBuf,
     pub editable: bool,
+    pub r#virtual: bool,
     pub hashes: Vec<HashDigest>,
 }
 
@@ -57,13 +58,15 @@ impl CachedDist {
                 hashes,
                 path,
                 editable: false,
+                r#virtual: false,
             }),
             Dist::Built(BuiltDist::Path(dist)) => Self::Url(CachedDirectUrlDist {
                 filename,
                 url: dist.url,
                 hashes,
                 path,
                 editable: false,
+                r#virtual: false,
             }),
             Dist::Source(SourceDist::Registry(_dist)) => Self::Registry(CachedRegistryDist {
                 filename,
@@ -76,27 +79,31 @@ impl CachedDist {
                 hashes,
                 path,
                 editable: false,
+                r#virtual: false,
             }),
             Dist::Source(SourceDist::Git(dist)) => Self::Url(CachedDirectUrlDist {
                 filename,
                 url: dist.url,
                 hashes,
                 path,
                 editable: false,
+                r#virtual: false,
             }),
             Dist::Source(SourceDist::Path(dist)) => Self::Url(CachedDirectUrlDist {
                 filename,
                 url: dist.url,
                 hashes,
                 path,
                 editable: false,
+                r#virtual: false,
             }),
             Dist::Source(SourceDist::Directory(dist)) => Self::Url(CachedDirectUrlDist {
                 filename,
                 url: dist.url,
                 hashes,
                 path,
                 editable: dist.editable,
+                r#virtual: dist.r#virtual,
             }),
         }
     }
@@ -124,6 +131,7 @@ impl CachedDist {
                         url: dist.url.raw().clone(),
                         install_path: path,
                         editable: dist.editable,
+                        r#virtual: dist.r#virtual,
                     })))
                 } else {
                     Ok(Some(ParsedUrl::try_from(dist.url.to_url())?))
@@ -161,6 +169,7 @@ impl CachedDirectUrlDist {
             hashes,
             path,
             editable: false,
+            r#virtual: false,
         }
     }
 }

crates/distribution-types/src/lib.rs

@@ -309,6 +309,8 @@ pub struct DirectorySourceDist {
     pub install_path: PathBuf,
     /// Whether the package should be installed in editable mode.
     pub editable: bool,
+    /// Whether the package should be built and installed.
+    pub r#virtual: bool,
     /// The URL as it was provided by the user.
     pub url: VerbatimUrl,
 }
@@ -404,6 +406,7 @@ impl Dist {
         url: VerbatimUrl,
         install_path: &Path,
         editable: bool,
+        r#virtual: bool,
     ) -> Result<Dist, Error> {
         // Convert to an absolute path.
         let install_path = path::absolute(install_path)?;
@@ -421,6 +424,7 @@ impl Dist {
             name,
             install_path,
             editable,
+            r#virtual,
             url,
         })))
     }
@@ -458,6 +462,7 @@ impl Dist {
                 url.verbatim,
                 &directory.install_path,
                 directory.editable,
+                directory.r#virtual,
             ),
             ParsedUrl::Git(git) => {
                 Self::from_git_url(name, url.verbatim, git.url, git.subdirectory)

crates/distribution-types/src/resolution.rs

@@ -220,6 +220,7 @@ impl From<&ResolvedDist> for Requirement {
                     install_path: sdist.install_path.clone(),
                     url: sdist.url.clone(),
                     editable: sdist.editable,
+                    r#virtual: sdist.r#virtual,
                 },
             },
             ResolvedDist::Installed(dist) => RequirementSource::Registry {

crates/pypi-types/src/parsed_url.rs

@@ -72,6 +72,7 @@ impl UnnamedRequirementUrl for VerbatimParsedUrl {
                 url: verbatim.to_url(),
                 install_path: verbatim.as_path()?,
                 editable: false,
+                r#virtual: false,
             })
         } else {
             ParsedUrl::Path(ParsedPathUrl {
@@ -101,6 +102,7 @@ impl UnnamedRequirementUrl for VerbatimParsedUrl {
                 url: verbatim.to_url(),
                 install_path: verbatim.as_path()?,
                 editable: false,
+                r#virtual: false,
             })
         } else {
             ParsedUrl::Path(ParsedPathUrl {
@@ -208,15 +210,17 @@ pub struct ParsedDirectoryUrl {
     /// The absolute path to the distribution which we use for installing.
     pub install_path: PathBuf,
     pub editable: bool,
+    pub r#virtual: bool,
 }
 
 impl ParsedDirectoryUrl {
     /// Construct a [`ParsedDirectoryUrl`] from a path requirement source.
-    pub fn from_source(install_path: PathBuf, editable: bool, url: Url) -> Self {
+    pub fn from_source(install_path: PathBuf, editable: bool, r#virtual: bool, url: Url) -> Self {
         Self {
             url,
             install_path,
             editable,
+            r#virtual,
         }
     }
 }
@@ -370,6 +374,7 @@ impl TryFrom<Url> for ParsedUrl {
                     url,
                     install_path: path.clone(),
                     editable: false,
+                    r#virtual: false,
                 }))
             } else {
                 Ok(Self::Path(ParsedPathUrl {

crates/pypi-types/src/requirement.rs

@@ -197,12 +197,14 @@ impl From<Requirement> for pep508_rs::Requirement<VerbatimParsedUrl> {
                 RequirementSource::Directory {
                     install_path,
                     editable,
+                    r#virtual,
                     url,
                 } => Some(VersionOrUrl::Url(VerbatimParsedUrl {
                     parsed_url: ParsedUrl::Directory(ParsedDirectoryUrl {
                         url: url.to_url(),
                         install_path,
                         editable,
+                        r#virtual,
                     }),
                     verbatim: url,
                 })),
@@ -360,6 +362,8 @@ pub enum RequirementSource {
         install_path: PathBuf,
         /// For a source tree (a directory), whether to install as an editable.
         editable: bool,
+        /// For a source tree (a directory), whether the project should be built and installed.
+        r#virtual: bool,
         /// The PEP 508 style URL in the format
         /// `file:///<path>#subdirectory=<subdirectory>`.
         url: VerbatimUrl,
@@ -379,6 +383,7 @@ impl RequirementSource {
             ParsedUrl::Directory(directory) => RequirementSource::Directory {
                 install_path: directory.install_path.clone(),
                 editable: directory.editable,
+                r#virtual: directory.r#virtual,
                 url,
             },
             ParsedUrl::Git(git) => RequirementSource::Git {
@@ -435,11 +440,13 @@ impl RequirementSource {
             Self::Directory {
                 install_path,
                 editable,
+                r#virtual,
                 url,
             } => Some(VerbatimParsedUrl {
                 parsed_url: ParsedUrl::Directory(ParsedDirectoryUrl::from_source(
                     install_path.clone(),
                     *editable,
+                    *r#virtual,
                     url.to_url(),
                 )),
                 verbatim: url.clone(),
@@ -515,11 +522,14 @@ impl RequirementSource {
             RequirementSource::Directory {
                 install_path,
                 editable,
+                r#virtual,
                 url,
+                ..
             } => Ok(Self::Directory {
                 install_path: relative_to(&install_path, path)
                     .or_else(|_| std::path::absolute(install_path))?,
                 editable,
+                r#virtual,
                 url,
             }),
         }
@@ -582,6 +592,8 @@ enum RequirementSourceWire {
     Directory { directory: PortablePathBuf },
     /// Ex) `source = { editable = "/home/ferris/iniconfig" }`
     Editable { editable: PortablePathBuf },
+    /// Ex) `source = { editable = "/home/ferris/iniconfig" }`
+    Virtual { r#virtual: PortablePathBuf },
     /// Ex) `source = { specifier = "foo >1,<2" }`
     Registry {
         #[serde(skip_serializing_if = "VersionSpecifiers::is_empty", default)]
@@ -668,12 +680,17 @@ impl From<RequirementSource> for RequirementSourceWire {
             RequirementSource::Directory {
                 install_path,
                 editable,
+                r#virtual,
                 url: _,
             } => {
                 if editable {
                     Self::Editable {
                         editable: PortablePathBuf::from(install_path),
                     }
+                } else if r#virtual {
+                    Self::Virtual {
+                        r#virtual: PortablePathBuf::from(install_path),
+                    }
                 } else {
                     Self::Directory {
                         directory: PortablePathBuf::from(install_path),
@@ -760,6 +777,7 @@ impl TryFrom<RequirementSourceWire> for RequirementSource {
                 Ok(Self::Directory {
                     install_path: directory,
                     editable: false,
+                    r#virtual: false,
                     url,
                 })
             }
@@ -769,6 +787,17 @@ impl TryFrom<RequirementSourceWire> for RequirementSource {
                 Ok(Self::Directory {
                     install_path: editable,
                     editable: true,
+                    r#virtual: false,
+                    url,
+                })
+            }
+            RequirementSourceWire::Virtual { r#virtual } => {
+                let r#virtual = PathBuf::from(r#virtual);
+                let url = VerbatimUrl::from_path(&r#virtual, &*CWD)?;
+                Ok(Self::Directory {
+                    install_path: r#virtual,
+                    editable: false,
+                    r#virtual: true,
                     url,
                 })
             }
@@ -825,6 +854,7 @@ mod tests {
             source: RequirementSource::Directory {
                 install_path: PathBuf::from(path),
                 editable: false,
+                r#virtual: false,
                 url: VerbatimUrl::from_absolute_path(path).unwrap(),
             },
             origin: None,

crates/uv-distribution/src/index/cached_wheel.rs

@@ -55,6 +55,7 @@ impl CachedWheel {
             url,
             path: self.entry.into_path_buf(),
             editable: false,
+            r#virtual: false,
             hashes: self.hashes,
         }
     }
@@ -66,6 +67,19 @@ impl CachedWheel {
             url,
             path: self.entry.into_path_buf(),
             editable: true,
+            r#virtual: false,
+            hashes: self.hashes,
+        }
+    }
+
+    /// Convert a [`CachedWheel`] into an editable [`CachedDirectUrlDist`].
+    pub fn into_virtual(self, url: VerbatimUrl) -> CachedDirectUrlDist {
+        CachedDirectUrlDist {
+            filename: self.filename,
+            url,
+            path: self.entry.into_path_buf(),
+            editable: false,
+            r#virtual: true,
             hashes: self.hashes,
         }
     }

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

@@ -12,7 +12,7 @@ use pypi_types::{ParsedUrlError, Requirement, RequirementSource, VerbatimParsedU
 use uv_git::GitReference;
 use uv_normalize::PackageName;
 use uv_warnings::warn_user_once;
-use uv_workspace::pyproject::Source;
+use uv_workspace::pyproject::{PyProjectToml, Source};
 use uv_workspace::Workspace;
 
 #[derive(Debug, Clone)]
@@ -148,10 +148,21 @@ impl LoweredRequirement {
                         "Invalid path in file URL",
                     ))
                 })?;
-                RequirementSource::Directory {
-                    install_path,
-                    url,
-                    editable: true,
+
+                if member.pyproject_toml().is_package() {
+                    RequirementSource::Directory {
+                        install_path,
+                        url,
+                        editable: true,
+                        r#virtual: false,
+                    }
+                } else {
+                    RequirementSource::Directory {
+                        install_path,
+                        url,
+                        editable: false,
+                        r#virtual: true,
+                    }
                 }
             }
             Source::CatchAll { .. } => {
@@ -372,17 +383,41 @@ fn path_source(
             "Invalid path in file URL",
         ))
     })?;
+
     let is_dir = if let Ok(metadata) = install_path.metadata() {
         metadata.is_dir()
     } else {
         install_path.extension().is_none()
     };
     if is_dir {
-        Ok(RequirementSource::Directory {
-            install_path,
-            url,
-            editable,
-        })
+        if editable {
+            Ok(RequirementSource::Directory {
+                install_path,
+                url,
+                editable,
+                r#virtual: false,
+            })
+        } else {
+            // Determine whether the project is a package or virtual.
+            let is_package = {
+                let pyproject_path = install_path.join("pyproject.toml");
+                fs_err::read_to_string(&pyproject_path)
+                    .ok()
+                    .and_then(|contents| PyProjectToml::from_string(contents).ok())
+                    .map(|pyproject_toml| pyproject_toml.is_package())
+                    .unwrap_or(true)
+            };
+
+            // If a project is not a package, treat it as a virtual dependency.
+            let r#virtual = !is_package;
+
+            Ok(RequirementSource::Directory {
+                install_path,
+                url,
+                editable,
+                r#virtual,
+            })
+        }
     } else {
         if editable {
             return Err(LoweringError::EditableFile(url.to_string()));