fix: keep non-picked route exports that picked exports reference#2205
Open
brenelz wants to merge 2 commits into
Open
fix: keep non-picked route exports that picked exports reference#2205brenelz wants to merge 2 commits into
brenelz wants to merge 2 commits into
Conversation
The route tree-shaker removed the entire binding of any export not in
the pick list, so an API handler calling a helper exported from the
same file crashed at runtime with a ReferenceError. Instead of deleting
the declaration, strip only the export keyword and register the kept
binding with the existing sweep pass, which still removes it (and its
imports) when nothing references it - so no extra code reaches a bundle
that didn't already reach it.
Also fixes an always-true condition in the export-specifier handling
that dropped picked exports declared via specifiers, e.g.
`export { POST as GET }`.
Mixed declarations (`export const a = ..., b = ...`) are split into one
declaration per declarator, preserving evaluation order, and only the
picked ones stay exported. Named default-export declarations are kept
in module scope when other exports reference them; anonymous ones are
still removed.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
🦋 Changeset detectedLatest commit: f70faed The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
✅ Deploy Preview for solid-start-landing-page ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
brenelz
commented
Jul 17, 2026
| // if opts.keep is true, we don't remove the routeData export | ||
| if (state.opts.pick && !state.opts.pick.includes("default")) { | ||
| exportNamedPath.remove(); | ||
| const decl = exportNamedPath.node.declaration; |
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
commit: |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #2100
What is the current behavior?
The route tree-shaker deletes the entire binding of any export not in the
?picklist. An API route likeis imported with
?pick=GET,hello's declaration is removed, and the handler crashes at runtime withReferenceError: hello is not defined(500 on every request). Verified end-to-end on a production build of the basic fixture.There is also an always-true condition in the export-specifier handling that removes picked specifiers, so
export { POST as GET }never survives — the root cause of #1659.What is the new behavior?
Non-picked exports are un-exported instead of deleted: the
exportkeyword is stripped and the binding stays in module scope. Kept bindings are registered with the existing sweep pass, which still removes them (and their imports) when nothing references them — so no code reaches a bundle it didn't already reach, addressing the server-code-leak concern discussed in #2104.Details:
export const a = ..., GET = ...) are split into one declaration per declarator, preserving evaluation order; only picked ones stay exported.export { a, b as c }specifiers are now filtered by their exported name (fixes [Bug?]:export { POST as GET }is now broken #1659).export classis handled likeexport function.Other information
Added
packages/start/src/config/fs-routes/tree-shake.spec.ts(9 tests, 5 of which fail without this fix). Full playwright e2e suite inapps/testspasses. Supersedes the approach in #2104, which un-exported picked declarators in mixed declarations, produced invalid syntax for anonymous default exports, and kept unreferenced non-picked exports in the output.🤖 Generated with Claude Code