chore(claude): consolidate code-intelligence LSP plugins#64
Conversation
typescript-lsp, rust-analyzer-lsp, eslint-lsp를 code-intelligence-lsp@code-intelligence 단일 플러그인으로 통합하고, code-review@pleaseai / please@passionfactory / standards@passionfactory / review@passionfactory / plannotator@passionfactory를 enabledPlugins에서 제거했다. bun@pleaseai와 claude-md-management@claude-plugins-official는 유지한다.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe change modifies the ChangesPlugin configuration update
Estimated code review effort: 1 (Trivial) | ~2 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Code Review
This pull request updates the '.claude/settings.json' file by removing several language-specific LSP and utility plugins in favor of a single 'code-intelligence-lsp' plugin. The review feedback correctly identifies that this consolidated plugin does not exist in the official marketplace, which would break code intelligence for TypeScript and Rust. The reviewer suggests using the official, individual LSP plugins instead.
Up to standards ✅🟢 Issues
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
There was a problem hiding this comment.
No issues found across 1 file
Architecture diagram
sequenceDiagram
participant Claude as Claude Desktop
participant Settings as .claude/settings.json
participant PluginLoader as Plugin Loader
participant CI as code-intelligence-lsp@code-intelligence
participant TS as typescript-lsp@code-intelligence
participant RA as rust-analyzer-lsp@code-intelligence
participant ESL as eslint-lsp@code-intelligence
participant Others as Removed Plugins
Note over Claude,Others: Configuration Parsing at Startup
Claude->>Settings: Read enabledPlugins
Settings-->>Claude: {"code-intelligence-lsp": true, "bun": true, "claude-md-management": true}
Claude->>PluginLoader: Initialize plugins from enabled list
PluginLoader->>CI: Load code-intelligence-lsp@code-intelligence
PluginLoader-xOthers: Skip code-review@pleaseai (not enabled)
PluginLoader-xOthers: Skip please@passionfactory
PluginLoader-xOthers: Skip standards@passionfactory
PluginLoader-xOthers: Skip review@passionfactory
PluginLoader-xOthers: Skip plannotator@passionfactory
PluginLoader-->>Claude: Plugin initialization complete
Note over CI,Runtime: Code Intelligence Features (now consolidated)
Claude->>CI: Request file analysis
CI->>CI: Handle TypeScript, Rust, ESLint via single plugin
Note right of CI: Unified LSP integration replaces 3 separate plugins
CI-->>Claude: Analysis results
Note over Others: Legacy plugins no longer loaded
|



Summary
Consolidate the code-intelligence LSP plugins into a single entry and prune unused/duplicate enabled plugins in
.claude/settings.json.Changes
typescript-lsp@code-intelligence,rust-analyzer-lsp@code-intelligence,eslint-lsp@code-intelligencewith the singlecode-intelligence-lsp@code-intelligenceplugincode-review@pleaseai,please@passionfactory,standards@passionfactory,review@passionfactory,plannotator@passionfactoryfromenabledPluginsbun@pleaseaiandclaude-md-management@claude-plugins-officialTest Plan
.claude/settings.jsonchanged, no runtime code affectedSummary by cubic
Consolidated LSP plugins in
.claude/settings.json: replacedtypescript-lsp@code-intelligence,rust-analyzer-lsp@code-intelligence, andeslint-lsp@code-intelligencewithcode-intelligence-lsp@code-intelligence; removedcode-review@pleaseai,please@passionfactory,standards@passionfactory,review@passionfactory,plannotator@passionfactory. Keptbun@pleaseaiandclaude-md-management@claude-plugins-official; simpler config, no runtime changes.Written for commit 92ee8b0. Summary will update on new commits.
Summary by CodeRabbit