-
Notifications
You must be signed in to change notification settings - Fork 367
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
fix(app-tools): the router-plugin should update source-map #4078
Conversation
🦋 Changeset detectedLatest commit: f27efa9 The changes in this PR will be included in the next version bump. This PR includes changesets to release 208 packages
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 |
8a35242
to
40839b4
Compare
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## main #4078 +/- ##
==========================================
- Coverage 57.59% 57.50% -0.10%
==========================================
Files 672 648 -24
Lines 17794 17654 -140
Branches 3876 3855 -21
==========================================
- Hits 10249 10152 -97
+ Misses 6934 6870 -64
- Partials 611 632 +21 ☔ View full report in Codecov by Sentry. |
Summary
🤖 Generated by Copilot at f881cf6
This pull request fixes a bug in the
RouterPlugin
that prevented the source-map from updating correctly for the router assets. It also updates the@modern-js/app-tools
package with a changeset file that documents the fix.Details
🤖 Generated by Copilot at f881cf6
RawSource
with theSourceMapSource
to handle the assets that have source maps (link, link)PROCESS_ASSETS_STAGE_ADDITIONS
to run the plugin before the assets are emitted (link, link)path
module from theRouterPlugin.ts
file (link)Related Issue
Checklist
pnpm run change
.