Skip to content

Empty NextNodeServer#handleNextImageRequest to avoid pulling unneed…#1098

Merged
vicb merged 4 commits intomainfrom
vicb/empty-handleNextImageRequest
Jan 28, 2026
Merged

Empty NextNodeServer#handleNextImageRequest to avoid pulling unneed…#1098
vicb merged 4 commits intomainfrom
vicb/empty-handleNextImageRequest

Conversation

@vicb
Copy link
Copy Markdown
Contributor

@vicb vicb commented Jan 27, 2026

…ed deps

We will get the bundle size reduction when we update to the Next aws release after 3.9.13

See opennextjs/opennextjs-aws#1098 (released in 3.9.14)

Closes #1094

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Jan 27, 2026

⚠️ No Changeset found

Latest commit: eb2933c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Jan 27, 2026

Open in StackBlitz

npm i https://pkg.pr.new/@opennextjs/cloudflare@1098

commit: eb2933c

},
"include": ["next-env.d.ts", "**/*.ts", "**/*.tsx", ".next/types/**/*.ts"],
"exclude": ["node_modules", "open-next.config.ts"]
"compilerOptions": {
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is Next updating the tsconfig after a pnpm -r build.
Note directly related to the other changes in this PR, but added because it doesn't harm.

@vicb vicb changed the title [BLOCKED on AWS] Empty NextNodeServer#handleNextImageRequest to avoid pulling unneed… Empty NextNodeServer#handleNextImageRequest to avoid pulling unneed… Jan 28, 2026
"paths": {
"@/*": ["./src/*"]
}
},
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@vicb vicb requested review from conico974, dario-piotrowicz, james-elicx and sommeeeer and removed request for dario-piotrowicz January 28, 2026 11:38
@vicb
Copy link
Copy Markdown
Contributor Author

vicb commented Jan 28, 2026

Thanks for the review Magnus!

@vicb vicb merged commit de13b7e into main Jan 28, 2026
7 checks passed
@vicb vicb deleted the vicb/empty-handleNextImageRequest branch January 28, 2026 12:09
vicb added a commit that referenced this pull request Jan 28, 2026
Co-authored-by: James Anderson <james@eli.cx>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Larger build when --turbo used

2 participants