cargo_cross/

cli.rs

//! Command-line argument parsing for cargo-cross using clap

use crate::config::{
    self, supported_freebsd_versions_str, supported_glibc_versions_str,
    supported_iphone_sdk_versions_str, supported_macos_sdk_versions_str,
    DEFAULT_CROSS_MAKE_VERSION, DEFAULT_FREEBSD_VERSION, DEFAULT_GLIBC_VERSION,
    DEFAULT_IPHONE_SDK_VERSION, DEFAULT_MACOS_SDK_VERSION, DEFAULT_NDK_VERSION,
    DEFAULT_QEMU_VERSION, SUPPORTED_FREEBSD_VERSIONS, SUPPORTED_GLIBC_VERSIONS,
    SUPPORTED_IPHONE_SDK_VERSIONS, SUPPORTED_MACOS_SDK_VERSIONS,
};
use crate::error::{CrossError, Result};
use clap::builder::styling::{AnsiColor, Effects, Styles};
use clap::{Args as ClapArgs, CommandFactory, FromArgMatches, Parser, Subcommand, ValueHint};
use std::path::PathBuf;
use std::sync::LazyLock;

/// Binary name from Cargo.toml (e.g., "cargo-cross")
const BIN_NAME: &str = env!("CARGO_PKG_NAME");

/// Define subcommand-related constants from a single literal
macro_rules! define_subcommand {
    ($name:literal) => {
        const SUBCOMMAND: &str = $name;
        const CARGO_DISPLAY_NAME: &str = concat!("cargo ", $name);
    };
}
define_subcommand!("cross");

/// Program name - either "cargo cross" or "cargo-cross" based on invocation style
static PROGRAM_NAME: LazyLock<&'static str> = LazyLock::new(|| {
    let args: Vec<String> = std::env::args().collect();
    let is_cargo_subcommand = std::env::var("CARGO").is_ok()
        && std::env::var("CARGO_HOME").is_ok()
        && args.get(1).map(String::as_str) == Some(SUBCOMMAND);

    if is_cargo_subcommand {
        CARGO_DISPLAY_NAME
    } else {
        BIN_NAME
    }
});

/// Get the program name for display
#[must_use]
pub fn program_name() -> &'static str {
    *PROGRAM_NAME
}

/// Custom styles for CLI help output
fn cli_styles() -> Styles {
    Styles::styled()
        .header(AnsiColor::BrightCyan.on_default() | Effects::BOLD)
        .usage(AnsiColor::BrightCyan.on_default() | Effects::BOLD)
        .literal(AnsiColor::BrightGreen.on_default())
        .placeholder(AnsiColor::BrightMagenta.on_default())
        .valid(AnsiColor::BrightGreen.on_default())
        .invalid(AnsiColor::BrightRed.on_default())
        .error(AnsiColor::BrightRed.on_default() | Effects::BOLD)
}

/// Cross-compilation tool for Rust projects
#[derive(Parser, Debug)]
#[command(name = "cargo-cross", version)]
#[command(about = "Cross-compilation tool for Rust projects, no Docker required")]
#[command(long_about = "\
Cross-compilation tool for Rust projects.

This tool provides cross-compilation support for Rust projects across multiple
platforms including Linux (musl/gnu), Windows, macOS, FreeBSD, iOS, and Android.
It automatically downloads and configures the appropriate cross-compiler toolchains.")]
#[command(propagate_version = true)]
#[command(arg_required_else_help = true)]
#[command(styles = cli_styles())]
#[command(override_usage = "cargo-cross [+toolchain] <COMMAND> [OPTIONS]")]
#[command(after_help = "\
Use 'cargo-cross <COMMAND> --help' for more information about a command.

TOOLCHAIN:
    If the first argument begins with +, it will be interpreted as a Rust toolchain
    name (such as +nightly, +stable, or +1.75.0). This follows the same convention
    as rustup and cargo.

EXAMPLES:
    cargo-cross build -t x86_64-unknown-linux-musl
    cargo-cross +nightly build -t aarch64-unknown-linux-gnu --profile release
    cargo-cross build -t '*-linux-musl' --crt-static true
    cargo-cross test -t x86_64-unknown-linux-musl -- --nocapture")]
pub struct Cli {
    #[command(subcommand)]
    pub command: CliCommand,
}

#[derive(Subcommand, Debug)]
pub enum CliCommand {
    /// Compile the current package
    #[command(visible_alias = "b")]
    #[command(long_about = "\
Compile the current package and all of its dependencies.

When no target selection options are given, this tool will build all binary
and library targets of the selected packages.")]
    Build(BuildArgs),

    /// Analyze the current package and report errors, but don't build object files
    #[command(visible_alias = "c")]
    #[command(long_about = "\
Check the current package and all of its dependencies for errors.

This will essentially compile packages without performing the final step of
code generation, which is faster than running build.")]
    Check(BuildArgs),

    /// Run a binary or example of the current package
    #[command(visible_alias = "r")]
    #[command(long_about = "\
Run a binary or example of the local package.

For cross-compilation targets, QEMU user-mode emulation is used to run the binary.")]
    Run(BuildArgs),

    /// Run the tests
    #[command(visible_alias = "t")]
    #[command(long_about = "\
Execute all unit and integration tests and build examples of a local package.

For cross-compilation targets, QEMU user-mode emulation is used to run tests.")]
    Test(BuildArgs),

    /// Run the benchmarks
    #[command(long_about = "\
Execute all benchmarks of a local package.

For cross-compilation targets, QEMU user-mode emulation is used to run benchmarks.")]
    Bench(BuildArgs),

    /// Display all supported cross-compilation targets
    #[command(long_about = "\
Display all supported cross-compilation targets.

You can also use glob patterns with --target to match multiple targets,
for example: --target '*-linux-musl' or --target 'aarch64-*'")]
    Targets(TargetsArgs),

    /// Print version information
    Version,
}

/// Output format for targets command
#[derive(Debug, Clone, Copy, Default, PartialEq, Eq, clap::ValueEnum)]
pub enum OutputFormat {
    /// Human-readable colored text (default)
    #[default]
    Text,
    /// JSON array format
    Json,
    /// Plain text, one target per line
    Plain,
}

#[derive(ClapArgs, Debug, Clone, Default)]
pub struct TargetsArgs {
    /// Output format
    #[arg(
        short = 'f',
        long = "format",
        value_enum,
        default_value = "text",
        help = "Output format (text, json, plain)"
    )]
    pub format: OutputFormat,
}

#[derive(ClapArgs, Debug, Clone, Default)]
#[command(next_help_heading = "Target Selection")]
pub struct BuildArgs {
    // ===== Target Selection =====
    /// Build for the target triple(s)
    #[arg(
        short = 't',
        long = "target",
        visible_alias = "targets",
        value_delimiter = ',',
        env = "TARGETS",
        value_name = "TRIPLE",
        help = "Build for the target triple(s), comma-separated",
        long_help = "\
Build for the specified target architecture. This flag may be specified multiple
times or with comma-separated values. Supports glob patterns like '*-linux-musl'.
The general format of the triple is <arch><sub>-<vendor>-<sys>-<abi>.
Run the 'targets' subcommand to see all supported targets.

Examples: -t x86_64-unknown-linux-musl, -t '*-linux-musl'"
    )]
    pub targets: Vec<String>,

    // ===== Feature Selection =====
    /// Space or comma separated list of features to activate
    #[arg(
        short = 'F',
        long,
        env = "FEATURES",
        value_name = "FEATURES",
        conflicts_with = "all_features",
        help_heading = "Feature Selection",
        long_help = "\
Space or comma separated list of features to activate. Features of workspace members
may be enabled with package-name/feature-name syntax. May be specified multiple times."
    )]
    pub features: Option<String>,

    /// Do not activate the `default` feature of the selected packages
    #[arg(long, env = "NO_DEFAULT_FEATURES", help_heading = "Feature Selection")]
    pub no_default_features: bool,

    /// Activate all available features of all selected packages
    #[arg(
        long,
        env = "ALL_FEATURES",
        conflicts_with = "features",
        help_heading = "Feature Selection"
    )]
    pub all_features: bool,

    // ===== Profile =====
    /// Build artifacts in release mode, with optimizations
    #[arg(
        short = 'r',
        long = "release",
        conflicts_with = "profile",
        help_heading = "Profile",
        long_help = "\
Build artifacts in release mode, with optimizations. Equivalent to --profile=release."
    )]
    pub release: bool,

    /// Build artifacts with the specified profile
    #[arg(
        long,
        default_value = "release",
        env = "PROFILE",
        value_name = "PROFILE-NAME",
        conflicts_with = "release",
        help_heading = "Profile",
        long_help = "\
Build artifacts with the specified profile. Built-in: dev, release, test, bench.
Custom profiles can be defined in Cargo.toml. Default is 'release' (differs from cargo's 'dev')."
    )]
    pub profile: String,

    // ===== Package Selection =====
    /// Package to build (see `cargo help pkgid`)
    #[arg(
        short = 'p',
        long,
        env = "PACKAGE",
        value_name = "SPEC",
        help_heading = "Package Selection",
        long_help = "\
Build only the specified packages. This flag may be specified multiple times
and supports common Unix glob patterns like *, ?, and []."
    )]
    pub package: Option<String>,

    /// Build all members in the workspace
    #[arg(
        long,
        visible_alias = "all",
        env = "BUILD_WORKSPACE",
        help_heading = "Package Selection"
    )]
    pub workspace: bool,

    /// Exclude packages from the build (must be used with --workspace)
    #[arg(
        long,
        env = "EXCLUDE",
        value_name = "SPEC",
        requires = "workspace",
        help_heading = "Package Selection",
        long_help = "\
Exclude the specified packages. Must be used in conjunction with the --workspace flag.
This flag may be specified multiple times and supports common Unix glob patterns."
    )]
    pub exclude: Option<String>,

    /// Build only the specified binary
    #[arg(
        long = "bin",
        env = "BIN_TARGET",
        value_name = "NAME",
        help_heading = "Package Selection",
        long_help = "\
Build the specified binary. This flag may be specified multiple times
and supports common Unix glob patterns."
    )]
    pub bin_target: Option<String>,

    /// Build all binary targets
    #[arg(long = "bins", env = "BUILD_BINS", help_heading = "Package Selection")]
    pub build_bins: bool,

    /// Build only this package's library
    #[arg(long = "lib", env = "BUILD_LIB", help_heading = "Package Selection")]
    pub build_lib: bool,

    /// Build only the specified example
    #[arg(
        long = "example",
        env = "EXAMPLE_TARGET",
        value_name = "NAME",
        help_heading = "Package Selection",
        long_help = "\
Build the specified example. This flag may be specified multiple times
and supports common Unix glob patterns."
    )]
    pub example_target: Option<String>,

    /// Build all example targets
    #[arg(
        long = "examples",
        env = "BUILD_EXAMPLES",
        help_heading = "Package Selection"
    )]
    pub build_examples: bool,

    /// Build only the specified test target
    #[arg(
        long = "test",
        env = "TEST_TARGET",
        value_name = "NAME",
        help_heading = "Package Selection",
        long_help = "\
Build the specified integration test. This flag may be specified multiple times
and supports common Unix glob patterns."
    )]
    pub test_target: Option<String>,

    /// Build all test targets (includes unit tests from lib/bins)
    #[arg(
        long = "tests",
        env = "BUILD_TESTS",
        help_heading = "Package Selection",
        long_help = "\
Build all targets that have the test = true manifest flag set. By default this
includes the library and binaries built as unittests, and integration tests."
    )]
    pub build_tests: bool,

    /// Build only the specified bench target
    #[arg(
        long = "bench",
        env = "BENCH_TARGET",
        value_name = "NAME",
        help_heading = "Package Selection",
        long_help = "\
Build the specified benchmark. This flag may be specified multiple times
and supports common Unix glob patterns."
    )]
    pub bench_target: Option<String>,

    /// Build all bench targets
    #[arg(
        long = "benches",
        env = "BUILD_BENCHES",
        help_heading = "Package Selection",
        long_help = "\
Build all targets that have the bench = true manifest flag set. By default this
includes the library and binaries built as benchmarks, and bench targets."
    )]
    pub build_benches: bool,

    /// Build all targets (equivalent to --lib --bins --tests --benches --examples)
    #[arg(
        long = "all-targets",
        env = "BUILD_ALL_TARGETS",
        help_heading = "Package Selection"
    )]
    pub build_all_targets: bool,

    /// Path to Cargo.toml
    #[arg(long, env = "MANIFEST_PATH", value_name = "PATH",
          value_hint = ValueHint::FilePath, help_heading = "Package Selection",
          long_help = "\
Path to Cargo.toml. By default, Cargo searches for the Cargo.toml file
in the current directory or any parent directory.")]
    pub manifest_path: Option<PathBuf>,

    // ===== Version Options =====
    /// Glibc version for Linux GNU targets
    #[arg(long, default_value = DEFAULT_GLIBC_VERSION, env = "GLIBC_VERSION",
          value_name = "VERSION", hide_default_value = true, help_heading = "Toolchain Versions")]
    pub glibc_version: String,

    /// iPhone SDK version for iOS targets
    #[arg(long, default_value = DEFAULT_IPHONE_SDK_VERSION, env = "IPHONE_SDK_VERSION",
          value_name = "VERSION", hide_default_value = true, help_heading = "Toolchain Versions")]
    pub iphone_sdk_version: String,

    /// Override iPhoneOS SDK path (skips version lookup)
    #[arg(long, env = "IPHONE_SDK_PATH", value_name = "PATH",
          value_hint = ValueHint::DirPath, help_heading = "Toolchain Versions",
          long_help = "\
Override iPhoneOS SDK path for device targets. Skips version lookup.")]
    pub iphone_sdk_path: Option<PathBuf>,

    /// Override iPhoneSimulator SDK path
    #[arg(long, env = "IPHONE_SIMULATOR_SDK_PATH", value_name = "PATH",
          value_hint = ValueHint::DirPath, help_heading = "Toolchain Versions",
          long_help = "\
Override iPhoneSimulator SDK path for simulator targets. Skips version lookup.")]
    pub iphone_simulator_sdk_path: Option<PathBuf>,

    /// macOS SDK version for Darwin targets
    #[arg(long, default_value = DEFAULT_MACOS_SDK_VERSION, env = "MACOS_SDK_VERSION",
          value_name = "VERSION", hide_default_value = true, help_heading = "Toolchain Versions")]
    pub macos_sdk_version: String,

    /// Override macOS SDK path (skips version lookup)
    #[arg(long, env = "MACOS_SDK_PATH", value_name = "PATH",
          value_hint = ValueHint::DirPath, help_heading = "Toolchain Versions",
          long_help = "\
Override macOS SDK path directly. Skips version lookup.")]
    pub macos_sdk_path: Option<PathBuf>,

    /// FreeBSD version for FreeBSD targets
    #[arg(long, default_value = DEFAULT_FREEBSD_VERSION, env = "FREEBSD_VERSION",
          value_name = "VERSION", hide_default_value = true, help_heading = "Toolchain Versions")]
    pub freebsd_version: String,

    /// Android NDK version
    #[arg(long, default_value = DEFAULT_NDK_VERSION, env = "NDK_VERSION",
          value_name = "VERSION", hide_default_value = true, help_heading = "Toolchain Versions",
          long_help = "\
Specify Android NDK version for Android targets. Auto-downloaded from Google's official repository.")]
    pub ndk_version: String,

    /// QEMU version for user-mode emulation
    #[arg(long, default_value = DEFAULT_QEMU_VERSION, env = "QEMU_VERSION",
          value_name = "VERSION", hide_default_value = true, help_heading = "Toolchain Versions",
          long_help = "\
Specify QEMU version for user-mode emulation. Used to run cross-compiled binaries during test/run/bench.")]
    pub qemu_version: String,

    /// Cross-compiler make version
    #[arg(long, default_value = DEFAULT_CROSS_MAKE_VERSION, env = "CROSS_MAKE_VERSION",
          value_name = "VERSION", hide_default_value = true, help_heading = "Toolchain Versions",
          long_help = "\
Specify cross-compiler make version. This determines which version of cross-compilation \
toolchains will be downloaded from the upstream repository.")]
    pub cross_make_version: String,

    // ===== Directories =====
    /// Directory for cross-compiler toolchains
    #[arg(long, env = "CROSS_COMPILER_DIR", value_name = "DIR",
          value_hint = ValueHint::DirPath, help_heading = "Directories",
          long_help = "\
Directory where cross-compiler toolchains will be downloaded and stored. Defaults to temp dir.
Set this to reuse downloaded toolchains across builds.")]
    pub cross_compiler_dir: Option<PathBuf>,

    /// Directory for all generated artifacts
    #[arg(long, visible_alias = "target-dir", env = "CARGO_TARGET_DIR", value_name = "DIR",
          value_hint = ValueHint::DirPath, help_heading = "Directories",
          long_help = "\
Directory for all generated artifacts and intermediate files. Defaults to 'target'.")]
    pub cargo_target_dir: Option<PathBuf>,

    /// Copy final artifacts to this directory (unstable)
    #[arg(long, env = "ARTIFACT_DIR", value_name = "DIR",
          value_hint = ValueHint::DirPath, help_heading = "Directories",
          long_help = "\
Copy final artifacts to this directory. Unstable, requires nightly toolchain.")]
    pub artifact_dir: Option<PathBuf>,

    // ===== Compiler Options =====
    /// Override C compiler path
    #[arg(long, env = "CC", value_name = "PATH",
          value_hint = ValueHint::ExecutablePath, help_heading = "Compiler Options",
          long_help = "\
Override the C compiler path. By default, the appropriate cross-compiler is auto-configured.")]
    pub cc: Option<PathBuf>,

    /// Override C++ compiler path
    #[arg(long, env = "CXX", value_name = "PATH",
          value_hint = ValueHint::ExecutablePath, help_heading = "Compiler Options",
          long_help = "\
Override the C++ compiler path. By default, the appropriate cross-compiler is auto-configured.")]
    pub cxx: Option<PathBuf>,

    /// Override archiver (ar) path
    #[arg(long, env = "AR", value_name = "PATH",
          value_hint = ValueHint::ExecutablePath, help_heading = "Compiler Options",
          long_help = "\
Override the archiver (ar) path. By default, the appropriate archiver is auto-configured.")]
    pub ar: Option<PathBuf>,

    /// Override linker path
    #[arg(long, env = "LINKER", value_name = "PATH",
          value_hint = ValueHint::ExecutablePath, help_heading = "Compiler Options",
          long_help = "\
Override the linker path. By default, the cross-compiler is used as linker.
This option takes precedence over auto-configured linker.")]
    pub linker: Option<PathBuf>,

    /// Additional flags for C compilation
    #[arg(
        long,
        env = "CFLAGS",
        value_name = "FLAGS",
        allow_hyphen_values = true,
        help_heading = "Compiler Options",
        long_help = "\
Additional flags to pass to the C compiler. Appended to default CFLAGS.
Example: --cflags '-O2 -Wall -march=native'"
    )]
    pub cflags: Option<String>,

    /// Additional flags for C++ compilation
    #[arg(
        long,
        env = "CXXFLAGS",
        value_name = "FLAGS",
        allow_hyphen_values = true,
        help_heading = "Compiler Options",
        long_help = "\
Additional flags to pass to the C++ compiler. Appended to default CXXFLAGS.
Example: --cxxflags '-O2 -Wall -std=c++17'"
    )]
    pub cxxflags: Option<String>,

    /// Additional flags for linking
    #[arg(
        long,
        env = "LDFLAGS",
        value_name = "FLAGS",
        allow_hyphen_values = true,
        help_heading = "Compiler Options",
        long_help = "\
Additional flags to pass to the linker. Appended to default LDFLAGS.
Example: --ldflags '-L/usr/local/lib -static'"
    )]
    pub ldflags: Option<String>,

    /// C++ standard library to use
    #[arg(
        long,
        env = "CXXSTDLIB",
        value_name = "LIB",
        help_heading = "Compiler Options",
        long_help = "\
Specify the C++ standard library to use (libc++, libstdc++, etc)."
    )]
    pub cxxstdlib: Option<String>,

    /// `CMake` generator to use (like cmake -G)
    #[arg(
        long,
        short = 'G',
        env = "CMAKE_GENERATOR",
        value_name = "GENERATOR",
        help_heading = "Compiler Options",
        long_help = "\
Specify the CMake generator to use. On Windows, this overrides the auto-detection.
Common generators: Ninja, 'MinGW Makefiles', 'Unix Makefiles', 'NMake Makefiles'.
If not specified, auto-detects: Ninja > MinGW Makefiles > Unix Makefiles."
    )]
    pub cmake_generator: Option<String>,

    /// Additional RUSTFLAGS (can be repeated)
    #[arg(long = "rustflag", visible_alias = "rustflags", value_name = "FLAG",
          env = "ADDITIONAL_RUSTFLAGS", allow_hyphen_values = true,
          action = clap::ArgAction::Append, help_heading = "Compiler Options",
          long_help = "\
Additional flags to pass to rustc via RUSTFLAGS. Can be specified multiple times.
Example: --rustflag '-C target-cpu=native' --rustflag '-C lto=thin'")]
    pub rustflags: Vec<String>,

    /// Rustc wrapper program (e.g., sccache, cachepot)
    #[arg(long, env = "RUSTC_WRAPPER", value_name = "PATH",
          value_hint = ValueHint::ExecutablePath,
          conflicts_with = "enable_sccache", help_heading = "Compiler Options",
          long_help = "\
Specify a rustc wrapper program (sccache, cachepot, etc) for compilation caching.")]
    pub rustc_wrapper: Option<PathBuf>,

    /// Skip cross-compilation toolchain setup
    #[arg(
        long,
        env = "NO_TOOLCHAIN_SETUP",
        help_heading = "Compiler Options",
        long_help = "\
Skip downloading and configuring cross-compilation toolchain.
Use this when you have pre-configured system compilers or want to use
only CLI-provided compiler options (--cc, --cxx, --ar, --linker)."
    )]
    pub no_toolchain_setup: bool,

    // ===== Sccache Options =====
    /// Enable sccache for compilation caching
    #[arg(
        long,
        env = "ENABLE_SCCACHE",
        conflicts_with = "rustc_wrapper",
        help_heading = "Sccache Options",
        long_help = "\
Enable sccache as the rustc wrapper for compilation caching.
Speeds up compilation by caching previous compilations."
    )]
    pub enable_sccache: bool,

    /// Directory for sccache local disk cache
    #[arg(long, env = "SCCACHE_DIR", value_name = "DIR",
          value_hint = ValueHint::DirPath, help_heading = "Sccache Options",
          long_help = "\
Directory for sccache's local disk cache. Defaults to $HOME/.cache/sccache.")]
    pub sccache_dir: Option<PathBuf>,

    /// Maximum cache size (e.g., '10G', '500M')
    #[arg(
        long,
        env = "SCCACHE_CACHE_SIZE",
        value_name = "SIZE",
        help_heading = "Sccache Options",
        long_help = "\
Maximum size of the local disk cache (e.g., '10G', '500M'). Default is 10GB."
    )]
    pub sccache_cache_size: Option<String>,

    /// Idle timeout in seconds for sccache server
    #[arg(
        long,
        env = "SCCACHE_IDLE_TIMEOUT",
        value_name = "SECONDS",
        help_heading = "Sccache Options",
        long_help = "\
Idle timeout in seconds for the sccache server. Set to 0 to run indefinitely."
    )]
    pub sccache_idle_timeout: Option<String>,

    /// Log level for sccache (error, warn, info, debug, trace)
    #[arg(
        long,
        env = "SCCACHE_LOG",
        value_name = "LEVEL",
        help_heading = "Sccache Options",
        long_help = "\
Log level for sccache. Valid: error, warn, info, debug, trace"
    )]
    pub sccache_log: Option<String>,

    /// Run sccache without the daemon (single process mode)
    #[arg(
        long,
        env = "SCCACHE_NO_DAEMON",
        help_heading = "Sccache Options",
        long_help = "\
Run sccache without daemon (single-process mode). May be slower but avoids daemon startup issues."
    )]
    pub sccache_no_daemon: bool,

    /// Enable sccache direct mode (bypass preprocessor)
    #[arg(
        long,
        env = "SCCACHE_DIRECT",
        help_heading = "Sccache Options",
        long_help = "\
Enable sccache direct mode. Caches based on source file content directly, bypassing preprocessor."
    )]
    pub sccache_direct: bool,

    // ===== CC Crate Options =====
    /// Disable CC crate default compiler flags
    #[arg(
        long,
        env = "CRATE_CC_NO_DEFAULTS",
        hide = true,
        help_heading = "CC Crate Options"
    )]
    pub cc_no_defaults: bool,

    /// Use shell-escaped flags for CC crate
    #[arg(
        long,
        env = "CC_SHELL_ESCAPED_FLAGS",
        hide = true,
        help_heading = "CC Crate Options"
    )]
    pub cc_shell_escaped_flags: bool,

    /// Enable CC crate debug output
    #[arg(
        long,
        env = "CC_ENABLE_DEBUG_OUTPUT",
        hide = true,
        help_heading = "CC Crate Options"
    )]
    pub cc_enable_debug: bool,

    // ===== Build Options =====
    /// Link the C runtime statically
    #[arg(long, value_parser = parse_optional_bool, env = "CRT_STATIC",
          value_name = "BOOL", num_args = 0..=1, default_missing_value = "true",
          help_heading = "Build Options",
          long_help = "\
Control whether the C runtime is statically linked. true=static (larger, portable),
false=dynamic (smaller, requires libc). Musl defaults to static, glibc to dynamic.")]
    pub crt_static: Option<bool>,

    /// Abort immediately on panic (smaller binary, implies --build-std)
    #[arg(
        long,
        env = "PANIC_IMMEDIATE_ABORT",
        help_heading = "Build Options",
        long_help = "\
Use panic=abort and remove panic formatting code for smaller binaries.
Requires nightly and implies --build-std. Stack traces will not be available."
    )]
    pub panic_immediate_abort: bool,

    /// Debug formatting mode (full, shallow, none) - requires nightly
    #[arg(long, value_name = "MODE", hide = true, help_heading = "Build Options")]
    pub fmt_debug: Option<String>,

    /// Location detail mode - requires nightly
    #[arg(long, value_name = "MODE", hide = true, help_heading = "Build Options")]
    pub location_detail: Option<String>,

    /// Build the standard library from source
    #[arg(long, value_parser = parse_build_std, env = "BUILD_STD",
          value_name = "CRATES", help_heading = "Build Options",
          num_args = 0..=1, default_missing_value = "true",
          long_help = "\
Build the standard library from source (requires nightly). Without arguments, builds 'std'.
Use 'true' for full std or specify crates like 'core,alloc'. Required for unsupported targets or panic=abort.")]
    pub build_std: Option<String>,

    /// Features to enable when building std
    #[arg(
        long,
        env = "BUILD_STD_FEATURES",
        value_name = "FEATURES",
        requires = "build_std",
        help_heading = "Build Options",
        long_help = "\
Space-separated features for std. Common: panic_immediate_abort, optimize_for_size"
    )]
    pub build_std_features: Option<String>,

    /// Trim paths in compiler output for reproducible builds
    #[arg(
        long,
        visible_alias = "trim-paths",
        env = "CARGO_TRIM_PATHS",
        value_name = "VALUE",
        num_args = 0..=1,
        default_missing_value = "true",
        help_heading = "Build Options",
        long_help = "\
Control how paths are trimmed in compiler output for reproducible builds.
Valid: true, macro, diagnostics, object, all, none (default: false)"
    )]
    pub cargo_trim_paths: Option<String>,

    /// Disable metadata embedding (requires nightly)
    #[arg(
        long,
        env = "NO_EMBED_METADATA",
        hide = true,
        help_heading = "Build Options"
    )]
    pub no_embed_metadata: bool,

    /// Set `RUSTC_BOOTSTRAP` for using nightly features on stable
    #[arg(
        long,
        env = "RUSTC_BOOTSTRAP",
        value_name = "VALUE",
        num_args = 0..=1,
        default_missing_value = "1",
        hide = true,
        help_heading = "Build Options"
    )]
    pub rustc_bootstrap: Option<String>,

    // ===== Output Options =====
    /// Use verbose output (-v, -vv for very verbose)
    #[arg(short = 'v', long = "verbose", action = clap::ArgAction::Count,
          env = "VERBOSE_LEVEL", conflicts_with = "quiet",
          help_heading = "Output Options",
          long_help = "\
Use verbose output. -v=commands/warnings, -vv=+deps/build scripts, -vvv=max verbosity")]
    pub verbose_level: u8,

    /// Do not print cargo log messages
    #[arg(
        short = 'q',
        long,
        env = "QUIET",
        conflicts_with = "verbose_level",
        help_heading = "Output Options",
        long_help = "\
Do not print cargo log messages. Shows only errors and warnings."
    )]
    pub quiet: bool,

    /// Diagnostic message format
    #[arg(
        long,
        env = "MESSAGE_FORMAT",
        value_name = "FMT",
        help_heading = "Output Options",
        long_help = "\
Output format for diagnostics. Valid: human (default), short, json"
    )]
    pub message_format: Option<String>,

    /// Control when colored output is used
    #[arg(
        long,
        env = "COLOR",
        value_name = "WHEN",
        help_heading = "Output Options",
        long_help = "\
Control when colored output is used. Valid: auto (default), always, never"
    )]
    pub color: Option<String>,

    /// Output the build plan in JSON (requires nightly)
    #[arg(long, env = "BUILD_PLAN", hide = true, help_heading = "Output Options")]
    pub build_plan: bool,

    /// Timing output formats (html, json)
    #[arg(long, env = "TIMINGS", value_name = "FMTS",
          num_args = 0..=1, default_missing_value = "true",
          help_heading = "Output Options",
          long_help = "\
Output timing information. --timings=HTML report, --timings=json for JSON. Saved to target/cargo-timings/.")]
    pub timings: Option<String>,

    // ===== Dependency Options =====
    /// Ignore `rust-version` specification in packages
    #[arg(
        long,
        env = "IGNORE_RUST_VERSION",
        help_heading = "Dependency Options",
        long_help = "\
Ignore rust-version specification in packages. Allows building with older Rust versions."
    )]
    pub ignore_rust_version: bool,

    /// Assert that Cargo.lock will remain unchanged
    #[arg(
        long,
        env = "LOCKED",
        help_heading = "Dependency Options",
        long_help = "\
Assert Cargo.lock will remain unchanged. Exits with error if missing or needs updating. Use in CI."
    )]
    pub locked: bool,

    /// Run without accessing the network
    #[arg(
        long,
        env = "OFFLINE",
        help_heading = "Dependency Options",
        long_help = "\
Prevent network access. Uses locally cached data. Run 'cargo fetch' first if needed."
    )]
    pub offline: bool,

    /// Require Cargo.lock and cache are up to date (implies --locked --offline)
    #[arg(
        long,
        env = "FROZEN",
        help_heading = "Dependency Options",
        long_help = "\
Equivalent to --locked --offline. Requires Cargo.lock and cache are up to date."
    )]
    pub frozen: bool,

    /// Path to Cargo.lock (unstable)
    #[arg(long, env = "LOCKFILE_PATH", value_name = "PATH",
          value_hint = ValueHint::FilePath, help_heading = "Dependency Options",
          long_help = "\
Override lockfile path from default (<workspace_root>/Cargo.lock). Requires nightly.")]
    pub lockfile_path: Option<PathBuf>,

    // ===== Build Configuration =====
    /// Number of parallel jobs to run
    #[arg(
        short = 'j',
        long,
        env = "JOBS",
        value_name = "N",
        help_heading = "Build Configuration",
        long_help = "\
Number of parallel jobs. Defaults to logical CPUs. Negative=CPUs+N. 'default' to reset."
    )]
    pub jobs: Option<String>,

    /// Build as many crates as possible, rather than aborting on first error
    #[arg(
        long,
        env = "KEEP_GOING",
        help_heading = "Build Configuration",
        long_help = "\
Build as many crates in the dependency graph as possible. Rather than aborting on the first
crate that fails to build, continue with other crates in the dependency graph."
    )]
    pub keep_going: bool,

    /// Output a future incompatibility report after the build
    #[arg(
        long,
        env = "FUTURE_INCOMPAT_REPORT",
        help_heading = "Build Configuration",
        long_help = "\
Displays a future-incompat report for any future-incompatible warnings produced during
execution of this command. See 'cargo report' for more information."
    )]
    pub future_incompat_report: bool,

    // ===== Additional Cargo Arguments =====
    /// Additional arguments to pass to cargo
    /// Note: `CARGO_ARGS` env var is handled manually in cargo.rs to support shell-style parsing
    #[arg(
        long,
        visible_alias = "args",
        value_name = "ARGS",
        hide = true,
        allow_hyphen_values = true,
        action = clap::ArgAction::Append,
        help_heading = "Additional Options"
    )]
    pub cargo_args: Vec<String>,

    /// Unstable (nightly-only) flags to Cargo
    #[arg(short = 'Z', value_name = "FLAG",
          action = clap::ArgAction::Append, help_heading = "Additional Options",
          long_help = "\
Unstable (nightly-only) flags to Cargo. Run 'cargo -Z help' for details on available flags.
Common flags: build-std, unstable-options")]
    pub cargo_z_flags: Vec<String>,

    /// Override a Cargo configuration value
    #[arg(long = "config", value_name = "KEY=VALUE",
          action = clap::ArgAction::Append, help_heading = "Additional Options",
          long_help = "\
Override a Cargo configuration value. The argument should be in TOML syntax of KEY=VALUE.
This flag may be specified multiple times.
Example: --config 'build.jobs=4' --config 'profile.release.lto=true'")]
    pub cargo_config: Vec<String>,

    /// Change to directory before doing anything
    #[arg(short = 'C', long = "directory", env = "CARGO_CWD",
          value_name = "DIR", value_hint = ValueHint::DirPath,
          help_heading = "Additional Options",
          long_help = "\
Changes the current working directory before executing any specified operations.
This affects where cargo looks for the project manifest (Cargo.toml) and .cargo/config.toml.")]
    pub cargo_cwd: Option<PathBuf>,

    /// Rust toolchain to use (alternative to +toolchain syntax)
    #[arg(
        long = "toolchain",
        env = "TOOLCHAIN",
        value_name = "TOOLCHAIN",
        help_heading = "Additional Options",
        long_help = "\
Specify the Rust toolchain to use for compilation. This is an alternative to the +toolchain
syntax (e.g., +nightly). Examples: --toolchain nightly, --toolchain stable, --toolchain 1.75.0"
    )]
    pub toolchain_option: Option<String>,

    /// GitHub mirror URL for downloading toolchains
    #[arg(long, visible_alias = "github-proxy-mirror", env = "GH_PROXY", value_name = "URL",
          value_hint = ValueHint::Url, hide_env = true,
          help_heading = "Additional Options",
          long_help = "\
Specify a GitHub mirror/proxy URL for downloading cross-compiler toolchains.
Useful in regions where GitHub access is slow or restricted.
Example: --github-proxy 'https://ghproxy.com/'")]
    pub github_proxy: Option<String>,

    /// Clean the target directory before building
    #[arg(
        long,
        env = "CLEAN_CACHE",
        help_heading = "Additional Options",
        long_help = "\
Clean the target directory before building. Equivalent to running 'cargo clean' before the build."
    )]
    pub clean_cache: bool,

    /// Arguments passed through to cargo (after --)
    /// Note: `CARGO_PASSTHROUGH_ARGS` env var is handled manually in cargo.rs to support shell-style parsing
    #[arg(
        last = true,
        allow_hyphen_values = true,
        value_name = "ARGS",
        help = "Arguments passed through to the underlying cargo command",
        long_help = "\
Arguments passed through to the underlying cargo command. Everything after -- is passed
directly to cargo/test runner. For test command, these are passed to the test binary.
Examples: test -- --nocapture --test-threads=1, run -- --arg1 --arg2"
    )]
    pub passthrough_args: Vec<String>,
}

impl BuildArgs {
    /// Create default `BuildArgs` with proper version defaults
    #[must_use]
    pub fn default_for_host() -> Self {
        Self {
            profile: "release".to_string(),
            glibc_version: DEFAULT_GLIBC_VERSION.to_string(),
            iphone_sdk_version: DEFAULT_IPHONE_SDK_VERSION.to_string(),
            macos_sdk_version: DEFAULT_MACOS_SDK_VERSION.to_string(),
            freebsd_version: DEFAULT_FREEBSD_VERSION.to_string(),
            ndk_version: DEFAULT_NDK_VERSION.to_string(),
            qemu_version: DEFAULT_QEMU_VERSION.to_string(),
            cross_make_version: DEFAULT_CROSS_MAKE_VERSION.to_string(),
            ..Default::default()
        }
    }
}

/// Parse optional bool value (true/false)
fn parse_optional_bool(s: &str) -> std::result::Result<bool, String> {
    match s.to_lowercase().as_str() {
        "true" | "1" | "yes" => Ok(true),
        "false" | "0" | "no" => Ok(false),
        _ => Err(format!("invalid bool value: {s}")),
    }
}

/// Parse build-std value (returns empty string for disabled, which is filtered later)
fn parse_build_std(s: &str) -> std::result::Result<String, String> {
    match s.to_lowercase().as_str() {
        "false" | "0" | "no" | "" => Ok(String::new()), // Will be converted to None later
        "true" | "1" | "yes" => Ok("true".to_string()),
        _ => Ok(s.to_string()),
    }
}

/// Cargo command to execute
#[derive(Debug, Clone, Copy, PartialEq, Eq, Default)]
pub enum Command {
    #[default]
    Build,
    Check,
    Run,
    Test,
    Bench,
}

impl Command {
    #[must_use]
    pub const fn as_str(&self) -> &'static str {
        match self {
            Self::Build => "build",
            Self::Check => "check",
            Self::Run => "run",
            Self::Test => "test",
            Self::Bench => "bench",
        }
    }

    #[must_use]
    pub const fn needs_runner(&self) -> bool {
        matches!(self, Self::Run | Self::Test | Self::Bench)
    }
}

/// Parsed and validated arguments
#[derive(Debug, Clone)]
pub struct Args {
    /// Rust toolchain to use (e.g., "nightly", "stable")
    pub toolchain: Option<String>,
    /// Cargo command to execute
    pub command: Command,
    /// Expanded target list (after glob pattern expansion)
    pub targets: Vec<String>,
    /// Skip passing --target to cargo (for host builds)
    pub no_cargo_target: bool,
    /// Cross-make version for toolchain downloads
    pub cross_make_version: String,
    /// Directory for cross-compiler toolchains
    pub cross_compiler_dir: PathBuf,
    /// All build arguments from CLI
    pub build: BuildArgs,
}

impl std::ops::Deref for Args {
    type Target = BuildArgs;

    fn deref(&self) -> &Self::Target {
        &self.build
    }
}

impl std::ops::DerefMut for Args {
    fn deref_mut(&mut self) -> &mut Self::Target {
        &mut self.build
    }
}

impl Args {
    /// Create Args from `BuildArgs` and Command
    fn from_build_args(b: BuildArgs, command: Command, toolchain: Option<String>) -> Result<Self> {
        let cross_compiler_dir = b
            .cross_compiler_dir
            .clone()
            .unwrap_or_else(|| std::env::temp_dir().join("rust-cross-compiler"));
        let targets = expand_target_list(&b.targets)?;

        Ok(Self {
            toolchain,
            command,
            targets,
            no_cargo_target: false,
            cross_make_version: b.cross_make_version.clone(),
            cross_compiler_dir,
            build: b,
        })
    }
}

/// Result of parsing CLI arguments
pub enum ParseResult {
    /// Normal build/check/run/test/bench command
    Build(Box<Args>),
    /// Show targets command
    ShowTargets(OutputFormat),
    /// Show version
    ShowVersion,
}

/// Remove empty environment variables that clap would incorrectly treat as having values.
/// Clap's `env = "VAR"` attribute treats empty strings as valid values, which causes
/// parsing errors for `PathBuf` and other types that don't accept empty strings.
fn sanitize_clap_env() {
    let empty_vars: Vec<_> = std::env::vars()
        .filter(|(_, v)| v.is_empty())
        .map(|(k, _)| k)
        .collect();

    for var in empty_vars {
        std::env::remove_var(&var);
    }
}

/// Parse command-line arguments
pub fn parse_args() -> Result<ParseResult> {
    let args: Vec<String> = std::env::args().collect();
    parse_args_from(args)
}

/// Parse arguments from a vector (for testing)
pub fn parse_args_from(args: Vec<String>) -> Result<ParseResult> {
    use std::env;

    // Remove empty environment variables that clap would incorrectly treat as having values
    // This must be done before clap parses, as clap reads from env vars with `env = "VAR"`
    sanitize_clap_env();

    let mut toolchain: Option<String> = None;

    // When invoked as `cargo cross`, cargo sets the CARGO env var and passes
    // args as ["cargo-cross", "cross", ...]. We need to skip both.
    // When invoked directly as `cargo-cross`, only skip the program name.
    let is_cargo_subcommand = env::var("CARGO").is_ok()
        && env::var("CARGO_HOME").is_ok()
        && args.get(1).map(String::as_str) == Some(SUBCOMMAND);

    let skip_count = if is_cargo_subcommand { 2 } else { 1 };
    let mut args: Vec<String> = args.iter().skip(skip_count).cloned().collect();

    // Extract +toolchain from args (can appear at the beginning)
    // e.g., cargo cross +nightly build -t x86_64-unknown-linux-musl
    if let Some(tc) = args.first().and_then(|a| a.strip_prefix('+')) {
        toolchain = Some(tc.to_string());
        args.remove(0);
    }

    // Prepend program name for clap (internal name, not displayed)
    args.insert(0, BIN_NAME.to_string());

    // Build command with dynamic help text
    let cmd = build_command_with_dynamic_help();

    // Try to parse with clap using modified command
    let cli = match cmd.try_get_matches_from(&args) {
        Ok(matches) => {
            Cli::from_arg_matches(&matches).map_err(|e| CrossError::ClapError(e.to_string()))?
        }
        Err(e) => {
            // For help/version/missing subcommand, let clap print and exit
            if matches!(
                e.kind(),
                clap::error::ErrorKind::DisplayHelp
                    | clap::error::ErrorKind::DisplayVersion
                    | clap::error::ErrorKind::DisplayHelpOnMissingArgumentOrSubcommand
            ) {
                e.exit();
            }
            // For argument errors, return Err so tests can catch them
            return Err(CrossError::ClapError(e.render().to_string()));
        }
    };

    process_cli(cli, toolchain)
}

/// Build the clap Command with dynamic help text based on invocation style
fn build_command_with_dynamic_help() -> clap::Command {
    let prog = program_name();

    // Build dynamic help strings
    let usage = format!("{prog} [+toolchain] <COMMAND> [OPTIONS]");
    let after_help = format!(
        "Use '{prog} <COMMAND> --help' for more information about a command.\n\n\
TOOLCHAIN:\n    \
    If the first argument begins with +, it will be interpreted as a Rust toolchain\n    \
    name (such as +nightly, +stable, or +1.75.0). This follows the same convention\n    \
    as rustup and cargo.\n\n\
EXAMPLES:\n    \
    {prog} build -t x86_64-unknown-linux-musl\n    \
    {prog} +nightly build -t aarch64-unknown-linux-gnu --profile release\n    \
    {prog} build -t '*-linux-musl' --crt-static true\n    \
    {prog} test -t x86_64-unknown-linux-musl -- --nocapture"
    );

    // Build dynamic version help texts
    let glibc_help = format!(
        "Specify glibc version for GNU libc targets. The version determines the minimum Linux kernel\n\
         version required. Lower versions provide better compatibility with older systems.\n\
         Supported: {}",
        supported_glibc_versions_str()
    );
    let freebsd_help = format!(
        "Specify FreeBSD version for FreeBSD targets. Supported: {}",
        supported_freebsd_versions_str()
    );
    let iphone_sdk_help = format!(
        "Specify iPhone SDK version for iOS targets. On Linux: uses pre-built SDK from releases.\n\
         On macOS: uses installed Xcode SDK. Supported on Linux: {}",
        supported_iphone_sdk_versions_str()
    );
    let macos_sdk_help = format!(
        "Specify macOS SDK version for Darwin targets. On Linux: uses osxcross with pre-built SDK.\n\
         On macOS: uses installed Xcode SDK. Supported on Linux: {}",
        supported_macos_sdk_versions_str()
    );

    // Get base command and modify it
    let mut cmd = Cli::command().override_usage(usage).after_help(after_help);

    // Update subcommand usage strings and version help texts
    for subcmd_name in &["build", "check", "run", "test", "bench"] {
        let glibc_help = glibc_help.clone();
        let freebsd_help = freebsd_help.clone();
        let iphone_sdk_help = iphone_sdk_help.clone();
        let macos_sdk_help = macos_sdk_help.clone();
        cmd = cmd.mut_subcommand(*subcmd_name, |subcmd| {
            subcmd
                .override_usage(format!(
                    "{prog} [+toolchain] {subcmd_name} [OPTIONS] [-- <PASSTHROUGH_ARGS>...]"
                ))
                .mut_arg("glibc_version", |arg| arg.long_help(glibc_help))
                .mut_arg("freebsd_version", |arg| arg.long_help(freebsd_help))
                .mut_arg("iphone_sdk_version", |arg| arg.long_help(iphone_sdk_help))
                .mut_arg("macos_sdk_version", |arg| arg.long_help(macos_sdk_help))
        });
    }

    cmd
}

fn process_cli(cli: Cli, toolchain: Option<String>) -> Result<ParseResult> {
    match cli.command {
        CliCommand::Build(args) => {
            let args = finalize_args(args, Command::Build, toolchain)?;
            Ok(ParseResult::Build(Box::new(args)))
        }
        CliCommand::Check(args) => {
            let args = finalize_args(args, Command::Check, toolchain)?;
            Ok(ParseResult::Build(Box::new(args)))
        }
        CliCommand::Run(args) => {
            let args = finalize_args(args, Command::Run, toolchain)?;
            Ok(ParseResult::Build(Box::new(args)))
        }
        CliCommand::Test(args) => {
            let args = finalize_args(args, Command::Test, toolchain)?;
            Ok(ParseResult::Build(Box::new(args)))
        }
        CliCommand::Bench(args) => {
            let args = finalize_args(args, Command::Bench, toolchain)?;
            Ok(ParseResult::Build(Box::new(args)))
        }
        CliCommand::Targets(args) => Ok(ParseResult::ShowTargets(args.format)),
        CliCommand::Version => Ok(ParseResult::ShowVersion),
    }
}

/// Check if a string is a glob pattern (contains *, ?, or [)
fn is_glob_pattern(s: &str) -> bool {
    s.contains('*') || s.contains('?') || s.contains('[')
}

/// Validate that a target triple only contains valid characters (a-z, 0-9, -, _)
fn validate_target_triple(target: &str) -> Result<()> {
    for c in target.chars() {
        if !c.is_ascii_lowercase() && !c.is_ascii_digit() && c != '-' && c != '_' {
            return Err(CrossError::InvalidTargetTriple {
                target: target.to_string(),
                char: c,
            });
        }
    }
    Ok(())
}

/// Expand target list, handling glob patterns
fn expand_target_list(targets: &[String]) -> Result<Vec<String>> {
    let mut result = Vec::new();
    for target in targets {
        // Split by comma or newline to support multiple delimiters
        for part in target.split([',', '\n']) {
            let part = part.trim();
            if part.is_empty() {
                continue;
            }
            let expanded = config::expand_targets(part);
            if expanded.is_empty() {
                // If it was a glob pattern that matched nothing, error
                if is_glob_pattern(part) {
                    return Err(CrossError::NoMatchingTargets {
                        pattern: part.to_string(),
                    });
                }
                // Not a glob pattern, validate and use as-is
                validate_target_triple(part)?;
                if !result.contains(&part.to_string()) {
                    result.push(part.to_string());
                }
            } else {
                for t in expanded {
                    let t = t.to_string();
                    if !result.contains(&t) {
                        result.push(t);
                    }
                }
            }
        }
    }
    Ok(result)
}

fn finalize_args(
    mut build_args: BuildArgs,
    command: Command,
    toolchain: Option<String>,
) -> Result<Args> {
    // Handle --release flag: set profile to "release"
    if build_args.release {
        build_args.profile = "release".to_string();
    }

    // Handle build_std: empty string means disabled (from env var "false")
    if build_args
        .build_std
        .as_ref()
        .is_some_and(std::string::String::is_empty)
    {
        build_args.build_std = None;
    }

    // Handle env vars for cargo_args and passthrough_args (command line takes priority)
    if build_args.cargo_args.is_empty() {
        if let Some(env_args) = parse_env_args("CARGO_ARGS") {
            build_args.cargo_args = env_args;
        }
    }
    if build_args.passthrough_args.is_empty() {
        if let Some(env_args) = parse_env_args("CARGO_PASSTHROUGH_ARGS") {
            build_args.passthrough_args = env_args;
        }
    }

    // Merge toolchain: +toolchain syntax takes precedence over --toolchain option
    let final_toolchain = toolchain.or_else(|| build_args.toolchain_option.clone());

    let mut args = Args::from_build_args(build_args, command, final_toolchain)?;

    // Validate versions
    validate_versions(&args)?;

    // Handle empty targets - default to host
    if args.targets.is_empty() {
        let host = config::HostPlatform::detect();
        args.targets.push(host.triple);
        args.no_toolchain_setup = true;
        args.no_cargo_target = true;
    }
    // Note: "host-tuple" is handled dynamically in execute_target

    Ok(args)
}

/// Parse environment variable as shell-style arguments using shlex
/// Returns None if env var is not set or empty
fn parse_env_args(env_name: &str) -> Option<Vec<String>> {
    let value = std::env::var(env_name).ok()?;
    let value = value.trim();
    if value.is_empty() {
        return None;
    }
    match shlex::split(value) {
        Some(parsed) if !parsed.is_empty() => Some(parsed),
        Some(_) => None,
        None => {
            eprintln!("Warning: Failed to parse {env_name} (mismatched quotes?): {value}");
            None
        }
    }
}

/// Validate version options
fn validate_versions(args: &Args) -> Result<()> {
    // Only validate glibc version if it's specified (non-empty)
    // Empty string means use default version, which is valid for both gnu and musl targets
    if !args.glibc_version.is_empty()
        && !SUPPORTED_GLIBC_VERSIONS.contains(&args.glibc_version.as_str())
    {
        return Err(CrossError::UnsupportedGlibcVersion {
            version: args.glibc_version.clone(),
            supported: SUPPORTED_GLIBC_VERSIONS.join(", "),
        });
    }

    let host = config::HostPlatform::detect();
    if !host.is_darwin()
        && !SUPPORTED_IPHONE_SDK_VERSIONS.contains(&args.iphone_sdk_version.as_str())
    {
        return Err(CrossError::UnsupportedIphoneSdkVersion {
            version: args.iphone_sdk_version.clone(),
            supported: SUPPORTED_IPHONE_SDK_VERSIONS.join(", "),
        });
    }

    if !host.is_darwin() && !SUPPORTED_MACOS_SDK_VERSIONS.contains(&args.macos_sdk_version.as_str())
    {
        return Err(CrossError::UnsupportedMacosSdkVersion {
            version: args.macos_sdk_version.clone(),
            supported: SUPPORTED_MACOS_SDK_VERSIONS.join(", "),
        });
    }

    if !SUPPORTED_FREEBSD_VERSIONS.contains(&args.freebsd_version.as_str()) {
        return Err(CrossError::UnsupportedFreebsdVersion {
            version: args.freebsd_version.clone(),
            supported: SUPPORTED_FREEBSD_VERSIONS.join(", "),
        });
    }

    Ok(())
}

/// Print all supported targets
pub fn print_all_targets(format: OutputFormat) {
    let mut targets: Vec<_> = config::all_targets().collect();
    targets.sort_unstable();

    match format {
        OutputFormat::Text => {
            use colored::Colorize;
            println!("{}", "Supported Rust targets:".bright_green());
            for target in &targets {
                println!("  {}", target.bright_cyan());
            }
        }
        OutputFormat::Json => {
            let json_array = serde_json::to_string(&targets).unwrap_or_else(|_| "[]".to_string());
            println!("{json_array}");
        }
        OutputFormat::Plain => {
            for target in &targets {
                println!("{target}");
            }
        }
    }

    // Output to GITHUB_OUTPUT if running in GitHub Actions
    if let Ok(github_output) = std::env::var("GITHUB_OUTPUT") {
        let json_array = serde_json::to_string(&targets).unwrap_or_else(|_| "[]".to_string());
        if let Ok(mut file) = std::fs::OpenOptions::new()
            .append(true)
            .open(&github_output)
        {
            use std::io::Write;
            let _ = writeln!(file, "all-targets={json_array}");
        }
    }
}

/// Print version information
pub fn print_version() {
    use colored::Colorize;

    let version = env!("CARGO_PKG_VERSION");
    let name = env!("CARGO_PKG_NAME");
    println!("{} {}", name.bright_green(), version.bright_cyan());
}

#[cfg(test)]
mod tests {
    use super::*;

    fn parse(args: &[&str]) -> Result<Args> {
        let args: Vec<String> = args.iter().map(std::string::ToString::to_string).collect();
        match parse_args_from(args)? {
            ParseResult::Build(args) => Ok(*args),
            ParseResult::ShowTargets(_) => panic!("unexpected ShowTargets"),
            ParseResult::ShowVersion => panic!("unexpected ShowVersion"),
        }
    }

    // Note: test_parse_empty_requires_subcommand removed because MissingSubcommand
    // now calls exit() which cannot be tested

    #[test]
    fn test_parse_build_command() {
        let args = parse(&["cargo-cross", "build"]).unwrap();
        assert_eq!(args.command, Command::Build);
    }

    #[test]
    fn test_parse_check_command() {
        let args = parse(&["cargo-cross", "check"]).unwrap();
        assert_eq!(args.command, Command::Check);
    }

    #[test]
    fn test_parse_target() {
        let args = parse(&["cargo-cross", "build", "-t", "x86_64-unknown-linux-musl"]).unwrap();
        assert_eq!(args.targets, vec!["x86_64-unknown-linux-musl"]);
    }

    #[test]
    fn test_parse_multiple_targets() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "-t",
            "x86_64-unknown-linux-musl,aarch64-unknown-linux-musl",
        ])
        .unwrap();
        assert_eq!(
            args.targets,
            vec!["x86_64-unknown-linux-musl", "aarch64-unknown-linux-musl"]
        );
    }

    #[test]
    fn test_parse_verbose() {
        let args = parse(&["cargo-cross", "build", "-vvv"]).unwrap();
        assert_eq!(args.verbose_level, 3);
    }

    #[test]
    fn test_parse_crt_static_flag() {
        let args = parse(&["cargo-cross", "build", "--crt-static", "true"]).unwrap();
        assert_eq!(args.crt_static, Some(true));
    }

    #[test]
    fn test_parse_crt_static_false() {
        let args = parse(&["cargo-cross", "build", "--crt-static", "false"]).unwrap();
        assert_eq!(args.crt_static, Some(false));
    }

    #[test]
    fn test_parse_crt_static_no_value() {
        // --crt-static without value should default to true
        let args = parse(&["cargo-cross", "build", "--crt-static"]).unwrap();
        assert_eq!(args.crt_static, Some(true));
    }

    #[test]
    fn test_parse_crt_static_not_provided() {
        // When --crt-static is not provided at all, value should be None
        let args = parse(&["cargo-cross", "build"]).unwrap();
        assert_eq!(args.crt_static, None);
    }

    #[test]
    fn test_parse_build_std() {
        let args = parse(&["cargo-cross", "build", "--build-std", "true"]).unwrap();
        assert_eq!(args.build_std, Some("true".to_string()));
    }

    #[test]
    fn test_parse_build_std_crates() {
        let args = parse(&["cargo-cross", "build", "--build-std", "core,alloc"]).unwrap();
        assert_eq!(args.build_std, Some("core,alloc".to_string()));
    }

    #[test]
    fn test_parse_build_std_false() {
        let args = parse(&["cargo-cross", "build", "--build-std", "false"]).unwrap();
        assert_eq!(args.build_std, None);
    }

    #[test]
    fn test_parse_build_std_no_value() {
        // --build-std without value should default to "true"
        let args = parse(&["cargo-cross", "build", "--build-std"]).unwrap();
        assert_eq!(args.build_std, Some("true".to_string()));
    }

    #[test]
    fn test_parse_features() {
        let args = parse(&["cargo-cross", "build", "--features", "foo,bar"]).unwrap();
        assert_eq!(args.features, Some("foo,bar".to_string()));
    }

    #[test]
    fn test_parse_no_default_features() {
        let args = parse(&["cargo-cross", "build", "--no-default-features"]).unwrap();
        assert!(args.no_default_features);
    }

    #[test]
    fn test_parse_profile() {
        let args = parse(&["cargo-cross", "build", "--profile", "dev"]).unwrap();
        assert_eq!(args.profile, "dev");
    }

    #[test]
    fn test_parse_jobs() {
        let args = parse(&["cargo-cross", "build", "-j", "4"]).unwrap();
        assert_eq!(args.jobs, Some("4".to_string()));
    }

    #[test]
    fn test_parse_passthrough_args() {
        let args = parse(&["cargo-cross", "build", "--", "--foo", "--bar"]).unwrap();
        assert_eq!(args.passthrough_args, vec!["--foo", "--bar"]);
    }

    #[test]
    fn test_parse_z_flag() {
        let args = parse(&["cargo-cross", "build", "-Z", "build-std"]).unwrap();
        assert_eq!(args.cargo_z_flags, vec!["build-std"]);
    }

    #[test]
    fn test_parse_config_flag() {
        let args = parse(&["cargo-cross", "build", "--config", "opt-level=3"]).unwrap();
        assert_eq!(args.cargo_config, vec!["opt-level=3"]);
    }

    #[test]
    fn test_targets_subcommand() {
        let args: Vec<String> = vec!["cargo-cross".to_string(), "targets".to_string()];
        match parse_args_from(args).unwrap() {
            ParseResult::ShowTargets(format) => {
                assert_eq!(format, OutputFormat::Text);
            }
            _ => panic!("expected ShowTargets"),
        }
    }

    #[test]
    fn test_targets_json_format() {
        let args: Vec<String> = vec![
            "cargo-cross".to_string(),
            "targets".to_string(),
            "--format".to_string(),
            "json".to_string(),
        ];
        match parse_args_from(args).unwrap() {
            ParseResult::ShowTargets(format) => {
                assert_eq!(format, OutputFormat::Json);
            }
            _ => panic!("expected ShowTargets"),
        }
    }

    #[test]
    fn test_targets_plain_format() {
        let args: Vec<String> = vec![
            "cargo-cross".to_string(),
            "targets".to_string(),
            "-f".to_string(),
            "plain".to_string(),
        ];
        match parse_args_from(args).unwrap() {
            ParseResult::ShowTargets(format) => {
                assert_eq!(format, OutputFormat::Plain);
            }
            _ => panic!("expected ShowTargets"),
        }
    }

    #[test]
    fn test_parse_toolchain() {
        let args = parse(&["cargo-cross", "+nightly", "build"]).unwrap();
        assert_eq!(args.toolchain, Some("nightly".to_string()));
        assert_eq!(args.command, Command::Build);
    }

    #[test]
    fn test_parse_toolchain_with_target() {
        let args = parse(&[
            "cargo-cross",
            "+nightly",
            "build",
            "-t",
            "x86_64-unknown-linux-musl",
        ])
        .unwrap();
        assert_eq!(args.toolchain, Some("nightly".to_string()));
        assert_eq!(args.targets, vec!["x86_64-unknown-linux-musl"]);
    }

    // Equals syntax vs space syntax tests

    #[test]
    fn test_equals_syntax_target() {
        let args = parse(&["cargo-cross", "build", "-t=x86_64-unknown-linux-musl"]).unwrap();
        assert_eq!(args.targets, vec!["x86_64-unknown-linux-musl"]);
    }

    #[test]
    fn test_equals_syntax_long_target() {
        let args = parse(&["cargo-cross", "build", "--target=x86_64-unknown-linux-musl"]).unwrap();
        assert_eq!(args.targets, vec!["x86_64-unknown-linux-musl"]);
    }

    #[test]
    fn test_equals_syntax_profile() {
        let args = parse(&["cargo-cross", "build", "--profile=dev"]).unwrap();
        assert_eq!(args.profile, "dev");
    }

    #[test]
    fn test_equals_syntax_features() {
        let args = parse(&["cargo-cross", "build", "--features=foo,bar"]).unwrap();
        assert_eq!(args.features, Some("foo,bar".to_string()));
    }

    #[test]
    fn test_equals_syntax_short_features() {
        let args = parse(&["cargo-cross", "build", "-F=foo,bar"]).unwrap();
        assert_eq!(args.features, Some("foo,bar".to_string()));
    }

    #[test]
    fn test_equals_syntax_jobs() {
        let args = parse(&["cargo-cross", "build", "-j=8"]).unwrap();
        assert_eq!(args.jobs, Some("8".to_string()));
    }

    #[test]
    fn test_equals_syntax_crt_static() {
        let args = parse(&["cargo-cross", "build", "--crt-static=true"]).unwrap();
        assert_eq!(args.crt_static, Some(true));
    }

    #[test]
    fn test_equals_syntax_build_std() {
        let args = parse(&["cargo-cross", "build", "--build-std=core,alloc"]).unwrap();
        assert_eq!(args.build_std, Some("core,alloc".to_string()));
    }

    #[test]
    fn test_equals_syntax_manifest_path() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "--manifest-path=/path/to/Cargo.toml",
        ])
        .unwrap();
        assert_eq!(
            args.manifest_path,
            Some(PathBuf::from("/path/to/Cargo.toml"))
        );
    }

    #[test]
    fn test_equals_syntax_cross_compiler_dir() {
        let args = parse(&["cargo-cross", "build", "--cross-compiler-dir=/opt/cross"]).unwrap();
        assert_eq!(args.cross_compiler_dir, PathBuf::from("/opt/cross"));
    }

    #[test]
    fn test_equals_syntax_glibc_version() {
        let args = parse(&["cargo-cross", "build", "--glibc-version=2.31"]).unwrap();
        assert_eq!(args.glibc_version, "2.31");
    }

    #[test]
    fn test_equals_syntax_cc_cxx_ar() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "--cc=/usr/bin/gcc",
            "--cxx=/usr/bin/g++",
            "--ar=/usr/bin/ar",
        ])
        .unwrap();
        assert_eq!(args.cc, Some(PathBuf::from("/usr/bin/gcc")));
        assert_eq!(args.cxx, Some(PathBuf::from("/usr/bin/g++")));
        assert_eq!(args.ar, Some(PathBuf::from("/usr/bin/ar")));
    }

    #[test]
    fn test_equals_syntax_linker() {
        let args = parse(&["cargo-cross", "build", "--linker=/usr/bin/ld.lld"]).unwrap();
        assert_eq!(args.linker, Some(PathBuf::from("/usr/bin/ld.lld")));
    }

    #[test]
    fn test_equals_syntax_cflags_with_spaces() {
        let args = parse(&["cargo-cross", "build", "--cflags=-O2 -Wall -Wextra"]).unwrap();
        assert_eq!(args.cflags, Some("-O2 -Wall -Wextra".to_string()));
    }

    #[test]
    fn test_equals_syntax_ldflags() {
        let args = parse(&["cargo-cross", "build", "--ldflags=-L/usr/local/lib -static"]).unwrap();
        assert_eq!(args.ldflags, Some("-L/usr/local/lib -static".to_string()));
    }

    #[test]
    fn test_equals_syntax_rustflag() {
        let args = parse(&["cargo-cross", "build", "--rustflag=-C opt-level=3"]).unwrap();
        assert_eq!(args.rustflags, vec!["-C opt-level=3"]);
    }

    #[test]
    fn test_equals_syntax_github_proxy() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "--github-proxy=https://mirror.example.com/",
        ])
        .unwrap();
        assert_eq!(
            args.github_proxy,
            Some("https://mirror.example.com/".to_string())
        );
    }

    #[test]
    fn test_equals_syntax_sccache_options() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "--enable-sccache",
            "--sccache-dir=/tmp/sccache",
            "--sccache-cache-size=20G",
        ])
        .unwrap();
        assert!(args.enable_sccache);
        assert_eq!(args.sccache_dir, Some(PathBuf::from("/tmp/sccache")));
        assert_eq!(args.sccache_cache_size, Some("20G".to_string()));
    }

    #[test]
    fn test_equals_syntax_config_with_equals_in_value() {
        let args = parse(&["cargo-cross", "build", "--config=build.jobs=4"]).unwrap();
        assert_eq!(args.cargo_config, vec!["build.jobs=4"]);
    }

    #[test]
    fn test_equals_syntax_multiple_options() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "-t=x86_64-unknown-linux-musl",
            "--profile=release",
            "-F=serde,json",
            "-j=8",
            "--crt-static=true",
            "--build-std=core,alloc",
        ])
        .unwrap();
        assert_eq!(args.targets, vec!["x86_64-unknown-linux-musl"]);
        assert_eq!(args.profile, "release");
        assert_eq!(args.features, Some("serde,json".to_string()));
        assert_eq!(args.jobs, Some("8".to_string()));
        assert_eq!(args.crt_static, Some(true));
        assert_eq!(args.build_std, Some("core,alloc".to_string()));
    }

    #[test]
    fn test_equals_syntax_toolchain() {
        let args = parse(&["cargo-cross", "build", "--toolchain=nightly-2024-01-01"]).unwrap();
        assert_eq!(args.toolchain, Some("nightly-2024-01-01".to_string()));
    }

    #[test]
    fn test_equals_syntax_target_dir() {
        let args = parse(&["cargo-cross", "build", "--target-dir=/tmp/target"]).unwrap();
        assert_eq!(args.cargo_target_dir, Some(PathBuf::from("/tmp/target")));
    }

    #[test]
    fn test_equals_syntax_z_flag() {
        let args = parse(&["cargo-cross", "build", "-Z=build-std"]).unwrap();
        assert_eq!(args.cargo_z_flags, vec!["build-std"]);
    }

    #[test]
    fn test_equals_syntax_directory() {
        let args = parse(&["cargo-cross", "build", "-C=/path/to/project"]).unwrap();
        assert_eq!(args.cargo_cwd, Some(PathBuf::from("/path/to/project")));
    }

    #[test]
    fn test_equals_syntax_message_format() {
        let args = parse(&["cargo-cross", "build", "--message-format=json"]).unwrap();
        assert_eq!(args.message_format, Some("json".to_string()));
    }

    #[test]
    fn test_equals_syntax_color() {
        let args = parse(&["cargo-cross", "build", "--color=always"]).unwrap();
        assert_eq!(args.color, Some("always".to_string()));
    }

    // Mixed flags and options tests

    #[test]
    fn test_mixed_crt_static_then_flag() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "--crt-static",
            "true",
            "--no-default-features",
        ])
        .unwrap();
        assert_eq!(args.crt_static, Some(true));
        assert!(args.no_default_features);
    }

    #[test]
    fn test_mixed_crt_static_then_short_option() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "--crt-static",
            "false",
            "-F",
            "serde",
        ])
        .unwrap();
        assert_eq!(args.crt_static, Some(false));
        assert_eq!(args.features, Some("serde".to_string()));
    }

    #[test]
    fn test_mixed_crt_static_with_target() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "--crt-static",
            "true",
            "-t",
            "x86_64-unknown-linux-musl",
            "--profile",
            "release",
        ])
        .unwrap();
        assert_eq!(args.crt_static, Some(true));
        assert_eq!(args.targets, vec!["x86_64-unknown-linux-musl"]);
        assert_eq!(args.profile, "release");
    }

    #[test]
    fn test_mixed_flag_then_crt_static() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "--no-default-features",
            "--crt-static",
            "true",
        ])
        .unwrap();
        assert!(args.no_default_features);
        assert_eq!(args.crt_static, Some(true));
    }

    #[test]
    fn test_mixed_multiple_flags_and_options() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "-t",
            "aarch64-unknown-linux-musl",
            "--no-default-features",
            "-F",
            "serde,json",
            "--crt-static",
            "true",
            "--profile",
            "release",
            "-vv",
        ])
        .unwrap();
        assert_eq!(args.targets, vec!["aarch64-unknown-linux-musl"]);
        assert!(args.no_default_features);
        assert_eq!(args.features, Some("serde,json".to_string()));
        assert_eq!(args.crt_static, Some(true));
        assert_eq!(args.profile, "release");
        assert_eq!(args.verbose_level, 2);
    }

    // Complex option ordering tests

    #[test]
    fn test_options_before_command_style() {
        // Options can come in any order
        let args = parse(&[
            "cargo-cross",
            "build",
            "--profile",
            "dev",
            "-t",
            "x86_64-unknown-linux-musl",
            "--features",
            "foo",
        ])
        .unwrap();
        assert_eq!(args.profile, "dev");
        assert_eq!(args.targets, vec!["x86_64-unknown-linux-musl"]);
        assert_eq!(args.features, Some("foo".to_string()));
    }

    #[test]
    fn test_interleaved_short_and_long_options() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "-t",
            "x86_64-unknown-linux-musl",
            "--profile",
            "release",
            "-F",
            "foo",
            "--no-default-features",
            "-j",
            "4",
            "--locked",
        ])
        .unwrap();
        assert_eq!(args.targets, vec!["x86_64-unknown-linux-musl"]);
        assert_eq!(args.profile, "release");
        assert_eq!(args.features, Some("foo".to_string()));
        assert!(args.no_default_features);
        assert_eq!(args.jobs, Some("4".to_string()));
        assert!(args.locked);
    }

    // Verbose flag variations

    #[test]
    fn test_verbose_single() {
        let args = parse(&["cargo-cross", "build", "-v"]).unwrap();
        assert_eq!(args.verbose_level, 1);
    }

    #[test]
    fn test_verbose_double() {
        let args = parse(&["cargo-cross", "build", "-vv"]).unwrap();
        assert_eq!(args.verbose_level, 2);
    }

    #[test]
    fn test_verbose_triple() {
        let args = parse(&["cargo-cross", "build", "-vvv"]).unwrap();
        assert_eq!(args.verbose_level, 3);
    }

    #[test]
    fn test_verbose_separate() {
        let args = parse(&["cargo-cross", "build", "-v", "-v", "-v"]).unwrap();
        assert_eq!(args.verbose_level, 3);
    }

    #[test]
    fn test_verbose_long_form() {
        let args = parse(&["cargo-cross", "build", "--verbose", "--verbose"]).unwrap();
        assert_eq!(args.verbose_level, 2);
    }

    #[test]
    fn test_verbose_mixed_with_options() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "-v",
            "-t",
            "x86_64-unknown-linux-musl",
            "-v",
        ])
        .unwrap();
        assert_eq!(args.verbose_level, 2);
        assert_eq!(args.targets, vec!["x86_64-unknown-linux-musl"]);
    }

    // Timings option (optional value) tests

    #[test]
    fn test_timings_without_value() {
        let args = parse(&["cargo-cross", "build", "--timings"]).unwrap();
        assert_eq!(args.timings, Some("true".to_string()));
    }

    #[test]
    fn test_timings_with_value() {
        let args = parse(&["cargo-cross", "build", "--timings=html"]).unwrap();
        assert_eq!(args.timings, Some("html".to_string()));
    }

    #[test]
    fn test_timings_followed_by_flag() {
        let args = parse(&["cargo-cross", "build", "--timings", "--locked"]).unwrap();
        assert_eq!(args.timings, Some("true".to_string()));
        assert!(args.locked);
    }

    #[test]
    fn test_timings_followed_by_option() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "--timings",
            "-t",
            "x86_64-unknown-linux-musl",
        ])
        .unwrap();
        assert_eq!(args.timings, Some("true".to_string()));
        assert_eq!(args.targets, vec!["x86_64-unknown-linux-musl"]);
    }

    // Multiple values / repeated options tests

    #[test]
    fn test_multiple_targets_comma_separated() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "-t",
            "x86_64-unknown-linux-musl,aarch64-unknown-linux-musl,armv7-unknown-linux-musleabihf",
        ])
        .unwrap();
        assert_eq!(args.targets.len(), 3);
        assert_eq!(args.targets[0], "x86_64-unknown-linux-musl");
        assert_eq!(args.targets[1], "aarch64-unknown-linux-musl");
        assert_eq!(args.targets[2], "armv7-unknown-linux-musleabihf");
    }

    #[test]
    fn test_multiple_targets_repeated_option() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "-t",
            "x86_64-unknown-linux-musl",
            "-t",
            "aarch64-unknown-linux-musl",
        ])
        .unwrap();
        assert_eq!(args.targets.len(), 2);
    }

    #[test]
    fn test_multiple_rustflags() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "--rustflag",
            "-C opt-level=3",
            "--rustflag",
            "-C lto=thin",
        ])
        .unwrap();
        assert_eq!(args.rustflags.len(), 2);
        assert_eq!(args.rustflags[0], "-C opt-level=3");
        assert_eq!(args.rustflags[1], "-C lto=thin");
    }

    #[test]
    fn test_multiple_config_flags() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "--config",
            "build.jobs=4",
            "--config",
            "profile.release.lto=true",
        ])
        .unwrap();
        assert_eq!(args.cargo_config.len(), 2);
    }

    #[test]
    fn test_multiple_z_flags() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "-Z",
            "build-std",
            "-Z",
            "unstable-options",
        ])
        .unwrap();
        assert_eq!(args.cargo_z_flags.len(), 2);
    }

    // Hyphen values tests (for compiler flags)

    #[test]
    fn test_cflags_with_hyphen() {
        let args = parse(&["cargo-cross", "build", "--cflags", "-O2 -Wall"]).unwrap();
        assert_eq!(args.cflags, Some("-O2 -Wall".to_string()));
    }

    #[test]
    fn test_ldflags_with_hyphen() {
        let args = parse(&["cargo-cross", "build", "--ldflags", "-L/usr/local/lib"]).unwrap();
        assert_eq!(args.ldflags, Some("-L/usr/local/lib".to_string()));
    }

    #[test]
    fn test_rustflag_with_hyphen() {
        let args = parse(&["cargo-cross", "build", "--rustflag", "-C target-cpu=native"]).unwrap();
        assert_eq!(args.rustflags, vec!["-C target-cpu=native"]);
    }

    // Passthrough arguments tests

    #[test]
    fn test_passthrough_single() {
        let args = parse(&["cargo-cross", "build", "--", "--nocapture"]).unwrap();
        assert_eq!(args.passthrough_args, vec!["--nocapture"]);
    }

    #[test]
    fn test_passthrough_multiple() {
        let args = parse(&[
            "cargo-cross",
            "test",
            "--",
            "--nocapture",
            "--test-threads=1",
        ])
        .unwrap();
        assert_eq!(
            args.passthrough_args,
            vec!["--nocapture", "--test-threads=1"]
        );
    }

    #[test]
    fn test_passthrough_with_hyphen_values() {
        let args = parse(&["cargo-cross", "build", "--", "-v", "--foo", "-bar"]).unwrap();
        assert_eq!(args.passthrough_args, vec!["-v", "--foo", "-bar"]);
    }

    #[test]
    fn test_passthrough_after_options() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "-t",
            "x86_64-unknown-linux-musl",
            "--profile",
            "release",
            "--",
            "--foo",
            "--bar",
        ])
        .unwrap();
        assert_eq!(args.targets, vec!["x86_64-unknown-linux-musl"]);
        assert_eq!(args.profile, "release");
        assert_eq!(args.passthrough_args, vec!["--foo", "--bar"]);
    }

    // Alias tests

    #[test]
    fn test_alias_targets() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "--targets",
            "x86_64-unknown-linux-musl",
        ])
        .unwrap();
        assert_eq!(args.targets, vec!["x86_64-unknown-linux-musl"]);
    }

    #[test]
    fn test_alias_workspace_all() {
        let args = parse(&["cargo-cross", "build", "--all"]).unwrap();
        assert!(args.workspace);
    }

    #[test]
    fn test_alias_rustflags() {
        let args = parse(&["cargo-cross", "build", "--rustflags", "-C lto"]).unwrap();
        assert_eq!(args.rustflags, vec!["-C lto"]);
    }

    #[test]
    fn test_alias_trim_paths() {
        let args = parse(&["cargo-cross", "build", "--trim-paths", "all"]).unwrap();
        assert_eq!(args.cargo_trim_paths, Some("all".to_string()));
    }

    #[test]
    fn test_trim_paths_no_value() {
        // --trim-paths without value should default to "true"
        let args = parse(&["cargo-cross", "build", "--trim-paths"]).unwrap();
        assert_eq!(args.cargo_trim_paths, Some("true".to_string()));
    }

    #[test]
    fn test_cargo_trim_paths_no_value() {
        // --cargo-trim-paths without value should default to "true"
        let args = parse(&["cargo-cross", "build", "--cargo-trim-paths"]).unwrap();
        assert_eq!(args.cargo_trim_paths, Some("true".to_string()));
    }

    #[test]
    fn test_rustc_bootstrap_no_value() {
        // --rustc-bootstrap without value should default to "1"
        let args = parse(&["cargo-cross", "build", "--rustc-bootstrap"]).unwrap();
        assert_eq!(args.rustc_bootstrap, Some("1".to_string()));
    }

    #[test]
    fn test_rustc_bootstrap_with_value() {
        let args = parse(&["cargo-cross", "build", "--rustc-bootstrap", "mycrate"]).unwrap();
        assert_eq!(args.rustc_bootstrap, Some("mycrate".to_string()));
    }

    // Command alias tests

    #[test]
    fn test_command_alias_b() {
        let args = parse(&["cargo-cross", "b"]).unwrap();
        assert_eq!(args.command, Command::Build);
    }

    #[test]
    fn test_command_alias_c() {
        let args = parse(&["cargo-cross", "c"]).unwrap();
        assert_eq!(args.command, Command::Check);
    }

    #[test]
    fn test_command_alias_r() {
        let args = parse(&["cargo-cross", "r"]).unwrap();
        assert_eq!(args.command, Command::Run);
    }

    #[test]
    fn test_command_alias_t() {
        let args = parse(&["cargo-cross", "t"]).unwrap();
        assert_eq!(args.command, Command::Test);
    }

    // Requires relationship tests

    #[test]
    fn test_requires_exclude_needs_workspace() {
        let result = parse(&["cargo-cross", "build", "--exclude", "foo"]);
        assert!(
            result.is_err() || {
                // clap exits on error, so we might not get here
                false
            }
        );
    }

    #[test]
    fn test_requires_exclude_with_workspace() {
        let args = parse(&["cargo-cross", "build", "--workspace", "--exclude", "foo"]).unwrap();
        assert!(args.workspace);
        assert_eq!(args.exclude, Some("foo".to_string()));
    }

    #[test]
    fn test_requires_build_std_features_with_build_std() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "--build-std",
            "core,alloc",
            "--build-std-features",
            "panic_immediate_abort",
        ])
        .unwrap();
        assert_eq!(args.build_std, Some("core,alloc".to_string()));
        assert_eq!(
            args.build_std_features,
            Some("panic_immediate_abort".to_string())
        );
    }

    // Conflicts relationship tests

    #[test]
    fn test_conflicts_quiet_verbose() {
        let result = parse(&["cargo-cross", "build", "--quiet", "--verbose"]);
        // This should fail due to conflict
        assert!(
            result.is_err() || {
                // clap exits, checking we don't panic
                false
            }
        );
    }

    #[test]
    fn test_conflicts_features_all_features() {
        let result = parse(&[
            "cargo-cross",
            "build",
            "--features",
            "foo",
            "--all-features",
        ]);
        assert!(result.is_err());
    }

    #[test]
    fn test_no_toolchain_setup() {
        let args = parse(&["cargo-cross", "build", "--no-toolchain-setup"]).unwrap();
        assert!(args.no_toolchain_setup);
    }

    #[test]
    fn test_linker_with_no_toolchain_setup() {
        // --linker and --no-toolchain-setup can be used together
        let args = parse(&[
            "cargo-cross",
            "build",
            "--linker",
            "/usr/bin/ld",
            "--no-toolchain-setup",
        ])
        .unwrap();
        assert!(args.no_toolchain_setup);
        assert_eq!(args.linker, Some(PathBuf::from("/usr/bin/ld")));
    }

    // Complex real-world scenario tests

    #[test]
    fn test_real_world_linux_musl_build() {
        let args = parse(&[
            "cargo-cross",
            "+nightly",
            "build",
            "-t",
            "x86_64-unknown-linux-musl",
            "--profile",
            "release",
            "--crt-static",
            "true",
            "--no-default-features",
            "-F",
            "serde,json",
            "-j",
            "8",
            "--locked",
        ])
        .unwrap();
        assert_eq!(args.toolchain, Some("nightly".to_string()));
        assert_eq!(args.command, Command::Build);
        assert_eq!(args.targets, vec!["x86_64-unknown-linux-musl"]);
        assert_eq!(args.profile, "release");
        assert_eq!(args.crt_static, Some(true));
        assert!(args.no_default_features);
        assert_eq!(args.features, Some("serde,json".to_string()));
        assert_eq!(args.jobs, Some("8".to_string()));
        assert!(args.locked);
    }

    #[test]
    fn test_real_world_multi_target_build() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "-t",
            "x86_64-unknown-linux-musl,aarch64-unknown-linux-musl",
            "--profile",
            "release",
            "--build-std",
            "core,alloc",
            "--build-std-features",
            "panic_immediate_abort",
            "-vv",
        ])
        .unwrap();
        assert_eq!(args.targets.len(), 2);
        assert_eq!(args.build_std, Some("core,alloc".to_string()));
        assert_eq!(
            args.build_std_features,
            Some("panic_immediate_abort".to_string())
        );
        assert_eq!(args.verbose_level, 2);
    }

    #[test]
    fn test_real_world_test_with_passthrough() {
        let args = parse(&[
            "cargo-cross",
            "test",
            "-t",
            "x86_64-unknown-linux-musl",
            "--",
            "--nocapture",
            "--test-threads=1",
        ])
        .unwrap();
        assert_eq!(args.command, Command::Test);
        assert_eq!(args.targets, vec!["x86_64-unknown-linux-musl"]);
        assert_eq!(
            args.passthrough_args,
            vec!["--nocapture", "--test-threads=1"]
        );
    }

    #[test]
    fn test_real_world_with_compiler_options() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "-t",
            "aarch64-unknown-linux-musl",
            "--cc",
            "/opt/cross/bin/aarch64-linux-musl-gcc",
            "--cxx",
            "/opt/cross/bin/aarch64-linux-musl-g++",
            "--ar",
            "/opt/cross/bin/aarch64-linux-musl-ar",
            "--cflags",
            "-O2 -march=armv8-a",
        ])
        .unwrap();
        assert_eq!(args.targets, vec!["aarch64-unknown-linux-musl"]);
        assert!(args.cc.is_some());
        assert!(args.cxx.is_some());
        assert!(args.ar.is_some());
        assert_eq!(args.cflags, Some("-O2 -march=armv8-a".to_string()));
    }

    #[test]
    fn test_real_world_sccache_build() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "-t",
            "x86_64-unknown-linux-musl",
            "--enable-sccache",
            "--sccache-dir",
            "/tmp/sccache",
            "--sccache-cache-size",
            "10G",
        ])
        .unwrap();
        assert!(args.enable_sccache);
        assert_eq!(args.sccache_dir, Some(PathBuf::from("/tmp/sccache")));
        assert_eq!(args.sccache_cache_size, Some("10G".to_string()));
    }

    #[test]
    fn test_real_world_workspace_build() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "--workspace",
            "--exclude",
            "test-crate",
            "-t",
            "x86_64-unknown-linux-musl",
            "--profile",
            "release",
        ])
        .unwrap();
        assert!(args.workspace);
        assert_eq!(args.exclude, Some("test-crate".to_string()));
        assert_eq!(args.targets, vec!["x86_64-unknown-linux-musl"]);
    }

    // Edge case tests

    #[test]
    fn test_edge_case_equals_in_value() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "--config",
            "build.rustflags=['-C', 'opt-level=3']",
        ])
        .unwrap();
        assert_eq!(
            args.cargo_config,
            vec!["build.rustflags=['-C', 'opt-level=3']"]
        );
    }

    #[test]
    fn test_edge_case_empty_passthrough() {
        let args = parse(&["cargo-cross", "build", "--"]).unwrap();
        assert!(args.passthrough_args.is_empty());
    }

    #[test]
    fn test_edge_case_target_with_numbers() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "-t",
            "armv7-unknown-linux-musleabihf",
        ])
        .unwrap();
        assert_eq!(args.targets, vec!["armv7-unknown-linux-musleabihf"]);
    }

    #[test]
    fn test_edge_case_all_bool_options() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "--no-default-features",
            "--workspace",
            "--bins",
            "--lib",
            "--examples",
            "--tests",
            "--benches",
            "--all-targets",
            "--locked",
            "--offline",
            "--keep-going",
        ])
        .unwrap();
        assert!(args.no_default_features);
        assert!(args.workspace);
        assert!(args.build_bins);
        assert!(args.build_lib);
        assert!(args.build_examples);
        assert!(args.build_tests);
        assert!(args.build_benches);
        assert!(args.build_all_targets);
        assert!(args.locked);
        assert!(args.offline);
        assert!(args.keep_going);
    }

    #[test]
    fn test_edge_case_mixed_equals_and_space() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "-t=x86_64-unknown-linux-musl",
            "--profile",
            "release",
            "-F=serde",
            "--crt-static",
            "true",
        ])
        .unwrap();
        assert_eq!(args.targets, vec!["x86_64-unknown-linux-musl"]);
        assert_eq!(args.profile, "release");
        assert_eq!(args.features, Some("serde".to_string()));
        assert_eq!(args.crt_static, Some(true));
    }

    #[test]
    fn test_edge_case_directory_option() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "-C",
            "/path/to/project",
            "-t",
            "x86_64-unknown-linux-musl",
        ])
        .unwrap();
        assert_eq!(args.cargo_cwd, Some(PathBuf::from("/path/to/project")));
    }

    #[test]
    fn test_edge_case_manifest_path() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "--manifest-path",
            "/path/to/Cargo.toml",
        ])
        .unwrap();
        assert_eq!(
            args.manifest_path,
            Some(PathBuf::from("/path/to/Cargo.toml"))
        );
    }

    // Cargo cross invocation style tests

    #[test]
    fn test_cargo_cross_style_build() {
        let args: Vec<String> = vec![
            "cargo-cross".to_string(),
            "cross".to_string(),
            "build".to_string(),
            "-t".to_string(),
            "x86_64-unknown-linux-musl".to_string(),
        ];
        match parse_args_from(args).unwrap() {
            ParseResult::Build(args) => {
                assert_eq!(args.command, Command::Build);
                assert_eq!(args.targets, vec!["x86_64-unknown-linux-musl"]);
            }
            _ => panic!("expected Build"),
        }
    }

    #[test]
    fn test_cargo_cross_style_with_toolchain() {
        let args: Vec<String> = vec![
            "cargo-cross".to_string(),
            "cross".to_string(),
            "+nightly".to_string(),
            "build".to_string(),
        ];
        match parse_args_from(args).unwrap() {
            ParseResult::Build(args) => {
                assert_eq!(args.toolchain, Some("nightly".to_string()));
                assert_eq!(args.command, Command::Build);
            }
            _ => panic!("expected Build"),
        }
    }

    #[test]
    fn test_cargo_cross_style_targets() {
        let args: Vec<String> = vec![
            "cargo-cross".to_string(),
            "cross".to_string(),
            "targets".to_string(),
        ];
        match parse_args_from(args).unwrap() {
            ParseResult::ShowTargets(_) => {}
            _ => panic!("expected ShowTargets"),
        }
    }

    // New alias and option tests

    #[test]
    fn test_github_proxy_mirror_alias() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "--github-proxy-mirror",
            "https://mirror.example.com/",
        ])
        .unwrap();
        assert_eq!(
            args.github_proxy,
            Some("https://mirror.example.com/".to_string())
        );
    }

    #[test]
    fn test_github_proxy_original() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "--github-proxy",
            "https://proxy.example.com/",
        ])
        .unwrap();
        assert_eq!(
            args.github_proxy,
            Some("https://proxy.example.com/".to_string())
        );
    }

    #[test]
    fn test_release_flag_short() {
        let args = parse(&["cargo-cross", "build", "-r"]).unwrap();
        assert!(args.release);
        assert_eq!(args.profile, "release");
    }

    #[test]
    fn test_release_flag_long() {
        let args = parse(&["cargo-cross", "build", "--release"]).unwrap();
        assert!(args.release);
        assert_eq!(args.profile, "release");
    }

    #[test]
    fn test_toolchain_option() {
        let args = parse(&["cargo-cross", "build", "--toolchain", "nightly"]).unwrap();
        assert_eq!(args.toolchain, Some("nightly".to_string()));
    }

    #[test]
    fn test_toolchain_option_with_version() {
        let args = parse(&["cargo-cross", "build", "--toolchain", "1.75.0"]).unwrap();
        assert_eq!(args.toolchain, Some("1.75.0".to_string()));
    }

    #[test]
    fn test_toolchain_plus_syntax_takes_precedence() {
        let args = parse(&["cargo-cross", "+nightly", "build", "--toolchain", "stable"]).unwrap();
        // +nightly syntax takes precedence over --toolchain
        assert_eq!(args.toolchain, Some("nightly".to_string()));
    }

    #[test]
    fn test_target_dir_alias() {
        let args = parse(&["cargo-cross", "build", "--target-dir", "/tmp/target"]).unwrap();
        assert_eq!(args.cargo_target_dir, Some(PathBuf::from("/tmp/target")));
    }

    #[test]
    fn test_cargo_target_dir_original() {
        let args = parse(&["cargo-cross", "build", "--cargo-target-dir", "/tmp/target"]).unwrap();
        assert_eq!(args.cargo_target_dir, Some(PathBuf::from("/tmp/target")));
    }

    #[test]
    fn test_args_alias() {
        let args = parse(&["cargo-cross", "build", "--args", "--verbose"]).unwrap();
        assert_eq!(args.cargo_args, vec!["--verbose"]);
    }

    #[test]
    fn test_cargo_args_original() {
        let args = parse(&["cargo-cross", "build", "--cargo-args", "--verbose"]).unwrap();
        assert_eq!(args.cargo_args, vec!["--verbose"]);
    }

    #[test]
    fn test_cargo_args_multiple() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "--cargo-args",
            "--verbose",
            "--cargo-args",
            "--locked",
        ])
        .unwrap();
        assert_eq!(args.cargo_args, vec!["--verbose", "--locked"]);
    }

    // Target validation tests

    #[test]
    fn test_invalid_target_triple_uppercase() {
        let result = parse(&["cargo-cross", "build", "-t", "X86_64-unknown-linux-musl"]);
        assert!(result.is_err());
        let err = result.unwrap_err();
        assert!(
            matches!(err, CrossError::InvalidTargetTriple { target, char } if target == "X86_64-unknown-linux-musl" && char == 'X')
        );
    }

    #[test]
    fn test_glob_pattern_matches_target() {
        // x86_64*unknown-linux-musl matches x86_64-unknown-linux-musl (glob * matches -)
        let args = parse(&["cargo-cross", "build", "-t", "x86_64*unknown-linux-musl"]).unwrap();
        assert!(args
            .targets
            .contains(&"x86_64-unknown-linux-musl".to_string()));
    }

    #[test]
    fn test_invalid_target_triple_slash() {
        // Slash is not a glob character, so it should fail validation as invalid character
        let result = parse(&["cargo-cross", "build", "-t", "x86_64/unknown-linux-musl"]);
        assert!(result.is_err());
        let err = result.unwrap_err();
        assert!(
            matches!(err, CrossError::InvalidTargetTriple { target, char } if target == "x86_64/unknown-linux-musl" && char == '/')
        );
    }

    #[test]
    fn test_invalid_target_triple_dot() {
        let result = parse(&["cargo-cross", "build", "-t", "x86_64.unknown-linux-musl"]);
        assert!(result.is_err());
        let err = result.unwrap_err();
        assert!(
            matches!(err, CrossError::InvalidTargetTriple { target, char } if target == "x86_64.unknown-linux-musl" && char == '.')
        );
    }

    #[test]
    fn test_no_matching_targets_glob() {
        let result = parse(&["cargo-cross", "build", "-t", "*mingw*"]);
        assert!(result.is_err());
        let err = result.unwrap_err();
        assert!(matches!(
            err,
            CrossError::NoMatchingTargets { pattern } if pattern == "*mingw*"
        ));
    }

    #[test]
    fn test_no_matching_targets_glob_complex() {
        let result = parse(&["cargo-cross", "build", "-t", "*nonexistent-platform*"]);
        assert!(result.is_err());
        let err = result.unwrap_err();
        assert!(matches!(
            err,
            CrossError::NoMatchingTargets { pattern } if pattern == "*nonexistent-platform*"
        ));
    }

    #[test]
    fn test_valid_target_triple_with_numbers() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "-t",
            "armv7-unknown-linux-gnueabihf",
        ])
        .unwrap();
        assert_eq!(args.targets, vec!["armv7-unknown-linux-gnueabihf"]);
    }

    #[test]
    fn test_valid_target_triple_underscore() {
        let args = parse(&["cargo-cross", "build", "-t", "x86_64_unknown_linux_musl"]).unwrap();
        assert_eq!(args.targets, vec!["x86_64_unknown_linux_musl"]);
    }

    #[test]
    fn test_valid_glob_pattern_matches() {
        let args = parse(&["cargo-cross", "build", "-t", "*-linux-musl"]).unwrap();
        assert!(!args.targets.is_empty());
        for target in &args.targets {
            assert!(target.ends_with("-linux-musl"));
        }
    }

    #[test]
    fn test_is_glob_pattern() {
        assert!(is_glob_pattern("*-linux-musl"));
        assert!(is_glob_pattern("x86_64-*-linux"));
        assert!(is_glob_pattern("x86_64-?-linux"));
        assert!(is_glob_pattern("[ab]-linux"));
        assert!(!is_glob_pattern("x86_64-unknown-linux-musl"));
        assert!(!is_glob_pattern("aarch64-linux-android"));
    }

    #[test]
    fn test_validate_target_triple() {
        assert!(validate_target_triple("x86_64-unknown-linux-musl").is_ok());
        assert!(validate_target_triple("aarch64-unknown-linux-gnu").is_ok());
        assert!(validate_target_triple("armv7-unknown-linux-gnueabihf").is_ok());
        assert!(validate_target_triple("i686-pc-windows-msvc").is_ok());
        assert!(validate_target_triple("x86_64-pc-windows-msvc").is_ok());
        assert!(validate_target_triple("X86_64-unknown-linux-musl").is_err()); // uppercase X
        assert!(validate_target_triple("x86_64*linux").is_err()); // special char *
        assert!(validate_target_triple("x86_64.linux").is_err()); // special char .
        assert!(validate_target_triple("x86_64 linux").is_err()); // space
    }

    // Short argument concatenation tests (no separator between flag and value)

    #[test]
    fn test_short_concat_features() {
        let args = parse(&["cargo-cross", "build", "-Ffoo,bar"]).unwrap();
        assert_eq!(args.features, Some("foo,bar".to_string()));
    }

    #[test]
    fn test_short_concat_jobs() {
        let args = parse(&["cargo-cross", "build", "-j4"]).unwrap();
        assert_eq!(args.jobs, Some("4".to_string()));
    }

    #[test]
    fn test_short_concat_package() {
        let args = parse(&["cargo-cross", "build", "-pmypackage"]).unwrap();
        assert_eq!(args.package, Some("mypackage".to_string()));
    }

    #[test]
    fn test_short_concat_target() {
        let args = parse(&["cargo-cross", "build", "-tx86_64-unknown-linux-musl"]).unwrap();
        assert_eq!(args.targets, vec!["x86_64-unknown-linux-musl"]);
    }

    #[test]
    fn test_short_concat_z_flag() {
        let args = parse(&["cargo-cross", "build", "-Zbuild-std"]).unwrap();
        assert_eq!(args.cargo_z_flags, vec!["build-std"]);
    }

    #[test]
    fn test_short_concat_directory() {
        let args = parse(&["cargo-cross", "build", "-C/path/to/project"]).unwrap();
        assert_eq!(args.cargo_cwd, Some(PathBuf::from("/path/to/project")));
    }

    #[test]
    fn test_short_concat_multiple() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "-tx86_64-unknown-linux-musl",
            "-Ffoo,bar",
            "-j8",
            "-pmypkg",
        ])
        .unwrap();
        assert_eq!(args.targets, vec!["x86_64-unknown-linux-musl"]);
        assert_eq!(args.features, Some("foo,bar".to_string()));
        assert_eq!(args.jobs, Some("8".to_string()));
        assert_eq!(args.package, Some("mypkg".to_string()));
    }

    #[test]
    fn test_short_concat_mixed_with_space() {
        let args = parse(&[
            "cargo-cross",
            "build",
            "-j4",
            "-t",
            "x86_64-unknown-linux-musl",
            "-Fbar",
        ])
        .unwrap();
        assert_eq!(args.jobs, Some("4".to_string()));
        assert_eq!(args.targets, vec!["x86_64-unknown-linux-musl"]);
        assert_eq!(args.features, Some("bar".to_string()));
    }

    // Tests for parse_env_args (shell-style argument parsing from env vars)

    #[test]
    fn test_parse_env_args_not_set() {
        std::env::remove_var("TEST_PARSE_ENV_ARGS_NOT_SET");
        let result = parse_env_args("TEST_PARSE_ENV_ARGS_NOT_SET");
        assert!(result.is_none());
    }

    #[test]
    fn test_parse_env_args_empty() {
        std::env::set_var("TEST_PARSE_ENV_ARGS_EMPTY", "");
        let result = parse_env_args("TEST_PARSE_ENV_ARGS_EMPTY");
        assert!(result.is_none());
        std::env::remove_var("TEST_PARSE_ENV_ARGS_EMPTY");
    }

    #[test]
    fn test_parse_env_args_simple() {
        std::env::set_var("TEST_PARSE_ENV_ARGS_SIMPLE", "--verbose --locked");
        let result = parse_env_args("TEST_PARSE_ENV_ARGS_SIMPLE");
        assert_eq!(
            result,
            Some(vec!["--verbose".to_string(), "--locked".to_string()])
        );
        std::env::remove_var("TEST_PARSE_ENV_ARGS_SIMPLE");
    }

    #[test]
    fn test_parse_env_args_with_single_quotes() {
        std::env::set_var(
            "TEST_PARSE_ENV_ARGS_SINGLE_QUOTES",
            "--config 'build.jobs=4' --verbose",
        );
        let result = parse_env_args("TEST_PARSE_ENV_ARGS_SINGLE_QUOTES");
        assert_eq!(
            result,
            Some(vec![
                "--config".to_string(),
                "build.jobs=4".to_string(),
                "--verbose".to_string()
            ])
        );
        std::env::remove_var("TEST_PARSE_ENV_ARGS_SINGLE_QUOTES");
    }

    #[test]
    fn test_parse_env_args_with_double_quotes() {
        std::env::set_var(
            "TEST_PARSE_ENV_ARGS_DOUBLE_QUOTES",
            "--message-format \"json with spaces\"",
        );
        let result = parse_env_args("TEST_PARSE_ENV_ARGS_DOUBLE_QUOTES");
        assert_eq!(
            result,
            Some(vec![
                "--message-format".to_string(),
                "json with spaces".to_string()
            ])
        );
        std::env::remove_var("TEST_PARSE_ENV_ARGS_DOUBLE_QUOTES");
    }

    #[test]
    fn test_parse_env_args_complex() {
        std::env::set_var(
            "TEST_PARSE_ENV_ARGS_COMPLEX",
            "--config 'key=\"value with spaces\"' --verbose",
        );
        let result = parse_env_args("TEST_PARSE_ENV_ARGS_COMPLEX");
        assert_eq!(
            result,
            Some(vec![
                "--config".to_string(),
                "key=\"value with spaces\"".to_string(),
                "--verbose".to_string()
            ])
        );
        std::env::remove_var("TEST_PARSE_ENV_ARGS_COMPLEX");
    }

    #[test]
    fn test_parse_env_args_whitespace_only() {
        std::env::set_var("TEST_PARSE_ENV_ARGS_WHITESPACE", "   ");
        let result = parse_env_args("TEST_PARSE_ENV_ARGS_WHITESPACE");
        assert!(result.is_none());
        std::env::remove_var("TEST_PARSE_ENV_ARGS_WHITESPACE");
    }

    #[test]
    fn test_musl_target_with_default_glibc() {
        // musl targets should work with default (empty) glibc version
        let args = parse(&[
            "cargo-cross",
            "build",
            "-t",
            "aarch64_be-unknown-linux-musl",
        ])
        .unwrap();
        assert_eq!(args.targets, vec!["aarch64_be-unknown-linux-musl"]);
        assert_eq!(args.glibc_version, ""); // default is empty string
    }
}