feat: support clean url and improve route resolve

Mister-Hope · Mister-Hope · commit 8a0c158aee20 · 2024-05-22T15:57:46.000+08:00
diff --git a/.eslintrc.cjs b/.eslintrc.cjs
@@ -1,6 +1,12 @@
 module.exports = {
   root: true,
   extends: 'vuepress',
+
+  // FIXME: This should be added to `eslint-config-vuepress`
+  globals: {
+    __VUEPRESS_CLEAN_URL__: 'readonly',
+  },
+
   overrides: [
     {
       files: ['*.ts', '*.vue', '*.cts'],
diff --git a/packages/bundler-vite/src/plugins/vuepressMainPlugin.ts b/packages/bundler-vite/src/plugins/vuepressMainPlugin.ts
@@ -205,6 +205,7 @@ const resolveDefine = async ({
   const define: UserConfig['define'] = {
     __VUEPRESS_VERSION__: JSON.stringify(app.version),
     __VUEPRESS_BASE__: JSON.stringify(app.options.base),
+    __VUEPRESS_CLEAN_URL__: JSON.stringify(app.options.route.cleanUrl),
     __VUEPRESS_DEV__: JSON.stringify(!isBuild),
     __VUEPRESS_SSR__: JSON.stringify(isServer),
     // @see http://link.vuejs.org/feature-flags
diff --git a/packages/bundler-webpack/src/config/handlePluginDefine.ts b/packages/bundler-webpack/src/config/handlePluginDefine.ts
@@ -21,6 +21,7 @@ export const handlePluginDefine = async ({
     {
       __VUEPRESS_VERSION__: JSON.stringify(app.version),
       __VUEPRESS_BASE__: JSON.stringify(app.options.base),
+      __VUEPRESS_CLEAN_URL__: JSON.stringify(app.options.route.cleanUrl),
       __VUEPRESS_DEV__: JSON.stringify(!isBuild),
       __VUEPRESS_SSR__: JSON.stringify(isServer),
       // @see http://link.vuejs.org/feature-flags
diff --git a/packages/cli/src/commands/dev/watchPageFiles.ts b/packages/cli/src/commands/dev/watchPageFiles.ts
@@ -41,7 +41,7 @@ export const watchPageFiles = (app: App): FSWatcher[] => {
   app.pages.forEach((page) => addDeps(page))
 
   // watch page files
-  const pagesWatcher = chokidar.watch(app.options.pagePatterns, {
+  const pagesWatcher = chokidar.watch(app.options.route.pagePatterns, {
     cwd: app.dir.source(),
     ignoreInitial: true,
   })
diff --git a/packages/client/src/router/index.ts b/packages/client/src/router/index.ts
@@ -3,4 +3,5 @@ export { useRoute, useRouter } from 'vue-router'
 
 export * from './resolveRoute.js'
 export * from './resolveRouteFullPath.js'
+export * from './resolveRouteKey.js'
 export * from './resolveRoutePath.js'
diff --git a/packages/client/src/router/resolveRoute.ts b/packages/client/src/router/resolveRoute.ts
@@ -1,7 +1,7 @@
-import { resolveRoutePathInfo } from '@vuepress/shared'
+import { resolvePathInfo, resolveRoutePathWithExt } from '@vuepress/shared'
 import { routes } from '../internal/routes.js'
 import type { Route, RouteMeta } from '../types/index.js'
-import { resolveRoutePath } from './resolveRoutePath.js'
+import { resolveRouteKey } from './resolveRouteKey.js'
 
 export interface ResolvedRoute<T extends RouteMeta = RouteMeta>
   extends Route<T> {
@@ -17,14 +17,16 @@ export const resolveRoute = <T extends RouteMeta = RouteMeta>(
   currentPath?: string,
 ): ResolvedRoute<T> => {
   // get only the pathname from the path
-  const [pathname, hashAndQueries] = resolveRoutePathInfo(path)
+  const [pathname, hashAndQueries] = resolvePathInfo(path)
 
   // resolve the route path
-  const routePath = resolveRoutePath(pathname, currentPath)
-  const routeFullPath = routePath + hashAndQueries
+  const routeKey = resolveRouteKey(pathname, currentPath)
+  const routeFullPath = __VUEPRESS_CLEAN_URL__
+    ? routeKey
+    : resolveRoutePathWithExt(routeKey) + hashAndQueries
 
   // the route not found
-  if (!routes.value[routePath]) {
+  if (!routes.value[routeKey]) {
     return {
       ...routes.value['/404.html'],
       path: routeFullPath,
@@ -33,7 +35,7 @@ export const resolveRoute = <T extends RouteMeta = RouteMeta>(
   }
 
   return {
-    ...routes.value[routePath],
+    ...routes.value[routeKey],
     path: routeFullPath,
     notFound: false,
   } as ResolvedRoute<T>
diff --git a/packages/client/src/router/resolveRouteFullPath.ts b/packages/client/src/router/resolveRouteFullPath.ts
@@ -1,4 +1,4 @@
-import { resolveRoutePathInfo } from '@vuepress/shared'
+import { resolvePathInfo } from '@vuepress/shared'
 import { resolveRoutePath } from './resolveRoutePath.js'
 
 /**
@@ -8,7 +8,7 @@ export const resolveRouteFullPath = (
   path: string,
   currentPath?: string,
 ): string => {
-  const [pathname, hashAndQueries] = resolveRoutePathInfo(path)
+  const [pathname, hashAndQueries] = resolvePathInfo(path)
 
   return resolveRoutePath(pathname, currentPath) + hashAndQueries
 }
diff --git a/packages/client/src/router/resolveRouteKey.ts b/packages/client/src/router/resolveRouteKey.ts
@@ -0,0 +1,34 @@
+import { normalizeRoutePath } from '@vuepress/shared'
+import { redirects, routes } from '../internal/routes.js'
+
+/**
+ * Resolve route path with given raw path
+ */
+export const resolveRouteKey = (
+  pathname: string,
+  currentPath?: string,
+): string => {
+  // normalized path
+  const routePath = normalizeRoutePath(pathname, currentPath)
+
+  // check if the normalized path is in routes
+  if (routes.value[routePath]) return routePath
+
+  // check encoded path
+  const encodedRoutePath = encodeURI(routePath)
+
+  if (routes.value[encodedRoutePath]) {
+    return encodedRoutePath
+  }
+
+  // check redirected path with normalized path and encoded path
+  const redirectedRoutePath =
+    redirects.value[routePath] || redirects.value[encodedRoutePath]
+
+  if (redirectedRoutePath) {
+    return redirectedRoutePath
+  }
+
+  // default to normalized route path
+  return routePath
+}
diff --git a/packages/client/src/router/resolveRoutePath.ts b/packages/client/src/router/resolveRoutePath.ts
@@ -1,5 +1,5 @@
-import { normalizeRoutePath } from '@vuepress/shared'
-import { redirects, routes } from '../internal/routes.js'
+import { resolveRoutePathWithExt } from '@vuepress/shared'
+import { resolveRouteKey } from './resolveRouteKey.js'
 
 /**
  * Resolve route path with given raw path
@@ -8,27 +8,8 @@ export const resolveRoutePath = (
   pathname: string,
   currentPath?: string,
 ): string => {
-  // normalized path
-  const normalizedRoutePath = normalizeRoutePath(pathname, currentPath)
+  // clean route path format used as key in routes
+  const routeKey = resolveRouteKey(pathname, currentPath)
 
-  // check if the normalized path is in routes
-  if (routes.value[normalizedRoutePath]) return normalizedRoutePath
-
-  // check encoded path
-  const encodedRoutePath = encodeURI(normalizedRoutePath)
-
-  if (routes.value[encodedRoutePath]) {
-    return encodedRoutePath
-  }
-
-  // check redirected path with normalized path and encoded path
-  const redirectedRoutePath =
-    redirects.value[normalizedRoutePath] || redirects.value[encodedRoutePath]
-
-  if (redirectedRoutePath) {
-    return redirectedRoutePath
-  }
-
-  // default to normalized route path
-  return normalizedRoutePath
+  return __VUEPRESS_CLEAN_URL__ ? routeKey : resolveRoutePathWithExt(routeKey)
 }
diff --git a/packages/client/types.d.ts b/packages/client/types.d.ts
@@ -1,6 +1,7 @@
 declare const __VUEPRESS_VERSION__: string
 declare const __VUEPRESS_BASE__: string
 declare const __VUEPRESS_DEV__: boolean
+declare const __VUEPRESS_CLEAN_URL__: boolean
 declare const __VUEPRESS_SSR__: boolean
 declare const __VUE_HMR_RUNTIME__: Record<string, any>
 declare const __VUE_OPTIONS_API__: boolean
diff --git a/packages/core/src/app/resolveAppOptions.ts b/packages/core/src/app/resolveAppOptions.ts
@@ -38,8 +38,16 @@ export const resolveAppOptions = ({
   bundler,
   debug = false,
   markdown = {},
-  pagePatterns = ['**/*.md', '!.vuepress', '!node_modules'],
-  permalinkPattern = null,
+  pagePatterns: _pagePatterns,
+  permalinkPattern: _permalinkPattern,
+  route: {
+    cleanUrl = false,
+    pagePatterns = ['**/*.md', '!.vuepress', '!node_modules'],
+    permalinkPattern = null,
+  } = {
+    pagePatterns: _pagePatterns,
+    permalinkPattern: _permalinkPattern,
+  },
   plugins = [],
   theme,
 }: AppConfig): AppOptions => ({
@@ -65,8 +73,11 @@ export const resolveAppOptions = ({
   bundler,
   debug,
   markdown,
-  pagePatterns,
-  permalinkPattern,
+  route: {
+    cleanUrl,
+    pagePatterns,
+    permalinkPattern,
+  },
   plugins,
   theme,
 })
diff --git a/packages/core/src/app/resolveAppPages.ts b/packages/core/src/app/resolveAppPages.ts
@@ -11,7 +11,7 @@ export const resolveAppPages = async (app: App): Promise<Page[]> => {
   log('resolveAppPages start')
 
   // resolve page absolute file paths according to the page patterns
-  const pageFilePaths = await globby(app.options.pagePatterns, {
+  const pageFilePaths = await globby(app.options.route.pagePatterns, {
     absolute: true,
     cwd: app.dir.source(),
   })
@@ -22,7 +22,7 @@ export const resolveAppPages = async (app: App): Promise<Page[]> => {
   )
 
   // find the 404 page
-  const notFoundPage = pages.find((page) => page.path === '/404.html')
+  const notFoundPage = pages.find((page) => page.path === '/404')
 
   // if there is a 404 page, set the default layout to NotFound
   if (notFoundPage) {
@@ -32,7 +32,7 @@ export const resolveAppPages = async (app: App): Promise<Page[]> => {
   else {
     pages.push(
       await createPage(app, {
-        path: '/404.html',
+        path: '/404',
         frontmatter: { layout: 'NotFound' },
         content: '404 Not Found',
       }),
diff --git a/packages/core/src/page/resolvePagePath.ts b/packages/core/src/page/resolvePagePath.ts
@@ -21,5 +21,15 @@ export const resolvePagePath = ({
     )
   }
 
-  return encodeURI(pagePath.split('/').map(sanitizeFileName).join('/'))
+  return encodeURI(
+    pagePath
+      // clean page path
+      .replace(/\.html$/, '')
+      .replace(/\/index$/i, '/')
+
+      // sanitize page path
+      .split('/')
+      .map(sanitizeFileName)
+      .join('/'),
+  )
 }
diff --git a/packages/core/src/page/resolvePagePermalink.ts b/packages/core/src/page/resolvePagePermalink.ts
@@ -34,7 +34,7 @@ export const resolvePagePermalink = ({
   }
 
   const permalinkPattern =
-    frontmatter.permalinkPattern || app.options.permalinkPattern
+    frontmatter.permalinkPattern || app.options.route.permalinkPattern
 
   if (!isString(permalinkPattern)) {
     return null
diff --git a/packages/core/src/types/app/options.ts b/packages/core/src/types/app/options.ts
@@ -5,6 +5,12 @@ import type { Bundler } from '../bundler.js'
 import type { PluginConfig } from '../plugin.js'
 import type { Theme } from '../theme.js'
 
+export interface RouteOptions {
+  cleanUrl?: boolean
+  pagePatterns?: string[]
+  permalinkPattern?: string | null
+}
+
 /**
  * Vuepress app common config that shared between dev and build
  */
@@ -14,11 +20,9 @@ export interface AppConfigCommon extends Partial<SiteData> {
   temp?: string
   cache?: string
   public?: string
-
   debug?: boolean
   markdown?: MarkdownOptions
-  pagePatterns?: string[]
-  permalinkPattern?: string | null
+  route?: RouteOptions
   bundler: Bundler
   theme: Theme
   plugins?: PluginConfig
@@ -95,9 +99,20 @@ export interface AppConfigBuild {
 /**
  * Vuepress app config
  */
-export type AppConfig = AppConfigCommon & AppConfigDev & AppConfigBuild
+export type AppConfig = AppConfigCommon &
+  AppConfigDev &
+  AppConfigBuild & {
+    /** @deprecated use route.pagePatterns instead */
+    pagePatterns?: string[]
+    /** @deprecated use route.permalinkPattern instead */
+    permalinkPattern?: string | null
+  }
 
 /**
  * Vuepress app options
  */
-export type AppOptions = Required<AppConfig>
+export type AppOptions = Required<
+  AppConfigCommon & AppConfigDev & AppConfigBuild
+> & {
+  route: Required<RouteOptions>
+}
diff --git a/packages/core/tests/app/resolveAppOptions.spec.ts b/packages/core/tests/app/resolveAppOptions.spec.ts
@@ -30,8 +30,11 @@ describe('core > app > resolveAppOptions', () => {
       host: '0.0.0.0',
       port: 8080,
       open: false,
-      pagePatterns: ['**/*.md', '!.vuepress', '!node_modules'],
-      permalinkPattern: null,
+      route: {
+        cleanUrl: false,
+        pagePatterns: ['**/*.md', '!.vuepress', '!node_modules'],
+        permalinkPattern: null,
+      },
       templateDev: path.normalize(
         require.resolve('@vuepress/client/templates/dev.html'),
       ),
diff --git a/packages/core/tests/app/resolveAppPages.spec.ts b/packages/core/tests/app/resolveAppPages.spec.ts
@@ -13,9 +13,9 @@ describe('core > app > resolveAppPages', () => {
     app.markdown = createMarkdown()
 
     const pages = await resolveAppPages(app)
-    const fooPage = pages.find((page) => page.path === '/foo.html')
-    const barPage = pages.find((page) => page.path === '/bar.html')
-    const notFoundPage = pages.find((page) => page.path === '/404.html')
+    const fooPage = pages.find((page) => page.path === '/foo')
+    const barPage = pages.find((page) => page.path === '/bar')
+    const notFoundPage = pages.find((page) => page.path === '/404')
 
     expect(pages).toHaveLength(3)
     expect(fooPage?.filePathRelative).toEqual('foo.md')
@@ -33,9 +33,9 @@ describe('core > app > resolveAppPages', () => {
     app.markdown = createMarkdown()
 
     const pages = await resolveAppPages(app)
-    const fooPage = pages.find((page) => page.path === '/foo.html')
-    const barPage = pages.find((page) => page.path === '/bar.html')
-    const notFoundPage = pages.find((page) => page.path === '/404.html')
+    const fooPage = pages.find((page) => page.path === '/foo')
+    const barPage = pages.find((page) => page.path === '/bar')
+    const notFoundPage = pages.find((page) => page.path === '/404')
 
     expect(pages).toHaveLength(3)
     expect(fooPage?.filePathRelative).toEqual('foo.md')
diff --git a/packages/core/tests/page/resolvePagePath.spec.ts b/packages/core/tests/page/resolvePagePath.spec.ts
@@ -40,7 +40,7 @@ const testCases: [
         },
       },
     ],
-    '/options.html',
+    '/options',
   ],
   // use permalink
   [
@@ -92,7 +92,7 @@ const testCases: [
         options: {},
       },
     ],
-    '/inferred.html',
+    '/inferred',
   ],
 ]
 
diff --git a/packages/markdown/src/plugins/linksPlugin/linksPlugin.ts b/packages/markdown/src/plugins/linksPlugin/linksPlugin.ts
diff --git a/packages/shared/src/utils/routes/index.ts b/packages/shared/src/utils/routes/index.ts
diff --git a/packages/shared/src/utils/routes/inferRoutePath.ts b/packages/shared/src/utils/routes/inferRoutePath.ts
diff --git a/packages/shared/src/utils/routes/resolvePathInfo.ts b/packages/shared/src/utils/routes/resolvePathInfo.ts
diff --git a/packages/shared/src/utils/routes/resolveRoutePathWithExt.ts b/packages/shared/src/utils/routes/resolveRoutePathWithExt.ts
diff --git a/packages/shared/tests/routes/inferRoutePath.spec.ts b/packages/shared/tests/routes/inferRoutePath.spec.ts
diff --git a/packages/shared/tests/routes/normalizeRoutePath.spec.ts b/packages/shared/tests/routes/normalizeRoutePath.spec.ts
diff --git a/packages/shared/tests/routes/resolvePathInfo.spec.ts b/packages/shared/tests/routes/resolvePathInfo.spec.ts

Original file line number	Diff line number	Diff line change
`@@ -21,6 +21,7 @@ export const handlePluginDefine = async ({`
`21`	`21`	`{`
`22`	`22`	`__VUEPRESS_VERSION__: JSON.stringify(app.version),`
`23`	`23`	`__VUEPRESS_BASE__: JSON.stringify(app.options.base),`
	`24`	`+ __VUEPRESS_CLEAN_URL__: JSON.stringify(app.options.route.cleanUrl),`
`24`	`25`	`__VUEPRESS_DEV__: JSON.stringify(!isBuild),`
`25`	`26`	`__VUEPRESS_SSR__: JSON.stringify(isServer),`
`26`	`27`	`// @see http://link.vuejs.org/feature-flags`