chore(ruff): Run automated fixes (97 errors)

ruff --show-fixes --fix .

- conftest.py:
    1 × I001 (unsorted-imports)
- docs/_ext/aafig.py:
    5 × C408 (unnecessary-collection-call)
    1 × RUF005 (collection-literal-concatenation)
- docs/conf.py:
    2 × C408 (unnecessary-collection-call)
    1 × I001 (unsorted-imports)
    1 × SIM108 (if-else-block-instead-of-if-exp)
    1 × RUF100 (unused-noqa)
- src/tmuxp/cli/debug_info.py:
    1 × I001 (unsorted-imports)
    1 × C417 (unnecessary-map)
- src/tmuxp/cli/freeze.py:
    1 × I001 (unsorted-imports)
- src/tmuxp/cli/load.py:
    1 × I001 (unsorted-imports)
    1 × UP031 (printf-string-formatting)
    1 × UP032 (f-string)
- src/tmuxp/cli/shell.py:
    1 × RUF100 (unused-noqa)
- src/tmuxp/cli/utils.py:
    1 × UP031 (printf-string-formatting)
    1 × UP032 (f-string)
- src/tmuxp/exc.py:
    1 × UP032 (f-string)
- src/tmuxp/shell.py:
    4 × RUF100 (unused-noqa)
- src/tmuxp/types.py:
    1 × RUF100 (unused-noqa)
- src/tmuxp/util.py:
    2 × RUF100 (unused-noqa)
    1 × RUF015 (unnecessary-iterable-allocation-for-first-element)
    1 × TRY201 (verbose-raise)
- src/tmuxp/workspace/builder.py:
    3 × SIM401 (if-else-block-instead-of-dict-get)
    1 × F841 (unused-variable)
    1 × B006 (mutable-argument-default)
    1 × RUF013 (implicit-optional)
    1 × TRY201 (verbose-raise)
- src/tmuxp/workspace/finders.py:
    2 × B006 (mutable-argument-default)
    1 × RUF013 (implicit-optional)
- src/tmuxp/workspace/freezer.py:
    1 × I001 (unsorted-imports)
    1 × UP018 (native-literals)
- src/tmuxp/workspace/importers.py:
    2 × B007 (unused-loop-control-variable)
    1 × SIM401 (if-else-block-instead-of-dict-get)
- src/tmuxp/workspace/loader.py:
    2 × SIM401 (if-else-block-instead-of-dict-get)
    1 × UP018 (native-literals)
    1 × SIM102 (collapsible-if)
- src/tmuxp/workspace/validation.py:
    1 × SIM102 (collapsible-if)
- tests/cli/test_cli.py:
    2 × I001 (unsorted-imports)
    2 × SIM105 (suppressible-exception)
    1 × PTH201 (path-constructor-current-directory)
- tests/cli/test_convert.py:
    2 × SIM105 (suppressible-exception)
    1 × I001 (unsorted-imports)
- tests/cli/test_freeze.py:
    2 × I001 (unsorted-imports)
    2 × SIM105 (suppressible-exception)
    2 × RUF005 (collection-literal-concatenation)
- tests/cli/test_import.py:
    2 × SIM105 (suppressible-exception)
    1 × I001 (unsorted-imports)
- tests/cli/test_load.py:
    5 × SIM105 (suppressible-exception)
    2 × I001 (unsorted-imports)
    1 × RUF005 (collection-literal-concatenation)
    1 × UP032 (f-string)
- tests/cli/test_ls.py:
    1 × SIM105 (suppressible-exception)
    1 × I001 (unsorted-imports)
- tests/cli/test_shell.py:
    3 × C408 (unnecessary-collection-call)
    2 × I001 (unsorted-imports)
    1 × SIM105 (suppressible-exception)
- tests/fixtures/__init__.py:
    1 × RUF100 (unused-noqa)
- tests/fixtures/import_teamocil/__init__.py:
    1 × RUF100 (unused-noqa)
- tests/fixtures/import_tmuxinator/__init__.py:
    1 × RUF100 (unused-noqa)
- tests/fixtures/workspace/__init__.py:
    1 × RUF100 (unused-noqa)
- tests/tests/test_helpers.py:
    1 × I001 (unsorted-imports)
- tests/workspace/test_builder.py:
    1 × I001 (unsorted-imports)
    1 × SIM300 (yoda-conditions)
    1 × SIM108 (if-else-block-instead-of-if-exp)
    1 × UP030 (format-literals)
    1 × UP032 (f-string)
- tests/workspace/test_config.py:
    2 × B007 (unused-loop-control-variable)

Author: tony

conftest.py

@@ -10,10 +10,9 @@
 import typing as t
 
 import pytest
-
 from _pytest.doctest import DoctestItem
-
 from libtmux.test import namer
+
 from tests.fixtures import utils as test_utils
 from tmuxp.workspace.finders import get_workspace_dir
 

docs/_ext/aafig.py

@@ -28,7 +28,7 @@
 
 logger = logging.getLogger(__name__)
 
-DEFAULT_FORMATS = dict(html="svg", latex="pdf", text=None)
+DEFAULT_FORMATS = {"html": "svg", "latex": "pdf", "text": None}
 
 
 def merge_dict(dst, src):
@@ -58,21 +58,21 @@ class AafigDirective(images.Image):
 
     has_content = True
     required_arguments = 0
-    own_option_spec = dict(
-        line_width=float,
-        background=str,
-        foreground=str,
-        fill=str,
-        aspect=nonnegative_int,
-        textual=flag,
-        proportional=flag,
-    )
+    own_option_spec = {
+        "line_width": float,
+        "background": str,
+        "foreground": str,
+        "fill": str,
+        "aspect": nonnegative_int,
+        "textual": flag,
+        "proportional": flag,
+    }
     option_spec = images.Image.option_spec.copy()
     option_spec.update(own_option_spec)
 
     def run(self):
-        aafig_options = dict()
-        own_options_keys = [self.own_option_spec.keys()] + ["scale"]
+        aafig_options = {}
+        own_options_keys = [self.own_option_spec.keys(), "scale"]
         for (k, v) in self.options.items():
             if k in own_options_keys:
                 # convert flags to booleans
@@ -88,7 +88,7 @@ def run(self):
         if isinstance(image_node, nodes.system_message):
             return [image_node]
         text = "\n".join(self.content)
-        image_node.aafig = dict(options=aafig_options, text=text)
+        image_node.aafig = {"options": aafig_options, "text": text}
         return [image_node]
 
 
@@ -206,7 +206,7 @@ def setup(app):
     app.add_directive("aafig", AafigDirective)
     app.connect("doctree-read", render_aafig_images)
     app.add_config_value("aafig_format", DEFAULT_FORMATS, "html")
-    app.add_config_value("aafig_default_options", dict(), "html")
+    app.add_config_value("aafig_default_options", {}, "html")
 
 
 # vim: set expandtab shiftwidth=4 softtabstop=4 :

docs/conf.py

@@ -1,10 +1,10 @@
 # flake8: NOQA: E501
 import contextlib
 import inspect
+import pathlib
 import sys
 import typing as t
 from os.path import relpath
-import pathlib
 
 import tmuxp
 
@@ -172,12 +172,12 @@
 }
 
 # aafig format, try to get working with pdf
-aafig_format = dict(latex="pdf", html="gif")
+aafig_format = {"latex": "pdf", "html": "gif"}
 
-aafig_default_options = dict(scale=0.75, aspect=0.5, proportional=True)
+aafig_default_options = {"scale": 0.75, "aspect": 0.5, "proportional": True}
 
 
-def linkcode_resolve(domain, info):  # NOQA: C901
+def linkcode_resolve(domain, info):
     """
     Determine the URL corresponding to Python object
 
@@ -224,10 +224,7 @@ def linkcode_resolve(domain, info):  # NOQA: C901
     except Exception:
         lineno = None
 
-    if lineno:
-        linespec = "#L%d-L%d" % (lineno, lineno + len(source) - 1)
-    else:
-        linespec = ""
+    linespec = "#L%d-L%d" % (lineno, lineno + len(source) - 1) if lineno else ""
 
     fn = relpath(fn, start=pathlib.Path(tmuxp.__file__).parent)
 

src/tmuxp/cli/debug_info.py

@@ -7,7 +7,6 @@
 import typing as t
 
 from colorama import Fore
-
 from libtmux.__about__ import __version__ as libtmux_version
 from libtmux.common import get_version, tmux_cmd
 
@@ -34,7 +33,7 @@ def prepend_tab(strings):
         """
         Prepend tab to strings in list.
         """
-        return list(map(lambda x: "\t%s" % x, strings))
+        return ["\t%s" % x for x in strings]
 
     def output_break():
         """

src/tmuxp/cli/freeze.py

@@ -5,6 +5,7 @@
 import typing as t
 
 from libtmux.server import Server
+
 from tmuxp.config_reader import ConfigReader
 from tmuxp.exc import TmuxpException
 from tmuxp.workspace.finders import get_workspace_dir

src/tmuxp/cli/load.py

@@ -16,6 +16,7 @@
 from libtmux.common import has_gte_version
 from libtmux.server import Server
 from libtmux.session import Session
+
 from tmuxp.types import StrPath
 
 from .. import config_reader, exc, log, util
@@ -90,11 +91,7 @@ def set_layout_hook(session: Session, hook_name: str) -> None:
         hook_cmd.append("selectw -p")
 
     # unset the hook immediately after executing
-    hook_cmd.append(
-        "set-hook -u -t {target_session} {hook_name}".format(
-            target_session=session.id, hook_name=hook_name
-        )
-    )
+    hook_cmd.append(f"set-hook -u -t {session.id} {hook_name}")
     hook_cmd.append(f"selectw -t {attached_window.id}")
 
     # join the hook's commands with semicolons
@@ -122,8 +119,9 @@ def load_plugins(session_config: t.Dict[str, t.Any]) -> t.List[t.Any]:
                 plugins.append(plugin())
             except exc.TmuxpPluginException as error:
                 if not prompt_yes_no(
-                    "%sSkip loading %s?"
-                    % (style(str(error), fg="yellow"), plugin_name),
+                    "{}Skip loading {}?".format(
+                        style(str(error), fg="yellow"), plugin_name
+                    ),
                     default=True,
                 ):
                     tmuxp_echo(

src/tmuxp/cli/shell.py

@@ -166,7 +166,7 @@ def command_shell(
         session=session, window_name=args.window_name, current_pane=current_pane
     )
 
-    pane = util.get_pane(window=window, current_pane=current_pane)  # NOQA: F841
+    pane = util.get_pane(window=window, current_pane=current_pane)
 
     if args.command is not None:
         exec(args.command)

src/tmuxp/cli/utils.py

@@ -126,7 +126,7 @@ def prompt_choices(
         if isinstance(choice, str):
             options.append(choice)
         elif isinstance(choice, tuple):
-            options.append("%s [%s]" % (choice, choice[0]))
+            options.append(f"{choice} [{choice[0]}]")
             choice = choice[0]
         _choices.append(choice)
 

src/tmuxp/exc.py

@@ -50,11 +50,11 @@ def __init__(
         self.cmd = cmd
         self.output = output
         self.message = (
-            "before_script failed with returncode {returncode}.\n"
-            "command: {cmd}\n"
+            f"before_script failed with returncode {self.returncode}.\n"
+            f"command: {self.cmd}\n"
             "Error output:\n"
-            "{output}"
-        ).format(returncode=self.returncode, cmd=self.cmd, output=self.output)
+            f"{self.output}"
+        )
 
     def __str__(self):
         return self.message

src/tmuxp/shell.py

@@ -32,7 +32,7 @@ def has_ipython() -> bool:
 
 def has_ptpython() -> bool:
     try:
-        from ptpython.repl import embed, run_config  # NOQA F841
+        from ptpython.repl import embed, run_config  # F841
     except ImportError:
         try:
             from prompt_toolkit.contrib.repl import embed, run_config  # NOQA F841
@@ -44,8 +44,8 @@ def has_ptpython() -> bool:
 
 def has_ptipython() -> bool:
     try:
-        from ptpython.ipython import embed  # NOQA F841
-        from ptpython.repl import run_config  # NOQA F841
+        from ptpython.ipython import embed  # F841
+        from ptpython.repl import run_config  # F841
     except ImportError:
         try:
             from prompt_toolkit.contrib.ipython import embed  # NOQA F841
@@ -80,7 +80,7 @@ def get_bpython(options, extra_args=None):
     if extra_args is None:
         extra_args = {}
 
-    from bpython import embed  # NOQA F841
+    from bpython import embed  # F841
 
     def launch_bpython():
         imported_objects = get_launch_args(**options)