Skip to content

Add lsp-setup skill for configuring LSP servers in Copilot CLI#1272

Open
brunoborges wants to merge 5 commits intostagedfrom
skill/lsp-setup
Open

Add lsp-setup skill for configuring LSP servers in Copilot CLI#1272
brunoborges wants to merge 5 commits intostagedfrom
skill/lsp-setup

Conversation

@brunoborges
Copy link
Copy Markdown
Contributor

@brunoborges brunoborges commented Apr 2, 2026

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:

  1. Asks which language(s) the user wants LSP support for
  2. Detects the user's OS (macOS / Linux / Windows)
  3. Installs the appropriate LSP server
  4. Writes the lsp-config.json configuration (user-level or repo-level)
  5. Verifies the setup

Included languages

Bundled reference data covers 14 languages with per-OS install commands and ready-to-use config snippets:

Language LSP Server
TypeScript / JavaScript typescript-language-server
Java Eclipse JDT Language Server (jdtls)
Python pylsp / Pyright
Go gopls
Rust rust-analyzer
C / C++ clangd
C# csharp-ls
Ruby solargraph
PHP intelephense
Kotlin kotlin-language-server
Swift sourcekit-lsp
Lua lua-language-server
YAML yaml-language-server
Bash bash-language-server

Files changed

  • skills/lsp-setup/SKILL.md — skill definition and agent instructions
  • skills/lsp-setup/references/lsp-servers.md — bundled reference data
  • docs/README.skills.md — auto-generated (via npm run build)

Validation

  • ✅ npm run skill:validate — all 272 skills valid
  • ✅ npm run build — README and marketplace regenerated
  • ✅ bash scripts/fix-line-endings.sh — line endings normalized

@brunoborges
Add lsp-setup skill for configuring LSP servers in Copilot CLI
c6590f5 
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 <[email protected]>
@brunoborges brunoborges requested a review from aaronpowell as a code owner April 2, 2026 19:21
Copilot AI review requested due to automatic review settings April 2, 2026 19:22
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 2, 2026 •
edited
Loading

🔍 Skill Validator Results

2 resource(s) checked | ✅ All checks passed

Full output 
Found 1 skill(s)
[lsp-setup] 📊 lsp-setup: 815 BPE tokens [chars/4: 830] (detailed ✓), 6 sections, 1 code blocks
�[32m✅ All checks passed (1 skill(s))�[0m

Copilot AI reviewed Apr 2, 2026
View reviewed changes
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 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-setup skill 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.md to 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.json and .github/lsp.json as config locations, but the Learning Hub plugin docs also mention lsp.json at the repo root as a valid location. If root lsp.json is 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`

@brunoborges
Address review: clarify config filenames and snippet format
9c76394 
- 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 <[email protected]>
@brunoborges brunoborges requested a review from Copilot April 2, 2026 19:36
Copilot AI reviewed Apr 2, 2026
View reviewed changes
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 3 out of 3 changed files in this pull request and generated 2 comments.

@brunoborges brunoborges enabled auto-merge April 6, 2026 16:09
@hubwriter
Copy link
Copy Markdown

hubwriter commented Apr 8, 2026

This works really nicely.

I tested it by installing LSP servers for Python and Rust.

The only thing I find a little strange (probably nothing to do with this PR) is that after the installation is complete I expected to be able to do /lsp reload and then see the new server listed when I did /lsp but in both cases I had to quit and start a new CLI session to see the new server listed.

This makes adding an LSP server way easier.

Is this going to be available from this page: https://awesome-copilot.github.com/skills/? I'll document that it's findable there by searching for "lsp" - if that's not correct please let me know.

aaronpowell
aaronpowell requested changes Apr 9, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@aaronpowell aaronpowell left a comment

Choose a reason for hiding this comment

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

Just want to get an update on the .NET/C# part to align with the guidance from the .NET team.

skills/lsp-setup/references/lsp-servers.md

## C# (.NET)

**Server**: [OmniSharp](https://github.com/OmniSharp/omnisharp-roslyn) or [csharp-ls](https://github.com/razzmatazz/csharp-language-server)
Copy link
Copy Markdown
Contributor

@aaronpowell aaronpowell Apr 9, 2026

Choose a reason for hiding this comment

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

This isn't correct, it should be using the roslyn-language-server, see https://github.com/dotnet/skills/blob/main/plugins/dotnet/lsp.json for a reference.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@aaronpowell aaronpowell aaronpowell requested changes

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@brunoborges @hubwriter @aaronpowell