Skip to content

Add copilot-resource-qa skill for Copilot customization QA 🤖🤖🤖 - #5718

Closed
prakash888kp wants to merge 1 commit into
github:mainfrom
prakash888kp:feat/copilot-resource-qa-skill
Closed

prakash888kp wants to merge 1 commit into
github:mainfrom
prakash888kp:feat/copilot-resource-qa-skill

Conversation

@prakash888kp

Copy link
Copy Markdown

Summary

  • Adds a new Agent Skill copilot-resource-qa that runs structured QA on GitHub Copilot customizations (agents, instructions, skills, hooks, agentic workflows, and plugins).
  • Covers frontmatter/manifest checks, naming, applyTo quality, overlap/conflict detection, Responsible AI/safety fit, and smoke prompts with a PASS / PASS WITH WARNINGS / FAIL verdict.
  • Includes references/checklist.md as a field-level companion checklist and regenerates docs/README.skills.md.

Why this is a valid contribution

  • Fills a catalog gap: existing resources create or suggest skills/agents (microsoft-skill-creator, suggest-awesome-github-copilot-*) or cover app QA (qa-subagent, test-gap-audit, quality-playbook). None provide a dedicated pre-merge QA gate for Copilot resource files themselves.
  • Specific and actionable (not generic “write better prompts”), with clear uplift over base-model defaults.
  • Follows Agent Skills conventions: folder/name match, trigger-rich description, MIT license, bundled reference under size limits.

Test plan

  • npm run skill:validate (copilot-resource-qa valid)
  • npm run build (skill appears in docs/README.skills.md)
  • Maintainer: spot-check skill against CONTRIBUTING / AGENTS.md checklists
  • Maintainer: smoke-prompt the skill on a sample agent/instruction/skill in this repo

Provides a structured read-only review workflow for agents, instructions, skills, hooks, workflows, and plugins before they ship.

Co-authored-by: Cursor <cursoragent@cursor.com>
@prakash888kp
prakash888kp requested review from a team as code owners October 27, 2026 08:05
@github-actions github-actions Bot added new-submission PR adds at least one new contribution skills PR touches skills labels Oct 27, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🔒 PR Risk Scan Results

Scanned 2 changed file(s).

Severity Count
🔴 High 0
🟠 Medium 0
ℹ️ Info 0

✅ No matching risk patterns were detected in changed files.

This is an automated soft-gate report. Findings indicate review targets and do not block merge by themselves.

@github-actions

github-actions Bot commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

🚦 Submission status: 👀 Ready for review

Risk tier: merge-risk:medium — New or substantially changed resource without privileged execution
Required to merge: passing submission-gate checks plus 1 approval from reviewers with write access.

Why this tier
  • Adds 2 file(s), e.g. skills/copilot-resource-qa/SKILL.md
  • Changes 284 lines (low-risk limit is 40)

Automated checks

Check Status Details
Line endings ✅ Passed Passed · logs
Spelling ✅ Passed Passed · logs
Generated README consistency ✅ Passed Passed · logs
Skill validation ✅ Passed Passed · logs
Skill lint (vally) ✅ Passed Passed · logs
Risk scan ✅ Passed Passed · logs
Contributor reputation ✅ Passed Passed · logs
Duplicate resource scan ✅ Passed Passed · logs
PR quality signal ⏭️ Skipped Skipped by its workflow · logs

Review

  • Approvals: 0/1
  • Assigned reviewer: not assigned yet — comment /request-review to ask for one
  • Review target date: not set
  • Still needed: 1 more approval(s)
  • No staffed reviewer pool owns these files yet; any reviewer with write access counts as the domain reviewer.

Commands

Command Who What it does
/rerun-checks PR author, maintainers Re-runs failed or incomplete checks and re-evaluates this gate
/request-review PR author, maintainers Asks the review rotation to assign a reviewer (adds needs-reviewer)

Updated for 5d85d42 · gate run · This comment is maintained automatically — see submission gate docs.

@github-actions

Copy link
Copy Markdown
Contributor

🔍 Vally Lint Results

✅ All checks passed

Scope Checked
Skills 1
Agents 0
Total 1
Severity Count
❌ Errors 0
⚠️ Warnings 0
ℹ️ Advisories 0

Summary

Level Finding
ℹ️ ✅ copilot-resource-qa (2/2 checks passed)
ℹ️ ✓ [spec-compliance] All 1 skill(s) are spec-compliant.
ℹ️ ✓ spec-compliance: All spec checks passed.
ℹ️ ✓ [valid-refs] All file references across 1 skill(s) are valid.
ℹ️ ✓ valid-refs: All file references resolve to existing files within the skill directory.
ℹ️ 1 skill(s) linted, 1 passed
Full linter output
### Linting skills/copilot-resource-qa
✅ copilot-resource-qa (2/2 checks passed)
    ✓ [spec-compliance] All 1 skill(s) are spec-compliant.
        ✓ spec-compliance: All spec checks passed.
    ✓ [valid-refs] All file references across 1 skill(s) are valid.
        ✓ valid-refs: All file references resolve to existing files within the skill directory.

1 skill(s) linted, 1 passed

@github-actions github-actions Bot added ready-for-review Submission passed intake validation and is ready for maintainer review and removed awaiting-automation labels Oct 27, 2026
@aaronpowell aaronpowell closed this Nov 3, 2026
@aaronpowell

Copy link
Copy Markdown
Contributor

This is a little too specific around how to write a submission for this repo and isn't broadly applicable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge-risk:medium new-submission PR adds at least one new contribution ready-for-review Submission passed intake validation and is ready for maintainer review skills PR touches skills

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants