Optimize dependencies found through route module graphs

[pcattori]

Fixes #8917 #9798
Might fix #8803 #8876

See changeset for description

[changeset-bot]

🦋 Changeset detected

Latest commit: 10f0715

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

This PR includes changesets to release 16 packages

Name	Type
@remix-run/dev	Patch
create-remix	Patch
remix	Patch
@remix-run/architect	Patch
@remix-run/cloudflare	Patch
@remix-run/cloudflare-pages	Patch
@remix-run/cloudflare-workers	Patch
@remix-run/css-bundle	Patch
@remix-run/deno	Patch
@remix-run/eslint-config	Patch
@remix-run/express	Patch
@remix-run/node	Patch
@remix-run/react	Patch
@remix-run/serve	Patch
@remix-run/server-runtime	Patch
@remix-run/testing	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

[pcattori]

Unrelated to this PR, our CI integration suite was failing due to outdated wrangler version, so batched that fix in here.

[markdalgleish]

Hmmm, why is this not on the type? 🤔

[markdalgleish]

Looking at the docs for optimizeDeps.entries:

If build.rollupOptions.input is specified, Vite will crawl those entry points instead

I wonder if the root cause is that we're only applying the Rollup build options when the build command is running:

remix/packages/remix-dev/vite/plugin.ts

Lines 1126 to 1134 in 8f38118

	...(viteCommand === "build"
	? {
	build: {
	cssMinify: viteUserConfig.build?.cssMinify ?? true,
	...(!viteConfigEnv.isSsrBuild
	? {
	manifest: true,
	outDir: getClientBuildDirectory(ctx.remixConfig),
	rollupOptions: {

[github-actions]

🤖 Hello there,

We just published version 2.12.0-pre.0 which includes this pull request. If you'd like to take it for a test run please try it out and let us know what you think!

Thanks!

[github-actions]

🤖 Hello there,

We just published version 2.12.0 which includes this pull request. If you'd like to take it for a test run please try it out and let us know what you think!

Thanks!