Skip to content

perf(simulator): lazy-load source maps - #1985

Open
TekenujUi90 wants to merge 1 commit into
dotandev:mainfrom
TekenujUi90:fix/issue-1936
Open

TekenujUi90 wants to merge 1 commit into
dotandev:mainfrom
TekenujUi90:fix/issue-1936

Conversation

@TekenujUi90

@TekenujUi90 TekenujUi90 commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Defer DWARF/source-map parsing until the first source lookup.
  • Initialize the parsed source map once per mapper and retain the existing lookup behavior.

Verification

  • cargo test --lib source_mapper::tests — 8 passed.
  • cargo fmt --manifest-path simulator/Cargo.toml -- --check — passed.
  • git diff --check — passed.

Fixes #1936

@vercel

vercel Bot commented Sep 28, 2026

Copy link
Copy Markdown

@zeemscript is attempting to deploy a commit to the jesusoflagos' projects Team on Vercel.

A member of the Team first needs to authorize it.

@TekenujUi90

Copy link
Copy Markdown
Contributor Author

@dotandev @TekenujUi90 This PR isolates the lazy source-map parsing change as requested and references only Fixes #1936. It touches only simulator/src/source_mapper.rs (commit d121c524). Focused source-mapper Rust tests are running; I will update with their result.

@zeemscript

Copy link
Copy Markdown

@dotandev @TekenujUi90 Final verification: cargo test --lib source_mapper::tests passed (8 tests), cargo fmt --manifest-path simulator/Cargo.toml -- --check passed, and git diff --check passed. GitHub shows only simulator/src/source_mapper.rs; the PR body references only Fixes #1936.

@dotandev

Copy link
Copy Markdown
Owner

do you mind checking your commit authors?

2 separate accounts? look into that.

This branch has not been deployed

No deployments
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.

[EXT] Lazy-load source maps to reduce debugger startup time

3 participants