fix: escape vite asset import suffixes

[barakcodes]

Add the regex to exclude all vite asset import suffixes, might be better done in the @hono/vite-dev-server(maybe), but this fixes the issue for now

Rel #44

[john-griffin]

Thanks, I checked this out locally and I still see the 404 in the example you updated.

[barakcodes]

Thanks, I checked this out locally and I still see the 404 in the example you updated.

Hmm, it works for me. I look more into it

[barakcodes]

Thanks, I checked this out locally and I still see the 404 in the example you updated.

@john-griffin could you give it another try?

[john-griffin]

Still a 404 with that pattern. I'm running via the npm run dev command if that makes any difference, not npm run start-with-adapter

[barakcodes]

Still a 404 with that pattern. I'm running via the npm run dev command if that makes any difference, not npm run start-with-adapter

shoot! I'll take a deeper look at this over the weekend

[barakcodes]

So

Still a 404 with that pattern. I'm running via the npm run dev command if that makes any difference, not npm run start-with-adapter

turns out I was testing with ?inline which did not have?import prefix in some cases for some reason. this should now work as it matches both ?inline and ?import&inline and similar combinations

[john-griffin]

Thanks, that pattern is now working, imports are being returned with a 200 👍

[yusukebe]

Hey @barakcodes

Thank you for fixing. I've added the test for this change.

[yusukebe]

LGTM!

[yusukebe]

Looks good! Let's land it.