Skip to content

Small session tweaks#300576

Merged
benibenj merged 1 commit intomainfrom
benibenj/chief-aardwolf
Mar 10, 2026
Merged

Small session tweaks#300576
benibenj merged 1 commit intomainfrom
benibenj/chief-aardwolf

Conversation

@benibenj
Copy link
Contributor

Copilot Generated Description: Introduce a mechanism to suppress selection changes temporarily and refactor the feedback submission process to improve focus handling in the editor. Additionally, update the session code review action to exclude certain agent session types.

Copilot AI review requested due to automatic review settings March 10, 2026 22:33
@benibenj benibenj enabled auto-merge March 10, 2026 22:33
@benibenj benibenj self-assigned this Mar 10, 2026
@vs-code-engineering vs-code-engineering bot added this to the 1.112.0 milestone Mar 10, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR makes small UX/behavior tweaks in the Sessions window: it adjusts when the “Run Code Review” action is shown and improves focus/selection-change handling when submitting agent feedback from the editor overlay widget.

Changes:

  • Hide the Sessions “Run Code Review” toolbar action for Cloud agent sessions.
  • Add a one-time selection-change suppression mechanism to prevent the feedback widget from re-opening during hide/refocus.
  • Refactor feedback submission flows to share a single hide+refocus helper.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/vs/sessions/contrib/codeReview/browser/codeReview.contributions.ts Adds a context key constraint so the code review action is not offered for Cloud agent sessions.
src/vs/sessions/contrib/agentFeedback/browser/agentFeedbackEditorInputContribution.ts Introduces one-shot selection-change suppression and centralizes hide+refocus logic after feedback actions.

You can also share your feedback on Copilot code review. Take the survey.

@benibenj benibenj merged commit 03b38ff into main Mar 10, 2026
23 checks passed
@benibenj benibenj deleted the benibenj/chief-aardwolf branch March 10, 2026 23:09
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.

3 participants