docs(agents): centralize workflow guidance in skills#1051
Merged
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Contributor
There was a problem hiding this comment.
Pull request overview
Centralizes duplicated monorepo workflow guidance into reusable agent “skills”, while simplifying per-package AGENTS.md files and tightening E2E testing instructions.
Changes:
- Removed duplicated workflow/safety/testing guidance from multiple package
AGENTS.mdfiles and referenced shared skills instead - Added
.agents/skills/docs for PR creation, testing workflow, and TypeScript review guardrails - Updated documentation and diagrams (notably E2E instructions and Mermaid identifier escaping)
Reviewed changes
Copilot reviewed 16 out of 16 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| packages/vscode/AGENTS.md | Removes duplicated workflow/testing/safety guidance from VS Code extension agent doc |
| packages/coverage-istanbul/AGENTS.md | Removes duplicated dependency/safety/stuck guidance while keeping package “key files” |
| packages/core/AGENTS.md | Removes large sections (incl. key files/testing notes) as part of centralization effort |
| packages/browser/src/client/AGENTS.md | Fixes Mermaid diagram identifier escaping for __*__ globals |
| packages/browser/src/AGENTS.md | Fixes Mermaid diagram identifier escaping for dispatch globals |
| packages/browser/AGENTS.md | Fixes Mermaid diagram identifier escaping for dispatch globals |
| packages/browser-ui/AGENTS.md | Compresses UI workflow guidance into shorter, more scannable bullets; removes safety/stuck sections |
| packages/browser-react/AGENTS.md | Removes file-scoped command tips and detailed exports/type documentation |
| packages/adapter-rslib/AGENTS.md | Removes usage/API/key-files/safety sections, leaving minimal guidance |
| packages/adapter-rsbuild/AGENTS.md | Adds new minimal adapter agent doc with module structure and commands |
| e2e/AGENTS.md | Deletes E2E agent doc (guidance moved elsewhere) |
| CONTRIBUTING.md | Updates headed browser E2E instructions to run from e2e/ via pnpm test |
| AGENTS.md | Simplifies root agent doc and introduces the “Skills” table referencing .agents/skills/ |
| .agents/skills/typescript/SKILL.md | Adds TypeScript review/writing guardrails (“anti-slop”) |
| .agents/skills/testing/SKILL.md | Adds centralized testing workflow for unit/package/E2E runs |
| .agents/skills/pr/SKILL.md | Adds PR creation workflow guidance (branching, validation, gh usage) |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
2 tasks
9aoy
approved these changes
Mar 12, 2026
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
Background
The repository's agent instructions had workflow steps duplicated across package guides, which made command guidance harder to maintain and left PR/test workflows split across multiple docs.
Implementation
human input:
User Impact
Contributors and coding agents get more consistent instructions with less duplicated maintenance.
Checklist