Skip to content

Strip sourcemap reference from dev toolbar entrypoint#16481

Open
matthewp wants to merge 1 commit intomainfrom
strip-toolbar-sourcemap
Open

Strip sourcemap reference from dev toolbar entrypoint#16481
matthewp wants to merge 1 commit intomainfrom
strip-toolbar-sourcemap

Conversation

@matthewp
Copy link
Copy Markdown
Contributor

@matthewp matthewp commented Apr 24, 2026

Changes

  • This removes the frequent 404 caused by the browser loading a sourcemap that doesn't exist.
  • esbuild creates the sourcemap link, we remove it from the file when its optimized.

Testing

  • Verified manually: the spurious .js.map request no longer appears in dev server logs after opening a page in the browser.

Docs

  • No docs update needed.

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Apr 24, 2026

🦋 Changeset detected

Latest commit: ae6b48e

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions Bot added the pkg: astro Related to the core `astro` package (scope) label Apr 24, 2026
@matthewp matthewp marked this pull request as ready for review April 24, 2026 17:47
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Apr 24, 2026

Merging this PR will not alter performance

✅ 18 untouched benchmarks


Comparing strip-toolbar-sourcemap (ae6b48e) with main (1058428)

Open in CodSpeed

@matthewp matthewp changed the title Strip spurious sourcemap reference from dev toolbar entrypoint Strip sourcemap reference from dev toolbar entrypoint Apr 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pkg: astro Related to the core `astro` package (scope)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant