Skip to content

Add lspServers config to clangd plugin#25

Closed
micahstubbs wants to merge 1 commit intoPiebald-AI:mainfrom
micahstubbs:12/clangd-add-lspservers-config
Closed

Add lspServers config to clangd plugin#25
micahstubbs wants to merge 1 commit intoPiebald-AI:mainfrom
micahstubbs:12/clangd-add-lspservers-config

Conversation

@micahstubbs
Copy link
Copy Markdown
Contributor

Summary

  • Add lspServers configuration to clangd plugin entry in marketplace.json

Fixes #12

Root Cause

Claude Code reads LSP configuration from the lspServers field in marketplace.json, not from .lsp.json files in plugin directories.

Test Plan

  • Install/update the clangd plugin
  • Restart Claude Code session
  • Verify LSP works on .c/.cpp/.h/.hpp files

Fixes Piebald-AI#12

Claude Code reads LSP configuration from the lspServers field in
marketplace.json, not from .lsp.json files in plugin directories.
@bl-ue bl-ue closed this Jan 30, 2026
@bl-ue
Copy link
Copy Markdown
Member

bl-ue commented Jan 30, 2026

@shyney7 I see you've reacted to this PR; are you able to help me understand the value of it? I'm not aware of .lsp.json not being used, and no one has really complained about these LSPs not working as they're currently configured. I'm totally fine with fixing them if there's a problem, but I'd like to understand the need first. This PR adds a lspServers field to marketplace.json, but the contents of that lspServers field are already in the clangd plugin's .lsp.json, so this would result in duplication.

@bl-ue bl-ue reopened this Jan 30, 2026
@jeong-sik
Copy link
Copy Markdown
Contributor

Friendly ping — any feedback on this? Happy to adjust if needed.

@bl-ue bl-ue closed this Mar 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

clangd plugin missing lspServers config in marketplace.json

3 participants