fix(runner): remove redundant module.instantiate calls

[chenjiahan]

Summary

Remove redundant module.instantiate() calls in the module runners. module.instantiate has been removed since Node.js >= 13.0.0.

Related Links

• vm: SourceTextModule refactor nodejs/node#29776

Checklist

• Tests updated (or not required).
• Documentation updated (or not required).

[netlify]

✅ Deploy Preview for rsbuild ready!

Name	Link
🔨 Latest commit	85f1a2b
🔍 Latest deploy log	https://app.netlify.com/projects/rsbuild/deploys/688dc4580c699800088ed0a9
😎 Deploy Preview	https://deploy-preview-5743--rsbuild.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
Lighthouse	1 paths audited Performance: 77 (🟢 up 23 from production) Accessibility: 97 (no change from production) Best Practices: 100 (no change from production) SEO: 100 (no change from production) PWA: 60 (no change from production) View the detailed breakdown and full score reports

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[Copilot]

Pull Request Overview

This PR removes redundant module.instantiate() calls from the module runners, as this method was removed from Node.js starting with version 13.0.0. The changes clean up outdated code that is no longer needed.

• Removes deprecated module.instantiate() calls from ESM and synthetic module runners
• Improves variable naming in the asModule function for better clarity
• Simplifies code structure by removing unnecessary conditional instantiate calls

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
packages/core/src/server/runner/esm.ts	Removes redundant esm.instantiate() call that is no longer needed
packages/core/src/server/runner/asModule.ts	Removes m.instantiate() call and improves variable naming from m to syntheticModule

Comments suppressed due to low confidence (1)

packages/core/src/server/runner/asModule.ts:32

• The removal of the instantiate() call should be covered by tests to ensure the module evaluation still works correctly without it, especially since this is a breaking change from older Node.js versions.

  await syntheticModule.evaluate();