From 2f1fbf8e487fdae4db5c144d97f9d20e42bb1603 Mon Sep 17 00:00:00 2001
From: vben <anncwb@126.com>
Date: Thu, 29 Oct 2020 23:01:11 +0800
Subject: [PATCH] style: add some notes

---
 .eslintignore                   |  3 ---
 build/constant.ts               |  3 +++
 build/getShortName.ts           |  4 +++
 build/script/buildConf.ts       | 11 ++++----
 build/utils.ts                  | 37 ++++++++++++++++++++++++++
 build/vite/hm.ts                |  4 +--
 build/vite/plugin/gzip/index.ts |  1 +
 build/vite/proxy.ts             |  6 +++++
 mock/_createProductionServer.ts |  3 +++
 src/App.vue                     |  6 +++++
 src/main.ts                     |  4 +++
 src/useApp.ts                   |  1 +
 src/utils/http/axios/const.ts   |  1 +
 vite.config.ts                  | 46 ++++++++++++++++++++-------------
 14 files changed, 101 insertions(+), 29 deletions(-)

diff --git a/.eslintignore b/.eslintignore
index 63fe48d1..11482780 100644
--- a/.eslintignore
+++ b/.eslintignore
@@ -20,9 +20,6 @@ lib
 Dockerfile
 vue.config.js
 commit-lint.js
-/src/assets/iconfont/
-/types/shims
-/src/types/shims
 postcss.config.js
 stylelint.config.js
 commitlint.config.js
diff --git a/build/constant.ts b/build/constant.ts
index 5e02f35c..b604acfb 100644
--- a/build/constant.ts
+++ b/build/constant.ts
@@ -1 +1,4 @@
+/**
+ * The name of the configuration file entered in the production environment
+ */
 export const GLOB_CONFIG_FILE_NAME = '_app.config.js';
diff --git a/build/getShortName.ts b/build/getShortName.ts
index 6a52fc34..b46205f0 100644
--- a/build/getShortName.ts
+++ b/build/getShortName.ts
@@ -1,3 +1,7 @@
+/**
+ * Get the configuration file variable name
+ * @param env
+ */
 export const getShortName = (env: any) => {
   return `__PRODUCTION__${env.VITE_GLOB_APP_SHORT_NAME || '__APP'}__CONF__`
     .toUpperCase()
diff --git a/build/script/buildConf.ts b/build/script/buildConf.ts
index b80cafc0..15e54d75 100644
--- a/build/script/buildConf.ts
+++ b/build/script/buildConf.ts
@@ -1,14 +1,12 @@
+/**
+ * Generate additional configuration files when used for packaging. The file can be configured with some global variables, so that it can be changed directly externally without repackaging
+ */
 import { GLOB_CONFIG_FILE_NAME } from '../constant';
 import fs, { writeFileSync } from 'fs-extra';
 
 import viteConfig from '../../vite.config';
 import { errorConsole, successConsole, getCwdPath, getEnvConfig } from '../utils';
-
-const getShortName = (env: any) => {
-  return `__PRODUCTION__${env.VITE_GLOB_APP_SHORT_NAME || '__APP'}__CONF__`
-    .toUpperCase()
-    .replace(/\s/g, '');
-};
+import { getShortName } from '../getShortName';
 
 function createConfig(
   {
@@ -20,6 +18,7 @@ function createConfig(
   try {
     const windowConf = `window.${configName}`;
     const outDir = viteConfig.outDir || 'dist';
+    // Ensure that the variable will not be modified
     const configStr = `${windowConf}=${JSON.stringify(config)};
 
       Object.freeze(${windowConf});
diff --git a/build/utils.ts b/build/utils.ts
index 87dc1a0d..86afb4b3 100644
--- a/build/utils.ts
+++ b/build/utils.ts
@@ -7,6 +7,7 @@ import chalk from 'chalk';
 
 export const isFunction = (arg: unknown): arg is (...args: any[]) => any =>
   typeof arg === 'function';
+
 export const isRegExp = (arg: unknown): arg is RegExp =>
   Object.prototype.toString.call(arg) === '[object RegExp]';
 
@@ -43,6 +44,9 @@ export function readAllFile(root: string, reg: RegExp) {
   return resultArr;
 }
 
+/**
+ * get client ip address
+ */
 export function getIPAddress() {
   let interfaces = networkInterfaces();
   for (let devName in interfaces) {
@@ -67,12 +71,23 @@ export function isProdFn(): boolean {
   return process.env.NODE_ENV === 'production';
 }
 
+/**
+ * Whether to generate package preview
+ */
 export function isReportMode(): boolean {
   return process.env.REPORT === 'true';
 }
+
+/**
+ * Whether to generate gzip for packaging
+ */
 export function isBuildGzip(): boolean {
   return process.env.VITE_BUILD_GZIP === 'true';
 }
+
+/**
+ *  Whether to generate package site
+ */
 export function isSiteMode(): boolean {
   return process.env.SITE === 'true';
 }
@@ -89,6 +104,7 @@ export interface ViteEnv {
   VITE_BUILD_GZIP: boolean;
 }
 
+// Read all environment variable configuration files to process.env
 export function loadEnv(): ViteEnv {
   const env = process.env.NODE_ENV;
   const ret: any = {};
@@ -116,6 +132,11 @@ export function loadEnv(): ViteEnv {
   return ret;
 }
 
+/**
+ * Get the environment variables starting with the specified prefix
+ * @param match prefix
+ * @param confFiles ext
+ */
 export function getEnvConfig(match = 'VITE_GLOB_', confFiles = ['.env', '.env.production']) {
   let envConfig = {};
   confFiles.forEach((item) => {
@@ -142,18 +163,34 @@ function consoleFn(color: string, message: any) {
   );
 }
 
+/**
+ * warnConsole
+ * @param message
+ */
 export function successConsole(message: any) {
   consoleFn('green', '✨ ' + message);
 }
 
+/**
+ * warnConsole
+ * @param message
+ */
 export function errorConsole(message: any) {
   consoleFn('red', '✨ ' + message);
 }
 
+/**
+ * warnConsole
+ * @param message message
+ */
 export function warnConsole(message: any) {
   consoleFn('yellow', '✨ ' + message);
 }
 
+/**
+ * Get user root directory
+ * @param dir file path
+ */
 export function getCwdPath(...dir: string[]) {
   return path.resolve(process.cwd(), ...dir);
 }
diff --git a/build/vite/hm.ts b/build/vite/hm.ts
index 2c3f3b6b..14abdd8b 100644
--- a/build/vite/hm.ts
+++ b/build/vite/hm.ts
@@ -1,5 +1,5 @@
-// 百度统计代码 用于站点部署
-// 只在build:site开启
+// Baidu statistics code for site deployment
+// Only open in build:site
 export const hmScript = `<script>
 var _hmt = _hmt || [];
 (function() {
diff --git a/build/vite/plugin/gzip/index.ts b/build/vite/plugin/gzip/index.ts
index 481aad9b..011b224b 100644
--- a/build/vite/plugin/gzip/index.ts
+++ b/build/vite/plugin/gzip/index.ts
@@ -1,6 +1,7 @@
 // 修改自https://github.com/kryops/rollup-plugin-gzip
 // 因为rollup-plugin-gzip不支持vite
 // vite对css打包独立的。所以不能在打包的时候顺带打包css
+// TODO rc.9会支持 configurBuild 配置项。到时候重新修改
 
 import { readFile, writeFile } from 'fs';
 import { basename } from 'path';
diff --git a/build/vite/proxy.ts b/build/vite/proxy.ts
index fbaee2ee..b64bcb5f 100644
--- a/build/vite/proxy.ts
+++ b/build/vite/proxy.ts
@@ -3,6 +3,11 @@ type ProxyItem = [string, string];
 type ProxyList = ProxyItem[];
 
 const reg = /^https:\/\//;
+
+/**
+ * Generate proxy
+ * @param list
+ */
 export function createProxy(list: ProxyList = []) {
   const ret: any = {};
   for (const [prefix, target] of list) {
@@ -12,6 +17,7 @@ export function createProxy(list: ProxyList = []) {
       target: target,
       changeOrigin: true,
       rewrite: (path: string) => path.replace(new RegExp(`^${prefix}`), ''),
+      // https is require secure=false
       ...(isHttps ? { secure: false } : {}),
     };
   }
diff --git a/mock/_createProductionServer.ts b/mock/_createProductionServer.ts
index 003a0984..f72b41ca 100644
--- a/mock/_createProductionServer.ts
+++ b/mock/_createProductionServer.ts
@@ -3,6 +3,9 @@ import userMock from './sys/user';
 import menuMock from './sys/menu';
 import tableDemoMock from './demo/table-demo';
 
+/**
+ * Used in a production environment. Need to manually import all modules
+ */
 export function setupProdMockServer() {
   createProdMockServer([...userMock, ...menuMock, ...tableDemoMock]);
 }
diff --git a/src/App.vue b/src/App.vue
index 92b6a482..15bb4577 100644
--- a/src/App.vue
+++ b/src/App.vue
@@ -23,14 +23,20 @@
     name: 'App',
     components: { ConfigProvider },
     setup() {
+      // Initialize application settings
       useInitAppConfigStore();
+      // Initialize network monitoring
       useListenerNetWork();
+      // Initialize breakpoint monitoring
       createBreakpointListen();
+      // Get system configuration
       const { projectSetting } = useSetting();
+      // Get ConfigProvider configuration
       const { transformCellText } = useConfigProvider();
 
       let lockOn = {};
       if (projectSetting.lockTime) {
+        // Monitor the mouse or keyboard time, used to recalculate the lock screen time
         const { on } = useLockPage();
         lockOn = on;
       }
diff --git a/src/main.ts b/src/main.ts
index 723451d2..7c6d1363 100644
--- a/src/main.ts
+++ b/src/main.ts
@@ -22,8 +22,10 @@ setupRouter(app);
 // store
 setupStore(app);
 
+// Directives
 setupDirectives(app);
 
+// error-handle
 setupErrorHandle(app);
 
 router.isReady().then(() => {
@@ -35,8 +37,10 @@ if (isDevMode()) {
   window.__APP__ = app;
 }
 
+// If you do not need to use the mock service in the production environment, you can comment the code
 if (isProdMode() && isUseMock()) {
   setupProdMockServer();
 }
 
+// Used to share app instances in other modules
 setApp(app);
diff --git a/src/useApp.ts b/src/useApp.ts
index 9525afc5..38f0c63f 100644
--- a/src/useApp.ts
+++ b/src/useApp.ts
@@ -1,3 +1,4 @@
+// Application related functions
 import type { ProjectConfig } from '/@/types/config';
 import type { App } from 'vue';
 import { computed, ref } from 'vue';
diff --git a/src/utils/http/axios/const.ts b/src/utils/http/axios/const.ts
index aed2f9a7..02b7c243 100644
--- a/src/utils/http/axios/const.ts
+++ b/src/utils/http/axios/const.ts
@@ -1 +1,2 @@
+// 接口返回值data不能为这个，否则会判为请求失败
 export const errorResult = '__ERROR_RESULT__';
diff --git a/vite.config.ts b/vite.config.ts
index fc7a01cf..f52528be 100644
--- a/vite.config.ts
+++ b/vite.config.ts
@@ -35,79 +35,89 @@ const viteConfig: UserConfig = {
   // TODO build error
   // entry: './public/index.html',
   /**
-   * 端口号
+   * port
    * @default '3000'
    */
   port: VITE_PORT,
   /**
-   * 服务地址
    * @default 'localhost'
    */
   hostname: 'localhost',
   /**
-   * 运行自动打开浏览器·
+   * Run to open the browser automatically
    * @default 'false'
    */
   open: false,
   /**
-   * 压缩代码
-   *  boolean | 'terser' | 'esbuild'
+   * Set to `false` to disable minification, or specify the minifier to use.
+   * Available options are 'terser' or 'esbuild'.
    * @default 'terser'
    */
   minify: isDevFn() ? 'esbuild' : 'terser',
   /**
-   * 基本公共路径
+   * Base public path when served in production.
    * @default '/'
    */
   base: VITE_PUBLIC_PATH,
 
   /**
-   * 打包输入路径
+   * Directory relative from `root` where build output will be placed. If the
+   * directory exists, it will be removed before the build.
    * @default 'dist'
    */
   outDir: 'dist',
   /**
-   * @default 'false'
+   * Whether to generate sourcemap
+   * @default false
    */
   sourcemap: false,
   /**
-   * 资源输出路径
+   * Directory relative from `outDir` where the built js/css/image assets will
+   * be placed.
    * @default '_assets'
    */
   assetsDir: '_assets',
   /**
-   * 静态资源小于该大小将会内联，默认4096kb
-   * @default '4096'
+   * Static asset files smaller than this number (in bytes) will be inlined as
+   * base64 strings. Default limit is `4096` (4kb). Set to `0` to disable.
+   * @default 4096
    */
   assetsInlineLimit: 4096,
   /**
-   * esbuild转换目标。
+   * Transpile target for esbuild.
    * @default 'es2020'
    */
   esbuildTarget: 'es2020',
+  /**
+   * Whether to log asset info to console
+   * @default false
+   */
   silent: false,
-  // 别名
+  /**
+   * Import alias. The entries can either be exact request -> request mappings
+   * (exact, no wildcard syntax), or request path -> fs directory mappings.
+   * When using directory mappings, the key **must start and end with a slash**.
+   * ```
+   */
   alias: {
     '/@/': pathResolve('src'),
   },
-  // terser配置
+  // terser options
   terserOptions: {
     compress: {
-      // 是否删除console
       drop_console: VITE_DROP_CONSOLE,
     },
   },
   define: {
     __VERSION__: pkg.version,
   },
-  // css预处理
   cssPreprocessOptions: {
     less: {
       modifyVars: modifyVars,
       javascriptEnabled: true,
     },
   },
-  // 会使用 rollup 对 包重新编译，将编译成符合 esm 模块规范的新的包放入 node_modules/.vite_opt_cache
+  // The package will be recompiled using rollup, and the new package compiled into the esm module specification will be put into node_modules/.vite_opt_cache
   optimizeDeps: {
     include: [
       'echarts',
@@ -118,7 +128,7 @@ const viteConfig: UserConfig = {
     ],
   },
 
-  // 本地跨域代理
+  // Local cross-domain proxy
   proxy: createProxy(VITE_PROXY),
   plugins: createVitePlugins(viteEnv),
   rollupInputOptions: {
-- 
GitLab