Skip to content

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

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

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

This branch has not been deployed

No deployments
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

1 participant