This repository was archived by the owner on Aug 1, 2025. It is now read-only.
feat: add default prompt mixin to web#1466
Closed
abeatrix wants to merge 1 commit into
Closed
Conversation
Contributor
|
This is meant for VS Code Web or Cody Web Client in SG? |
Contributor
Author
@thenamankumar woops thanks for letting me know. I just found the set up guide for Cody web, will take a look tomorrow. Thanks again! |
abeatrix
referenced
this pull request
Oct 24, 2023
Moved from https://github.com/sourcegraph/cody/pull/1466 refactor: add default prompt mixin to transcripts by default - Remove initialization of default prompt mixin by clients. - Update Transcript to add the default prompt mixin to all interactions on creation. - The PromptMixin class allows mixins to be defined that prepend text to prompts. - Transcript mixes in all registered PromptMixin instances into the human prompt text before sending to the assistant. - This centralizes default prompt handling in Transcript. Clients no longer need to add the default prompt mixin themselves. - This also adds the default prompt mixin to web and other clients that previously have not initialized the PromptMixin at activation. ## Test plan <!-- Required. See https://docs.sourcegraph.com/dev/background-information/testing_principles. --> You should see `(Reply as Cody, a coding assistant developed by Sourcegraph...)` prepended to all users' questions:  --------- Co-authored-by: Rafał Gajdulewicz <rafax@users.noreply.github.com>
burmudar
pushed a commit
that referenced
this pull request
Nov 21, 2024
…ch don't support inlay models. (#1490) ImaginaryEditor does not support InlayModel and throws an UnsupportedOperationException; IntentionPreviewEditor throws an IntentionPreviewUnsupportedOperationException: UnsupportedOperationException, etc. Fixes #1488, #1486, #1485, #1480, #1466, #1462, #1435, #1363 ## Test plan ``` ./gradlew test --tests InlayModelUtilTest ``` Ran the IDE and clicked between editors, requesting autocompletes.
This file contains hidden or 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
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
RE: https://sourcegraph.slack.com/archives/C052G9Y5Y8H/p1697711213702149
The defaultPromptMixin is added and PromptMixin.add is called in activate event for web client.
Test plan
See if
defaultPromptMixinwas added to every message submitted by users on web.