skill-reviewer
- Repo stars 205
- License MIT
- Author repo GitHub-Copilot-for-Azure
Skill PR Reviewer
Performs thorough, structured code reviews of skill PRs — severity-classified findings with actionable fixes, positive acknowledgment, and a summary table.
When to Use
- Reviewing a PR that adds or modifies a skill under
plugin/skills/or.github/skills/ - Checking skill compliance before submitting a PR
- Auditing an existing skill for quality issues
💡 Note:
.github/skills/meta-skills have different conventions — checklist sections 8-9 apply only toplugin/skills/service skills.
Review Workflow
- Collect — Identify all changed skill files (SKILL.md, references, tests, scripts)
- Check — Run each category from the review checklist
- Classify — Assign severity per the severity guide
- Analyze Routing — Check triggers for conflicts per routing analysis
- Draft — Write the review per the output format
- Validate — Verify suggested fixes are actionable with accurate file/line references
Error Handling
| Error | Remediation |
|---|---|
| Cannot determine changed files | Ask user for the file list or PR number |
| Token counting unavailable | Estimate at ~4 chars per token |
References
- Fluxly category
- Engineering
- Author-declared agents
- No explicit declaration found; this is not inferred or tested compatibility
- Static check
- 94 / 100 · heuristic scan, not runtime safety proof
- Author / version / license
- @microsoft · MIT
- Fluxly token estimate
- Lean
- Fluxly setup estimate
- Plug-and-play
- External API key
- No requirement detected
- Detected OS requirements
- Unspecified
- Runtime requirements
- Unspecified
- Detected file/system behavior
-
- Read-only
- Write / modify
- Detected network behavior
- Local-only
- Install commands
- None (reference only)
Profile is derived at build time from SKILL.md and install vectors. Subject to drift from author intent.
Heads up: 未限定 allowed-tools,默认拥有全部工具权限。
The current SKILL.md does not define a fixed output example. Reviewing a PR that adds or modifies a skill under plugin/skills/ or .github/skills/ Checking skill compliance before submitting a PR Auditing an existing skill for quality issues
Collect — Identify all changed skill files (SKILL.md, references, tests, scripts) Check — Run each category from the review checklist Classify — Assign severity per the severity guide
Error · Remediation Cannot determine changed files · Ask user for the file list or PR number Token counting unavailable · Estimate at ~4 chars per token
Review Checklist Severity Classification Routing Analysis
# Skill PR Reviewer
Performs thorough, structured code reviews of skill PRs — severity-classified findings with actionable fixes, positive acknowledgment, and a summary table.
## When to Use
- Reviewing a PR that adds or modifies a skill under `plugin/skills/` or `.github/skills/`
- Checking skill compliance before submitting a PR
- Auditing an existing skill for quality issues
> 💡 **Note:** `.github/skills/` meta-skills have different conventions — checklist sections 8-9 apply only to `plugin/skills/` service skills.
## Review Workflow
1. **Collect** — Identify all changed skill files (SKILL.md, references, tests, scripts)
2. **Check** — Run each category from the [review checklist](references/review-checklist.md)
3. **Classify** — Assign severity per the [severity guide](references/severity-classification.md)
4. **Analyze Routing** — Check triggers for conflicts per [routing analysis](references/routing-analysis.md)
5. **Draft** — Write the review per the [output format](references/output-format.md)
6. **Validate** — Verify suggested fixes are actionable with accurate file/line references
## Error Handling
| Error | Remediation |
|-------|-------------|
| Cannot determine changed files | Ask user for the file list or PR number |
| Token counting unavailable | Estimate at ~4 chars per token |
## References
- [Review Checklist](references/review-checklist.md)
- [Severity Classification](references/severity-classification.md)
- [Routing Analysis](references/routing-analysis.md)
- [Output Format](references/output-format.md) Author text anchors workflow facts; Fluxly only indexes current sections, terms, files, and commands.
sections -> When to Use → Review Workflow → Error Handling → References
terms -> Note · Collect · Check · Classify · Analyze Routing · Draft · Validate
files/cmd -> plugin/skills/ · .github/skills/ · plugin/skills · .github/skills · references/review-checklist.md · references/severity-classification.md · references/routing-analysis.md · references/output-format.md
body sha256 -> 49eda3deae7a
Decide Fit First
Design Intent
How To Use It
Boundaries And Review