fix usage string and some errors

golemcloud · Dec 5, 2024 · 0f5847b · 0f5847b
1 parent b797ade
commit 0f5847b
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 3 deletions.
diff --git a/wasm-rpc-stubgen/src/commands/app.rs b/wasm-rpc-stubgen/src/commands/app.rs
@@ -741,7 +741,7 @@ fn collect_sources(mode: &ApplicationSourceMode) -> ValidatedResult<Vec<PathBuf>
                         })
                 }
             }
-            None => ValidatedResult::from_error("No config file found!".to_string()),
+            None => ValidatedResult::from_error("No application manifest found!".to_string()),
         },
         ApplicationSourceMode::Explicit(sources) => {
             let non_unique_source_warns: Vec<_> = sources

diff --git a/wasm-rpc-stubgen/src/lib.rs b/wasm-rpc-stubgen/src/lib.rs
@@ -307,7 +307,7 @@ pub fn initialize_workspace(
 }
 
 pub async fn run_app_command<CPE: ComponentPropertiesExtensions>(
-    clap_command: &mut clap::Command,
+    mut clap_command: clap::Command,
     command: App,
 ) -> anyhow::Result<()> {
     let (mut config, subcommand) = app_command_to_config_and_subcommand::<CPE>(command);

diff --git a/wasm-rpc-stubgen/src/main.rs b/wasm-rpc-stubgen/src/main.rs
@@ -46,7 +46,20 @@ async fn main() -> ExitCode {
         #[cfg(feature = "app-command")]
         Command::App { command } => {
             run_app_command::<ComponentPropertiesExtensionsNone>(
-                clap_command.find_subcommand_mut("app").unwrap(),
+                {
+                    // TODO: it would be nice to use the same logic which is used by default for handling help,
+                    //       and that way include the current context (bin name and parent commands),
+                    //       but that seems to be using errors, error formating and exit directly;
+                    //       and quite different code path compared to calling print_help
+                    clap_command
+                        .find_subcommand_mut("app")
+                        .unwrap()
+                        .clone()
+                        .override_usage(format!(
+                            "{} [OPTIONS] [COMMAND]",
+                            "wasm-rpc-stubgen app".bold()
+                        ))
+                },
                 command,
             )
             .await