fix: restore ruff baseline ignores for hatch fmt CI compatibility

Author: anticomputer

pyproject.toml

@@ -165,37 +165,90 @@ exclude_lines = [
 target-version = "py310"
 
 [tool.ruff.lint]
-# Project-wide style choices and pragmatic suppressions.
-# Rules removed from the original baseline are now enforced.
+# Baseline suppressions carried from the original codebase.
+# Many of these fire in pre-existing MCP server code or reflect
+# deliberate style choices.  Rules can be tightened incrementally.
 ignore = [
-  # Style choices — these are deliberate project conventions
-  "EM101",    # Exception string literals (pragmatic for this codebase)
-  "EM102",    # Exception f-strings (pragmatic for this codebase)
-  "G004",     # Logging f-strings (clearer than % formatting)
-  "T201",     # print() used intentionally for user output
-  "TRY003",   # Raise with inline message strings (pragmatic)
+  # Style choices — deliberate project conventions
+  "EM101",    # Exception string literals
+  "EM102",    # Exception f-strings
+  "G004",     # Logging f-strings
+  "T201",     # print() used for user output
+  "TRY003",   # Raise with inline message strings
 
   # Backwards-compatibility suppressions for existing code
-  "A001",     # Variable shadows built-in (existing API names)
-  "A002",     # Argument shadows built-in (existing API signatures)
-  "FBT001",   # Boolean positional arg (existing API)
-  "FBT002",   # Boolean default value (existing API)
-  "N802",     # Function name casing (existing API: get_AI_endpoint etc.)
-  "N806",     # Variable casing (existing code conventions)
-  "SLF001",   # Private member access (needed for MCP wrapper internals)
+  "A001",     # Variable shadows built-in
+  "A002",     # Argument shadows built-in
+  "A004",     # Import shadows built-in
+  "FBT001",   # Boolean positional arg
+  "FBT002",   # Boolean default value
+  "N801",     # Class name casing
+  "N802",     # Function name casing
+  "N806",     # Variable casing
+  "N818",     # Exception name suffix
+  "SLF001",   # Private member access (MCP internals)
 
   # Framework / ecosystem constraints
-  "ARG001",   # Unused function argument (required by hook/callback signatures)
-  "B023",     # Function uses loop variable (async closures in runner)
-  "INP001",   # Implicit namespace package (project uses src layout)
-  "PLW2901",  # Outer loop variable overwritten (iteration patterns)
-  "S701",     # Jinja2 autoescape=False (YAML context, not HTML)
+  "ARG001",   # Unused function argument (hook/callback signatures)
+  "B023",     # Function uses loop variable (async closures)
+  "INP001",   # Implicit namespace package (src layout)
+  "PLW2901",  # Outer loop variable overwritten
+  "S701",     # Jinja2 autoescape=False (YAML, not HTML)
+  "TID252",   # Relative imports from parent modules
 
-  # Low-signal rules for this project
+  # Import organisation (deferred imports used for cycle-breaking)
+  "I001",     # Import block unsorted
+  "PLC0414",  # Useless import alias (intentional re-exports)
+  "PLC0415",  # Import not at top-level (deferred imports)
+
+  # Exception handling patterns
+  "B904",     # raise without from in except
+  "BLE001",   # Blind except (Exception)
+  "TRY004",   # Prefer TypeError
+  "TRY300",   # try-except-return
+  "TRY301",   # Abstract raise to inner function
+  "TRY400",   # logging.exception vs logging.error
+
+  # Typing / annotation style
+  "FA100",    # Missing from __future__ import annotations
+  "FA102",    # Missing annotations import in stub
+  "PYI036",   # __exit__ signature
+  "PYI041",   # Use float instead of int | float
+  "TC001",    # Move import into TYPE_CHECKING
+  "TC002",    # Move import into TYPE_CHECKING
+  "TC003",    # Move import into TYPE_CHECKING
+  "UP006",    # Use X | Y for isinstance
+  "UP007",    # Use X | Y union type
+  "UP035",    # Import from collections.abc
+  "UP045",    # Use X | None
+
+  # Low-signal or noisy rules
+  "B006",     # Mutable default argument (pre-existing in MCP servers)
+  "B007",     # Unused loop control variable
+  "B008",     # Function call in argument defaults
+  "C416",     # Unnecessary comprehension
+  "LOG015",   # Root logger usage
+  "PERF102",  # Use keys()/values() instead of items()
   "PLR2004",  # Magic value comparisons
+  "PLW0602",  # Global variable not assigned
+  "PLW0603",  # Using global statement
+  "PLW1508",  # Invalid envvar default
+  "PT011",    # pytest.raises too broad
   "RET503",   # Missing explicit return
+  "RET504",   # Unnecessary assignment before return
   "RET505",   # Unnecessary else after return
-  "SIM102",   # Collapsible if (readability preference)
+  "RSE102",   # Unnecessary parentheses on raised exception
+  "RUF005",   # Unpack instead of concatenation
+  "RUF022",   # __all__ not sorted
+  "RUF023",   # __slots__ not sorted
+  "RUF059",   # Unused unpacked variable
+  "RUF100",   # Unused noqa directive
+  "S108",     # Hardcoded temp file
+  "S607",     # Partial path to executable
+  "SIM102",   # Collapsible if
+  "SIM115",   # Use context handler for file
+  "SIM210",   # Use ternary operator
+  "UP015",    # Unnecessary open mode
 ]
 
 [tool.ruff.lint.per-file-ignores]