Skip to content

Commit 8c8f587

Browse files
authored
feat: add ssrFixStacktrace / ssrRewriteStacktrace to removeSsrLoadModule future deprecation (#20435)
1 parent 8ad388a commit 8c8f587

2 files changed

Lines changed: 12 additions & 0 deletions

File tree

docs/changes/ssr-using-modulerunner.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,3 +19,5 @@ The `server.ssrLoadModule(url)` only allows importing modules in the `ssr` e
1919
## Migration Guide
2020

2121
Check out the [Environment API for Frameworks Guide](../guide/api-environment-frameworks.md).
22+
23+
`server.ssrFixStacktrace` and `server.ssrRewriteStacktrace` does not have to be called when using the Module Runner APIs. The stack traces will be updated unless `sourcemapInterceptor` is set to `false`.

packages/vite/src/node/server/index.ts

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -609,9 +609,19 @@ export async function _createServer(
609609
return ssrLoadModule(url, server, opts?.fixStacktrace)
610610
},
611611
ssrFixStacktrace(e) {
612+
warnFutureDeprecation(
613+
config,
614+
'removeSsrLoadModule',
615+
"ssrFixStacktrace doesn't need to be used for Environment Module Runners.",
616+
)
612617
ssrFixStacktrace(e, server.environments.ssr.moduleGraph)
613618
},
614619
ssrRewriteStacktrace(stack: string) {
620+
warnFutureDeprecation(
621+
config,
622+
'removeSsrLoadModule',
623+
"ssrRewriteStacktrace doesn't need to be used for Environment Module Runners.",
624+
)
615625
return ssrRewriteStacktrace(stack, server.environments.ssr.moduleGraph)
616626
},
617627
async reloadModule(module) {

0 commit comments

Comments
 (0)