fix(sort_routes.ts): optional route parameters sorting (#1077)

fixes #1016

Author: robisim74

packages/qwik-city/buildtime/routing/sort-routes.ts

@@ -9,10 +9,10 @@ export function routeSortCompare(a: BuildRoute, b: BuildRoute) {
 
     // /x < /x/y, but /[...x]/y < /[...x]
     if (!sa) {
-      return a.id.includes('[...') ? 1 : -1;
+      return a.pathname.includes('[...') ? 1 : -1;
     }
     if (!sb) {
-      return b.id.includes('[...') ? -1 : 1;
+      return b.pathname.includes('[...') ? -1 : 1;
     }
 
     const maxParts = Math.max(sa.length, sb.length);

packages/qwik-city/buildtime/routing/sort-routes.unit.ts

@@ -1,6 +1,7 @@
 import { test } from 'uvu';
 import { equal } from 'uvu/assert';
 import type { BuildRoute } from '../types';
+import { createFileId } from '../utils/fs';
 import { parseRoutePathname } from './parse-pathname';
 import { routeSortCompare } from './sort-routes';
 
@@ -45,7 +46,7 @@ test('routeSortCompare', () => {
 function route(r: TestRoute) {
   const pathname = r.pathname || '/';
   const route: BuildRoute = {
-    id: pathname,
+    id: createFileId('', pathname),
     filePath: pathname,
     pathname,
     ext: '.tsx',