Add doc-pr-reviewer skill for single-PR docs accuracy review#1298
Open
IEvangelist wants to merge 5 commits into
Open
Add doc-pr-reviewer skill for single-PR docs accuracy review#1298IEvangelist wants to merge 5 commits into
IEvangelist wants to merge 5 commits into
Conversation
Adds a new .agents/skills/doc-pr-reviewer skill derived from the docs-accuracy reviewer workflow prompt, reworked to review a single PR passed as a parameter (number or URL) instead of selecting PRs by the docs-from-code label. Removes the selection loop, the rinse-and-repeat loop, and the draft/conflict/branch-mismatch skip preconditions. Keeps the Phase A (claims verification against microsoft/aspire source), Phase B (doc-tester run), and Phase C (merged single review) discipline. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a new agent skill (doc-pr-reviewer) intended to review a single aspire.dev documentation PR for factual accuracy against microsoft/aspire, then validate the docs experience via doc-tester, and finally post one merged review.
Changes:
- Introduces
.agents/skills/doc-pr-reviewer/SKILL.mddefining a 3-phase single-PR docs accuracy workflow (claims verification, doc-tester run, merged review). - Documents branch/SHA “source of truth” selection for
microsoft/aspire, including a fallback when no matching branch exists. - Defines a combined output format and review-verdict rules based on Phase A/Phase B results.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…right-cli Routes claims to CommunityToolkit/Aspire as the source of truth for Community Toolkit integration changes (CommunityToolkit.Aspire.* packages / integrations docs), while keeping microsoft/aspire for core Aspire. Clarifies that Phase A and Phase B are independent and may run in parallel for speed (Phase B's only input is the PR's changed files, not Phase A's catalog), and makes Phase B explicitly require the playwright-cli skill rather than Playwright directly. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Removes manual mid-paragraph line breaks so paragraphs and list items are single lines, matching sibling skills (tooling soft-wraps the text). No content changes. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…ost-review cleanup to doc-pr-reviewer Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…R review Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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
Adds a new
doc-pr-reviewerskill at.agents/skills/doc-pr-reviewer/SKILL.md, derived from the docs-accuracy reviewer workflow prompt but reworked to review a single pull request passed as a parameter rather than running on a schedule.What it does
The skill verifies that every factual claim a documentation PR makes about Aspire matches the
microsoft/aspiresource code (Phase A), validates the docs as a new user via thedoc-testerskill (Phase B), and posts a single merged review (Phase C).Changes from the original reviewer prompt
# Inputreplaces# Selection— takes one PR (number or URL); nodocs-from-codelabel filter, oldest-first sort, or skip-already-reviewed loop.# Rinse and repeat— no looping over multiple PRs.doc-testerpath to this repo's actual location (.agents/skills/doc-tester/SKILL.md).# Safety— removed the hourly-loop "next run continues" language; kept read-only rules and[partial]labeling.