From 27c6e7ea1eed1b855ac85b88e5b2533c9c3888e0 Mon Sep 17 00:00:00 2001 From: AmosOO7 Date: Sat, 29 Nov 2025 20:12:23 +0100 Subject: [PATCH 1/2] Add -p short flag for pretty command --- src/commands.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/commands.rs b/src/commands.rs index d3f2d98..1fccd56 100644 --- a/src/commands.rs +++ b/src/commands.rs @@ -51,7 +51,7 @@ pub struct CliOpts { #[arg(env = "DATADIR", short = 'd', long = "datadir")] pub datadir: Option, /// Output results in pretty format (instead of JSON). - #[arg(long = "pretty", global = true)] + #[arg(long = "pretty", short = 'p', global = true)] pub pretty: bool, /// Top level cli sub-commands. #[command(subcommand)] From e4e6b3b23beb615df8b5b1655520f35259ee8195 Mon Sep 17 00:00:00 2001 From: AmosOO7 Date: Sat, 29 Nov 2025 20:30:11 +0100 Subject: [PATCH 2/2] fix(cli): resolve short flag conflict for --pretty The -p short flag was conflicting with other options (proxy, password, path), causing CLI parsing errors and failing tests. Changed the short flag for --pretty to -P to avoid conflicts while keeping it intuitive. This restores test passes for all CLI flag positions and maintains backward compatibility. --- src/commands.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/commands.rs b/src/commands.rs index 1fccd56..dc0bfbc 100644 --- a/src/commands.rs +++ b/src/commands.rs @@ -51,7 +51,7 @@ pub struct CliOpts { #[arg(env = "DATADIR", short = 'd', long = "datadir")] pub datadir: Option, /// Output results in pretty format (instead of JSON). - #[arg(long = "pretty", short = 'p', global = true)] + #[arg(long = "pretty", short = 'P', global = true)] pub pretty: bool, /// Top level cli sub-commands. #[command(subcommand)]