Files
Miranda LimonczenkoandNik Richers f5fcecf3d7 docs: point the authoring skills at the style guide (#50744)
Part 2 of 3. Stack: #50742 → #50744 → #50743. Review #50742 first.


## Problem

Six skills restated style rules inline, so a rule could be corrected in
the guide and stay wrong in a skill. `edit-the-docs` alone carried a
second copy of the procedure format, the information-type classification
rules, and the tables outline example.

Two reference files said in their own text that they should be retired
once a style guide existed.

## Solution

Replace the restatements with pointers to the file that owns each rule.

**Retired, as each file asked:**

- `style-fallback.md` is deleted. It ended by telling an agent to follow
the nearest comparable page, which launders whatever that page happens
to do into a rule. The guide's References section replaces it.
- `common-pitfalls.md` becomes a pointer, per the note at its own line
94.

**Rewired**: `write-the-docs`, `edit-the-docs`, `review-the-docs`,
`pm-the-docs`'s checklist, and `drafting-mechanics.md`.

Skills cite a specific file rather than the directory, so one file can
be loaded instead of the whole guide.

`review-the-docs` gains a docs-tooling check for the inverse case: a
style rule added to a skill belongs in the guide, with the skill
pointing at it.

## Notes for review

**`edit-the-docs`' PR 1 / PR 2 boundary is unchanged on purpose.** That
split is by kind of diff — PR 1 is inline changes only, nothing moves a
line — which is what makes each PR reviewable. The guide's files split
by the size of the thing they govern, and the two cut across each other:
choosing an admonition is an element decision but an inline diff, and
chunking is a page-structure decision but currently applied in PR 1.
Forcing them to match would stop PR 1 being a pure inline pass.

Dropping the dash-aside rule from `write-the-docs` here lost it
entirely, since it had no home in the guide. #50742 restores it in
`01-voice-and-tone.md`. I audited the other four rules this PR removes
from that checklist; only that one was lost.

The skill's document-type list keeps `troubleshooting`, which CodeRabbit
flagged as a fifth type not in the guide. The repo has 219
troubleshooting pages and a content-type gate that treats them as
authorable, so the gap was in the guide. #50742 now lists five document
types.

## Manual testing

1. Run `grep -rn "style-fallback\|apps/docs/WORD_LIST" .agents/skills/`
and confirm no matches.
2. Open each rewired skill and confirm every style guide link resolves,
including anchors such as `03-page-structure.md#chunking`.
3. Invoke `/edit-the-docs` and confirm it reads the guide rather than
restating rules.
4. Run `npx prettier --config prettier.config.mjs --check
".agents/skills/*-the-docs/**/*.md"`.


<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

* **Documentation**
* Updated documentation writing, editing, and review guidance to
reference the dedicated style guide for voice, terminology, page
structure, and content elements.
* Clarified how to classify and organize sections, and expanded
style-consistency checks.
* Updated related checklists and references to distinguish style
guidance from repository contribution instructions.
* Consolidated common drafting advice into the style guide and updated
the page-type table layout.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

## Stack order

This PR moved above the `CONTRIBUTING.md` trim after review. The trim
deletes the
style sections that seven skill instructions still referenced, so
trimming first
left those references dangling until this PR landed. Rewiring the skills
first
removes that intermediate state: the skills point at the guide while
`CONTRIBUTING.md` is still whole, and the trim then breaks nothing.

---------

Co-authored-by: Nik Richers <nik@validmind.ai>
2026-09-25 20:11:58 -07:00
..