Skip to content

Conversation

@justschen
Copy link
Collaborator

fix #284810

Copilot AI review requested due to automatic review settings January 2, 2026 07:55
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 fixes issue #284810 by removing the thinking dropdown UI when there's only a single tool invocation with no thinking content. Instead of displaying a collapsible thinking section containing one tool, the tool is shown directly in its original position.

Key Changes

  • Modified the logic to track and restore single tool invocations to their original DOM positions
  • Removed the currentToolCallLabel field which was previously used to set the thinking dropdown's title for single tools
  • Added restoration logic that moves the tool element out of the thinking container and hides the dropdown when appropriate

Reviewed changes

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

File Description
chatListRenderer.ts Passes templateData.value as the originalParent parameter to appendItem calls, enabling the thinking part to restore tools to their original container
chatThinkingContentPart.ts Implements single-tool restoration by tracking the tool element's original position and moving it back when the thinking dropdown would only contain one tool without thinking text

@justschen justschen marked this pull request as ready for review January 2, 2026 08:31
@justschen justschen enabled auto-merge (squash) January 2, 2026 08:31
@vs-code-engineering vs-code-engineering bot added this to the December / January 2026 milestone Jan 2, 2026
@justschen justschen merged commit 9f05c21 into microsoft:main Jan 2, 2026
17 checks passed
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.

Replacing single tool call with LLM summary

2 participants