Review feedback

Author: zanieb

crates/uv/src/commands/python/install.rs

@@ -138,10 +138,14 @@ pub(crate) async fn install(
     let start = std::time::Instant::now();
 
     if default && !preview.is_enabled() {
-        writeln!(printer.stderr(), "The `--default` flag is only available in preview mode; add the `--preview` flag to use `--default.")?;
+        writeln!(printer.stderr(), "The `--default` flag is only available in preview mode; add the `--preview` flag to use `--default")?;
         return Ok(ExitStatus::Failure);
     }
 
+    if default && targets.len() > 1 {
+        anyhow::bail!("The `--default` flag cannot be used with multiple targets");
+    }
+
     // Resolve the requests
     let mut is_default_install = false;
     let requests: Vec<_> = if targets.is_empty() {

crates/uv/tests/it/python_install.rs

@@ -507,7 +507,7 @@ fn python_install_default() {
     ----- stdout -----
 
     ----- stderr -----
-    The `--default` flag is only available in preview mode; add the `--preview` flag to use `--default.
+    The `--default` flag is only available in preview mode; add the `--preview` flag to use `--default
     "###);
 
     // Install a specific version
@@ -594,14 +594,23 @@ fn python_install_default() {
 
     // Install multiple versions, with the `--default` flag
     uv_snapshot!(context.filters(), context.python_install().arg("--preview").arg("3.12").arg("3.13").arg("--default"), @r###"
+    success: false
+    exit_code: 2
+    ----- stdout -----
+
+    ----- stderr -----
+    error: The `--default` flag cannot be used with multiple targets
+    "###);
+
+    // Install 3.12 as a new default
+    uv_snapshot!(context.filters(), context.python_install().arg("--preview").arg("3.12").arg("--default"), @r###"
     success: true
     exit_code: 0
     ----- stdout -----
 
     ----- stderr -----
-    Installed 2 versions in [TIME]
+    Installed Python 3.12.7 in [TIME]
      + cpython-3.12.7-[PLATFORM] (python, python3, python3.12)
-     + cpython-3.13.0-[PLATFORM] (python3.13)
     "###);
 
     let bin_python_minor_12 = context
@@ -610,7 +619,6 @@ fn python_install_default() {
         .child(format!("python3.12{}", std::env::consts::EXE_SUFFIX));
 
     // All the executables should exist
-    bin_python_minor_13.assert(predicate::path::exists());
     bin_python_minor_12.assert(predicate::path::exists());
     bin_python_major.assert(predicate::path::exists());
     bin_python_default.assert(predicate::path::exists());
@@ -625,14 +633,6 @@ fn python_install_default() {
             );
         });
 
-        insta::with_settings!({
-            filters => context.filters(),
-        }, {
-            insta::assert_snapshot!(
-                read_link_path(&bin_python_minor_13), @"[TEMP_DIR]/managed/cpython-3.13.0-[PLATFORM]/bin/python3.13"
-            );
-        });
-
         insta::with_settings!({
             filters => context.filters(),
         }, {
@@ -657,14 +657,6 @@ fn python_install_default() {
             );
         });
 
-        insta::with_settings!({
-            filters => context.filters(),
-        }, {
-            insta::assert_snapshot!(
-                read_link_path(&bin_python_minor_13), @"[TEMP_DIR]/managed/cpython-3.13.0-[PLATFORM]/python"
-            );
-        });
-
         insta::with_settings!({
             filters => context.filters(),
         }, {
@@ -690,7 +682,7 @@ fn python_install_default() {
 
     ----- stderr -----
     Installed Python 3.13.0 in [TIME]
-     + cpython-3.13.0-[PLATFORM] (python, python3)
+     + cpython-3.13.0-[PLATFORM] (python, python3, python3.13)
     "###);
 
     // All the executables should exist