Skip to content

style: sweep em-dashes from non-.md files#23

Merged
mattmaynes merged 1 commit into
mainfrom
chore/ascii-non-md
Jun 25, 2026
Merged

style: sweep em-dashes from non-.md files#23
mattmaynes merged 1 commit into
mainfrom
chore/ascii-non-md

Conversation

@mattmaynes

Copy link
Copy Markdown
Collaborator

Finishes the em-dash sweep. PR #20 covered only .md files; this does the rest of the repo per the relaxed Trellis rule.

Swept (25 dashes, 11 files)

test.sh, scripts/token-report.sh, scripts/check-commit-msg.sh, .github/workflows/ci.yml, spectra/hooks/pre-commit, spectra/personas.config, the three marketplace.json manifests, and both SVG label texts.

Every occurrence was in a comment, a description string, echo text, or an SVG label - none in a regex, generated output, or the token-comparison block. Verified:

  • JSON manifests still parse.
  • bash test.sh -> PASS; README token figures regenerated (spectra/personas.config shrank by 2 bytes).
  • Zero em/en dashes remain anywhere in the tree.

Pure formatting, no behavior change -> self-review per the protocol's review scoping.

🤖 Generated with Claude Code

PR #20's sweep was .md-only; this finishes the job across the rest of the repo
per the relaxed Trellis rule (em/en dashes banned, other Unicode kept). All 25
were in comments, description strings, echo text, or SVG labels - no regex,
generated output, or token-comparison content. JSON manifests still parse;
README token figures regenerated since spectra/personas.config shrank.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@mattmaynes mattmaynes merged commit d330222 into main Jun 25, 2026
3 checks 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