refactor: added more strict app segment config validation #70480
Conversation
This file contains 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
This was referenced Sep 25, 2024
Tests Passed |
wyattjoh
force-pushed
the
refactor/app-segment-config
branch
from
4858c86
to
6a4a0c0
Compare
wyattjoh
force-pushed
the
refactor/app-segment-config-validation
branch
from
1ebdb45
to
c261a85
Compare
wyattjoh
force-pushed
the
refactor/app-segment-config
branch
from
6a4a0c0
to
685e01a
Compare
wyattjoh
force-pushed
the
refactor/app-segment-config-validation
branch
from
c261a85
to
026cc27
Compare
wyattjoh
force-pushed
the
refactor/app-segment-config
branch
from
685e01a
to
0901fa1
Compare
wyattjoh
force-pushed
the
refactor/app-segment-config-validation
branch
from
026cc27
to
c552009
Compare
wyattjoh
force-pushed
the
refactor/app-segment-config
branch
from
0901fa1
to
ba16a69
Compare
wyattjoh
force-pushed
the
refactor/app-segment-config-validation
branch
from
c552009
to
998c585
Compare
wyattjoh
force-pushed
the
refactor/app-segment-config
branch
from
ba16a69
to
a428fa1
Compare
wyattjoh
force-pushed
the
refactor/app-segment-config-validation
branch
from
998c585
to
1ff36ff
Compare
wyattjoh
force-pushed
the
refactor/app-segment-config
branch
from
a428fa1
to
a06c4b7
Compare
wyattjoh
force-pushed
the
refactor/app-segment-config-validation
branch
from
1ff36ff
to
351bbbc
Compare
wyattjoh
force-pushed
the
refactor/app-segment-config
branch
from
a06c4b7
to
dfa9a4e
Compare
wyattjoh
force-pushed
the
refactor/app-segment-config-validation
branch
from
351bbbc
to
3686097
Compare
wyattjoh
force-pushed
the
refactor/app-segment-config
branch
from
dfa9a4e
to
3789363
Compare
wyattjoh
force-pushed
the
refactor/app-segment-config-validation
branch
from
3686097
to
b8f312e
Compare
wyattjoh
force-pushed
the
refactor/app-segment-config
branch
from
3789363
to
6ceba21
Compare
wyattjoh
force-pushed
the
refactor/app-segment-config-validation
branch
from
b8f312e
to
3edece9
Compare
wyattjoh
force-pushed
the
refactor/app-segment-config
branch
from
6ceba21
to
aa07524
Compare
wyattjoh
force-pushed
the
refactor/app-segment-config-validation
branch
2 times, most recently
from
7a1b186
to
6f319f5
Compare
wyattjoh
force-pushed
the
refactor/app-segment-config
branch
from
aa07524
to
fd8a8dd
Compare
wyattjoh
force-pushed
the
refactor/app-segment-config-validation
branch
from
6f319f5
to
80164aa
Compare
wyattjoh
force-pushed
the
refactor/app-segment-config-validation
branch
2 times, most recently
from
aa1af00
to
a44b164
Compare
wyattjoh
force-pushed
the
refactor/app-segment-config-validation
branch
3 times, most recently
from
d750e5a
to
3d0e18a
Compare
|
New dependencies detected. Learn more about Socket for GitHub ↗︎
|
wyattjoh
force-pushed
the
refactor/app-segment-config-validation
branch
15 times, most recently
from
cd8c4c3
to
6f99389
Compare
ztanner
reviewed
Oct 4, 2024
|
|
||
| if (staticInfo?.type === PAGE_TYPES.PAGES) { | ||
| if ( | ||
| staticInfo.config?.config?.amp === true || |
There was a problem hiding this comment.
how come this is a doubly nested config?
There was a problem hiding this comment.
pages/ supports the following:
export const config = { runtime: "edge" }
ztanner
reviewed
Oct 4, 2024
| params: GetPageStaticInfoParams | ||
| ): Promise<PageStaticInfo> { | ||
| if (params.pageType === PAGE_TYPES.APP) { | ||
| return getAppPageStaticInfo(params) |
packages/next/src/build/entries.ts
Outdated
| @@ -148,45 +150,32 @@ export async function getStaticInfoIncludingLayouts({ | |||
| dir = join(dir, '..') | |||
| } | |||
|
|
|||
| // Reverse the layout files so we can use unshift to add them to the | |||
| // segments array. | |||
| layoutFiles.reverse() | |||
There was a problem hiding this comment.
this part is a little strange to me, could you clarify in the comment why we do this?
ztanner
approved these changes
Oct 4, 2024
wyattjoh
force-pushed
the
refactor/app-segment-config-validation
branch
from
6f99389
to
64eae6e
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This increases the strictness that Next.js uses while parsing configuration from applications as well as removing some legacy options (in preparation for Next.js 15).
export const runtime = "experimental-edge", they will be required to switch toexport const runtime = "edge". They were the same here anyways but this stabilizes it.