Fire end in microtask

0YJ · Apr 16, 2024 · 28ae1e0 · 28ae1e0
1 parent 762cbd6
commit 28ae1e0
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/vs/workbench/api/browser/mainThreadTerminalShellIntegration.ts b/src/vs/workbench/api/browser/mainThreadTerminalShellIntegration.ts
@@ -69,7 +69,10 @@ export class MainThreadTerminalShellIntegration extends Disposable implements Ma
 			currentCommand = undefined;
 			const instanceId = e.instance.instanceId;
 			instanceDataListeners.get(instanceId)?.dispose();
-			this._proxy.$shellExecutionEnd(instanceId, e.data.command, convertToExtHostCommandLineConfidence(e.data), e.data.isTrusted, e.data.exitCode);
+			// Send end in a microtask to ensure the data events are sent first
+			queueMicrotask(() => {
+				this._proxy.$shellExecutionEnd(instanceId, e.data.command, convertToExtHostCommandLineConfidence(e.data), e.data.isTrusted, e.data.exitCode);
+			});
 		}));
 
 		// onDidChangeTerminalShellIntegration via cwd