@eggjs/core 6.0.0-beta.1 → 6.0.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@eggjs/core",
-  "version": "6.0.0-beta.1",
+  "version": "6.0.0",
   "publishConfig": {
     "access": "public"
   },
@@ -13,8 +13,10 @@
   "description": "A core plugin framework based on @eggjs/koa",
   "scripts": {
     "lint": "eslint src test --ext ts",
+    "pretest": "npm run prepublishOnly",
     "test": "npm run lint -- --fix && npm run test-local",
     "test-local": "egg-bin test",
+    "preci": "npm run prepublishOnly",
     "ci": "npm run lint && egg-bin cov && npm run prepublishOnly",
     "contributor": "git-contributor",
     "prepublishOnly": "tshy && tshy-after"
@@ -36,6 +38,7 @@
   "dependencies": {
     "@eggjs/koa": "^2.18.2",
     "@eggjs/router": "^3.0.5",
+    "@eggjs/utils": "^4.0.2",
     "egg-logger": "^3.5.0",
     "egg-path-matching": "^2.0.0",
     "extend2": "^4.0.0",
@@ -45,7 +48,7 @@
     "node-homedir": "^2.0.0",
     "performance-ms": "^1.1.0",
     "ready-callback": "^4.0.0",
-    "tsconfig-paths": "^4.1.1",
+    "tsconfig-paths": "^4.2.0",
     "utility": "^2.1.0"
   },
   "devDependencies": {
@@ -57,14 +60,12 @@
     "await-event": "2",
     "coffee": "5",
     "egg-bin": "6",
-    "egg-utils": "2",
     "eslint": "8",
     "eslint-config-egg": "13",
     "gals": "1",
     "git-contributor": "2",
     "js-yaml": "3",
     "mm": "3",
-    "spy": "1",
     "supertest": "7",
     "ts-node": "10",
     "tshy": "1",

package/src/loader/context_loader.ts

@@ -42,7 +42,7 @@ export interface ContextLoaderOptions extends Omit<FileLoaderOptions, 'target'>
   /** required inject */
   inject: Record<string, any>;
   /** property name defined to target */
-  property: string;
+  property: string | symbol;
   /** determine the field name of inject object. */
   fieldClass?: string;
 }
@@ -86,7 +86,7 @@ export class ContextLoader extends FileLoader {
         if (!ctx[CLASS_LOADER]) {
           ctx[CLASS_LOADER] = new Map();
         }
-        const classLoader: Map<string, ClassLoader> = ctx[CLASS_LOADER];
+        const classLoader: Map<string | symbol, ClassLoader> = ctx[CLASS_LOADER];
         let instance = classLoader.get(property);
         if (!instance) {
           instance = getInstance(target, ctx);

package/src/loader/egg_loader.ts

@@ -148,16 +148,12 @@ export class EggLoader {
      * loader will find all directories from the prototype of Application,
      * you should define `Symbol.for('egg#eggPath')` property.
      *
-     * ```
-     * // lib/example.js
-     * const egg = require('egg');
-     * class ExampleApplication extends egg.Application {
-     *   constructor(options) {
-     *     super(options);
-     *   }
-     *
+     * ```ts
+     * // src/example.ts
+     * import { Application } from 'egg';
+     * class ExampleApplication extends Application {
      *   get [Symbol.for('egg#eggPath')]() {
-     *     return path.join(__dirname, '..');
+     *     return baseDir;
      *   }
      * }
      * ```
@@ -370,17 +366,20 @@ export class EggLoader {
       if (proto === Object.prototype || proto === EggCore?.prototype) {
         break;
       }
-
-      assert(proto.hasOwnProperty(Symbol.for('egg#eggPath')), 'Symbol.for(\'egg#eggPath\') is required on Application');
       const eggPath = Reflect.get(proto, Symbol.for('egg#eggPath'));
-      assert(eggPath && typeof eggPath === 'string', 'Symbol.for(\'egg#eggPath\') should be string');
+      if (!eggPath) {
+        // if (EggCore) {
+        //   throw new TypeError('Symbol.for(\'egg#eggPath\') is required on Application');
+        // }
+        continue;
+      }
+      assert(typeof eggPath === 'string', 'Symbol.for(\'egg#eggPath\') should be string');
       assert(fs.existsSync(eggPath), `${eggPath} not exists`);
       const realpath = fs.realpathSync(eggPath);
       if (!eggPaths.includes(realpath)) {
         eggPaths.unshift(realpath);
       }
     }
-
     return eggPaths;
   }
 
@@ -1269,6 +1268,11 @@ export class EggLoader {
 
     this.options.logger.info('[@eggjs/core:egg_loader] Loaded middleware from %j', middlewarePaths);
     this.timing.end('Load Middleware');
+
+    // add router middleware, make sure router is the last middleware
+    const mw = this.app.router.middleware();
+    Reflect.set(mw, '_name', 'routerMiddleware');
+    this.app.use(mw);
   }
   /** end Middleware loader */
 
@@ -1334,10 +1338,6 @@ export class EggLoader {
   async loadRouter() {
     this.timing.start('Load Router');
     await this.loadFile(path.join(this.options.baseDir, 'app/router'));
-    // add router middleware
-    const mw = this.app.router.middleware();
-    Reflect.set(mw, '_name', 'routerMiddleware');
-    this.app.use(mw);
     this.timing.end('Load Router');
   }
   /** end Router loader */
@@ -1493,12 +1493,12 @@ export class EggLoader {
    * @param {Object} options - see {@link FileLoader}
    * @since 1.0.0
    */
-  async loadToApp(directory: string | string[], property: string, options: FileLoaderOptions) {
+  async loadToApp(directory: string | string[], property: string | symbol, options?: FileLoaderOptions) {
     const target = {};
     Reflect.set(this.app, property, target);
     options = {
       ...options,
-      directory: options.directory ?? directory,
+      directory: options?.directory ?? directory,
       target,
       inject: this.app,
     };
@@ -1516,7 +1516,7 @@ export class EggLoader {
    * @param {Object} options - see {@link ContextLoader}
    * @since 1.0.0
    */
-  async loadToContext(directory: string | string[], property: string, options?: ContextLoaderOptions) {
+  async loadToContext(directory: string | string[], property: string | symbol, options?: ContextLoaderOptions) {
     options = {
       ...options,
       directory: options?.directory || directory,

package/src/utils/index.ts

@@ -1,9 +1,8 @@
 import { debuglog } from 'node:util';
 import path from 'node:path';
 import fs from 'node:fs';
-import { pathToFileURL } from 'node:url';
 import BuiltinModule from 'node:module';
-import { createRequire } from 'node:module';
+import { importResolve, importModule } from '@eggjs/utils';
 
 const debug = debuglog('@eggjs/core:utils');
 
@@ -19,15 +18,6 @@ const extensions = (Module as any)._extensions;
 const extensionNames = Object.keys(extensions).concat([ '.cjs', '.mjs' ]);
 debug('Module extensions: %j', extensionNames);
 
-let _customRequire: NodeRequire;
-function getCustomRequire() {
-  if (!_customRequire && typeof require === 'undefined') {
-    _customRequire = createRequire(process.cwd());
-    // _customRequire = createRequire(import.meta.url);
-  }
-  return _customRequire;
-}
-
 export default {
   deprecated(message: string) {
     console.warn('[@eggjs/core:deprecated] %s', message);
@@ -35,19 +25,6 @@ export default {
 
   extensions,
 
-  // async _importOrRequire(filepath: string) {
-  //   // try import first
-  //   let obj: any;
-  //   try {
-  //     obj = await import(filepath);
-  //   } catch (err: any) {
-  //     debug('await import error, use require instead, %s', err);
-  //     // use custom require
-  //     obj = getCustomRequire()(filepath);
-  //   }
-  //   return obj;
-  // },
-
   async loadFile(filepath: string) {
     try {
       // if not js module, just return content buffer
@@ -55,33 +32,7 @@ export default {
       if (extname && !extensionNames.includes(extname)) {
         return fs.readFileSync(filepath);
       }
-      let obj: any;
-      let isESM = false;
-      if (typeof require === 'function') {
-        // commonjs
-        obj = require(filepath);
-        debug('[loadFile] require %s => %o', filepath, obj);
-        if (obj && obj.__esModule) {
-          isESM = true;
-        }
-      } else {
-        // esm
-        debug('[loadFile] await import start: %s', filepath);
-        const fileUrl = pathToFileURL(filepath).toString();
-        obj = await import(fileUrl);
-        debug('[loadFile] await import end: %s => %o', filepath, obj);
-        isESM = true;
-        if (obj && typeof obj === 'object' && 'default' in obj) {
-          // default: { default: [Function (anonymous)] }
-          obj = obj.default;
-        }
-      }
-      if (!obj) return obj;
-      // it's es module, use default export
-      if (isESM && typeof obj === 'object') {
-        obj = 'default' in obj ? obj.default : obj;
-      }
-      debug('[loadFile] return %s => %o', filepath, obj);
+      const obj = await importModule(filepath, { importDefaultOnly: true });
       return obj;
     } catch (e: any) {
       const err = new Error(`[@eggjs/core] load file: ${filepath}, error: ${e.message}`);
@@ -92,10 +43,7 @@ export default {
   },
 
   resolvePath(filepath: string, options?: { paths?: string[] }) {
-    if (typeof require !== 'undefined') {
-      return require.resolve(filepath, options);
-    }
-    return getCustomRequire().resolve(filepath, options);
+    return importResolve(filepath, options);
   },
 
   methods: [ 'head', 'options', 'get', 'put', 'patch', 'post', 'delete' ],