Add exe check.

crates/cli/src/app.rs

@@ -23,12 +23,11 @@ use crate::commands::task::TaskArgs;
 use crate::enums::{CacheMode, LogLevel};
 use clap::builder::styling::{Color, Style, Styles};
 use clap::{Parser, Subcommand};
+use moon_common::consts::BIN_NAME;
 use starbase::State;
 use starbase_styles::color::Color as ColorType;
 use std::path::PathBuf;
 
-pub const BIN_NAME: &str = if cfg!(windows) { "moon.exe" } else { "moon" };
-
 #[derive(Clone, Debug, Subcommand)]
 pub enum DockerCommands {
     #[command(

crates/cli/src/commands/upgrade.rs

@@ -1,10 +1,10 @@
-use crate::app::BIN_NAME;
 use crate::helpers::create_progress_bar;
 use bytes::Buf;
 use itertools::Itertools;
 use miette::{miette, IntoDiagnostic};
 use moon_api::Launchpad;
 use moon_cache::CacheEngine;
+use moon_common::consts::BIN_NAME;
 use moon_utils::{get_workspace_root, semver::Version};
 use starbase::system;
 use starbase_utils::{dirs, fs};

crates/cli/src/lib.rs

@@ -33,15 +33,14 @@ use app::App as CLI;
 use clap::Parser;
 use commands::migrate::FromTurborepoArgs;
 use enums::{CacheMode, LogLevel};
+use moon_common::consts::BIN_NAME;
 use starbase::{tracing::TracingOptions, App, AppResult};
 use starbase_styles::color;
 use starbase_utils::string_vec;
 use std::env;
 use std::ffi::OsString;
 use tracing::debug;
 
-pub use app::BIN_NAME;
-
 fn setup_logging(level: &LogLevel) {
     env::set_var("MOON_APP_LOG", level.to_string());
 

crates/cli/src/main.rs

@@ -1,7 +1,7 @@
 use miette::IntoDiagnostic;
 use mimalloc::MiMalloc;
-use moon_cli::{run_cli, BIN_NAME};
-use moon_common::consts::CONFIG_DIRNAME;
+use moon_cli::run_cli;
+use moon_common::consts::{BIN_NAME, CONFIG_DIRNAME};
 use moon_node_lang::{
     node::{extract_canonical_node_module_bin, BinFile},
     NODE,

crates/rust/platform/src/rust_platform.rs

@@ -2,7 +2,7 @@ use crate::{
     find_cargo_lock, get_cargo_home, target_hash::RustTargetHash, toolchain_hash::RustToolchainHash,
 };
 use moon_action_context::ActionContext;
-use moon_common::{is_ci, Id};
+use moon_common::{is_ci, path::exe_name, Id};
 use moon_config::{
     BinEntry, HasherConfig, PlatformType, ProjectConfig, ProjectsAliasesMap, ProjectsSourcesMap,
     RustConfig, UnresolvedVersionSpec,
@@ -282,9 +282,7 @@ impl Platform for RustPlatform {
             let globals_dir = self.get_globals_dir(Some(tool));
 
             // Install cargo-binstall if it does not exist
-            if !globals_dir.join("cargo-binstall").exists()
-                && !globals_dir.join("cargo-binstall.exe").exists()
-            {
+            if !globals_dir.join(exe_name("cargo-binstall")).exists() {
                 debug!(
                     target: LOG_TARGET,
                     "{} does not exist, installing",
@@ -525,10 +523,10 @@ impl Platform for RustPlatform {
             // Binary may be installed to ~/.cargo/bin
             _ => {
                 let globals_dir = self.get_globals_dir(self.toolchain.get().ok());
-                let global_bin_path = globals_dir.join(&task.command);
+                let global_bin_path = globals_dir.join(exe_name(&task.command));
 
                 let cargo_bin = task.command.strip_prefix("cargo-").unwrap_or(&task.command);
-                let cargo_bin_path = globals_dir.join(format!("cargo-{}", cargo_bin));
+                let cargo_bin_path = globals_dir.join(exe_name(format!("cargo-{}", cargo_bin)));
 
                 // Must run through cargo
                 if cargo_bin_path.exists() {

nextgen/common/src/path.rs

@@ -74,3 +74,16 @@ pub fn to_string<T: AsRef<Path>>(path: T) -> miette::Result<String> {
 pub fn to_virtual_string<T: AsRef<Path>>(path: T) -> miette::Result<String> {
     Ok(standardize_separators(to_string(path)?))
 }
+
+#[inline]
+pub fn exe_name<T: AsRef<str>>(name: T) -> String {
+    #[cfg(windows)]
+    {
+        format!("{}.exe", name.as_ref())
+    }
+
+    #[cfg(not(windows))]
+    {
+        name.as_ref().into()
+    }
+}