chore: remove locale sort TODO comments

bartlomieju · claude · bartlomieju · commit 822059166006 · 2026-04-21T18:52:57.000+02:00
Plain `&lt;` comparison is correct for route paths (ASCII) and preferable
to localeCompare which would reintroduce platform-dependent ordering.

Co-Authored-By: Claude Opus 4.6 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/packages/fresh/src/fs_routes.ts b/packages/fresh/src/fs_routes.ts
@@ -246,7 +246,6 @@ export function sortRoutePaths(a: string, b: string) {
       const scoreB = getRoutePathScore(charB, b, bIdx);
       if (scoreA === scoreB) {
         if (charA !== charB) {
-          // TODO: Do we need localeSort here or is this good enough?
           return charA < charB ? -1 : 1;
         }
         continue;
@@ -256,7 +255,6 @@ export function sortRoutePaths(a: string, b: string) {
     }
 
     if (charA !== charB) {
-      // TODO: Do we need localeSort here or is this good enough?
       return charA < charB ? -1 : 1;
     }
 

Original file line number	Diff line number	Diff line change
`@@ -246,7 +246,6 @@ export function sortRoutePaths(a: string, b: string) {`
`246`	`246`	`const scoreB = getRoutePathScore(charB, b, bIdx);`
`247`	`247`	`if (scoreA === scoreB) {`
`248`	`248`	`if (charA !== charB) {`
`249`		`- // TODO: Do we need localeSort here or is this good enough?`
`250`	`249`	`return charA < charB ? -1 : 1;`
`251`	`250`	`}`
`252`	`251`	`continue;`
`@@ -256,7 +255,6 @@ export function sortRoutePaths(a: string, b: string) {`
`256`	`255`	`}`
`257`	`256`
`258`	`257`	`if (charA !== charB) {`
`259`		`- // TODO: Do we need localeSort here or is this good enough?`
`260`	`258`	`return charA < charB ? -1 : 1;`
`261`	`259`	`}`
`262`	`260`