@modern-js/server 1.1.5-beta.1 → 1.2.1-beta.2

package/CHANGELOG.md

@@ -1,5 +1,25 @@
 # @modern-js/server
 
+## 1.2.0
+
+### Minor Changes
+
+- 5a4c557e: feat: support bff test
+
+### Patch Changes
+
+- e04914ce: add route types, fix metrics types
+- e04914ce: add route types, fix metrics types
+- Updated dependencies [90eeb72c]
+- Updated dependencies [e04914ce]
+- Updated dependencies [5a4c557e]
+- Updated dependencies [e04914ce]
+- Updated dependencies [ca7dcb32]
+- Updated dependencies [ecb344dc]
+  - @modern-js/core@1.2.0
+  - @modern-js/server-plugin@1.1.3
+  - @modern-js/utils@1.1.5
+
 ## 1.1.4
 
 ### Patch Changes

package/dist/js/modern/index.js

@@ -1,14 +1,8 @@
 import { Server } from "./server";
-export * from "./type";
-export * from "./libs/context";
-export * from "./libs/route";
 export { Server };
 export default (options => {
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
-  const allowEnvs = ['production', 'development', 'test'];
-
   if (options == null) {
-    throw new Error();
+    throw new Error('can not start mserver without options');
   }
 
   const server = new Server(options);

package/dist/js/modern/libs/context/context.js

@@ -13,10 +13,7 @@ export class ModernServerContext {
   /**
    * url params
    */
-  constructor(req, res, {
-    logger,
-    metrics
-  }) {
+  constructor(req, res) {
     this.req = void 0;
     this.res = void 0;
     this.params = {};
@@ -24,8 +21,8 @@ export class ModernServerContext {
     this.metrics = void 0;
     this.req = req;
     this.res = res;
-    this.logger = logger;
-    this.metrics = metrics;
+    this.logger = req.logger;
+    this.metrics = req.metrics;
     this.bind();
   }
 

package/dist/js/modern/libs/context/index.js

@@ -1,9 +1,3 @@
 import { ModernServerContext } from "./context";
-export const createContext = (req, res, {
-  logger,
-  metrics
-}) => new ModernServerContext(req, res, {
-  logger,
-  metrics
-});
+export const createContext = (req, res) => new ModernServerContext(req, res);
 export { ModernServerContext };

package/dist/js/modern/libs/hook-api/template.js

@@ -23,28 +23,28 @@ class TemplateAPI {
     this.content = content;
   }
 
-  beforeHead(fragment) {
+  prependHead(fragment) {
     const {
       head
     } = RegList.before;
     return this.replace(head, `${head}${fragment}`);
   }
 
-  afterHead(fragment) {
+  appendHead(fragment) {
     const {
       head
     } = RegList.after;
     return this.replace(head, `${fragment}${head}`);
   }
 
-  beforeBody(fragment) {
+  prependBody(fragment) {
     const {
       body
     } = RegList.before;
     return this.replace(body, `${body}${fragment}`);
   }
 
-  afterBody(fragment) {
+  appendBody(fragment) {
     const {
       body
     } = RegList.after;

package/dist/js/modern/libs/proxy.js

@@ -31,7 +31,7 @@ export const createProxyHandler = proxyOptions => {
       }, []));
     }
   } else {
-    formatedProxy.concat(proxyOptions);
+    formatedProxy.push(...proxyOptions);
   }
 
   const middlewares = formatedProxy.map(option => {

package/dist/js/modern/libs/render/cache/__tests__/cache.test.js

@@ -19,6 +19,7 @@ const createCacheConfig = (config = {}) => _objectSpread({
   matches: null
 }, config);
 
+jest.setTimeout(60000);
 describe('cache', () => {
   it('should cache correctly', async () => {
     destroyCache();
@@ -178,7 +179,6 @@ describe('cache', () => {
       }
     }
   });
-  jest.setTimeout(1000 * 10);
   it('should stale cache correctly', async () => {
     destroyCache();
     const cache = createCache();
@@ -204,7 +204,6 @@ describe('cache', () => {
     const staleResult = await cache.get(context);
     expect(staleResult === null || staleResult === void 0 ? void 0 : staleResult.isStale).toBe(true);
   });
-  jest.setTimeout(1000 * 15);
   it('should garbage cache correctly', async () => {
     destroyCache();
     const cache = createCache();

package/dist/js/modern/libs/render/reader.js

@@ -15,9 +15,9 @@ const createCacheItem = async (filepath, mtime) => {
 };
 
 class LruReader {
+  // private timer?: NodeJS.Timeout;
   constructor() {
     this.cache = void 0;
-    this.timer = void 0;
     this.cache = new LRU({
       max: 256 * MB,
       length: getContentLength,
@@ -26,14 +26,12 @@ class LruReader {
     });
   }
 
-  init() {
-    this.timeTask();
+  init() {// this.timeTask();
   }
 
-  close() {
-    if (this.timer) {
-      clearInterval(this.timer);
-    }
+  close() {// if (this.timer) {
+    //   clearInterval(this.timer);
+    // }
   }
 
   async read(filepath) {
@@ -92,11 +90,10 @@ class LruReader {
         cache.del(filepath);
       }
     }
-  }
+  } // private timeTask() {
+  // this.timer = setInterval(() => this.update, 5 * 60 * 1000).unref();
+  // }
 
-  timeTask() {
-    this.timer = setInterval(() => this.update, 5 * 60 * 1000).unref();
-  }
 
 }
 

package/dist/js/modern/libs/route/index.js

@@ -65,4 +65,5 @@ export class RouteMatchManager {
     return bundles;
   }
 
-}
+}
+export { RouteMatcher };

package/dist/js/modern/server/index.js

@@ -53,7 +53,7 @@ export class Server {
   }
 
   listen(port = 8080, listener) {
-    this.app.listen(port, () => {
+    this.app.listen(process.env.PORT || port, () => {
       if (listener) {
         listener();
       }
@@ -120,9 +120,11 @@ export class Server {
     });
     const appContext = await this.initAppContext();
     serverManager.run(() => {
+      var _options$config$outpu;
+
       ConfigContext.set(this.options.config);
       AppContext.set(_objectSpread(_objectSpread({}, appContext), {}, {
-        distDirectory: path.join(options.pwd, options.config.output.path || 'dist')
+        distDirectory: path.join(options.pwd, ((_options$config$outpu = options.config.output) === null || _options$config$outpu === void 0 ? void 0 : _options$config$outpu.path) || 'dist')
       }));
     });
     return serverManager.init({});

package/dist/js/modern/server/modern-server.js

@@ -44,6 +44,8 @@ export class ModernServer {
     metrics,
     proxyTarget
   }) {
+    var _config$output;
+
     this.pwd = void 0;
     this.distDir = void 0;
     this.workDir = void 0;
@@ -60,15 +62,15 @@ export class ModernServer {
     this.routeRenderHandler = void 0;
     this.frameWebHandler = null;
     this.frameAPIHandler = null;
+    this.proxyHandler = null;
     this._handler = void 0;
     this.staticGenerate = false;
-    this.proxyHandler = null;
 
     require('ignore-styles');
 
     this.isDev = Boolean(dev);
     this.pwd = pwd;
-    this.distDir = path.join(pwd, config.output.path || 'dist');
+    this.distDir = path.join(pwd, ((_config$output = config.output) === null || _config$output === void 0 ? void 0 : _config$output.path) || 'dist');
     this.workDir = this.isDev ? pwd : this.distDir;
     this.conf = config;
     this.logger = logger;
@@ -128,7 +130,7 @@ export class ModernServer {
     const {
       favicon,
       faviconByEntries
-    } = this.conf.output;
+    } = this.conf.output || {};
     const favicons = this.prepareFavicons(favicon, faviconByEntries); // Only work when without setting `assetPrefix`.
     // Setting `assetPrefix` means these resources should be uploaded to CDN.
 
@@ -175,6 +177,7 @@ export class ModernServer {
 
     return [];
   } // add promisify request handler to server
+  // handler should do not do more things after invoke next
 
 
   addHandler(handler) {
@@ -336,7 +339,7 @@ export class ModernServer {
       context
     }); // match routes in the route spec
 
-    const matched = this.router.match(context.url);
+    const matched = this.router.match(context.path);
 
     if (!matched) {
       this.render404(context);
@@ -477,7 +480,7 @@ export class ModernServer {
       const injection = JSON.stringify(_objectSpread(_objectSpread({}, manifest), {}, {
         modules
       }));
-      templateAPI.afterHead(`<script>window.modern_manifest=${injection}</script>`);
+      templateAPI.appendHead(`<script>window.modern_manifest=${injection}</script>`);
     } catch (e) {
       context.error(ERROR_DIGEST.EMICROINJ, e);
     }
@@ -524,10 +527,9 @@ export class ModernServer {
   requestHandler(req, res, next = () => {// empty
   }) {
     res.statusCode = 200;
-    const context = createContext(req, res, {
-      logger: this.logger,
-      metrics: this.metrics
-    });
+    req.logger = req.logger || this.logger;
+    req.metrics = req.metrics || this.metrics;
+    const context = createContext(req, res);
 
     try {
       this._handler(context, next);

package/dist/js/modern/utils.js

@@ -47,12 +47,4 @@ export const createErrorDocument = (status, text) => {
   </body>
   </html>
   `;
-}; // This can live anywhere in your codebase:
-
-export function applyMixins(derivedCtor, constructors) {
-  constructors.forEach(baseCtor => {
-    Object.getOwnPropertyNames(baseCtor.prototype).forEach(name => {
-      Object.defineProperty(derivedCtor.prototype, name, Object.getOwnPropertyDescriptor(baseCtor.prototype, name) || Object.create(null));
-    });
-  });
-}
+};

package/dist/js/node/index.js

@@ -3,9 +3,6 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-var _exportNames = {
-  Server: true
-};
 Object.defineProperty(exports, "Server", {
   enumerable: true,
   get: function () {
@@ -16,54 +13,9 @@ exports.default = void 0;
 
 var _server = require("./server");
 
-var _type = require("./type");
-
-Object.keys(_type).forEach(function (key) {
-  if (key === "default" || key === "__esModule") return;
-  if (Object.prototype.hasOwnProperty.call(_exportNames, key)) return;
-  if (key in exports && exports[key] === _type[key]) return;
-  Object.defineProperty(exports, key, {
-    enumerable: true,
-    get: function () {
-      return _type[key];
-    }
-  });
-});
-
-var _context = require("./libs/context");
-
-Object.keys(_context).forEach(function (key) {
-  if (key === "default" || key === "__esModule") return;
-  if (Object.prototype.hasOwnProperty.call(_exportNames, key)) return;
-  if (key in exports && exports[key] === _context[key]) return;
-  Object.defineProperty(exports, key, {
-    enumerable: true,
-    get: function () {
-      return _context[key];
-    }
-  });
-});
-
-var _route = require("./libs/route");
-
-Object.keys(_route).forEach(function (key) {
-  if (key === "default" || key === "__esModule") return;
-  if (Object.prototype.hasOwnProperty.call(_exportNames, key)) return;
-  if (key in exports && exports[key] === _route[key]) return;
-  Object.defineProperty(exports, key, {
-    enumerable: true,
-    get: function () {
-      return _route[key];
-    }
-  });
-});
-
 var _default = options => {
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
-  const allowEnvs = ['production', 'development', 'test'];
-
   if (options == null) {
-    throw new Error();
+    throw new Error('can not start mserver without options');
   }
 
   const server = new _server.Server(options);

package/dist/js/node/libs/context/context.js

@@ -25,10 +25,7 @@ class ModernServerContext {
   /**
    * url params
    */
-  constructor(req, res, {
-    logger,
-    metrics
-  }) {
+  constructor(req, res) {
     this.req = void 0;
     this.res = void 0;
     this.params = {};
@@ -36,8 +33,8 @@ class ModernServerContext {
     this.metrics = void 0;
     this.req = req;
     this.res = res;
-    this.logger = logger;
-    this.metrics = metrics;
+    this.logger = req.logger;
+    this.metrics = req.metrics;
     this.bind();
   }
 

package/dist/js/node/libs/context/index.js

@@ -13,12 +13,6 @@ exports.createContext = void 0;
 
 var _context = require("./context");
 
-const createContext = (req, res, {
-  logger,
-  metrics
-}) => new _context.ModernServerContext(req, res, {
-  logger,
-  metrics
-});
+const createContext = (req, res) => new _context.ModernServerContext(req, res);
 
 exports.createContext = createContext;

package/dist/js/node/libs/hook-api/template.js

@@ -29,28 +29,28 @@ class TemplateAPI {
     this.content = content;
   }
 
-  beforeHead(fragment) {
+  prependHead(fragment) {
     const {
       head
     } = RegList.before;
     return this.replace(head, `${head}${fragment}`);
   }
 
-  afterHead(fragment) {
+  appendHead(fragment) {
     const {
       head
     } = RegList.after;
     return this.replace(head, `${fragment}${head}`);
   }
 
-  beforeBody(fragment) {
+  prependBody(fragment) {
     const {
       body
     } = RegList.before;
     return this.replace(body, `${body}${fragment}`);
   }
 
-  afterBody(fragment) {
+  appendBody(fragment) {
     const {
       body
     } = RegList.after;

package/dist/js/node/libs/proxy.js

@@ -39,7 +39,7 @@ const createProxyHandler = proxyOptions => {
       }, []));
     }
   } else {
-    formatedProxy.concat(proxyOptions);
+    formatedProxy.push(...proxyOptions);
   }
 
   const middlewares = formatedProxy.map(option => {

package/dist/js/node/libs/render/cache/__tests__/cache.test.js

@@ -24,6 +24,7 @@ const createCacheConfig = (config = {}) => _objectSpread({
   matches: null
 }, config);
 
+jest.setTimeout(60000);
 describe('cache', () => {
   it('should cache correctly', async () => {
     (0, _spr.destroyCache)();
@@ -183,7 +184,6 @@ describe('cache', () => {
       }
     }
   });
-  jest.setTimeout(1000 * 10);
   it('should stale cache correctly', async () => {
     (0, _spr.destroyCache)();
     const cache = (0, _spr.createCache)();
@@ -209,7 +209,6 @@ describe('cache', () => {
     const staleResult = await cache.get(context);
     expect(staleResult === null || staleResult === void 0 ? void 0 : staleResult.isStale).toBe(true);
   });
-  jest.setTimeout(1000 * 15);
   it('should garbage cache correctly', async () => {
     (0, _spr.destroyCache)();
     const cache = (0, _spr.createCache)();

package/dist/js/node/libs/render/reader.js

@@ -26,9 +26,9 @@ const createCacheItem = async (filepath, mtime) => {
 };
 
 class LruReader {
+  // private timer?: NodeJS.Timeout;
   constructor() {
     this.cache = void 0;
-    this.timer = void 0;
     this.cache = new _lruCache.default({
       max: 256 * MB,
       length: getContentLength,
@@ -37,14 +37,12 @@ class LruReader {
     });
   }
 
-  init() {
-    this.timeTask();
+  init() {// this.timeTask();
   }
 
-  close() {
-    if (this.timer) {
-      clearInterval(this.timer);
-    }
+  close() {// if (this.timer) {
+    //   clearInterval(this.timer);
+    // }
   }
 
   async read(filepath) {
@@ -105,11 +103,10 @@ class LruReader {
         cache.del(filepath);
       }
     }
-  }
+  } // private timeTask() {
+  // this.timer = setInterval(() => this.update, 5 * 60 * 1000).unref();
+  // }
 
-  timeTask() {
-    this.timer = setInterval(() => this.update, 5 * 60 * 1000).unref();
-  }
 
 }
 

package/dist/js/node/libs/route/index.js

@@ -4,6 +4,12 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.RouteMatchManager = void 0;
+Object.defineProperty(exports, "RouteMatcher", {
+  enumerable: true,
+  get: function () {
+    return _matcher.RouteMatcher;
+  }
+});
 
 var _matcher = require("./matcher");
 

package/dist/js/node/server/index.js

@@ -69,7 +69,7 @@ class Server {
   }
 
   listen(port = 8080, listener) {
-    this.app.listen(port, () => {
+    this.app.listen(process.env.PORT || port, () => {
       if (listener) {
         listener();
       }
@@ -137,10 +137,12 @@ class Server {
     const appContext = await this.initAppContext();
 
     _serverPlugin.serverManager.run(() => {
+      var _options$config$outpu;
+
       _core.ConfigContext.set(this.options.config);
 
       _core.AppContext.set(_objectSpread(_objectSpread({}, appContext), {}, {
-        distDirectory: _path.default.join(options.pwd, options.config.output.path || 'dist')
+        distDirectory: _path.default.join(options.pwd, ((_options$config$outpu = options.config.output) === null || _options$config$outpu === void 0 ? void 0 : _options$config$outpu.path) || 'dist')
       }));
     });
 

package/dist/js/node/server/modern-server.js

@@ -74,6 +74,8 @@ class ModernServer {
     metrics,
     proxyTarget
   }) {
+    var _config$output;
+
     this.pwd = void 0;
     this.distDir = void 0;
     this.workDir = void 0;
@@ -90,15 +92,15 @@ class ModernServer {
     this.routeRenderHandler = void 0;
     this.frameWebHandler = null;
     this.frameAPIHandler = null;
+    this.proxyHandler = null;
     this._handler = void 0;
     this.staticGenerate = false;
-    this.proxyHandler = null;
 
     require('ignore-styles');
 
     this.isDev = Boolean(dev);
     this.pwd = pwd;
-    this.distDir = _path.default.join(pwd, config.output.path || 'dist');
+    this.distDir = _path.default.join(pwd, ((_config$output = config.output) === null || _config$output === void 0 ? void 0 : _config$output.path) || 'dist');
     this.workDir = this.isDev ? pwd : this.distDir;
     this.conf = config;
     this.logger = logger;
@@ -158,7 +160,7 @@ class ModernServer {
     const {
       favicon,
       faviconByEntries
-    } = this.conf.output;
+    } = this.conf.output || {};
     const favicons = this.prepareFavicons(favicon, faviconByEntries); // Only work when without setting `assetPrefix`.
     // Setting `assetPrefix` means these resources should be uploaded to CDN.
 
@@ -206,6 +208,7 @@ class ModernServer {
 
     return [];
   } // add promisify request handler to server
+  // handler should do not do more things after invoke next
 
 
   addHandler(handler) {
@@ -371,7 +374,7 @@ class ModernServer {
       context
     }); // match routes in the route spec
 
-    const matched = this.router.match(context.url);
+    const matched = this.router.match(context.path);
 
     if (!matched) {
       this.render404(context);
@@ -512,7 +515,7 @@ class ModernServer {
       const injection = JSON.stringify(_objectSpread(_objectSpread({}, manifest), {}, {
         modules
       }));
-      templateAPI.afterHead(`<script>window.modern_manifest=${injection}</script>`);
+      templateAPI.appendHead(`<script>window.modern_manifest=${injection}</script>`);
     } catch (e) {
       context.error(_constants.ERROR_DIGEST.EMICROINJ, e);
     }
@@ -559,10 +562,9 @@ class ModernServer {
   requestHandler(req, res, next = () => {// empty
   }) {
     res.statusCode = 200;
-    const context = (0, _context.createContext)(req, res, {
-      logger: this.logger,
-      metrics: this.metrics
-    });
+    req.logger = req.logger || this.logger;
+    req.metrics = req.metrics || this.metrics;
+    const context = (0, _context.createContext)(req, res);
 
     try {
       this._handler(context, next);

package/dist/js/node/utils.js

@@ -3,7 +3,6 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.applyMixins = applyMixins;
 exports.toMessage = exports.noop = exports.mergeExtension = exports.createErrorDocument = void 0;
 
 const mergeExtension = users => {
@@ -64,15 +63,6 @@ const createErrorDocument = (status, text) => {
   </body>
   </html>
   `;
-}; // This can live anywhere in your codebase:
-
-
-exports.createErrorDocument = createErrorDocument;
+};
 
-function applyMixins(derivedCtor, constructors) {
-  constructors.forEach(baseCtor => {
-    Object.getOwnPropertyNames(baseCtor.prototype).forEach(name => {
-      Object.defineProperty(derivedCtor.prototype, name, Object.getOwnPropertyDescriptor(baseCtor.prototype, name) || Object.create(null));
-    });
-  });
-}
+exports.createErrorDocument = createErrorDocument;

package/dist/types/index.d.ts

@@ -1,10 +1,8 @@
 import { ModernServerOptions } from './type';
 import { Server } from './server';
-export * from './type';
-export * from './libs/context';
-export * from './libs/route';
 export type { SSRServerContext } from './libs/render/type';
 export { Server };
+export type { ModernServerOptions };
 
 declare const _default: (options: ModernServerOptions) => Promise<Server>;