open/llm-ingestion-okf
1
0
Fork 0
Code Issues Releases 1 Activity

style(extract,propose): the formatter was never in the acceptance, so it went red unseen

Browse source 
`ruff format --check .` was clean on `fbad5b0` and red on `6ff18fd`: blank
lines around the new `strip_converter_attribute` and the new `member` closure.
Round 9's order named only `ruff check`, and STATE's "ruff clean" read wider
than what was measured -- an acceptance that names one of a tool's two gates
reports green for the gate it did not run.

Three hunks, blank lines only, no behaviour. From here the acceptance is BOTH
`ruff check` and `ruff format --check .`, both named in the report with the
version they ran under (0.15.22 here).

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
Kjell Tore Guttormsen 2026-09-09 22:05:51 +02:00
commit 6ff84d71c8
2 changed files with 2 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
src/llm_ingestion_okf/extract.py
View file

@ -753,7 +753,6 @@ _CONVERTER_ATTRIBUTE = re.compile(r"\s*\{#[A-Za-z0-9_.:-]+\}\s*$")
_TABLE_LINE = "|" _TABLE_LINE = "|"
def strip_converter_attribute(title: str) -> str: def strip_converter_attribute(title: str) -> str:
"""Remove a trailing pandoc attribute anchor from a heading's title. """Remove a trailing pandoc attribute anchor from a heading's title.
@ -776,6 +775,7 @@ def strip_converter_attribute(title: str) -> str:
""" """
return _CONVERTER_ATTRIBUTE.sub("", title) return _CONVERTER_ATTRIBUTE.sub("", title)
@dataclass(frozen=True) @dataclass(frozen=True)
class SourceUnits: class SourceUnits:
"""Where in the ORIGINAL each stretch of the extracted text came from. """Where in the ORIGINAL each stretch of the extracted text came from.

1
src/llm_ingestion_okf/propose.py
View file

@ -934,6 +934,7 @@ def _contents_run_positions(
concept while its neighbours were not. What still bounds the rule is the run concept while its neighbours were not. What still bounds the rule is the run
LENGTH, which is what the `CONTENTS_RUN` sweep bought and is unchanged. LENGTH, which is what the `CONTENTS_RUN` sweep bought and is unchanged.
""" """
def member(candidate: Candidate) -> bool: def member(candidate: Candidate) -> bool:
if candidate.rule == RULE_TABLE_BLOCK: if candidate.rule == RULE_TABLE_BLOCK:
return False return False