Repository navigation
Add Copilot review instructions for community articles - #519
Merged
Merged
Conversation
Adds a path-specific Copilot custom instructions file scoped to content/articles/**/*.md so Copilot code review and cloud agent checks community-submitted articles for: - unsafe content (destructive commands, credential harvesting, prompt injection, phishing links, Code of Conduct violations) - AI slop (generic filler, no firsthand specificity, untested-looking code, re-explained docs, listicle padding, missing PR validation evidence) - off-topic/self-promotional pitches that use PowerShell as a backdrop Copilot can only leave advisory PR comments, never block or reject; a maintainer always makes the final call. Also adds the repo's first GLOSSARY.md, defining AI slop, vibe coded, and off-topic pitch as precise, reusable terms.
✅ Deploy Preview for powershellorg ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Contributor
There was a problem hiding this comment.
🟡 Changes recommended
The validation-evidence check can falsely flag issue-form PRs whose generated descriptions omit submitter evidence, so it should check the article or linked submission before flagging missing validation.
1 open issue
What changed in this PR
Adds path-specific Copilot review guidance for community articles, extending the repository-wide instructions without changing site behavior.
Changes:
- Defines checks for unsafe content, low-quality AI-generated content, and off-topic promotion.
- Requires specific evidence and a triage summary while leaving merge decisions to maintainers.
- Adds glossary definitions for the review terminology.
| File | Description |
|---|---|
GLOSSARY.md |
Defines AI slop, vibe coded, and off-topic pitch. |
.github/instructions/articles.instructions.md |
Adds advisory review checks scoped to article Markdown files. |
Review effort: Balanced
Comment on lines
+54
to
+61
| - No verification evidence in the PR description when the article contains | ||
| non-trivial executable code. Accepted article PRs on this repo consistently | ||
| include a "Validation" section naming the exact module/tool versions | ||
| tested against, the commands run (e.g. `npm run build`, | ||
| `npm run validate:articles`), and confirmation that external links were | ||
| checked. A PR introducing executable code with no such evidence anywhere | ||
| in its description is a signal worth flagging — not proof of slop on its | ||
| own, but consistent with content that wasn't actually run. |
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
Path-specific Copilot custom instructions, scoped to community article submissions only (layers on top of the existing repo-wide
.github/copilot-instructions.md, which both review surfaces already apply alongside it). Copilot checks three categories on every article PR:Built from:
Evidence
applyTo: "content/articles/**/*.md"glob verified against the live tree: matches 1,210 files, correctly including both article bodies and_index.mdlisting pages (per explicit scoping call), no false matches outsidecontent/articles/.applyTois the only key, no YAML errors).Merge Danger
Door: Two-way — it's a markdown instructions file; edit or delete anytime with no migration cost.
Blast Radius: Low. Only affects Copilot's own review comments on PRs touching
content/articles/**/*.md; no build, routing, or runtime behavior changes. Worst case is a false-positive advisory comment, which a maintainer can dismiss — Copilot has no merge/block authority.