Dont tear down shell startup when switched to command (microsoft#1071)

anthonykim1 · web-flow · commit f026743692d1 · 2025-12-17T12:02:35.000-08:00
Resolves: microsoft#1023 Came from: microsoft#1070 Tests coming: microsoft#1019
diff --git a/src/features/terminal/terminalManager.ts b/src/features/terminal/terminalManager.ts
@@ -135,11 +135,8 @@ export class TerminalManagerImpl implements TerminalManager {
                         }
                     } else {
                         traceVerbose(
-                            `Auto activation type changed to ${actType}, we are cleaning up shell startup setup`,
+                            `Auto activation type changed to ${actType}, not tearing down shell startup scripts on activation type switch; scripts are only removed via explicit revert.`,
                         );
-                        // Teardown scripts when switching away from shell startup activation
-                        await Promise.all(this.startupScriptProviders.map((p) => p.teardownScripts()));
-                        this.shellSetup.clear();
                     }
                 }
             }),

Original file line number	Diff line number	Diff line change
`@@ -135,11 +135,8 @@ export class TerminalManagerImpl implements TerminalManager {`
`135`	`135`	`}`
`136`	`136`	`} else {`
`137`	`137`	`traceVerbose(`
`138`		- `Auto activation type changed to ${actType}, we are cleaning up shell startup setup`,
	`138`	+ `Auto activation type changed to ${actType}, not tearing down shell startup scripts on activation type switch; scripts are only removed via explicit revert.`,
`139`	`139`	`);`
`140`		`- // Teardown scripts when switching away from shell startup activation`
`141`		`- await Promise.all(this.startupScriptProviders.map((p) => p.teardownScripts()));`
`142`		`- this.shellSetup.clear();`
`143`	`140`	`}`
`144`	`141`	`}`
`145`	`142`	`}),`