Skip to content

write-chapter: allow Unicode box-drawing inside fenced diagrams#41

Merged
yagop merged 1 commit into
mainfrom
skill-allow-unicode-diagrams
Jun 17, 2026
Merged

write-chapter: allow Unicode box-drawing inside fenced diagrams#41
yagop merged 1 commit into
mainfrom
skill-allow-unicode-diagrams

Conversation

@yagop

@yagop yagop commented Jun 17, 2026

Copy link
Copy Markdown
Owner

Follows the commit-hook change that now permits non-ASCII inside ``` fenced blocks. This relaxes the matching skill convention so generated chapters can draw cleaner diagrams.

What changed in SKILL.md

  • Appendix B, punctuation rule: "ASCII punctuation only" → "ASCII punctuation in prose and code", with an explicit exception: inside a fenced diagram you may use Unicode box-drawing and arrow characters. Everything outside a fence stays ASCII.
  • Appendix B, visual-aids rule: "AT MOST one small ASCII diagram" → "one small diagram", with guidance to prefer Unicode box-drawing inside a fence (ASCII art still fine), line connectors up under the node they descend from, give every descending line an arrowhead, and render the page to check it.
  • Appendix A, review checklist: the "Non-ASCII punctuation" item is now scoped to outside fenced diagrams, and a new item says to render the page and eyeball the diagram for alignment / missing arrowheads — the exact defect behind the chapter 3 and chapter 5 flowcharts.

The skill text itself stays pure ASCII (it describes the chars by name, doesn't embed them), so the convention is just guidance — nothing about existing chapters changes.

Verification

  • SKILL.md has zero non-ASCII bytes (so this commit passed the hook normally).
  • bunx tsc --noEmit clean and bun x vitepress build passes — existing chapters (all ASCII diagrams) are unaffected.

Note: this is the convention/guidance side. The enabling change is in the global commit hook (~/.claude/hooks/check-commit-chars.mjs), which now skips the blocklist for lines inside ``` fences — that's not in this repo, so it's not part of this PR.

🤖 Generated with Claude Code

Relax the ASCII-punctuation convention: prose and code stay ASCII, but a
fenced diagram (``` block) may use Unicode box-drawing and arrows for a
cleaner picture - the commit hook now permits non-ASCII inside fences.
Also add a review-checklist item to render the page and check that
diagram connectors line up and every descending line has an arrowhead
(the defect behind the chapter 3 and 5 flowcharts).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@yagop yagop merged commit 3c5db59 into main Jun 17, 2026
1 check passed
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