Add copilot queries to query history #510
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…texts. This includes getting rid of RpcRequestContext/RpcNotificationContext, which just uses base class functionality. Web*Context overrides the base class behavior because a particular session is needed and the server implementation doesn't have that detail.
Also add type annotations various places to trigger editor type error help. Formatting changes.
lossyrob
force-pushed
the
feature/chat-query-history
branch
from
24b2cb1 to
107446a
Compare
mmcfarland
reviewed
Feb 14, 2025
lossyrob
force-pushed
the
feature/chat-query-history
branch
from
107446a to
fca7712
Compare
lossyrob
force-pushed
the
feature/chat-query-history
branch
from
fca7712 to
22eeae9
Compare
lossyrob
force-pushed
the
feature/chat-query-history
branch
2 times, most recently
from
712e4f7 to
826dcdb
Compare
lossyrob
force-pushed
the
feature/chat-query-history
branch
from
826dcdb to
3525c27
Compare
This was applicable when pytest_asyncio was used; we are using the AsyncRunner to execute async code and so do not need pytest_asyncio
lossyrob
force-pushed
the
feature/chat-query-history
branch
from
c24a14c to
f619fd6
Compare
lossyrob
force-pushed
the
feature/chat-query-history
branch
from
f619fd6 to
47ded48
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds a feature where notifications are sent to VSCode each time a query is executed by the copilot.
It also adds a feature where the MessageServer is able to handle request responses and errors. This was implemented as I originally implemented the query history messages as requests, but move to notifications.
Also, a PR GitHub action was added to the project, along with a docker-compose that brings up a local postgres instance for integration tests that can be used locally and is used in the PR tests.
It also refactored a few things: