Skip to content

Conversation

meganrogge
Copy link
Contributor

fix #248160

Screenshot 2025-05-05 at 3 10 58 PM Screenshot 2025-05-05 at 3 11 11 PM

@meganrogge meganrogge self-assigned this May 5, 2025
@meganrogge meganrogge requested review from connor4312 and roblourens and removed request for connor4312 May 5, 2025 19:21
@meganrogge meganrogge added this to the May 2025 milestone May 5, 2025
@connor4312
Copy link
Member

On the 2nd screenshot, before the tool calls maybe we should have a section header for Tool calls used in this response: so the user can easily tell what's part of the response vs metadata used to generate it.

@meganrogge meganrogge enabled auto-merge (squash) May 6, 2025 20:59
@meganrogge meganrogge closed this May 8, 2025
auto-merge was automatically disabled May 8, 2025 19:22

Pull request was closed

@meganrogge meganrogge reopened this May 8, 2025
@meganrogge meganrogge enabled auto-merge (squash) May 8, 2025 19:23
@meganrogge meganrogge merged commit b801005 into main May 8, 2025
8 checks passed
@meganrogge meganrogge deleted the merogge/accessible-view-updates branch May 8, 2025 21:47
@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators Jun 22, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

provide richer info about tool calls in accessible view
3 participants