fix(shared): improve normalizeRoutePath edge cases handling

Author: meteorlxy

packages/shared/src/utils/normalizeRoutePath.ts

@@ -2,15 +2,27 @@
  * Normalize the given path to the final route path
  */
 export const normalizeRoutePath = (path: string): string => {
-  const convertedMdPath = path.endsWith('README.md')
-    ? path.substring(0, path.length - 9)
-    : path.endsWith('.md')
-      ? path.substring(0, path.length - 3) + '.html'
-      : path
+  if (!path || path.endsWith('/')) {
+    return path
+  }
 
-  return convertedMdPath.endsWith('/index.html')
-    ? convertedMdPath.substring(0, convertedMdPath.length - 10)
-    : convertedMdPath.endsWith('.html') || convertedMdPath.endsWith('/')
-      ? convertedMdPath
-      : convertedMdPath + '.html'
+  // convert README.md to index.html
+  let routePath = path.replace(/(^|\/)README.md$/i, '$1index.html')
+
+  // convert /foo/bar.md to /foo/bar.html
+  if (routePath.endsWith('.md')) {
+    routePath = routePath.substring(0, routePath.length - 3) + '.html'
+  }
+
+  // convert /foo/bar to /foo/bar.html
+  if (!routePath.endsWith('.html')) {
+    routePath = routePath + '.html'
+  }
+
+  // convert /foo/index.html to /foo/
+  if (routePath.endsWith('/index.html')) {
+    return routePath.substring(0, routePath.length - 10)
+  }
+
+  return routePath
 }

packages/shared/tests/normalizeRoutePath.spec.ts

@@ -2,23 +2,51 @@ import { expect, it } from 'vitest'
 import { normalizeRoutePath } from '../src/index.js'
 
 const testCases = [
+  // index
   ['/', '/'],
   ['/README.md', '/'],
+  ['/readme.md', '/'],
   ['/index.md', '/'],
   ['/index.html', '/'],
-  ['/foo', '/foo.html'],
-  ['/foo.md', '/foo.html'],
+  ['/index', '/'],
   ['/foo/', '/foo/'],
   ['/foo/README.md', '/foo/'],
+  ['/foo/readme.md', '/foo/'],
   ['/foo/index.md', '/foo/'],
   ['/foo/index.html', '/foo/'],
+  ['/foo/index', '/foo/'],
+  ['', ''],
+  ['README.md', 'index.html'],
+  ['readme.md', 'index.html'],
+  ['index.md', 'index.html'],
+  ['index.html', 'index.html'],
+  ['index', 'index.html'],
+  ['foo/', 'foo/'],
+  ['foo/README.md', 'foo/'],
+  ['foo/readme.md', 'foo/'],
+  ['foo/index.md', 'foo/'],
+  ['foo/index.html', 'foo/'],
+  ['foo/index', 'foo/'],
+
+  // non-index
+  ['/foo', '/foo.html'],
+  ['/foo.md', '/foo.html'],
+  ['/foo.html', '/foo.html'],
   ['/foo/bar', '/foo/bar.html'],
-  ['/foo/bar/', '/foo/bar/'],
-  ['/foo/bar/README.md', '/foo/bar/'],
-  ['/foo/bar/index.md', '/foo/bar/'],
-  ['/foo/bar/index.html', '/foo/bar/'],
   ['/foo/bar.md', '/foo/bar.html'],
   ['/foo/bar.html', '/foo/bar.html'],
+  ['foo', 'foo.html'],
+  ['foo.md', 'foo.html'],
+  ['foo.html', 'foo.html'],
+  ['foo/bar', 'foo/bar.html'],
+  ['foo/bar.md', 'foo/bar.html'],
+  ['foo/bar.html', 'foo/bar.html'],
+
+  // unexpected corner cases
+  ['.md', '.html'],
+  ['foo/.md', 'foo/.html'],
+  ['/.md', '/.html'],
+  ['/foo/.md', '/foo/.html'],
 ]
 
 testCases.forEach(([path, expected]) =>