From b86420fc0622b2ca5b80d5cbe2c8d807d8124f63 Mon Sep 17 00:00:00 2001
From: zthxxx <zthxxx.me@gmail.com>
Date: Fri, 1 Jan 2021 23:58:28 +0800
Subject: [PATCH] fix: turn on react-dev-inspector only for locally developed
 devs (#7859)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- 在正式编译为页面后， react-dev-inspector 一般就没有意义了，不应该在正式编译是开启，推荐只在本地开发时使用
- 不在 build 时开启还能优化减少 build 耗时
- inspector 插件本身需要配合前端组件 `<Inspector>` 使用，而原本组件的代码中也只在 dev 下才开启

rel: 818926f 9862c94
---
 config/config.dev.ts | 15 +++++++++++++++
 config/config.ts     |  9 +--------
 package.json         | 12 ++++++------
 3 files changed, 22 insertions(+), 14 deletions(-)
 create mode 100644 config/config.dev.ts

diff --git a/config/config.dev.ts b/config/config.dev.ts
new file mode 100644
index 00000000..ab0e590f
--- /dev/null
+++ b/config/config.dev.ts
@@ -0,0 +1,15 @@
+// https://umijs.org/config/
+import { defineConfig } from 'umi';
+
+export default defineConfig({
+  plugins: [
+    // https://github.com/zthxxx/react-dev-inspector
+    'react-dev-inspector/plugins/umi/react-inspector',
+  ],
+  // https://github.com/zthxxx/react-dev-inspector#inspector-loader-props
+  inspectorConfig: {
+    exclude: [],
+    babelPlugins: [],
+    babelOptions: {},
+  },
+});
diff --git a/config/config.ts b/config/config.ts
index a0675002..d5d6fb9d 100644
--- a/config/config.ts
+++ b/config/config.ts
@@ -41,12 +41,5 @@ export default defineConfig({
     basePath: '/',
   },
   esbuild: {},
-  // https://github.com/zthxxx/react-dev-inspector
-  plugins: ['react-dev-inspector/plugins/umi/react-inspector'],
-  inspectorConfig: {
-    // loader options type and docs see below
-    exclude: [],
-    babelPlugins: [],
-    babelOptions: {},
-  },
+  plugins: [],
 });
diff --git a/package.json b/package.json
index 3a62e664..377dc47c 100644
--- a/package.json
+++ b/package.json
@@ -28,12 +28,12 @@
     "precommit": "lint-staged",
     "prettier": "prettier -c --write \"src/**/*\"",
     "site": "npm run fetch:blocks && npm run build",
-    "start": "umi dev",
-    "start:dev": "cross-env REACT_APP_ENV=dev MOCK=none umi dev",
-    "start:no-mock": "cross-env MOCK=none umi dev",
-    "start:no-ui": "cross-env UMI_UI=none umi dev",
-    "start:pre": "cross-env REACT_APP_ENV=pre umi dev",
-    "start:test": "cross-env REACT_APP_ENV=test MOCK=none umi dev",
+    "start": "cross-env UMI_ENV=dev umi dev",
+    "start:dev": "cross-env REACT_APP_ENV=dev MOCK=none UMI_ENV=dev umi dev",
+    "start:no-mock": "cross-env MOCK=none UMI_ENV=dev umi dev",
+    "start:no-ui": "cross-env UMI_UI=none UMI_ENV=dev umi dev",
+    "start:pre": "cross-env REACT_APP_ENV=pre UMI_ENV=dev umi dev",
+    "start:test": "cross-env REACT_APP_ENV=test MOCK=none UMI_ENV=dev umi dev",
     "pretest": "node ./tests/beforeTest",
     "test": "umi test",
     "test:all": "node ./tests/run-tests.js",