fix: avoid import duplicate chunks #2906

[psaren]

Description

Fix #2906 Maximum call stack size exceeded because import duplicate chunks.

Additional context

I have build vite in my local npm through verdaccio,
and install it to in Reproduction repo https://github.com/g4rcez/vite-testing;
I run yarn build successfully.

---

What is the purpose of this pull request?

• Bug fix
• New Feature
• Documentation update
• Other

Before submitting the PR, please make sure you do the following

• Read the Contributing Guidelines.
• Read the Pull Request Guidelines and follow the Commit Convention.
• Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
• Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
• Ideally, include relevant tests that fail without this PR but pass with it.

[Shinigami92]

Cause tests are failing here, I converted this PR to a draft

[psaren]

@Shinigami92
I check out the latest main branch of vite repo & yarn test-build, it also shown the same test fail message.

[Shinigami92]

But something needs to be wrong on your side
cause the last commit to main branch is green and other issues already based on the newest main are also green (e.g. #2939)

[g4rcez]

Awesome @psaren. Thanks for fixing #2906

[psaren]

@g4rcez some tests failed, i try to fix it, but it hasn't been fixed yet

[psaren]

@g4rcez
in your viteconfig.js, Maximum call stack size exceeded because import duplicate chunks.
I find the mthod getCssTagsForChunk in plugin [vite:html] is recursive call, so we need to filter the same chunk to avoid Infinite call.
so i add the code in fbba7b2.
but some tests failed.

[psaren]

@Shinigami92 Hello, I have solved the problem that the some test case failed.