search

TerminalFi / LSP-copilot

GitHub Copilot support for Sublime Text LSP plugin provided through Copilot.vim.
625 stars 25 forks source link

chore: bump server to 1.10.0 #119

Closed jfcherng closed 10 months ago

jfcherng commented 10 months ago

This PR doesn't work by simply updating the server. I got

:: [05:53:05.974] <-  LSP-copilot LogMessage: {'message': '[DEBUG] [getCompletions] [2023-08-19T21:53:05.968Z] Producing empty completions due to document version mismatch. Completions requested for document version 0 but document version was 34.', 'level': 0, 'extra': ['Producing empty completions due to document version mismatch. Completions requested for document version 0 but document version was 34.'], 'metadataStr': '[DEBUG] [getCompletions] [2023-08-19T21:53:05.968Z]'}
:: [05:53:05.974] <<< LSP-copilot (49) (duration: 8ms): {'cancellationReason': 'DocumentVersionMismatch', 'completions': []}

in LSP's log panel.

Also, there is a new notification type.

:: [05:55:55.381] <?  LSP-copilot featureFlagsNotification: {'ssc': False, 'chat': True, 'rt': False}
TerminalFi commented 10 months ago

Guess we need to finally handle doc. Versions

TerminalFi commented 10 months ago

@jfcherng We can update utils.py:198 with "version": view.change_count() for now. Which is what LSP uses as the version. Copilot accepts this as well

jfcherng commented 10 months ago

Tested PR is working on my Win64 PC.