git_paw/

cli.rs

//! CLI argument parsing.
//!
//! Defines the command-line interface using `clap` v4 with derive macros.
//! All subcommands, flags, and options are declared here.

use clap::{Parser, Subcommand, ValueEnum};

/// Spec format selector for the `--specs-format` flag.
///
/// Three formats are supported:
/// - `openspec` — `openspec/changes/<name>/` directory layout.
/// - `markdown` — single-file Markdown specs with YAML frontmatter.
/// - `speckit` — GitHub Spec Kit `.specify/specs/<feature>/` layout.
#[derive(Debug, Clone, Copy, PartialEq, Eq, ValueEnum)]
#[clap(rename_all = "lowercase")]
pub enum SpecsFormat {
    /// `OpenSpec` format (directory of `<change>/tasks.md`).
    Openspec,
    /// Markdown format (one `.md` file per spec with frontmatter).
    Markdown,
    /// Spec Kit format (`.specify/specs/<feature>/`).
    Speckit,
}

impl SpecsFormat {
    /// Returns the backend-dispatch string for this format.
    #[must_use]
    pub fn as_str(self) -> &'static str {
        match self {
            Self::Openspec => "openspec",
            Self::Markdown => "markdown",
            Self::Speckit => "speckit",
        }
    }
}

/// Parallel AI Worktrees — orchestrate multiple AI coding CLI sessions
/// across git worktrees from a single terminal using tmux.
#[derive(Debug, Parser)]
#[command(
    name = "git-paw",
    version,
    about = "Parallel AI Worktrees — orchestrate multiple AI coding CLI sessions across git worktrees",
    long_about = "git-paw orchestrates multiple AI coding CLI sessions (Claude, Codex, Gemini, etc.) \
                  across git worktrees from a single terminal using tmux. Each branch gets its own \
                  worktree and AI session, running in parallel.",
    after_help = "\x1b[1mQuick Start:\x1b[0m\n\n  \
                  # Launch interactive session (picks CLI and branches)\n  \
                  git paw\n\n  \
                  # Use Claude on specific branches\n  \
                  git paw start --cli claude --branches feat/auth,feat/api\n\n  \
                  # Check session status\n  \
                  git paw status\n\n  \
                  # Pause session (detaches client, stops broker, keeps CLIs alive)\n  \
                  git paw pause\n\n  \
                  # Stop session (kills CLIs, preserves worktrees for later)\n  \
                  git paw stop\n\n  \
                  # Remove everything\n  \
                  git paw purge"
)]
pub struct Cli {
    /// Subcommand to run. Defaults to `start` if omitted.
    #[command(subcommand)]
    pub command: Option<Command>,
}

/// Available subcommands.
#[derive(Debug, Subcommand)]
pub enum Command {
    /// Launch a new session or reattach to an existing one
    #[command(
        about = "Launch a new session or reattach to an existing one",
        long_about = "Smart start: reattaches if a session is active, recovers if stopped/crashed, \
                      or launches a new interactive session.\n\n\
                      By default, every existing agent branch is rebased onto the repository's \
                      default branch (whatever `origin/HEAD` tracks — typically `main`) before \
                      its worktree is opened, so agents always start from current main. Pass \
                      `--no-rebase` to skip this step and reproduce the pre-v0.6 behaviour \
                      (useful when you have local pinned SHAs or are deliberately working off a \
                      stale baseline). If the rebase hits a conflict, the affected branch is \
                      left at its pre-rebase HEAD and `git paw start` exits with an error \
                      listing the conflicting files.\n\n\
                      Examples:\n  \
                      git paw start\n  \
                      git paw start --cli claude\n  \
                      git paw start --cli claude --branches feat/auth,feat/api\n  \
                      git paw start --from-all-specs\n  \
                      git paw start --from-all-specs --cli claude\n  \
                      git paw start --specs add-auth,fix-session\n  \
                      git paw start --specs   # opens spec picker (TTY required)\n  \
                      git paw start --dry-run\n  \
                      git paw start --preset backend\n  \
                      git paw start --supervisor   # auto-approve safe prompts via [supervisor.auto_approve]\n  \
                      git paw start --no-supervisor  # disable supervisor for this session (overrides config)\n  \
                      git paw start --no-rebase   # skip rebasing agent branches onto the default branch"
    )]
    Start {
        /// AI CLI to use (e.g., claude, codex, gemini). Skips CLI picker if provided.
        #[arg(long, help = "AI CLI to use (skips CLI picker)")]
        cli: Option<String>,

        /// Comma-separated branch names. Skips branch picker if provided.
        #[arg(
            long,
            value_delimiter = ',',
            help = "Comma-separated branches (skips branch picker)"
        )]
        branches: Option<Vec<String>>,

        /// Launch worktrees for every discovered spec across all configured formats.
        #[arg(
            long,
            alias = "from-specs",
            help = "Launch from every discovered spec across all configured formats"
        )]
        from_all_specs: bool,

        /// Narrow the session to named specs, or open the multi-select picker
        /// when given without values.
        ///
        /// `--specs add-auth,fix-session` runs only those specs. Bare `--specs`
        /// opens a multi-select picker; an interactive terminal is required
        /// (otherwise the command exits with an actionable error pointing at
        /// `--specs NAME[,NAME...]` and `--from-all-specs`).
        ///
        /// Mutually exclusive with `--from-all-specs`.
        #[arg(
            long,
            value_delimiter = ',',
            num_args = 0..,
            conflicts_with = "from_all_specs",
            help = "Comma-separated spec names; bare flag opens picker (TTY required)"
        )]
        specs: Option<Vec<String>>,

        /// Override the spec format used for `--from-all-specs` / `--specs` scanning.
        ///
        /// Accepted values: `openspec`, `markdown`, `speckit`. Overrides both
        /// the `[specs] type` setting in `.git-paw/config.toml` and the
        /// auto-detection of `.specify/` at the repo root.
        #[arg(
            long,
            value_enum,
            help = "Override spec format (openspec, markdown, speckit)"
        )]
        specs_format: Option<SpecsFormat>,

        /// Preview the session plan without executing.
        #[arg(long, help = "Preview the session plan without executing")]
        dry_run: bool,

        /// Use a named preset from config.
        #[arg(long, help = "Use a named preset from config")]
        preset: Option<String>,

        /// Enable supervisor mode for this session.
        #[arg(
            long,
            default_value_t = false,
            help = "Enable supervisor mode for this session"
        )]
        supervisor: bool,

        /// Disable supervisor mode for this session, overriding any config setting.
        #[arg(
            long,
            conflicts_with = "supervisor",
            default_value_t = false,
            help = "Disable supervisor for this session, overriding any [supervisor] enabled = true in config"
        )]
        no_supervisor: bool,

        /// Bypass uncommitted-spec validation warning.
        #[arg(long, help = "Bypass uncommitted-spec validation warning")]
        force: bool,

        /// Skip rebasing existing agent branches onto the default branch
        /// before opening their worktrees.
        ///
        /// By default, `git paw start` rebases every existing agent branch
        /// onto the repository's default branch (whatever `origin/HEAD`
        /// tracks, typically `main`) before opening or reopening its
        /// worktree, so agents always start from current `main`. Pass
        /// `--no-rebase` to skip the rebase step entirely and reproduce the
        /// pre-v0.6 behaviour. Newly created branches (no prior commits) are
        /// not rebased regardless of this flag.
        #[arg(
            long,
            default_value_t = false,
            help = "Skip rebasing existing agent branches onto the default branch before opening worktrees"
        )]
        no_rebase: bool,
    },

    /// Attach a new worktree + agent pane to a running session
    #[command(
        about = "Attach a new worktree + agent pane to a running session",
        long_about = "Hot-attaches a worktree and agent pane to an already-running session — \
                      no stop/purge/restart, the other agents keep working undisturbed. The \
                      agent grid re-tiles to the layout a start of that many agents would \
                      produce, the new branch is registered in the session, and the agent boots \
                      with the same broker boot block + initial prompt a start-time agent gets.\n\n\
                      Provide a branch name, or use --from-spec to derive the branch (and CLI) \
                      from a discovered spec. Adding past the 25-agent cap is rejected. When the \
                      session is paused, the new pane starts paused too and begins on the next \
                      `git paw resume`. The supervisor (if any) discovers the new agent on its \
                      next broker poll — no restart.\n\n\
                      Examples:\n  \
                      git paw add feat/new-thing\n  \
                      git paw add feat/api --cli codex\n  \
                      git paw add --from-spec add-export"
    )]
    Add {
        /// Branch to attach. Omit when using --from-spec (the branch is
        /// derived from the spec).
        #[arg(
            required_unless_present = "from_spec",
            help = "Branch to attach (omit when using --from-spec)"
        )]
        branch: Option<String>,

        /// AI CLI to launch in the new pane (defaults to the session's CLI).
        #[arg(
            long,
            help = "AI CLI for the new pane (defaults to the session's default CLI)"
        )]
        cli: Option<String>,

        /// Resolve the branch name and CLI from a discovered spec instead of a
        /// positional branch argument.
        #[arg(
            long,
            conflicts_with = "branch",
            help = "Derive branch + CLI from a spec (OpenSpec change, Markdown spec, or Spec Kit feature)"
        )]
        from_spec: Option<String>,
    },

    /// Detach a single agent from a running session
    #[command(
        about = "Detach a single agent from a running session",
        long_about = "Removes one agent from an active session: closes its tmux pane, re-tiles \
                      the grid for the smaller agent count, removes its worktree (reusing \
                      `git paw purge`'s per-worktree teardown), and drops it from the session. \
                      The other agents are left untouched.\n\n\
                      Safe by default: `remove` refuses to delete a worktree with uncommitted \
                      changes (it lists what would be lost) unless you pass --force. Pass \
                      --keep-worktree to detach the pane + session entry but leave the worktree \
                      and branch on disk (this skips the uncommitted-work check, since nothing \
                      is deleted). `remove supervisor` is refused — use `git paw stop` to end \
                      the whole session. The supervisor notices the departure on its next broker \
                      poll (the agent stops heartbeating) — no restart.\n\n\
                      Examples:\n  \
                      git paw remove feat/done-thing\n  \
                      git paw remove feat/wip --force\n  \
                      git paw remove feat/keep --keep-worktree"
    )]
    Remove {
        /// Branch of the agent to remove.
        #[arg(help = "Branch of the agent to remove")]
        branch: String,

        /// Detach the pane + session entry but leave the worktree and branch
        /// on disk (skips the uncommitted-work safety check).
        #[arg(
            long,
            help = "Leave the worktree + branch on disk; only detach the pane and session entry"
        )]
        keep_worktree: bool,

        /// Remove the worktree even when it has uncommitted changes.
        #[arg(
            long,
            help = "Remove even with uncommitted changes (bypass the safety check)"
        )]
        force: bool,
    },

    /// Pause the session (detaches client, stops broker, leaves CLIs running)
    #[command(
        about = "Pause the session (detaches client, stops broker, leaves CLIs running)",
        long_about = "Detaches the tmux client and stops the broker, but leaves all CLI \
                      processes running in the background. This preserves agent conversation \
                      state for instant resume via `git paw start`. RAM stays allocated \
                      (~300 MB per Claude pane).\n\n\
                      Use pause for short breaks (lunch, meetings, end-of-day). For longer \
                      breaks, use `git paw stop` to kill the CLIs and release RAM (worktrees \
                      preserved). A future `git paw hibernate` (v1.0.0) will snapshot state \
                      to disk.\n\n\
                      Example:\n  git paw pause"
    )]
    Pause,

    /// Stop the session (kills tmux, keeps worktrees and state)
    #[command(
        about = "Stop the session (kills tmux, keeps worktrees and state)",
        long_about = "Kills the tmux session and every CLI pane process, but preserves \
                      worktrees and session state on disk. CLI conversation context is lost. \
                      Run `git paw start` later to recover the session with fresh CLI \
                      processes.\n\n\
                      Three teardown verbs:\n  \
                      pause — soft stop (detach + broker stop; CLIs keep running, RAM held)\n  \
                      stop  — kills CLI processes; preserves worktrees on disk (this command)\n  \
                      purge — full reset; removes worktrees, branches, and state\n\n\
                      `stop` prompts for confirmation in interactive terminals. Use \
                      `--force` to skip the prompt (scripts) or pipe stdin from \
                      `/dev/null` for non-interactive contexts.\n\n\
                      Examples:\n  git paw stop\n  git paw stop --force"
    )]
    Stop {
        /// Skip confirmation prompt.
        #[arg(long, default_value_t = false, help = "Skip confirmation prompt")]
        force: bool,
    },

    /// Remove everything (tmux session, worktrees, and state)
    #[command(
        about = "Remove everything (tmux session, worktrees, and state)",
        long_about = "Nuclear option: kills the tmux session, removes all worktrees, and deletes \
                      session state. Requires confirmation unless --force is used.\n\n\
                      Use --stale to purge only sessions whose tmux session is gone (a stale \
                      receipt). Live sessions are left untouched, so --stale is safe in cleanup \
                      scripts. Pairing --stale with --force is a no-op (--force is redundant on \
                      a stale entry).\n\n\
                      Examples:\n  git paw purge\n  git paw purge --force\n  git paw purge --stale"
    )]
    Purge {
        /// Skip confirmation prompt.
        #[arg(long, help = "Skip confirmation prompt")]
        force: bool,
        /// Purge only stale sessions (receipt claims active but tmux is gone).
        #[arg(
            long,
            help = "Purge only stale sessions (receipt claims active but tmux is gone); \
                    live sessions untouched"
        )]
        stale: bool,
    },

    /// Show session state for the current repo
    #[command(
        about = "Show session state for the current repo",
        long_about = "Displays the current session status, branches, CLIs, and worktree paths \
                      for the repository in the current directory.\n\n\
                      Status is one of 🟢 active (tmux running), 🔵 paused, 🟡 stopped, or \
                      🔴 stale (the receipt claims active but the tmux session no longer \
                      exists — a crash or release-boundary carry-over). Run `git paw start` to \
                      self-heal a stale receipt, or `git paw purge --stale` to clear it.\n\n\
                      Pass --json for machine-readable output (the `status` field is one of \
                      active/paused/stopped/stale).\n\n\
                      Examples:\n  git paw status\n  git paw status --json"
    )]
    Status {
        /// Emit machine-readable JSON instead of the human-readable display.
        #[arg(long, help = "Emit machine-readable JSON")]
        json: bool,
    },

    /// List detected and custom AI CLIs
    #[command(
        about = "List detected and custom AI CLIs",
        long_about = "Shows all AI CLIs found on PATH (auto-detected) and any custom CLIs \
                      registered in your config.\n\n\
                      Example:\n  git paw list-clis"
    )]
    ListClis,

    /// Register a custom AI CLI
    #[command(
        about = "Register a custom AI CLI",
        long_about = "Adds a custom CLI to your global config (~/.config/git-paw/config.toml). \
                      The command can be an absolute path or a binary name on PATH.\n\n\
                      Examples:\n  \
                      git paw add-cli my-agent /usr/local/bin/my-agent\n  \
                      git paw add-cli my-agent my-agent --display-name \"My Agent\""
    )]
    AddCli {
        /// Name to register the CLI as.
        #[arg(help = "Name to register the CLI as")]
        name: String,

        /// Command or path to the CLI binary.
        #[arg(help = "Command or path to the CLI binary")]
        command: String,

        /// Optional display name for the CLI.
        #[arg(long, help = "Display name shown in prompts")]
        display_name: Option<String>,
    },

    /// Unregister a custom AI CLI
    #[command(
        about = "Unregister a custom AI CLI",
        long_about = "Removes a custom CLI from your global config. Only custom CLIs can be \
                      removed — auto-detected CLIs cannot.\n\n\
                      Example:\n  git paw remove-cli my-agent"
    )]
    RemoveCli {
        /// Name of the custom CLI to remove.
        #[arg(help = "Name of the custom CLI to remove")]
        name: String,
    },

    /// Initialize .git-paw/ directory and configuration
    #[command(
        about = "Initialize .git-paw/ directory and configuration",
        long_about = "Creates the .git-paw/ directory with a default config and sets up \
                      .gitignore for logs.\n\n\
                      Examples:\n  git paw init"
    )]
    Init,

    /// Internal: run the broker and dashboard in pane 0
    #[command(
        hide = true,
        name = "__dashboard",
        about = "Internal: run the broker and dashboard in pane 0",
        long_about = "Internal subcommand used by git-paw to run the broker and dashboard TUI \
                      in pane 0 of a tmux session. Not intended for direct invocation."
    )]
    Dashboard,

    /// View captured session logs
    #[command(
        about = "View captured session logs",
        long_about = "Reads session logs captured by pipe-pane. By default, strips ANSI codes \
                      for clean output. Use --color to view with colors via less -R.\n\n\
                      Examples:\n  \
                      git paw replay --list\n  \
                      git paw replay feat/add-auth\n  \
                      git paw replay feat/add-auth --color\n  \
                      git paw replay feat/add-auth --session paw-myproject"
    )]
    Replay {
        /// Branch name to replay (fuzzy-matched against log filenames).
        #[arg(required_unless_present = "list", help = "Branch to replay")]
        branch: Option<String>,

        /// List available log sessions and branches.
        #[arg(long, help = "List available log sessions and branches")]
        list: bool,

        /// Display with ANSI colors via less -R.
        #[arg(long, help = "Display with colors via less -R")]
        color: bool,

        /// Session name to replay from (defaults to most recent).
        #[arg(long, help = "Session to replay from (defaults to most recent)")]
        session: Option<String>,
    },

    /// Report manually-approved command patterns for a session
    #[command(
        about = "Report manually-approved command patterns for a session",
        long_about = "Lists the command patterns you manually approved during a session — the \
                      prompts the auto-approve preset did NOT match — sorted by how often each \
                      was approved. Each row carries a SUGGEST hint for where the pattern might \
                      be promoted: the project-local allowlist (project-specific scripts/paths) \
                      or the bundled dev-allowlist preset (general commands like `make <target>`). \
                      The suggestion is a hint, not a rule.\n\n\
                      Reads `.git-paw/sessions/<session>.manual-approvals.jsonl`. Defaults to the \
                      active session; pass --session to target another. Recording is controlled by \
                      `[supervisor] manual_approvals_log` (default on).\n\n\
                      Examples:\n  \
                      git paw approvals\n  \
                      git paw approvals --json\n  \
                      git paw approvals --session paw-myproject\n  \
                      git paw approvals --limit 5"
    )]
    Approvals {
        /// Session to read approvals from (defaults to the active session).
        #[arg(long, help = "Session to read from (defaults to the active session)")]
        session: Option<String>,

        /// Cap the output to the top N patterns by count.
        #[arg(long, help = "Show at most N patterns (top N by count)")]
        limit: Option<usize>,

        /// Emit machine-readable JSON instead of the text table.
        #[arg(long, help = "Emit machine-readable JSON")]
        json: bool,
    },
}

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

    /// Helper: parse args as if running `git-paw <args>`.
    fn parse(args: &[&str]) -> Cli {
        let mut full = vec!["git-paw"];
        full.extend(args);
        Cli::try_parse_from(full).expect("failed to parse")
    }

    // -- Default subcommand --

    #[test]
    fn no_args_defaults_to_none_command() {
        let cli = parse(&[]);
        assert!(
            cli.command.is_none(),
            "no args should yield None (handled as Start in main)"
        );
    }

    // -- Start subcommand --

    #[test]
    fn start_with_no_flags() {
        let cli = parse(&["start"]);
        match cli.command.unwrap() {
            Command::Start {
                cli,
                branches,
                from_all_specs,
                specs,
                specs_format,
                dry_run,
                preset,
                supervisor,
                no_supervisor,
                force,
                no_rebase,
            } => {
                assert!(cli.is_none());
                assert!(branches.is_none());
                assert!(!from_all_specs);
                assert!(specs.is_none());
                assert!(specs_format.is_none());
                assert!(!dry_run);
                assert!(preset.is_none());
                assert!(!supervisor);
                assert!(!no_supervisor);
                assert!(!force);
                assert!(!no_rebase);
            }
            other => panic!("expected Start, got {other:?}"),
        }
    }

    #[test]
    fn start_with_cli_flag() {
        let cli = parse(&["start", "--cli", "claude"]);
        match cli.command.unwrap() {
            Command::Start { cli, .. } => assert_eq!(cli.as_deref(), Some("claude")),
            other => panic!("expected Start, got {other:?}"),
        }
    }

    #[test]
    fn start_with_from_all_specs_sets_flag_and_leaves_specs_unset() {
        let cli = parse(&["start", "--from-all-specs"]);
        match cli.command.unwrap() {
            Command::Start {
                from_all_specs,
                specs,
                ..
            } => {
                assert!(from_all_specs);
                assert!(specs.is_none());
            }
            other => panic!("expected Start, got {other:?}"),
        }
    }

    #[test]
    fn start_with_from_specs_alias_parses_identically_to_from_all_specs() {
        let alias_args = parse(&["start", "--from-specs"]);
        let canonical_args = parse(&["start", "--from-all-specs"]);
        match (alias_args.command.unwrap(), canonical_args.command.unwrap()) {
            (
                Command::Start {
                    from_all_specs: a_all,
                    specs: a_specs,
                    supervisor: a_sup,
                    ..
                },
                Command::Start {
                    from_all_specs: c_all,
                    specs: c_specs,
                    supervisor: c_sup,
                    ..
                },
            ) => {
                assert_eq!(a_all, c_all);
                assert_eq!(a_specs, c_specs);
                assert_eq!(a_sup, c_sup);
                assert!(a_all);
            }
            other => panic!("expected two Start variants, got {other:?}"),
        }
    }

    #[test]
    fn start_with_bare_specs_yields_empty_vec_picker_mode() {
        let cli = parse(&["start", "--specs"]);
        match cli.command.unwrap() {
            Command::Start {
                from_all_specs,
                specs,
                ..
            } => {
                assert!(!from_all_specs);
                assert_eq!(specs, Some(Vec::<String>::new()));
            }
            other => panic!("expected Start, got {other:?}"),
        }
    }

    #[test]
    fn start_with_specs_single_name() {
        let cli = parse(&["start", "--specs", "add-auth"]);
        match cli.command.unwrap() {
            Command::Start { specs, .. } => {
                assert_eq!(specs, Some(vec!["add-auth".to_string()]));
            }
            other => panic!("expected Start, got {other:?}"),
        }
    }

    #[test]
    fn start_with_specs_two_comma_separated_names() {
        let cli = parse(&["start", "--specs", "add-auth,fix-session"]);
        match cli.command.unwrap() {
            Command::Start { specs, .. } => {
                assert_eq!(
                    specs,
                    Some(vec!["add-auth".to_string(), "fix-session".to_string()])
                );
            }
            other => panic!("expected Start, got {other:?}"),
        }
    }

    #[test]
    fn start_with_specs_three_comma_separated_names() {
        let cli = parse(&["start", "--specs", "add-auth,fix-session,add-logging"]);
        match cli.command.unwrap() {
            Command::Start { specs, .. } => {
                assert_eq!(
                    specs,
                    Some(vec![
                        "add-auth".to_string(),
                        "fix-session".to_string(),
                        "add-logging".to_string(),
                    ])
                );
            }
            other => panic!("expected Start, got {other:?}"),
        }
    }

    #[test]
    fn start_with_from_all_specs_and_specs_is_rejected() {
        let result = Cli::try_parse_from([
            "git-paw",
            "start",
            "--from-all-specs",
            "--specs",
            "add-auth",
        ]);
        assert!(result.is_err());
        let err = result.unwrap_err().to_string();
        assert!(err.contains("--from-all-specs"), "got: {err}");
        assert!(err.contains("--specs"), "got: {err}");
    }

    #[test]
    fn start_with_from_specs_alias_and_specs_is_rejected() {
        let result =
            Cli::try_parse_from(["git-paw", "start", "--from-specs", "--specs", "add-auth"]);
        assert!(result.is_err());
    }

    #[test]
    fn start_with_from_all_specs_and_supervisor_sets_both_flags() {
        let cli = parse(&["start", "--from-all-specs", "--supervisor"]);
        match cli.command.unwrap() {
            Command::Start {
                from_all_specs,
                specs,
                supervisor,
                ..
            } => {
                assert!(from_all_specs);
                assert!(supervisor);
                assert!(specs.is_none());
            }
            other => panic!("expected Start, got {other:?}"),
        }
    }

    #[test]
    fn start_with_supervisor_only_leaves_spec_mode_unset() {
        let cli = parse(&["start", "--supervisor"]);
        match cli.command.unwrap() {
            Command::Start {
                from_all_specs,
                specs,
                supervisor,
                ..
            } => {
                assert!(!from_all_specs);
                assert!(specs.is_none());
                assert!(supervisor);
            }
            other => panic!("expected Start, got {other:?}"),
        }
    }

    #[test]
    fn start_help_contains_from_all_specs_and_specs_but_not_alias() {
        let result = Cli::try_parse_from(["git-paw", "start", "--help"]);
        let err = result.unwrap_err();
        assert_eq!(err.kind(), clap::error::ErrorKind::DisplayHelp);
        let help = err.to_string();
        assert!(
            help.contains("--from-all-specs"),
            "start --help should contain --from-all-specs; got: {help}"
        );
        assert!(
            help.contains("--specs"),
            "start --help should contain --specs; got: {help}"
        );
        assert!(
            !help.contains("--from-specs"),
            "start --help should NOT contain hidden alias --from-specs; got: {help}"
        );
    }

    #[test]
    fn start_with_branches_flag_comma_separated() {
        let cli = parse(&["start", "--branches", "feat/a,feat/b,fix/c"]);
        match cli.command.unwrap() {
            Command::Start { branches, .. } => {
                let b = branches.expect("branches should be set");
                assert_eq!(b, vec!["feat/a", "feat/b", "fix/c"]);
            }
            other => panic!("expected Start, got {other:?}"),
        }
    }

    #[test]
    fn start_with_dry_run() {
        let cli = parse(&["start", "--dry-run"]);
        match cli.command.unwrap() {
            Command::Start { dry_run, .. } => assert!(dry_run),
            other => panic!("expected Start, got {other:?}"),
        }
    }

    #[test]
    fn start_with_preset() {
        let cli = parse(&["start", "--preset", "backend"]);
        match cli.command.unwrap() {
            Command::Start { preset, .. } => assert_eq!(preset.as_deref(), Some("backend")),
            other => panic!("expected Start, got {other:?}"),
        }
    }

    #[test]
    fn start_with_supervisor_flag() {
        let cli = parse(&["start", "--supervisor"]);
        match cli.command.unwrap() {
            Command::Start { supervisor, .. } => assert!(supervisor),
            other => panic!("expected Start, got {other:?}"),
        }
    }

    #[test]
    fn start_without_supervisor_defaults_false() {
        let cli = parse(&["start", "--cli", "claude"]);
        match cli.command.unwrap() {
            Command::Start { supervisor, .. } => assert!(!supervisor),
            other => panic!("expected Start, got {other:?}"),
        }
    }

    #[test]
    fn start_with_supervisor_and_other_flags() {
        let cli = parse(&[
            "start",
            "--supervisor",
            "--cli",
            "claude",
            "--branches",
            "feat/a,feat/b",
        ]);
        match cli.command.unwrap() {
            Command::Start {
                supervisor,
                cli,
                branches,
                ..
            } => {
                assert!(supervisor);
                assert_eq!(cli.as_deref(), Some("claude"));
                assert_eq!(branches.unwrap(), vec!["feat/a", "feat/b"]);
            }
            other => panic!("expected Start, got {other:?}"),
        }
    }

    // -- --specs-format flag --

    #[test]
    fn start_with_specs_format_speckit() {
        let cli = parse(&["start", "--from-specs", "--specs-format", "speckit"]);
        match cli.command.unwrap() {
            Command::Start { specs_format, .. } => {
                assert_eq!(specs_format, Some(SpecsFormat::Speckit));
            }
            other => panic!("expected Start, got {other:?}"),
        }
    }

    #[test]
    fn start_with_specs_format_openspec() {
        let cli = parse(&["start", "--from-specs", "--specs-format", "openspec"]);
        match cli.command.unwrap() {
            Command::Start { specs_format, .. } => {
                assert_eq!(specs_format, Some(SpecsFormat::Openspec));
            }
            other => panic!("expected Start, got {other:?}"),
        }
    }

    #[test]
    fn start_with_specs_format_markdown() {
        let cli = parse(&["start", "--from-specs", "--specs-format", "markdown"]);
        match cli.command.unwrap() {
            Command::Start { specs_format, .. } => {
                assert_eq!(specs_format, Some(SpecsFormat::Markdown));
            }
            other => panic!("expected Start, got {other:?}"),
        }
    }

    #[test]
    fn start_rejects_unknown_specs_format() {
        let result = Cli::try_parse_from([
            "git-paw",
            "start",
            "--from-specs",
            "--specs-format",
            "unknown-value",
        ]);
        assert!(result.is_err(), "unknown value should be rejected");
        let err = result.unwrap_err().to_string();
        assert!(
            err.contains("openspec") && err.contains("markdown") && err.contains("speckit"),
            "error should list all three valid values, got: {err}"
        );
    }

    #[test]
    fn specs_format_as_str_matches_backend_names() {
        assert_eq!(SpecsFormat::Openspec.as_str(), "openspec");
        assert_eq!(SpecsFormat::Markdown.as_str(), "markdown");
        assert_eq!(SpecsFormat::Speckit.as_str(), "speckit");
    }

    #[test]
    fn start_help_shows_specs_format_flag() {
        let result = Cli::try_parse_from(["git-paw", "start", "--help"]);
        assert!(result.is_err());
        let err = result.unwrap_err();
        assert_eq!(err.kind(), clap::error::ErrorKind::DisplayHelp);
        let help = err.to_string();
        assert!(
            help.contains("--specs-format"),
            "start --help should contain --specs-format"
        );
    }

    #[test]
    fn start_help_shows_supervisor_flag() {
        let result = Cli::try_parse_from(["git-paw", "start", "--help"]);
        assert!(result.is_err());
        let err = result.unwrap_err();
        assert_eq!(err.kind(), clap::error::ErrorKind::DisplayHelp);
        let help = err.to_string();
        assert!(
            help.contains("--supervisor"),
            "start --help should contain --supervisor"
        );
    }

    // -- --no-supervisor flag --

    #[test]
    fn start_with_no_supervisor_flag() {
        let cli = parse(&["start", "--no-supervisor"]);
        match cli.command.unwrap() {
            Command::Start {
                supervisor,
                no_supervisor,
                ..
            } => {
                assert!(no_supervisor);
                assert!(!supervisor);
            }
            other => panic!("expected Start, got {other:?}"),
        }
    }

    #[test]
    fn start_without_flags_leaves_no_supervisor_false() {
        let cli = parse(&["start"]);
        match cli.command.unwrap() {
            Command::Start {
                supervisor,
                no_supervisor,
                ..
            } => {
                assert!(!no_supervisor);
                assert!(!supervisor);
            }
            other => panic!("expected Start, got {other:?}"),
        }
    }

    #[test]
    fn start_with_supervisor_and_no_supervisor_is_rejected() {
        let result = Cli::try_parse_from(["git-paw", "start", "--supervisor", "--no-supervisor"]);
        assert!(
            result.is_err(),
            "--supervisor + --no-supervisor must be rejected by clap"
        );
        let err = result.unwrap_err();
        let msg = err.to_string();
        assert!(
            msg.contains("--supervisor") && msg.contains("--no-supervisor"),
            "error should mention both flags, got: {msg}"
        );
    }

    #[test]
    fn start_with_no_supervisor_and_supervisor_reversed_is_also_rejected() {
        // clap's conflicts_with is bidirectional; order of flags shouldn't matter.
        let result = Cli::try_parse_from(["git-paw", "start", "--no-supervisor", "--supervisor"]);
        assert!(result.is_err());
    }

    #[test]
    fn start_no_supervisor_combines_with_other_flags() {
        let cli = parse(&[
            "start",
            "--no-supervisor",
            "--cli",
            "claude",
            "--branches",
            "feat/a,feat/b",
        ]);
        match cli.command.unwrap() {
            Command::Start {
                no_supervisor,
                cli,
                branches,
                ..
            } => {
                assert!(no_supervisor);
                assert_eq!(cli.as_deref(), Some("claude"));
                assert_eq!(branches.unwrap(), vec!["feat/a", "feat/b"]);
            }
            other => panic!("expected Start, got {other:?}"),
        }
    }

    #[test]
    fn start_help_shows_no_supervisor_flag() {
        let result = Cli::try_parse_from(["git-paw", "start", "--help"]);
        assert!(result.is_err());
        let err = result.unwrap_err();
        assert_eq!(err.kind(), clap::error::ErrorKind::DisplayHelp);
        let help = err.to_string();
        assert!(
            help.contains("--no-supervisor"),
            "start --help should contain --no-supervisor, got: {help}"
        );
    }

    #[test]
    fn start_with_all_flags() {
        let cli = parse(&[
            "start",
            "--cli",
            "gemini",
            "--branches",
            "a,b",
            "--dry-run",
            "--preset",
            "dev",
        ]);
        match cli.command.unwrap() {
            Command::Start {
                cli,
                branches,
                dry_run,
                preset,
                ..
            } => {
                assert_eq!(cli.as_deref(), Some("gemini"));
                assert_eq!(branches.unwrap(), vec!["a", "b"]);
                assert!(dry_run);
                assert_eq!(preset.as_deref(), Some("dev"));
            }
            other => panic!("expected Start, got {other:?}"),
        }
    }

    // -- Add subcommand --

    #[test]
    fn add_with_branch_only() {
        let cli = parse(&["add", "feat/new"]);
        match cli.command.unwrap() {
            Command::Add {
                branch,
                cli,
                from_spec,
            } => {
                assert_eq!(branch.as_deref(), Some("feat/new"));
                assert!(cli.is_none());
                assert!(from_spec.is_none());
            }
            other => panic!("expected Add, got {other:?}"),
        }
    }

    #[test]
    fn add_with_branch_and_cli() {
        let cli = parse(&["add", "feat/x", "--cli", "codex"]);
        match cli.command.unwrap() {
            Command::Add { branch, cli, .. } => {
                assert_eq!(branch.as_deref(), Some("feat/x"));
                assert_eq!(cli.as_deref(), Some("codex"));
            }
            other => panic!("expected Add, got {other:?}"),
        }
    }

    #[test]
    fn add_with_from_spec_only_needs_no_branch() {
        let cli = parse(&["add", "--from-spec", "add-export"]);
        match cli.command.unwrap() {
            Command::Add {
                branch, from_spec, ..
            } => {
                assert!(branch.is_none());
                assert_eq!(from_spec.as_deref(), Some("add-export"));
            }
            other => panic!("expected Add, got {other:?}"),
        }
    }

    #[test]
    fn add_with_no_branch_and_no_from_spec_is_rejected() {
        let result = Cli::try_parse_from(["git-paw", "add"]);
        assert!(
            result.is_err(),
            "add requires either a branch or --from-spec"
        );
    }

    #[test]
    fn add_with_branch_and_from_spec_is_rejected() {
        let result = Cli::try_parse_from(["git-paw", "add", "feat/x", "--from-spec", "change"]);
        assert!(
            result.is_err(),
            "branch and --from-spec are mutually exclusive"
        );
    }

    #[test]
    fn add_help_lists_flags_and_examples() {
        let result = Cli::try_parse_from(["git-paw", "add", "--help"]);
        let err = result.unwrap_err();
        assert_eq!(err.kind(), clap::error::ErrorKind::DisplayHelp);
        let help = err.to_string();
        assert!(help.contains("--cli"), "got: {help}");
        assert!(help.contains("--from-spec"), "got: {help}");
        assert!(
            help.contains("git paw add feat/api --cli codex"),
            "add --help should include copy-pasteable examples; got: {help}"
        );
    }

    // -- Remove subcommand --

    #[test]
    fn remove_with_branch_only() {
        let cli = parse(&["remove", "feat/done"]);
        match cli.command.unwrap() {
            Command::Remove {
                branch,
                keep_worktree,
                force,
            } => {
                assert_eq!(branch, "feat/done");
                assert!(!keep_worktree);
                assert!(!force);
            }
            other => panic!("expected Remove, got {other:?}"),
        }
    }

    #[test]
    fn remove_with_keep_worktree_and_force() {
        let cli = parse(&["remove", "feat/x", "--keep-worktree", "--force"]);
        match cli.command.unwrap() {
            Command::Remove {
                keep_worktree,
                force,
                ..
            } => {
                assert!(keep_worktree);
                assert!(force);
            }
            other => panic!("expected Remove, got {other:?}"),
        }
    }

    #[test]
    fn remove_without_branch_is_rejected() {
        let result = Cli::try_parse_from(["git-paw", "remove"]);
        assert!(result.is_err(), "remove requires a branch");
    }

    #[test]
    fn remove_help_lists_flags_and_examples() {
        let result = Cli::try_parse_from(["git-paw", "remove", "--help"]);
        let err = result.unwrap_err();
        assert_eq!(err.kind(), clap::error::ErrorKind::DisplayHelp);
        let help = err.to_string();
        assert!(help.contains("--keep-worktree"), "got: {help}");
        assert!(help.contains("--force"), "got: {help}");
        assert!(
            help.contains("git paw remove feat/wip --force"),
            "remove --help should include copy-pasteable examples; got: {help}"
        );
    }

    #[test]
    fn root_help_lists_add_and_remove() {
        let result = Cli::try_parse_from(["git-paw", "--help"]);
        let help = result.unwrap_err().to_string();
        assert!(
            help.contains("add"),
            "root help should list add; got: {help}"
        );
        assert!(
            help.contains("remove"),
            "root help should list remove; got: {help}"
        );
    }

    // -- Pause subcommand --

    #[test]
    fn pause_parses() {
        let cli = parse(&["pause"]);
        assert!(matches!(cli.command.unwrap(), Command::Pause));
    }

    #[test]
    fn pause_help_mentions_ram_tradeoff() {
        let result = Cli::try_parse_from(["git-paw", "pause", "--help"]);
        assert!(result.is_err());
        let err = result.unwrap_err();
        assert_eq!(err.kind(), clap::error::ErrorKind::DisplayHelp);
        let help = err.to_string();
        assert!(
            help.to_lowercase().contains("ram"),
            "pause --help should mention RAM tradeoff, got: {help}"
        );
        assert!(
            help.contains("stop"),
            "pause --help should cross-reference stop, got: {help}"
        );
    }

    #[test]
    fn pause_rejects_unknown_flags() {
        let result = Cli::try_parse_from(["git-paw", "pause", "--anything"]);
        assert!(result.is_err(), "pause should reject unknown flags");
    }

    #[test]
    fn root_help_lists_pause() {
        let result = Cli::try_parse_from(["git-paw", "--help"]);
        assert!(result.is_err());
        let err = result.unwrap_err();
        assert_eq!(err.kind(), clap::error::ErrorKind::DisplayHelp);
        let help = err.to_string();
        assert!(
            help.contains("pause"),
            "root --help should list pause subcommand, got: {help}"
        );
        // Quick-start block should also reference pause.
        assert!(
            help.contains("git paw pause"),
            "after_help quick-start should mention `git paw pause`"
        );
    }

    // -- Stop subcommand --

    #[test]
    fn stop_parses() {
        let cli = parse(&["stop"]);
        assert!(matches!(
            cli.command.unwrap(),
            Command::Stop { force: false }
        ));
    }

    #[test]
    fn stop_without_force() {
        let cli = parse(&["stop"]);
        match cli.command.unwrap() {
            Command::Stop { force } => assert!(!force),
            other => panic!("expected Stop, got {other:?}"),
        }
    }

    #[test]
    fn stop_with_force() {
        let cli = parse(&["stop", "--force"]);
        match cli.command.unwrap() {
            Command::Stop { force } => assert!(force),
            other => panic!("expected Stop, got {other:?}"),
        }
    }

    #[test]
    fn stop_help_mentions_pause_and_purge() {
        let result = Cli::try_parse_from(["git-paw", "stop", "--help"]);
        assert!(result.is_err());
        let err = result.unwrap_err();
        assert_eq!(err.kind(), clap::error::ErrorKind::DisplayHelp);
        let help = err.to_string();
        assert!(
            help.contains("pause"),
            "stop --help should reference pause, got: {help}"
        );
        assert!(
            help.contains("purge"),
            "stop --help should reference purge, got: {help}"
        );
        assert!(
            help.contains("--force"),
            "stop --help should list --force, got: {help}"
        );
    }

    // -- Purge subcommand --

    #[test]
    fn purge_without_force() {
        let cli = parse(&["purge"]);
        match cli.command.unwrap() {
            Command::Purge { force, stale } => {
                assert!(!force);
                assert!(!stale);
            }
            other => panic!("expected Purge, got {other:?}"),
        }
    }

    #[test]
    fn purge_with_force() {
        let cli = parse(&["purge", "--force"]);
        match cli.command.unwrap() {
            Command::Purge { force, stale } => {
                assert!(force);
                assert!(!stale);
            }
            other => panic!("expected Purge, got {other:?}"),
        }
    }

    #[test]
    fn purge_with_stale() {
        let cli = parse(&["purge", "--stale"]);
        match cli.command.unwrap() {
            Command::Purge { force, stale } => {
                assert!(!force);
                assert!(stale);
            }
            other => panic!("expected Purge, got {other:?}"),
        }
    }

    #[test]
    fn purge_with_stale_and_force() {
        let cli = parse(&["purge", "--stale", "--force"]);
        match cli.command.unwrap() {
            Command::Purge { force, stale } => {
                assert!(force);
                assert!(stale);
            }
            other => panic!("expected Purge, got {other:?}"),
        }
    }

    // -- Status subcommand --

    #[test]
    fn status_parses() {
        let cli = parse(&["status"]);
        match cli.command.unwrap() {
            Command::Status { json } => assert!(!json),
            other => panic!("expected Status, got {other:?}"),
        }
    }

    #[test]
    fn status_with_json() {
        let cli = parse(&["status", "--json"]);
        match cli.command.unwrap() {
            Command::Status { json } => assert!(json),
            other => panic!("expected Status, got {other:?}"),
        }
    }

    // -- List-CLIs subcommand --

    #[test]
    fn list_clis_parses() {
        let cli = parse(&["list-clis"]);
        assert!(matches!(cli.command.unwrap(), Command::ListClis));
    }

    // -- Add-CLI subcommand --

    #[test]
    fn add_cli_with_required_args() {
        let cli = parse(&["add-cli", "my-agent", "/usr/local/bin/my-agent"]);
        match cli.command.unwrap() {
            Command::AddCli {
                name,
                command,
                display_name,
            } => {
                assert_eq!(name, "my-agent");
                assert_eq!(command, "/usr/local/bin/my-agent");
                assert!(display_name.is_none());
            }
            other => panic!("expected AddCli, got {other:?}"),
        }
    }

    #[test]
    fn add_cli_with_display_name() {
        let cli = parse(&[
            "add-cli",
            "my-agent",
            "my-agent",
            "--display-name",
            "My Agent",
        ]);
        match cli.command.unwrap() {
            Command::AddCli {
                name,
                command,
                display_name,
            } => {
                assert_eq!(name, "my-agent");
                assert_eq!(command, "my-agent");
                assert_eq!(display_name.as_deref(), Some("My Agent"));
            }
            other => panic!("expected AddCli, got {other:?}"),
        }
    }

    // -- Remove-CLI subcommand --

    #[test]
    fn remove_cli_parses() {
        let cli = parse(&["remove-cli", "my-agent"]);
        match cli.command.unwrap() {
            Command::RemoveCli { name } => assert_eq!(name, "my-agent"),
            other => panic!("expected RemoveCli, got {other:?}"),
        }
    }

    // -- Help text quality --

    #[test]
    fn version_flag_is_accepted() {
        let result = Cli::try_parse_from(["git-paw", "--version"]);
        // clap returns Err(DisplayVersion) for --version, which is expected
        assert!(result.is_err());
        let err = result.unwrap_err();
        assert_eq!(err.kind(), clap::error::ErrorKind::DisplayVersion);
    }

    #[test]
    fn help_flag_is_accepted() {
        let result = Cli::try_parse_from(["git-paw", "--help"]);
        assert!(result.is_err());
        let err = result.unwrap_err();
        assert_eq!(err.kind(), clap::error::ErrorKind::DisplayHelp);
    }

    // --- Gap #6: init_parses ---

    #[test]
    fn init_parses() {
        let cli = parse(&["init"]);
        assert!(matches!(cli.command.unwrap(), Command::Init));
    }

    // --- Gap #7: init_help_text ---

    #[test]
    fn init_help_text() {
        let result = Cli::try_parse_from(["git-paw", "init", "--help"]);
        assert!(result.is_err());
        let err = result.unwrap_err();
        assert_eq!(err.kind(), clap::error::ErrorKind::DisplayHelp);
    }

    #[test]
    fn unknown_subcommand_is_rejected() {
        let result = Cli::try_parse_from(["git-paw", "unknown-command"]);
        assert!(result.is_err());
    }

    #[test]
    fn add_cli_missing_required_args_is_rejected() {
        let result = Cli::try_parse_from(["git-paw", "add-cli"]);
        assert!(result.is_err());
    }

    // -- Replay subcommand --

    #[test]
    fn replay_with_branch() {
        let cli = parse(&["replay", "feat/add-auth"]);
        match cli.command.unwrap() {
            Command::Replay {
                branch,
                list,
                color,
                session,
            } => {
                assert_eq!(branch.as_deref(), Some("feat/add-auth"));
                assert!(!list);
                assert!(!color);
                assert!(session.is_none());
            }
            other => panic!("expected Replay, got {other:?}"),
        }
    }

    #[test]
    fn replay_with_list() {
        let cli = parse(&["replay", "--list"]);
        match cli.command.unwrap() {
            Command::Replay { branch, list, .. } => {
                assert!(list);
                assert!(branch.is_none());
            }
            other => panic!("expected Replay, got {other:?}"),
        }
    }

    #[test]
    fn replay_with_color() {
        let cli = parse(&["replay", "feat/add-auth", "--color"]);
        match cli.command.unwrap() {
            Command::Replay { color, .. } => assert!(color),
            other => panic!("expected Replay, got {other:?}"),
        }
    }

    #[test]
    fn replay_with_session() {
        let cli = parse(&["replay", "feat/add-auth", "--session", "paw-myproject"]);
        match cli.command.unwrap() {
            Command::Replay { session, .. } => {
                assert_eq!(session.as_deref(), Some("paw-myproject"));
            }
            other => panic!("expected Replay, got {other:?}"),
        }
    }

    #[test]
    fn replay_no_args_fails() {
        let result = Cli::try_parse_from(["git-paw", "replay"]);
        assert!(result.is_err());
    }

    #[test]
    fn replay_help_text() {
        let result = Cli::try_parse_from(["git-paw", "replay", "--help"]);
        assert!(result.is_err());
        let err = result.unwrap_err();
        assert_eq!(err.kind(), clap::error::ErrorKind::DisplayHelp);
        let help = err.to_string();
        assert!(help.contains("--list"));
        assert!(help.contains("--color"));
        assert!(help.contains("--session"));
    }

    #[test]
    fn help_shows_replay_subcommand() {
        let result = Cli::try_parse_from(["git-paw", "--help"]);
        let err = result.unwrap_err();
        let help = err.to_string();
        assert!(
            help.contains("replay"),
            "help should list the replay subcommand"
        );
    }

    // -- __dashboard subcommand --

    #[test]
    fn dashboard_parses() {
        let cli = parse(&["__dashboard"]);
        assert!(matches!(cli.command.unwrap(), Command::Dashboard));
    }

    // -- --no-rebase flag --

    #[test]
    fn start_with_no_rebase_flag_sets_no_rebase_true() {
        let cli = parse(&["start", "--no-rebase"]);
        match cli.command.unwrap() {
            Command::Start { no_rebase, .. } => assert!(no_rebase),
            other => panic!("expected Start, got {other:?}"),
        }
    }

    #[test]
    fn start_without_no_rebase_defaults_to_false() {
        let cli = parse(&["start"]);
        match cli.command.unwrap() {
            Command::Start { no_rebase, .. } => assert!(!no_rebase),
            other => panic!("expected Start, got {other:?}"),
        }
    }

    #[test]
    fn start_no_rebase_combines_with_supervisor() {
        let cli = parse(&["start", "--no-rebase", "--supervisor"]);
        match cli.command.unwrap() {
            Command::Start {
                no_rebase,
                supervisor,
                ..
            } => {
                assert!(no_rebase);
                assert!(supervisor);
            }
            other => panic!("expected Start, got {other:?}"),
        }
    }

    #[test]
    fn start_help_shows_no_rebase_flag() {
        let result = Cli::try_parse_from(["git-paw", "start", "--help"]);
        assert!(result.is_err());
        let err = result.unwrap_err();
        assert_eq!(err.kind(), clap::error::ErrorKind::DisplayHelp);
        let help = err.to_string();
        assert!(
            help.contains("--no-rebase"),
            "start --help should contain --no-rebase, got: {help}"
        );
    }

    // -- Approvals subcommand --

    #[test]
    fn approvals_parses_with_no_flags() {
        let cli = parse(&["approvals"]);
        match cli.command.unwrap() {
            Command::Approvals {
                session,
                limit,
                json,
            } => {
                assert!(session.is_none());
                assert!(limit.is_none());
                assert!(!json);
            }
            other => panic!("expected Approvals, got {other:?}"),
        }
    }

    #[test]
    fn approvals_with_session_limit_and_json() {
        let cli = parse(&[
            "approvals",
            "--session",
            "paw-other",
            "--limit",
            "5",
            "--json",
        ]);
        match cli.command.unwrap() {
            Command::Approvals {
                session,
                limit,
                json,
            } => {
                assert_eq!(session.as_deref(), Some("paw-other"));
                assert_eq!(limit, Some(5));
                assert!(json);
            }
            other => panic!("expected Approvals, got {other:?}"),
        }
    }

    #[test]
    fn approvals_rejects_non_numeric_limit() {
        let result = Cli::try_parse_from(["git-paw", "approvals", "--limit", "lots"]);
        assert!(result.is_err());
    }

    #[test]
    fn approvals_help_lists_flags_and_examples() {
        let result = Cli::try_parse_from(["git-paw", "approvals", "--help"]);
        assert!(result.is_err());
        let err = result.unwrap_err();
        assert_eq!(err.kind(), clap::error::ErrorKind::DisplayHelp);
        let help = err.to_string();
        assert!(help.contains("--session"), "got: {help}");
        assert!(help.contains("--limit"), "got: {help}");
        assert!(help.contains("--json"), "got: {help}");
        assert!(
            help.contains("git paw approvals --json"),
            "help should include examples, got: {help}"
        );
    }

    #[test]
    fn help_shows_approvals_subcommand() {
        let result = Cli::try_parse_from(["git-paw", "--help"]);
        let err = result.unwrap_err();
        let help = err.to_string();
        assert!(
            help.contains("approvals"),
            "root help should list approvals subcommand, got: {help}"
        );
    }

    #[test]
    fn dashboard_does_not_appear_in_help() {
        let result = Cli::try_parse_from(["git-paw", "--help"]);
        let err = result.unwrap_err();
        let help = err.to_string();
        assert!(
            !help.contains("__dashboard"),
            "hidden __dashboard subcommand should not appear in help output"
        );
    }
}