chore(lint): enable flake8-boolean-trap ruff rules

- FBT001: Boolean positional arg in function definition
- FBT002: Boolean default value in function definition
- FBT003: Boolean positional value in function call

https://beta.ruff.rs/docs/rules/#flake8-boolean-trap-fbt

Author: László Vaskó

pyproject.toml

@@ -68,6 +68,7 @@ select = [
   "D",      # pydocstyle
   "E",      # pycodestyle (errors)
   "F",      # pyflakes
+  "FBT",    # flake8-boolean-trap
   "I",      # isort
   "ISC",    # flake8-implicit-str-concats
   "N",      # pep8-naming
@@ -108,6 +109,10 @@ ignore-init-module-imports = true
 "build.py" = [
   "T201"  # print
 ]
+"tests/*" = [
+  "FBT001",  # boolean-positional-arg-in-function-definition: allowed in `pytest.param()`
+  "FBT003",  # boolean-positional-value-in-function-call:     allowed test parameters
+]
 
 [tool.ruff.flake8-comprehensions]
 allow-dict-calls-with-keyword-arguments = true  # code like `dict(a=1, b=2)` is allowed

unblob/cli.py

@@ -26,7 +26,7 @@
 
 
 def show_external_dependencies(
-    ctx: click.Context, _param: click.Option, value: bool
+    ctx: click.Context, _param: click.Option, value: bool  # noqa: FBT001
 ) -> None:
     if not value or ctx.resilient_parsing:
         return
@@ -179,13 +179,13 @@ def cli(
     file: Path,
     extract_root: Path,
     report_file: Optional[Path],
-    force: bool,
+    force: bool,  # noqa: FBT001
     process_num: int,
     depth: int,
     entropy_depth: int,
     skip_magic: Iterable[str],
-    skip_extraction: bool,
-    keep_extracted_chunks: bool,
+    skip_extraction: bool,  # noqa: FBT001
+    keep_extracted_chunks: bool,  # noqa: FBT001
     handlers: Handlers,
     plugins_path: Optional[Path],
     plugin_manager: UnblobPluginManager,

unblob/handlers/archive/_safe_tarfile.py

@@ -12,7 +12,9 @@
 
 
 class SafeTarFile(TarFile):
-    def extract(self, member, path="", set_attrs=True, *, numeric_owner=False):
+    def extract(
+        self, member, path="", set_attrs=True, *, numeric_owner=False  # noqa: FBT002
+    ):
         path_as_path = Path(str(path))
         member_name_path = Path(str(member.name))