fix(5457): normalize path on prefetch

[GauBen]

Fixes #5457

[changeset-bot]

🦋 Changeset detected

Latest commit: 29a7196

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@sveltejs/kit	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[GauBen]

I have no idea what's happening but this is fun to watch.

As a sidenote, I was thinking about normalizing before matching:

const path = decodeURI(url.pathname.slice(base.length) || '/');
const normalized_path = normalize_path(url.pathname, trailing_slash);

for (const route of routes) {
	  const params = route.exec(normalized_path);
	  
	  if (params) {
		  const id = normalized_path + url.search;
		  /** @type {import('./types').NavigationIntent} */
		  const intent = { id, route, params, url };
		  return intent;
	  }
}

It may solve current and future issues, or may not, I have no idea.

[Rich-Harris]

Thanks!

I was thinking about normalizing before matching:

I don't think there's any benefit to doing so — it's the same amount of work in either case