python-linters 0.2.0__tar.gz → 0.2.1__tar.gz

{python_linters-0.2.0 → python_linters-0.2.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-linters
-Version: 0.2.0
+Version: 0.2.1
 Summary: Run ruff, ty and pymodularity on a Python project and print one JSON report grouped by rule.
 Author: Tilo Himmelsbach
 License-Expression: MIT

{python_linters-0.2.0 → python_linters-0.2.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "python-linters"
-version = "0.2.0"
+version = "0.2.1"
 description = "Run ruff, ty and pymodularity on a Python project and print one JSON report grouped by rule."
 readme = "README.md"
 license = "MIT"

{python_linters-0.2.0 → python_linters-0.2.1}/pyproject.toml.orig

@@ -1,6 +1,6 @@
 [project]
 name = "python-linters"
-version = "0.2.0"
+version = "0.2.1"
 description = "Run ruff, ty and pymodularity on a Python project and print one JSON report grouped by rule."
 readme = "README.md"
 license = "MIT"

{python_linters-0.2.0 → python_linters-0.2.1}/src/python_linters/config_check.py

@@ -169,7 +169,9 @@ def _covers_rule(
         return False
     if selector in meta_prefixes:
         return True
-    selector_linters = {r["linter"] for r in rules if r["code"] and r["code"].startswith(selector)}
+    selector_linters = {
+        r["linter"] for r in rules if r["code"] and r["code"].startswith(selector)
+    }
     return selector_linters == {linter}
 
 
@@ -221,9 +223,14 @@ def _get_new_categories(
     for prefix, linter in sorted(categories.items()):
         if any(_covers(k, prefix, category_linters, meta_prefixes) for k in known):
             continue
-        prefix_rules = [r for r in rules if r["code"] and r["code"].rstrip(string.digits) == prefix]
+        prefix_rules = [
+            r for r in rules if r["code"] and r["code"].rstrip(string.digits) == prefix
+        ]
         if all(
-            any(_covers_rule(k, r["code"], r["linter"], rules, meta_prefixes) for k in known)
+            any(
+                _covers_rule(k, r["code"], r["linter"], rules, meta_prefixes)
+                for k in known
+            )
             for r in prefix_rules
         ):
             continue
@@ -255,9 +262,13 @@ def check_ruff_config(toml_path: Path) -> RuffConfigCheck:
     commented = _parse_toml_comments(toml_path)
 
     known = selected | commented
-    meta_prefixes = {"PL"} | {p for p, linters in category_linters.items() if len(linters) > 1}
+    meta_prefixes = {"PL"} | {
+        p for p, linters in category_linters.items() if len(linters) > 1
+    }
 
-    new_categories = _get_new_categories(rules, categories, category_linters, known, meta_prefixes)
+    new_categories = _get_new_categories(
+        rules, categories, category_linters, known, meta_prefixes
+    )
     removed = _get_removed(rules, selected)
 
     untriaged_codeless = [n for n in _get_codeless_rules(rules) if n not in known]
@@ -293,24 +304,36 @@ def format_ruff_report(toml_path: Path, check: RuffConfigCheck) -> str:
     if check.new_categories:
         lines.append(f"NEW categories not yet triaged ({len(check.new_categories)}):")
         lines.append("  Add to select or comment out with '# excluded' in ruff.toml:\n")
-        lines.extend(f'    "{prefix}",\t# {linter}' for prefix, linter in check.new_categories)
+        lines.extend(
+            f'    "{prefix}",\t# {linter}' for prefix, linter in check.new_categories
+        )
         lines.append("")
 
     if check.removed:
-        lines.append(f"REMOVED/RENAMED categories still in select ({len(check.removed)}):")
+        lines.append(
+            f"REMOVED/RENAMED categories still in select ({len(check.removed)}):"
+        )
         lines.extend(f"    {code}" for code in check.removed)
         lines.append("")
 
     if check.untriaged_codeless:
-        lines.append(f"Code-less rules not yet triaged ({len(check.untriaged_codeless)}):")
-        lines.append("  Add to select by name or comment out with '# excluded' in ruff.toml:\n")
+        lines.append(
+            f"Code-less rules not yet triaged ({len(check.untriaged_codeless)}):"
+        )
+        lines.append(
+            "  Add to select by name or comment out with '# excluded' in ruff.toml:\n"
+        )
         lines.extend(f'    "{name}",' for name in check.untriaged_codeless)
         lines.append("")
 
     if check.untriaged_preview:
         lines.append(f"Preview rules not yet triaged ({len(check.untriaged_preview)}):")
-        lines.append("  Add to select by name or cover with an '# excluded' line in ruff.toml:\n")
-        lines.extend(f'    "{name}",\t# {code}' for code, name in check.untriaged_preview)
+        lines.append(
+            "  Add to select by name or cover with an '# excluded' line in ruff.toml:\n"
+        )
+        lines.extend(
+            f'    "{name}",\t# {code}' for code, name in check.untriaged_preview
+        )
         lines.append("")
 
     if check.is_clean:
@@ -395,7 +418,9 @@ def format_ty_report(config_path: Path, check: TyConfigCheck) -> str:
     ]
 
     if check.untriaged:
-        lines.append(f"UNTRIAGED rules ty knows that the config omits ({len(check.untriaged)}):")
+        lines.append(
+            f"UNTRIAGED rules ty knows that the config omits ({len(check.untriaged)}):"
+        )
         lines.extend(
             f'    {name} = "???"  # default: {check.default_levels[name]}'
             for name in check.untriaged
@@ -403,7 +428,9 @@ def format_ty_report(config_path: Path, check: TyConfigCheck) -> str:
         lines.append("")
 
     if check.stale:
-        lines.append(f"STALE rules named in config that ty no longer knows ({len(check.stale)}):")
+        lines.append(
+            f"STALE rules named in config that ty no longer knows ({len(check.stale)}):"
+        )
         lines.extend(f"    {name}" for name in check.stale)
         lines.append("")
 

{python_linters-0.2.0 → python_linters-0.2.1}/src/python_linters/lint_report.py

@@ -61,7 +61,12 @@ class Location:
     message: str
 
     def to_dict(self) -> dict[str, object]:
-        return {"file": self.file, "line": self.line, "col": self.col, "message": self.message}
+        return {
+            "file": self.file,
+            "line": self.line,
+            "col": self.col,
+            "message": self.message,
+        }
 
 
 @dataclass
@@ -123,7 +128,16 @@ def run_ruff(project_path: Path) -> dict[str, FindingGroup]:
     # selector, bad config, ...). Empty stdout on exit 2 must never read
     # as "0 findings" — `run_checked` raises instead of returning in that case.
     result = run_checked(
-        [sys.executable, "-m", "ruff", "check", "--no-cache", "--output-format", "json", "."],
+        [
+            sys.executable,
+            "-m",
+            "ruff",
+            "check",
+            "--no-cache",
+            "--output-format",
+            "json",
+            ".",
+        ],
         project_path,
         "ruff",
         ok_codes=(0, 1),
@@ -226,8 +240,15 @@ def run_pymodularity(project_path: Path) -> dict[str, FindingGroup]:
     # 2 `uv` missing or an uncaught exception (docs/rules.md "JSON output").
     # A failure must never read as "0 findings".
     result = run_checked(
-        [sys.executable, "-m", "pymodularity", "check", "--output-format", "json",
-         str(project_path)],
+        [
+            sys.executable,
+            "-m",
+            "pymodularity",
+            "check",
+            "--output-format",
+            "json",
+            str(project_path),
+        ],
         project_path,
         "pymodularity",
         ok_codes=(0, 1),
@@ -259,7 +280,11 @@ def build_report(project_path: Path) -> tuple[dict[str, object], int]:
     sections: dict[str, object] = {}
     grand_total = 0
 
-    for tool, run in (("ruff", run_ruff), ("ty", run_ty), ("pymodularity", run_pymodularity)):
+    for tool, run in (
+        ("ruff", run_ruff),
+        ("ty", run_ty),
+        ("pymodularity", run_pymodularity),
+    ):
         print(f"Running {tool} on {project_path} ...", file=sys.stderr)
         violations = run(project_path)
         total = sum(group.count for group in violations.values())

{python_linters-0.2.0 → python_linters-0.2.1}/src/python_linters/references/ruff.reference.toml

@@ -321,9 +321,9 @@ ignore = [
     # ANN401 (typing.Any disallowed) stays ON: with basedpyright dropped
     # in favor of ty, ANN401 is the gate's only coverage for explicit
     # Any (basedpyright's reportExplicitAny): ty has no equivalent rule.
-    # missing return type on __init__, __enter__, __exit__ — always obvious
-    # from context
-    "missing-return-type-special-method",
+    # missing-return-type-special-method stays ON: ty types the target of
+    # `with X() as x` as Unknown when `__enter__` has no return type. ruff
+    # fixes `__init__ -> None` and `__str__ -> str` itself (safe fix).
     "missing-type-args",  # missing type on *args — typically pass-through,
                           #   type depends on caller
     "missing-type-kwargs",  # missing type on **kwargs — same reasoning as
@@ -339,18 +339,18 @@ ignore = [
                               #   than max_snr
 
     # ── Code style ──
-    "lambda-assignment",  # `transform = lambda x: x.strip()` is sometimes
-                          #   clearer than a named function for one-off local
-                          #   transforms
+    # lambda-assignment stays ON: a lambda cannot annotate its parameters,
+    # so ty types `strip = lambda x: x.strip()` as `(x) -> Unknown` and
+    # accepts `strip(1)`.
     "in-dict-keys",  # use `key in dict` — `key in dict.keys()` is more
                      #   explicit about intent; Python Zen says explicit >
                      #   implicit
     "superfluous-else-return",  # the else branch makes the two paths visually
                                 #   symmetric and easier to read
     "superfluous-else-raise",  # same reasoning as superfluous-else-return
-    # super() issues with dataclass(slots=True), see
-    # https://stackoverflow.com/questions/73268995
-    "super-call-with-parameters",
+    # super-call-with-parameters stays ON: zero-argument super() fails in a
+    # dataclass(slots=True) before Python 3.14, but ruff skips slots
+    # dataclasses for this rule (checked on ruff 0.17.3).
     "compare-to-empty-string",  # `s == ""` is more explicit than `not s` (the
                                 #   latter also matches None, 0, [])
 
@@ -396,6 +396,11 @@ ignore = [
     "unused-async",  # async test doubles mirror the awaited signature they replace
     "float-equality-comparison",  # asserting an exactly representable result is correct
     "too-many-locals",  # an integration test sets up many real collaborators
+    "missing-type-function-argument",  # pytest fixtures: the fixture defines the type
+    "missing-return-type-private-function",  # helpers and fakes local to a test module
+    "missing-return-type-special-method",  # `__init__` / `__enter__` on test doubles
+    "import-outside-top-level",  # imports inside a test patch or isolate the import
+    "private-member-access",  # tests inspect internal state on purpose
 ]
 "test_*.py" = [
     "assert",
@@ -409,6 +414,11 @@ ignore = [
     "unused-async",
     "float-equality-comparison",
     "too-many-locals",
+    "missing-type-function-argument",
+    "missing-return-type-private-function",
+    "missing-return-type-special-method",
+    "import-outside-top-level",
+    "private-member-access",
 ]
 
 # ── isort ───────────────────────────────────────────────────────────────────