Merge pull request microsoft#184363 from microsoft/merogge/fish-auto

don't automatically inject fish shell integration

Author: meganrogge

src/vs/platform/terminal/node/terminalEnvironment.ts

@@ -176,11 +176,12 @@ export function getShellIntegrationInjection(
 		case 'fish': {
 			// The injection mechanism used for fish is to add a custom dir to $XDG_DATA_DIRS which
 			// is similar to $ZDOTDIR in zsh but contains a list of directories to run from.
-			const oldDataDirs = env?.XDG_DATA_DIRS ?? '/usr/local/share:/usr/share';
-			const newDataDir = path.join(appRoot, 'out/vs/workbench/contrib/terminal/browser/media/fish_xdg_data');
-			envMixin['XDG_DATA_DIRS'] = `${oldDataDirs}:${newDataDir}`;
-			addEnvMixinPathPrefix(options, envMixin);
-			return { newArgs: undefined, envMixin };
+			// const oldDataDirs = env?.XDG_DATA_DIRS ?? '/usr/local/share:/usr/share';
+			// const newDataDir = path.join(appRoot, 'out/vs/workbench/contrib/terminal/browser/media/fish_xdg_data');
+			// envMixin['XDG_DATA_DIRS'] = `${oldDataDirs}:${newDataDir}`;
+			// addEnvMixinPathPrefix(options, envMixin);
+			// return { newArgs: undefined, envMixin };
+			return undefined;
 		}
 		case 'pwsh': {
 			if (!originalArgs || arePwshImpliedArgs(originalArgs)) {