@octanejs/remix-router 0.1.0 → 0.1.2

package/src/lib/href.ts

@@ -1,4 +1,4 @@
-// Vendored from react-router@7.18.1 packages/react-router/lib/href.ts — unmodified.
+// Vendored from react-router@8.2.0 packages/react-router/lib/href.ts — unmodified.
 // Re-vendor with `node scripts/vendor-remix-router.mjs`; never hand-edit.
 import type { Pages } from './types/register';
 import type { Equal } from './types/utils';
@@ -14,6 +14,10 @@ type ToArgs<Params extends Record<string, string | undefined>> =
   // otherwise, require `params` arg
   [Params];
 
+function stringify(p: any) {
+	return p == null ? '' : typeof p === 'string' ? p : String(p);
+}
+
 /**
   Returns a resolved URL path for the specified route.
 
@@ -35,16 +39,17 @@ export function href<Path extends keyof Args>(path: Path, ...args: Args[Path]):
 				if (isRequired && value === undefined) {
 					throw new Error(`Path '${path}' requires param '${param}' but it was not provided`);
 				}
-				return value === undefined ? '' : '/' + value;
+				return value == null ? '' : '/' + encodeURIComponent(stringify(value));
 			},
 		);
 
 	if (path.endsWith('*')) {
 		// treat trailing splat the same way as compilePath, and force it to be as if it were `/*`.
-		// `react-router typegen` will not generate the params for a malformed splat, causing a type error, but we can still do the correct thing here.
+		// `react-router typegen` will not generate the params for a malformed splat,
+		// causing a type error, but we can still do the correct thing here.
 		const value = params?.['*'];
 		if (value !== undefined) {
-			result += '/' + value;
+			result += '/' + stringify(value).split('/').map(encodeURIComponent).join('/');
 		}
 	}
 

package/src/lib/router/history.ts

@@ -1,4 +1,4 @@
-// Vendored from react-router@7.18.1 packages/react-router/lib/router/history.ts — unmodified.
+// Vendored from react-router@8.2.0 packages/react-router/lib/router/history.ts — unmodified.
 // Re-vendor with `node scripts/vendor-remix-router.mjs`; never hand-edit.
 import { PROTOCOL_RELATIVE_URL_REGEX } from './url';
 
@@ -53,7 +53,7 @@ export interface Path {
 	hash: string;
 }
 
-// TODO: (v7) Change the Location generic default from `any` to `unknown` and
+// TODO: (v9) Change the Location generic default from `any` to `unknown` and
 // remove Remix `useLocation` wrapper.
 
 /**