Merge branch 'main' into meteorlxy-markdown-to-vue

Author: meteorlxy

e2e/docs/.vuepress/plugins/foo/fooPlugin.ts

@@ -1,8 +1,9 @@
+import type { Plugin } from 'vuepress/core'
 import { getDirname, path } from 'vuepress/utils'
 
 const __dirname = getDirname(import.meta.url)
 
-export const fooPlugin = {
+export const fooPlugin: Plugin = {
   name: 'test-plugin',
   clientConfigFile: path.resolve(
     __dirname,

packages/cli/tests/config/loadUserConfig.spec.ts

@@ -47,31 +47,29 @@ const MJS_CASES: [string, unknown][] = [
   ],
 ]
 
-describe('cli > config > loadUserConfig', () => {
-  describe('should load ts config file correctly', () => {
-    TS_CASES.forEach(([source, expected]) => {
-      it(JSON.stringify(source), async () => {
-        const { userConfig } = await loadUserConfig(source)
-        expect(userConfig).toEqual(expected)
-      })
+describe('should load ts config file correctly', () => {
+  TS_CASES.forEach(([source, expected]) => {
+    it(JSON.stringify(source), async () => {
+      const { userConfig } = await loadUserConfig(source)
+      expect(userConfig).toEqual(expected)
     })
   })
+})
 
-  describe('should load js config file correctly', () => {
-    JS_CASES.forEach(([source, expected]) => {
-      it(JSON.stringify(source), async () => {
-        const { userConfig } = await loadUserConfig(source)
-        expect(userConfig).toEqual(expected)
-      })
+describe('should load js config file correctly', () => {
+  JS_CASES.forEach(([source, expected]) => {
+    it(JSON.stringify(source), async () => {
+      const { userConfig } = await loadUserConfig(source)
+      expect(userConfig).toEqual(expected)
     })
   })
+})
 
-  describe('should load mjs config file correctly', () => {
-    MJS_CASES.forEach(([source, expected]) => {
-      it(JSON.stringify(source), async () => {
-        const { userConfig } = await loadUserConfig(source)
-        expect(userConfig).toEqual(expected)
-      })
+describe('should load mjs config file correctly', () => {
+  MJS_CASES.forEach(([source, expected]) => {
+    it(JSON.stringify(source), async () => {
+      const { userConfig } = await loadUserConfig(source)
+      expect(userConfig).toEqual(expected)
     })
   })
 })

packages/cli/tests/config/resolveUserConfigConventionalPath.spec.ts

@@ -14,13 +14,11 @@ const TEST_CASES: [string, string][] = [
   [resolveFixtures('case6'), '.vuepress/config.mjs'],
 ]
 
-describe('cli > config > resolveUserConfigConventionalPath', () => {
-  describe('should resolve conventional config file correctly', () => {
-    TEST_CASES.forEach(([source, expected]) => {
-      it(expected, () => {
-        const configFile = resolveUserConfigConventionalPath(source, source)
-        expect(configFile).toEqual(path.resolve(source, expected))
-      })
+describe('should resolve conventional config file correctly', () => {
+  TEST_CASES.forEach(([source, expected]) => {
+    it(expected, () => {
+      const configFile = resolveUserConfigConventionalPath(source, source)
+      expect(configFile).toEqual(path.resolve(source, expected))
     })
   })
 })

packages/cli/tests/config/resolveUserConfigPath.spec.ts

@@ -1,32 +1,30 @@
 import { path } from '@vuepress/utils'
-import { describe, expect, it, vi } from 'vitest'
+import { expect, it, vi } from 'vitest'
 import { resolveUserConfigPath } from '../../src/index.js'
 
 const resolveFixtures = (str: string): string =>
   path.resolve(__dirname, '../__fixtures__/config', str)
 
-describe('cli > config > resolveUserConfigPath', () => {
-  it('should resolve absolute file path correctly', () => {
-    const absolutePath = resolveFixtures('custom-config.ts')
-    const configFile = resolveUserConfigPath(absolutePath)
-    expect(configFile).toEqual(absolutePath)
-  })
+it('should resolve absolute file path correctly', () => {
+  const absolutePath = resolveFixtures('custom-config.ts')
+  const configFile = resolveUserConfigPath(absolutePath)
+  expect(configFile).toEqual(absolutePath)
+})
 
-  it('should resolve relative file path correctly', () => {
-    const relativePath = 'custom-config.ts'
-    const configFile = resolveUserConfigPath(relativePath, resolveFixtures(''))
-    expect(configFile).toEqual(resolveFixtures(relativePath))
-  })
+it('should resolve relative file path correctly', () => {
+  const relativePath = 'custom-config.ts'
+  const configFile = resolveUserConfigPath(relativePath, resolveFixtures(''))
+  expect(configFile).toEqual(resolveFixtures(relativePath))
+})
 
-  it('should throw an error if file does not exist', () => {
-    const consoleError = console.error
-    console.error = vi.fn()
+it('should throw an error if file does not exist', () => {
+  const consoleError = console.error
+  console.error = vi.fn()
 
-    expect(() => {
-      resolveUserConfigPath('4-0-4')
-    }).toThrow()
-    expect(console.error).toHaveBeenCalled()
+  expect(() => {
+    resolveUserConfigPath('4-0-4')
+  }).toThrow()
+  expect(console.error).toHaveBeenCalled()
 
-    console.error = consoleError
-  })
+  console.error = consoleError
 })

packages/core/src/app/appInit.ts

@@ -9,6 +9,8 @@ const log = debug('vuepress:core/app')
  * Initialize a vuepress app
  *
  * Plugins should be used before initialization.
+ *
+ * @internal
  */
 export const appInit = async (app: App): Promise<void> => {
   log('init start')

packages/core/src/app/appPrepare.ts

@@ -16,6 +16,8 @@ const log = debug('vuepress:core/app')
  * - routes
  * - site data
  * - other files that generated by plugins
+ *
+ * @internal
  */
 export const appPrepare = async (app: App): Promise<void> => {
   log('prepare start')

packages/core/src/app/appUse.ts

@@ -4,6 +4,13 @@ import { resolvePluginObject } from './resolvePluginObject.js'
 
 const log = debug('vuepress:core/app')
 
+/**
+ * Use a plugin in vuepress app.
+ *
+ * Should be called before initialization.
+ *
+ * @internal
+ */
 export const appUse = (app: App, rawPlugin: Plugin): App => {
   const pluginObject = resolvePluginObject(app, rawPlugin)
 

packages/core/src/app/createBaseApp.ts

@@ -1,5 +1,10 @@
 import { createPluginApi } from '../pluginApi/index.js'
-import type { App, AppConfig, Plugin } from '../types/index.js'
+import type {
+  App,
+  AppConfig,
+  AppPropertiesBase,
+  Plugin,
+} from '../types/index.js'
 import { appInit } from './appInit.js'
 import { appPrepare } from './appPrepare.js'
 import { appUse } from './appUse.js'
@@ -12,12 +17,16 @@ import { resolveAppWriteTemp } from './resolveAppWriteTemp.js'
 import { setupAppThemeAndPlugins } from './setupAppThemeAndPlugins.js'
 
 /**
- * Create vuepress app
+ * Create base vuepress app.
+ *
+ * Notice that the base app could not be used for dev nor build.
+ *
+ * It would be used for creating dev app or build app, or for testing.
  */
-export const createBaseApp = (config: AppConfig, isBuild = false): App => {
+export const createBaseApp = (config: AppConfig): App => {
   const options = resolveAppOptions(config)
   const dir = resolveAppDir(options)
-  const env = resolveAppEnv(options, isBuild)
+  const env = resolveAppEnv(options)
   const pluginApi = createPluginApi()
   const siteData = resolveAppSiteData(options)
   const version = resolveAppVersion()
@@ -38,7 +47,7 @@ export const createBaseApp = (config: AppConfig, isBuild = false): App => {
     use: (plugin: Plugin) => appUse(app, plugin),
     init: async () => appInit(app),
     prepare: async () => appPrepare(app),
-  } as App
+  } satisfies AppPropertiesBase as App
 
   // setup theme and plugins
   // notice that we setup theme before plugins,

packages/core/src/app/createBuildApp.ts

@@ -2,10 +2,14 @@ import type { AppConfig, BuildApp } from '../types/index.js'
 import { createBaseApp } from './createBaseApp.js'
 
 /**
- * Create vuepress build app
+ * Create vuepress build app.
  */
 export const createBuildApp = (config: AppConfig): BuildApp => {
-  const app = createBaseApp(config, true) as BuildApp
+  const app = createBaseApp(config) as BuildApp
+
+  // set env flag and add build method
+  app.env.isBuild = true
   app.build = async () => app.options.bundler.build(app)
+
   return app
 }

packages/core/src/app/createDevApp.ts

@@ -2,10 +2,14 @@ import type { AppConfig, DevApp } from '../types/index.js'
 import { createBaseApp } from './createBaseApp.js'
 
 /**
- * Create vuepress dev app
+ * Create vuepress dev app.
  */
 export const createDevApp = (config: AppConfig): DevApp => {
-  const app = createBaseApp(config, false) as DevApp
+  const app = createBaseApp(config) as DevApp
+
+  // set env flag and add dev method
+  app.env.isDev = true
   app.dev = async () => app.options.bundler.dev(app)
+
   return app
 }