-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Adds AI feedback support for changelog markdown editors #4479
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
Conversation
Hi @eamodio This PR is the implementation for Changelogs in a similar manner how it's been done for markdown previews. Please let me know if you think that something needs to be improved here. Also it has a follow-up where I suggest to wrap static values to a disposable provider. |
4437459
to
8b7c748
Compare
Enables the use of helpful/unhelpful AI feedback buttons in untitled markdown editors generated by the changelog command. Tracks feedback context for these documents, updates relevant context keys, and ensures telemetry compatibility. Also improves feedback instructions in generated changelogs to encourage user interaction. Updates telemetry and menu configuration to support this new AI feedback capability for changelog editors. (#4449, #4479)
8b7c748
to
5233993
Compare
Enables the use of helpful/unhelpful AI feedback buttons in untitled markdown editors generated by the changelog command. Tracks feedback context for these documents, updates relevant context keys, and ensures telemetry compatibility. Also improves feedback instructions in generated changelogs to encourage user interaction. Updates telemetry and menu configuration to support this new AI feedback capability for changelog editors. (#4449, #4479)
5233993
to
28c7ed5
Compare
Enables the use of helpful/unhelpful AI feedback buttons in untitled markdown editors generated by the changelog command. Tracks feedback context for these documents, updates relevant context keys, and ensures telemetry compatibility. Also improves feedback instructions in generated changelogs to encourage user interaction. Updates telemetry and menu configuration to support this new AI feedback capability for changelog editors. (#4449, #4479)
28c7ed5
to
60b3f8f
Compare
Enables the use of helpful/unhelpful AI feedback buttons in untitled markdown editors generated by the changelog command. Tracks feedback context for these documents, updates relevant context keys, and ensures telemetry compatibility. Also improves feedback instructions in generated changelogs to encourage user interaction. Updates telemetry and menu configuration to support this new AI feedback capability for changelog editors. (#4449, #4479)
60b3f8f
to
91f9b4a
Compare
eee474b
to
91f9b4a
Compare
Enables the use of helpful/unhelpful AI feedback buttons in untitled markdown editors generated by the changelog command. Tracks feedback context for these documents, updates relevant context keys, and ensures telemetry compatibility. Also improves feedback instructions in generated changelogs to encourage user interaction. Updates telemetry and menu configuration to support this new AI feedback capability for changelog editors. (#4449, #4479)
91f9b4a
to
13055df
Compare
@@ -98,12 +132,22 @@ export async function generateChangelogAndOpenMarkdownDocument( | |||
if (result === 'cancelled') return; | |||
|
|||
const { range, changes: { length: count } = [] } = await changes.value; | |||
const feedbackContext = result && getAIResultContext(result); |
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.
Is this always guaranteed to hit the case uri?.scheme === Schemes.GitLensAIMarkdown
? Otherwise on a quick read it looks like it would result in a getChangelogFeedbackContext
which wouldn't exist yet (I must be missing something).
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.
@axosoft-ramint
It is rather guaranteed that changelog's scheme is not GitLensAIMarkdown
. It's "untitled"
.
getAIResultContext
is an util function creates an object assigned tofeedbackContext
.const document = await workspace.openTextDocument
creates a document withuntitled
scheme.- then
setChangelogFeedbackContext
stores the context.
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
Enables the use of helpful/unhelpful AI feedback buttons in untitled markdown editors generated by the changelog command. Tracks feedback context for these documents, updates relevant context keys, and ensures telemetry compatibility. Also improves feedback instructions in generated changelogs to encourage user interaction. Updates telemetry and menu configuration to support this new AI feedback capability for changelog editors. (#4449, #4479)
13055df
to
5fd6a0f
Compare
Enables the use of helpful/unhelpful AI feedback buttons in untitled markdown editors generated by the changelog command. Tracks feedback context for these documents, updates relevant context keys, and ensures telemetry compatibility. Also improves feedback instructions in generated changelogs to encourage user interaction. Updates telemetry and menu configuration to support this new AI feedback capability for changelog editors. (#4449, #4479)
5fd6a0f
to
621283b
Compare
Enables the use of helpful/unhelpful AI feedback buttons in untitled markdown editors generated by the changelog command. Tracks feedback context for these documents, updates relevant context keys, and ensures telemetry compatibility. Also improves feedback instructions in generated changelogs to encourage user interaction. Updates telemetry and menu configuration to support this new AI feedback capability for changelog editors. (#4449, #4479)
Description
This is a follow-up of #4449
Enables the use of helpful/unhelpful AI feedback buttons in untitled markdown editors generated by the changelog command.
Tracks feedback context for these documents, updates relevant context keys.
Checklist
Fixes $XXX -
orCloses #XXX -
prefix to auto-close the issue that your PR addresses