by default, play tool user action required when window is not focused (microsoft#249379)

fix issue

Author: meganrogge

src/vs/base/browser/dom.ts

@@ -944,7 +944,11 @@ export function getActiveWindow(): CodeWindow {
 
 export function getFocusedWindow(): CodeWindow | undefined {
 	const document = getActiveDocument();
-	return (document.defaultView?.window) as CodeWindow | undefined;
+	// This check is needed to ensure the window has focus
+	if (document.defaultView?.window.document.hasFocus()) {
+		return (document.defaultView?.window) as CodeWindow;
+	}
+	return;
 }
 
 interface IMutationObserver {

src/vs/workbench/contrib/accessibility/browser/accessibilityConfiguration.ts

@@ -753,15 +753,15 @@ const configuration: IConfigurationNode = {
 		},
 		'accessibility.signals.chatUserActionRequired': {
 			...signalFeatureBase,
-			'markdownDescription': localize('accessibility.signals.chatUserActionRequired', "Plays a signal - sound (audio cue) and/or announcement (alert) - when user action is required in the chat."),
+			'markdownDescription': localize('accessibility.signals.chatUserActionRequired', "Plays a signal - sound (audio cue) and/or announcement (alert) - when user action is required in the chat. Note that {0} will impact the `auto` behavior of this setting.", '`#chat.focusWindowOnConfirmation#`'),
 			'properties': {
 				'sound': {
 					'description': localize('accessibility.signals.chatUserActionRequired.sound', "Plays a sound when user action is required in the chat."),
 					'type': 'string',
 					'enum': ['auto', 'on', 'off'],
 					'default': 'auto',
 					'enumDescriptions': [
-						localize('sound.enabled.autoWindow', "Enable sound when a screen reader is attached or when the current window is focused."),
+						localize('sound.enabled.autoWindow', "Enable sound when a screen reader is attached or when the current window is not focused."),
 						localize('sound.enabled.on', "Enable sound."),
 						localize('sound.enabled.off', "Disable sound.")
 					],

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

@@ -367,7 +367,7 @@ export class LanguageModelToolsService extends Disposable implements ILanguageMo
 	private playAccessibilitySignal(toolInvocations: ChatToolInvocation[]): void {
 		const hasFocusedWindow = getFocusedWindow();
 		const setting: { sound: 'auto' | 'on' | 'off' } = this._configurationService.getValue(AccessibilitySignal.chatUserActionRequired.settingsKey);
-		if (setting.sound === 'on' || (setting.sound === 'auto' && (this._accessibilityService.isScreenReaderOptimized() || hasFocusedWindow))) {
+		if (setting.sound === 'on' || (setting.sound === 'auto' && (this._accessibilityService.isScreenReaderOptimized() || !hasFocusedWindow))) {
 			this._accessibilitySignalService.playSignal(AccessibilitySignal.chatUserActionRequired, { customAlertMessage: this._instantiationService.invokeFunction(getToolConfirmationAlert, toolInvocations), userGesture: true });
 		}
 	}