Skip to content

Remove references to vscode-chat-issues and vscode-chat-release from chat feedback reporting#321076

Merged
dmitrivMS merged 19 commits into
mainfrom
copilot/remove-chat-feedback-command
Jun 13, 2026
Merged

Remove references to vscode-chat-issues and vscode-chat-release from chat feedback reporting#321076
dmitrivMS merged 19 commits into
mainfrom
copilot/remove-chat-feedback-command

Conversation

Copilot AI commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Description

Use the vscode repo/target when reporting chat-related issues and remove legacy chat issue/release destinations.

Also addresses follow-up feedback in openIssueReporter: routing to the internal backlog is now explicit opt-in (public === false) instead of the previous falsy check, so unspecified calls no longer default to internal backlog.

Testing

  • Ran npm run typecheck in extensions/copilot
  • Ran targeted unit test: npm run test:unit -- src/extension/conversation/vscode-node/test/newWorkspaceFollowup.spec.ts

Copilot AI requested review from Copilot and removed request for Copilot June 12, 2026 03:24
Co-authored-by: dmitrivMS <9581278+dmitrivMS@users.noreply.github.com>
Copilot AI requested review from Copilot and removed request for Copilot June 12, 2026 03:31
Copilot AI changed the title [WIP] Remove 'Chat: Send Chat Feedback' command due to archiving Route Chat feedback command to the Copilot issue reporter Jun 12, 2026
Copilot AI requested a review from dmitrivMS June 12, 2026 03:31
Comment thread extensions/copilot/src/extension/conversation/vscode-node/chatFeedbackCommand.ts Outdated
Co-authored-by: dmitrivMS <9581278+dmitrivMS@users.noreply.github.com>
Copilot AI requested review from Copilot and removed request for Copilot June 12, 2026 03:36
Copilot AI requested a review from dmitrivMS June 12, 2026 03:36
Co-authored-by: dmitrivMS <9581278+dmitrivMS@users.noreply.github.com>
Copilot AI requested review from Copilot and removed request for Copilot June 12, 2026 03:41
@dmitrivMS

Copy link
Copy Markdown
Contributor

@copilot In feedbackContribution.ts we are sending EXTENSION_ID for copilot-chat even though it's built-in now - we should probably not do so as it leads to reporting on an empty extension.

Co-authored-by: dmitrivMS <9581278+dmitrivMS@users.noreply.github.com>
Copilot AI requested review from Copilot and removed request for Copilot June 12, 2026 03:49
Co-authored-by: dmitrivMS <9581278+dmitrivMS@users.noreply.github.com>
@dmitrivMS dmitrivMS marked this pull request as ready for review June 12, 2026 22:21
@dmitrivMS dmitrivMS enabled auto-merge (squash) June 12, 2026 22:21
@dmitrivMS dmitrivMS changed the title Route Chat feedback command to the Copilot issue reporter Remove references to vscode-chat-issues and vscode-chat-release from the codebase Jun 12, 2026
@dmitrivMS dmitrivMS requested a review from Copilot June 12, 2026 22:22

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot's findings

  • Files reviewed: 5/5 changed files
  • Comments generated: 2

Comment thread extensions/copilot/README.md
alexdima
alexdima previously approved these changes Jun 12, 2026
Co-authored-by: dmitrivMS <9581278+dmitrivMS@users.noreply.github.com>
Copilot AI changed the title Remove references to vscode-chat-issues and vscode-chat-release from the codebase Remove references to vscode-chat-issues and vscode-chat-release from chat feedback reporting Jun 12, 2026
Copilot AI requested a review from dmitrivMS June 12, 2026 22:46
@dmitrivMS dmitrivMS enabled auto-merge (squash) June 12, 2026 22:49
@dmitrivMS dmitrivMS merged commit e830697 into main Jun 13, 2026
39 of 40 checks passed
@dmitrivMS dmitrivMS deleted the copilot/remove-chat-feedback-command branch June 13, 2026 00:15
@vs-code-engineering vs-code-engineering Bot added this to the 1.125.0 milestone Jun 13, 2026
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.

5 participants