CancellationToken#onCancellationRequested is a once-event by definiti…

…on, helps with #81574

src/vs/base/common/cancellation.ts

@@ -14,7 +14,10 @@ export interface CancellationToken {
 	readonly isCancellationRequested: boolean;
 
 	/**
-	 * An event which fires when cancellation is requested.
+	 * An event which fires when cancellation is requested. This event
+	 * only ever fires `once` as cancellation can only happen once. Listeners
+	 * that are registered after cancellation will be called (next event loop run),
+	 * but also only once.
 	 *
 	 * @event
 	 */

src/vs/base/node/zip.ts

@@ -12,7 +12,6 @@ import { mkdirp, rimraf } from 'vs/base/node/pfs';
 import { open as _openZip, Entry, ZipFile } from 'yauzl';
 import * as yazl from 'yazl';
 import { CancellationToken } from 'vs/base/common/cancellation';
-import { Event } from 'vs/base/common/event';
 
 export interface IExtractOptions {
 	overwrite?: boolean;
@@ -80,7 +79,7 @@ function extractEntry(stream: Readable, fileName: string, mode: number, targetPa
 
 	let istream: WriteStream;
 
-	Event.once(token.onCancellationRequested)(() => {
+	token.onCancellationRequested(() => {
 		if (istream) {
 			istream.destroy();
 		}
@@ -107,7 +106,7 @@ function extractZip(zipfile: ZipFile, targetPath: string, options: IOptions, tok
 	let last = createCancelablePromise<void>(() => Promise.resolve());
 	let extractedEntriesCount = 0;
 
-	Event.once(token.onCancellationRequested)(() => {
+	token.onCancellationRequested(() => {
 		last.cancel();
 		zipfile.close();
 	});

src/vs/monaco.d.ts

@@ -59,7 +59,10 @@ declare namespace monaco {
 		 */
 		readonly isCancellationRequested: boolean;
 		/**
-		 * An event which fires when cancellation is requested.
+		 * An event which fires when cancellation is requested. This event
+		 * only ever fires `once` as cancellation can only happen once. Listeners
+		 * that are registered after cancellation will be called (next event loop run),
+		 * but also only once.
 		 *
 		 * @event
 		 */

src/vs/workbench/services/remote/common/remoteAgentFileSystemChannel.ts

@@ -3,7 +3,7 @@
  *  Licensed under the MIT License. See License.txt in the project root for license information.
  *--------------------------------------------------------------------------------------------*/
 
-import { Emitter, Event } from 'vs/base/common/event';
+import { Emitter } from 'vs/base/common/event';
 import { Disposable, IDisposable, toDisposable } from 'vs/base/common/lifecycle';
 import { URI, UriComponents } from 'vs/base/common/uri';
 import { generateUuid } from 'vs/base/common/uuid';
@@ -153,7 +153,7 @@ export class RemoteFileSystemProvider extends Disposable implements
 
 		// Support cancellation
 		if (token) {
-			Event.once(token.onCancellationRequested)(() => {
+			token.onCancellationRequested(() => {
 
 				// Ensure to end the stream properly with an error
 				// to indicate the cancellation.