href types normalize route full path

Author: pcattori

integration/typegen-test.ts

@@ -401,6 +401,7 @@ test.describe("typegen", () => {
           route("no-params", "routes/no-params.tsx"),
           route("required-param/:req", "routes/required-param.tsx"),
           route("optional-param/:opt?", "routes/optional-param.tsx"),
+          route("/leading-and-trailing-slash/", "routes/leading-and-trailing-slash.tsx"),
           route("some-other-route", "routes/some-other-route.tsx"),
         ] satisfies RouteConfig;
       `,
@@ -413,6 +414,9 @@ test.describe("typegen", () => {
       "app/routes/optional-param.tsx": tsx`
         export default function Component() {}
       `,
+      "app/routes/leading-and-trailing-slash.tsx": tsx`
+        export default function Component() {}
+      `,
       "app/routes/some-other-route.tsx": tsx`
         import { href } from "react-router"
 
@@ -428,6 +432,10 @@ test.describe("typegen", () => {
         href("/optional-param/:opt?")
         href("/optional-param/:opt?", { opt: "hello" })
 
+        href("/leading-and-trailing-slash")
+        // @ts-expect-error
+        href("/leading-and-trailing-slash/")
+
         export default function Component() {}
       `,
     });

packages/react-router-dev/typegen/route.ts

@@ -16,8 +16,11 @@ export function lineage(
 
 export function fullpath(lineage: RouteManifestEntry[]) {
   if (lineage.length === 1 && lineage[0].id === "root") return "/";
-  return lineage
-    .map((route) => route.path)
-    .filter((path) => path !== undefined)
-    .join("/");
+  return (
+    "/" +
+    lineage
+      .map((route) => route.path?.replace(/^\//, "")?.replace(/\/$/, ""))
+      .filter((path) => path !== undefined && path !== "")
+      .join("/")
+  );
 }