[Vite] Name route folder chunks the same as standard route chunks

[AdiRishi]

Hey all, I came accross some odd behavior from the vite compiler and wanted to raise a discussion/issue about it.

To demonstrate what I mean, let's take a simple remix project made with --template remix-run/remix/templates/vite
This project has three routes - _index.tsx, about.tsx and terms.tsx.

When we run remix vite:build we see the following output

vite v5.1.5 building for production...
✓ 84 modules transformed.
build/client/.vite/manifest.json                1.33 kB │ gzip:  0.31 kB
build/client/assets/terms-B-PUXzpg.js           0.25 kB │ gzip:  0.21 kB
build/client/assets/about-DR8A8W8z.js           0.25 kB │ gzip:  0.21 kB
build/client/assets/_index-Bqmz9KMR.js          0.25 kB │ gzip:  0.21 kB
build/client/assets/root-CXCOh6Pt.js            1.44 kB │ gzip:  0.83 kB
build/client/assets/jsx-runtime-BlSqMCxk.js     8.09 kB │ gzip:  3.04 kB
build/client/assets/entry.client-BJ12z_Rq.js   14.04 kB │ gzip:  5.00 kB
build/client/assets/components-t0vHlsCC.js    212.81 kB │ gzip: 68.87 kB
✓ built in 518ms

As you can see, Vite correctly names the route module chunks after the route name.

Now let's see what happens if we use route module folders. Aka if I change them to _index/route.tsx, about/route.tsx and terms/route.tsx.
Again, running remix vite:build we see

vite v5.1.5 building for production...
✓ 84 modules transformed.
build/client/.vite/manifest.json                1.35 kB │ gzip:  0.31 kB
build/client/assets/_index-Bqmz9KMR.js          0.25 kB │ gzip:  0.21 kB
build/client/assets/route-B-PUXzpg.js           0.25 kB │ gzip:  0.21 kB
build/client/assets/route-DR8A8W8z.js           0.25 kB │ gzip:  0.21 kB
build/client/assets/root-CXCOh6Pt.js            1.44 kB │ gzip:  0.83 kB
build/client/assets/jsx-runtime-BlSqMCxk.js     8.09 kB │ gzip:  3.04 kB
build/client/assets/entry.client-BJ12z_Rq.js   14.04 kB │ gzip:  5.00 kB
build/client/assets/components-t0vHlsCC.js    212.81 kB │ gzip: 68.87 kB
✓ built in 505ms

The chunks are named after the file itself (route.tsx) instead of the actual route name. While this may seem minor, I personally think this really hurts.
It's quite convenient to look at the build output and know at a glance how large each route file is.
Right now, it would be very difficult to figure out which route built to which file.

I'm honestly not sure if this is just a bug or an oversight (I don't know enough about Vite). It's interesting that the _index route still maintains its filename correctly while the others become generic.
Looking forward to hearing more 🙏