fix: deterministic param parser types order

posva · posva · commit bf0fc9b60552 · 2026-05-20T11:10:09.000+02:00
diff --git a/packages/router/src/unplugin/codegen/generateParamParsers.spec.ts b/packages/router/src/unplugin/codegen/generateParamParsers.spec.ts
@@ -230,8 +230,8 @@ describe('generateParamParsersTypesDeclarations', () => {
 
     const result = generateParamParsersTypesDeclarations(paramParsers)
     expect(result).toMatchInlineSnapshot(`
-      "type Param_uuid = _ExtractParamParserType<typeof import('./parsers/uuid').parser>
-      type Param_slug = _ExtractParamParserType<typeof import('./parsers/slug').parser>"
+      "type Param_slug = _ExtractParamParserType<typeof import('./parsers/slug').parser>
+      type Param_uuid = _ExtractParamParserType<typeof import('./parsers/uuid').parser>"
     `)
   })
 })
@@ -667,8 +667,8 @@ describe('generateParamParserCustomType', () => {
 
     expect(generateParamParsersTypesDeclarations(paramParsers))
       .toMatchInlineSnapshot(`
-        "type Param_userId = _ExtractParamParserType<typeof import('./parsers/user-id').parser>
-        type Param_dateWithDashes = _ExtractParamParserType<typeof import('./parsers/date-with-dashes').parser>"
+        "type Param_dateWithDashes = _ExtractParamParserType<typeof import('./parsers/date-with-dashes').parser>
+        type Param_userId = _ExtractParamParserType<typeof import('./parsers/user-id').parser>"
       `)
 
     const param: TreePathParam = {
diff --git a/packages/router/src/unplugin/codegen/generateParamParsers.ts b/packages/router/src/unplugin/codegen/generateParamParsers.ts
@@ -90,14 +90,18 @@ export function collectMissingParamParsers(
 export function generateParamParsersTypesDeclarations(
   paramParsers: ParamParsersMap
 ) {
-  return Array.from(paramParsers.values())
-    .map(({ typeName, relativePath }) => {
-      const importPath = relativePath.startsWith('.')
-        ? relativePath
-        : './' + relativePath
-      return `type ${typeName} = _ExtractParamParserType<typeof import('${importPath}').parser>`
-    })
-    .join('\n')
+  return (
+    Array.from(paramParsers.values())
+      .map(({ typeName, relativePath }) => {
+        const importPath = relativePath.startsWith('.')
+          ? relativePath
+          : './' + relativePath
+        return `type ${typeName} = _ExtractParamParserType<typeof import('${importPath}').parser>`
+      })
+      // ensure deterministic order for testing, readability and git
+      .sort()
+      .join('\n')
+  )
 }
 
 export function generateParamsTypes(
