Add comment about concurrent activateConsoleSession()

lionel- · lionel- · commit d87869a79363 · 2025-12-05T09:57:42.000+01:00
diff --git a/extensions/positron-r/src/session-manager.ts b/extensions/positron-r/src/session-manager.ts
@@ -99,6 +99,9 @@ export class RSessionManager implements vscode.Disposable {
 			throw Error(`Foreground session with ID ${sessionId} must not be a background session.`);
 		}
 
+		// Multiple `activateConsoleSession()` might run concurrently if the
+		// `didChangeForegroundSession` event fires rapidly. We might want to queue
+		// the handling.
 		this._lastForegroundSessionId = session.metadata.sessionId;
 		await this.activateConsoleSession(session, 'foreground session changed');
 	}

Original file line number	Diff line number	Diff line change
`@@ -99,6 +99,9 @@ export class RSessionManager implements vscode.Disposable {`
`99`	`99`	throw Error(`Foreground session with ID ${sessionId} must not be a background session.`);
`100`	`100`	`}`
`101`	`101`
	`102`	+ // Multiple `activateConsoleSession()` might run concurrently if the
	`103`	+ // `didChangeForegroundSession` event fires rapidly. We might want to queue
	`104`	`+ // the handling.`
`102`	`105`	`this._lastForegroundSessionId = session.metadata.sessionId;`
`103`	`106`	`await this.activateConsoleSession(session, 'foreground session changed');`
`104`	`107`	`}`