Skip to content

feat: add Skills and VS Code Extension categories to issue/PR templates#410

Merged
WilliamBerryiii merged 9 commits intomainfrom
copilot/add-skills-and-vscode-categories
Feb 4, 2026
Merged

feat: add Skills and VS Code Extension categories to issue/PR templates#410
WilliamBerryiii merged 9 commits intomainfrom
copilot/add-skills-and-vscode-categories

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 4, 2026

Implementation Plan

  • Update bug-report.yml to add "Skills" and "VS Code Extension" to Component dropdown
  • Create new skill-request.yml template following the pattern of existing request templates
  • Update PULL_REQUEST_TEMPLATE.md to add Skills checkbox and reference link
  • Remove single-platform options from skill-request.yml (only cross-platform allowed)
  • Remove platform requirements section entirely (doesn't add value)
  • Update AI artifact checklist to include "skill" in the comment
  • Add PR template to frontmatter validation exclusion list to prevent CI failures
  • Sync package.json frontmatter exclusions with workflow for consistent dev/CI experience
  • Run validation (lint:yaml, lint:md, lint:frontmatter) to ensure templates pass all checks
  • Verify all changes are correctly implemented
Original prompt

This section details on the original issue you should resolve

<issue_title>[Issue]: add VS Code Extension and Skills categories to issue and PR templates</issue_title>
<issue_description>### Issue Description

Background

The repository recently added Skills as a new AI artifact type (.github/skills/*/SKILL.md) alongside agents, prompts, and instructions. The video-to-gif skill exists and is documented at docs/contributing/skills.md, but the issue templates and PR template do not recognize Skills as a contribution type.

Additionally, the VS Code Extension (extension/) is a core deliverable but is missing from the bug report component list, preventing users from categorizing extension-specific bugs correctly.

Current State

Template Issue
.github/ISSUE_TEMPLATE/bug-report.yml Missing "Skills" and "VS Code Extension" in Component dropdown
.github/ISSUE_TEMPLATE/skill-request.yml Does not exist
.github/PULL_REQUEST_TEMPLATE.md Missing Skills checkbox in AI Artifacts section

Work Required

1. Update Bug Report Template

File: .github/ISSUE_TEMPLATE/bug-report.yml

Add two new options to the Component dropdown:

options:
  - Instruction Files
  - Prompts
  - Custom Agents
  - Skills              # ADD
  - VS Code Extension   # ADD
  - Scripts
  - Documentation
  - Other

2. Create Skill Request Issue Template

File: .github/ISSUE_TEMPLATE/skill-request.yml (new file)

Create template with:

  • name: Skill Request
  • description: Request a new skill package
  • title: [Skill]:
  • labels: skill, needs-triage
  • Fields:
    • Skill Name (input, required)
    • Purpose (textarea, required)
    • Platform Requirements (dropdown: Both bash and PowerShell, Bash only, PowerShell only)
    • Prerequisites (textarea, optional) - external tools needed
    • Requirements (textarea, optional)

3. Update PR Template

File: .github/PULL_REQUEST_TEMPLATE.md

Add Skills checkbox to AI Artifacts section:

**AI Artifacts:**

- [ ] Reviewed contribution with `prompt-builder` agent and addressed all feedback
- [ ] Copilot instructions (`.github/instructions/*.instructions.md`)
- [ ] Copilot prompt (`.github/prompts/*.prompt.md`)
- [ ] Copilot agent (`.github/agents/*.agent.md`)
- [ ] Copilot skill (`.github/skills/*/SKILL.md`)   # ADD

Add Skills reference to the note section:

> - **Skills**: Must include both bash and PowerShell scripts. See [Skills](../docs/contributing/skills.md).

Acceptance Criteria

  • Bug report template includes Skills and VS Code Extension options
  • New skill-request.yml template exists and matches pattern of other request templates
  • PR template includes Skills checkbox and reference link
  • All templates pass npm run lint:yaml validation

Additional Context

No response</issue_description>

<agent_instructions>Start implementation, ensure you run the available validation tools like npm run lint:md``, and npm run lint:frontmatter` or other applicable ones to the types of files changed. Only finish and report back once validation passes.</agent_instructions>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…/PR templates

- Add "Skills" and "VS Code Extension" to bug-report.yml Component dropdown
- Create new skill-request.yml template for requesting new skill packages
- Add Skills checkbox to PR template AI Artifacts section
- Add Skills reference to PR template notes and documentation references

🎯 - Generated by Copilot

Co-authored-by: katriendg <838216+katriendg@users.noreply.github.com>
Copilot AI changed the title [WIP] Add Skills and VS Code Extension categories to templates feat: add Skills and VS Code Extension categories to issue/PR templates Feb 4, 2026
Copilot AI requested a review from katriendg February 4, 2026 13:54
@katriendg
Copy link
Copy Markdown
Contributor

The changes look great, but we need a change that has a discrepancy: in skill-request.yml the drop-down platform-requirements allows a choice that is either or both bash and powershell. Should we remove this as per contributing guidance this is not allowed.

@katriendg
Copy link
Copy Markdown
Contributor

@copilot The changes look great, but we need a change that has a discrepancy: in skill-request.yml the drop-down platform-requirements allows a choice that is either one or the other, both bash AND powershell. Should we remove this as per contributing guidance this is not allowed - and only cross-platform is allowed.

…uidelines

Single-platform skills are explicitly rejected per docs/contributing/skills.md.
Replaced dropdown with markdown notice explaining cross-platform requirement.

Addresses feedback from @katriendg

🔧 - Generated by Copilot

Co-authored-by: katriendg <838216+katriendg@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Feb 4, 2026

@copilot The changes look great, but we need a change that has a discrepancy: in skill-request.yml the drop-down platform-requirements allows a choice that is either one or the other, both bash AND po...

Removed the single-platform options. The template now displays a markdown notice explaining that all skills must support both bash and PowerShell, with a link to the contributing guidelines that explicitly reject single-platform skills.

Commit: 467027b

@katriendg
Copy link
Copy Markdown
Contributor

@copilot remove the platform options entirely, it does not add any value and might confuse users.

The platform requirements section has been removed as it doesn't add value
and might confuse users. All skills must be cross-platform by default per
contributing guidelines.

Addresses feedback from @katriendg

🔧 - Generated by Copilot

Co-authored-by: katriendg <838216+katriendg@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Feb 4, 2026

@copilot remove the platform options entirely, it does not add any value and might confuse users.

Removed the entire platform requirements section. The template now only includes the essential fields: Skill Name, Purpose, Prerequisites, and Requirements.

Commit: 08960c1

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 4, 2026

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@katriendg katriendg marked this pull request as ready for review February 4, 2026 14:44
@katriendg katriendg requested a review from a team as a code owner February 4, 2026 14:44
Copilot AI review requested due to automatic review settings February 4, 2026 14:44
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds “Skills” and “VS Code Extension” as first-class contribution categories across GitHub issue and PR templates to improve triage and contribution guidance (fixes #409).

Changes:

  • Updated bug report “Component” dropdown to include Skills and VS Code Extension.
  • Added a new Skill Request issue form template.
  • Updated the PR template to include Copilot skill in AI Artifacts and linked Skills contribution guidance.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
.github/ISSUE_TEMPLATE/bug-report.yml Extends the Component dropdown to properly categorize Skills/Extension bugs.
.github/ISSUE_TEMPLATE/skill-request.yml Introduces a dedicated issue form for requesting new skill packages.
.github/PULL_REQUEST_TEMPLATE.md Adds Skills to the AI Artifacts checklist and links to Skills contribution docs.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Feb 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.63%. Comparing base (2ce6371) to head (0fc8d03).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #410      +/-   ##
==========================================
- Coverage   74.66%   74.63%   -0.04%     
==========================================
  Files          19       19              
  Lines        3296     3296              
==========================================
- Hits         2461     2460       -1     
- Misses        835      836       +1     
Flag Coverage Δ
pester 74.63% <ø> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@katriendg
Copy link
Copy Markdown
Contributor

@copilot this is accurate feedback:

Changing this file will cause it to be picked up by the repo’s frontmatter validation job (changed-files-only + warnings-as-errors). Since the template has no YAML frontmatter block and no standard Copilot footer, the workflow will emit warnings (treated as errors) and fail CI. Either add the required frontmatter/footer to this template, or update the frontmatter validation configuration to exclude PR/issue templates (for example via ExcludePaths / FooterExcludePaths, or by exempting TEMPLATE.md in the validator).

Add the exclusion for the specific file .github/PULL_REQUEST_TEMPLATE.md.

…from frontmatter validation

- Update PR template checklist to include "skill" alongside agent/prompt/instruction
- Add .github/PULL_REQUEST_TEMPLATE.md to frontmatter validation exclusion list
- Prevents CI failure from warnings-as-errors on GitHub template files

Addresses bot feedback comments #2764415814 and @katriendg feedback #3848338950

🔧 - Generated by Copilot

Co-authored-by: katriendg <838216+katriendg@users.noreply.github.com>
@katriendg
Copy link
Copy Markdown
Contributor

@copilot great work, ensure one final thing. For the frontmatter markdown validation, ensure the exclude paths you changed in ‎.github/workflows/frontmatter-validation.yml are also updated in the developer inner loop option in package.json, line 16 "lint:frontmatter": "pwsh -NoProfile -Command \"& scripts/linting/Validate-MarkdownFrontmatter.ps1 -WarningsAsErrors -EnableSchemaValidation -ExcludePaths 'scripts/tests/Fixtures/**'\"", it is not up to date. Workflow file has the right list.

Update package.json lint:frontmatter script to match the exclusion list
in frontmatter-validation.yml workflow. Ensures consistent validation
between local development (npm run) and CI.

Exclusions now match workflow:
- scripts/tests/Fixtures/**
- extension/README.md
- .github/PULL_REQUEST_TEMPLATE.md

Addresses @katriendg feedback #3848570457

🔧 - Generated by Copilot

Co-authored-by: katriendg <838216+katriendg@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Feb 4, 2026

@copilot great work, ensure one final thing. For the frontmatter markdown validation, ensure the exclude paths you changed in ‎.github/workflows/frontmatter-validation.yml are also updated in the ...

Updated package.json line 16 to include all three exclusions matching the workflow file: scripts/tests/Fixtures/**, extension/README.md, and .github/PULL_REQUEST_TEMPLATE.md. The developer inner loop now matches CI validation.

Commit: 3e1e7eb

@katriendg katriendg marked this pull request as draft February 4, 2026 17:00
@katriendg katriendg marked this pull request as ready for review February 4, 2026 17:57
Copilot AI review requested due to automatic review settings February 4, 2026 17:57
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.

@katriendg
Copy link
Copy Markdown
Contributor

Looks good to me but since I was the initiator of the PR I am not going to approve it. @WilliamBerryiii adding you as a reviewer because of this?

Copy link
Copy Markdown
Member

@WilliamBerryiii WilliamBerryiii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LTGM

@WilliamBerryiii WilliamBerryiii merged commit 108e160 into main Feb 4, 2026
16 checks passed
@WilliamBerryiii WilliamBerryiii deleted the copilot/add-skills-and-vscode-categories branch February 4, 2026 18:54
WilliamBerryiii pushed a commit that referenced this pull request Feb 6, 2026
🤖 I have created a release *beep* *boop*
---


##
[2.2.0](hve-core-v2.1.0...hve-core-v2.2.0)
(2026-02-06)


### ✨ Features

* add incident response prompt template
([#386](#386))
([0adb35c](0adb35c))
* add Skills and VS Code Extension categories to issue/PR templates
([#410](#410))
([108e160](108e160))
* **hve-core-guidance-instructions:** update guidance artifacts and MCP
config ([#402](#402))
([25b34de](25b34de))
* **security:** add action version consistency validation
([#423](#423))
([f3bb787](f3bb787))
* **workflows:** add copyright header validation CI workflow
([#429](#429))
([c53de22](c53de22))


### 🐛 Bug Fixes

* **docs:** add missing Copilot footers, consolidate validation
exclusions ([#419](#419))
([e40f960](e40f960))
* **scripts:** include CIHelpers module + packaging script testability
([#420](#420))
([da26edf](da26edf))


### ♻️ Refactoring

* migrate inline CI code to CIHelpers module
([#393](#393))
([adf6a5f](adf6a5f))


### 🔧 Maintenance

* **templates:** align issue templates with conventional commit format
([#427](#427))
([2d28702](2d28702))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: hve-core-release-please[bot] <254602402+hve-core-release-please[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Issue]: add VS Code Extension and Skills categories to issue and PR templates

5 participants