fix(core): nesting npm lock file deps should support multi nests

[meeroslav]

Imagine the following situation:

A@v1 -> D@v2 -> B@v2
B@v1
C@v1 -> B@v2
D@v1

Package B@v2 will have two nests - one as a direct child of C and another one as a child of A (since A has no B child of its own). Current implementation explicitly checks that there is only a single slot to place B@v2 causing this package to never be nested.

We need to support multiple nesting slots.

Current Behavior

If the package version has multiple nesting places, the stringifier ends up in the loop

Expected Behavior

Npm stringifier should support multiple nesting slots for the same package

Related Issue(s)

Fixes #

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated
nx-dev	✅ Ready (Inspect)	Visit Preview	💬 Add your feedback	Feb 13, 2023 at 9:41AM (UTC)

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit ba256f2. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch

---

✅ Successfully ran 8 targets

• nx run-many -t e2e-macos --parallel=1 --exclude=e2e-detox
• nx affected --target=e2e --base=c712949b7e8df9ffdaf7f711157d2bf1cfd48868 --head=ba256f24ce762758346b8ecbe4f2a2da89635133 --exclude=e2e-storybook,e2e-storybook-angular --parallel=1
• nx affected --target=test --base=c712949b7e8df9ffdaf7f711157d2bf1cfd48868 --head=ba256f24ce762758346b8ecbe4f2a2da89635133 --parallel=1
• nx affected --target=build --base=c712949b7e8df9ffdaf7f711157d2bf1cfd48868 --head=ba256f24ce762758346b8ecbe4f2a2da89635133 --parallel=3
• nx run-many -t root-lint check-imports check-commit check-lock-files depcheck --parallel=1 --no-dte
• nx affected --target=lint --base=c712949b7e8df9ffdaf7f711157d2bf1cfd48868 --head=ba256f24ce762758346b8ecbe4f2a2da89635133 --parallel=3
• nx-cloud record -- nx format:check --base=c712949b7e8df9ffdaf7f711157d2bf1cfd48868 --head=ba256f24ce762758346b8ecbe4f2a2da89635133
• nx documentation --no-dte

---

Sent with 💌 from NxCloud.

[github-actions]

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.