🐛 Make second column of Rich help output reflect the type consistently, even when using metavar

docs_src/arguments/help/tutorial006_an_py39.py

@@ -6,7 +6,7 @@
 
 
 @app.command()
-def main(name: Annotated[str, typer.Argument(metavar="✨username✨")] = "World"):
+def main(name: Annotated[str, typer.Argument(metavar="✨user✨")] = "World"):
     print(f"Hello {name}")
 
 

docs_src/arguments/help/tutorial006_py39.py

@@ -4,7 +4,7 @@
 
 
 @app.command()
-def main(name: str = typer.Argument("World", metavar="✨username✨")):
+def main(name: str = typer.Argument("World", metavar="✨user✨")):
     print(f"Hello {name}")
 
 

tests/test_rich_utils.py

@@ -73,7 +73,7 @@ def main(
 
     result = runner.invoke(app, ["--help"])
     assert "Usage" in result.stdout
-    assert "name" in result.stdout
+    assert "NAME" in result.stdout
     assert "option-1" in result.stdout
     assert "option-2" in result.stdout
     assert result.stdout.count("[default: None]") == 0
@@ -99,3 +99,43 @@ def main(bar: str):
     result = runner.invoke(app, ["--help"])
     assert "Usage" in result.stdout
     assert "BAR" in result.stdout
+
+
+def test_rich_help_metavar():
+    app = typer.Typer(rich_markup_mode="rich")
+
+    @app.command()
+    def main(
+        *,
+        arg1: int,
+        arg2: int = 42,
+        arg3: int = typer.Argument(...),
+        arg4: int = typer.Argument(42),
+        arg5: int = typer.Option(...),
+        arg6: int = typer.Option(42),
+        arg7: int = typer.Argument(42, metavar="meta7"),
+        arg8: int = typer.Argument(metavar="ARG8"),
+        arg9: int = typer.Argument(metavar="arg9"),
+    ):
+        pass  # pragma: no cover
+
+    result = runner.invoke(app, ["--help"])
+    assert "Usage: main [OPTIONS] ARG1 ARG3 [ARG4] [meta7] ARG8 arg9" in result.stdout
+
+    out_nospace = result.stdout.replace(" ", "")
+
+    # arguments
+    assert "ARG1INTEGER" in out_nospace
+    assert "ARG3INTEGER" in out_nospace
+    assert "[ARG4]INTEGER" in out_nospace
+    assert "[meta7]INTEGER" in out_nospace
+    assert "ARG8INTEGER" in out_nospace
+    assert "arg9INTEGER" in out_nospace
+
+    assert "arg7" not in result.stdout.lower()
+    assert "ARG9" not in result.stdout
+
+    # options
+    assert "arg2INTEGER" in out_nospace
+    assert "arg5INTEGER" in out_nospace
+    assert "arg6INTEGER" in out_nospace

tests/test_tutorial/test_arguments/test_help/test_tutorial006.py

@@ -4,6 +4,7 @@
 from types import ModuleType
 
 import pytest
+import typer
 from typer.testing import CliRunner
 
 runner = CliRunner()
@@ -22,16 +23,31 @@ def get_mod(request: pytest.FixtureRequest) -> ModuleType:
     return mod
 
 
-def test_help(mod: ModuleType):
-    result = runner.invoke(mod.app, ["--help"])
+@pytest.fixture(
+    name="app",
+    params=[
+        pytest.param(None),
+        pytest.param("rich"),
+    ],
+)
+def get_app(mod: ModuleType, request: pytest.FixtureRequest) -> typer.Typer:
+    rich_markup_mode = request.param
+    app = typer.Typer(rich_markup_mode=rich_markup_mode)
+    app.command()(mod.main)
+    return app
+
+
+def test_help(app):
+    result = runner.invoke(app, ["--help"])
     assert result.exit_code == 0
-    assert "Usage: main [OPTIONS] [✨username✨]" in result.output
+    assert "Usage: main [OPTIONS] [✨user✨]" in result.output
     assert "Arguments" in result.output
+    assert "name" not in result.output
     assert "[default: World]" in result.output
 
 
-def test_call_arg(mod: ModuleType):
-    result = runner.invoke(mod.app, ["Camila"])
+def test_call_arg(app):
+    result = runner.invoke(app, ["Camila"])
     assert result.exit_code == 0
     assert "Hello Camila" in result.output
 

tests/test_tutorial/test_commands/test_help/test_tutorial007.py

@@ -40,10 +40,10 @@ def test_main_help(mod: ModuleType):
 def test_create_help(mod: ModuleType):
     result = runner.invoke(mod.app, ["create", "--help"])
     assert result.exit_code == 0
-    assert "username" in result.output
+    assert "USERNAME" in result.output
     assert "The username to create" in result.output
     assert "Secondary Arguments" in result.output
-    assert "lastname" in result.output
+    assert "LASTNAME" in result.output
     assert "The last name of the new user" in result.output
     assert "--force" in result.output
     assert "--no-force" in result.output

typer/rich_utils.py

@@ -30,8 +30,8 @@
 STYLE_SWITCH = "bold green"
 STYLE_NEGATIVE_OPTION = "bold magenta"
 STYLE_NEGATIVE_SWITCH = "bold red"
-STYLE_METAVAR = "bold yellow"
-STYLE_METAVAR_SEPARATOR = "dim"
+STYLE_TYPES = "bold yellow"
+STYLE_TYPES_SEPARATOR = "dim"
 STYLE_USAGE = "yellow"
 STYLE_USAGE_COMMAND = "bold"
 STYLE_DEPRECATED = "red"
@@ -108,7 +108,7 @@ class OptionHighlighter(RegexHighlighter):
     highlights = [
         r"(^|\W)(?P<switch>\-\w+)(?![a-zA-Z0-9])",
         r"(^|\W)(?P<option>\-\-[\w\-]+)(?![a-zA-Z0-9])",
-        r"(?P<metavar>\<[^\>]+\>)",
+        r"(?P<types>\<[^\>]+\>)",
         r"(?P<usage>Usage: )",
     ]
 
@@ -132,8 +132,8 @@ def _get_rich_console(stderr: bool = False) -> Console:
                 "switch": STYLE_SWITCH,
                 "negative_option": STYLE_NEGATIVE_OPTION,
                 "negative_switch": STYLE_NEGATIVE_SWITCH,
-                "metavar": STYLE_METAVAR,
-                "metavar_sep": STYLE_METAVAR_SEPARATOR,
+                "types": STYLE_TYPES,
+                "types_sep": STYLE_TYPES_SEPARATOR,
                 "usage": STYLE_USAGE,
             },
         ),
@@ -343,38 +343,47 @@ def _print_options_panel(
         opt_short_strs = []
         secondary_opt_long_strs = []
         secondary_opt_short_strs = []
+
+        # check whether argument has a metavar name or type set
+        metavar_name = None
+        metavar_type = None
+        # TODO: when deprecating Click < 8.2, make ctx required
+        signature = inspect.signature(param.make_metavar)
+        if "ctx" in signature.parameters:
+            metavar_str = param.make_metavar(ctx=ctx)
+        else:
+            # Click < 8.2
+            metavar_str = param.make_metavar()  # type: ignore[call-arg]
+        if isinstance(param, click.Argument):
+            metavar_name = metavar_str
+        if isinstance(param, click.Option):
+            metavar_type = metavar_str
+
         for opt_str in param.opts:
             if "--" in opt_str:
                 opt_long_strs.append(opt_str)
+            elif metavar_name:
+                opt_short_strs.append(metavar_name)
             else:
                 opt_short_strs.append(opt_str)
         for opt_str in param.secondary_opts:
             if "--" in opt_str:
                 secondary_opt_long_strs.append(opt_str)
+            elif metavar_name:  # pragma: no cover
+                secondary_opt_short_strs.append(metavar_name)
             else:
                 secondary_opt_short_strs.append(opt_str)
 
-        # Column for a metavar, if we have one
-        metavar = Text(style=STYLE_METAVAR, overflow="fold")
-        # TODO: when deprecating Click < 8.2, make ctx required
-        signature = inspect.signature(param.make_metavar)
-        if "ctx" in signature.parameters:
-            metavar_str = param.make_metavar(ctx=ctx)
-        else:
-            # Click < 8.2
-            metavar_str = param.make_metavar()  # type: ignore[call-arg]
+        # Column for recording the type
+        types = Text(style=STYLE_TYPES, overflow="fold")
 
-        # Do it ourselves if this is a positional argument
-        if (
-            isinstance(param, click.Argument)
-            and param.name
-            and metavar_str == param.name.upper()
-        ):
-            metavar_str = param.type.name.upper()
-
-        # Skip booleans and choices (handled above)
-        if metavar_str != "BOOLEAN":
-            metavar.append(metavar_str)
+        # Fetch type
+        if metavar_type and metavar_type != "BOOLEAN":
+            types.append(metavar_type)
+        else:
+            type_str = param.type.name.upper()
+            if type_str != "BOOLEAN":
+                types.append(type_str)
 
         # Range - from
         # https://github.com/pallets/click/blob/c63c70dabd3f86ca68678b4f00951f78f52d0270/src/click/core.py#L2698-L2706  # noqa: E501
@@ -386,22 +395,22 @@ def _print_options_panel(
         ):
             range_str = param.type._describe_range()
             if range_str:
-                metavar.append(RANGE_STRING.format(range_str))
+                types.append(RANGE_STRING.format(range_str))
 
         # Required asterisk
         required: Union[str, Text] = ""
         if param.required:
             required = Text(REQUIRED_SHORT_STRING, style=STYLE_REQUIRED_SHORT)
 
         # Highlighter to make [ | ] and <> dim
-        class MetavarHighlighter(RegexHighlighter):
+        class TypesHighlighter(RegexHighlighter):
             highlights = [
-                r"^(?P<metavar_sep>(\[|<))",
-                r"(?P<metavar_sep>\|)",
-                r"(?P<metavar_sep>(\]|>)$)",
+                r"^(?P<types_sep>(\[|<))",
+                r"(?P<types_sep>\|)",
+                r"(?P<types_sep>(\]|>)$)",
             ]
 
-        metavar_highlighter = MetavarHighlighter()
+        types_highlighter = TypesHighlighter()
 
         required_rows.append(required)
         options_rows.append(
@@ -410,7 +419,7 @@ class MetavarHighlighter(RegexHighlighter):
                 highlighter(",".join(opt_short_strs)),
                 negative_highlighter(",".join(secondary_opt_long_strs)),
                 negative_highlighter(",".join(secondary_opt_short_strs)),
-                metavar_highlighter(metavar),
+                types_highlighter(types),
                 _get_parameter_help(
                     param=param,
                     ctx=ctx,