Fix panic on startup in debug build

[ackintosh]

Issue Addressed

Lighthouse in debug build panics on startup with the following error:

$ cargo build --bin lighthouse
$ target/debug/lighthouse bn

thread 'main' panicked at /Users/akihito/.cargo/registry/src/index.crates.io-6f17d22bba15001f/clap_builder-4.5.2/src/builder/debug_asserts.rs:341:13:
Command Lighthouse: command `account_manager` alias `account_manager` is duplicated
stack backtrace:
   0: rust_begin_unwind
             at /rustc/9b00956e56009bab2aa15d7bff10916599e3d6d6/library/std/src/panicking.rs:645:5
   1: core::panicking::panic_fmt
             at /rustc/9b00956e56009bab2aa15d7bff10916599e3d6d6/library/core/src/panicking.rs:72:14
   2: clap_builder::builder::debug_asserts::assert_app
             at /Users/akihito/.cargo/registry/src/index.crates.io-6f17d22bba15001f/clap_builder-4.5.2/src/builder/debug_asserts.rs:341:13
   3: clap_builder::builder::command::Command::_build_self
             at /Users/akihito/.cargo/registry/src/index.crates.io-6f17d22bba15001f/clap_builder-4.5.2/src/builder/command.rs:4123:13
   4: clap_builder::builder::command::Command::_do_parse
             at /Users/akihito/.cargo/registry/src/index.crates.io-6f17d22bba15001f/clap_builder-4.5.2/src/builder/command.rs:3994:9
   5: clap_builder::builder::command::Command::try_get_matches_from_mut
             at /Users/akihito/.cargo/registry/src/index.crates.io-6f17d22bba15001f/clap_builder-4.5.2/src/builder/command.rs:830:9
   6: clap_builder::builder::command::Command::get_matches_from
             at /Users/akihito/.cargo/registry/src/index.crates.io-6f17d22bba15001f/clap_builder-4.5.2/src/builder/command.rs:701:9
   7: clap_builder::builder::command::Command::get_matches
             at /Users/akihito/.cargo/registry/src/index.crates.io-6f17d22bba15001f/clap_builder-4.5.2/src/builder/command.rs:610:9
   8: lighthouse::main
             at ./lighthouse/src/main.rs:79:19
   9: core::ops::function::FnOnce::call_once
             at /rustc/9b00956e56009bab2aa15d7bff10916599e3d6d6/library/core/src/ops/function.rs:250:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

thread 'main' panicked at /Users/akihito/.cargo/registry/src/index.crates.io-6f17d22bba15001f/clap_builder-4.5.2/src/builder/debug_asserts.rs:341:13:
Command Lighthouse: command `validator_manager` alias `validator_manager` is duplicated
stack backtrace:
   0: rust_begin_unwind
             at /rustc/9b00956e56009bab2aa15d7bff10916599e3d6d6/library/std/src/panicking.rs:645:5
   1: core::panicking::panic_fmt
             at /rustc/9b00956e56009bab2aa15d7bff10916599e3d6d6/library/core/src/panicking.rs:72:14
   2: clap_builder::builder::debug_asserts::assert_app
             at /Users/akihito/.cargo/registry/src/index.crates.io-6f17d22bba15001f/clap_builder-4.5.2/src/builder/debug_asserts.rs:341:13
   3: clap_builder::builder::command::Command::_build_self
             at /Users/akihito/.cargo/registry/src/index.crates.io-6f17d22bba15001f/clap_builder-4.5.2/src/builder/command.rs:4123:13
   4: clap_builder::builder::command::Command::_do_parse
             at /Users/akihito/.cargo/registry/src/index.crates.io-6f17d22bba15001f/clap_builder-4.5.2/src/builder/command.rs:3994:9
   5: clap_builder::builder::command::Command::try_get_matches_from_mut
             at /Users/akihito/.cargo/registry/src/index.crates.io-6f17d22bba15001f/clap_builder-4.5.2/src/builder/command.rs:830:9
   6: clap_builder::builder::command::Command::get_matches_from
             at /Users/akihito/.cargo/registry/src/index.crates.io-6f17d22bba15001f/clap_builder-4.5.2/src/builder/command.rs:701:9
   7: clap_builder::builder::command::Command::get_matches
             at /Users/akihito/.cargo/registry/src/index.crates.io-6f17d22bba15001f/clap_builder-4.5.2/src/builder/command.rs:610:9
   8: lighthouse::main
             at ./lighthouse/src/main.rs:79:19
   9: core::ops::function::FnOnce::call_once
             at /rustc/9b00956e56009bab2aa15d7bff10916599e3d6d6/library/core/src/ops/function.rs:250:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

Proposed Changes

It was because the alias was specified as the same as the subcommand name, so I removed it.

[ackintosh]

The CI failure in compile-with-beta-compiler will be fixed by #5916

[jimmygchen]

LGTM!
I also ran into the same issue and made the changes locally but never remembered to commit 💀 . Thanks!

[jimmygchen]

@ackintosh looks like CI needs some fixes - I think the beta compiler failure may not be related to your changes but a new Rust beta version.

The CLI help needs a fix though, if you run ``make cli-locallocally and commit the changes it should fix thecli-check` job.

[jimmygchen]

Beta compiler fix is in #5916

[ackintosh]

Thanks @jimmygchen !

[michaelsproul]

@Mergifyio queue

[mergify]

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at 5789db0