fix interpreters manager active session (#2785)

posit-dev · Apr 17, 2024 · e790eef · e790eef
1 parent 342264f
commit e790eef
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/...rkbench/browser/parts/positronTopActionBar/components/topActionBarInterpretersManager.tsx b/...rkbench/browser/parts/positronTopActionBar/components/topActionBarInterpretersManager.tsx
@@ -69,10 +69,10 @@ export const TopActionBarInterpretersManager = (props: TopActionBarInterpretersM
 		// Create the disposable store for cleanup.
 		const disposableStore = new DisposableStore();
 
-		// Add the onDidStartRuntime event handler.
+		// Add the onDidChangeForegroundSession event handler.
 		disposableStore.add(
-			context.runtimeSessionService.onDidStartRuntime(session => {
-				if (session.metadata.sessionMode === LanguageRuntimeSessionMode.Console) {
+			context.runtimeSessionService.onDidChangeForegroundSession(session => {
+				if (session?.metadata.sessionMode === LanguageRuntimeSessionMode.Console) {
 					setActiveSession(
 						context.runtimeSessionService.foregroundSession);
 				}

diff --git a/src/vs/workbench/services/runtimeSession/common/runtimeSession.ts b/src/vs/workbench/services/runtimeSession/common/runtimeSession.ts
@@ -436,7 +436,7 @@ export class RuntimeSessionService extends Disposable implements IRuntimeSession
 
 		this._foregroundSession = session;
 
-		// Fire the onDidChangeActiveRuntime event.
+		// Fire the onDidChangeForegroundSession event.
 		this._onDidChangeForegroundSessionEmitter.fire(this._foregroundSession);
 	}