Repository navigation
docs: remove prose em dashes from evi and skills surfaces (U-14) - #703
Merged
Merged
Conversation
…ration contract Every dash is judged against the U-14 exceptions (code blocks, verbatim quotes, numeric ranges); code-block and verbatim-template occurrences stay. The self-review contract check now names the first-class vs guide-level distinction EVL-361 landed in the content doctrine.
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
|
Important Review skippedBot user detected. To trigger a single review, invoke the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
commit: |
Contributor
|
Thank you for following the naming conventions! 🙏 |
HugoRCD
marked this pull request as ready for review
September 17, 2026 21:10
This was referenced Sep 18, 2026
This branch had an error being deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Removes every prose em dash from the surfaces the daily content pass cannot reach, and aligns the self-review contract check with the tiered integration contract EVL-361 landed in the content doctrine.
Why
U-14 (
no em dashes, no en dashes in prose, on any surface) applies everywhere, but the content-lint corpus only walksapps/docs/content,skills/,.agents/skills/, and AGENTS.md files;apps/evi/docs/andapps/evi/agent/skills/are never scanned, so drift there survives. A hand scan found prose dashes in all of those unscanned trees, plus oneU-14candidate on a scanned docs page (5.use-cases/5.eve.md).Files (18)
apps/evi/docs/(observability, authorization, notes, vision, capability-placement): headings like### evlog/eve — let ...become### evlog/eve: let ..., appositives become commas or parentheses, sentence joins become periods or colons.apps/evi/agent/skills/(before-after, daily-digest, cost-watchdog, linear-maintenance, self-review, upstream-sync): prose and frontmatter descriptions; theDaily digest — YYYY-MM-DDdocument-title templates becomeDaily digest: YYYY-MM-DD.skills/(review-logging-patterns, build-audit-logs): table glosses become colons or commas. The en-dash numeric ranges (1 – 7 years,30 – 90 days) stay: U-14 exempts ranges between two numbers..agents/skills/(create-map-rule, create-framework-integration, create-enricher): prose, frontmatter templates, and table cells.apps/docs/content/5.use-cases/5.eve.md: the one scanned page still carrying prose dashes.apps/evi/agent/skills/self-review/SKILL.md: the integration contract check now says "every first-class framework integration", and names the guide-level tier, matching the doctrine correction in EVL-361.What stays
Dashes inside code blocks and verbatim templates stay (U-14 exempts them): the AGENTS.md tree diagram, code comments in skill templates, and the
### {title} — {question}heading format increate-map-rule, which prescribes the format the docs rules pages actually use.Checks
pnpm run lint: 22/22 tasks successful.pnpm content:linton all 129 pages: zeroU-14candidates remaining; the self-review skill'sT-15critical was also resolved by the contract rewrite (the old row'sevlog/sharedliteral tripped the retired-entrypoint rule).No changeset:
apps/*and skills prose only, no package consumer impact.