pnpm rebuild not working when node-linker=hoisted

[balajiv113]

pnpm version:

pnpm - 7.14.0

Code to reproduce the issue:

https://github.com/balajiv113/pnpm-test

Expected behavior:

pnpm rebuild should build native modules even when node-linker=hoisted

Actual behavior:

pnpm rebuild not running as expected node-linker=hoisted

Additional information:

• macOS - intel machine

Due to this issue, we are not able to do build electron application using electron-builder for multiple architectures (generating arm apps in intel and vice-versa). The app gets built, but the native binaries are on wrong architecture

[zkochan]

Looks like the rebuild command can only lookup packages inside the node_modules/.pnpm directory as of now:

pnpm/packages/plugin-commands-rebuild/src/implementation/index.ts

Lines 273 to 280 in b234923

	const pkgSnapshot = pkgSnapshots[depPath]
	const pkgInfo = nameVerFromPkgSnapshot(depPath, pkgSnapshot)
	const pkgRoot = path.join(ctx.virtualStoreDir, dp.depPathToFilename(depPath), 'node_modules', pkgInfo.name)
	try {
	const modules = path.join(ctx.virtualStoreDir, dp.depPathToFilename(depPath), 'node_modules')
	const binPath = path.join(pkgRoot, 'node_modules', '.bin')
	await linkBins(modules, binPath, { extraNodePaths: ctx.extraNodePaths, warn })
	await runPostinstallHooks({

Why do you need the rebuild command at all? The deps are built on pnpm install.

[balajiv113]

Electron builder makes use of this rebuild command to rebuild the native dependency for different arch.

Due to this issue, for all arch (amd64, arm64) the native dependency are compiled to only to one (amd64).