Skip to content

Add VS Code and VS Code Insiders as skill-supporting clients#4773

Merged
samuv merged 2 commits intomainfrom
skills-vscode
Apr 14, 2026
Merged

Add VS Code and VS Code Insiders as skill-supporting clients#4773
samuv merged 2 commits intomainfrom
skills-vscode

Conversation

@samuv
Copy link
Copy Markdown
Contributor

@samuv samuv commented Apr 13, 2026

Summary

Enables VS Code and VS Code Insiders (GitHub Copilot) as skill-supporting clients alongside claude-code, codex, opencode, and cursor.

Both variants run the same GitHub Copilot extension and share the same skill directories:

  • Personal skills: ~/.copilot/skills/<name> (user scope)
  • Project skills: <project>/.github/skills/<name> (project scope)

Paths sourced from the VS Code Agent Skills docs. No platform prefix is needed — unlike VS Code's MCP config (which lives under Library/Application Support or .config/Code/), the skills dirs are a flat ~/.copilot/ and .github/ on all platforms.

Type of change

  • New feature

Test plan

  • Unit tests (task test)
  • Linting (task lint-fix)

Changes

File Change
pkg/client/config.go Add SupportsSkills, SkillsGlobalPath, SkillsProjectPath to both the VS Code and VS Code Insiders entries
pkg/client/skills_test.go Update fixtures, add TestSupportsSkills cases, bump list count to 7, add path test cases for both clients
pkg/client/discovery_test.go Update VS Code fixture and flip SupportsSkills assertion

Does this introduce a user-facing change?

Yes — thv skill install --clients vscode and thv skill install --clients vscode-insider (or --clients all) now work. Skills land in ~/.copilot/skills/ so GitHub Copilot picks them up automatically in both VS Code variants.

@github-actions github-actions bot added the size/XS Extra small PR: < 100 lines changed label Apr 13, 2026
JAORMX
JAORMX previously approved these changes Apr 13, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.92%. Comparing base (6a79259) to head (a9e2657).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4773      +/-   ##
==========================================
- Coverage   68.94%   68.92%   -0.02%     
==========================================
  Files         517      517              
  Lines       54741    54741              
==========================================
- Hits        37742    37732      -10     
- Misses      14095    14107      +12     
+ Partials     2904     2902       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

@github-actions github-actions bot added size/XS Extra small PR: < 100 lines changed and removed size/XS Extra small PR: < 100 lines changed labels Apr 13, 2026
@samuv samuv changed the title Add VS Code as a skill-supporting client Add VS Code and VS Code Insiders as skill-supporting clients Apr 13, 2026
@github-actions github-actions bot added size/XS Extra small PR: < 100 lines changed and removed size/XS Extra small PR: < 100 lines changed labels Apr 13, 2026
@github-actions github-actions bot added size/XS Extra small PR: < 100 lines changed and removed size/XS Extra small PR: < 100 lines changed labels Apr 13, 2026
@samuv samuv requested a review from JAORMX April 13, 2026 13:46
@github-actions github-actions bot added size/XS Extra small PR: < 100 lines changed and removed size/XS Extra small PR: < 100 lines changed labels Apr 13, 2026
@samuv samuv merged commit 32c23f4 into main Apr 14, 2026
40 of 41 checks passed
@samuv samuv deleted the skills-vscode branch April 14, 2026 07:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/XS Extra small PR: < 100 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants