⚗️ Add bsdtar-compatible --block-size option to stdio command (#2566)

* feat(cli): Add bsdtar-compatible `-b` and `--block-size` options to stdio

This commit introduces the `-b` and `--block-size` command-line options
to the `pna experimental stdio` command for improved compatibility with
bsdtar.

These options are for interface compatibility only and are no-ops. They
do not affect the runtime behavior of the command. When used, a warning
is logged to inform the user that the option is ignored.

- Added `block_size: Option<String>` to the `StdioCommand` struct.
- Implemented a warning in `run_stdio` when the option is present.
- Added integration tests to verify that both `-b` and `--block-size` are
  accepted and trigger the warning.

* feat(cli): Add bsdtar-compatible `-b` and `--block-size` options to stdio

This commit introduces the `-b` and `--block-size` command-line options
to the `pna experimental stdio` command for improved compatibility with
bsdtar.

These options are for interface compatibility only and are no-ops. They
do not affect the runtime behavior of the command. When used, a warning
is logged to inform the user that the option is ignored.

- Added `block_size: Option<usize>` to the `StdioCommand` struct.
- Implemented a warning in `run_stdio` when the option is present.
- Added integration tests to verify that both `-b` and `--block-size` are
  accepted and trigger the warning.
- Hid the option from the help output as it's for compatibility only.

---------

Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>

Author: google-labs-jules[bot]

cli/src/command/stdio.rs

@@ -487,6 +487,13 @@ pub(crate) struct StdioCommand {
     no_safe_writes: bool,
     #[arg(short = 'a', long = "auto-compress", hide = true)]
     auto_compress: bool,
+    #[arg(
+        short = 'b',
+        long = "block-size",
+        value_name = "blocksize",
+        hide = true
+    )]
+    block_size: Option<usize>,
     #[arg(long, action = clap::ArgAction::Version, help = "Print version")]
     version: (),
     #[arg(long, action = clap::ArgAction::Help, help = "Print help")]
@@ -525,6 +532,11 @@ fn run_stdio(args: StdioCommand) -> anyhow::Result<()> {
     if args.auto_compress {
         log::warn!("Option '--auto-compress' is accepted for compatibility but will be ignored.");
     }
+    if let Some(block_size) = &args.block_size {
+        log::warn!(
+            "Option '--block-size {block_size}' is accepted for compatibility but will be ignored."
+        );
+    }
     if args.create {
         run_create_archive(args)
     } else if args.extract {

cli/tests/cli/stdio.rs

@@ -1,6 +1,7 @@
 mod exclude_vcs;
 mod files_from;
 mod option_auto_compress;
+mod option_block_size;
 mod option_check_links;
 mod option_no_recursive;
 mod option_update;

cli/tests/cli/stdio/option_block_size.rs

@@ -0,0 +1,45 @@
+#![cfg(not(target_family = "wasm"))]
+use crate::utils::setup;
+use assert_cmd::cargo::cargo_bin_cmd;
+use predicates::prelude::predicate;
+use std::fs;
+
+#[test]
+fn stdio_block_size_option() {
+    setup();
+    let file = "stdio_block_size_option.txt";
+    fs::write(file, "").unwrap();
+
+    let mut cmd = cargo_bin_cmd!("pna");
+    cmd.arg("experimental")
+        .arg("stdio")
+        .arg("-c")
+        .arg("--block-size")
+        .arg("20")
+        .arg(file)
+        .assert()
+        .success()
+        .stderr(predicate::str::contains(
+            "Option '--block-size 20' is accepted for compatibility but will be ignored.",
+        ));
+}
+
+#[test]
+fn stdio_b_option() {
+    setup();
+    let file = "stdio_b_option.txt";
+    fs::write(file, "").unwrap();
+
+    let mut cmd = cargo_bin_cmd!("pna");
+    cmd.arg("experimental")
+        .arg("stdio")
+        .arg("-c")
+        .arg("-b")
+        .arg("20")
+        .arg(file)
+        .assert()
+        .success()
+        .stderr(predicate::str::contains(
+            "Option '--block-size 20' is accepted for compatibility but will be ignored.",
+        ));
+}