@modern-js/server 1.2.0 → 1.2.1-rc.0

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) {
@@ -348,7 +351,11 @@ class ModernServer {
   }
 
   async handleWeb(context, route) {
-    return this.routeRenderHandler(context, route);
+    return this.routeRenderHandler({
+      ctx: context,
+      route,
+      runner: this.runner
+    });
   }
 
   verifyMatch(_c, _m) {// empty
@@ -367,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);
@@ -508,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);
     }
@@ -555,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);
@@ -590,7 +596,11 @@ class ModernServer {
 
       if (entryName === status.toString() || entryName === '_error') {
         try {
-          const file = await this.routeRenderHandler(context, route);
+          const file = await this.routeRenderHandler({
+            route,
+            ctx: context,
+            runner: this.runner
+          });
 
           if (file) {
             context.res.end(file.content);

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>;
 

package/dist/types/libs/context/context.d.ts

@@ -1,8 +1,14 @@
+/// <reference path="../../type.d.ts" />
 /// <reference types="node" />
+/// <reference types="node/http" />
 import { IncomingMessage, ServerResponse } from 'http';
 import { URL } from 'url';
 import qs from 'querystring';
 import type { ModernServerContext as ModernServerContextInterface, Metrics, Logger } from '@modern-js/types/server';
+export declare type ContextOptions = {
+  logger?: Logger;
+  metrics?: Metrics;
+};
 export declare class ModernServerContext implements ModernServerContextInterface {
   /**
    * http request
@@ -20,13 +26,7 @@ export declare class ModernServerContext implements ModernServerContextInterface
   params: Record<string, string>;
   logger: Logger;
   metrics?: Metrics;
-  constructor(req: IncomingMessage, res: ServerResponse, {
-    logger,
-    metrics
-  }: {
-    logger: Logger;
-    metrics: Metrics;
-  });
+  constructor(req: IncomingMessage, res: ServerResponse);
   private bind;
   setParams(params: Record<string, string>): void;
   getReqHeader(key: string): string | string[];

package/dist/types/libs/context/index.d.ts

@@ -1,11 +1,4 @@
 import { IncomingMessage, ServerResponse } from 'http';
-import { Metrics, Logger } from '../../type';
 import { ModernServerContext } from './context';
-export declare const createContext: (req: IncomingMessage, res: ServerResponse, {
-  logger,
-  metrics
-}: {
-  logger: Logger;
-  metrics: Metrics;
-}) => ModernServerContext;
+export declare const createContext: (req: IncomingMessage, res: ServerResponse) => ModernServerContext;
 export { ModernServerContext };

package/dist/types/libs/hook-api/template.d.ts

@@ -3,10 +3,10 @@ declare class TemplateAPI {
   constructor(content: string);
   get(): string;
   set(content: string): void;
-  beforeHead(fragment: string): this;
-  afterHead(fragment: string): this;
-  beforeBody(fragment: string): this;
-  afterBody(fragment: string): this;
+  prependHead(fragment: string): this;
+  appendHead(fragment: string): this;
+  prependBody(fragment: string): this;
+  appendBody(fragment: string): this;
   replace(reg: RegExp | string, text: string): this;
 }
 

package/dist/types/libs/render/index.d.ts

@@ -1,10 +1,19 @@
 import { RenderResult } from '../../type';
 import { ModernRoute } from '../route';
 import { ModernServerContext } from '../context';
+import { ServerHookRunner } from "../../type.d";
 export declare const createRenderHandler: ({
   distDir,
   staticGenerate
 }: {
   distDir: string;
   staticGenerate: boolean;
-}) => (ctx: ModernServerContext, route: ModernRoute) => Promise<RenderResult | null>;
+}) => ({
+  ctx,
+  route,
+  runner
+}: {
+  ctx: ModernServerContext;
+  route: ModernRoute;
+  runner: ServerHookRunner;
+}) => Promise<RenderResult | null>;

package/dist/types/libs/render/ssr.d.ts

@@ -1,9 +1,10 @@
 import { ModernServerContext } from '../context';
 import { RenderResult } from '../../type';
+import { ServerHookRunner } from "../../type.d";
 export declare const render: (ctx: ModernServerContext, renderOptions: {
   distDir: string;
   bundle: string;
   template: string;
   entryName: string;
   staticGenerate: boolean;
-}) => Promise<RenderResult>;
+}, runner: ServerHookRunner) => Promise<RenderResult>;

package/dist/types/libs/render/type.d.ts

@@ -1,7 +1,5 @@
-/// <reference types="node" />
 /// <reference types="react" />
-import { IncomingHttpHeaders } from 'http';
-import { Metrics, Logger } from '../../type';
+import { BaseSSRServerContext } from '@modern-js/types/server';
 declare type MetaKeyMap = {
   header?: string[];
   query?: string[];
@@ -24,24 +22,7 @@ export declare enum RenderLevel {
   SERVER_PREFETCH = 1,
   SERVER_RENDER = 2,
 }
-export declare type SSRServerContext = {
-  request: {
-    params: Record<string, string>;
-    pathname: string;
-    query: Record<string, string>;
-    headers: IncomingHttpHeaders;
-    cookie?: string;
-  };
-  redirection: {
-    url?: string;
-    status?: number;
-  };
-  distDir: string;
-  template: string;
-  entryName: string;
-  logger: Logger;
-  metrics?: Metrics;
-  loadableManifest?: string;
+export declare type SSRServerContext = BaseSSRServerContext & {
   cacheConfig?: CacheConfig;
   staticGenerate?: boolean;
 };

package/dist/types/libs/route/index.d.ts

@@ -11,4 +11,5 @@ export declare class RouteMatchManager {
   matchEntry(entryname: string): RouteMatcher | undefined;
   getBundles(): (string | undefined)[];
 }
-export type { ModernRouteInterface, RouteMatcher, ModernRoute };
+export type { ModernRouteInterface, ModernRoute };
+export { RouteMatcher };

package/dist/types/server/modern-server.d.ts

@@ -26,9 +26,9 @@ export declare class ModernServer {
   private routeRenderHandler;
   private frameWebHandler;
   private frameAPIHandler;
+  private proxyHandler;
   private _handler;
   private readonly staticGenerate;
-  private proxyHandler;
   constructor({
     pwd,
     config,

package/dist/types/type.d.ts

@@ -4,6 +4,12 @@ import { serverManager } from '@modern-js/server-plugin';
 import type { NormalizedConfig } from '@modern-js/core';
 import type { Metrics, Logger, NextFunction } from '@modern-js/types/server';
 import { ModernRouteInterface } from './libs/route';
+declare module 'http' {
+  interface IncomingMessage {
+    logger: Logger;
+    metrics: Metrics;
+  }
+}
 declare module '@modern-js/core' {
   interface UserConfig {
     bff: {

package/dist/types/utils.d.ts

@@ -3,5 +3,4 @@ export declare const mergeExtension: (users: any[]) => {
 };
 export declare const toMessage: (dig: string, e: Error | string) => string;
 export declare const noop: () => void;
-export declare const createErrorDocument: (status: number, text: string) => string;
-export declare function applyMixins(derivedCtor: any, constructors: any[]): void;
+export declare const createErrorDocument: (status: number, text: string) => string;

package/package.json

@@ -11,7 +11,7 @@
     "modern",
     "modern.js"
   ],
-  "version": "1.2.0",
+  "version": "1.2.1-rc.0",
   "jsnext:source": "./src/index.ts",
   "types": "./dist/types/index.d.ts",
   "main": "./dist/js/node/index.js",
@@ -33,12 +33,12 @@
     "@babel/preset-typescript": "^7.15.0",
     "@babel/register": "^7.15.3",
     "@babel/runtime": "^7",
-    "@modern-js/core": "^1.2.0",
-    "@modern-js/hmr-client": "^1.1.1",
-    "@modern-js/server-plugin": "^1.1.3",
-    "@modern-js/server-utils": "^1.1.2",
-    "@modern-js/bff-utils": "^1.1.1",
-    "@modern-js/utils": "^1.1.5",
+    "@modern-js/core": "^1.2.1-rc.0",
+    "@modern-js/hmr-client": "^1.1.2-rc.0",
+    "@modern-js/server-plugin": "^1.1.4-rc.0",
+    "@modern-js/server-utils": "^1.1.3-rc.0",
+    "@modern-js/bff-utils": "^1.1.2-rc.0",
+    "@modern-js/utils": "^1.1.6-rc.0",
     "axios": "^0.21.4",
     "babel-plugin-module-resolver": "^4.1.0",
     "chokidar": "^3.5.2",
@@ -61,9 +61,9 @@
     "ws": "^8.2.0"
   },
   "devDependencies": {
-    "@modern-js/module-tools": "^1.1.2",
-    "@modern-js/plugin-testing": "^1.1.1",
-    "@modern-js/types": "^1.1.4",
+    "@modern-js/module-tools": "^1.1.4",
+    "@modern-js/plugin-testing": "^1.2.2",
+    "@modern-js/types": "^1.1.5-rc.0",
     "@types/jest": "^26",
     "@types/lru-cache": "^5.1.1",
     "@types/mime-types": "^2.1.0",
@@ -78,7 +78,9 @@
     "@types/webpack-dev-middleware": "^5.0.2",
     "@types/ws": "^7.4.7",
     "typescript": "^4",
-    "webpack": "^5.54.0"
+    "webpack": "^5.54.0",
+    "node-mocks-http": "^1.11.0",
+    "portfinder": "^1.0.28"
   },
   "peerDependencies": {
     "webpack": "^5.54.0"
@@ -97,7 +99,7 @@
     "new": "modern new",
     "build": "modern build",
     "dev": "modern build --watch",
-    "test": "modern test --passWithNoTests"
+    "test": "modern test"
   },
   "readme": "\n<p align=\"center\">\n  <a href=\"https://modernjs.dev\" target=\"blank\"><img src=\"https://lf3-static.bytednsdoc.com/obj/eden-cn/ylaelkeh7nuhfnuhf/modernjs-cover.png\" width=\"300\" alt=\"Modern.js Logo\" /></a>\n</p>\n<p align=\"center\">\n现代 Web 工程体系\n  <br/>\n  <a href=\"https://modernjs.dev\" target=\"blank\">\n    modernjs.dev\n  </a>\n</p>\n<p align=\"center\">\n  The meta-framework suite designed from scratch for frontend-focused modern web development\n</p>\n\n# Introduction\n\n> The doc site ([modernjs.dev](https://modernjs.dev)) and articles are only available in Chinese for now, we are planning to add English versions soon.\n\n- [Modern.js: Hello, World!](https://zhuanlan.zhihu.com/p/426707646)\n\n## Getting Started\n\n- [Quick Start](https://modernjs.dev/docs/start)\n- [Guides](https://modernjs.dev/docs/guides)\n- [API References](https://modernjs.dev/docs/apis)\n\n## Contributing\n\n- [Contributing Guide](https://github.com/modern-js-dev/modern.js/blob/main/CONTRIBUTING.md)\n"
 }

package/src/index.ts

@@ -1,19 +1,13 @@
 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 };
 
 export default (options: ModernServerOptions): Promise<Server> => {
-  // 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/src/libs/context/context.ts

@@ -8,6 +8,11 @@ import type {
 } from '@modern-js/types/server';
 import { toMessage } from '../../utils';
 
+export type ContextOptions = {
+  logger?: Logger;
+  metrics?: Metrics;
+};
+
 export class ModernServerContext implements ModernServerContextInterface {
   /**
    * http request
@@ -28,15 +33,11 @@ export class ModernServerContext implements ModernServerContextInterface {
 
   public metrics?: Metrics;
 
-  constructor(
-    req: IncomingMessage,
-    res: ServerResponse,
-    { logger, metrics }: { logger: Logger; metrics: Metrics },
-  ) {
+  constructor(req: IncomingMessage, res: ServerResponse) {
     this.req = req;
     this.res = res;
-    this.logger = logger;
-    this.metrics = metrics;
+    this.logger = req.logger;
+    this.metrics = req.metrics;
 
     this.bind();
   }

package/src/libs/context/index.ts

@@ -1,11 +1,7 @@
 import { IncomingMessage, ServerResponse } from 'http';
-import { Metrics, Logger } from '../../type';
 import { ModernServerContext } from './context';
 
-export const createContext = (
-  req: IncomingMessage,
-  res: ServerResponse,
-  { logger, metrics }: { logger: Logger; metrics: Metrics },
-) => new ModernServerContext(req, res, { logger, metrics });
+export const createContext = (req: IncomingMessage, res: ServerResponse) =>
+  new ModernServerContext(req, res);
 
 export { ModernServerContext };

package/src/libs/hook-api/template.ts

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

package/src/libs/proxy.ts

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

package/src/libs/render/cache/__tests__/cache.test.ts

@@ -15,6 +15,8 @@ const createCacheConfig = (config: any = {}) => ({
   ...config,
 });
 
+jest.setTimeout(60000);
+
 describe('cache', () => {
   it('should cache correctly', async () => {
     destroyCache();
@@ -179,7 +181,6 @@ describe('cache', () => {
     }
   });
 
-  jest.setTimeout(1000 * 10);
   it('should stale cache correctly', async () => {
     destroyCache();
     const cache = createCache();
@@ -207,7 +208,6 @@ describe('cache', () => {
     expect(staleResult?.isStale).toBe(true);
   });
 
-  jest.setTimeout(1000 * 15);
   it('should garbage cache correctly', async () => {
     destroyCache();
     const cache = createCache();

package/src/libs/render/index.ts

@@ -9,6 +9,7 @@ import { readFile } from './reader';
 import * as ssr from './ssr';
 import { supportModern, getModernEntry } from './modern';
 import { ERROR_DIGEST } from '@/constants';
+import { ServerHookRunner } from '@/type';
 
 export const createRenderHandler = ({
   distDir,
@@ -17,10 +18,15 @@ export const createRenderHandler = ({
   distDir: string;
   staticGenerate: boolean;
 }) =>
-  async function render(
-    ctx: ModernServerContext,
-    route: ModernRoute,
-  ): Promise<RenderResult | null> {
+  async function render({
+    ctx,
+    route,
+    runner,
+  }: {
+    ctx: ModernServerContext;
+    route: ModernRoute;
+    runner: ServerHookRunner;
+  }): Promise<RenderResult | null> {
     if (ctx.resHasHandled()) {
       return null;
     }
@@ -43,13 +49,17 @@ export const createRenderHandler = ({
     // handles ssr first
     if (route.isSSR) {
       try {
-        const result = await ssr.render(ctx, {
-          distDir,
-          entryName: route.entryName,
-          bundle: route.bundle,
-          template: templateHTML,
-          staticGenerate,
-        });
+        const result = await ssr.render(
+          ctx,
+          {
+            distDir,
+            entryName: route.entryName,
+            bundle: route.bundle,
+            template: templateHTML,
+            staticGenerate,
+          },
+          runner,
+        );
         return result;
       } catch (err) {
         ctx.error(ERROR_DIGEST.ERENDER, (err as Error).stack);

package/src/libs/render/reader.ts

@@ -24,7 +24,7 @@ const createCacheItem = async (filepath: string, mtime: Date) => {
 class LruReader {
   private readonly cache: LRU<string, FileCache>;
 
-  private timer?: NodeJS.Timeout;
+  // private timer?: NodeJS.Timeout;
 
   constructor() {
     this.cache = new LRU({
@@ -35,13 +35,13 @@ class LruReader {
   }
 
   public init() {
-    this.timeTask();
+    // this.timeTask();
   }
 
   public close() {
-    if (this.timer) {
-      clearInterval(this.timer);
-    }
+    // if (this.timer) {
+    //   clearInterval(this.timer);
+    // }
   }
 
   public async read(filepath: string) {
@@ -94,9 +94,9 @@ class LruReader {
     }
   }
 
-  private timeTask() {
-    this.timer = setInterval(() => this.update, 5 * 60 * 1000).unref();
-  }
+  // private timeTask() {
+  // this.timer = setInterval(() => this.update, 5 * 60 * 1000).unref();
+  // }
 }
 
 const reader = new LruReader();

package/src/libs/render/ssr.ts

@@ -5,6 +5,7 @@ import { ModernServerContext } from '../context';
 import { RenderResult } from '../../type';
 import cache from './cache';
 import { SSRServerContext } from './type';
+import { ServerHookRunner } from '@/type';
 
 export const render = async (
   ctx: ModernServerContext,
@@ -15,6 +16,7 @@ export const render = async (
     entryName: string;
     staticGenerate: boolean;
   },
+  runner: ServerHookRunner,
 ): Promise<RenderResult> => {
   const { bundle, distDir, template, entryName, staticGenerate } =
     renderOptions;
@@ -37,6 +39,8 @@ export const render = async (
     metrics: ctx.metrics,
   };
 
+  runner.extendSSRContext(context);
+
   const serverRender = require(bundleJS)[SERVER_RENDER_FUNCTION_NAME];
 
   const html = await cache(serverRender, ctx)(context);

package/src/libs/render/type.ts

@@ -1,5 +1,4 @@
-import { IncomingHttpHeaders } from 'http';
-import { Metrics, Logger } from '../../type';
+import { BaseSSRServerContext } from '@modern-js/types/server';
 
 type MetaKeyMap = {
   header?: string[];
@@ -26,21 +25,7 @@ export enum RenderLevel {
   SERVER_RENDER,
 }
 
-export type SSRServerContext = {
-  request: {
-    params: Record<string, string>;
-    pathname: string;
-    query: Record<string, string>;
-    headers: IncomingHttpHeaders;
-    cookie?: string;
-  };
-  redirection: { url?: string; status?: number };
-  distDir: string;
-  template: string;
-  entryName: string;
-  logger: Logger;
-  metrics?: Metrics;
-  loadableManifest?: string;
+export type SSRServerContext = BaseSSRServerContext & {
   cacheConfig?: CacheConfig;
   staticGenerate?: boolean;
 };

package/src/libs/route/index.ts

@@ -73,4 +73,5 @@ export class RouteMatchManager {
   }
 }
 
-export type { ModernRouteInterface, RouteMatcher, ModernRoute };
+export type { ModernRouteInterface, ModernRoute };
+export { RouteMatcher };