Skip to content

feat: enhance chat session option updates#299109

Merged
DonJayamanne merged 1 commit intomainfrom
don/variable-quokka
Mar 4, 2026
Merged

feat: enhance chat session option updates#299109
DonJayamanne merged 1 commit intomainfrom
don/variable-quokka

Conversation

@DonJayamanne
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings March 4, 2026 04:14
@DonJayamanne DonJayamanne self-assigned this Mar 4, 2026
osortega
osortega previously approved these changes Mar 4, 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 refines how chat session “agent” option updates are propagated from the chat input when the current chat mode changes, aiming to avoid unnecessary extension notifications. It also adds a sample custom agent definition under .github/agents for use in repository workflows/testing.

Changes:

  • Add conditional logic to only call notifySessionOptionsChange when the “agent” session option appears to require an update.
  • Introduce .github/agents/test.agent.md as a custom agent template/example.

Reviewed changes

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

File Description
src/vs/workbench/contrib/chat/browser/widget/input/chatInputPart.ts Adds a needsUpdate check before notifying extensions about the agent session option change.
.github/agents/test.agent.md Adds a sample custom agent definition (YAML front matter + instructions).

@DonJayamanne DonJayamanne marked this pull request as ready for review March 4, 2026 05:05
@DonJayamanne DonJayamanne merged commit b16ecea into main Mar 4, 2026
20 checks passed
@DonJayamanne DonJayamanne deleted the don/variable-quokka branch March 4, 2026 05:05
@vs-code-engineering vs-code-engineering bot added this to the 1.111.0 milestone Mar 4, 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