fix possible error with '+' character in the route pathname

.gitignore

@@ -1,4 +1,6 @@
 .DS_Store
 
 node_modules
-dist
+dist
+
+debug*.ts

package.json

@@ -1,6 +1,6 @@
 {
   "name": "route-fn",
-  "version": "1.3.2",
+  "version": "1.3.3",
   "description": "A simple utility function for typesafe urls.",
   "license": "MIT",
   "type": "module",

src/index.ts

@@ -8,6 +8,7 @@ import type {
   RouteParams,
   SearchParams,
 } from "./types"
+import { safeUrl } from "./safe-url"
 
 const fakeOrigin = "http://localhost"
 
@@ -62,9 +63,9 @@ export function createRouteFn<const Routes extends string[]>(routes: Routes) {
 
   fn.params = function (url: string): DynamicParamsDictionary<Routes> {
     const urlWithOrigin = new URL(url, fakeOrigin).href
-    const input = urlWithOrigin.split("?")[0]
+    const input = safeUrl(urlWithOrigin.split("?")[0])
 
-    const patterns = sortedRoutes.map((route) => new URLPattern({ pathname: route }))
+    const patterns = sortedRoutes.map((route) => new URLPattern({ pathname: safeUrl(route) }))
 
     for (const pattern of patterns) {
       const patternResult = pattern.exec(input)
@@ -83,9 +84,9 @@ export function createRouteFn<const Routes extends string[]>(routes: Routes) {
     const matchingRoutes = typeof routeIds === "string" ? [routeIds] : routeIds
     const bestMatch = sortedRoutes.find((route) => {
       const urlWithOrigin = new URL(url, fakeOrigin).href
-      const input = urlWithOrigin.split("?")[0]
+      const input = safeUrl(urlWithOrigin.split("?")[0])
 
-      const pattern = new URLPattern({ pathname: route })
+      const pattern = new URLPattern({ pathname: safeUrl(route) })
 
       if (pattern.test(input)) {
         return true

src/safe-url.ts

@@ -0,0 +1,3 @@
+export function safeUrl(url: string) {
+  return url.replace(/[+]/g, encodeURIComponent("+"))
+}

tests/index.test.ts

@@ -64,6 +64,15 @@ describe("route-fn.params", () => {
       project: "route-fn",
     })
   })
+
+  it("should handle special characters", () => {
+    const route = createRouteFn(["/:org/:id/+"])
+
+    expect(route.params("/apple/iphone/+")).toEqual({
+      org: "apple",
+      id: "iphone",
+    })
+  })
 })
 
 describe("route-fn.test", () => {
@@ -109,4 +118,11 @@ describe("route-fn.test", () => {
     expect(route.test("/apple/settings/billing", "/:org/:project/posts")).toEqual(false)
     expect(route.test("/apple/settings/billing", "/:org/:project/settings")).toEqual(false)
   })
+
+  it("should handle special characters", () => {
+    const route = createRouteFn(["/:org/+"])
+
+    expect(route.test("/apple/new", "/:org/+")).toEqual(false)
+    expect(route.test("/apple/+", "/:org/+")).toEqual(true)
+  })
 })