Skip to content
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

next.js: add option to use bundled runtime #10485

Merged
merged 22 commits into from
Sep 12, 2023
Merged

Conversation

feedthejim
Copy link
Contributor

@feedthejim feedthejim commented Sep 11, 2023

This PR adds an environment variable that should allow us to test the bundled version for Next.js on Vercel, see vercel/next.js#52997 for reference.

The changes include:

  • a new environment variable VERCEL_NEXT_BUNDLED_SERVER
  • some logic changes that will put app route handlers into their own lambda groups
  • extra logic to require early the rendering runtimes (see PR above for details)

@changeset-bot
Copy link

changeset-bot bot commented Sep 11, 2023

🦋 Changeset detected

Latest commit: 1994698

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@vercel/next Patch
vercel Patch

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

Copy link

@orca-security-us orca-security-us bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca

appRouteHandlers.push(page);
} else {
appRouterPages.push(page);
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change is not behind a flag. Is that intentional?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, we should have app route handlers into their own lambdas!

Copy link
Member

@styfle styfle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Might be good to have @ijjk take a look too

@styfle styfle added area: next pr: automerge Automatically merge the PR when checks pass labels Sep 12, 2023
@kodiakhq kodiakhq bot merged commit 2f461a8 into main Sep 12, 2023
106 checks passed
@kodiakhq kodiakhq bot deleted the feedthejim/add-minimal branch September 12, 2023 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: next pr: automerge Automatically merge the PR when checks pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants