chore(skills): standardize script path references in SKILL.md files by katriendg · Pull Request #768 · microsoft/hve-core

katriendg · 2026-02-25T11:04:02Z

Description

Standardized all script path references in SKILL.md files to use bare relative paths from the skill root directory, removing ./ prefixes and repo-root-relative paths that break portability.

The prompt-builder.instructions.md guidance states that all paths within a skill must be relative to the skill root, never repo-root-relative. These SKILL.md files were not following that convention, which breaks when skills are distributed via extension, CLI, or personal skill directories.

Updated video-to-gif SKILL.md with 21 path reference changes across Quick Start, Script Reference, Examples, and Parameters sections. Two repo-root-relative paths (./.github/skills/experimental/video-to-gif/scripts/convert.*) were replaced with scripts/convert.*, and 19 ./convert.* references were standardized to scripts/convert.* for both bash and PowerShell code blocks.

Updated pr-reference SKILL.md with 15 path reference changes across Quick Start, Additional Scripts Reference, and Parameters Reference sections. All ./scripts/ prefixes were replaced with bare scripts/ for both bash and PowerShell code blocks.

Related Issue(s)

Closes #767

Type of Change

Select all that apply:

Code & Documentation:

Bug fix (non-breaking change fixing an issue)
New feature (non-breaking change adding functionality)
Breaking change (fix or feature causing existing functionality to change)
Documentation update

Infrastructure & Configuration:

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)

Note for AI Artifact Contributors:

Agents: Research, indexing/referencing other project (using standard VS Code GitHub Copilot/MCP tools), planning, and general implementation agents likely already exist. Review .github/agents/ before creating new ones.

Skills: Must include both bash and PowerShell scripts. See Skills.

Model Versions: Only contributions targeting the latest Anthropic and OpenAI models will be accepted. Older model versions (e.g., GPT-3.5, Claude 3) will be rejected.

See Agents Not Accepted and Model Version Requirements.

Note on Prompt-builder feedback during implementation:

    ### Prompt-Builder Instructions Analysis

    Our prompt-builder.instructions.md already has excellent guidance in the "File References" section:

    > All paths within a skill must be relative to the skill root, never repo-root-relative.
    > Repo-root-relative paths like `./.github/skills/<collection>/<skill>/scripts/...` break portability across all distributed contexts.

    And in the "Skill Invocation from Callers" section:

    **The guidance is correct, but our skills don't follow it.**

Other:

Script/automation (.ps1, .sh, .py)
Other (please describe):

Sample Prompts (for AI Artifact Contributions)

User Request:

Invoke the pr-reference or video-to-gif skill using the script path examples in the SKILL.md documentation.

Execution Flow:

User reads SKILL.md Quick Start or Script Reference section
Copies a script command from the code block (e.g., scripts/generate.sh or scripts/convert.sh input.mp4)
Runs the command from the skill root directory
Script resolves correctly regardless of installation context (repo checkout, extension, CLI plugin)

Output Artifacts:

pr-reference: .copilot-tracking/pr/pr-reference.xml containing commit history and unified diffs
video-to-gif: output.gif file converted from the input video

Testing

Markdown linting (npm run lint:md): ✅ Passed (252 files, 0 errors)
Spell checking (npm run spell-check): ✅ Passed (189 files, 0 issues)
Frontmatter validation (npm run lint:frontmatter): ✅ Passed (238 files, 0 errors)
Skill structure validation (npm run validate:skills): ✅ Passed (2 skills, 0 errors)
Link validation (npm run lint:md-links): ✅ Passed
PowerShell analysis (npm run lint:ps): ✅ Passed
Security analysis: No sensitive data exposure, dependency vulnerabilities, or privilege escalation concerns identified. Changes are limited to documentation path references within markdown code blocks.
Diff-based assessment: Verified zero remaining ./scripts/, ./convert.*, or .github/skills/.*/scripts/ references in either SKILL.md file.

Manual testing performed with VS code, scripts paths resolved correctly when invoking the skill through /pull-request prompt.

Checklist

Required Checks

Documentation is updated (if applicable)
Files follow existing naming conventions
Changes are backwards compatible (if applicable) (N/A — no functional changes, path references only)
Tests added for new functionality (if applicable) (N/A — no new functionality)

AI Artifact Contributions

Used /prompt-analyze to review contribution
Addressed all feedback from prompt-builder review
Verified contribution follows common standards and type-specific requirements

Required Automated Checks

The following validation commands must pass before merging:

Markdown linting: npm run lint:md
Spell checking: npm run spell-check
Frontmatter validation: npm run lint:frontmatter
Skill structure validation: npm run validate:skills
Link validation: npm run lint:md-links
PowerShell analysis: npm run lint:ps

GHCP Artifact Maturity

File	Type	Maturity	Notes
`video-to-gif/SKILL.md`	Skill	✅ stable	All builds
`pr-reference/SKILL.md`	Skill	✅ stable	All builds

…kills 🔧 - Generated by Copilot

github-actions · 2026-02-25T11:04:26Z

Dependency Review

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

Scanned Files

None

- remove changes covered by PR #768 🔄 - Generated by Copilot

## Pre-Release 3.1.44 ### ✨ Features - add Docusaurus 3 documentation site with GitHub Pages deployment (#680) - add workflow permissions validation for OpenSSF Scorecard compliance (#759) - add DT coach return path handoff to task-researcher (#591) (#758) - add DT subagent handoff workflow instructions (#592) (#757) - create dt-method-06-deep.instructions.md (#602) (#748) - create dt-method-05-deep.instructions.md (#747) - add DT-aware task-implementor context instructions (#755) - extract embedded PowerShell from workflows into testable scripts (#738) - add gitleaks binary-based secret scanning as PR gate (#734) - add SBOM generation, attestation, and diff tooling to release pipeline (#730) - add dt-learning-tutor agent for DT education (#662) - add DT image prompt generation guidance for Method 5 (#726) - add DT-aware task-reviewer review context (#714) - add dt-method-next routing prompt (#713) - create dt-method-04-deep.instructions.md (#709) - add Implementation Space exit handoff prompt for DT workflows (#708) - add Write-CIStepSummary markdown table to Test-SHAStaleness github output (#660) - add dt-handoff-solution-space prompt for Solution Spac… (#707) ### 🐛 Bug Fixes - update sidebar link color to meet WCAG AA contrast requirements (#814) - harden even/odd versioning against regression and syntax errors (#816) - replace even/odd versioning with SemVer -rc.N suffixes (#811) - ensure prerelease label exists before PR creation (#806) - replace Docusaurus favicons with Microsoft logo (#808) - add missing subagents and shared instructions to collection manifests (#804) - standardize file path conventions for copilot-tracking output (#784) - enforce project-scoped artifact isolation across DT files (#766) - add top-level permissions to copilot-setup-steps.yml (#760) - update broken file directives and markdown links after collection directory reorg (#743) - add pre-release companion pipeline with even/odd versioning (#735) - exclude auto-generated CHANGELOG.md from spell check (#756) - add job-level permissions to extension-publish.yml (#729) - resolve handoff dependencies using display names (#727) - add job-level permissions to validate-version in extension-publish-prerelease (#731) - replace parent-directory VS Code settings paths with per-subdirectory enumeration (#732) ### 📚 Documentation - add Design Thinking documentation and DT-to-RPI handoff (#789) - add customization guides for HVE Core artifacts (#772) - reconcile documentation against implementation (#771) - document accepted Token-Permissions risks and add lint:dependency-pinning (#763) - add Design Thinking section to hve-core-all collection description (#762) ### ♻️ Refactoring - move collection scripts from plugins to collections (#728) - remove duplicate git diff logic in frontmatter validator (#473) ### 🔧 Maintenance - bump basic-ftp from 5.0.5 to 5.2.0 (#780) - standardize script path references in SKILL.md files (#768) - bump the github-actions group across 1 directory with 2 updates (#752) --- *Managed automatically by pre-release workflow.* Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

## Pre-Release 3.1.46 ### ✨ Features - add Docusaurus 3 documentation site with GitHub Pages deployment (#680) - add workflow permissions validation for OpenSSF Scorecard compliance (#759) - add DT coach return path handoff to task-researcher (#591) (#758) - add DT subagent handoff workflow instructions (#592) (#757) - create dt-method-06-deep.instructions.md (#602) (#748) - create dt-method-05-deep.instructions.md (#747) - add DT-aware task-implementor context instructions (#755) - extract embedded PowerShell from workflows into testable scripts (#738) - add gitleaks binary-based secret scanning as PR gate (#734) - add SBOM generation, attestation, and diff tooling to release pipeline (#730) - add dt-learning-tutor agent for DT education (#662) - add DT image prompt generation guidance for Method 5 (#726) - add DT-aware task-reviewer review context (#714) - add dt-method-next routing prompt (#713) - create dt-method-04-deep.instructions.md (#709) - add Implementation Space exit handoff prompt for DT workflows (#708) - add Write-CIStepSummary markdown table to Test-SHAStaleness github output (#660) - add dt-handoff-solution-space prompt for Solution Spac… (#707) ### 🐛 Bug Fixes - update prerelease publish to use even/odd convention (#822) - update sidebar link color to meet WCAG AA contrast requirements (#814) - harden even/odd versioning against regression and syntax errors (#816) - replace even/odd versioning with SemVer -rc.N suffixes (#811) - ensure prerelease label exists before PR creation (#806) - replace Docusaurus favicons with Microsoft logo (#808) - add missing subagents and shared instructions to collection manifests (#804) - standardize file path conventions for copilot-tracking output (#784) - enforce project-scoped artifact isolation across DT files (#766) - add top-level permissions to copilot-setup-steps.yml (#760) - update broken file directives and markdown links after collection directory reorg (#743) - add pre-release companion pipeline with even/odd versioning (#735) - exclude auto-generated CHANGELOG.md from spell check (#756) - add job-level permissions to extension-publish.yml (#729) - resolve handoff dependencies using display names (#727) - add job-level permissions to validate-version in extension-publish-prerelease (#731) - replace parent-directory VS Code settings paths with per-subdirectory enumeration (#732) ### 📚 Documentation - add Design Thinking documentation and DT-to-RPI handoff (#789) - add customization guides for HVE Core artifacts (#772) - reconcile documentation against implementation (#771) - document accepted Token-Permissions risks and add lint:dependency-pinning (#763) - add Design Thinking section to hve-core-all collection description (#762) ### ♻️ Refactoring - move collection scripts from plugins to collections (#728) - remove duplicate git diff logic in frontmatter validator (#473) ### 🔧 Maintenance - pre-release 3.1.44 (#819) - bump basic-ftp from 5.0.5 to 5.2.0 (#780) - standardize script path references in SKILL.md files (#768) - bump the github-actions group across 1 directory with 2 updates (#752) --- *Managed automatically by pre-release workflow.* Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

chore(skills): update script paths in video-to-gif and pr-reference s…

6795a03

…kills 🔧 - Generated by Copilot

katriendg requested a review from a team as a code owner February 25, 2026 11:04

katriendg added a commit that referenced this pull request Feb 25, 2026

revert(skills): restore pr-reference SKILL.md to main

c87dc14

- remove changes covered by PR #768 🔄 - Generated by Copilot

katriendg added a commit that referenced this pull request Feb 25, 2026

revert(agents): restore agents to main

2aeab9f

- remove changes covered by PR #768 🔄 - Generated by Copilot

WilliamBerryiii approved these changes Feb 25, 2026

View reviewed changes

Merge branch 'main' into chore/767-skill-child-paths

fe31e23

WilliamBerryiii merged commit 4de0a01 into main Feb 25, 2026
23 checks passed

hve-core-release-please bot mentioned this pull request Feb 28, 2026

chore(main): pre-release 3.1.46 #823

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

chore(skills): standardize script path references in SKILL.md files#768

chore(skills): standardize script path references in SKILL.md files#768
WilliamBerryiii merged 2 commits intomainfrom
chore/767-skill-child-paths

katriendg commented Feb 25, 2026

Uh oh!

github-actions bot commented Feb 25, 2026 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

katriendg commented Feb 25, 2026

Description

Related Issue(s)

Type of Change

Sample Prompts (for AI Artifact Contributions)

Testing

Checklist

Required Checks

AI Artifact Contributions

Required Automated Checks

GHCP Artifact Maturity

Uh oh!

github-actions bot commented Feb 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Dependency Review

Scanned Files

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

github-actions bot commented Feb 25, 2026 •

edited

Loading