Refine --no-sources reinstall logic in do_sync function

- Updated the logic to selectively reinstall URL-based packages only when the resolution selects a registry distribution, preventing unnecessary reinstalls for packages specified via direct URL.
- Enhanced debug logging for better clarity on package evaluation during the reinstall process.
- Adjusted related test expectations to reflect the changes in package installation behavior.

This change improves the efficiency of package management when source installations are disabled.

Author: yumeminami

crates/uv/src/commands/project/sync.rs

@@ -19,7 +19,8 @@ use uv_configuration::{
 use uv_dispatch::BuildDispatch;
 use uv_distribution::LoweredExtraBuildDependencies;
 use uv_distribution_types::{
-    DirectorySourceDist, Dist, Index, Name, Requirement, Resolution, ResolvedDist, SourceDist,
+    BuiltDist, DirectorySourceDist, Dist, Index, Name, Requirement, Resolution, ResolvedDist,
+    SourceDist,
 };
 use uv_fs::{PortablePathBuf, Simplified};
 use uv_installer::SitePackages;
@@ -788,22 +789,50 @@ pub(super) async fn do_sync(
 
     let site_packages = SitePackages::from_environment(venv)?;
 
-    // When --no-sources is used, force reinstall of URL-based packages
-    // to ensure they are replaced with registry versions
+    // When --no-sources is used, force reinstall only when the current install is editable/URL
+    // but the resolution selects a registry distribution for that package. This avoids
+    // unnecessary reinstalls for packages that are legitimately specified via direct URL.
     let reinstall = if matches!(sources, uv_configuration::SourceStrategy::Disabled) {
-        tracing::debug!("Sources are disabled, checking for URL-based packages to reinstall");
-        let mut updated_reinstall = reinstall.clone();
+        tracing::debug!("Sources are disabled; evaluating packages to selectively reinstall");
+
+        // Collect package names which the resolution intends to install from a registry.
+        let resolved_registry_names = {
+            let mut names = std::collections::HashSet::new();
+            for resolved in resolution.distributions() {
+                if let ResolvedDist::Installable { dist, .. } = resolved {
+                    match dist.as_ref() {
+                        Dist::Built(BuiltDist::Registry(_))
+                        | Dist::Source(SourceDist::Registry(_)) => {
+                            names.insert(dist.name().clone());
+                        }
+                        _ => {}
+                    }
+                }
+            }
+            names
+        };
 
+        let mut updated_reinstall = reinstall.clone();
         for dist in site_packages.iter() {
-            if matches!(dist, uv_distribution_types::InstalledDist::Url(_)) {
-                tracing::debug!("Marking URL-based package for reinstall: {}", dist.name());
+            let is_url_like = matches!(
+                dist,
+                uv_distribution_types::InstalledDist::Url(_)
+                    | uv_distribution_types::InstalledDist::LegacyEditable(_)
+            );
+            if is_url_like && resolved_registry_names.contains(dist.name()) {
+                tracing::debug!(
+                    "Marking package for reinstall due to --no-sources registry preference: {}",
+                    dist.name()
+                );
                 updated_reinstall = match updated_reinstall {
                     uv_configuration::Reinstall::None => {
                         uv_configuration::Reinstall::Packages(vec![dist.name().clone()], Vec::new())
                     }
                     uv_configuration::Reinstall::All => updated_reinstall,
                     uv_configuration::Reinstall::Packages(mut packages, paths) => {
-                        packages.push(dist.name().clone());
+                        if !packages.contains(dist.name()) {
+                            packages.push(dist.name().clone());
+                        }
                         uv_configuration::Reinstall::Packages(packages, paths)
                     }
                 };

crates/uv/tests/it/sync.rs

@@ -12995,13 +12995,12 @@ fn sync_no_sources_editable_to_package_switch() -> Result<()> {
 
     ----- stderr -----
     Resolved 4 packages in [TIME]
-    Uninstalled 2 packages in [TIME]
-    Installed 4 packages in [TIME]
+    Uninstalled 1 package in [TIME]
+    Installed 3 packages in [TIME]
      - anyio==4.3.0 (from file://[TEMP_DIR]/local_dep)
      + anyio==4.3.0
      + idna==3.6
      + sniffio==1.3.1
-     ~ test-no-sources==0.0.1 (from file://[TEMP_DIR]/)
     ");
 
     Ok(())