Skip to content

copilot chat poc #343

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Conversation

knilink
Copy link
Contributor

@knilink knilink commented Sep 25, 2024

chrome-extension-nlipoenfbbikpbjkfpfillcgkoblgpmj-setup-react-html-from-install.webm

just to share some findings about how conversation feature can possibly be implemented for anyone who is interested

language server will behave in 2 different way depends on "source" parameter provided in the conversation/create request:

  • panel: server will analysis the conversation at the end of the response and will suggest title and follow up question
  • inline: server will try to update the code and notify editor about the new file content, may need to select a region otherwise it's assumed the entire filed is selected and the llm may be confused about what it needs to do

the server use $/progress notification to send all kinds of responses
the server send conversation/context request to collected chat context, if not handled, server will assume no filed is opened

simply delete *Copilot Chat* buffer to start a new conversation

@knilink knilink mentioned this pull request Sep 25, 2024
@knilink knilink marked this pull request as draft September 25, 2024 09:26
@bbatsov
Copy link
Collaborator

bbatsov commented Apr 3, 2025

Let's revisit this PR after #339 gets merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants