handle alerts in audio cue service, add List Alerts command and settings

src/vs/platform/audioCues/browser/audioCueService.ts

@@ -30,6 +30,11 @@ export interface IAudioCueService {
 export interface IAudioCueOptions {
 	allowManyInParallel?: boolean;
 	source?: string;
+	/**
+	 * For actions like save or format, depending on the
+	 * configured value, we will only
+	 * play the sound if the user triggered the action.
+	 */
 	userGesture?: boolean;
 }
 
@@ -45,17 +50,18 @@ export class AudioCueService extends Disposable implements IAudioCueService {
 	constructor(
 		@IConfigurationService private readonly configurationService: IConfigurationService,
 		@IAccessibilityService private readonly accessibilityService: IAccessibilityService,
-		@ITelemetryService private readonly telemetryService: ITelemetryService) {
+		@ITelemetryService private readonly telemetryService: ITelemetryService,
+	) {
 		super();
 	}
 
 	public async playAudioCue(cue: AudioCue, options: IAudioCueOptions = {}): Promise<void> {
 		const alertMessage = cue.alertMessage;
-		if (this.isAlertEnabled(cue) && alertMessage) {
+		if (this.isAlertEnabled(cue, options.userGesture) && alertMessage) {
 			this.accessibilityService.alert(alertMessage);
 		}
 
-		if (this.isCueEnabled(cue)) {
+		if (this.isCueEnabled(cue, options.userGesture)) {
 			this.sendAudioCueTelemetry(cue, options.source);
 			await this.playSound(cue.sound.getSound(), options.allowManyInParallel);
 		}
@@ -218,12 +224,12 @@ export class AudioCueService extends Disposable implements IAudioCueService {
 		});
 	});
 
-	public isAlertEnabled(cue: AudioCue): boolean {
-		return this.isAlertEnabledCache.get(cue).get() ?? false;
+	public isAlertEnabled(cue: AudioCue, userGesture?: boolean): boolean {
+		return this.isAlertEnabledCache.get(cue, userGesture).get() ?? false;
 	}
 
-	public isCueEnabled(cue: AudioCue): boolean {
-		return this.isCueEnabledCache.get(cue).get() ?? false;
+	public isCueEnabled(cue: AudioCue, userGesture?: boolean): boolean {
+		return this.isCueEnabledCache.get(cue, userGesture).get() ?? false;
 	}
 
 	public onEnabledChanged(cue: AudioCue): Event<void> {
@@ -254,17 +260,17 @@ function playAudio(url: string, volume: number): Promise<HTMLAudioElement> {
 }
 
 class Cache<TArg, TValue> {
-	private readonly map = new Map<TArg, TValue>();
-	constructor(private readonly getValue: (value: TArg) => TValue) {
+	private readonly map = new Map<{ arg: TArg; optionalArg: any }, TValue>();
+	constructor(private readonly getValue: (value: TArg, optionalArg?: any) => TValue) {
 	}
 
-	public get(arg: TArg): TValue {
-		if (this.map.has(arg)) {
-			return this.map.get(arg)!;
+	public get(arg: TArg, optionalArg?: any): TValue {
+		if (this.map.has({ arg, optionalArg })) {
+			return this.map.get({ arg, optionalArg })!;
 		}
 
-		const value = this.getValue(arg);
-		this.map.set(arg, value);
+		const value = this.getValue(arg, optionalArg);
+		this.map.set({ arg, optionalArg }, value);
 		return value;
 	}
 }