Module graph reports multiple times same modules imports

[underfisk]

Describe the bug

I have been playing around with the module graph and I was curious if the reported modules are actually duplicated or if it's just an analysis of the file imports because it looks like lodash is represented more than once and some other packages (mostly @mui related but even testing library appears multiple times).

Reproduction

https://github.com/underfisk/vitest-module-graph

System Info

System:
    OS: macOS 12.3.1
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
    Memory: 2.49 GB / 64.00 GB
    Shell: 3.2.2 - /usr/local/bin/fish
  Binaries:
    Node: 16.15.0 - ~/.volta/tools/image/node/16.15.0/bin/node
    Yarn: 1.22.18 - ~/.volta/tools/image/yarn/1.22.18/bin/yarn
    npm: 8.5.5 - ~/.volta/tools/image/node/16.15.0/bin/npm
  Browsers:
    Chrome: 101.0.4951.64
    Safari: 15.4

Used Package Manager

yarn

Validations

• Follow our Code of Conduct
• Read the Contributing Guidelines.
• Read the docs.
• Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
• Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
• The provided reproduction is a minimal reproducible example of the bug.

[nieyuyao]

It seems that there should actually be two modules @testing-library/jest-dom and @testing-library/react. But showing the same name?

[underfisk]

@nieyuyao Correct, i believe that for the testing library it does make sense but jest-dom is invoked only at the setup file so its not directly invoked through the file.

[sheremet-va]

@nieyuyao Correct, i believe that for the testing library it does make sense but jest-dom is invoked only at the setup file so its not directly invoked through the file.

Setup file is part of the suite, tho. Module Graph shows you every module that was invoked durung the run.

[nieyuyao]

The following code truncates the dependency name. So how to deal with dependencies with the same package name?

vitest/packages/ui/client/composables/module-graph.ts

Lines 12 to 16 in c3621c9

	function defineExternalModuleNode(module: string): ModuleNode {
	let label = module
	if (label.includes('/node_modules/'))
	label = label.split(/\/node_modules\//g).pop()!.split(/\//g).shift()!
	else