Skip to content

Fix heading swallowed into body text at Bylaws Article 5, section 5.7.5 - #75

Open
amfriesz wants to merge 3 commits into
developmentfrom
58-fix-5-7-5-heading-swallowed
Open

Fix heading swallowed into body text at Bylaws Article 5, section 5.7.5#75
amfriesz wants to merge 3 commits into
developmentfrom
58-fix-5-7-5-heading-swallowed

Conversation

@amfriesz

@amfriesz amfriesz commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Insert the missing blank line before ### **5.7.5** in bylaws/article-05-director-conduct.qmd so it renders as its own heading and appears in the page's table of contents, instead of being glued onto the 5.7.4 paragraph text.
  • Wrap Article 5 prose to ~80 columns, matching the convention established in commit 06815ad for Policy 1.6 (tables/headings/separators untouched, no long token like a URL broken across lines, list hanging indents sized to marker width). Content is unchanged — verified identical under whitespace normalization.
  • Add a non-blocking PreToolUse git-commit hook (.claude/hooks/check-prose-wrap.py + .claude/settings.json) that warns when staged .qmd/.md prose exceeds 80 columns and could have been wrapped, so this convention is easier to keep going forward.

Fixes #58

Test plan

  • Confirmed the heading now renders and TOC includes 5.7.5 (blank-line diff verified as a single-line insertion)
  • Verified prose-wrap diff is whitespace-normalization-identical to the pre-wrap content (no wording changes)
  • Ran the new hook against the staged file and confirmed zero violations
  • Visual check of the rendered Quarto page once built

🤖 Generated with Claude Code

amfriesz and others added 3 commits August 21, 2026 16:14
The heading "### **5.7.5**" had no blank line before it, directly
following the 5.7.4 paragraph text. This caused the literal heading
markup to render as plain text glued onto 5.7.4's paragraph instead of
becoming its own heading, and 5.7.5 was missing from the page's table
of contents.

Fixes #58

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Matches the convention established in commit 06815ad for Policy 1.6:
prose paragraphs and list items wrapped to ~80 chars, tables/headings/
separators untouched, no long token (e.g. a URL) broken across lines,
and list item hanging indents sized to each item's marker width.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Warns (non-blocking) when staged .qmd/.md prose lines exceed 80
columns and could have been wrapped, per the convention set in commit
06815ad. Skips headings, tables, separators, YAML front matter, code
blocks, and lines whose overflow is a single unbreakable token (e.g.
a URL).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@amfriesz
amfriesz requested a review from BenGalewsky August 21, 2026 20:16
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.

1 participant