Bun Crashes when changing client.js javascript code. How to disable HMR javascript injection on the HTML page?

[dicinohit]

How can we reproduce the crash?

I have a server.js importing and serving a static HTML homepage that uses an ES module called client.js. When I change the client code, Bun crashes.

By the way, I do not like that you guys inject JavaScript code for the "hmr" functionality. I don't need any of this HMR stuff. In the browser network information, I see a pending socket connection. How can I disable this HMR code injection? I see no documentation about it anywhere in your docs. It's annoying that you do code injection without any docs or mention of how to disable this.

I just started coding this, so the source code is very small... On the Bun server.js:

import homepage from "./index.html";

Bun.serve({
static: {
"/": homepage,
},
port: 3000
}

The HTML has this javascript " <script type="module" src="client.js"></script>". The client code is just a class.

I guess this error message is because the port was not freed when it crashed. So, I don't think this log helps much:

Relevant log output

Bun.serve({
         ^
error: Failed to start server. Is port 3000 in use?
 syscall: "listen",
   errno: 0,
    code: "EADDRINUSE"

      at C:\Dev\Projects\snamba\server.js:18:5

Stack Trace (bun.report)

Bun v1.2.4 (fd9a5ea) on windows x86_64 [AutoCommand]

panic: Internal assertion failure

• array_list.zig:71: deinit
• DevServer.zig:2464: finalizeBundle
• bundle_v2.zig:2726: finishFromBakeDevServer
• memory_allocator.zig:31: mimalloc_free
• memory_allocator.zig:31: mimalloc_free
• event_loop.zig:1377: tickConcurrent
• javascript.zig:1256: handlePendingInternalPromiseRejection
• bindings.cpp:5248: JSC__VM__holdAPILock

Features: tsconfig, tsconfig_paths, process_dlopen, Bun.stderr, Bun.stdout, bunfig, http_server, jsc, dev_server

_{Sentry Issue: BUN-D41}

[kravetsone]

U can disable HMR with `development: {hmr: false} at Bun.serve option

[dicinohit]

Great. Removing HMR helped, and Bun is not crashing when changing the client code because it stopped doing hot reloading.

I suggest setting "hmr: false" as the default until HMR is production-ready because it is not working now even with a simple code setup.