Revert "fix: clean up ESM registry when require() of ESM module fails…

[dylan-conway]

… (#27288)"

This reverts commit 21c3439.

[robobun]

^{Updated 2:23 AM PT - Feb 21st, 2026}

❌ @dylan-conway, your commit ba4fc52 has 5 failures in Build #37857 (All Failures):

• test/js/node/worker_threads/worker_threads.test.ts - pid 28691 segmentation fault on 🐧 25.04 x64
• test/cli/hot/watch-many-dirs.test.ts - pid 4529 segmentation fault on 🐧 3.23 x64
• test/regression/issue/24364.test.ts - 1 failing on 🪟 11 aarch64
• test/regression/issue/11297/11297.test.ts - 1 failing on 🪟 11 aarch64
• cmake/targets/BuildBun.cmake - cmake error on 🐧 x64-baseline - build-cpp

---

🧪   To try this PR locally:

bunx bun-pr 27325

That installs a local version of the PR into your bun-27325 executable, so you can run:

bun-27325 --bun

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

---

Walkthrough

Modified error handling in CommonJS module loading to preserve Loader.registry entries on exception, and removed a regression test for module evaluation behavior involving CJS require() and ESM import() interaction.

Changes

Cohort / File(s)	Summary
CommonJS Error Handling src/js/builtins/CommonJS.ts	Modified exception handling in overridableRequire and requireESMFromHijackedExtension to preserve Loader.registry entries while removing only per-id require map entries on error.
Test Removal test/regression/issue/27287.test.ts	Removed regression test verifying CJS require() and ESM import() behavior when an ESM module throws during evaluation.

🚥 Pre-merge checks | ✅ 1 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description is incomplete. It lacks the required 'What does this PR do?' and 'How did you verify your code works?' sections from the template.	Add the missing template sections: explain why the previous fix is being reverted and describe verification steps taken (e.g., test results demonstrating the revert resolves issues).

✅ Passed checks (1 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly indicates a revert operation of a specific commit related to ESM registry cleanup, directly reflecting the main changeset.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[claude]

✅ ba4fc — Looks good!

Reviewed 2 files across src/js/builtins/ and test/regression/issue/: Reverts the ESM registry cleanup logic that was added when require() of an ESM module fails, along with its associated regression test for issue #27287.