chore(ruff) Automated fixes for Python 3.9

ruff check . --select ALL --fix --unsafe-fixes --preview --show-fixes --ignore T201 --ignore PT014 --ignore RUF100; ruff format .;

Fixed 174 errors:
- conftest.py:
     3 × UP006 (non-pep585-annotation)
- docs/_ext/aafig.py:
     7 × UP006 (non-pep585-annotation)
     2 × RUF052 (used-dummy-variable)
- docs/conf.py:
     6 × UP006 (non-pep585-annotation)
- src/tmuxp/_internal/config_reader.py:
     5 × UP006 (non-pep585-annotation)
     2 × TC006 (runtime-cast-value)
- src/tmuxp/_internal/types.py:
     3 × UP006 (non-pep585-annotation)
     1 × F401 (unused-import)
- src/tmuxp/cli/__init__.py:
     1 × UP006 (non-pep585-annotation)
- src/tmuxp/cli/debug_info.py:
     2 × UP006 (non-pep585-annotation)
- src/tmuxp/cli/freeze.py:
     2 × UP006 (non-pep585-annotation)
     1 × RUF052 (used-dummy-variable)
     1 × TC006 (runtime-cast-value)
- src/tmuxp/cli/import_config.py:
     2 × UP006 (non-pep585-annotation)
- src/tmuxp/cli/load.py:
     3 × UP006 (non-pep585-annotation)
- src/tmuxp/cli/utils.py:
     6 × UP006 (non-pep585-annotation)
     5 × RUF052 (used-dummy-variable)
- src/tmuxp/log.py:
     1 × UP006 (non-pep585-annotation)
- src/tmuxp/plugin.py:
     5 × UP006 (non-pep585-annotation)
- src/tmuxp/shell.py:
     7 × UP006 (non-pep585-annotation)
- src/tmuxp/workspace/builder.py:
     8 × UP006 (non-pep585-annotation)
- src/tmuxp/workspace/finders.py:
     4 × UP006 (non-pep585-annotation)
     1 × PLC1802 (len-test)
- src/tmuxp/workspace/freezer.py:
     5 × UP006 (non-pep585-annotation)
- src/tmuxp/workspace/importers.py:
     4 × UP006 (non-pep585-annotation)
- src/tmuxp/workspace/loader.py:
     6 × UP006 (non-pep585-annotation)
- tests/cli/test_cli.py:
     1 × UP006 (non-pep585-annotation)
- tests/cli/test_convert.py:
     2 × UP006 (non-pep585-annotation)
     1 × F401 (unused-import)
- tests/cli/test_freeze.py:
     4 × UP006 (non-pep585-annotation)
- tests/cli/test_import.py:
     5 × UP006 (non-pep585-annotation)
     1 × F401 (unused-import)
- tests/cli/test_load.py:
    16 × UP006 (non-pep585-annotation)
- tests/cli/test_shell.py:
    19 × UP006 (non-pep585-annotation)
- tests/fixtures/pluginsystem/partials/_types.py:
     3 × UP006 (non-pep585-annotation)
     1 × F401 (unused-import)
- tests/fixtures/utils.py:
     2 × RUF052 (used-dummy-variable)
- tests/fixtures/workspace/expand1.py:
     1 × UP006 (non-pep585-annotation)
- tests/fixtures/workspace/shell_command_before.py:
     2 × UP006 (non-pep585-annotation)
- tests/workspace/test_builder.py:
    11 × RUF052 (used-dummy-variable)
     2 × UP006 (non-pep585-annotation)
- tests/workspace/test_config.py:
     1 × UP006 (non-pep585-annotation)
- tests/workspace/test_import_teamocil.py:
     7 × UP006 (non-pep585-annotation)
- tests/workspace/test_import_tmuxinator.py:
     2 × UP006 (non-pep585-annotation)

Found 1371 errors (174 fixed, 1197 remaining).
2 files reformatted, 97 files left unchanged

Author: tony

conftest.py

@@ -84,7 +84,7 @@ def monkeypatch_plugin_test_packages(monkeypatch: pytest.MonkeyPatch) -> None:
 
 
 @pytest.fixture
-def session_params(session_params: t.Dict[str, t.Any]) -> t.Dict[str, t.Any]:
+def session_params(session_params: dict[str, t.Any]) -> dict[str, t.Any]:
     """Terminal-friendly tmuxp session_params for dimensions."""
     session_params.update({"x": 800, "y": 600})
     return session_params
@@ -99,7 +99,7 @@ def socket_name(request: pytest.FixtureRequest) -> str:
 @pytest.fixture(autouse=True)
 def add_doctest_fixtures(
     request: pytest.FixtureRequest,
-    doctest_namespace: t.Dict[str, t.Any],
+    doctest_namespace: dict[str, t.Any],
     tmp_path: pathlib.Path,
 ) -> None:
     """Harness pytest fixtures to doctests namespace."""

docs/_ext/aafig.py

@@ -40,9 +40,9 @@
 
 
 def merge_dict(
-    dst: t.Dict[str, t.Optional[str]],
-    src: t.Dict[str, t.Optional[str]],
-) -> t.Dict[str, t.Optional[str]]:
+    dst: dict[str, t.Optional[str]],
+    src: dict[str, t.Optional[str]],
+) -> dict[str, t.Optional[str]]:
     for k, v in src.items():
         if k not in dst:
             dst[k] = v
@@ -51,15 +51,15 @@ def merge_dict(
 
 def get_basename(
     text: str,
-    options: t.Dict[str, str],
+    options: dict[str, str],
     prefix: t.Optional[str] = "aafig",
 ) -> str:
     options = options.copy()
     if "format" in options:
         del options["format"]
     hashkey = text + str(options)
-    _id = sha(hashkey.encode("utf-8")).hexdigest()
-    return f"{prefix}-{_id}"
+    id_ = sha(hashkey.encode("utf-8")).hexdigest()
+    return f"{prefix}-{id_}"
 
 
 class AafigError(SphinxError):
@@ -83,7 +83,7 @@ class AafigDirective(images.Image):  # type:ignore
     option_spec = images.Image.option_spec.copy()
     option_spec.update(own_option_spec)
 
-    def run(self) -> t.List[nodes.Node]:
+    def run(self) -> list[nodes.Node]:
         aafig_options = {}
         own_options_keys = [self.own_option_spec.keys(), "scale"]
         for k, v in self.options.items():
@@ -120,13 +120,13 @@ def render_aafig_images(app: "Sphinx", doctree: nodes.Node) -> None:
             continue
         options = img.aafig["options"]
         text = img.aafig["text"]
-        _format = app.builder.format
+        format_ = app.builder.format
         merge_dict(options, app.builder.config.aafig_default_options)
-        if _format in format_map:
-            options["format"] = format_map[_format]
+        if format_ in format_map:
+            options["format"] = format_map[format_]
         else:
             logger.warning(
-                f'unsupported builder format "{_format}", please '
+                f'unsupported builder format "{format_}", please '
                 "add a custom entry in aafig_format config "
                 "option for this builder",
             )
@@ -159,8 +159,8 @@ def __init__(self, *args: object, **kwargs: object) -> None:
 def render_aafigure(
     app: "Sphinx",
     text: str,
-    options: t.Dict[str, str],
-) -> t.Tuple[str, str, t.Optional[str], t.Optional[str]]:
+    options: dict[str, str],
+) -> tuple[str, str, t.Optional[str], t.Optional[str]]:
     """Render an ASCII art figure into the requested format output file."""
     if aafigure is None:
         raise AafigureNotInstalled

docs/conf.py

@@ -22,7 +22,7 @@
 sys.path.insert(0, str(cwd / "_ext"))
 
 # package data
-about: t.Dict[str, str] = {}
+about: dict[str, str] = {}
 with (src_root / "tmuxp" / "__about__.py").open() as fp:
     exec(fp.read(), about)
 
@@ -73,8 +73,8 @@
 html_static_path = ["_static"]
 html_favicon = "_static/favicon.ico"
 html_theme = "furo"
-html_theme_path: t.List[str] = []
-html_theme_options: t.Dict[str, t.Union[str, t.List[t.Dict[str, str]]]] = {
+html_theme_path: list[str] = []
+html_theme_options: dict[str, t.Union[str, list[dict[str, str]]]] = {
     "light_logo": "img/tmuxp.svg",
     "dark_logo": "img/tmuxp.svg",
     "footer_icons": [
@@ -143,7 +143,7 @@
 }
 
 
-def linkcode_resolve(domain: str, info: t.Dict[str, str]) -> t.Union[None, str]:
+def linkcode_resolve(domain: str, info: dict[str, str]) -> t.Union[None, str]:
     """
     Determine the URL corresponding to Python object.
 

src/tmuxp/_internal/config_reader.py

@@ -11,7 +11,7 @@
 
     FormatLiteral = t.Literal["json", "yaml"]
 
-    RawConfigData: TypeAlias = t.Dict[t.Any, t.Any]
+    RawConfigData: TypeAlias = dict[t.Any, t.Any]
 
 
 class ConfigReader:
@@ -28,7 +28,7 @@ def __init__(self, content: "RawConfigData") -> None:
         self.content = content
 
     @staticmethod
-    def _load(fmt: "FormatLiteral", content: str) -> t.Dict[str, t.Any]:
+    def _load(fmt: "FormatLiteral", content: str) -> dict[str, t.Any]:
         """Load raw config data and directly return it.
 
         >>> ConfigReader._load("json", '{ "session_name": "my session" }')
@@ -39,14 +39,14 @@ def _load(fmt: "FormatLiteral", content: str) -> t.Dict[str, t.Any]:
         """
         if fmt == "yaml":
             return t.cast(
-                t.Dict[str, t.Any],
+                "dict[str, t.Any]",
                 yaml.load(
                     content,
                     Loader=yaml.SafeLoader,
                 ),
             )
         if fmt == "json":
-            return t.cast(t.Dict[str, t.Any], json.loads(content))
+            return t.cast("dict[str, t.Any]", json.loads(content))
         msg = f"{fmt} not supported in configuration"
         raise NotImplementedError(msg)
 
@@ -74,7 +74,7 @@ def load(cls, fmt: "FormatLiteral", content: str) -> "ConfigReader":
         )
 
     @classmethod
-    def _from_file(cls, path: pathlib.Path) -> t.Dict[str, t.Any]:
+    def _from_file(cls, path: pathlib.Path) -> dict[str, t.Any]:
         r"""Load data from file path directly to dictionary.
 
         **YAML file**

src/tmuxp/_internal/types.py

@@ -10,19 +10,17 @@
 ...
 """
 
-import typing as t
-
 from typing_extensions import NotRequired, TypedDict
 
 
 class PluginConfigSchema(TypedDict):
     plugin_name: NotRequired[str]
     tmux_min_version: NotRequired[str]
     tmux_max_version: NotRequired[str]
-    tmux_version_incompatible: NotRequired[t.List[str]]
+    tmux_version_incompatible: NotRequired[list[str]]
     libtmux_min_version: NotRequired[str]
     libtmux_max_version: NotRequired[str]
-    libtmux_version_incompatible: NotRequired[t.List[str]]
+    libtmux_version_incompatible: NotRequired[list[str]]
     tmuxp_min_version: NotRequired[str]
     tmuxp_max_version: NotRequired[str]
-    tmuxp_version_incompatible: NotRequired[t.List[str]]
+    tmuxp_version_incompatible: NotRequired[list[str]]

src/tmuxp/cli/__init__.py

@@ -118,7 +118,7 @@ class CLINamespace(argparse.Namespace):
 ns = CLINamespace()
 
 
-def cli(_args: t.Optional[t.List[str]] = None) -> None:
+def cli(_args: t.Optional[list[str]] = None) -> None:
     """Manage tmux sessions.
 
     Pass the "--help" argument to any command to see detailed help.

src/tmuxp/cli/debug_info.py

@@ -31,7 +31,7 @@ def command_debug_info(
 ) -> None:
     """Entrypoint for ``tmuxp debug-info`` to print debug info to submit with issues."""
 
-    def prepend_tab(strings: t.List[str]) -> t.List[str]:
+    def prepend_tab(strings: list[str]) -> list[str]:
         """Prepend tab to strings in list."""
         return [f"\t{x}" for x in strings]
 

src/tmuxp/cli/freeze.py

@@ -165,7 +165,7 @@ def command_freeze(
     dest = os.path.abspath(os.path.relpath(os.path.expanduser(dest)))
     workspace_format = args.workspace_format
 
-    valid_workspace_formats: t.List[CLIOutputFormatLiteral] = ["json", "yaml"]
+    valid_workspace_formats: list[CLIOutputFormatLiteral] = ["json", "yaml"]
 
     def is_valid_ext(stem: t.Optional[str]) -> "TypeGuard[CLIOutputFormatLiteral]":
         return stem in valid_workspace_formats
@@ -184,15 +184,15 @@ def extract_workspace_format(
 
         workspace_format = extract_workspace_format(dest)
         if not is_valid_ext(workspace_format):
-            _workspace_format = prompt_choices(
+            workspace_format_ = prompt_choices(
                 "Couldn't ascertain one of [{}] from file name. Convert to".format(
                     ", ".join(valid_workspace_formats),
                 ),
-                choices=t.cast(t.List[str], valid_workspace_formats),
+                choices=t.cast("list[str]", valid_workspace_formats),
                 default="yaml",
             )
-            assert is_valid_ext(_workspace_format)
-            workspace_format = _workspace_format
+            assert is_valid_ext(workspace_format_)
+            workspace_format = workspace_format_
 
     if workspace_format == "yaml":
         workspace = configparser.dump(

src/tmuxp/cli/import_config.py

@@ -128,7 +128,7 @@ def create_import_subparser(
 class ImportConfigFn(t.Protocol):
     """Typing for import configuration callback function."""
 
-    def __call__(self, workspace_dict: t.Dict[str, t.Any]) -> t.Dict[str, t.Any]:
+    def __call__(self, workspace_dict: dict[str, t.Any]) -> dict[str, t.Any]:
         """Execute tmuxp import function."""
         ...
 

src/tmuxp/cli/load.py

@@ -36,7 +36,7 @@ class OptionOverrides(TypedDict):
 class CLILoadNamespace(argparse.Namespace):
     """Typed :class:`argparse.Namespace` for tmuxp load command."""
 
-    workspace_files: t.List[str]
+    workspace_files: list[str]
     socket_name: t.Optional[str]
     socket_path: t.Optional[str]
     tmux_config_file: t.Optional[str]
@@ -47,7 +47,7 @@ class CLILoadNamespace(argparse.Namespace):
     log_file: t.Optional[str]
 
 
-def load_plugins(session_config: t.Dict[str, t.Any]) -> t.List[t.Any]:
+def load_plugins(session_config: dict[str, t.Any]) -> list[t.Any]:
     """Load and return plugins in workspace."""
     plugins = []
     if "plugins" in session_config: