Skip to content

chat setup - improve detection for panel welcome on context keys that are relevant#293603

Merged
bpasero merged 2 commits intomainfrom
ben/tense-penguin
Feb 7, 2026
Merged

chat setup - improve detection for panel welcome on context keys that are relevant#293603
bpasero merged 2 commits intomainfrom
ben/tense-penguin

Conversation

@bpasero
Copy link
Member

@bpasero bpasero commented Feb 7, 2026

No description provided.

Copilot AI review requested due to automatic review settings February 7, 2026 08:33
@bpasero bpasero enabled auto-merge (squash) February 7, 2026 08:33
@bpasero bpasero self-assigned this Feb 7, 2026
@vs-code-engineering vs-code-engineering bot added this to the February 2026 milestone Feb 7, 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

Improves the chat setup flow’s detection logic for whether the panel can show a “welcome / guidance” view by only reacting to context key changes that are relevant to the registered welcome descriptors.

Changes:

  • Tracks the set of context keys referenced by registered chatViewsWelcomeRegistry descriptors.
  • Filters onDidChangeContext handling to only fire when one of those descriptor keys changes.

@bpasero bpasero marked this pull request as draft February 7, 2026 09:19
auto-merge was automatically disabled February 7, 2026 09:19

Pull request was converted to draft

Copy link
Contributor

Copilot AI commented Feb 7, 2026

@bpasero I've opened a new pull request, #293607, to work on those changes. Once the pull request is ready, I'll request review from you.

…ry registry reads (#293607)

* Initial plan

* Optimize context key detection to only update descriptorKeys on registry changes

Co-authored-by: bpasero <900690+bpasero@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: bpasero <900690+bpasero@users.noreply.github.com>
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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@bpasero bpasero marked this pull request as ready for review February 7, 2026 16:57
@bpasero bpasero enabled auto-merge (squash) February 7, 2026 16:57
@bpasero bpasero merged commit 3770523 into main Feb 7, 2026
23 of 24 checks passed
@bpasero bpasero deleted the ben/tense-penguin branch February 7, 2026 17:10
daviddossett pushed a commit to daviddossett/vscode that referenced this pull request Feb 7, 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.

3 participants