Merge import.meta.require and require to be the same thing

[paperdave]

We used to implement require twice. once for import.meta.require and then once in the actual require implementation. This is just old code from before the CJS rewrite.

import.meta.require and createRequire now create an empty JSCommonJSModule and return it's require function.

Fixes #3724, because the old require implementation would not append __esModule to certain case.

[github-actions]

❌ @paperdave 1 files with test failures on linux-x64-baseline:

• test/bundler/esbuild/ts.test.ts

View test output

^{#bc138117301c523e52c13c30c1f852b2abdb1335}

[github-actions]

❌ @paperdave 1 files with test failures on linux-x64:

• test/bundler/esbuild/ts.test.ts

View test output

^{#bc138117301c523e52c13c30c1f852b2abdb1335}

[github-actions]

❌ @paperdave 7 files with test failures on bun-darwin-aarch64:

• test/bundler/esbuild/ts.test.ts
• test/cli/install/bun-remove.test.ts
• test/cli/install/bunx.test.ts
• test/js/bun/net/socket.test.ts
• test/js/bun/spawn/spawn.test.ts
• test/js/bun/test/test-test.test.ts
• test/js/web/fetch/fetch-leak.test.js

View test output

^{#bc138117301c523e52c13c30c1f852b2abdb1335}

[github-actions]

❌ @paperdave 11 files with test failures on bun-darwin-x64-baseline:

• test/bundler/bundler_compile.test.ts
• test/bundler/bundler_npm.test.ts
• test/bundler/esbuild/ts.test.ts
• test/js/bun/http/serve.test.ts
• test/js/bun/spawn/spawn-streaming-stdin.test.ts
• test/js/bun/spawn/spawn-streaming-stdout.test.ts
• test/js/bun/sqlite/sqlite.test.js
• test/js/bun/util/sleepSync.test.ts
• test/js/third_party/webpack/webpack.test.ts
• test/js/web/timers/setInterval.test.js
• test/js/web/timers/setTimeout.test.js

View test output

^{#bc138117301c523e52c13c30c1f852b2abdb1335}

[Jarred-Sumner]

Looks like this breaks plugins because plugins for "real" CommonJS aren't actually implemented yet (that's why plugins + svelte tests are failing)

I think that will need to be addressed before this PR can move forward.