fix(deploy): always purge modules directory

close #6510
pnpm · May 7, 2023 · 4916a62 · 4916a62
1 parent 5e0165c
commit 4916a62
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 2 deletions.
diff --git a/pkg-manager/core/src/install/extendInstallOptions.ts b/pkg-manager/core/src/install/extendInstallOptions.ts
@@ -123,6 +123,7 @@ export interface StrictInstallOptions {
   dedupePeerDependents: boolean
   extendNodePath: boolean
   excludeLinksFromLockfile: boolean
+  confirmModulesPurge: boolean
 }
 
 export type InstallOptions =
@@ -139,6 +140,7 @@ const defaults = async (opts: InstallOptions) => {
     allowNonAppliedPatches: false,
     autoInstallPeers: true,
     childConcurrency: 5,
+    confirmModulesPurge: opts.force,
     depth: 0,
     enablePnp: false,
     engineStrict: false,

diff --git a/pkg-manager/get-context/src/index.ts b/pkg-manager/get-context/src/index.ts
@@ -72,6 +72,7 @@ interface HookOptions {
 
 export interface GetContextOptions {
   allProjects: Array<ProjectOptions & HookOptions>
+  confirmModulesPurge?: boolean
   force: boolean
   forceNewModules?: boolean
   forceSharedLockfile: boolean
@@ -116,7 +117,7 @@ export async function getContext (
       registries: opts.registries,
       storeDir: opts.storeDir,
       virtualStoreDir,
-      confirmModulesPurge: !opts.force && !isCI,
+      confirmModulesPurge: opts.confirmModulesPurge && !isCI,
 
       forceHoistPattern: opts.forceHoistPattern,
       hoistPattern: opts.hoistPattern,
@@ -408,6 +409,7 @@ export async function getContextForSingleImporter (
     force: boolean
     forceNewModules?: boolean
     forceSharedLockfile: boolean
+    confirmModulesPurge?: boolean
     extraBinPaths: string[]
     extendNodePath?: boolean
     lockfileDir: string
@@ -472,7 +474,7 @@ export async function getContextForSingleImporter (
       registries: opts.registries,
       storeDir: opts.storeDir,
       virtualStoreDir,
-      confirmModulesPurge: !opts.force && !isCI,
+      confirmModulesPurge: opts.confirmModulesPurge && !isCI,
 
       forceHoistPattern: opts.forceHoistPattern,
       hoistPattern: opts.hoistPattern,

diff --git a/pkg-manager/plugin-commands-installation/src/install.ts b/pkg-manager/plugin-commands-installation/src/install.ts
@@ -305,6 +305,7 @@ export type InstallCommandOptions = Pick<Config,
   saveLockfile?: boolean
   workspace?: boolean
   includeOnlyPackageFiles?: boolean
+  confirmModulesPurge?: boolean
 } & Partial<Pick<Config, 'modulesCacheMaxAge' | 'pnpmHomeDir' | 'preferWorkspacePackages'>>
 
 export async function handler (

diff --git a/releasing/plugin-commands-deploy/src/deploy.ts b/releasing/plugin-commands-deploy/src/deploy.ts
@@ -78,6 +78,7 @@ export async function handler (
   await copyProject(deployedDir, deployDir, { includeOnlyPackageFiles })
   await install.handler({
     ...opts,
+    confirmModulesPurge: false,
     depth: Infinity,
     hooks: {
       ...opts.hooks,