Skip to content

Commit aaebd3e

Browse files
Add --ignore-scripts flag to package manager prune commands (#192)
* Add --ignore-scripts flag to package manager prune commands * Oops! missed it --------- Co-authored-by: Jake Runzer <[email protected]>
1 parent 1379922 commit aaebd3e

4 files changed

+7
-6
lines changed

core/__snapshots__/TestGenerateBuildPlanForExamples_php-laravel-11-react_1.snap.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -235,7 +235,7 @@
235235
],
236236
"commands": [
237237
{
238-
"cmd": "npm prune --omit=dev"
238+
"cmd": "npm prune --omit=dev --ignore-scripts"
239239
}
240240
],
241241
"inputs": [

core/__snapshots__/TestGenerateBuildPlanForExamples_php-laravel-12-react_1.snap.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -235,7 +235,7 @@
235235
],
236236
"commands": [
237237
{
238-
"cmd": "npm prune --omit=dev"
238+
"cmd": "npm prune --omit=dev --ignore-scripts"
239239
}
240240
],
241241
"inputs": [

core/__snapshots__/TestGenerateBuildPlanForExamples_ruby-with-node_1.snap.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -193,7 +193,7 @@
193193
],
194194
"commands": [
195195
{
196-
"cmd": "npm prune --omit=dev"
196+
"cmd": "npm prune --omit=dev --ignore-scripts"
197197
}
198198
],
199199
"inputs": [

core/providers/node/package_manager.go

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -129,12 +129,12 @@ func (p PackageManager) PruneDeps(ctx *generate.GenerateContext, prune *generate
129129

130130
switch p {
131131
case PackageManagerNpm:
132-
prune.AddCommand(plan.NewExecCommand("npm prune --omit=dev"))
132+
prune.AddCommand(plan.NewExecCommand("npm prune --omit=dev --ignore-scripts"))
133133
case PackageManagerPnpm:
134-
prune.AddCommand(plan.NewExecCommand("pnpm prune --prod"))
134+
prune.AddCommand(plan.NewExecCommand("pnpm prune --prod --ignore-scripts"))
135135
case PackageManagerBun:
136136
// Prune is not supported in Bun. https://github.com/oven-sh/bun/issues/3605
137-
prune.AddCommand(plan.NewExecShellCommand("rm -rf node_modules && bun install --production"))
137+
prune.AddCommand(plan.NewExecShellCommand("rm -rf node_modules && bun install --production --ignore-scripts"))
138138
case PackageManagerYarn1:
139139
prune.AddCommand(plan.NewExecCommand("yarn install --production=true"))
140140
case PackageManagerYarnBerry:
@@ -155,6 +155,7 @@ func (p PackageManager) pruneYarnBerry(ctx *generate.GenerateContext, prune *gen
155155
}
156156

157157
// Yarn 2 and 4+ support workspaces focus (also fallback for unknown versions)
158+
// Note: yarn workspaces focus doesn't support --ignore-scripts flag
158159
prune.AddCommand(plan.NewExecCommand("yarn workspaces focus --production --all"))
159160
}
160161

0 commit comments

Comments
 (0)