Skip to content

Split four sections of the doc-writer skill into reference files - #1816

Merged
David Pine (IEvangelist) merged 3 commits into
microsoft:mainfrom
alistairmatthews:split-doc-writer-skill-info-references
Oct 6, 2026
Merged

David Pine (IEvangelist) merged 3 commits into
microsoft:mainfrom
alistairmatthews:split-doc-writer-skill-info-references

Conversation

@alistairmatthews

Copy link
Copy Markdown
Collaborator

Summary

The PR moves the following sections from the doc writer SKILL.md file into reference files:

  • Integration Documentation
  • Testing Your Documentation
  • Cross-Referencing
  • Common Documentation Issues

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.

  • Links: Each file has exactly one link in SKILL.md, and every path points to a file that exists. The new cross-referencing link is at SKILL.md:12, and terminal-recordings.md is linked at SKILL.md:678.
  • Content: All six files are different from each other, and each starts with the right heading.
  • No leftovers: The old section is gone from SKILL.md, so the guidance isn't in two places.
  • Links inside the reference files: None of the six files has a relative link, so nothing broke when they moved into references/.
  • Git: Your fix is committed as b738d4c, and the folder has no uncommitted changes.

@alistairmatthews
Alistair Matthews (alistairmatthews) marked this pull request as ready for review October 2, 2026 14:59
Copilot AI balanced review requested due to automatic review settings October 2, 2026 14:59

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 Medium severity · 1 Low severity

Open (4)
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.md to 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.

Comment thread .agents/skills/doc-writer/references/integration-documentation.md
Comment thread .agents/skills/doc-writer/references/testing-your-documentation.md
Comment thread .agents/skills/doc-writer/references/testing-your-documentation.md Outdated
Comment thread .agents/skills/doc-writer/SKILL.md Outdated
@IEvangelist
David Pine (IEvangelist) enabled auto-merge (squash) October 6, 2026 01:29
@IEvangelist
David Pine (IEvangelist) merged commit b591436 into microsoft:main Oct 6, 2026
10 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.

Doc writer skill is too large

3 participants