Warn on existing directory if not tty

Author: jtfmumm

Cargo.lock

@@ -27,6 +27,7 @@ uv-pypi-types = { workspace = true }
 uv-python = { workspace = true }
 uv-shell = { workspace = true }
 uv-version = { workspace = true }
+uv-warnings = { workspace = true }
 
 console = { workspace = true }
 fs-err = { workspace = true }

crates/uv-virtualenv/Cargo.toml

@@ -19,6 +19,7 @@ use uv_python::managed::{PythonMinorVersionLink, create_link_to_executable};
 use uv_python::{Interpreter, VirtualEnvironment};
 use uv_shell::escape_posix_for_single_quotes;
 use uv_version::version;
+use uv_warnings::warn_user_once;
 
 use crate::{Error, Prompt};
 
@@ -118,17 +119,27 @@ pub(crate) fn create(
                 {
                     debug!("Ignoring empty directory");
                 } else {
-                    return Err(Error::Io(io::Error::new(
-                        io::ErrorKind::AlreadyExists,
-                        format!(
-                            "The directory `{}` exists. \n\n{}{} Use `{}` to remove the directory first or `{}` to write to the directory without clearing",
-                            location.user_display(),
-                            "hint".bold().cyan(),
-                            ":".bold(),
-                            "--clear".green(),
-                            "--allow-existing".green(),
-                        ),
-                    )));
+                    if Term::stderr().is_term() {
+                        return Err(Error::Io(io::Error::new(
+                            io::ErrorKind::AlreadyExists,
+                            format!(
+                                "The directory `{}` exists. \n\n{}{} Use `{}` to remove the directory first or `{}` to write to the directory without clearing",
+                                location.user_display(),
+                                "hint".bold().cyan(),
+                                ":".bold(),
+                                "--clear".green(),
+                                "--allow-existing".green(),
+                            ),
+                        )));
+                    }
+
+                    // If this is not a tty, warn for now.
+                    warn_user_once!(
+                        "The directory `{}` exists. In the future, uv will require `{}` to remove the directory first or `{}` to write to the directory without clearing",
+                        location.user_display(),
+                        "--clear".green(),
+                        "--allow-existing".green(),
+                    );
                 }
             }
         }

crates/uv-virtualenv/src/virtualenv.rs

@@ -36,19 +36,15 @@ fn create_venv() {
         .arg(context.venv.as_os_str())
         .arg("--python")
         .arg("3.12"), @r"
-    success: false
-    exit_code: 1
+    success: true
+    exit_code: 0
     ----- stdout -----
 
     ----- stderr -----
     Using CPython 3.12.[X] interpreter at: [PYTHON-3.12]
     Creating virtual environment at: .venv
-    uv::venv::creation
-
-      × Failed to create virtualenv
-      ╰─▶ The directory `.venv` exists.
-
-          hint: Use `--clear` to remove the directory first or `--allow-existing` to write to the directory without clearing
+    warning: The directory `.venv` exists. In the future, uv will require `--clear` to remove the directory first or `--allow-existing` to write to the directory without clearing
+    Activate with: source .venv/[BIN]/activate
     "
     );
 
@@ -994,21 +990,17 @@ fn non_empty_dir_exists() -> Result<()> {
     uv_snapshot!(context.filters(), context.venv()
         .arg(context.venv.as_os_str())
         .arg("--python")
-        .arg("3.12"), @r###"
-    success: false
-    exit_code: 1
+        .arg("3.12"), @r"
+    success: true
+    exit_code: 0
     ----- stdout -----
 
     ----- stderr -----
     Using CPython 3.12.[X] interpreter at: [PYTHON-3.12]
     Creating virtual environment at: .venv
-    uv::venv::creation
-
-      × Failed to create virtualenv
-      ╰─▶ The directory `.venv` exists.
-
-          hint: Use `--clear` to remove the directory first or `--allow-existing` to write to the directory without clearing
-    "###
+    warning: The directory `.venv` exists. In the future, uv will require `--clear` to remove the directory first or `--allow-existing` to write to the directory without clearing
+    Activate with: source .venv/[BIN]/activate
+    "
     );
 
     Ok(())
@@ -1026,21 +1018,17 @@ fn non_empty_dir_exists_allow_existing() -> Result<()> {
     uv_snapshot!(context.filters(), context.venv()
         .arg(context.venv.as_os_str())
         .arg("--python")
-        .arg("3.12"), @r###"
-    success: false
-    exit_code: 1
+        .arg("3.12"), @r"
+    success: true
+    exit_code: 0
     ----- stdout -----
 
     ----- stderr -----
     Using CPython 3.12.[X] interpreter at: [PYTHON-3.12]
     Creating virtual environment at: .venv
-    uv::venv::creation
-
-      × Failed to create virtualenv
-      ╰─▶ The directory `.venv` exists.
-
-          hint: Use `--clear` to remove the directory first or `--allow-existing` to write to the directory without clearing
-    "###
+    warning: The directory `.venv` exists. In the future, uv will require `--clear` to remove the directory first or `--allow-existing` to write to the directory without clearing
+    Activate with: source .venv/[BIN]/activate
+    "
     );
 
     uv_snapshot!(context.filters(), context.venv()