Description

Depends on #10406. The change is 3789d8f.

This fixes what #10450 tried to fix.

When

• an app depends on A
• A depends on B
• A and B are both written in CJS
• B is externalized
• A is no-externalized

require('B') written in A is converted to import 'B'. But converting require into import is not always safe. For example, if B uses exports field and uses different files.
This PR fixes this by preserving require as-is for SSR with node target.

Additional context

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.