Finish reverting ruff rule TC006 changes and added that to the ignore list

This rule forced every use of typing inside a cast to be quoted. In reality, it is only forward reference types which needed this.

Author: tleonhardt

cmd2/argparse_custom.py

@@ -447,7 +447,7 @@ def _action_get_choices_callable(self: argparse.Action) -> Optional[ChoicesCalla
     :param self: argparse Action being queried
     :return: A ChoicesCallable instance or None if attribute does not exist
     """
-    return cast("Optional[ChoicesCallable]", getattr(self, ATTR_CHOICES_CALLABLE, None))
+    return cast(Optional[ChoicesCallable], getattr(self, ATTR_CHOICES_CALLABLE, None))
 
 
 setattr(argparse.Action, 'get_choices_callable', _action_get_choices_callable)
@@ -537,7 +537,7 @@ def _action_get_descriptive_header(self: argparse.Action) -> Optional[str]:
     :param self: argparse Action being queried
     :return: The value of descriptive_header or None if attribute does not exist
     """
-    return cast("Optional[str]", getattr(self, ATTR_DESCRIPTIVE_HEADER, None))
+    return cast(Optional[str], getattr(self, ATTR_DESCRIPTIVE_HEADER, None))
 
 
 setattr(argparse.Action, 'get_descriptive_header', _action_get_descriptive_header)
@@ -574,7 +574,7 @@ def _action_get_nargs_range(self: argparse.Action) -> Optional[tuple[int, Union[
     :param self: argparse Action being queried
     :return: The value of nargs_range or None if attribute does not exist
     """
-    return cast("Optional[tuple[int, Union[int, float]]]", getattr(self, ATTR_NARGS_RANGE, None))
+    return cast(Optional[tuple[int, Union[int, float]]], getattr(self, ATTR_NARGS_RANGE, None))
 
 
 setattr(argparse.Action, 'get_nargs_range', _action_get_nargs_range)
@@ -611,7 +611,7 @@ def _action_get_suppress_tab_hint(self: argparse.Action) -> bool:
     :param self: argparse Action being queried
     :return: The value of suppress_tab_hint or False if attribute does not exist
     """
-    return cast("bool", getattr(self, ATTR_SUPPRESS_TAB_HINT, False))
+    return cast(bool, getattr(self, ATTR_SUPPRESS_TAB_HINT, False))
 
 
 setattr(argparse.Action, 'get_suppress_tab_hint', _action_get_suppress_tab_hint)
@@ -921,7 +921,7 @@ def _ArgumentParser_get_ap_completer_type(self: argparse.ArgumentParser) -> Opti
     :param self: ArgumentParser being queried
     :return: An ArgparseCompleter-based class or None if attribute does not exist
     """
-    return cast("Optional[type[ArgparseCompleter]]", getattr(self, ATTR_AP_COMPLETER_TYPE, None))
+    return cast(Optional[type['ArgparseCompleter']], getattr(self, ATTR_AP_COMPLETER_TYPE, None))
 
 
 setattr(argparse.ArgumentParser, 'get_ap_completer_type', _ArgumentParser_get_ap_completer_type)

cmd2/cmd2.py

@@ -175,7 +175,7 @@
         )
 
         rl_basic_quote_characters = ctypes.c_char_p.in_dll(readline_lib, "rl_basic_quote_characters")
-        orig_rl_basic_quotes = cast("bytes", ctypes.cast(rl_basic_quote_characters, ctypes.c_void_p).value)
+        orig_rl_basic_quotes = cast(bytes, ctypes.cast(rl_basic_quote_characters, ctypes.c_void_p).value)
 
 
 class _SavedReadlineSettings:
@@ -707,7 +707,7 @@ def register_command_set(self, cmdset: CommandSet) -> None:
 
         cmdset.on_register(self)
         methods = cast(
-            "list[tuple[str, Callable[..., Any]]]",
+            list[tuple[str, Callable[..., Any]]],
             inspect.getmembers(
                 cmdset,
                 predicate=lambda meth: isinstance(meth, Callable)  # type: ignore[arg-type]
@@ -970,7 +970,7 @@ def find_subcommand(action: argparse.ArgumentParser, subcmd_names: list[str]) ->
 
             target_parser = find_subcommand(command_parser, subcommand_names)
 
-            subcmd_parser = cast("argparse.ArgumentParser", self._build_parser(cmdset, subcmd_parser_builder))
+            subcmd_parser = cast(argparse.ArgumentParser, self._build_parser(cmdset, subcmd_parser_builder))
             from .decorators import (
                 _set_parser_prog,
             )
@@ -1153,7 +1153,7 @@ def allow_style_type(value: str) -> ansi.AllowStyle:
                 'Allow ANSI text style sequences in output (valid values: '
                 f'{ansi.AllowStyle.ALWAYS}, {ansi.AllowStyle.NEVER}, {ansi.AllowStyle.TERMINAL})',
                 self,
-                choices_provider=cast("ChoicesProviderFunc", get_allow_style_choices),
+                choices_provider=cast(ChoicesProviderFunc, get_allow_style_choices),
             )
         )
 
@@ -1945,7 +1945,7 @@ def _display_matches_gnu_readline(
 
                 # rl_display_match_list() expects matches to be in argv format where
                 # substitution is the first element, followed by the matches, and then a NULL.
-                strings_array = cast("list[Optional[bytes]]", (ctypes.c_char_p * (1 + len(encoded_matches) + 1))())
+                strings_array = cast(list[Optional[bytes]], (ctypes.c_char_p * (1 + len(encoded_matches) + 1))())
 
                 # Copy in the encoded strings and add a NULL to the end
                 strings_array[0] = encoded_substitution
@@ -2868,7 +2868,7 @@ def _redirect_output(self, statement: Statement) -> utils.RedirectionSavedState:
 
         # Initialize the redirection saved state
         redir_saved_state = utils.RedirectionSavedState(
-            cast("TextIO", self.stdout), sys.stdout, self._cur_pipe_proc_reader, self._redirecting
+            cast(TextIO, self.stdout), sys.stdout, self._cur_pipe_proc_reader, self._redirecting
         )
 
         # The ProcReader for this command
@@ -2884,7 +2884,7 @@ def _redirect_output(self, statement: Statement) -> utils.RedirectionSavedState:
 
             # Open each side of the pipe
             subproc_stdin = open(read_fd)
-            new_stdout: TextIO = cast("TextIO", open(write_fd, 'w'))
+            new_stdout: TextIO = cast(TextIO, open(write_fd, 'w'))
 
             # Create pipe process in a separate group to isolate our signals from it. If a Ctrl-C event occurs,
             # our sigint handler will forward it only to the most recent pipe process. This makes sure pipe
@@ -2925,7 +2925,7 @@ def _redirect_output(self, statement: Statement) -> utils.RedirectionSavedState:
                 new_stdout.close()
                 raise RedirectionError(f'Pipe process exited with code {proc.returncode} before command could run')
             redir_saved_state.redirecting = True  # type: ignore[unreachable]
-            cmd_pipe_proc_reader = utils.ProcReader(proc, cast("TextIO", self.stdout), sys.stderr)
+            cmd_pipe_proc_reader = utils.ProcReader(proc, cast(TextIO, self.stdout), sys.stderr)
             sys.stdout = self.stdout = new_stdout
 
         elif statement.output:
@@ -2935,7 +2935,7 @@ def _redirect_output(self, statement: Statement) -> utils.RedirectionSavedState:
                 mode = 'a' if statement.output == constants.REDIRECTION_APPEND else 'w'
                 try:
                     # Use line buffering
-                    new_stdout = cast("TextIO", open(utils.strip_quotes(statement.output_to), mode=mode, buffering=1))
+                    new_stdout = cast(TextIO, open(utils.strip_quotes(statement.output_to), mode=mode, buffering=1))
                 except OSError as ex:
                     raise RedirectionError(f'Failed to redirect because: {ex}')
 
@@ -2956,7 +2956,7 @@ def _redirect_output(self, statement: Statement) -> utils.RedirectionSavedState:
                 # no point opening up the temporary file
                 current_paste_buffer = get_paste_buffer()
                 # create a temporary file to store output
-                new_stdout = cast("TextIO", tempfile.TemporaryFile(mode="w+"))
+                new_stdout = cast(TextIO, tempfile.TemporaryFile(mode="w+"))
                 redir_saved_state.redirecting = True
                 sys.stdout = self.stdout = new_stdout
 
@@ -2989,8 +2989,8 @@ def _restore_output(self, statement: Statement, saved_redir_state: utils.Redirec
                 pass
 
             # Restore the stdout values
-            self.stdout = cast("TextIO", saved_redir_state.saved_self_stdout)
-            sys.stdout = cast("TextIO", saved_redir_state.saved_sys_stdout)
+            self.stdout = cast(TextIO, saved_redir_state.saved_self_stdout)
+            sys.stdout = cast(TextIO, saved_redir_state.saved_sys_stdout)
 
             # Check if we need to wait for the process being piped to
             if self._cur_pipe_proc_reader is not None:
@@ -3016,7 +3016,7 @@ def cmd_func(self, command: str) -> Optional[CommandFunc]:
         """
         func_name = constants.COMMAND_FUNC_PREFIX + command
         func = getattr(self, func_name, None)
-        return cast("CommandFunc", func) if callable(func) else None
+        return cast(CommandFunc, func) if callable(func) else None
 
     def onecmd(self, statement: Union[Statement, str], *, add_to_history: bool = True) -> bool:
         """This executes the actual do_* method for a command.
@@ -3276,7 +3276,7 @@ def _set_up_cmd2_readline(self) -> _SavedReadlineSettings:
                 # We don't want this behavior since cmd2 only adds a closing quote when self.allow_closing_quote is True.
                 # To fix this behavior, set readline's rl_basic_quote_characters to NULL. We don't need to worry about setting
                 # rl_completion_suppress_quote since we never declared rl_completer_quote_characters.
-                readline_settings.basic_quotes = cast("bytes", ctypes.cast(rl_basic_quote_characters, ctypes.c_void_p).value)
+                readline_settings.basic_quotes = cast(bytes, ctypes.cast(rl_basic_quote_characters, ctypes.c_void_p).value)
                 rl_basic_quote_characters.value = None
 
             readline_settings.completer = readline.get_completer()
@@ -3941,7 +3941,7 @@ def _build_command_info(self) -> tuple[dict[str, list[str]], list[str], list[str
         cmds_undoc: list[str] = []
         cmds_cats: dict[str, list[str]] = {}
         for command in visible_commands:
-            func = cast("CommandFunc", self.cmd_func(command))
+            func = cast(CommandFunc, self.cmd_func(command))
             has_help_func = False
             has_parser = func in self._command_parsers
 
@@ -4011,7 +4011,7 @@ def _print_topics(self, header: str, cmds: list[str], verbose: bool) -> None:
                             stdout_orig = self.stdout
                             try:
                                 # redirect our internal stdout
-                                self.stdout = cast("TextIO", result)
+                                self.stdout = cast(TextIO, result)
                                 help_func()
                             finally:
                                 # restore internal stdout
@@ -4078,7 +4078,7 @@ def select(self, opts: Union[str, list[str], list[tuple[Any, Optional[str]]]], p
                                 the text advertised to the user"""
         local_opts: Union[list[str], list[tuple[Any, Optional[str]]]]
         if isinstance(opts, str):
-            local_opts = cast("list[tuple[Any, Optional[str]]]", list(zip(opts.split(), opts.split())))
+            local_opts = cast(list[tuple[Any, Optional[str]]], list(zip(opts.split(), opts.split())))
         else:
             local_opts = opts
         fulloptions: list[tuple[Any, Optional[str]]] = []
@@ -4278,7 +4278,7 @@ def do_shell(self, args: argparse.Namespace) -> None:
                 **kwargs,
             )
 
-            proc_reader = utils.ProcReader(proc, cast("TextIO", self.stdout), sys.stderr)  # type: ignore[arg-type]
+            proc_reader = utils.ProcReader(proc, cast(TextIO, self.stdout), sys.stderr)  # type: ignore[arg-type]
             proc_reader.wait()
 
             # Save the return code of the application for use in a pyscript
@@ -4337,7 +4337,7 @@ def _set_up_py_shell_env(self, interp: InteractiveConsole) -> _SavedCmd2Env:
                 # rlcompleter relies on the default settings of the Python readline module
                 if rl_type == RlType.GNU:
                     cmd2_env.readline_settings.basic_quotes = cast(
-                        "bytes", ctypes.cast(rl_basic_quote_characters, ctypes.c_void_p).value
+                        bytes, ctypes.cast(rl_basic_quote_characters, ctypes.c_void_p).value
                     )
                     rl_basic_quote_characters.value = orig_rl_basic_quotes
 
@@ -4986,8 +4986,8 @@ def _generate_transcript(
                 transcript += command
 
                 # Use a StdSim object to capture output
-                stdsim = utils.StdSim(cast("TextIO", self.stdout))
-                self.stdout = cast("TextIO", stdsim)
+                stdsim = utils.StdSim(cast(TextIO, self.stdout))
+                self.stdout = cast(TextIO, stdsim)
 
                 # then run the command and let the output go into our buffer
                 try:
@@ -5012,7 +5012,7 @@ def _generate_transcript(
             with self.sigint_protection:
                 # Restore altered attributes to their original state
                 self.echo = saved_echo
-                self.stdout = cast("TextIO", saved_stdout)
+                self.stdout = cast(TextIO, saved_stdout)
 
         # Check if all commands ran
         if commands_run < len(history):
@@ -5226,7 +5226,7 @@ class TestMyAppCase(Cmd2TestCase):
         setattr(self.__class__, 'testfiles', transcripts_expanded)
         sys.argv = [sys.argv[0]]  # the --test argument upsets unittest.main()
         testcase = TestMyAppCase()
-        stream = cast("TextIO", utils.StdSim(sys.stderr))
+        stream = cast(TextIO, utils.StdSim(sys.stderr))
         runner = unittest.TextTestRunner(stream=stream)
         start_time = time.time()
         test_results = runner.run(testcase)
@@ -5615,7 +5615,7 @@ def register_postloop_hook(self, func: Callable[[], None]) -> None:
     @classmethod
     def _validate_postparsing_callable(cls, func: Callable[[plugin.PostparsingData], plugin.PostparsingData]) -> None:
         """Check parameter and return types for postparsing hooks"""
-        cls._validate_callable_param_count(cast("Callable[..., Any]", func), 1)
+        cls._validate_callable_param_count(cast(Callable[..., Any], func), 1)
         signature = inspect.signature(func)
         _, param = list(signature.parameters.items())[0]
         if param.annotation != plugin.PostparsingData:
@@ -5637,7 +5637,7 @@ def _validate_prepostcmd_hook(
         """Check parameter and return types for pre and post command hooks."""
         signature = inspect.signature(func)
         # validate that the callable has the right number of parameters
-        cls._validate_callable_param_count(cast("Callable[..., Any]", func), 1)
+        cls._validate_callable_param_count(cast(Callable[..., Any], func), 1)
         # validate the parameter has the right annotation
         paramname = list(signature.parameters.keys())[0]
         param = signature.parameters[paramname]

cmd2/transcript.py

@@ -48,7 +48,7 @@ def setUp(self) -> None:
 
             # Trap stdout
             self._orig_stdout = self.cmdapp.stdout
-            self.cmdapp.stdout = cast('TextIO', utils.StdSim(cast('TextIO', self.cmdapp.stdout)))
+            self.cmdapp.stdout = cast(TextIO, utils.StdSim(cast(TextIO, self.cmdapp.stdout)))
 
     def tearDown(self) -> None:
         if self.cmdapp:
@@ -63,7 +63,7 @@ def runTest(self) -> None:  # was testall
 
     def _fetchTranscripts(self) -> None:
         self.transcripts = {}
-        testfiles = cast('list[str]', getattr(self.cmdapp, 'testfiles', []))
+        testfiles = cast(list[str], getattr(self.cmdapp, 'testfiles', []))
         for fname in testfiles:
             tfile = open(fname)
             self.transcripts[fname] = iter(tfile.readlines())

pyproject.toml

@@ -222,6 +222,7 @@ select = [
 ]
 ignore = [
     # `uv run ruff rule E501` for a description of that rule
+    "TC006", # Add quotes to type expression in typing.cast() (not needed except for forward references)
     "UP007", # Use X | Y for type annotations (requires Python 3.10+)
     "UP017", # Use datetime.UTC alias (requires Python 3.11+)
 ]