From 4c7f9bd4f078411cc30e315a19cda2f4caf30443 Mon Sep 17 00:00:00 2001
From: Kemal Akkoyun <kakkoyun@gmail.com>
Date: Thu, 26 Sep 2024 15:35:02 +0200
Subject: [PATCH] Address review comments: Optional RunCommand

Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>
---
 crates/uv/src/commands/project/run.rs |  8 ++++----
 crates/uv/src/lib.rs                  | 14 ++++++--------
 crates/uv/tests/run.rs                |  4 ++--
 3 files changed, 12 insertions(+), 14 deletions(-)

diff --git a/crates/uv/src/commands/project/run.rs b/crates/uv/src/commands/project/run.rs
index 5747ac8fc06a5..956a085529cea 100644
--- a/crates/uv/src/commands/project/run.rs
+++ b/crates/uv/src/commands/project/run.rs
@@ -53,7 +53,7 @@ use crate::settings::ResolverInstallerSettings;
 pub(crate) async fn run(
     project_dir: &Path,
     script: Option<Pep723Script>,
-    command: RunCommand,
+    command: Option<RunCommand>,
     requirements: Vec<RequirementsSource>,
     show_resolution: bool,
     locked: bool,
@@ -721,7 +721,7 @@ pub(crate) async fn run(
 
     // Check if any run command is given.
     // If not, print the available scripts for the current interpreter.
-    if let RunCommand::Empty = command {
+    let Some(command) = command else {
         writeln!(
             printer.stdout(),
             "Provide a command or script to invoke with `uv run <command>` or `uv run <script>.py`.\n"
@@ -753,7 +753,7 @@ pub(crate) async fn run(
             .filter(|path| is_executable(path))
             .map(|path| {
                 if cfg!(windows) {
-                    // remove the extensions
+                    // Remove the extensions.
                     path.with_extension("")
                 } else {
                     path
@@ -766,7 +766,7 @@ pub(crate) async fn run(
                     .to_string()
             })
             .filter(|command| {
-                !command.starts_with("activate") || !command.starts_with("deactivate")
+                !command.starts_with("activate") && !command.starts_with("deactivate")
             })
             .collect_vec();
         commands.sort();
diff --git a/crates/uv/src/lib.rs b/crates/uv/src/lib.rs
index 0645e8d1a6e80..1b504273f4a2a 100644
--- a/crates/uv/src/lib.rs
+++ b/crates/uv/src/lib.rs
@@ -131,11 +131,12 @@ async fn run(cli: Cli) -> Result<ExitStatus> {
 
     // Parse the external command, if necessary.
     let run_command = if let Commands::Project(command) = &*cli.command {
-        if let ProjectCommand::Run(uv_cli::RunArgs { command, .. }) = &**command {
-            match command {
-                Some(command) => Some(RunCommand::try_from(command)?),
-                None => Some(RunCommand::Empty),
-            }
+        if let ProjectCommand::Run(uv_cli::RunArgs {
+            command: Some(command),
+            ..
+        }) = &**command
+        {
+            Some(RunCommand::try_from(command)?)
         } else {
             None
         }
@@ -1230,9 +1231,6 @@ async fn run_project(
                 )
                 .collect::<Vec<_>>();
 
-            // Given `ProjectCommand::Run`, we always expect a `RunCommand` to be present.
-            let command = command.expect("run command is required");
-
             Box::pin(commands::run(
                 project_dir,
                 script,
diff --git a/crates/uv/tests/run.rs b/crates/uv/tests/run.rs
index 04cc9c159a4d5..50a486bd9da59 100644
--- a/crates/uv/tests/run.rs
+++ b/crates/uv/tests/run.rs
@@ -241,8 +241,8 @@ fn run_no_args() -> Result<()> {
 
     #[cfg(windows)]
     uv_snapshot!(context.filters(), context.run(), @r###"
-    success: true
-    exit_code: 0
+    success: false
+    exit_code: 1
     ----- stdout -----
     Provide a command or script to invoke with `uv run <command>` or `uv run <script>.py`.