Repository navigation
Split four sections of the doc-writer skill into reference files - #1816
Merged
David Pine (IEvangelist) merged 3 commits intoOct 6, 2026
Conversation
Alistair Matthews (alistairmatthews)
marked this pull request as ready for review
October 2, 2026 14:59
Alistair Matthews (alistairmatthews)
requested a review
from David Pine (IEvangelist)
as a code owner
October 2, 2026 14:59
Contributor
There was a problem hiding this comment.
Warning
Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.
Copilot review overview
Review effort: Lite
Findings: 3
Open (4)
This table uses a double leading pipe (||), which many Markdown renderers interpret as an extra… · New The Playwright command example uses<relevant-spec>.ts, but the file pattern specified just above… · New The Playwright command example uses<relevant-spec>.ts, but the file pattern specified just above… · New There are a couple of grammatical/wording issues in the new link intro lines: (1) 'Cross… · New
What changed in this PR
This PR reduces the size of the doc-writer skill by moving four large guidance sections out of SKILL.md into dedicated reference files and linking to them from the skill entrypoint.
Changes:
- Added four new reference files under
.agents/skills/doc-writer/references/for integration docs, testing, cross-referencing, and common issues. - Updated
.agents/skills/doc-writer/SKILL.mdto link to the new references and removed the inlined sections.
| File | Description |
|---|---|
| .agents/skills/doc-writer/SKILL.md | Adds links to the new reference files and removes the large inlined guidance blocks to keep the skill smaller. |
| .agents/skills/doc-writer/references/integration-documentation.md | New reference containing integration doc location + structure templates. |
| .agents/skills/doc-writer/references/testing-your-documentation.md | New reference describing validation strategy, test expectations, and local testing workflow. |
| .agents/skills/doc-writer/references/cross-referencing.md | New reference describing how to link to docs, use ApiReference, and format “See also”. |
| .agents/skills/doc-writer/references/common-documentation-issues.md | New reference compiling common PR feedback rules for docs authoring. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
David Pine (IEvangelist)
enabled auto-merge (squash)
October 6, 2026 01:29
David Pine (IEvangelist)
approved these changes
Oct 6, 2026
David Pine (IEvangelist)
disabled auto-merge
October 6, 2026 01:29
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.


Summary
The PR moves the following sections from the doc writer SKILL.md file into reference files:
Links to all four new reference files are included in SKILL.md.
This ensures that, when agents load the doc-writer skill, they apply these four sections reliably. When SKILL.md files become as large as it was, agents sometime don't load the entire skill.
Fixes: #1811
Third-party links and affiliations
None
Validation
I asked Claude to check that an agent using the doc-writer skill would be able to access and use all the reference files. The response was:
An agent using the doc-writer skill can now reach all six reference files.