refactor: duplicate teh shared code for better dx

package.json

@@ -27,13 +27,12 @@
   },
   "homepage": "https://github.com/vuejs/vitepress/tree/master/#readme",
   "scripts": {
-    "dev": "run-p dev-client dev-client-copy dev-node dev-shared",
+    "dev": "run-p dev-client dev-client-copy dev-node",
     "dev-client": "tsc -w -p src/client",
     "dev-client-copy": "node scripts/watchAndCopy",
     "dev-node": "tsc -w -p src/node",
-    "dev-shared": "tsc -w -p src/shared",
     "release": "bash scripts/release.sh",
-    "build": "rimraf -rf dist && tsc -p src/client && tsc -p src/node && tsc -p src/shared && node scripts/copy",
+    "build": "rimraf -rf dist && tsc -p src/client && tsc -p src/node && node scripts/copy",
     "changelog": "conventional-changelog -p angular -i CHANGELOG.md -s"
   },
   "engines": {

src/client/app/composables/siteDataByRoute.ts

@@ -1,10 +1,75 @@
 import { computed } from 'vue'
-import { resolveSiteDataByRoute } from '/@shared/config'
-import { siteDataRef } from './siteData'
+import { SiteData } from '../../../../types/shared'
 import { useRoute } from '../router'
+import { siteDataRef } from './siteData'
+
+const inBrowser = typeof window !== 'undefined'
 
 export function useSiteDataByRoute(route = useRoute()) {
   return computed(() => {
     return resolveSiteDataByRoute(siteDataRef.value, route.path)
   })
 }
+
+function findMatchRoot(route: string, roots: string[]) {
+  // first match to the routes with the most deep level.
+  roots.sort((a, b) => {
+    const levelDelta = b.split('/').length - a.split('/').length
+    if (levelDelta !== 0) {
+      return levelDelta
+    } else {
+      return b.length - a.length
+    }
+  })
+
+  for (const r of roots) {
+    if (route.startsWith(r)) return r
+  }
+  return undefined
+}
+
+function resolveLocales<T>(
+  locales: Record<string, T>,
+  route: string
+): T | undefined {
+  const localeRoot = findMatchRoot(route, Object.keys(locales))
+  return localeRoot ? locales[localeRoot] : undefined
+}
+
+// this merges the locales data to the main data by the route
+export function resolveSiteDataByRoute(siteData: SiteData, route: string) {
+  route = cleanRoute(siteData, route)
+
+  const localeData = resolveLocales(siteData.locales || {}, route) || {}
+  const localeThemeConfig =
+    resolveLocales<any>(
+      (siteData.themeConfig && siteData.themeConfig.locales) || {},
+      route
+    ) || {}
+
+  return {
+    ...siteData,
+    ...localeData,
+    themeConfig: {
+      ...siteData.themeConfig,
+      ...localeThemeConfig,
+      // clean the locales to reduce the bundle size
+      locales: {}
+    },
+    locales: {}
+  }
+}
+
+/**
+ * Clean up the route by removing the `base` path if it's set in config.
+ */
+function cleanRoute(siteData: SiteData, route: string): string {
+  if (!inBrowser) {
+    return route
+  }
+
+  const base = siteData.base
+  const baseWithoutSuffix = base.endsWith('/') ? base.slice(0, -1) : base
+
+  return route.slice(baseWithoutSuffix.length)
+}

src/client/tsconfig.json

@@ -2,7 +2,7 @@
   "extends": "../../tsconfig.base.json",
   "compilerOptions": {
     "baseUrl": ".",
-    "outDir": "../../dist",
+    "outDir": "../../dist/client",
     "module": "esnext",
     "lib": ["ESNext", "DOM"],
     "types": ["vite"],
@@ -14,10 +14,7 @@
     }
   },
   "include": [
-    ".", 
+    ".",
     "../../types/shared.d.ts",
-  ],
-  "exclude": [
-    "../shared"
   ]
 }

src/node/config.ts

@@ -2,24 +2,13 @@ import path from 'path'
 import fs from 'fs-extra'
 import chalk from 'chalk'
 import globby from 'globby'
-import { createResolver, APP_PATH } from './resolver'
 import { Resolver } from 'vite'
 import { SiteData, HeadConfig, LocaleConfig } from '../../types/shared'
-export { resolveSiteDataByRoute } from '../shared/config'
+import { createResolver, APP_PATH } from './resolver'
 
 const debug = require('debug')('vitepress:config')
 
-export interface UserConfig<ThemeConfig = any> {
-  lang?: string
-  base?: string
-  title?: string
-  description?: string
-  head?: HeadConfig[]
-  themeConfig?: ThemeConfig
-  locales?: Record<string, LocaleConfig>
-  alias?: Record<string, string>
-  // TODO locales support etc.
-}
+const inBrowser = typeof window !== 'undefined'
 
 export interface SiteConfig<ThemeConfig = any> {
   root: string
@@ -32,6 +21,17 @@ export interface SiteConfig<ThemeConfig = any> {
   pages: string[]
 }
 
+export interface UserConfig<ThemeConfig = any> {
+  lang?: string
+  base?: string
+  title?: string
+  description?: string
+  head?: HeadConfig[]
+  themeConfig?: ThemeConfig
+  locales?: Record<string, LocaleConfig>
+  alias?: Record<string, string>
+}
+
 const resolve = (root: string, file: string) =>
   path.resolve(root, `.vitepress`, file)
 
@@ -90,3 +90,68 @@ export async function resolveSiteData(root: string): Promise<SiteData> {
     locales: userConfig.locales || {}
   }
 }
+
+/**
+ * This method merges the locales data to the main data by the route.
+ */
+export function resolveSiteDataByRoute(siteData: SiteData, route: string) {
+  route = cleanRoute(siteData, route)
+
+  const localeData = resolveLocales(siteData.locales || {}, route) || {}
+  const localeThemeConfig =
+    resolveLocales<any>(
+      (siteData.themeConfig && siteData.themeConfig.locales) || {},
+      route
+    ) || {}
+
+  return {
+    ...siteData,
+    ...localeData,
+    themeConfig: {
+      ...siteData.themeConfig,
+      ...localeThemeConfig,
+      // clean the locales to reduce the bundle size
+      locales: {}
+    },
+    locales: {}
+  }
+}
+
+function resolveLocales<T>(
+  locales: Record<string, T>,
+  route: string
+): T | undefined {
+  const localeRoot = findMatchRoot(route, Object.keys(locales))
+  return localeRoot ? locales[localeRoot] : undefined
+}
+
+function findMatchRoot(route: string, roots: string[]) {
+  // first match to the routes with the most deep level.
+  roots.sort((a, b) => {
+    const levelDelta = b.split('/').length - a.split('/').length
+    if (levelDelta !== 0) {
+      return levelDelta
+    } else {
+      return b.length - a.length
+    }
+  })
+
+  for (const r of roots) {
+    if (route.startsWith(r)) return r
+  }
+  return undefined
+}
+
+/**
+ * Clean up the route by removing the `base` path if it's set in config.
+ */
+function cleanRoute(siteData: SiteData, route: string): string {
+  if (!inBrowser) {
+    return route
+  }
+
+  const base = siteData.base
+  const baseWithoutSuffix = base.endsWith('/') ? base.slice(0, -1) : base
+
+  return route.slice(baseWithoutSuffix.length)
+}

src/node/tsconfig.json

@@ -1,10 +1,14 @@
 {
   "extends": "../../tsconfig.base.json",
   "compilerOptions": {
-    "outDir": "../../dist",
+    "baseUrl": ".",
+    "outDir": "../../dist/node",
     "module": "commonjs",
     "lib": ["ESNext", "DOM"],
     "sourceMap": true
   },
-  "include": [".", "../shared", "../../types/shared.d.ts"]
+  "include": [
+    ".",
+    "../../types/shared.d.ts"
+  ]
 }

src/shared/config.ts

@@ -1,66 +0,0 @@
-import { SiteData } from '../../types/shared'
-
-const inBrowser = typeof window !== 'undefined'
-
-function findMatchRoot(route: string, roots: string[]) {
-  // first match to the routes with the most deep level.
-  roots.sort((a, b) => {
-    const levelDelta = b.split('/').length - a.split('/').length
-    if (levelDelta !== 0) {
-      return levelDelta
-    } else {
-      return b.length - a.length
-    }
-  })
-
-  for (const r of roots) {
-    if (route.startsWith(r)) return r
-  }
-  return undefined
-}
-
-function resolveLocales<T>(
-  locales: Record<string, T>,
-  route: string
-): T | undefined {
-  const localeRoot = findMatchRoot(route, Object.keys(locales))
-  return localeRoot ? locales[localeRoot] : undefined
-}
-
-// this merges the locales data to the main data by the route
-export function resolveSiteDataByRoute(siteData: SiteData, route: string) {
-  route = cleanRoute(siteData, route)
-
-  const localeData = resolveLocales(siteData.locales || {}, route) || {}
-  const localeThemeConfig =
-    resolveLocales<any>(
-      (siteData.themeConfig && siteData.themeConfig.locales) || {},
-      route
-    ) || {}
-
-  return {
-    ...siteData,
-    ...localeData,
-    themeConfig: {
-      ...siteData.themeConfig,
-      ...localeThemeConfig,
-      // clean the locales to reduce the bundle size
-      locales: {}
-    },
-    locales: {}
-  }
-}
-
-/**
- * Clean up the route by removing the `base` path if it's set in config.
- */
-function cleanRoute(siteData: SiteData, route: string): string {
-  if (!inBrowser) {
-    return route
-  }
-
-  const base = siteData.base
-  const baseWithoutSuffix = base.endsWith('/') ? base.slice(0, -1) : base
-
-  return route.slice(baseWithoutSuffix.length)
-}

src/shared/tsconfig.json

@@ -1,13 +0,0 @@
-{
-  "extends": "../../tsconfig.base.json",
-  "compilerOptions": {
-    "baseUrl": ".",
-    "outDir": "../../dist/client/shared",
-    "module": "esnext",
-    "lib": ["ESNext", "DOM"],
-  },
-  "include": [
-    ".", 
-    "../../types/shared.d.ts",
-  ]
-}