-
Notifications
You must be signed in to change notification settings - Fork 370
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
feat(smart-apply): Adds custom model prompt for smart apply. #7328
Conversation
@@ -15,27 +11,3 @@ export interface ModelParametersInput { | |||
export interface ModelParameterProvider { | |||
getModelParameters(args: ModelParametersInput): CompletionParameters | |||
} | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Moved this to vscode/src/edit/adapters/default.ts
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for splitting things out, LGTM.
Is there any way we can test this without running a local dev instance?
${getGuidelinesPrompt()}` | ||
} | ||
|
||
function getUserPrompt( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be useful to add some simple snapshot tests for this file, just to view the raw prompt output and check that it looks correct and isn't broken by someone forgetting to add a </
in future
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point, let me add this in the follow up PR.
Currently the backend PR is not merged, we should be able to test this when the SG get deloyed tomorrow |
e68c366
to
8f33ffb
Compare
Test plan