A few fixes related to CommonJS module loading

[Jarred-Sumner]

What does this PR do?

• Add extra checks to ensure source code is always freed when it should be
• Optimization: precompute the hash upfront for source code transpiled on another thread. This seems
• Make Bun.gc(true) free more memory
• Make CommonJSModule store the JSC::SourceCode directly instead of using JSC::JSSourceCode. Goal is to free it earlier instead of whenever GC decides to free it.
• Fixes signal SIGSEGV: invalid address #6946
• Fixes Crash in module loader when cycles exist between ESM & CJS #8965
• Fixes a bug where bun loads & transpiles the same code twice(!)

How did you verify your code works?

It doesn't work yet

It crashes in release builds, but not debug builds

I suspect something fishy with either GC or a C++ use-after-move somewhere. Worst case I revert it back

[github-actions]

❌ @Jarred-Sumner 4 files with test failures on linux-x64:

• test/integration/next/default-pages-dir/test/dev-server.test.ts 1 failing
• test/integration/next/default-pages-dir/test/next-build.test.ts SIGTERM
• test/js/node/fs/fs.test.ts 2 failing
• test/js/third_party/jsonwebtoken/claim-aud.test.js 1 failing

View test output

^{#1f63bc70da1480bcf6e8b0814cf96b3fb369ca8e}

[github-actions]

❌ @Jarred-Sumner 5 files with test failures on linux-x64-baseline:

• test/integration/next/default-pages-dir/test/dev-server.test.ts 1 failing
• test/integration/next/default-pages-dir/test/next-build.test.ts SIGTERM
• test/js/node/fs/fs.test.ts 2 failing
• test/js/node/watch/fs.watchFile.test.ts 1 failing
• test/js/web/timers/setTimeout.test.js 1 failing

View test output

^{#1f63bc70da1480bcf6e8b0814cf96b3fb369ca8e}

[github-actions]

❌🪟 @Jarred-Sumner, there are 10 test regressions on Windows x86_64

• test\cli\hot\hot.test.ts
• test\cli\install\bun-run-bunfig.test.ts
• test\cli\install\bunx.test.ts
• test\cli\install\registry\bun-install-registry.test.ts
• test\js\bun\console\console-iterator.test.ts
• test\js\bun\glob\scan.test.ts
• test\js\bun\shell\commands\rm.test.ts
• test\js\node\dns\node-dns.test.js
• test\js\web\fetch\body-stream.test.ts
• test\js\web\websocket\websocket.test.js

Full Test Output

[github-actions]

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

• test/integration/next/default-pages-dir/test/next-build.test.ts SIGTERM
• test/js/bun/http/serve.test.ts SIGTERM
• test/js/bun/shell/bunshell.test.ts 1 failing
• test/js/bun/shell/leak.test.ts SIGTERM
• test/js/bun/test/test-test.test.ts 1 failing
• test/js/node/fs/fs.test.ts 2 failing
• test/js/third_party/prisma/prisma.test.ts 1 failing

View test output

^{#1f63bc70da1480bcf6e8b0814cf96b3fb369ca8e}

[github-actions]

❌ @Jarred-Sumner 4 files with test failures on bun-darwin-x64:

• test/integration/next/default-pages-dir/test/next-build.test.ts SIGTERM
• test/js/bun/shell/leak.test.ts 1 failing
• test/js/node/fs/fs.test.ts 2 failing
• test/js/third_party/prisma/prisma.test.ts 2 failing

View test output

^{#1f63bc70da1480bcf6e8b0814cf96b3fb369ca8e}

[staabm]

• Optimization: precompute the hash upfront for source code transpiled on another thread. This seems

nit: it seems the PR description is truncated at this point

[paperdave]

looks good now.

if you are worried about JS code being wrong, feel free to add more $asserts into it (they are tree-shaken)