GeezerVison

Summary

• Fix --quiet and --force help strings in tmuxp freeze, both of which described behaviour the flags do not have
• Fix CLIFreezeNamespace.save_to, typed str | None while -o/--save-to parses with type=pathlib.Path
• Fix CLIColorsLiteral, which admitted 56 (a value nothing produces) and excluded 256 (the value -2 assigns)
• Remove three namespace annotations declared for flags that do not exist

Each was found while documenting these classes: writing an accurate description of a field is what exposed that the field's declaration and its runtime behaviour disagreed.

Based on class-variables, which supplies the docstrings these declarations belong to.

Changes

src/tmuxp/cli/freeze.py

--quiet: help said "don't prompt for confirmation". The flag only guards informational output — every if not args.quiet wraps a tmuxp_echo call. Confirmation prompts still run; --yes is the flag that skips them. A user reaching for -q to get an unattended freeze would still be sitting at a prompt.

--force: help said "overwrite the workspace file". It only relaxes the existence check inside the interactive destination prompt (if not args.force and os.path.exists(dest_prompt)). When -o is given there is no existence check at all, so --force is a no-op on the path most users pair it with.

save_to: declared str | None, but argparse stores a PosixPath. Correcting the annotation surfaced three genuine type errors that the wrong declaration had been masking — the destination is funnelled through a local that becomes a str. That conversion is now explicit at the boundary.

src/tmuxp/cli/load.py

CLIColorsLiteral was Literal[56, 88]. -2 stores const=256 and -8 stores const=88, so the type admitted a value nothing produces and rejected one of the two the flags actually assign. 56 reads as a dropped digit from 256.

src/tmuxp/cli/shell.py, src/tmuxp/cli/__init__.py

CLIShellNamespace.colors and .log_file are declared, but the shell subparser defines no -2/-8 and no --log-file, and command_shell reads neither. CLINamespace.version is declared, but --version uses argparse's version action, which prints and exits with dest suppressed. All three are absent from the parsed namespace.

Verification

Confirm -2 yields the value the corrected literal admits:

$ python -c "from tmuxp.cli import create_parser; print(create_parser().parse_args(['load','x','-2']).colors)"

Confirm the removed annotations never had a value:

$ python -c "from tmuxp.cli import create_parser; n=create_parser().parse_args(['shell']); print(hasattr(n,'colors'), hasattr(n,'log_file'))"

Test plan

• test_freeze_save_to_parses_as_a_path — the parsed value is a pathlib.Path, matching the declaration
• test_freeze_quiet_help_describes_what_it_silences — help names output suppression and points at --yes
• test_freeze_force_help_scopes_itself_to_the_prompt — help scopes itself to the prompted destination
• uv run ruff format / ruff check clean
• uv run mypy src/ tests/ clean, including the three errors the corrected save_to type exposed
• Full suite green