fix(lsp): correct workspace and document color capabilities and add tailwind lsp - #2736
fix(lsp): correct workspace and document color capabilities and add tailwind lsp#2736bajrangCoder wants to merge 2 commits into
Conversation
Greptile SummaryThe PR corrects LSP workspace-root and document-color capability handling and adds an optional built-in Tailwind CSS server.
Confidence Score: 4/5The watched-file capability should be removed or fully implemented before merging because enabled Tailwind servers can retain stale project state after watched files change. Tailwind now negotiates dynamic watched-file registration, but Acode silently discards the resulting registrations and has no path that sends workspace/didChangeWatchedFiles notifications. Files Needing Attention: src/cm/lsp/clientManager.ts and src/cm/lsp/transport.ts Important Files Changed
Sequence DiagramsequenceDiagram
participant Client as Acode LSP client
participant Server as Tailwind server
participant Files as Project files
Client->>Server: initialize(dynamicRegistration: true)
Server->>Client: client/registerCapability(watched files)
Client-->>Server: null result
Note over Client: Registration is not retained
Files-->>Client: Matching file changes
Note over Client,Server: No workspace/didChangeWatchedFiles notification
Reviews (1): Last reviewed commit: "feat(lsp): add Tailwind CSS server" | Re-trigger Greptile |
| didChangeWatchedFiles: { | ||
| dynamicRegistration: true, |
There was a problem hiding this comment.
Watched-file registration is discarded
When an enabled Tailwind server dynamically registers workspace/didChangeWatchedFiles, the transport acknowledges but discards the registration and the client never emits matching notifications, causing Tailwind completions and diagnostics to remain stale after watched project files change.
Knowledge Base Used: LSP Integration
No description provided.