fix: correct native dependency tree mismatch in app-builder rebuild

[leey0818]

This PR fixes the mismatch in the dependency tree structure provided by the rebuild-node-modules command in the app-builder binary.

[changeset-bot]

🦋 Changeset detected

Latest commit: 831a81e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 8 packages

Name	Type
app-builder-lib	Patch
electron-builder	Patch
dmg-builder	Patch
electron-builder-squirrel-windows	Patch
electron-forge-maker-appimage	Patch
electron-forge-maker-nsis-web	Patch
electron-forge-maker-nsis	Patch
electron-forge-maker-snap	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[netlify]

✅ Deploy Preview for car-park-attendant-cleat-11576 ready!

Name	Link
🔨 Latest commit	831a81e
🔍 Latest deploy log	https://app.netlify.com/sites/car-park-attendant-cleat-11576/deploys/66d9d663408ba70008a3f060
😎 Deploy Preview	https://deploy-preview-8450--car-park-attendant-cleat-11576.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[mmaietta]

@beyondkmp can you take a look at this when you have a chance? This PR is proposing conditionally applying the --flatten flag.

[mmaietta]

@leey0818 do you have a github issue that this is fixing? Or can you provide a minimal repro repo as to what the issue is originally? I'd like to create a unit test for this if possible

[beyondkmp]

@mmaietta This PR might be able to fix the issue(#8426 (comment))

I have a question: will the legacy configuration for nativeRebuilder be deprecated in the future, or will it be retained permanently?

nativeRebuilder = sequential “legacy” | “sequential” | “parallel” | “undefined” - Use legacy app-builder binary for installing native dependencies, or @electron/rebuild in sequential or parallel compilation modes.

[mmaietta]

I have a question: will the legacy configuration for nativeRebuilder be deprecated in the future, or will it be retained permanently?

Probably retained for quite some time, but eventually deprecated and finally removed. Just want to make it easy to debug any issues that arise between app-builder-bin migration to electron-rebuild. electron-builder v25 still doesn't have broad adoption as latest tag

[leey0818]

@mmaietta I have created a GitHub issue (#8459).

Using the legacy option to utilize pre-built native dependencies in a cross-platform build. It seems that @electron/rebuild does not yet support cross-platform builds.

[beyondkmp]

LGTM

[leey0818]

Fixed #8459