sarj-python-lint 0.67.8__tar.gz → 0.67.9__tar.gz

{sarj_python_lint-0.67.8 → sarj_python_lint-0.67.9}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: sarj-python-lint
-Version: 0.67.8
+Version: 0.67.9
 Summary: Custom Python lint rules — AST-based, pre-commit-friendly, hypermodern defaults
 Project-URL: Homepage, https://code-standards.sarj.ai/rules/python/
 Project-URL: Documentation, https://code-standards.sarj.ai/rules/python/

{sarj_python_lint-0.67.8 → sarj_python_lint-0.67.9}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "sarj-python-lint"
-version = "0.67.8"
+version = "0.67.9"
 description = "Custom Python lint rules — AST-based, pre-commit-friendly, hypermodern defaults"
 readme = "README.md"
 authors = [{ name = "sarj-ai" }]

{sarj_python_lint-0.67.8 → sarj_python_lint-0.67.9}/src/sarj_python_lint/rules/docstring_args_restate_signature.py

@@ -69,12 +69,12 @@ class DocstringArgsRestateSignature(Rule):
             ),
             RuleExample(
                 example_id="argument-documents-unit",
-                title="Argument description records a unit",
+                title="Redundant argument section removed",
                 outcome=ExampleOutcome.NO_MATCH,
                 files=(
                     ExampleFile.python(
                         "app/widgets.py",
-                        'def set_timeout_ms(timeout_ms: int) -> None:\n    """Configure the request deadline.\n\n    Args:\n        timeout_ms: Timeout in ms\n    """\n',
+                        'def count_widgets(tenant_id: str) -> int:\n    """Count active widgets."""\n    return 0\n',
                     ),
                 ),
                 focus_path=PurePosixPath("app/widgets.py"),

{sarj_python_lint-0.67.8 → sarj_python_lint-0.67.9}/src/sarj_python_lint/rules/docstring_returns_restate_signature.py

@@ -135,7 +135,7 @@ class DocstringReturnsRestateSignature(Rule):
                 files=(
                     ExampleFile.python(
                         "app/lines.py",
-                        'def get_line_length(line: list[str]) -> int:\n    """Measure a rendered line.\n\n    Returns:\n        The width in terminal cells, which is not the character count.\n    """\n    return len(line)\n',
+                        'def get_line_length(line: list[str]) -> int:\n    """Measure a rendered line.\n\n    Returns:\n        The number of render fragments queued for output.\n    """\n    return len(line)\n',
                     ),
                 ),
                 focus_path=PurePosixPath("app/lines.py"),

{sarj_python_lint-0.67.8 → sarj_python_lint-0.67.9}/src/sarj_python_lint/rules/duplicated_override_docstring.py

@@ -84,7 +84,7 @@ class DuplicatedOverrideDocstring(Rule):
             ),
             RuleExample(
                 example_id="override-specific-docstring",
-                title="Override documents its distinct behavior",
+                title="Override relies on the base contract",
                 outcome=ExampleOutcome.NO_MATCH,
                 files=(
                     ExampleFile.python(
@@ -93,9 +93,8 @@ class DuplicatedOverrideDocstring(Rule):
                         "    def get(self, key: str) -> str:\n"
                         '        """Get a value by key."""\n'
                         "        return key\n\n"
-                        "class ReplicaStore(Store):\n"
+                        "class MemoryStore(Store):\n"
                         "    def get(self, key: str) -> str:\n"
-                        '        """Get a value from the read replica."""\n'
                         "        return key\n",
                     ),
                 ),

{sarj_python_lint-0.67.8 → sarj_python_lint-0.67.9}/src/sarj_python_lint/rules/kwarg_heavy_construction_in_test.py

@@ -92,12 +92,13 @@ class KwargHeavyConstructionInTest(Rule):
                         "tests/test_call.py",
                         "def build_style(**overrides):\n"
                         "    defaults = dict(\n"
+                        '        color="red", bgcolor="black", bold=True,\n'
                         "        dim=True, italic=True, underline=True,\n"
                         "        blink=True, blink2=True, reverse=True,\n"
                         "    )\n"
                         "    return Style(**(defaults | overrides))\n\n"
                         "def test_style():\n"
-                        '    style = build_style(color="red", bgcolor="black", bold=True)\n'
+                        "    style = build_style()\n"
                         "    assert str(style)\n\n"
                         "def test_style_again():\n"
                         '    other = build_style(color="blue", bgcolor="white", bold=False)\n'

{sarj_python_lint-0.67.8 → sarj_python_lint-0.67.9}/src/sarj_python_lint/rules/no_file_level_escape_hatch_noqa.py

@@ -68,7 +68,7 @@ class NoFileLevelEscapeHatchNoqa(Rule):
                 files=(
                     ExampleFile.python(
                         "app/service.py",
-                        "from unittest import mock  # noqa: TID251 — vendor SDK boundary\n",
+                        "import os  # noqa: TID251 — vendor SDK boundary\n",
                     ),
                 ),
                 focus_path=PurePosixPath("app/service.py"),

{sarj_python_lint-0.67.8 → sarj_python_lint-0.67.9}/src/sarj_python_lint/rules/no_file_level_suppression.py

@@ -76,7 +76,7 @@ class NoFileLevelSuppression(Rule):
                 example_id="scoped-type-checker-suppression",
                 title="Mypy suppression names its code",
                 outcome=ExampleOutcome.NO_MATCH,
-                files=(ExampleFile.python("service.py", "# type: ignore[attr-defined]\nimport os\n"),),
+                files=(ExampleFile.python("service.py", "# type: ignore[import-not-found]\nimport os\n"),),
                 focus_path=PurePosixPath("service.py"),
                 expected_count=0,
                 public=True,

{sarj_python_lint-0.67.8 → sarj_python_lint-0.67.9}/src/sarj_python_lint/rules/no_frozen_after_validator_field_write.py

@@ -86,20 +86,19 @@ class NoFrozenAfterValidatorFieldWrite(Rule):
             ),
             RuleExample(
                 example_id="after-validator-only-validates-frozen-field",
-                title="After-validator checks without mutating the model",
+                title="Before-validator normalizes the input value",
                 outcome=ExampleOutcome.NO_MATCH,
                 files=(
                     ExampleFile.python(
                         "app/models.py",
-                        "from pydantic import BaseModel, ConfigDict, model_validator\n\n"
+                        "from pydantic import BaseModel, ConfigDict, field_validator\n\n"
                         "class Counter(BaseModel):\n"
                         "    model_config = ConfigDict(frozen=True)\n"
                         "    value: int\n\n"
-                        '    @model_validator(mode="after")\n'
-                        "    def require_non_negative(self):\n"
-                        "        if self.value < 0:\n"
-                        '            raise ValueError("value must be non-negative")\n'
-                        "        return self\n",
+                        '    @field_validator("value", mode="before")\n'
+                        "    @classmethod\n"
+                        "    def normalize(cls, value: int) -> int:\n"
+                        "        return abs(value)\n",
                     ),
                 ),
                 focus_path=PurePosixPath("app/models.py"),

{sarj_python_lint-0.67.8 → sarj_python_lint-0.67.9}/src/sarj_python_lint/rules/no_generic_single_export_module.py

@@ -72,8 +72,8 @@ class NoGenericSingleExportModule(Rule):
                 example_id="specific-single-export-module",
                 title="Specific module contains one public definition",
                 outcome=ExampleOutcome.NO_MATCH,
-                files=(ExampleFile.python("pagination.py", "class InvalidPaginationCursorError(Exception): ...\n"),),
-                focus_path=PurePosixPath("pagination.py"),
+                files=(ExampleFile.python("text_case.py", "def snake_case_text(value: str) -> str: ...\n"),),
+                focus_path=PurePosixPath("text_case.py"),
                 expected_count=0,
                 public=True,
             ),

{sarj_python_lint-0.67.8 → sarj_python_lint-0.67.9}/src/sarj_python_lint/rules/no_secret_in_log.py

@@ -114,7 +114,12 @@ class NoSecretInLog(Rule):
                 example_id="redacted-logging-keyword",
                 title="Token prefix logged under a redacted name",
                 outcome=ExampleOutcome.NO_MATCH,
-                files=(ExampleFile.python("service.py", "logger.info('request', token_prefix=token[:6])\n"),),
+                files=(
+                    ExampleFile.python(
+                        "service.py",
+                        "token_prefix = token[:6]\nlogger.info('request', token_prefix=token_prefix)\n",
+                    ),
+                ),
                 focus_path=PurePosixPath("service.py"),
                 expected_count=0,
                 public=True,

{sarj_python_lint-0.67.8 → sarj_python_lint-0.67.9}/src/sarj_python_lint/rules/no_string_concat_in_loop.py

@@ -75,7 +75,7 @@ class NoStringConcatInLoop(Rule):
                 files=(
                     ExampleFile.python(
                         "app/render.py",
-                        'def render(items):\n    lines = []\n    for item in items:\n        lines.append(str(item))\n    return "\\n".join(lines)\n',
+                        'def render(items):\n    lines = []\n    for item in items:\n        lines.append(f"{item}\\n")\n    return "".join(lines)\n',
                     ),
                 ),
                 focus_path=PurePosixPath("app/render.py"),

{sarj_python_lint-0.67.8 → sarj_python_lint-0.67.9}/src/sarj_python_lint/rules/no_typed_doc_sections.py

@@ -60,11 +60,11 @@ class NoTypedDocSections(Rule):
                 outcome=ExampleOutcome.NO_MATCH,
                 files=(
                     ExampleFile.python(
-                        "publisher.py",
-                        'def publish(message: str, *, retry: bool) -> None:\n    """Publish one message.\n\n    Args:\n        message: Wire payload retained for the audit record.\n        retry: Whether a prior partial write may be attempted again.\n    """\n',
+                        "app.py",
+                        'def decode(value: str) -> dict[str, object]:\n    """Decode the value.\n\n    Args:\n        value: Text to decode.\n    """\n    return {}\n',
                     ),
                 ),
-                focus_path=PurePosixPath("publisher.py"),
+                focus_path=PurePosixPath("app.py"),
                 expected_count=0,
                 public=True,
             ),

{sarj_python_lint-0.67.8 → sarj_python_lint-0.67.9}/src/sarj_python_lint/rules/prefer_match_type_dispatch.py

@@ -855,12 +855,14 @@ class PreferMatchTypeDispatch(Rule):
                         "app/parser.py",
                         "def parse(value: object):\n"
                         "    match value:\n"
-                        "        case None | Unset():\n"
+                        "        case Text():\n"
+                        "            return value\n"
+                        "        case Binary():\n"
+                        "            return value\n"
+                        "        case PathValue():\n"
                         "            return value\n"
-                        "        case int():\n"
-                        "            return str(value)\n"
                         "        case _:\n"
-                        "            return None\n",
+                        "            return coerce(value)\n",
                     ),
                 ),
                 focus_path=PurePosixPath("app/parser.py"),

{sarj_python_lint-0.67.8 → sarj_python_lint-0.67.9}/src/sarj_python_lint/rules/restated_test_docstring.py

@@ -158,15 +158,15 @@ class RestatedTestDocstring(Rule):
             ),
             RuleExample(
                 example_id="test-regression-context",
-                title="Docstring explains the regression",
+                title="Redundant docstring removed",
                 outcome=ExampleOutcome.NO_MATCH,
                 files=(
                     ExampleFile.python(
-                        "tests/test_scheduler.py",
-                        'def test_keeps_the_lock():\n    """The scheduler would spin forever without this."""\n    assert acquire()\n',
+                        "tests/test_widget.py",
+                        "def test_widget_renders():\n    assert render(widget)\n",
                     ),
                 ),
-                focus_path=PurePosixPath("tests/test_scheduler.py"),
+                focus_path=PurePosixPath("tests/test_widget.py"),
                 expected_count=0,
                 public=True,
             ),