Skip to content

Conversation

@timtmok
Copy link
Contributor

@timtmok timtmok commented Dec 5, 2025

Address #10364

Shiny has responses that include command buttons and they aren't handled by Assistant when it tries to review the chat history.

Release Notes

New Features

  • N/A

Bug Fixes

  • Fix error when chat history has a participant

QA Notes

@timtmok timtmok requested a review from sharon-wang December 5, 2025 21:42
@github-actions
Copy link

github-actions bot commented Dec 5, 2025

E2E Tests 🚀
This PR will run tests tagged with: @:critical

readme  valid tags

Copy link
Member

@sharon-wang sharon-wang left a comment

Choose a reason for hiding this comment

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

LGTM -- working for me!

One question re: support for other part types

Comment on lines +338 to 340
} else if (content instanceof vscode.ChatResponseCommandButtonPart) {
return acc;
} else {
Copy link
Member

Choose a reason for hiding this comment

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

do you think it's worth supporting the other part types we don't currently list here?

/**
* Represents the different chat response types.
*/
export type ChatResponsePart = ChatResponseMarkdownPart | ChatResponseFileTreePart | ChatResponseAnchorPart
| ChatResponseProgressPart | ChatResponseReferencePart | ChatResponseCommandButtonPart;

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