Bug in create venv flow with selecting skipping package installation (#928)

eleanorjboyd · amunger · web-flow · commit 49e4242c1927 · 2025-10-10T13:33:03.000-07:00
fixes #930 --------- Co-authored-by: Aaron Munger <2019016+amunger@users.noreply.github.com>
diff --git a/src/managers/builtin/pipUtils.ts b/src/managers/builtin/pipUtils.ts
@@ -129,7 +129,7 @@ async function selectWorkspaceOrCommon(
                 return await selectFromCommonPackagesToInstall(common, installed, undefined, { showBackButton });
             } else if (selected.label === PackageManagement.skipPackageInstallation) {
                 traceInfo('Package Installer: user selected skip package installation');
-                return undefined;
+                return { install: [], uninstall: [] } satisfies PipPackages;
             } else {
                 return undefined;
             }

Original file line number	Diff line number	Diff line change
`@@ -129,7 +129,7 @@ async function selectWorkspaceOrCommon(`
`129`	`129`	`return await selectFromCommonPackagesToInstall(common, installed, undefined, { showBackButton });`
`130`	`130`	`} else if (selected.label === PackageManagement.skipPackageInstallation) {`
`131`	`131`	`traceInfo('Package Installer: user selected skip package installation');`
`132`		`- return undefined;`
	`132`	`+ return { install: [], uninstall: [] } satisfies PipPackages;`
`133`	`133`	`} else {`
`134`	`134`	`return undefined;`
`135`	`135`	`}`