Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Possible to hit "HTML bundle not registered correctly" assertion failure #17956

Open
escobadio opened this issue Mar 7, 2025 · 3 comments
Open
Assignees
Labels
assertion failed Crashes in a build with assertions enabled bake:dev hot-reloading dev server for client+server applications needs investigate Needs to be investigated to find the root cause

Comments

@escobadio
Copy link

How can we reproduce the crash?

No response

Relevant log output

Stack Trace (bun.report)

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

panic: Internal assertion failure: HTML bundle not registered correctly

  • bake.DevServer.finalizeBundle
  • bundler.bundle_v2.BundleV2.onAfterDecrementScanCounter
  • bun.js.event_loop.ManagedTask.New
  • bun.js.event_loop.EventLoop.tickQueueWithCount__anon_516803
  • bun.js.event_loop.EventLoop.tick
  • bun.js.javascript.OpaqueWrap__anon_34405__struct_239833.callback
  • JSC__VM__holdAPILock
  • bun_js.Run.boot
  • cli.run_command.RunCommand._bootAndHandleError

Features: dotenv, http_server, jsc, dev_server

Sentry Issue: BUN-DVC

@escobadio escobadio added the crash An issue that could cause a crash label Mar 7, 2025
@github-actions github-actions bot added the macOS An issue that occurs on macOS label Mar 7, 2025
@paperclover paperclover added bake:dev hot-reloading dev server for client+server applications assertion failed Crashes in a build with assertions enabled and removed crash An issue that could cause a crash macOS An issue that occurs on macOS labels Mar 7, 2025
@paperclover
Copy link
Member

thank you for sending. it would help to have a reproduction for this, but just this is enough because it proves that it is possible to hit this panic:

Image

notice how this data is being overloaded with source map data. this is fine because HTML does not have a source map. the assertion fails because something is trying to add a source map, probably a blank one, to this file.

i have some big fixes i want to do first but this one should be easy to find when i give it the time

@escobadio
Copy link
Author

escobadio commented Mar 7, 2025 via email

@paperclover
Copy link
Member

I don’t use a source map anywhere currently it all just works.

sorry, im referring to the internal implementation. source maps are always enabled in development.

to be clear, no action is needed from you. enjoy your weekend

@paperclover paperclover self-assigned this Mar 7, 2025
@paperclover paperclover changed the title you asked me to send it ... Possible to hit "HTML bundle not registered correctly" assertion failure Mar 7, 2025
@paperclover paperclover added the needs investigate Needs to be investigated to find the root cause label Mar 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assertion failed Crashes in a build with assertions enabled bake:dev hot-reloading dev server for client+server applications needs investigate Needs to be investigated to find the root cause
Projects
None yet
Development

No branches or pull requests

2 participants