Remove data-astro-source-file and data-astro-source-loc attributes in dev mode

[natemoo-re]

Changes

• To support the Audit dev toolbar app, Astro's compiler injects data-astro-source-file and data-astro-source-loc attributes on every element in dev mode.
• In this PR, the Audit app has been updated to extract the data from these attributes, store them in a WeakMap that is keyed by each Element, and then remove the attributes.
• This should improve the dev mode experience by simplifying the markup when inspecting the page.
• Changeset added.

Impact

This admittedly might break some third-party dev toolbar apps that rely on these annotations. Ideally we would expose a public API to get the annotation info for an element.

Searching all of GitHub for data-astro-source-file yields a decent number of repos, but the vast majority of uses are workarounds to remove the attributes.

Testing

Tested manually because it involves dev mode-specific client-side script behavior, which would be flaky to run in CI.

Docs

As far as I can tell, this is an internal API that we've never documented!

These attributes were added in #9016 and first released in astro@3.5.0 in November 2023. I assume users have gotten used to ignoring them, but they're still visual noise. Since this approach allows us to reduce noise without removing functionality, I think we should.

[changeset-bot]

🦋 Changeset detected

Latest commit: 99f6bca

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

[pkg-pr-new]

astro

npm i https://pkg.pr.new/astro@13602

@astrojs/cloudflare

npm i https://pkg.pr.new/@astrojs/cloudflare@13602

@astrojs/netlify

npm i https://pkg.pr.new/@astrojs/netlify@13602

@astrojs/node

npm i https://pkg.pr.new/@astrojs/node@13602

@astrojs/vercel

npm i https://pkg.pr.new/@astrojs/vercel@13602

commit: 99f6bca

[codspeed-hq]

CodSpeed Performance Report

Merging #13602 will not alter performance

_{Comparing fix/audit-annotations (99f6bca) with main (a0774b3)}

Summary

✅ 6 untouched benchmarks

[Princesseuh]

Looks good to me, awesome work