Links to static files should not be handled by Sveltekit

[liegeandlief]

Describe the bug

When linking to a file in the static directory and there is a [...path] route defined this route tries to handle the static link rather than letting the browser handle the link.

I think it would make sense to include static files in the manifest and when a link points to one then Sveltekit should not try to handle the link itself and should let the browser handle it.

Adding data-sveltekit-reload to the link avoids the problem but is a burden on the developer to remember to do it when linking to static files.

Reproduction

https://stackblitz.com/edit/sveltejs-kit-template-default-5zqccl?file=src/routes/+page.svelte

Logs

No response

System Info

System:
    OS: macOS 13.2.1
    CPU: (10) arm64 Apple M1 Pro
    Memory: 240.78 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 18.15.0 - ~/.volta/tools/image/node/18.15.0/bin/node
    Yarn: 3.3.0 - ~/.volta/tools/image/yarn/3.3.0/bin/yarn
    npm: 9.5.0 - ~/.volta/tools/image/npm/9.5.0/bin/npm
  Browsers:
    Chrome: 111.0.5563.64
    Firefox: 110.0.1
    Safari: 16.3
  npmPackages:
    @sveltejs/adapter-vercel: ^2.0.2 => 2.3.1
    @sveltejs/kit: ^1.0.0 => 1.11.0
    svelte: ^3.44.0 => 3.56.0
    vite: ^4.0.0 => 4.1.4

Severity

serious, but I can work around it

Additional Information

No response

[dummdidumm]

This is tricky. To do this, we would need to add the paths of all static assets (which could be a lot) to the route manifest. The alternative would be to do a "test load" behind the scenes, but that would result in needless requests for the vast majority of cases.

[liegeandlief]

Perhaps a sensible compromise could be to do the "test load" only when the path requested ends with a filename e.g. /*.*? This would help to minimise the number of unnecessary test loads.

[artemkovalyov]

I seem to suffer from this when trying to add links to favico and apple-touch-image from svelte:head

Svelte seem to process this links that should be just static content and break the whole app hydration because of JS errors.

[artemkovalyov]

It kinda works if the link is added from app.html instead of using svelte:head