Skip to content

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

Merged
WilliamBerryiii merged 2 commits intomainfrom
chore/767-skill-child-paths
Feb 25, 2026
Merged

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

Conversation

@katriendg
Copy link
Contributor

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:

  • GitHub Actions workflow
  • Linting configuration (markdown, PowerShell, etc.)
  • Security configuration
  • DevContainer configuration
  • Dependency update

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:

  1. User reads SKILL.md Quick Start or Script Reference section
  2. Copies a script command from the code block (e.g., scripts/generate.sh or scripts/convert.sh input.mp4)
  3. Runs the command from the skill root directory
  4. 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
@katriendg katriendg requested a review from a team as a code owner February 25, 2026 11:04
@github-actions
Copy link
Contributor

github-actions bot commented Feb 25, 2026

Dependency Review

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

Scanned Files

None
katriendg added a commit that referenced this pull request Feb 25, 2026
- remove changes covered by PR #768

🔄 - Generated by Copilot
katriendg added a commit that referenced this pull request Feb 25, 2026
- remove changes covered by PR #768

🔄 - Generated by Copilot
@WilliamBerryiii WilliamBerryiii merged commit 4de0a01 into main Feb 25, 2026
23 checks passed
WilliamBerryiii pushed a commit that referenced this pull request Feb 28, 2026
## 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>
WilliamBerryiii pushed a commit that referenced this pull request Feb 28, 2026
## 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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants