fix: backport linked strategy fixes from multiple PRs to v10

[manzoorwanijk]

This contains the changes from

• a2154cd (fix: linked strategy fixes for scoped packages, aliases, and peer deps #8996)
• 880ecb7 (fix(arborist): skip postinstall on store links in linked strategy #9013)
• 26fa40e (fix: workspace-filtered install with linked strategy #9041)
• 983742b (fix: isolated mode code cleanup #9055)
• 10d5302 (fix(arborist): unwrap Link nodes in legacyPeerDeps for linked strategy #9051)
• a29aeee (fix(arborist): retry bin-links on Windows EPERM #9028)
• 16fbe13 (fix: resolve relative file: dependencies correctly with install-strategy=linked #9030)
• 8614b2a (fix(arborist): avoid full reinstall on subsequent linked strategy runs #9031)

Since Node 22 doesn't have npm 11 yet, it would be better to have this change backported to v10

Also, we wish to use install-strategy=linked in the Gutenberg monorepo, which powers the WordPress Block Editor. We are still on v10. So, these fixes will help.

[manzoorwanijk]

I am holding this off until #9013 is merged and then we can backport that as well.

[wraithgar]

When it comes time to update this w/ the other PRs we have been landing please just bring in the new file as-is in one commit.

I do not care about my own git attribution, there's no need to try to preserve any of that.

[manzoorwanijk]

When it comes time to update this w/ the other PRs we have been landing please just bring in the new file as-is in one commit.

Thank you.

Do we also need the changes from #9055?

These are the changes I am planning to cherry-pick

• a2154cd (fix: linked strategy fixes for scoped packages, aliases, and peer deps #8996)
• 880ecb7 (fix(arborist): skip postinstall on store links in linked strategy #9013)
• 26fa40e (fix: workspace-filtered install with linked strategy #9041)
• 10d5302 (fix(arborist): unwrap Link nodes in legacyPeerDeps for linked strategy #9051)
• a29aeee (fix(arborist): retry bin-links on Windows EPERM #9028)
• 16fbe13 (fix: resolve relative file: dependencies correctly with install-strategy=linked #9030)
• 8614b2a (fix(arborist): avoid full reinstall on subsequent linked strategy runs #9031)

[wraithgar]

Do we also need the changes from #9055?

Yes I think it will make things much easier to bring in, and make future updates easier.

[wraithgar]

I am also giving myself half of today to decide if #5820 is still worth looking at or if we will just close it and move on.

[manzoorwanijk]

@wraithgar, thank you for the hint. This is now ready for review.

[manzoorwanijk]

I am not sure how to fix the "Check Git Status" failure, though. I ran node ./scripts/resetdeps.js and it updated the lockfile. Looks like @gar/promise-retry has incompatible engines.

[wraithgar]

We just landed a deps update to the v10 branch, it's possible your fork is out of date.

[wraithgar]

Oooooh the promise-retry! You'll wanna use promise-retry instead, v10 did not move over to @gar/promisify.

[manzoorwanijk]

Switching to promise-retry seems to work.

[wraithgar]

@manzoorwanijk are we good to go here?

[manzoorwanijk]

@manzoorwanijk are we good to go here?

Yes, I think so.