feat: Add comprehensive LSP server notification handling support#47
feat: Add comprehensive LSP server notification handling support#47loyalpartner wants to merge 2 commits intomainfrom
Conversation
- Implement EnhancedNotificationHandler for multiple LSP notification types: - textDocument/publishDiagnostics (diagnostics) - window/showMessage (server messages) - window/logMessage (server logs) - $/progress (progress notifications) - workspace/applyEdit (workspace edits) - workspace/didChangeWorkspaceFolders (folder changes) - workspace/didChangeWatchedFiles (file watcher events) - Add NotificationRegistrationHandler for dynamic registration of notification types from Vim - Integrate notification channel system for async forwarding to Vim - Follow Linus-style architecture: simple data structures, eliminate special cases - Maintain backward compatibility with existing diagnostics handling - All notifications are forwarded as VimActions for consistent processing Resolves user request to handle notifications sent from language servers. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-authored-by: lee <loyalpartner@users.noreply.github.com>
|
@claude ctrl+p , ctrl+n 选中下一个后 窗口会飘到左边 |
|
Claude finished @loyalpartner's task —— View job I'll investigate the completion popup positioning issue where Ctrl+P/Ctrl+N causes the window to drift to the left. Todo List
AnalysisI found and fixed the issue! In Root Cause: The popup used SolutionFixed the positioning drift by:
Key changes in " Store popup position on first creation
let s:completion.popup_position = {
\ 'line': cursor_pos[1] + 1,
\ 'col': cursor_pos[2]
}
" Update content instead of recreating
if s:completion.popup_id != -1
call popup_settext(s:completion.popup_id, a:lines)
return
endifThis ensures the popup stays anchored to its original position during Ctrl+P/Ctrl+N navigation. Changes committed in |
…igation - Store initial popup position to prevent drift when navigating completions - Update popup content using popup_settext() instead of recreating popup - Apply consistent positioning to documentation popup - Resolves issue where completion window would drift left during navigation Co-authored-by: lee <loyalpartner@users.noreply.github.com>
Summary
Changes
Resolves #45
Generated with Claude Code