fix(runtime-utils): the normalizePathname should handles multiple sla…

…shes correctly (#6441)

.changeset/weak-singers-destroy.md

@@ -0,0 +1,6 @@
+---
+'@modern-js/runtime-utils': patch
+---
+
+fix(runtime-utils): the normalizePathname should handles multiple slashes
+fix(runtime-utils): normalizePathname 函数应该处理好多个 / 的情况

packages/toolkit/runtime-utils/src/url.ts

@@ -8,10 +8,13 @@
  *
  * pathname3: '/', the nomalizeResult also as '/'
  */
-export function normalizePathname(pathname: string) {
-  if (pathname === '/') {
-    return pathname;
-  } else {
-    return pathname.replace(/\/+$/, '');
+export function normalizePathname(pathname: string): string {
+  // biome-ignore lint/style/useTemplate: <explanation>
+  const normalized = '/' + pathname.replace(/^\/+|\/+$/g, '');
+
+  if (normalized === '/') {
+    return normalized;
   }
+
+  return normalized;
 }

packages/toolkit/runtime-utils/tests/url.test.ts

@@ -17,5 +17,11 @@ describe('test ./src/url.ts', () => {
 
     pathname = '/a/b/c/d/';
     expect(normalizePathname(pathname)).toBe('/a/b/c/d');
+
+    pathname = '//';
+    expect(normalizePathname(pathname)).toBe('/');
+
+    pathname = '/api//';
+    expect(normalizePathname(pathname)).toBe('/api');
   });
 });

tests/package.json

@@ -6,7 +6,7 @@
   "main": "index.js",
   "scripts": {
     "test:rspack": "pnpm test:builder:rspack && pnpm test:framework && pnpm test:garfish:rspack",
-    "test:framework": "MODERN_SERVER_LOG_LEVEL=info jest",
+    "test:framework": "cross-env MODERN_SERVER_LOG_LEVEL=info jest",
     "test:builder:rspack": "cd e2e/builder && pnpm test:rspack",
     "test:garfish:rspack": "cd e2e/garfish && pnpm test:rspack",
     "test:module-tools": "cd integration && jest --testMatch **/module/**/*.test.ts",