fix for 264315 (#269632)

* fix for 264315

* Update src/vs/workbench/contrib/chat/common/chatService.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Updating silent property for onload scenario

* implementing code comments

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Author: dileepyavan

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

@@ -40,7 +40,7 @@ export class ChatConfirmationContentPart extends Disposable implements IChatCont
 				{ label: localize('accept', "Accept"), data: confirmation.data },
 				{ label: localize('dismiss', "Dismiss"), data: confirmation.data, isSecondary: true },
 			];
-		const confirmationWidget = this._register(this.instantiationService.createInstance(SimpleChatConfirmationWidget, context.container, { title: confirmation.title, buttons, message: confirmation.message }));
+		const confirmationWidget = this._register(this.instantiationService.createInstance(SimpleChatConfirmationWidget, context.container, { title: confirmation.title, buttons, message: confirmation.message, silent: confirmation.isLive === false }));
 		confirmationWidget.setShowButtons(!confirmation.isUsed);
 
 		this._register(confirmationWidget.onDidChangeHeight(() => this._onDidChangeHeight.fire()));

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

@@ -43,6 +43,7 @@ export interface IChatConfirmationWidgetOptions<T> {
 	subtitle?: string | IMarkdownString;
 	buttons: IChatConfirmationButton<T>[];
 	toolbarData?: { arg: any; partType: string; partSource?: string };
+	silent?: boolean;
 }
 
 export class ChatQueryTitlePart extends Disposable {
@@ -129,6 +130,7 @@ abstract class BaseSimpleChatConfirmationWidget<T> extends Disposable {
 	protected readonly markdownRenderer: MarkdownRenderer;
 	private readonly title: string | IMarkdownString;
 
+	private readonly silent: boolean;
 	private readonly notification = this._register(new MutableDisposable<DisposableStore>());
 
 	constructor(
@@ -142,8 +144,9 @@ abstract class BaseSimpleChatConfirmationWidget<T> extends Disposable {
 	) {
 		super();
 
-		const { title, subtitle, message, buttons } = options;
+		const { title, subtitle, message, buttons, silent } = options;
 		this.title = title;
+		this.silent = !!silent;
 
 
 		const elements = dom.h('.chat-confirmation-widget-container@container', [
@@ -235,7 +238,7 @@ abstract class BaseSimpleChatConfirmationWidget<T> extends Disposable {
 	protected renderMessage(element: HTMLElement, listContainer: HTMLElement): void {
 		this.messageElement.append(element);
 
-		if (this.showingButtons && this._configurationService.getValue<boolean>('chat.notifyWindowOnConfirmation')) {
+		if (this.showingButtons && this._configurationService.getValue<boolean>('chat.notifyWindowOnConfirmation') && !this.silent) {
 			const targetWindow = dom.getWindow(listContainer);
 			if (!targetWindow.document.hasFocus()) {
 				this.notifyConfirmationNeeded(targetWindow);

src/vs/workbench/contrib/chat/common/chatModel.ts

@@ -1852,6 +1852,8 @@ export class ChatModel extends Disposable implements IChatModel {
 									id: item.id,
 									metadata: item.metadata
 								};
+							} else if (item.kind === 'confirmation') {
+								return { ...item, isLive: false };
 							} else {
 								// eslint-disable-next-line local/code-no-any-casts
 								return item as any; // TODO

src/vs/workbench/contrib/chat/common/chatService.ts

@@ -260,6 +260,8 @@ export interface IChatConfirmation {
 	title: string;
 	message: string | IMarkdownString;
 	data: any;
+	/** Indicates whether this came from a current chat session (true/undefined) or a restored historic session (false) */
+	isLive?: boolean;
 	buttons?: string[];
 	isUsed?: boolean;
 	kind: 'confirmation';