avoid Rollup double bundling for pre-bundled static assets for server routes

[thescientist13]

Type of Change

• New Feature Request
• Documentation / Website
• Improvement / Suggestion
• Bug
• Other (please clarify below)

Summary

The current implementation for server rendering routes requires that we run the final HTML on the fly through Rollup at minimum because bundled JS / CSS files will get content hashed, which means that the paths in the templates, e.g.

<script type="module" src="/components/header.js

Get properly turned into something like

<script type="module" src="/components/header.sdf234.js

Details

At its simplest, we could easily add a check to Rollup to check for the existence of a bundled file first and thus avoid the whole emitFile pass, and just rely on HTML updating logic.

The main reason a heavy handed approach was taken because in theory users could additional script tags on the fly in their getTemplate function that may never get statically pre-bundled, so this makes sure that all cases are covered, but perhaps this may not be needed? But maybe there's an even better way to do this?