python-checks 0.1.1__tar.gz → 0.1.2__tar.gz

{python_checks-0.1.1 → python_checks-0.1.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-checks
-Version: 0.1.1
+Version: 0.1.2
 Summary: Проверки архитектурных соглашений проекта
 Author: Armontex
 Author-email: Armontex <windle1337@gmail.com>
@@ -181,6 +181,13 @@ text("SELECT 1")  # db-ok: raw-sql: проба живости, формы ORM н
 правило. У каждой группы есть своё короткое слово (`# db-ok`, `# type-ok`,
 `# signature-ok`, …): человек помнит группу, а не двадцать семь кодов.
 
+Пометок в строке может стоять несколько — подпись в столбик собирает их на
+последней строке:
+
+```python
+    ) -> object:  # signature-ok: так зовёт pydantic  # type-ok: сырой ввод
+```
+
 Пометка без кода, с опечаткой в коде или без причины — сама нарушение. Молча
 неработающая пометка выглядит как отключённая проверка, а на деле проверка
 работает и просто её не видит.

{python_checks-0.1.1 → python_checks-0.1.2}/README.md

@@ -165,6 +165,13 @@ text("SELECT 1")  # db-ok: raw-sql: проба живости, формы ORM н
 правило. У каждой группы есть своё короткое слово (`# db-ok`, `# type-ok`,
 `# signature-ok`, …): человек помнит группу, а не двадцать семь кодов.
 
+Пометок в строке может стоять несколько — подпись в столбик собирает их на
+последней строке:
+
+```python
+    ) -> object:  # signature-ok: так зовёт pydantic  # type-ok: сырой ввод
+```
+
 Пометка без кода, с опечаткой в коде или без причины — сама нарушение. Молча
 неработающая пометка выглядит как отключённая проверка, а на деле проверка
 работает и просто её не видит.

{python_checks-0.1.1 → python_checks-0.1.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "python-checks"
-version = "0.1.1"
+version = "0.1.2"
 description = "Проверки архитектурных соглашений проекта"
 readme = "README.md"
 license = "MIT"

{python_checks-0.1.1 → python_checks-0.1.2}/pyproject.toml.orig

@@ -1,6 +1,6 @@
 [project]
 name = "python-checks"
-version = "0.1.1"
+version = "0.1.2"
 description = "Проверки архитектурных соглашений проекта"
 readme = "README.md"
 license = "MIT"

{python_checks-0.1.1 → python_checks-0.1.2}/src/py_checks/core/__init__.py

@@ -11,7 +11,7 @@ from py_checks.core._edit import Edit, apply, column
 from py_checks.core._errors import ParseError, UnknownCheckError
 from py_checks.core._fixer import fix
 from py_checks.core._format import reformat
-from py_checks.core._markers import MARKER, Marker, complaints, read, surviving
+from py_checks.core._markers import MARKER, Marker, complaints, markers, surviving
 from py_checks.core._protocols import Check, FileCheck, ProjectCheck, Scope
 from py_checks.core._registry import Checks, available, get
 from py_checks.core._report import report
@@ -21,19 +21,19 @@ from py_checks.core._source import ParsedFile
 from py_checks.core._violation import Violation
 
 __all__ = [
-    "EXIT_OK",
-    "EXIT_VIOLATION",
-    "GROUP",
-    "MARKER",
-    "SYNTAX",
     "Check",
     "Checks",
+    "EXIT_OK",
+    "EXIT_VIOLATION",
     "Edit",
     "FileCheck",
-    "ProjectCheck",
+    "GROUP",
+    "MARKER",
     "Marker",
     "ParseError",
     "ParsedFile",
+    "ProjectCheck",
+    "SYNTAX",
     "Scope",
     "SettingsMismatchError",
     "UnknownCheckError",
@@ -42,15 +42,15 @@ __all__ = [
     "available",
     "column",
     "complaints",
-    "fix",
     "examine",
+    "fix",
     "get",
     "inspect",
+    "markers",
     "python_files",
-    "read",
     "reformat",
-    "surviving",
-    "settings_as",
     "report",
+    "settings_as",
     "survey",
+    "surviving",
 ]

{python_checks-0.1.1 → python_checks-0.1.2}/src/py_checks/core/_markers.py

@@ -49,33 +49,73 @@ class Marker:
     column: int
 
 
-def read(
+def markers(
     *,
     line: str,
     aliases: Mapping[str, frozenset[str]],
-) -> Marker | None:
-    """Маркер из строки, если он там есть.
+) -> tuple[Marker, ...]:
+    """Маркеры строки: их может быть несколько.
+
+    Подпись в столбик собирает пометки в одну строку — `# signature-ok: так
+    зовёт библиотека  # type-ok: сырой ввод`, — и снимать они должны обе.
 
     Разбор нарочно не падает на кривой записи: маркер без кода или без причины
     читается и попадает в `complaints`, иначе о нём никто бы не узнал.
     """
-    if MARKER in line:
-        codes, _, reason = line.split(MARKER, maxsplit=1)[1].partition(":")
-        if not _named(text=codes):
-            return None
-        return Marker(
-            codes=frozenset(_codes(text=codes)),
-            reason=reason.strip(),
-            column=line.index(MARKER) + 1,
+    found = sorted(
+        _starts(
+            line=line,
+            aliases=aliases,
         )
-    for text, codes in aliases.items():
-        if text in line:
-            return Marker(
-                codes=codes,
-                reason=line.split(text, maxsplit=1)[1].removeprefix(":").strip(),
-                column=line.index(text) + 1,
+    )
+    bounds = [*(start for start, _ in found), len(line)]
+    return tuple(
+        marker
+        for number, (start, word) in enumerate(found)
+        if (
+            marker := _marker(
+                text=line[start : bounds[number + 1]],
+                word=word,
+                aliases=aliases,
+                column=start + 1,
             )
-    return None
+        )
+    )
+
+
+def _starts(
+    *,
+    line: str,
+    aliases: Mapping[str, frozenset[str]],
+) -> Iterator[tuple[int, str]]:
+    for word in (MARKER, *aliases):
+        start = line.find(word)
+        if start != -1:
+            yield start, word
+
+
+def _marker(
+    *,
+    text: str,
+    word: str,
+    aliases: Mapping[str, frozenset[str]],
+    column: int,
+) -> Marker | None:
+    tail = text.removeprefix(word)
+    if word != MARKER:
+        return Marker(
+            codes=aliases[word],
+            reason=tail.removeprefix(":").strip(),
+            column=column,
+        )
+    codes, _, reason = tail.partition(":")
+    if not _named(text=codes):
+        return None
+    return Marker(
+        codes=frozenset(_codes(text=codes)),
+        reason=reason.strip(),
+        column=column,
+    )
 
 
 def surviving(
@@ -109,18 +149,16 @@ def complaints(
     нарушение.
     """
     for number, line in enumerate(file.lines, start=1):
-        marker = read(
+        for marker in markers(
             line=line,
             aliases=aliases,
-        )
-        if marker is None:
-            continue
-        yield from _wrong(
-            marker=marker,
-            path=file.path,
-            line=number,
-            known=known,
-        )
+        ):
+            yield from _wrong(
+                marker=marker,
+                path=file.path,
+                line=number,
+                known=known,
+            )
 
 
 def _named(*, text: str) -> bool:
@@ -146,17 +184,17 @@ def _covered(
     file: ParsedFile,
     aliases: Mapping[str, frozenset[str]],
 ) -> bool:
-    for line in _span(
-        violation=violation,
-        file=file,
-    ):
-        marker = read(
+    return any(
+        violation.code in marker.codes
+        for line in _span(
+            violation=violation,
+            file=file,
+        )
+        for marker in markers(
             line=line,
             aliases=aliases,
         )
-        if marker is not None and violation.code in marker.codes:
-            return True
-    return False
+    )
 
 
 def _span(