fix: avoid stripping query params from entries

Fixes #92

src/runtime/sitemap/entries/normalise.ts

@@ -1,11 +1,12 @@
-import { hasProtocol, joinURL, parseURL } from 'ufo'
+import { hasProtocol, joinURL } from 'ufo'
 import { defu } from 'defu'
+import { fixSlashes } from 'site-config-stack'
 import type {
   BuildSitemapIndexInput,
   BuildSitemapInput,
   ResolvedSitemapEntry,
-  SitemapEntryInput,
   SitemapEntry,
+  SitemapEntryInput,
   SitemapRenderCtx,
 } from '../../types'
 import { createFilter } from '../../util/urlFilter'
@@ -54,7 +55,7 @@ export async function normaliseSitemapData(data: SitemapEntryInput[], options: B
         delete e.url
       }
       // we want a uniform loc so we can dedupe using it, remove slashes and only get the path
-      e.loc = parseURL(e.loc).pathname
+      e.loc = fixSlashes(false, e.loc)
       e = defu(e, defaultEntryData)
       return e
     })

test/queryRoutes.test.ts

@@ -0,0 +1,32 @@
+import { describe, expect, it } from 'vitest'
+import { createResolver } from '@nuxt/kit'
+import { $fetch, setup } from '@nuxt/test-utils'
+
+const { resolve } = createResolver(import.meta.url)
+
+await setup({
+  rootDir: resolve('../.playground'),
+  build: true,
+  server: true,
+  nuxtConfig: {
+    sitemap: {
+      urls: [
+        '/',
+        '/query-no-slash?foo=bar',
+        '/query-slash/?foo=bar',
+        '/query-slash-hash/?foo=bar#hash',
+      ],
+      autoLastmod: false,
+      sitemaps: false,
+    },
+  },
+})
+describe('query routes', () => {
+  it('basic', async () => {
+    const sitemap = await $fetch('/sitemap.xml')
+
+    expect(sitemap).toContain('<loc>https://nuxtseo.com/query-no-slash?foo=bar</loc>')
+    expect(sitemap).toContain('<loc>https://nuxtseo.com/query-slash?foo=bar</loc>')
+    expect(sitemap).toContain('<loc>https://nuxtseo.com/query-slash-hash?foo=bar#hash</loc>')
+  }, 60000)
+})

test/unit/normalise.test.ts

@@ -0,0 +1,41 @@
+import { describe, expect, it } from 'vitest'
+import { fixSlashes } from 'site-config-stack'
+import { normaliseSitemapData } from '../../src/runtime/sitemap/entries'
+import type { BuildSitemapInput } from '../../src/runtime/types'
+
+const normaliseOptions: BuildSitemapInput = {
+  // @ts-expect-error test hack
+  moduleConfig: {},
+  // @ts-expect-error test hack
+  buildTimeMeta: {},
+  getRouteRulesForPath: () => ({}),
+  canonicalUrlResolver: (path: string) => fixSlashes(true, path),
+  nitroUrlResolver: (path: string) => path,
+  relativeBaseUrlResolver: (path: string) => path,
+  pages: [],
+  urls: [],
+}
+describe('normalise', () => {
+  it('query', async () => {
+    const normalisedWithoutSlash = await normaliseSitemapData([
+      { loc: '/query?foo=bar' },
+    ], normaliseOptions)
+    expect(normalisedWithoutSlash).toMatchInlineSnapshot(`
+      [
+        {
+          "loc": "/query/?foo=bar",
+        },
+      ]
+    `)
+    const normalisedWithSlash = await normaliseSitemapData([
+      { loc: '/query/?foo=bar' },
+    ], normaliseOptions)
+    expect(normalisedWithSlash).toMatchInlineSnapshot(`
+      [
+        {
+          "loc": "/query/?foo=bar",
+        },
+      ]
+    `)
+  })
+})