subedit 0.1.0__tar.gz → 0.1.2__tar.gz

{subedit-0.1.0 → subedit-0.1.2}/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "subedit",
   "displayName": "subedit",
-  "version": "0.1.0",
+  "version": "0.1.2",
   "description": "Repairs comment layout silently on write, and returns only what needs rewriting.",
   "author": {
     "name": "mroops0111"

{subedit-0.1.0 → subedit-0.1.2}/.github/workflows/ci.yml

@@ -50,6 +50,20 @@ jobs:
           python-version: ${{ matrix.python-version }}
           enable-cache: true
       - run: uv sync --extra dev
+
+      # Without Vale, every test against it skips and the self-hosting check never runs it.
+      # AutoCorrect needs no step, since it arrives as a Python dependency.
+      - name: Install Vale
+        env:
+          VALE_VERSION: 3.18.0
+        run: |
+          curl -sSfL "https://github.com/errata-ai/vale/releases/download/v${VALE_VERSION}/vale_${VALE_VERSION}_Linux_64-bit.tar.gz" \
+            | tar -xz -C "$RUNNER_TEMP" vale
+          echo "$RUNNER_TEMP" >> "$GITHUB_PATH"
+
+      - name: Confirm Vale
+        run: vale --version
+
       - run: uv run pytest tests/ -v --tb=short
 
       - name: Self-hosting

subedit-0.1.2/.vale.ini

@@ -0,0 +1,13 @@
+StylesPath = src/subedit/vale/styles
+MinAlertLevel = warning
+
+[*.{py,ts,tsx,js,go,rs}]
+BasedOnStyles = subedit
+subedit.MidSentenceColonMarkup = NO
+
+[*.md]
+BasedOnStyles = subedit
+subedit.MidSentenceColon = NO
+
+[tests/fixtures/vale/*]
+BasedOnStyles =

{subedit-0.1.0 → subedit-0.1.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: subedit
-Version: 0.1.0
+Version: 0.1.2
 Summary: A linter and formatter for the prose inside code comments, across languages.
 Project-URL: Homepage, https://github.com/mroops0111/subedit
 Project-URL: Repository, https://github.com/mroops0111/subedit
@@ -18,14 +18,18 @@ Classifier: Programming Language :: Python :: 3.12
 Classifier: Programming Language :: Python :: 3.13
 Classifier: Topic :: Software Development :: Quality Assurance
 Requires-Python: >=3.11
+Requires-Dist: autocorrect-py>=2.14.0
 Requires-Dist: pydantic>=2.13.5
-Requires-Dist: tree-sitter-language-pack>=1.17.0
+Requires-Dist: tree-sitter-python>=0.25.0
 Requires-Dist: tree-sitter>=0.26.0
 Requires-Dist: typer>=0.27.2
 Provides-Extra: dev
 Requires-Dist: basedpyright>=1.40.1; extra == 'dev'
 Requires-Dist: pytest>=9.0.0; extra == 'dev'
 Requires-Dist: ruff>=0.16.7; extra == 'dev'
+Requires-Dist: tree-sitter-typescript>=0.23.2; extra == 'dev'
+Provides-Extra: vale
+Requires-Dist: vale>=3.22.0.0; extra == 'vale'
 Description-Content-Type: text/markdown
 
 # subedit
@@ -70,7 +74,7 @@ Install the command line tool from PyPI.
 uv tool install subedit
 ```
 
-`pipx install subedit` and `pip install subedit` work as well.
+`pipx install subedit` and `pip install subedit` work as well. Add the `vale` extra, as in `uv tool install 'subedit[vale]'`, to have Vale fetched for you.
 
 ## Commands
 
@@ -105,7 +109,7 @@ Rewriting a sentence needs its meaning, which a deterministic process cannot sup
 
 ### Claude Code
 
-The plugin registers the hook. It cannot install the tool, so install that first.
+The plugin needs only [uv](https://docs.astral.sh/uv/). It runs subedit from PyPI with the `vale` extra, and warms that environment in the background when a session starts, so no edit waits on the first download.
 
 ```text
 /plugin marketplace add mroops0111/subedit
@@ -159,14 +163,22 @@ block-too-long = { severity = "warning", line = 3, doc-public = 20 }
 
 Rule names describe the problem they report. `docs/rules.md` lists every rule, generated from the rules themselves.
 
+A rule at `error` fails `check` and stops an agent, while one at `warning` is shown as advice and blocks nothing. The same table sets the delegated Vale rules by name.
+
+```toml
+[rules]
+"subedit.MidSentenceColon" = "error"
+"subedit.DateRef" = "off"
+```
+
 ## Delegated Tools
 
-`subedit check` also runs two tools where they are installed, and folds their findings into its own report.
+`check`, `fix`, and the agent hook also run two tools, and fold their findings into the same report.
 
-- **Vale**: punctuation, banned phrasing, and anything a markup aware prose linter already handles
-- **AutoCorrect**: spacing and punctuation width between CJK and Latin text
+- **Vale**: punctuation, banned phrasing, and anything a markup aware prose linter already handles. Vale has no autofix, so its findings come back as rewrites. It runs where it is installed, or where the `vale` extra fetched it. A repository without its own `.vale.ini` gets the style that ships with subedit, whose rules are named `subedit.*`.
+- **AutoCorrect**: spacing and punctuation width between CJK and Latin text. Its corrections are applied silently, like any other mechanical fix. It arrives as a Python dependency, and reads `.autocorrectrc` from the working directory.
 
-`--no-delegate` skips both. `fix` and the agent hook do not run them.
+`--no-delegate` skips both for `check` and `fix`.
 
 ## Surfaces
 

{subedit-0.1.0 → subedit-0.1.2}/README.md

@@ -40,7 +40,7 @@ Install the command line tool from PyPI.
 uv tool install subedit
 ```
 
-`pipx install subedit` and `pip install subedit` work as well.
+`pipx install subedit` and `pip install subedit` work as well. Add the `vale` extra, as in `uv tool install 'subedit[vale]'`, to have Vale fetched for you.
 
 ## Commands
 
@@ -75,7 +75,7 @@ Rewriting a sentence needs its meaning, which a deterministic process cannot sup
 
 ### Claude Code
 
-The plugin registers the hook. It cannot install the tool, so install that first.
+The plugin needs only [uv](https://docs.astral.sh/uv/). It runs subedit from PyPI with the `vale` extra, and warms that environment in the background when a session starts, so no edit waits on the first download.
 
 ```text
 /plugin marketplace add mroops0111/subedit
@@ -129,14 +129,22 @@ block-too-long = { severity = "warning", line = 3, doc-public = 20 }
 
 Rule names describe the problem they report. `docs/rules.md` lists every rule, generated from the rules themselves.
 
+A rule at `error` fails `check` and stops an agent, while one at `warning` is shown as advice and blocks nothing. The same table sets the delegated Vale rules by name.
+
+```toml
+[rules]
+"subedit.MidSentenceColon" = "error"
+"subedit.DateRef" = "off"
+```
+
 ## Delegated Tools
 
-`subedit check` also runs two tools where they are installed, and folds their findings into its own report.
+`check`, `fix`, and the agent hook also run two tools, and fold their findings into the same report.
 
-- **Vale**: punctuation, banned phrasing, and anything a markup aware prose linter already handles
-- **AutoCorrect**: spacing and punctuation width between CJK and Latin text
+- **Vale**: punctuation, banned phrasing, and anything a markup aware prose linter already handles. Vale has no autofix, so its findings come back as rewrites. It runs where it is installed, or where the `vale` extra fetched it. A repository without its own `.vale.ini` gets the style that ships with subedit, whose rules are named `subedit.*`.
+- **AutoCorrect**: spacing and punctuation width between CJK and Latin text. Its corrections are applied silently, like any other mechanical fix. It arrives as a Python dependency, and reads `.autocorrectrc` from the working directory.
 
-`--no-delegate` skips both. `fix` and the agent hook do not run them.
+`--no-delegate` skips both for `check` and `fix`.
 
 ## Surfaces
 

subedit-0.1.2/docs/rules.md

@@ -0,0 +1,18 @@
+# Rules
+
+Generated from `RuleMetadata`. Edit the rule, not this file.
+
+The fix kind states who resolves a finding. A mechanical kind is repaired without telling the author, so only a rewrite reaches an agent.
+
+The level says whether a rewrite blocks. An error fails check and stops an agent, while a warning is shown as advice. Set either per rule under `[rules]`, and the Vale rules the same way by name, as in `"subedit.DateRef" = "off"`.
+
+| Rule | Fix kind | Level | Summary |
+| --- | --- | --- | --- |
+| `line-too-long` | reflow | error | A physical comment line must fit the configured width, counting indent and delimiter. |
+| `mid-clause-break` | reflow | error | A comment block must be laid out one sentence per line, splitting only at commas. |
+| `block-too-long` | rewrite | warning | A comment block long enough to restate the code has stopped earning its place. |
+| `file-header-comment` | rewrite | warning | A file must not open with a banner comment before its first declaration. |
+| `plain-comment-on-public` | rewrite | error | A public symbol must be documented in the doc comment form its language renders. |
+| `signature-restating-tag` | rewrite | error | A doc comment must not restate what the type signature already carries. |
+| `stub-fragment` | rewrite | warning | A sentence forced to break must not leave a stub of a line behind. |
+| `unbreakable-sentence` | rewrite | error | A sentence over the width must carry a comma so it has somewhere legal to break. |

subedit-0.1.2/examples/walkthrough.py

@@ -0,0 +1,71 @@
+import sys
+
+from subedit.application import Linter, RuleEngine
+from subedit.domain import Document, RuleSet
+from subedit.infrastructure import ALL_RULES, DEFAULT_PARSER
+
+
+SOURCE = """def build(value):
+    # cap the size to the shared limit so the mmap does not
+    # blow past it, an unbounded value faults the runner.
+    return value
+"""
+
+WIDTH = int(sys.argv[1]) if len(sys.argv) > 1 else 60
+
+
+def show(step_number, title, body):
+    print(f'\n{"─" * 62}\n{step_number}. {title}\n{"─" * 62}')
+    print(body)
+
+
+show(1, 'Source (just a string)', SOURCE.rstrip())
+
+document = Document(uri='file:///demo.py', text=SOURCE, language_id='python')
+group = DEFAULT_PARSER.parse(document)[0]
+
+show(
+    2,
+    'The parser turns it into a CommentGroup',
+    f"""form       = {group.form.value}          shape of the comment
+placement  = {group.placement.value}   where it sits
+attachment = {group.attachment}      what it documents
+line_count = {group.line_count}
+widest     = {group.longest_line} characters""",
+)
+
+show(
+    3,
+    'Each line splits into three parts (the key to measuring width)',
+    '\n'.join(
+        f'  indent={line.indent!r:6s} delimiter={line.delimiter!r:5s} text={line.text!r}' for line in group.lines
+    ),
+)
+
+show(
+    4,
+    'Prose is the plain text left once the markers are stripped',
+    f"""physical_lines = {group.prose().physical_lines}
+
+flattened      = {group.prose().flattened!r}
+
+sentences      = {[sentence.text for sentence in group.prose().sentences()]}""",
+)
+
+show(5, f'reflow computes the right layout (one sentence per line, width {WIDTH})', group.reflow(WIDTH).render())
+
+report = Linter(DEFAULT_PARSER, RuleEngine(ALL_RULES)).run(
+    document, RuleSet(enabled_rule_ids=frozenset(rule.metadata.rule_id for rule in ALL_RULES), line_width=WIDTH)
+)
+
+show(
+    6,
+    'Each of the 8 rules asks one question, and these answered',
+    '\n'.join(
+        f'  {diagnostic.rule_id:22s} fixed by {"the tool" if diagnostic.is_mechanical else "the author"}'
+        for diagnostic in report.diagnostics
+    ),
+)
+
+print(f'\n  → {len(report.mechanical)} mechanical, repaired silently by the tool')
+print(f'  → {len(report.needing_author)} need the author, and only these reach an agent')

{subedit-0.1.0 → subedit-0.1.2}/hooks/hooks.json

@@ -12,6 +12,17 @@
           }
         ]
       }
+    ],
+    "SessionStart": [
+      {
+        "hooks": [
+          {
+            "type": "command",
+            "command": "${CLAUDE_PLUGIN_ROOT}/hooks/run-subedit-hook.sh prepare",
+            "timeout": 10
+          }
+        ]
+      }
     ]
   }
 }

subedit-0.1.2/hooks/run-subedit-hook.sh

@@ -0,0 +1,27 @@
+#!/usr/bin/env bash
+# Runs subedit from PyPI through uvx, so the plugin needs nothing installed but uv.
+# With "prepare", it warms the same environment in the background instead,
+# because the first run installs subedit and downloads Vale, which no edit should wait on.
+set -uo pipefail
+
+SPEC='subedit[vale]>=0.1.2'
+# A bare uvx takes the first Python it finds, which on a stock Mac is too old for subedit.
+PYTHON='>=3.11'
+
+if ! command -v uvx >/dev/null 2>&1; then
+    cat >&2 <<'MESSAGE'
+subedit needs uv, so comment checks are not running.
+
+    curl -LsSf https://astral.sh/uv/install.sh | sh
+
+Install it, or remove the subedit plugin.
+MESSAGE
+    exit 1
+fi
+
+if [ "${1:-}" = prepare ]; then
+    nohup uvx --python "$PYTHON" --from "$SPEC" vale --version >/dev/null 2>&1 &
+    exit 0
+fi
+
+exec uvx --python "$PYTHON" --from "$SPEC" subedit-hook

{subedit-0.1.0 → subedit-0.1.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "subedit"
-version = "0.1.0"
+version = "0.1.2"
 description = "A linter and formatter for the prose inside code comments, across languages."
 authors = [
     {name = "Yun-Tai Yang"},
@@ -21,9 +21,10 @@ classifiers = [
 ]
 
 dependencies = [
+    "autocorrect-py>=2.14.0",
     "pydantic>=2.13.5",
     "tree-sitter>=0.26.0",
-    "tree-sitter-language-pack>=1.17.0",
+    "tree-sitter-python>=0.25.0",
     "typer>=0.27.2",
 ]
 
@@ -32,6 +33,10 @@ dev = [
     "pytest>=9.0.0",
     "basedpyright>=1.40.1",
     "ruff>=0.16.7",
+    "tree-sitter-typescript>=0.23.2",
+]
+vale = [
+    "vale>=3.22.0.0",
 ]
 
 [project.scripts]

{subedit-0.1.0 → subedit-0.1.2}/src/subedit/__init__.py

@@ -1,3 +1,3 @@
-__version__ = '0.1.0'
+__version__ = '0.1.2'
 
 __all__ = ['__version__']

{subedit-0.1.0 → subedit-0.1.2}/src/subedit/application/fixer.py

@@ -4,7 +4,9 @@ from ..domain.models.diagnostic import Diagnostic, FixKind, Report
 from ..domain.models.document import Document
 from ..domain.models.rule_set import RuleSet
 from ..domain.models.text import TextEdit, apply_edits
+from ..domain.ports.external_linter import ExternalLinter
 from .linter import Linter
+from .report_aggregator import ReportAggregator
 
 
 MAX_ROUNDS = 10
@@ -23,19 +25,25 @@ class FixResult:
 
     @property
     def unresolved_findings(self) -> tuple[Diagnostic, ...]:
-        return self.report.needing_author
+        return self.report.blocking
 
 
 class Fixer:
-    def __init__(self, linter: Linter, max_rounds: int = MAX_ROUNDS) -> None:
-        self._linter = linter
+    def __init__(
+        self,
+        linter: Linter,
+        external_linters: tuple[ExternalLinter, ...] = (),
+        max_rounds: int = MAX_ROUNDS,
+    ) -> None:
+        # External findings join every round, so an edit they carry is applied like a native one.
+        self._aggregator = ReportAggregator(linter, external_linters)
         self._max_rounds = max_rounds
 
     def run(self, document: Document, rule_set: RuleSet) -> FixResult:
         working_document = document
         applied_edit_count = 0
         round_count = 0
-        report = self._linter.run(working_document, rule_set)
+        report = self._aggregator.run(working_document, rule_set)
 
         while round_count < self._max_rounds:
             edits = self._next_edits(report)
@@ -44,7 +52,7 @@ class Fixer:
             working_document = working_document.with_text(apply_edits(working_document.text, edits))
             applied_edit_count += len(edits)
             round_count += 1
-            report = self._linter.run(working_document, rule_set)
+            report = self._aggregator.run(working_document, rule_set)
 
         return FixResult(
             document=working_document, report=report, round_count=round_count, applied_edit_count=applied_edit_count

subedit-0.1.2/src/subedit/application/report_aggregator.py

@@ -0,0 +1,35 @@
+import dataclasses
+
+from ..domain.models.diagnostic import Diagnostic, Report
+from ..domain.models.document import Document
+from ..domain.models.rule_set import RuleSet
+from ..domain.ports.external_linter import ExternalLinter
+from .linter import Linter
+
+
+class ReportAggregator:
+    def __init__(self, linter: Linter, external_linters: tuple[ExternalLinter, ...] = ()) -> None:
+        self._linter = linter
+        self._external_linters = external_linters
+
+    def run(self, document: Document, rule_set: RuleSet) -> Report:
+        report = self._linter.run(document, rule_set)
+        missing_sources: list[str] = []
+        for external_linter in self._external_linters:
+            if not external_linter.is_available():
+                missing_sources.append(external_linter.name)
+                continue
+            report = report.merged_with(Report(self._configured(external_linter.run(document), rule_set)))
+        if missing_sources:
+            report = report.merged_with(Report(unavailable_sources=tuple(missing_sources)))
+        return report
+
+    # The project's config has the last word on a delegated rule, as it does on a native one.
+    def _configured(self, diagnostics: tuple[Diagnostic, ...], rule_set: RuleSet) -> tuple[Diagnostic, ...]:
+        configured: list[Diagnostic] = []
+        for diagnostic in diagnostics:
+            if not rule_set.allows_external(diagnostic.rule_id):
+                continue
+            severity = rule_set.settings_for(diagnostic.rule_id).severity
+            configured.append(dataclasses.replace(diagnostic, severity=severity) if severity else diagnostic)
+        return tuple(configured)

{subedit-0.1.0 → subedit-0.1.2}/src/subedit/domain/models/comment.py

@@ -97,24 +97,34 @@ class CommentGroup:
         ]
         return Prose('\n'.join(rendered_lines))
 
-    def reflow(self, width: int) -> 'CommentGroup':
-        texts = self._reflow_into_texts(width)
+    def reflow(self, width: int, stub_floor: int = 0) -> 'CommentGroup':
+        # A one line docstring that fits is the summary convention, not a block to lay out.
+        if self.form is CommentForm.DOC and self.line_count == 1 and self.longest_line <= width:
+            return self
+        texts = self._reflow_into_texts(width, stub_floor)
         if not texts:
             return self
         return self._rebuild_lines(texts)
 
-    def _reflow_into_texts(self, width: int) -> tuple[str, ...]:
+    # A sentence whose split strands a stub needs its author,
+    # so the block keeps the layout they gave it rather than one its own check rejects.
+    def _reflow_into_texts(self, width: int, stub_floor: int) -> tuple[str, ...]:
         budget = max(20, width - self.lines[0].prefix_width)
         out: list[str] = []
-        for paragraph, is_literal in self._split_into_paragraphs():
+        for paragraph_index, (paragraph, is_literal) in enumerate(self._split_into_paragraphs()):
             if out:
                 out.append('')
             if is_literal:
                 out.extend(paragraph)
                 continue
             joined_text = ' '.join(line.strip() for line in paragraph if line.strip())
-            for sentence in Prose(joined_text).sentences():
-                out.extend(reflow_sentence(sentence.text, budget))
+            for sentence_index, sentence in enumerate(Prose(joined_text).sentences()):
+                if sentence_index == 1 and paragraph_index == 0 and self.form is CommentForm.DOC:
+                    out.append('')
+                sentence_lines = reflow_sentence(sentence.text, budget)
+                if len(sentence_lines) > 1 and min(map(len, sentence_lines)) < stub_floor:
+                    return ()
+                out.extend(sentence_lines)
         return tuple(out)
 
     def _split_into_paragraphs(self) -> tuple[tuple[tuple[str, ...], bool], ...]:
@@ -138,7 +148,8 @@ class CommentGroup:
         closes_alone = len(self.lines) > 1 and not closing.text.strip()
         body = [dataclasses.replace(opening, delimiter='', suffix='', text=text, range=opening.range) for text in texts]
         body[0] = dataclasses.replace(body[0], delimiter=opening.delimiter)
-        if closes_alone or len(texts) > 1:
+        # Text that fits one line closes on that line, as ruff and black format it.
+        if len(texts) > 1:
             body.append(
                 dataclasses.replace(closing, delimiter=closing.delimiter or opening.delimiter, text='', suffix='')
             )

{subedit-0.1.0 → subedit-0.1.2}/src/subedit/domain/models/diagnostic.py

@@ -96,6 +96,19 @@ class Report:
     def needing_author(self) -> tuple[Diagnostic, ...]:
         return tuple(diagnostic for diagnostic in self.diagnostics if diagnostic.needs_author)
 
+    @property
+    def blocking(self) -> tuple[Diagnostic, ...]:
+        """The findings an author must resolve, since a warning reaches them as advice and blocks nothing."""
+        return tuple(diagnostic for diagnostic in self.needing_author if diagnostic.severity is Severity.ERROR)
+
+    @property
+    def advisory(self) -> tuple[Diagnostic, ...]:
+        return tuple(diagnostic for diagnostic in self.needing_author if diagnostic.severity is not Severity.ERROR)
+
+    @property
+    def has_errors(self) -> bool:
+        return any(diagnostic.severity is Severity.ERROR for diagnostic in self.diagnostics)
+
     @property
     def is_clean(self) -> bool:
         return not self.diagnostics

{subedit-0.1.0 → subedit-0.1.2}/src/subedit/domain/models/prose.py

@@ -26,6 +26,7 @@ _FENCE = re.compile(r'^\s*(```|~~~)')
 _LIST_DELIMITER = re.compile(r'^\s*(?:[0-9]+[.)]|[-*+]|[a-zA-Z][.)])\s')
 _LITERAL_DELIMITER = '\\b'
 _CLOSING_MARKS = '"\')]}'
+_APOSTROPHES = ("'", '\u2019')
 
 # Only before a number, since "No. 5" abbreviates while a trailing no ends its sentence.
 NUMBER_ABBREVIATION = 'no'
@@ -161,6 +162,9 @@ class Prose:
         if offset + 1 < len(text) and text[offset + 1].isdigit():
             return False
         trailing_word = _TRAILING_WORD.search(text[:offset])
+        # The s of a possessive is no initial, so a sentence closing on a word like API's still ends there.
+        if trailing_word and text[: trailing_word.start()].endswith(_APOSTROPHES):
+            return self._followed_by_break(text, offset)
         word = trailing_word.group(1).lower().rstrip('.') if trailing_word else ''
         if word in ABBREVIATIONS:
             return False

{subedit-0.1.0 → subedit-0.1.2}/src/subedit/domain/models/rule_set.py

@@ -19,6 +19,8 @@ class RuleSet:
     enabled_rule_ids: frozenset[str]
     settings: typing.Mapping[str, RuleSettings] = dataclasses.field(default_factory=dict)
     line_width: int = 120
+    # A delegated tool's rules are not known up front, so they are listed when switched off instead.
+    disabled_external_ids: frozenset[str] = frozenset()
 
     def is_enabled(self, rule_id: str) -> bool:
         return rule_id in self.enabled_rule_ids
@@ -28,3 +30,6 @@ class RuleSet:
 
     def severity_for(self, metadata: RuleMetadata) -> Severity:
         return self.settings_for(metadata.rule_id).severity or metadata.default_severity
+
+    def allows_external(self, rule_id: str) -> bool:
+        return rule_id not in self.disabled_external_ids

{subedit-0.1.0 → subedit-0.1.2}/src/subedit/domain/ports/external_linter.py

@@ -12,3 +12,6 @@ class ExternalLinter(abc.ABC):
 
     @abc.abstractmethod
     def run(self, document: Document) -> tuple[Diagnostic, ...]: ...
+
+    def prepare(self) -> None:  # noqa: B027, an optional hook rather than a required one
+        """Fetch what the tool needs, for a caller that can afford to wait. Most tools need nothing."""

{subedit-0.1.0 → subedit-0.1.2}/src/subedit/domain/rules/layout.py

@@ -3,6 +3,7 @@ import typing
 from ..models.comment import CommentGroup
 from ..models.diagnostic import Diagnostic, Fix, FixKind, RuleMetadata
 from ..ports.rule import Rule, RuleContext
+from .rewrite import stub_floor
 
 
 MID_CLAUSE_BREAK = RuleMetadata(
@@ -22,7 +23,7 @@ class MidClauseBreakRule(Rule):
     metadata = MID_CLAUSE_BREAK
 
     def check(self, group: CommentGroup, context: RuleContext) -> typing.Iterable[Diagnostic]:
-        reflowed = group.reflow(context.line_width)
+        reflowed = group.reflow(context.line_width, stub_floor(context))
         if reflowed.render() == group.render():
             return
         yield Diagnostic(
@@ -43,7 +44,7 @@ class LineTooLongRule(Rule):
         width = context.line_width
         if group.longest_line <= width:
             return
-        reflowed = group.reflow(width)
+        reflowed = group.reflow(width, stub_floor(context))
         if reflowed.longest_line > width:
             return
         yield Diagnostic(

{subedit-0.1.0 → subedit-0.1.2}/src/subedit/domain/rules/rewrite.py

@@ -1,7 +1,7 @@
 import typing
 
 from ..models.comment import CommentGroup
-from ..models.diagnostic import Diagnostic, Fix, FixKind, RuleMetadata
+from ..models.diagnostic import Diagnostic, Fix, FixKind, RuleMetadata, Severity
 from ..models.prose import Prose, reflow_sentence
 from ..ports.rule import Rule, RuleContext
 
@@ -10,6 +10,7 @@ STUB_FRAGMENT = RuleMetadata(
     rule_id='stub-fragment',
     summary='A sentence forced to break must not leave a stub of a line behind.',
     fix_kind=FixKind.REWRITE,
+    default_severity=Severity.WARNING,
 )
 
 UNBREAKABLE_SENTENCE = RuleMetadata(
@@ -35,14 +36,19 @@ def sentence_budget(group: CommentGroup, context: RuleContext) -> int:
     return max(20, context.line_width - group.lines[0].prefix_width)
 
 
+def stub_floor(context: RuleContext) -> int:
+    """Reflow reads the same floor, so a fix never lays out a stub that this rule then rejects."""
+    if not context.rule_set.is_enabled(STUB_FRAGMENT.rule_id):
+        return 0
+    return context.settings(STUB_FRAGMENT.rule_id).integer('minimum_characters', DEFAULT_MINIMUM_CHARACTERS)
+
+
 class StubFragmentRule(Rule):
     metadata = STUB_FRAGMENT
 
     def check(self, group: CommentGroup, context: RuleContext) -> typing.Iterable[Diagnostic]:
         budget = sentence_budget(group, context)
-        minimum_length = context.settings(self.metadata.rule_id).integer(
-            'minimum_characters', DEFAULT_MINIMUM_CHARACTERS
-        )
+        minimum_length = stub_floor(context)
         for sentence in prose_sentences(group):
             split_pieces = reflow_sentence(sentence, budget)
             if len(split_pieces) < 2:

{subedit-0.1.0 → subedit-0.1.2}/src/subedit/domain/rules/structure.py

@@ -1,7 +1,7 @@
 import typing
 
 from ..models.comment import CommentForm, CommentGroup, CommentPlacement
-from ..models.diagnostic import Diagnostic, Fix, FixKind, RuleMetadata
+from ..models.diagnostic import Diagnostic, Fix, FixKind, RuleMetadata, Severity
 from ..ports.language_conventions import LanguageConventions
 from ..ports.rule import Rule, RuleContext
 
@@ -10,6 +10,7 @@ FILE_HEADER_COMMENT = RuleMetadata(
     rule_id='file-header-comment',
     summary='A file must not open with a banner comment before its first declaration.',
     fix_kind=FixKind.REWRITE,
+    default_severity=Severity.WARNING,
 )
 
 PLAIN_COMMENT_ON_PUBLIC = RuleMetadata(
@@ -104,6 +105,7 @@ BLOCK_TOO_LONG = RuleMetadata(
     rule_id='block-too-long',
     summary='A comment block long enough to restate the code has stopped earning its place.',
     fix_kind=FixKind.REWRITE,
+    default_severity=Severity.WARNING,
 )
 
 

{subedit-0.1.0 → subedit-0.1.2}/src/subedit/infrastructure/config_sources/toml_config_source.py

@@ -83,13 +83,16 @@ class TomlConfigSource:
 
     def _to_rule_set(self, config_file: ConfigFile) -> RuleSet:
         enabled_ids = set(self._available_rule_ids)
+        disabled_external_ids: set[str] = set()
         settings: dict[str, RuleSettings] = {}
 
+        # A name subedit does not define belongs to a delegated tool, such as subedit.Semicolon from Vale.
         for rule_id, rule_entry in config_file.rules.items():
-            if rule_id not in self._available_rule_ids:
-                continue
             if rule_entry is False or rule_entry == 'off':
-                enabled_ids.discard(rule_id)
+                if rule_id in self._available_rule_ids:
+                    enabled_ids.discard(rule_id)
+                else:
+                    disabled_external_ids.add(rule_id)
             elif isinstance(rule_entry, RuleTable):
                 settings[rule_id] = RuleSettings(severity=rule_entry.resolved_severity, options=rule_entry.options)
             elif isinstance(rule_entry, str):
@@ -99,4 +102,5 @@ class TomlConfigSource:
             enabled_rule_ids=frozenset(enabled_ids),
             settings=settings,
             line_width=config_file.subedit.line_width,
+            disabled_external_ids=frozenset(disabled_external_ids),
         )