Add lsp-setup skill for configuring LSP servers in Copilot CLI#1272
Open
brunoborges wants to merge 3 commits intostagedfrom
Open
Add lsp-setup skill for configuring LSP servers in Copilot CLI#1272brunoborges wants to merge 3 commits intostagedfrom
brunoborges wants to merge 3 commits intostagedfrom
Conversation
Adds a new skill that helps users install and configure Language Server Protocol servers for GitHub Copilot CLI. Includes: - Interactive workflow: detect OS, install server, write config, verify - Bundled reference with 14 languages (Java/jdtls, TypeScript, Python, Go, Rust, C/C++, C#, Ruby, PHP, Kotlin, Swift, Lua, YAML, Bash) - Per-OS install commands and ready-to-use lsp-config.json snippets - Support for both user-level and repo-level configuration Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
🔍 Skill Validator Results2 resource(s) checked | ✅ All checks passed Full output |
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a new lsp-setup skill to help users install/configure Language Server Protocol (LSP) servers for GitHub Copilot CLI, including bundled per-language install guidance and example configuration snippets.
Changes:
- Add
skills/lsp-setupskill instructions describing an interactive LSP setup workflow and config locations/format. - Add bundled reference data (
references/lsp-servers.md) covering install commands + config snippets for 14 languages. - Update
docs/README.skills.mdto include the new skill entry.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| skills/lsp-setup/SKILL.md | Defines the LSP setup workflow, config locations, and expected JSON structure. |
| skills/lsp-setup/references/lsp-servers.md | Provides per-language install commands and config snippets used by the skill. |
| docs/README.skills.md | Registers the new skill in the skills index. |
Comments suppressed due to low confidence (1)
skills/lsp-setup/SKILL.md:28
- This section lists only
~/.copilot/lsp-config.jsonand.github/lsp.jsonas config locations, but the Learning Hub plugin docs also mentionlsp.jsonat the repo root as a valid location. If rootlsp.jsonis supported, include it here (and in the scope question) so the skill doesn’t omit a documented option.
Copilot CLI reads LSP configuration from two locations (repo-level takes precedence):
- **User-level**: `~/.copilot/lsp-config.json`
- **Repo-level**: `.github/lsp.json`
- Update SKILL.md description to mention both config paths (user-level and repo-level) instead of only lsp-config.json - Clarify workflow step 6 with explicit file paths - Add note in references/lsp-servers.md explaining snippets are objects to insert under the lspServers key - Regenerate docs/README.skills.md with updated description Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Comment on lines
+24
to
+28
| Copilot CLI reads LSP configuration from two locations (repo-level takes precedence): | ||
|
|
||
| - **User-level**: `~/.copilot/lsp-config.json` | ||
| - **Repo-level**: `.github/lsp.json` | ||
|
|
There was a problem hiding this comment.
The skill only offers repo-level config at .github/lsp.json, but repo docs also mention lsp.json at the repo root as a valid location for LSP server configs. Consider offering both options (and clarifying precedence) so users don’t end up writing to a path Copilot CLI won’t read in their setup.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds a new lsp-setup skill that guides users through installing and configuring Language Server Protocol (LSP) servers for GitHub Copilot CLI.
What it does
The skill provides an interactive workflow:
lsp-config.jsonconfiguration (user-level or repo-level)Included languages
Bundled reference data covers 14 languages with per-OS install commands and ready-to-use config snippets:
Files changed
skills/lsp-setup/SKILL.md— skill definition and agent instructionsskills/lsp-setup/references/lsp-servers.md— bundled reference datadocs/README.skills.md— auto-generated (vianpm run build)Validation
npm run skill:validate— all 272 skills validnpm run build— README and marketplace regeneratedbash scripts/fix-line-endings.sh— line endings normalized