refactor(cli): Unify CLI parameters and refactor CLI definition

[dknopik]

Issue Addressed

• Unify CLI parameters #266

Proposed Changes

• Refactor logging crate
• Introduce global config crate that contains shared options
• Pass global config to subcommands to avoid duplicated logic about datadir and ssv network
• Set default network to Hoodi
• Adjust docs

Additional Info

This is not as scary to review as the lines changed seem to indicate, a lot comes from moving the logging crate around.

[Zacholme7]

Passing thought, do we want to allow config via env vars?

[dknopik]

@Zacholme7 good idea!

At least for the secondary commands it might be interesting to support e.g. ETH_RPC_URL to mimic Foundry's cast send

[dknopik]

Any open questions/blockers on this?

[Zacholme7]

small nits, lgtm otherwise

[mergify]

Some required checks have failed. Could you please take a look @dknopik? 🙏

[dknopik]

Simplified this for easier review and a smaller changeset. Logging refactoring will be reintroduced in an upcoming PR

[Copilot]

Pull Request Overview

Refactors the CLI by introducing a shared GlobalConfig crate, unifying common flags across subcommands, and cleaning up logging initialization.

• Introduce common/global_config crate for shared CLI options (--data-dir, --network, etc.)
• Refactor enable_logging to accept global flags and consolidate file logging layers
• Update CLI documentation to show unified global options and adjust commands accordingly

Reviewed Changes

Copilot reviewed 19 out of 20 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
book/src/cli.md	Consolidated global flags table and updated examples
anchor/src/main.rs	Use GlobalConfig for data dir/network and refactored logging setup
anchor/logging/src/logging.rs	Rewrote FileLoggingFlags, removed legacy config, updated init_file_logging
anchor/common/global_config/src/lib.rs	Added GlobalFlags and TryFrom for GlobalConfig
anchor/common/global_config/Cargo.toml	Declared new global_config crate in workspace
anchor/keysplit/src/cli.rs	Removed per-command --network flag, rely on global flag
anchor/keygen/src/lib.rs	Updated run_keygen to write to global data directory

Comments suppressed due to low confidence (4)

book/src/cli.md:25

• The table still lists 'Mainnet' as a supported network, but GlobalFlags only allows holesky or hoodi. Update this row to reflect the actual supported networks.

| `--network <NETWORK>` | Network to use (Mainnet, Holesky, Hoodi) | `hoodi`               |

book/src/cli.md:250

• [nitpick] The example places --network under the keysplit onchain subcommand, but --network is now a global flag. Consider moving it before the subcommand or noting that it can be used anywhere.

  --network mainnet

anchor/common/global_config/src/lib.rs:1

• The new GlobalConfig parsing logic lacks unit tests. Consider adding tests for GlobalConfig::try_from to validate default data dir, network parsing, and error cases.

use std::{path::PathBuf, str::FromStr};

anchor/common/global_config/Cargo.toml:11

• The strum dependency is declared but not used in global_config/src/lib.rs. You can remove it to keep dependencies minimal.

strum = { workspace = true }

[diegomrsantos]

LGTM, only small nits. Please take a look at the last copilot comments.

[diegomrsantos]

Thanks!