Skip to content

[V2] [Fix] Skip debug ID injection for modulesOnly bundles#1299

Draft
sbarrio wants to merge 1 commit into
feature/v2from
sbarrio/v2/fix/metro-modules-only-debug-id-crash
Draft

[V2] [Fix] Skip debug ID injection for modulesOnly bundles#1299
sbarrio wants to merge 1 commit into
feature/v2from
sbarrio/v2/fix/metro-modules-only-debug-id-crash

Conversation

@sbarrio
Copy link
Copy Markdown
Contributor

@sbarrio sbarrio commented Jun 3, 2026

What does this PR do?

Reported by: #1297

The metro serializer crashed when Metro asked for a modulesOnly bundle (lazy imports / split chunks), because those bundles don't carry a pre modules section that the Debug ID injection step relies on.

This fix skips running the serializer for modulesOnly=true bundles, avoiding the crash.

Motivation

The debug id implementation should not clash with the dev metro server.

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)
  • If this PR is auto-generated, please make sure also to manually update the code related to the change

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant