fix: more logical sorting of urls

src/runtime/sitemap/entries/normalise.ts

@@ -213,13 +213,10 @@ export async function normaliseSitemapData(data: SitemapEntryInput[], options: B
 
   function normaliseEntries(entries: SitemapEntry[]) {
     return mergeOnKey(entries.map(normaliseEntry), 'loc')
-      // sort based on logical string sorting of the loc
+      // sort based on logical string sorting of the loc, we need to properly account for numbers here
+      // so that urls: /route/1, /route/2 is displayed instead of /route/1, /route/10
       .sort((a, b) => {
-        if (a.loc > b.loc)
-          return 1
-        if (a.loc < b.loc)
-          return -1
-        return 0
+        return a.loc.localeCompare(b.loc, undefined, { numeric: true })
       })
       .sort((a, b) => {
         // we need to sort based on the path segments as well

test/unit/normalise.test.ts

@@ -1,6 +1,6 @@
 import { describe, expect, it } from 'vitest'
 import { fixSlashes } from 'site-config-stack'
-import { normaliseSitemapData } from '../../src/runtime/sitemap/entries'
+import {normaliseEntries, normaliseSitemapData} from '../../src/runtime/sitemap/entries'
 import type { BuildSitemapInput } from '../../src/runtime/types'
 
 const normaliseOptions: BuildSitemapInput = {
@@ -38,4 +38,36 @@ describe('normalise', () => {
       ]
     `)
   })
+  it('sorting', async () => {
+    const data = await normaliseSitemapData([
+      { loc: '/a' },
+      { loc: '/b' },
+      { loc: '/c' },
+      { loc: '/1' },
+      { loc: '/2' },
+      { loc: '/10' },
+    ], normaliseOptions)
+    expect(data).toMatchInlineSnapshot(`
+      [
+        {
+          "loc": "/1/",
+        },
+        {
+          "loc": "/2/",
+        },
+        {
+          "loc": "/10/",
+        },
+        {
+          "loc": "/a/",
+        },
+        {
+          "loc": "/b/",
+        },
+        {
+          "loc": "/c/",
+        },
+      ]
+    `)
+  })
 })