Fix ruff formatting for PR

- Format analysis.py and test_bypasses.py with ruff
- Remove mypy from pre-commit hooks (was already skipped in CI)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: dguido

fickling/analysis.py

@@ -249,18 +249,59 @@ class UnsafeImportsML(Analysis):
 
     # Builtins that are safe to import - pure functions and type constructors
     # that cannot be used for code execution or system access
-    SAFE_BUILTINS = frozenset([
-        # Type constructors (create data, cannot execute code)
-        "bool", "int", "float", "complex", "str", "bytes", "bytearray",
-        "list", "tuple", "set", "frozenset", "dict",
-        # Pure functions (no side effects, no code execution)
-        "len", "abs", "sum", "min", "max", "round", "pow", "divmod",
-        "sorted", "reversed", "enumerate", "zip", "range",
-        "map", "filter", "slice", "iter", "next",
-        "all", "any", "hash", "id", "repr", "ascii", "bin", "hex", "oct", "ord", "chr",
-        "isinstance", "issubclass", "type", "object",
-        "callable", "format",
-    ])
+    SAFE_BUILTINS = frozenset(
+        [
+            # Type constructors (create data, cannot execute code)
+            "bool",
+            "int",
+            "float",
+            "complex",
+            "str",
+            "bytes",
+            "bytearray",
+            "list",
+            "tuple",
+            "set",
+            "frozenset",
+            "dict",
+            # Pure functions (no side effects, no code execution)
+            "len",
+            "abs",
+            "sum",
+            "min",
+            "max",
+            "round",
+            "pow",
+            "divmod",
+            "sorted",
+            "reversed",
+            "enumerate",
+            "zip",
+            "range",
+            "map",
+            "filter",
+            "slice",
+            "iter",
+            "next",
+            "all",
+            "any",
+            "hash",
+            "id",
+            "repr",
+            "ascii",
+            "bin",
+            "hex",
+            "oct",
+            "ord",
+            "chr",
+            "isinstance",
+            "issubclass",
+            "type",
+            "object",
+            "callable",
+            "format",
+        ]
+    )
 
     # Builtins that are DANGEROUS and must remain blocked
     # (for documentation - these are NOT in SAFE_BUILTINS)

test/test_bypasses.py

@@ -426,8 +426,7 @@ def test_unsafe_builtins_still_flagged(self):
         # Should be flagged by at least one of the unsafe import checkers
         detailed = res.detailed_results().get("AnalysisResult", {})
         has_unsafe_import = (
-            detailed.get("UnsafeImports") is not None
-            or detailed.get("UnsafeImportsML") is not None
+            detailed.get("UnsafeImports") is not None or detailed.get("UnsafeImportsML") is not None
         )
         self.assertTrue(has_unsafe_import)