fix(arborist): backport linked strategy hoisting fixes to v10

[manzoorwanijk]

Cherry pick of:

• 5fd40b6 (fix(arborist): do not hoist undeclared workspaces in linked strategy #9076)
• e2c2443 (fix(arborist): omit root dev deps in linked strategy when shared with workspaces #9081)

Additional changes:

• Ported shouldOmit method to v10's Node class (exists on latest but was missing on v10) with full test coverage

References

[manzoorwanijk]

CC @wraithgar

[wraithgar]

This is a hairy one. Normally I'd say this should go on the node itself but can see why we would want it to be inversed (pass in the node and have the omits as context instead) so that we minimize how much we have to backport.

We're getting into quite a bit of divergence from the main branch and this is going to start to become harder and harder to maintain as time goes on.

[manzoorwanijk]

This is a hairy one. Normally I'd say this should go on the node itself but can see why we would want it to be inversed (pass in the node and have the omits as context instead) so that we minimize how much we have to backport.

We're getting into quite a bit of divergence from the main branch and this is going to start to become harder and harder to maintain as time goes on.

Should we move shouldOmit to the Node class and update the tests?

[manzoorwanijk]

Moved shouldOmit to the Node class itself to match latest, keeping the same API surface on both branches should make future backports easier.