From 12373f48a1dd4a638d82d4ea935d1a87094e052d Mon Sep 17 00:00:00 2001 From: Justin Chen <54879025+justschen@users.noreply.github.com> Date: Tue, 17 Mar 2026 21:06:29 -0700 Subject: [PATCH] revert disposable handling of newly created tools. (#302646) fix broken terminals in uncollapsed thinking modes --- .../contrib/chat/browser/widget/chatListRenderer.ts | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/vs/workbench/contrib/chat/browser/widget/chatListRenderer.ts b/src/vs/workbench/contrib/chat/browser/widget/chatListRenderer.ts index 6e6c928e69c..be0d5847e3c 100644 --- a/src/vs/workbench/contrib/chat/browser/widget/chatListRenderer.ts +++ b/src/vs/workbench/contrib/chat/browser/widget/chatListRenderer.ts @@ -1969,10 +1969,10 @@ export class ChatListItemRenderer extends Disposable implements ITreeRenderer { + const createToolPart = (): { domNode: HTMLElement; part: ChatToolInvocationPart } => { lazilyCreatedPart = this.instantiationService.createInstance(ChatToolInvocationPart, toolInvocation, context, this.chatContentMarkdownRenderer, this._contentReferencesListPool, this._toolEditorPool, () => this._currentLayoutWidth.get(), this._toolInvocationCodeBlockCollection, this._announcedToolProgressKeys, codeBlockStartIndex); this.handleRenderedCodeblocks(context.element, lazilyCreatedPart, codeBlockStartIndex); - return { domNode: lazilyCreatedPart.domNode, disposable: lazilyCreatedPart, part: lazilyCreatedPart }; + return { domNode: lazilyCreatedPart.domNode, part: lazilyCreatedPart }; }; // handling for when we want to put tool invocations inside a thinking part @@ -2028,7 +2028,7 @@ export class ChatListItemRenderer extends Disposable implements ITreeRenderer ChatToolInvocationPart | undefined, - createToolPart: () => { domNode: HTMLElement; disposable: ChatToolInvocationPart; part: ChatToolInvocationPart }, + createToolPart: () => { domNode: HTMLElement; part: ChatToolInvocationPart }, context: IChatContentPartRenderContext, templateData: IChatListItemTemplate ): void {