fix(@angular/build): remove Vite "/@id/" prefix for explicit external dependencies

[kfrederix]

This PR adds a Vite plugin to remove the /@id/ prefix (which gets inserted by Vite during import-analysis) for explicit externalDependencies.

PR Checklist

Please check to confirm your PR fulfills the following requirements:

• The commit message follows our guidelines: https://github.com/angular/angular-cli/blob/main/CONTRIBUTING.md#-commit-message-guidelines
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

• Bugfix
• Feature
• Code style update (formatting, local variables)
• Refactoring (no functional changes, no api changes)
• Build related changes
• CI related changes
• Documentation content changes
• Other... Please describe:

What is the current behavior?

Issue Number: #28028

What is the new behavior?

The /@id/ prefix gets removed from a new Vite plugin.

Does this PR introduce a breaking change?

• Yes
• No

Other information

[clydin]

This approach appears viable as a workaround.
I left several initial comments.
There is the potential for a false positive replacement. However, the likelihood is very low and would only occur if externalDependencies is used which is not overly common in its own. So I think that is acceptable as a (hopefully) temporary workaround.

[clydin]

This approach appears viable as a workaround.
I left several initial comments.
There is the potential for a false positive replacement. However, the likelihood is very low and would only occur if externalDependencies is used which is not overly common in its own. So I think that is acceptable as a (hopefully) temporary workaround.

[kfrederix]

Also added a test now. Let me know if you have more suggestions or concerns.

[clydin]

Just one comment otherwise LGTM.

Thank you for the contribution.

[kfrederix]

Thank you for the fast reviews!

[dgp1130]

The changes were merged into the following branches: main, 18.1.x

[angular-automatic-lock-bot]

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}