Skip to content

Commit 1ea1331

Browse files
committed
Remove unneeded param
1 parent 45a3b5b commit 1ea1331

File tree

5 files changed

+5
-7
lines changed

5 files changed

+5
-7
lines changed

src/vs/workbench/contrib/chat/browser/chatContentParts/chatMcpServersInteractionContentPart.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,6 @@ export class ChatMcpServersInteractionContentPart extends Disposable implements
142142
true, // forceShowSpinner
143143
true, // forceShowMessage
144144
undefined, // icon
145-
undefined, // confirmedReason
146145
undefined, // toolInvocation
147146
));
148147
this.domNode.appendChild(this.workingProgressPart.domNode);

src/vs/workbench/contrib/chat/browser/chatContentParts/chatProgressContentPart.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ import { IMarkdownRenderer } from '../../../../../platform/markdown/browser/mark
1313
import { IRenderedMarkdown } from '../../../../../base/browser/markdownRenderer.js';
1414
import { IInstantiationService } from '../../../../../platform/instantiation/common/instantiation.js';
1515
import { localize } from '../../../../../nls.js';
16-
import { IChatProgressMessage, IChatTask, IChatTaskSerialized, IChatToolInvocation, IChatToolInvocationSerialized, ToolConfirmKind, type ConfirmedReason } from '../../common/chatService.js';
16+
import { IChatProgressMessage, IChatTask, IChatTaskSerialized, IChatToolInvocation, IChatToolInvocationSerialized, ToolConfirmKind } from '../../common/chatService.js';
1717
import { IChatRendererContent, isResponseVM } from '../../common/chatViewModel.js';
1818
import { ChatTreeItem } from '../chat.js';
1919
import { renderFileWidgets } from '../chatInlineAnchorWidget.js';
@@ -37,7 +37,6 @@ export class ChatProgressContentPart extends Disposable implements IChatContentP
3737
forceShowSpinner: boolean | undefined,
3838
forceShowMessage: boolean | undefined,
3939
icon: ThemeIcon | undefined,
40-
confirmedReason: boolean | ConfirmedReason | undefined,
4140
private readonly toolInvocation: IChatToolInvocation | IChatToolInvocationSerialized | undefined,
4241
@IInstantiationService private readonly instantiationService: IInstantiationService,
4342
@IChatMarkdownAnchorService private readonly chatMarkdownAnchorService: IChatMarkdownAnchorService,
@@ -190,7 +189,7 @@ export class ChatWorkingProgressContentPart extends ChatProgressContentPart impl
190189
kind: 'progressMessage',
191190
content: new MarkdownString().appendText(localize('workingMessage', "Working..."))
192191
};
193-
super(progressMessage, chatContentMarkdownRenderer, context, undefined, undefined, undefined, undefined, undefined, instantiationService, chatMarkdownAnchorService, configurationService);
192+
super(progressMessage, chatContentMarkdownRenderer, context, undefined, undefined, undefined, undefined, instantiationService, chatMarkdownAnchorService, configurationService);
194193
}
195194

196195
override hasSameContent(other: IChatRendererContent, followingContent: IChatRendererContent[], element: ChatTreeItem): boolean {

src/vs/workbench/contrib/chat/browser/chatContentParts/chatTaskContentPart.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ export class ChatTaskContentPart extends Disposable implements IChatContentPart
4242
this.isSettled = isSettled;
4343
const showSpinner = !isSettled && !context.element.isComplete;
4444
// TODO: Pass in confirmed reason
45-
const progressPart = this._register(instantiationService.createInstance(ChatProgressContentPart, task, chatContentMarkdownRenderer, context, showSpinner, true, undefined, undefined, undefined));
45+
const progressPart = this._register(instantiationService.createInstance(ChatProgressContentPart, task, chatContentMarkdownRenderer, context, showSpinner, true, undefined, undefined));
4646
this.domNode = progressPart.domNode;
4747
this.onDidChangeHeight = Event.None;
4848
}

src/vs/workbench/contrib/chat/browser/chatContentParts/toolInvocationParts/chatToolProgressPart.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ export class ChatToolProgressSubPart extends BaseChatToolInvocationSubPart {
7777
this.provideScreenReaderStatus(content);
7878
}
7979

80-
return this.instantiationService.createInstance(ChatProgressContentPart, progressMessage, this.renderer, this.context, undefined, true, this.getIcon(), this.toolInvocation.isConfirmed, this.toolInvocation);
80+
return this.instantiationService.createInstance(ChatProgressContentPart, progressMessage, this.renderer, this.context, undefined, true, this.getIcon(), this.toolInvocation);
8181
}
8282

8383
private getAnnouncementKey(kind: 'progress' | 'complete'): string {

src/vs/workbench/contrib/chat/browser/chatListRenderer.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1265,7 +1265,7 @@ export class ChatListItemRenderer extends Disposable implements ITreeRenderer<Ch
12651265
} else if (content.kind === 'multiDiffData') {
12661266
return this.renderMultiDiffData(content, templateData, context);
12671267
} else if (content.kind === 'progressMessage') {
1268-
return this.instantiationService.createInstance(ChatProgressContentPart, content, this.chatContentMarkdownRenderer, context, undefined, undefined, undefined, undefined, undefined);
1268+
return this.instantiationService.createInstance(ChatProgressContentPart, content, this.chatContentMarkdownRenderer, context, undefined, undefined, undefined, undefined);
12691269
} else if (content.kind === 'working') {
12701270
return this.instantiationService.createInstance(ChatWorkingProgressContentPart, content, this.chatContentMarkdownRenderer, context);
12711271
} else if (content.kind === 'progressTask' || content.kind === 'progressTaskSerialized') {

0 commit comments

Comments
 (0)