Consolidate the AI review rubric into AGENTS.md - #5158
Conversation
Scope the automated review to collateral changes, structural fit, and text tone, and drop the checks that Vale and CI already cover. Removes .github/skills/code-review/SKILL.md. Copilot reads a skill only when its description matches, and Codex does not read .github/skills at all, so AGENTS.md is the only file every reviewer reads.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
📖 Docs PR preview linksThis PR does not change any pages in /docs. If you make updates, links to the modified pages will appear here. |
There was a problem hiding this comment.
Pull request overview
Consolidates automated documentation-review guidance in the repository-wide AGENTS.md and removes the redundant Copilot-specific skill.
Changes:
- Focuses reviews on collateral changes, structural fit, and text tone.
- Documents checks handled by CI or Vale as out of scope.
- Removes the superseded code-review skill.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
AGENTS.md |
Adds the consolidated review rubric. |
.github/skills/code-review/SKILL.md |
Removes duplicate review guidance. |
💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
@codex review this |
|
To use Codex here, create a Codex account and connect to github. |
|
@codex review this |
|
Codex Review: Didn't find any major issues. Breezy! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
What
Scopes the automated review guidance to the three things a lint rule cannot check: collateral changes, structural fit, and text tone. Drops the terminology, redirect, sidebar, link, and Mermaid checks that Vale and CI already run.
Moves the rubric into
AGENTS.mdand removes.github/skills/code-review/SKILL.md. Copilot loads a skill only when its description matches, and Codex does not read.github/skills, soAGENTS.mdis the only file every reviewer reads.