fix: only import needed runtime symbols

[hyf0]

Description

The fix is done by more delicate analyzing the module type and import kind.

This also fixes #319 (comment).

Test Plan

---

[hyf0]

Current dependencies on/for this PR:

• main
  • PR feat: add ExportsKind::None #315
    • PR chore: tweak visitability of Scanner #316
      • PR feat: empty module should have ExportsKind::None #317
        • PR feat: treat required module with ExportKind::None as commonjs #319
          • PR chore: remove unnecessary casting #320
            • PR chore(js): deny lint warnings in CI #321
              • PR refactor: create wrapper_ref after wrapping #322
                • PR fix: ensure valid output for import the same cjs twice #323
                  • PR fix: only import needed runtime symbols #324 👈
                  • PR chore: remove unused LinkingInfo#export_from_map #325
                  • PR feat: make BuildError newtype #326
                  • PR feat: introduce trait BuildErrorLike #327
                  • PR feat: store source in BuildError #328
                  • PR feat: print fancy messages for errors #329
                  • PR feat: basic warning mechanism #332

This stack of pull requests is managed by Graphite.

[hyf0]

Merge activity

• Nov 19, 7:13 AM: @hyf0 started a stack merge that includes this pull request via Graphite.
• Nov 19, 7:13 AM: Graphite rebased this pull request as part of a merge.
• Nov 19, 7:14 AM: @hyf0 merged this pull request with Graphite.