TerminalFi / LSP-copilot

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

Follow up link rendering / command issue #182

Closed TerminalFi closed 4 months ago

TerminalFi commented 4 months ago

Sometimes copilot returns a link with ` in it. This breaks the link.

Need to investigate

TerminalFi commented 4 months ago

TerminalFi 2024-07-12 at 21 21 03@2x

Looks like we need to insert follow up after mdpopup renders?

jfcherng commented 4 months ago

Could the issue be that double quotes in following up are not escaped?

TerminalFi commented 4 months ago

mdpopups see it and converts it to code. Maybe escaping fixes that

jfcherng commented 4 months ago

Do you know any question that would likely reproduce this issue?

TerminalFi commented 4 months ago

Do you know any question that would likely reproduce this issue?

You can modify the follow up by modifying windows settings for testing and debugging :)

jfcherng commented 4 months ago

nah, I don't even know what the server payload is.

jfcherng commented 4 months ago

Fixed by https://github.com/TerminalFi/LSP-copilot/pull/188