Vite: Work around crash when .svg file contains # or ?

[philipp-spiess]

Closes #16877

This PR works around #16877 by not registering .svg files containing a # or ? as a watch dependency for now.

Test plan

• Add a file to the Vite playground called src/c#.svg
• Observe Vite no longer prints errors

[RobinMalfait]

That's unfortunate that Vite crashes here; Does that mean that if you have these SVGs in your project that changes are never reflected and you have to restart your build for them to take effect?

Because I think the only reason Vite is crashing due to our plugin is just because we mark it with addWatchFile?

[philipp-spiess]

Does that mean that if you have these SVGs in your project that changes are never reflected and you have to restart your build for them to take effect?

Correct if you have .svg with # in the name right now, Vite will print a warning and not pick up any changes. This change makes it so that Vite wont print a warning and won't error in some other setups but also does not pick up any changes. We can revert this once the upstream issue is fixed: vitejs/vite#19563