Skip to content

hygiene(_patterns.md): line-leading list-marker confusion as recurring class#470

Merged
AceHack merged 1 commit intomainfrom
drain/patterns-line-leading-list-marker
Apr 25, 2026
Merged

hygiene(_patterns.md): line-leading list-marker confusion as recurring class#470
AceHack merged 1 commit intomainfrom
drain/patterns-line-leading-list-marker

Conversation

@AceHack
Copy link
Copy Markdown
Member

@AceHack AceHack commented Apr 25, 2026

Summary

Promote the 'CommonMark MD032 — line-leading `+` / `-` parsed as list marker' pattern to a named recurring-findings class in `docs/pr-preservation/_patterns.md` after 5 instances surfaced in a single tick (2026-04-25):

Two-branch fix decision tree

The pattern documents both fix paths:

Composes with the existing Class A inline-code-span line-wrap pattern (same shape: line-wrap hits a CommonMark special character at line start). Pre-commit-lint candidate noted for the doc-lint suite tracked in #465.

Test plan

🤖 Generated with Claude Code

Promote the 'CommonMark MD032 — line-leading + / - parsed as
list marker' pattern to a named recurring-findings class in
_patterns.md after 5 instances in a single tick (2026-04-25):

- #377: '+ manifest files' continuation line
- #280: '+ provenance-aware scoring' continuation
- #195: '+ Kenji (Architect)' owner-list continuation
- #235: '+ git history' (Amara verbatim ferry)
- #219: '- [link]' (Amara verbatim ferry)

Document the two-branch fix decision tree:

- Author-controlled prose → reflow inline (replace '+'-separator
  with comma, extend prior line, etc).
- Verbatim ferry / courier content (Otto-227) → add to
  markdownlint ignore list (precedent: docs/aurora/** in #469,
  docs/amara-full-conversation/**).

Composes with Class A inline-code-span line-wrap (same shape:
line-wrap hits a CommonMark special character at line start).
Pre-commit-lint candidate noted for the doc-lint suite from
#465.
@AceHack AceHack enabled auto-merge (squash) April 25, 2026 08:50
Copilot AI review requested due to automatic review settings April 25, 2026 08:50
@AceHack AceHack merged commit 5ef3fe7 into main Apr 25, 2026
15 checks passed
@AceHack AceHack deleted the drain/patterns-line-leading-list-marker branch April 25, 2026 08:53
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Promotes the recurring markdownlint MD032 “line-leading list marker after a wrap” issue into a named pattern entry in docs/pr-preservation/_patterns.md, documenting both author-controlled and verbatim-preservation fix paths.

Changes:

  • Adds a new recurring-findings entry documenting MD032-triggering line-wraps that start with +/-/*/1..
  • Captures a decision tree distinguishing author-editable prose vs Otto-227 verbatim ferry content.
  • Notes a potential pre-commit lint direction for preventing recurrences.

Comment thread docs/pr-preservation/_patterns.md
Comment thread docs/pr-preservation/_patterns.md
Comment thread docs/pr-preservation/_patterns.md
AceHack added a commit that referenced this pull request Apr 25, 2026
- L139: 'CommonMark MD032' → 'markdownlint MD032' (MD032 is the
  markdownlint rule ID 'blanks-around-lists', not a CommonMark
  identifier; the trigger uses CommonMark-style list markers but
  the lint rule itself is markdownlint's).
- L145: '#280' citation reframed — there's no
  docs/pr-preservation/280-drain-log.md in the corpus. The #280
  fix landed at write-time (commit cce1df2) before any drain log
  was filed for it. Cite the fix-commit rather than a per-PR log
  so the observation is verifiable from the stated corpus.
- L170: 'Pre-commit-lint candidate' wording was vague about
  whether detection exists. The repo already has
  tools/hygiene/audit-md032-plus-linestart.sh covering the
  '+'-at-line-start case. Update to point at the existing audit +
  state the new work clearly: extend to '- '/'* ' or promote to
  pre-commit enforcement. Drop 'Class A' label (this doc doesn't
  use that label; replace with the actual prior heading
  'Inline-code-span line-wrap rendering bug').
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants