Fix rye list overwrites .venv (#942)

CHANGELOG.md

@@ -11,6 +11,9 @@ _Unreleased_
 
 - Always create `.gitignore` file in `rye init`.  #919
 
+- `rye list` always prints the currently installed packages even this project is not managed by Rye.  #940
+
+- Fix error on using -v or -q with `rye fmt` or `rye lint`. #959
 - Fix error on using -v or -q with `rye fmt` or `rye lint`.  #959
 
 - Fix rye fetch detection of registered toolchain.  #931

rye/src/cli/list.rs

@@ -9,7 +9,7 @@ use crate::config::Config;
 use crate::consts::VENV_BIN;
 use crate::pyproject::PyProject;
 use crate::utils::{get_venv_python_bin, CommandOutput};
-use crate::uv::UvBuilder;
+use crate::uv::{UvBuilder, UvWithVenv};
 
 /// Prints the currently installed packages.
 #[derive(Parser, Debug)]
@@ -28,16 +28,20 @@ pub fn execute(cmd: Args) -> Result<(), Error> {
     let self_venv = ensure_self_venv(CommandOutput::Normal)?;
 
     if Config::current().use_uv() {
-        UvBuilder::new()
+        let uv = UvBuilder::new()
             .with_output(CommandOutput::Normal)
-            .ensure_exists()?
-            .venv(
+            .ensure_exists()?;
+        if !project.rye_managed() {
+            UvWithVenv::new(uv, &project.venv_path(), &project.venv_python_version()?).freeze()?;
+        } else {
+            uv.venv(
                 &project.venv_path(),
                 &python,
                 &project.venv_python_version()?,
                 None,
             )?
             .freeze()?;
+        }
     } else {
         let status = Command::new(self_venv.join(VENV_BIN).join("pip"))
             .arg("--python")

rye/src/uv.rs

@@ -359,7 +359,7 @@ pub struct UvWithVenv {
 }
 
 impl UvWithVenv {
-    fn new(uv: Uv, venv_dir: &Path, version: &PythonVersion) -> Self {
+    pub fn new(uv: Uv, venv_dir: &Path, version: &PythonVersion) -> Self {
         UvWithVenv {
             uv,
             py_version: version.clone(),

rye/tests/test-list.rs

@@ -0,0 +1,58 @@
+use crate::common::{rye_cmd_snapshot, Space};
+use toml_edit::value;
+
+mod common;
+
+#[test]
+fn test_basic_list() {
+    let space = Space::new();
+    space.init("my-project");
+
+    space
+        .rye_cmd()
+        .arg("add")
+        .arg("jinja2")
+        .status()
+        .expect("ok");
+
+    rye_cmd_snapshot!(
+        space.rye_cmd().arg("list"), @r###"
+    success: true
+    exit_code: 0
+    ----- stdout -----
+    jinja2==3.1.2
+    markupsafe==2.1.3
+    -e file:[TEMP_PATH]/project
+
+    ----- stderr -----
+    "###);
+}
+
+#[test]
+fn test_list_not_rye_managed() {
+    let space = Space::new();
+    space.init("my-project");
+
+    space.edit_toml("pyproject.toml", |doc| {
+        doc["tool"]["rye"]["managed"] = value(false);
+    });
+
+    space
+        .rye_cmd()
+        .arg("add")
+        .arg("jinja2")
+        .status()
+        .expect("Add package failed");
+
+    rye_cmd_snapshot!(
+        space.rye_cmd().arg("list"), @r###"
+    success: true
+    exit_code: 0
+    ----- stdout -----
+    jinja2==3.1.2
+    markupsafe==2.1.3
+    -e file:[TEMP_PATH]/project
+
+    ----- stderr -----
+    "###);
+}