chore(ruff): fixes from ruff 0.4.5

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

Fixed 29 errors:
- conftest.py:
    1 × UP037 (quoted-annotation)
- src/tmuxp/_internal/config_reader.py:
    1 × UP037 (quoted-annotation)
- src/tmuxp/cli/convert.py:
    1 × UP037 (quoted-annotation)
- src/tmuxp/cli/debug_info.py:
    3 × COM812 (missing-trailing-comma)
- src/tmuxp/cli/freeze.py:
    1 × UP037 (quoted-annotation)
    1 × COM812 (missing-trailing-comma)
- src/tmuxp/cli/load.py:
    1 × COM812 (missing-trailing-comma)
- src/tmuxp/plugin.py:
    1 × UP037 (quoted-annotation)
- src/tmuxp/shell.py:
    6 × PGH004 (blanket-noqa)
- src/tmuxp/util.py:
    1 × PGH004 (blanket-noqa)
- src/tmuxp/workspace/builder.py:
    1 × UP037 (quoted-annotation)
- tests/fixtures/pluginsystem/partials/all_pass.py:
    1 × UP037 (quoted-annotation)
- tests/fixtures/pluginsystem/partials/libtmux_version_fail.py:
    3 × UP037 (quoted-annotation)
- tests/fixtures/pluginsystem/partials/tmux_version_fail.py:
    3 × UP037 (quoted-annotation)
- tests/fixtures/pluginsystem/partials/tmuxp_version_fail.py:
    3 × UP037 (quoted-annotation)
- tests/fixtures/pluginsystem/plugins/tmuxp_test_plugin_fail/tmuxp_test_plugin_fail/plugin.py:
    1 × UP037 (quoted-annotation)

Author: tony

conftest.py

@@ -105,7 +105,7 @@ def add_doctest_fixtures(
     """Harness pytest fixtures to doctests namespace."""
     if isinstance(request._pyfuncitem, DoctestItem) and shutil.which("tmux"):
         doctest_namespace["server"] = request.getfixturevalue("server")
-        session: "Session" = request.getfixturevalue("session")
+        session: Session = request.getfixturevalue("session")
         doctest_namespace["session"] = session
         doctest_namespace["window"] = session.active_window
         doctest_namespace["pane"] = session.active_pane

src/tmuxp/_internal/config_reader.py

@@ -107,7 +107,7 @@ def _from_file(cls, path: pathlib.Path) -> t.Dict[str, t.Any]:
         content = path.open().read()
 
         if path.suffix in {".yaml", ".yml"}:
-            fmt: "FormatLiteral" = "yaml"
+            fmt: FormatLiteral = "yaml"
         elif path.suffix == ".json":
             fmt = "json"
         else:

src/tmuxp/cli/convert.py

@@ -68,7 +68,7 @@ def command_convert(
 
     _, ext = os.path.splitext(workspace_file)
     ext = ext.lower()
-    to_filetype: "AllowedFileTypes"
+    to_filetype: AllowedFileTypes
     if ext == ".json":
         to_filetype = "yaml"
     elif ext in {".yaml", ".yml"}:

src/tmuxp/cli/debug_info.py

@@ -62,7 +62,7 @@ def format_tmux_resp(std_resp: tmux_cmd) -> str:
                         f"version: {platform.version()}",
                     ],
                 ),
-            )
+            ),
         ),
         output_break(),
         "python version: {}".format(" ".join(sys.version.split("\n"))),
@@ -78,10 +78,10 @@ def format_tmux_resp(std_resp: tmux_cmd) -> str:
         "tmux windows:\n{}".format(format_tmux_resp(tmux_cmd("list-windows"))),
         "tmux panes:\n{}".format(format_tmux_resp(tmux_cmd("list-panes"))),
         "tmux global options:\n{}".format(
-            format_tmux_resp(tmux_cmd("show-options", "-g"))
+            format_tmux_resp(tmux_cmd("show-options", "-g")),
         ),
         "tmux window options:\n{}".format(
-            format_tmux_resp(tmux_cmd("show-window-options", "-g"))
+            format_tmux_resp(tmux_cmd("show-window-options", "-g")),
         ),
     ]
 

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: t.List[CLIOutputFormatLiteral] = ["json", "yaml"]
 
     def is_valid_ext(stem: t.Optional[str]) -> "TypeGuard[CLIOutputFormatLiteral]":
         return stem in valid_workspace_formats
@@ -186,7 +186,7 @@ def extract_workspace_format(
         if not is_valid_ext(workspace_format):
             _workspace_format = prompt_choices(
                 "Couldn't ascertain one of [{}] from file name. Convert to".format(
-                    ", ".join(valid_workspace_formats)
+                    ", ".join(valid_workspace_formats),
                 ),
                 choices=t.cast(t.List[str], valid_workspace_formats),
                 default="yaml",

src/tmuxp/cli/load.py

@@ -344,7 +344,7 @@ def load_workspace(
             answer_yes
             or prompt_yes_no(
                 "{} is already running. Attach?".format(
-                    style(session_name, fg="green")
+                    style(session_name, fg="green"),
                 ),
                 default=True,
             )

src/tmuxp/plugin.py

@@ -152,7 +152,7 @@ def __init__(self, **kwargs: "Unpack[PluginConfigSchema]") -> None:
         self.libtmux_version = libtmux.__about__.__version__
         self.tmuxp_version = Version(__version__)
 
-        self.version_constraints: "TmuxpPluginVersionConstraints" = {
+        self.version_constraints: TmuxpPluginVersionConstraints = {
             "tmux": {
                 "version": self.tmux_version,
                 "vmin": config["tmux_min_version"],

src/tmuxp/shell.py

@@ -51,10 +51,10 @@ class LaunchImports(t.TypedDict):
 def has_ipython() -> bool:
     """Return True if ipython is installed."""
     try:
-        from IPython import start_ipython  # NOQA F841
+        from IPython import start_ipython  # NOQA: F841
     except ImportError:
         try:
-            from IPython.Shell import IPShell  # NOQA F841
+            from IPython.Shell import IPShell  # NOQA: F841
         except ImportError:
             return False
 
@@ -67,7 +67,7 @@ def has_ptpython() -> bool:
         from ptpython.repl import embed, run_config  # F841
     except ImportError:
         try:
-            from prompt_toolkit.contrib.repl import embed, run_config  # NOQA F841
+            from prompt_toolkit.contrib.repl import embed, run_config  # NOQA: F841
         except ImportError:
             return False
 
@@ -81,8 +81,8 @@ def has_ptipython() -> bool:
         from ptpython.repl import run_config  # F841
     except ImportError:
         try:
-            from prompt_toolkit.contrib.ipython import embed  # NOQA F841
-            from prompt_toolkit.contrib.repl import run_config  # NOQA F841
+            from prompt_toolkit.contrib.ipython import embed  # NOQA: F841
+            from prompt_toolkit.contrib.repl import run_config  # NOQA: F841
         except ImportError:
             return False
 
@@ -92,7 +92,7 @@ def has_ptipython() -> bool:
 def has_bpython() -> bool:
     """Return True if bpython is installed."""
     try:
-        from bpython import embed  # NOQA F841
+        from bpython import embed  # NOQA: F841
     except ImportError:
         return False
     return True

src/tmuxp/util.py

@@ -69,7 +69,7 @@ def oh_my_zsh_auto_title() -> None:
     if (
         "SHELL" in os.environ
         and "zsh" in os.environ.get("SHELL", "")
-        and os.path.exists(os.path.expanduser("~/.oh-my-zsh"))  # NOQA PTH110, PTH111
+        and os.path.exists(os.path.expanduser("~/.oh-my-zsh"))  # NOQA: PTH110, PTH111
         and (
             "DISABLE_AUTO_TITLE" not in os.environ
             or os.environ.get("DISABLE_AUTO_TITLE") == "false"

src/tmuxp/workspace/builder.py

@@ -271,7 +271,7 @@ def build(self, session: t.Optional[Session] = None, append: bool = False) -> No
 
         assert session.server is not None
 
-        self.server: "Server" = session.server
+        self.server: Server = session.server
         assert self.server.sessions is not None
         assert self.server.has_session(session.name)
         assert session.id