fix(commonjs): Only proxy detected commonjs entry points

[lukastaegert]

Rollup Plugin Name: commonjs

This PR contains:

• bugfix
• feature
• refactor
• documentation
• other

Are tests included?

• yes (bugfixes and features will not be merged without tests)
• no

Breaking Changes?

• yes (breaking changes will not be merged unless absolutely necessary)
• no

If yes, then include "BREAKING CHANGES:" in the first commit message body, followed by a description of what is breaking.

List any relevant issue numbers:
Resolves #1169

Description

This will ease the issue in #1169 by only proxying detected CommonJS entry points (and not just every entry point). It does so via:

• Initially using the include/exclude filtering mechanism
• Using the detected module type information

Thus if you do not have commonjs entry points, you should no longer see the ?entry-proxy modules replacing your entry points.

[silkfire]

NB: Node.js v12 went EOL today, maybe remove it and introduce v17 as it goes LTS tomorrow?

[sodatea]

I can confirm that this PR fixes most issues that prevent Vite from updating the commonjs plugin.

Before patching with this fix, most test cases in Vite fail (likely due to this bug interfering with the default .html entry); after patching, only 1 case is still failing (and I'm still investigating, maybe not caused by the CJS plugin).

[lukastaegert]

I will ensure it is released