Revert "Reland fix custom task shell doesn't work without manually pa…

…ssing in "run command" arg/flag" (microsoft#236726) Revert "Reland fix custom task shell doesn't work without manually passing in…" This reverts commit 330ab6c.
thegecko · Dec 20, 2024 · 23aee3d · 23aee3d
1 parent 131ee0e
commit 23aee3d
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 21 deletions.
diff --git a/src/vs/platform/terminal/common/terminal.ts b/src/vs/platform/terminal/common/terminal.ts
@@ -872,7 +872,6 @@ export interface ITerminalProfile {
 	overrideName?: boolean;
 	color?: string;
 	icon?: ThemeIcon | URI | { light: URI; dark: URI };
-	isAutomationShell?: boolean;
 }
 
 export interface ITerminalDimensionsOverride extends Readonly<ITerminalDimensions> {

diff --git a/src/vs/workbench/contrib/tasks/browser/terminalTaskSystem.ts b/src/vs/workbench/contrib/tasks/browser/terminalTaskSystem.ts
@@ -1112,6 +1112,7 @@ export class TerminalTaskSystem extends Disposable implements ITaskSystem {
 				color: task.configurationProperties.icon?.color || undefined,
 				waitOnExit
 			};
+			let shellSpecified: boolean = false;
 			const shellOptions: IShellConfiguration | undefined = task.command.options && task.command.options.shell;
 			if (shellOptions) {
 				if (shellOptions.executable) {
@@ -1120,12 +1121,12 @@ export class TerminalTaskSystem extends Disposable implements ITaskSystem {
 						shellLaunchConfig.args = undefined;
 					}
 					shellLaunchConfig.executable = await this._resolveVariable(variableResolver, shellOptions.executable);
+					shellSpecified = true;
 				}
 				if (shellOptions.args) {
 					shellLaunchConfig.args = await this._resolveVariables(variableResolver, shellOptions.args.slice());
 				}
 			}
-			const taskShellArgsSpecified = (defaultProfile.isAutomationShell ? shellLaunchConfig.args : shellOptions?.args) !== undefined;
 			if (shellLaunchConfig.args === undefined) {
 				shellLaunchConfig.args = [];
 			}
@@ -1138,34 +1139,29 @@ export class TerminalTaskSystem extends Disposable implements ITaskSystem {
 				windowsShellArgs = true;
 				// If we don't have a cwd, then the terminal uses the home dir.
 				const userHome = await this._pathService.userHome();
-				if (basename === 'cmd.exe') {
-					if ((options.cwd && isUNC(options.cwd)) || (!options.cwd && isUNC(userHome.fsPath))) {
-						return undefined;
-					}
-					if (!taskShellArgsSpecified) {
-						toAdd.push('/d', '/c');
-					}
-				} else if ((basename === 'powershell.exe') || (basename === 'pwsh.exe')) {
-					if (!taskShellArgsSpecified) {
+				if (basename === 'cmd.exe' && ((options.cwd && isUNC(options.cwd)) || (!options.cwd && isUNC(userHome.fsPath)))) {
+					return undefined;
+				}
+				if ((basename === 'powershell.exe') || (basename === 'pwsh.exe')) {
+					if (!shellSpecified) {
 						toAdd.push('-Command');
 					}
 				} else if ((basename === 'bash.exe') || (basename === 'zsh.exe')) {
 					windowsShellArgs = false;
-					if (!taskShellArgsSpecified) {
+					if (!shellSpecified) {
 						toAdd.push('-c');
 					}
 				} else if (basename === 'wsl.exe') {
-					if (!taskShellArgsSpecified) {
+					if (!shellSpecified) {
 						toAdd.push('-e');
 					}
 				} else {
-					if (!taskShellArgsSpecified) {
-						// Push `-c` for unknown shells if the user didn't specify the args
-						toAdd.push('-c');
+					if (!shellSpecified) {
+						toAdd.push('/d', '/c');
 					}
 				}
 			} else {
-				if (!taskShellArgsSpecified) {
+				if (!shellSpecified) {
 					// Under Mac remove -l to not start it as a login shell.
 					if (platform === Platform.Platform.Mac) {
 						// Background on -l on osx https://github.com/microsoft/vscode/issues/107563
@@ -1272,12 +1268,11 @@ export class TerminalTaskSystem extends Disposable implements ITaskSystem {
 		const combinedShellArgs: string[] = Objects.deepClone(configuredShellArgs);
 		shellCommandArgs.forEach(element => {
 			const shouldAddShellCommandArg = configuredShellArgs.every((arg, index) => {
-				const isDuplicated = arg.toLowerCase() === element.toLowerCase();
-				if (isDuplicated && (configuredShellArgs.length > index + 1)) {
+				if ((arg.toLowerCase() === element) && (configuredShellArgs.length > index + 1)) {
 					// We can still add the argument, but only if not all of the following arguments begin with "-".
 					return !configuredShellArgs.slice(index + 1).every(testArg => testArg.startsWith('-'));
 				} else {
-					return !isDuplicated;
+					return arg.toLowerCase() !== element;
 				}
 			});
 			if (shouldAddShellCommandArg) {

diff --git a/src/vs/workbench/contrib/terminal/browser/terminalProfileResolverService.ts b/src/vs/workbench/contrib/terminal/browser/terminalProfileResolverService.ts
@@ -268,7 +268,6 @@ export abstract class BaseTerminalProfileResolverService extends Disposable impl
 		const automationProfile = this._configurationService.getValue(`terminal.integrated.automationProfile.${this._getOsKey(options.os)}`);
 		if (this._isValidAutomationProfile(automationProfile, options.os)) {
 			automationProfile.icon = this._getCustomIcon(automationProfile.icon) || Codicon.tools;
-			automationProfile.isAutomationShell = true;
 			return automationProfile;
 		}