From 86892f61a1bce1f855463477a278e24a41f8e9a7 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 |  4 ++--
 crates/uv/src/lib.rs                  | 14 ++++++--------
 2 files changed, 8 insertions(+), 10 deletions(-)

diff --git a/crates/uv/src/commands/project/run.rs b/crates/uv/src/commands/project/run.rs
index 5747ac8fc06a..c68013a3c93f 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"
diff --git a/crates/uv/src/lib.rs b/crates/uv/src/lib.rs
index 0645e8d1a6e8..1b504273f4a2 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,