From 39da37592d32bd16e2a5546314dcba30a3466255 Mon Sep 17 00:00:00 2001
From: Kartik Raj <karraj@microsoft.com>
Date: Thu, 20 Apr 2023 20:49:53 +0000
Subject: [PATCH] Attempt to fix smoke tests

This reverts commit be89fd23d09ab9dbeb961c63a1208f4259f7d9b4.
---
 src/vs/server/node/remoteTerminalChannel.ts                  | 4 +---
 .../contrib/terminal/common/remoteTerminalChannel.ts         | 5 +----
 .../configurationResolver/common/terminalResolver.ts         | 4 ++--
 3 files changed, 4 insertions(+), 9 deletions(-)

diff --git a/src/vs/server/node/remoteTerminalChannel.ts b/src/vs/server/node/remoteTerminalChannel.ts
index 209a33f6a6f4d..fddf20d605dfe 100644
--- a/src/vs/server/node/remoteTerminalChannel.ts
+++ b/src/vs/server/node/remoteTerminalChannel.ts
@@ -32,7 +32,6 @@ import { IProductService } from 'vs/platform/product/common/productService';
 import { IExtensionManagementService } from 'vs/platform/extensionManagement/common/extensionManagement';
 import { IConfigurationService } from 'vs/platform/configuration/common/configuration';
 import { withNullAsUndefined } from 'vs/base/common/types';
-import { getWorkspaceForTerminal } from 'vs/workbench/services/configurationResolver/common/terminalResolver';
 
 class CustomVariableResolver extends AbstractVariableResolverService {
 	constructor(
@@ -239,9 +238,8 @@ export class RemoteTerminalChannel extends Disposable implements IServerChannel<
 			}
 			const envVariableCollections = new Map<string, IEnvironmentVariableCollection>(entries);
 			const mergedCollection = new MergedEnvironmentVariableCollection(envVariableCollections);
-			const cwdWorkspaceFolder = getWorkspaceForTerminal(shellLaunchConfig.cwd, args.workspaceContextService, undefined);
 			const workspaceFolder = activeWorkspaceFolder ? withNullAsUndefined(activeWorkspaceFolder) : undefined;
-			await mergedCollection.applyToProcessEnvironment(env, { workspaceFolder: cwdWorkspaceFolder ?? workspaceFolder }, variableResolver);
+			await mergedCollection.applyToProcessEnvironment(env, { workspaceFolder }, variableResolver);
 		}
 
 		// Fork the process and listen for messages
diff --git a/src/vs/workbench/contrib/terminal/common/remoteTerminalChannel.ts b/src/vs/workbench/contrib/terminal/common/remoteTerminalChannel.ts
index 24fc081a00a02..f725cd971b95e 100644
--- a/src/vs/workbench/contrib/terminal/common/remoteTerminalChannel.ts
+++ b/src/vs/workbench/contrib/terminal/common/remoteTerminalChannel.ts
@@ -51,8 +51,6 @@ export interface ICreateTerminalProcessArguments {
 	rows: number;
 	unicodeVersion: '6' | '11';
 	resolverEnv: { [key: string]: string | null } | undefined;
-	// TODO: Remove this when last active workspace is fixed
-	workspaceContextService: IWorkspaceContextService;
 }
 
 export interface ICreateTerminalProcessResult {
@@ -184,8 +182,7 @@ export class RemoteTerminalChannelClient implements IPtyHostController {
 			cols,
 			rows,
 			unicodeVersion,
-			resolverEnv,
-			workspaceContextService: this._workspaceContextService,
+			resolverEnv
 		};
 		return await this._channel.call<ICreateTerminalProcessResult>('$createProcess', args);
 	}
diff --git a/src/vs/workbench/services/configurationResolver/common/terminalResolver.ts b/src/vs/workbench/services/configurationResolver/common/terminalResolver.ts
index cf19d4d31f870..f6f118027900a 100644
--- a/src/vs/workbench/services/configurationResolver/common/terminalResolver.ts
+++ b/src/vs/workbench/services/configurationResolver/common/terminalResolver.ts
@@ -8,13 +8,13 @@ import { URI } from 'vs/base/common/uri';
 import { IWorkspaceContextService, IWorkspaceFolder } from 'vs/platform/workspace/common/workspace';
 import { IHistoryService } from 'vs/workbench/services/history/common/history';
 
-export function getWorkspaceForTerminal(cwd: URI | string | undefined, workspaceContextService: IWorkspaceContextService, historyService: IHistoryService | undefined): IWorkspaceFolder | undefined {
+export function getWorkspaceForTerminal(cwd: URI | string | undefined, workspaceContextService: IWorkspaceContextService, historyService: IHistoryService): IWorkspaceFolder | undefined {
 	const cwdUri = typeof cwd === 'string' ? URI.parse(cwd) : cwd;
 	let workspaceFolder = cwdUri ? withNullAsUndefined(workspaceContextService.getWorkspaceFolder(cwdUri)) : undefined;
 	if (!workspaceFolder) {
 		// fallback to last active workspace if cwd is not available or it is not in workspace
 		// TOOD: last active workspace is known to be unreliable, we should remove this fallback eventually
-		const activeWorkspaceRootUri = historyService?.getLastActiveWorkspaceRoot();
+		const activeWorkspaceRootUri = historyService.getLastActiveWorkspaceRoot();
 		workspaceFolder = activeWorkspaceRootUri ? withNullAsUndefined(workspaceContextService.getWorkspaceFolder(activeWorkspaceRootUri)) : undefined;
 	}
 	return workspaceFolder;