fix: Resolver recursion in getDependencyVersion

[DeMoorJasper]

What kind of change does this PR introduce?

Fixes an infinite recursion between getDependencyVersion and resolvePath in fetch-npm-module

Sandbox to see this in action: https://codesandbox.io/s/yq0z0j (beware this will likely crash your browser-tab)

[CompuIves]

Good find. LGTM!

One case I had in mind, sometimes a dependency calls itself. E.g.

// node_modules/a/b.js

require('a/a'); // <- should resolve to node_modules/a/a.js

I've seen this case before once, do you think that still works?

[codesandbox-ci]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit a45529d:

Sandbox	Source
Notifications Test	Configuration
FileList	PR

[lbogdan]

Build for latest commit a45529d is at https://pr6487.build.csb.dev/s/new.

[DeMoorJasper]

Good find. LGTM!

One case I had in mind, sometimes a dependency calls itself. E.g.

// node_modules/a/b.js



require('a/a'); // <- should resolve to node_modules/a/a.js

I've seen this case before once, do you think that still works?

That should still work afaik, there's also an integration test for that case so tests should catch that