From 674f2a5234a31aaf125f65685fc6636ebe15eb67 Mon Sep 17 00:00:00 2001 From: cmj Date: Wed, 23 Sep 2026 08:21:25 +0800 Subject: [PATCH 1/2] feat(lsp): say whether a repo needs a language server Add the `lsp` skill, triggered by `need lsp`. It counts tracked source files per language, judges whether a language server would pay off, and checks whether the official LSP plugin and its binary are already on. It reports the missing install steps and never installs anything. Smith does not call it automatically; the magic word is the trigger. Refs: #32 Co-Authored-By: Claude Opus 5.5 (1M context) --- .claude-plugin/marketplace.json | 4 +- README.md | 1 + README.zh-TW.md | 1 + plugins/wisdom/.claude-plugin/plugin.json | 2 +- plugins/wisdom/skills/lsp/SKILL.md | 67 +++++++++++++++++++++++ 5 files changed, 72 insertions(+), 3 deletions(-) create mode 100644 plugins/wisdom/skills/lsp/SKILL.md diff --git a/.claude-plugin/marketplace.json b/.claude-plugin/marketplace.json index 3bacce7..23c65ab 100644 --- a/.claude-plugin/marketplace.json +++ b/.claude-plugin/marketplace.json @@ -1,6 +1,6 @@ { "name": "wisdom", - "version": "3.0.0", + "version": "3.1.0", "owner": { "name": "cmj", "url": "https://github.com/cmj0121" @@ -9,7 +9,7 @@ { "name": "wisdom", "description": "Single-purpose skills to plan, build, review and ship software, led by smith.", - "version": "3.0.0", + "version": "3.1.0", "source": "./plugins/wisdom" } ] diff --git a/README.md b/README.md index e55f2e0..0e1f5fa 100644 --- a/README.md +++ b/README.md @@ -68,6 +68,7 @@ other skills in turn. Every skill also works on its own. | `audit` | Audit dependencies for known vulnerabilities and staleness | `audit deps` | | `ops` | Review operational readiness before a release | `ops review` | | `challenge` | Argue against a plan and name its blind spots | `tenth man` | +| `lsp` | Say whether the repo needs a language server, and which | `need lsp` | ### Output diff --git a/README.zh-TW.md b/README.zh-TW.md index 7446e9f..2ef03d4 100644 --- a/README.zh-TW.md +++ b/README.zh-TW.md @@ -65,6 +65,7 @@ v3 從零重寫，圍繞三條規則。 | `audit` | 稽核相依套件的已知漏洞與過期版本 | `audit deps` | | `ops` | 發佈前審查維運就緒程度 | `ops review` | | `challenge` | 反對一項計畫，點出其盲點 | `tenth man` | +| `lsp` | 判斷專案是否需要 language server | `need lsp` | ### 輸出 diff --git a/plugins/wisdom/.claude-plugin/plugin.json b/plugins/wisdom/.claude-plugin/plugin.json index 71f1bae..95bf650 100644 --- a/plugins/wisdom/.claude-plugin/plugin.json +++ b/plugins/wisdom/.claude-plugin/plugin.json @@ -1,7 +1,7 @@ { "name": "wisdom", "description": "Single-purpose skills to plan, build, review and ship software, led by smith.", - "version": "3.0.0", + "version": "3.1.0", "license": "MIT", "author": { "name": "cmj", diff --git a/plugins/wisdom/skills/lsp/SKILL.md b/plugins/wisdom/skills/lsp/SKILL.md new file mode 100644 index 0000000..c5ba7e5 --- /dev/null +++ b/plugins/wisdom/skills/lsp/SKILL.md @@ -0,0 +1,67 @@ +--- +name: lsp +description: Checks whether a repo is large and typed enough for a language server to pay off, and names the official LSP plugin to enable. Use before heavy code work. +license: MIT +metadata: + shortcut: "need lsp" +--- + +# LSP + +Decide whether enabling a language server is worth it for this repo. A language server gives +the agent go-to-definition, find-references and post-edit diagnostics: fewer broad greps and +whole-file reads, and type errors caught before a build. It only pays off where there is +enough typed code to navigate. Report — never install a plugin, a binary or a setting. + +## Steps + +1. **Count source files** per language with `git ls-files`, so vendored and ignored files + stay out. Map extensions with the table below; anything else is not a candidate. + + | Language | Extensions | Plugin | Binary | + | -------- | --------------------- | ------------------- | ---------------------------- | + | TS / JS | `.ts .tsx .js .jsx` | `typescript-lsp` | `typescript-language-server` | + | Python | `.py .pyi` | `pyright-lsp` | `pyright-langserver` | + | Go | `.go` | `gopls-lsp` | `gopls` | + | Rust | `.rs` | `rust-analyzer-lsp` | `rust-analyzer` | + | C / C++ | `.c .h .cpp .cc .hpp` | `clangd-lsp` | `clangd` | + | C# | `.cs` | `csharp-lsp` | `csharp-ls` | + | Java | `.java` | `jdtls-lsp` | `jdtls` | + | Kotlin | `.kt .kts` | `kotlin-lsp` | `kotlin-lsp` | + | Swift | `.swift` | `swift-lsp` | `sourcekit-lsp` | + | Ruby | `.rb .rake .erb` | `ruby-lsp` | `ruby-lsp` | + | PHP | `.php` | `php-lsp` | `intelephense` | + | Lua | `.lua` | `lua-lsp` | `lua-language-server` | + + Plugins come from the `claude-plugins-official` marketplace. If its catalogue is cached + locally, check it for the current names before trusting this table. + +2. **Judge each language.** Worth it when the language has **100 or more** files, or fewer + but one package with deep cross-file calls (a monorepo of small files counts). Not worth + it for a repo that is mostly Markdown, YAML or shell, or for a language under 20 files. + The numbers are a rule of thumb; say so when a language sits near them. + +3. **Check what is already on.** For each worth-it language: + - **Plugin enabled:** `@claude-plugins-official` is `true` under `enabledPlugins` + in `~/.claude/settings.json`, `.claude/settings.json` or `.claude/settings.local.json`. + An `LSP` tool in this session means at least one server is loaded. + - **Binary installed:** `command -v `. The plugin does not ship it; without it + the plugin loads and reports "Executable not found in $PATH" under `/plugin` → Errors. + +## Output + +One row per language that has any files, largest first: + +| Language | Files | Worth it? | Plugin | Enabled | Binary | +| -------- | ----- | --------- | ------ | ------- | ------ | + +Then, for each worth-it language not fully on, the missing steps only: + +```text +/plugin install @claude-plugins-official +``` + +and that `` must be installed on `PATH` with the language's own toolchain — see the +plugin's README. Restart the session after enabling so the server loads. + +End with one sentence: enable LSP for which languages, or not needed and why. From e6ca15533af757921aab28fd8479ed429d0b5072 Mon Sep 17 00:00:00 2001 From: cmj Date: Wed, 23 Sep 2026 08:22:19 +0800 Subject: [PATCH 2/2] chore: release the lsp skill as 3.0.1 Co-Authored-By: Claude Opus 5.5 (1M context) --- .claude-plugin/marketplace.json | 4 ++-- plugins/wisdom/.claude-plugin/plugin.json | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/.claude-plugin/marketplace.json b/.claude-plugin/marketplace.json index 23c65ab..2d90925 100644 --- a/.claude-plugin/marketplace.json +++ b/.claude-plugin/marketplace.json @@ -1,6 +1,6 @@ { "name": "wisdom", - "version": "3.1.0", + "version": "3.0.1", "owner": { "name": "cmj", "url": "https://github.com/cmj0121" @@ -9,7 +9,7 @@ { "name": "wisdom", "description": "Single-purpose skills to plan, build, review and ship software, led by smith.", - "version": "3.1.0", + "version": "3.0.1", "source": "./plugins/wisdom" } ] diff --git a/plugins/wisdom/.claude-plugin/plugin.json b/plugins/wisdom/.claude-plugin/plugin.json index 95bf650..f2e574c 100644 --- a/plugins/wisdom/.claude-plugin/plugin.json +++ b/plugins/wisdom/.claude-plugin/plugin.json @@ -1,7 +1,7 @@ { "name": "wisdom", "description": "Single-purpose skills to plan, build, review and ship software, led by smith.", - "version": "3.1.0", + "version": "3.0.1", "license": "MIT", "author": { "name": "cmj",