@umijs/core 4.0.0-rc.5 → 4.0.0-rc.6

package/dist/service/path.d.ts

@@ -7,6 +7,7 @@ export declare function getPaths(opts: {
     cwd: string;
     absSrcPath: string;
     absPagesPath: string;
+    absApiRoutesPath: string;
     absTmpPath: string;
     absNodeModulesPath: string;
     absOutputPath: string;

package/dist/service/path.js

@@ -13,6 +13,7 @@ function getPaths(opts) {
     const src = winJoin(cwd, 'src');
     const absSrcPath = (0, fs_1.existsSync)(src) && (0, fs_1.statSync)(src).isDirectory() ? src : cwd;
     const absPagesPath = winJoin(absSrcPath, 'pages');
+    const absApiRoutesPath = winJoin(absSrcPath, 'api');
     const tmp = opts.env === types_1.Env.development
         ? `.${opts.prefix}`
         : `.${opts.prefix}-${opts.env}`;
@@ -23,6 +24,7 @@ function getPaths(opts) {
         cwd,
         absSrcPath,
         absPagesPath,
+        absApiRoutesPath,
         absTmpPath,
         absNodeModulesPath,
         absOutputPath,

package/dist/service/pluginAPI.d.ts

@@ -33,7 +33,7 @@ export declare class PluginAPI {
     }): void;
     registerPresets(source: Plugin[], presets: any[]): void;
     registerPlugins(source: Plugin[], plugins: any[]): void;
-    skipPlugins(ids: string[]): void;
+    skipPlugins(keys: string[]): void;
     static proxyPluginAPI(opts: {
         pluginAPI: PluginAPI;
         service: Service;

package/dist/service/pluginAPI.js

@@ -116,9 +116,11 @@ class PluginAPI {
             source.splice(0, 0, ...mappedPlugins);
         }
     }
-    skipPlugins(ids) {
-        ids.forEach((id) => {
-            this.service.skipPluginIds.add(id);
+    skipPlugins(keys) {
+        keys.forEach((key) => {
+            (0, assert_1.default)(!(this.plugin.key === key), `plugin ${key} can't skip itself!`);
+            (0, assert_1.default)(this.service.keyToPluginMap[key], `key: ${key} is not be registered by any plugin. You can't skip it!`);
+            this.service.skipPluginIds.add(this.service.keyToPluginMap[key].id);
         });
     }
     static proxyPluginAPI(opts) {

package/dist/service/service.d.ts

@@ -39,6 +39,7 @@ export declare class Service {
         cwd?: string;
         absSrcPath?: string;
         absPagesPath?: string;
+        absApiRoutesPath?: string;
         absTmpPath?: string;
         absNodeModulesPath?: string;
         absOutputPath?: string;

package/dist/service/service.js

@@ -81,7 +81,7 @@ class Service {
                         if (!this.isPluginEnable(hook))
                             continue;
                         tAdd.tapPromise({
-                            name: hook.plugin.id,
+                            name: hook.plugin.key,
                             stage: hook.stage,
                             before: hook.before,
                         }, (memo) => __awaiter(this, void 0, void 0, function* () {
@@ -96,7 +96,7 @@ class Service {
                         if (!this.isPluginEnable(hook))
                             continue;
                         tModify.tapPromise({
-                            name: hook.plugin.id,
+                            name: hook.plugin.key,
                             stage: hook.stage,
                             before: hook.before,
                         }, (memo) => __awaiter(this, void 0, void 0, function* () {
@@ -110,7 +110,7 @@ class Service {
                         if (!this.isPluginEnable(hook))
                             continue;
                         tEvent.tapPromise({
-                            name: hook.plugin.id,
+                            name: hook.plugin.key,
                             stage: hook.stage || 0,
                             before: hook.before,
                         }, () => __awaiter(this, void 0, void 0, function* () {
@@ -187,10 +187,6 @@ class Service {
             while (plugins.length) {
                 yield this.initPlugin({ plugin: plugins.shift(), plugins });
             }
-            // keyToPluginMap
-            for (const id of Object.keys(this.plugins)) {
-                this.keyToPluginMap[this.plugins[id].key] = this.plugins[id];
-            }
             // collect configSchemas and configDefaults
             for (const id of Object.keys(this.plugins)) {
                 const { config, key } = this.plugins[id];
@@ -291,7 +287,7 @@ class Service {
         });
     }
     initPlugin(opts) {
-        var _a;
+        var _a, _b;
         return __awaiter(this, void 0, void 0, function* () {
             // register to this.plugins
             (0, assert_1.default)(!this.plugins[opts.plugin.id], `${opts.plugin.type} ${opts.plugin.id} is already registered by ${(_a = this.plugins[opts.plugin.id]) === null || _a === void 0 ? void 0 : _a.path}, ${opts.plugin.type} from ${opts.plugin.path} register failed.`);
@@ -335,6 +331,9 @@ class Service {
             if (opts.plugin.type === 'plugin') {
                 (0, assert_1.default)(!ret, `plugin should return nothing`);
             }
+            // key should be unique
+            (0, assert_1.default)(!this.keyToPluginMap[opts.plugin.key], `key ${opts.plugin.key} is already registered by ${(_b = this.keyToPluginMap[opts.plugin.key]) === null || _b === void 0 ? void 0 : _b.path}, ${opts.plugin.type} from ${opts.plugin.path} register failed.`);
+            this.keyToPluginMap[opts.plugin.key] = opts.plugin;
             if (ret === null || ret === void 0 ? void 0 : ret.presets) {
                 ret.presets = ret.presets.map((preset) => new plugin_1.Plugin({
                     path: preset,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@umijs/core",
-  "version": "4.0.0-rc.5",
+  "version": "4.0.0-rc.6",
   "homepage": "https://github.com/umijs/umi-next/tree/master/packages/core#readme",
   "bugs": "https://github.com/umijs/umi-next/issues",
   "repository": {
@@ -20,8 +20,8 @@
     "dev": "pnpm build -- --watch"
   },
   "dependencies": {
-    "@umijs/bundler-utils": "4.0.0-rc.5",
-    "@umijs/utils": "4.0.0-rc.5"
+    "@umijs/bundler-utils": "4.0.0-rc.6",
+    "@umijs/utils": "4.0.0-rc.6"
   },
   "devDependencies": {
     "@hapi/joi": "17.1.1",