@modern-js/plugin-proxy 1.1.2-rc.0 → 1.2.0

package/CHANGELOG.md

@@ -1,18 +1,21 @@
 # @modern-js/plugin-proxy
 
-## 1.1.2-rc.0
+## 1.2.0
+
+### Minor Changes
+
+- cfe11628: Make Modern.js self bootstraping
 
 ### Patch Changes
 
-- Updated dependencies [d927bc83]
-- Updated dependencies [d73ff455]
-- Updated dependencies [9c1ab865]
-- Updated dependencies [d73ff455]
-- Updated dependencies [d73ff455]
-- Updated dependencies [d73ff455]
-- Updated dependencies [d73ff455]
-  - @modern-js/utils@1.1.4-rc.0
-  - @modern-js/core@1.1.4-rc.0
+- f6932de7: fix proxy bin error
+- bc87647d: fix: terminate whistle proxy when exiting the process
+- Updated dependencies [2da09c69]
+- Updated dependencies [fc71e36f]
+- Updated dependencies [c3d46ee4]
+- Updated dependencies [cfe11628]
+  - @modern-js/utils@1.2.0
+  - @modern-js/core@1.3.0
 
 ## 1.1.1
 

package/dist/js/modern/index.js

@@ -2,6 +2,7 @@ import { createPlugin, useAppContext, useResolvedConfigContext } from '@modern-j
 import { PLUGIN_SCHEMAS } from '@modern-js/utils';
 import { createProxyRule } from "./utils/createProxyRule";
 import WhistleProxy from "./utils/whistleProxy";
+let proxyServer;
 export default createPlugin(() => ({
   validateSchema() {
     return PLUGIN_SCHEMAS['@modern-js/plugin-proxy'];
@@ -22,15 +23,18 @@ export default createPlugin(() => ({
     }
 
     const rule = createProxyRule(internalDirectory, dev.proxy);
-    const proxyServer = new WhistleProxy({
+    proxyServer = new WhistleProxy({
       port: 8899,
       rule
     });
-    await proxyServer.start(); // TODO
-    // should exit proxy server before process exit
-    // api.on('process-exit', () => {
-    //   proxyServer.close();
-    // });
+    await proxyServer.start();
+  },
+
+  beforeExit() {
+    var _proxyServer;
+
+    // terminate whistle proxy
+    (_proxyServer = proxyServer) === null || _proxyServer === void 0 ? void 0 : _proxyServer.close();
   }
 
 }), {

package/dist/js/modern/utils/macCAManager.js

@@ -35,7 +35,7 @@ export const generateRootCA = () => new Promise((resolve, reject) => {
 
   fs.ensureDirSync(defaultCertDir);
   const stream = fs.createWriteStream(defaultRootCA);
-  http.get('http://127.0.0.1:8899/cgi-bin/rootca', response => {
+  http.get('http://localhost:8899/cgi-bin/rootca', response => {
     response.pipe(stream);
     stream.on('finish', () => {
       resolve(defaultRootCA);

package/dist/js/modern/utils/macProxyManager.js

@@ -16,7 +16,7 @@ const getNetworkType = () => {
 };
 
 export const enableGlobalProxy = (ip, port) => {
-  const networkType = getNetworkType(); // && networksetup -setproxybypassdomains ${networkType} 127.0.0.1 localhost
+  const networkType = getNetworkType(); // && networksetup -setproxybypassdomains ${networkType} localhost localhost
 
   execSync(`networksetup -setwebproxy ${networkType} ${ip} ${port}`);
   execSync(`networksetup -setsecurewebproxy ${networkType} ${ip} ${port}`);

package/dist/js/modern/utils/whistleProxy.js

@@ -39,14 +39,14 @@ export default class WhistleProxy {
     execSync(`${this.bin} start --certDir=${this.certDir} --port=${this.port}`);
     execSync(`${this.bin} use ${this.rule} --force`);
     await this.installRootCA();
-    enableGlobalProxy('127.0.0.1', this.port);
+    enableGlobalProxy('localhost', this.port);
     logger.info(`Proxy Server start on localhost:${this.port}\n`);
   }
 
   close() {
     execSync(`${this.bin} stop`);
     disableGlobalProxy();
-    logger.info(`Proxy Server has closed`);
+    logger.info(`Proxy Server has been closed`);
   }
 
 }

package/dist/js/node/index.js

@@ -15,6 +15,8 @@ var _whistleProxy = _interopRequireDefault(require("./utils/whistleProxy"));
 
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
+let proxyServer;
+
 var _default = (0, _core.createPlugin)(() => ({
   validateSchema() {
     return _utils.PLUGIN_SCHEMAS['@modern-js/plugin-proxy'];
@@ -35,15 +37,18 @@ var _default = (0, _core.createPlugin)(() => ({
     }
 
     const rule = (0, _createProxyRule.createProxyRule)(internalDirectory, dev.proxy);
-    const proxyServer = new _whistleProxy.default({
+    proxyServer = new _whistleProxy.default({
       port: 8899,
       rule
     });
-    await proxyServer.start(); // TODO
-    // should exit proxy server before process exit
-    // api.on('process-exit', () => {
-    //   proxyServer.close();
-    // });
+    await proxyServer.start();
+  },
+
+  beforeExit() {
+    var _proxyServer;
+
+    // terminate whistle proxy
+    (_proxyServer = proxyServer) === null || _proxyServer === void 0 ? void 0 : _proxyServer.close();
   }
 
 }), {

package/dist/js/node/utils/macCAManager.js

@@ -62,7 +62,7 @@ const generateRootCA = () => new Promise((resolve, reject) => {
 
   const stream = _utils.fs.createWriteStream(defaultRootCA);
 
-  _http.default.get('http://127.0.0.1:8899/cgi-bin/rootca', response => {
+  _http.default.get('http://localhost:8899/cgi-bin/rootca', response => {
     response.pipe(stream);
     stream.on('finish', () => {
       resolve(defaultRootCA);

package/dist/js/node/utils/macProxyManager.js

@@ -26,7 +26,7 @@ const getNetworkType = () => {
 };
 
 const enableGlobalProxy = (ip, port) => {
-  const networkType = getNetworkType(); // && networksetup -setproxybypassdomains ${networkType} 127.0.0.1 localhost
+  const networkType = getNetworkType(); // && networksetup -setproxybypassdomains ${networkType} localhost localhost
 
   (0, _execSync.default)(`networksetup -setwebproxy ${networkType} ${ip} ${port}`);
   (0, _execSync.default)(`networksetup -setsecurewebproxy ${networkType} ${ip} ${port}`);

package/dist/js/node/utils/whistleProxy.js

@@ -52,7 +52,7 @@ class WhistleProxy {
     (0, _execSync.default)(`${this.bin} start --certDir=${this.certDir} --port=${this.port}`);
     (0, _execSync.default)(`${this.bin} use ${this.rule} --force`);
     await this.installRootCA();
-    enableGlobalProxy('127.0.0.1', this.port);
+    enableGlobalProxy('localhost', this.port);
 
     _utils.logger.info(`Proxy Server start on localhost:${this.port}\n`);
   }
@@ -61,7 +61,7 @@ class WhistleProxy {
     (0, _execSync.default)(`${this.bin} stop`);
     disableGlobalProxy();
 
-    _utils.logger.info(`Proxy Server has closed`);
+    _utils.logger.info(`Proxy Server has been closed`);
   }
 
 }

package/jest.config.js

@@ -0,0 +1,8 @@
+const sharedConfig = require('@scripts/jest-config');
+
+/** @type {import('@jest/types').Config.InitialOptions} */
+module.exports = {
+  // eslint-disable-next-line node/no-unsupported-features/es-syntax
+  ...sharedConfig,
+  rootDir: __dirname,
+};

package/modern.config.js

@@ -1,10 +1,2 @@
 /** @type {import('@modern-js/module-tools').UserConfig} */
-module.exports = {
-  testing: {
-    jest: {
-      collectCoverage: true,
-      collectCoverageFrom: ['src/**/*.ts'],
-      coveragePathIgnorePatterns: ['/node_modules/'],
-    },
-  },
-};
+module.exports = {};

package/package.json

@@ -11,7 +11,7 @@
     "modern",
     "modern.js"
   ],
-  "version": "1.1.2-rc.0",
+  "version": "1.2.0",
   "jsnext:source": "./src/index.ts",
   "types": "./dist/types/index.d.ts",
   "main": "./dist/js/node/index.js",
@@ -20,31 +20,33 @@
   "exports": {
     ".": {
       "node": {
+        "jsnext:source": "./src/index.ts",
         "import": "./dist/js/modern/index.js",
         "require": "./dist/js/node/index.js"
       },
       "default": "./dist/js/treeshaking/index.js"
     },
-    "./cli": "./dist/js/node/index.js"
-  },
-  "bin": {
-    "modern": "./bin/modern.js"
+    "./cli": {
+      "jsnext:source": "./src/index.ts",
+      "default": "./dist/js/node/index.js"
+    }
   },
   "dependencies": {
     "@babel/runtime": "^7",
-    "@modern-js/utils": "^1.1.4-rc.0",
+    "@modern-js/utils": "^1.2.0",
     "whistle": "^2.7.18"
   },
   "devDependencies": {
     "@types/jest": "^26",
     "@types/node": "^14",
     "typescript": "^4",
-    "@modern-js/core": "^1.1.4-rc.0",
-    "@modern-js/plugin-testing": "^1.1.1",
-    "@modern-js/module-tools": "^1.1.1"
+    "@modern-js/core": "^1.3.0",
+    "@scripts/build": "0.0.0",
+    "jest": "^27",
+    "@scripts/jest-config": "0.0.0"
   },
   "peerDependencies": {
-    "@modern-js/core": "^1.1.4-rc.0"
+    "@modern-js/core": "^1.3.0"
   },
   "sideEffects": false,
   "modernConfig": {
@@ -54,12 +56,13 @@
   },
   "publishConfig": {
     "registry": "https://registry.npmjs.org/",
-    "access": "public"
+    "access": "public",
+    "types": "./dist/types/index.d.ts"
   },
   "scripts": {
     "new": "modern new",
     "build": "modern build",
-    "test": "modern test --passWithNoTests"
+    "test": "jest --passWithNoTests"
   },
   "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

@@ -7,6 +7,7 @@ import { PLUGIN_SCHEMAS } from '@modern-js/utils';
 import { createProxyRule } from './utils/createProxyRule';
 import WhistleProxy from './utils/whistleProxy';
 
+let proxyServer: WhistleProxy;
 export default createPlugin(
   () => ({
     validateSchema() {
@@ -23,14 +24,12 @@ export default createPlugin(
       }
 
       const rule = createProxyRule(internalDirectory, (dev as any).proxy);
-      const proxyServer = new WhistleProxy({ port: 8899, rule });
+      proxyServer = new WhistleProxy({ port: 8899, rule });
       await proxyServer.start();
-
-      // TODO
-      // should exit proxy server before process exit
-      // api.on('process-exit', () => {
-      //   proxyServer.close();
-      // });
+    },
+    beforeExit() {
+      // terminate whistle proxy
+      proxyServer?.close();
     },
   }),
   { name: '@modern-js/plugin-proxy' },

package/src/utils/macCAManager.ts

@@ -41,7 +41,7 @@ export const generateRootCA = () =>
     const stream = fs.createWriteStream(defaultRootCA);
 
     http
-      .get('http://127.0.0.1:8899/cgi-bin/rootca', response => {
+      .get('http://localhost:8899/cgi-bin/rootca', response => {
         response.pipe(stream);
         stream
           .on('finish', () => {

package/src/utils/macProxyManager.ts

@@ -19,7 +19,7 @@ const getNetworkType = () => {
 export const enableGlobalProxy = (ip: string, port: string) => {
   const networkType = getNetworkType();
 
-  // && networksetup -setproxybypassdomains ${networkType} 127.0.0.1 localhost
+  // && networksetup -setproxybypassdomains ${networkType} localhost localhost
   execSync(`networksetup -setwebproxy ${networkType} ${ip} ${port}`);
   execSync(`networksetup -setsecurewebproxy ${networkType} ${ip} ${port}`);
 };

package/src/utils/whistleProxy.ts

@@ -55,13 +55,13 @@ export default class WhistleProxy {
     execSync(`${this.bin} use ${this.rule} --force`);
     await this.installRootCA();
 
-    enableGlobalProxy('127.0.0.1', this.port);
+    enableGlobalProxy('localhost', this.port);
     logger.info(`Proxy Server start on localhost:${this.port}\n`);
   }
 
   close() {
     execSync(`${this.bin} stop`);
     disableGlobalProxy();
-    logger.info(`Proxy Server has closed`);
+    logger.info(`Proxy Server has been closed`);
   }
 }

package/tests/.eslintrc.js

@@ -0,0 +1,6 @@
+module.exports = {
+  extends: ['@modern-js'],
+  parserOptions: {
+    project: require.resolve('./tsconfig.json'),
+  },
+};

package/tests/__snapshots__/cli.test.ts.snap

@@ -0,0 +1,19 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`plugin proxy test config 1`] = `Array []`;
+
+exports[`plugin proxy test schema 1`] = `
+Array [
+  Array [
+    Object {
+      "schema": Object {
+        "typeof": Array [
+          "string",
+          "object",
+        ],
+      },
+      "target": "dev.proxy",
+    },
+  ],
+]
+`;

package/tests/cli.test.ts

@@ -0,0 +1,39 @@
+import path from 'path';
+import { manager } from '@modern-js/core';
+import plugin from '../src';
+
+const root = path.normalize(path.resolve(__dirname, '../../../../'));
+expect.addSnapshotSerializer({
+  test: val =>
+    typeof val === 'string' &&
+    (val.includes('modern.js') ||
+      val.includes('node_modules') ||
+      val.includes(root)),
+  print: val =>
+    // eslint-disable-next-line no-nested-ternary
+    typeof val === 'string'
+      ? // eslint-disable-next-line no-nested-ternary
+        val.includes('node_modules')
+        ? `"${val.replace(/.+node_modules/, '')}"`
+        : val.includes('modern.js')
+        ? `"${val.replace(/.+modern\.js/, '')}"`
+        : `"${val.replace(root, '')}"`
+      : (val as string),
+});
+
+describe('plugin proxy test', () => {
+  it('schema', async () => {
+    const main = manager.clone().usePlugin(plugin);
+    const runner = await main.init();
+    const result = await runner.validateSchema();
+
+    expect(result).toMatchSnapshot();
+  });
+
+  it('config', async () => {
+    const main = manager.clone().usePlugin(plugin);
+    const runner = await main.init();
+    const result = await runner.config();
+    expect(result).toMatchSnapshot();
+  });
+});

package/tests/tsconfig.json

@@ -0,0 +1,14 @@
+{
+  "extends": "@modern-js/tsconfig/base",
+  "compilerOptions": {
+    "declaration": true,
+    "jsx": "preserve",
+    "baseUrl": ".",
+    "noEmit": true,
+    "skipLibCheck": true,
+    "esModuleInterop": true,
+    "isolatedModules": true,
+    "paths": {},
+    "types": ["node", "jest"]
+  }
+}

package/tsconfig.json

@@ -5,9 +5,7 @@
     "jsx": "preserve",
     "baseUrl": "./",
     "isolatedModules": true,
-    "paths": {
-      "@/*": ["./src/*"]
-    }
+    "paths": {}
   },
   "include": ["src"]
 }