build task cacophony / debounce audio cues

[meganrogge]

When running a build task (like VS Code's build task) w the new fail/success audio cues, it sounds like the failure noise starts to play and then the success one is overlaid. We should make sure to manage this better

[meganrogge]

related #163731

[meganrogge]

the audio cue is always enabled here

vscode/src/vs/workbench/contrib/audioCues/browser/audioCueService.ts

Lines 41 to 48 in 287d963

	public async playAudioCue(cue: AudioCue): Promise<void> {
	if (this.isEnabled(cue).get()) {
	await this.playSound(cue.sound);
	}
	}
	public async playAudioCues(cues: AudioCue[]): Promise<void> {

[meganrogge]

@hediet are consumers of this expected to handle repeat requests?

[hediet]

Currently, the audio cue service does not do any debouncing.

What do you suggest?

[meganrogge]

I would suggest the audio cue service drops any requests to play a sound that's already playing

[hediet]

Sounds reasonable.

[meganrogge]

I'd suggest the same thing for playSound which would fix #163731