fix: add electron-winstaller in onlyBuiltDependencies (#3924)

wy-luke · BlackHole1 · VerteDinde · web-flow · commit 01dc08027661 · 2025-05-07T21:48:45.000Z
* fix: add electron-winstaller in onlyBuiltDependencies Add `electron-winstaller` in `onlyBuiltDependencies` when using pnpm. Related issue: #3892 * style: format Co-authored-by: Kevin Cui <github@bugs.cc> --------- Co-authored-by: Kevin Cui <github@bugs.cc> Co-authored-by: Keeley Hammond <vertedinde@electronjs.org> Co-authored-by: Erick Zhao <erick@hotmail.ca>
diff --git a/packages/template/base/src/BaseTemplate.ts b/packages/template/base/src/BaseTemplate.ts
@@ -108,7 +108,7 @@ export class BaseTemplate implements ForgeTemplate {
     // As of pnpm v10, no postinstall scripts will run unless allowlisted through `onlyBuiltDependencies`
     if (pm.executable === 'pnpm') {
       packageJSON.pnpm = {
-        onlyBuiltDependencies: ['electron'],
+        onlyBuiltDependencies: ['electron', 'electron-winstaller'],
       };
     }
 

Original file line number	Diff line number	Diff line change
`@@ -108,7 +108,7 @@ export class BaseTemplate implements ForgeTemplate {`
`108`	`108`	// As of pnpm v10, no postinstall scripts will run unless allowlisted through `onlyBuiltDependencies`
`109`	`109`	`if (pm.executable === 'pnpm') {`
`110`	`110`	`packageJSON.pnpm = {`
`111`		`- onlyBuiltDependencies: ['electron'],`
	`111`	`+ onlyBuiltDependencies: ['electron', 'electron-winstaller'],`
`112`	`112`	`};`
`113`	`113`	`}`
`114`	`114`