chore: minor improvements (#1185)

Author: chimurai

.devcontainer/devcontainer.json

@@ -43,7 +43,7 @@
   // },
 
   // Use 'postCreateCommand' to run commands after the container is created.
-  "postCreateCommand": "yarn install"
+  "postCreateCommand": "yarn install && yarn patch-package"
 
   // Uncomment to connect as root instead. More info: https://aka.ms/dev-containers-non-root.
   // "remoteUser": "root"

README.md

@@ -450,13 +450,12 @@ The following options are provided by the underlying [http-proxy](https://github
 - **option.buffer**: stream of data to send as the request body. Maybe you have some middleware that consumes the request stream before proxying it on e.g. If you read the body of a request into a field called 'req.rawbody' you could restream this field in the buffer option:
 
   ```javascript
-  'use strict';
+  import httpProxy from 'http-proxy';
+  import streamify from 'stream-array';
 
-  const streamify = require('stream-array');
-  const HttpProxy = require('http-proxy');
-  const proxy = new HttpProxy();
+  const proxy = httpProxy.createProxyServer();
 
-  module.exports = (req, res, next) => {
+  export default function proxyWithBody(req, res, next) {
     proxy.web(
       req,
       res,
@@ -466,7 +465,7 @@ The following options are provided by the underlying [http-proxy](https://github
       },
       next,
     );
-  };
+  }
   ```
 
 ## WebSocket

package.json

@@ -20,10 +20,10 @@
     "install:all": "yarn && (cd examples && yarn)",
     "lint": "yarn prettier && yarn eslint",
     "lint:fix": "yarn prettier:fix && yarn eslint:fix",
-    "eslint": "eslint '{src,test,examples}/**/*.{js,ts}' --cache",
+    "eslint": "eslint --cache '**/*.{js,ts,mjs,mts}'",
     "eslint:fix": "yarn eslint --fix",
-    "prettier": "prettier --list-different \"**/*.{js,ts,md,yml,json,html}\"",
-    "prettier:fix": "prettier --write \"**/*.{js,ts,md,yml,json,html}\"",
+    "prettier": "prettier --list-different \"**/*.{js,ts,mjs,mts,md,yml,json,html}\"",
+    "prettier:fix": "prettier --write \"**/*.{js,ts,mjs,mts,md,yml,json,html}\"",
     "build": "tsc --build",
     "test": "vitest run",
     "test:types": "tsc --project tsconfig.test.json --noEmit",

test/e2e/router.spec.ts

@@ -8,7 +8,8 @@ import { createApp, createAppWithPath, createProxyMiddleware } from './test-kit.
 
 const untrustedCACert = generateCACertificate({ bits: 2048 });
 
-process.env.NODE_TLS_REJECT_UNAUTHORIZED = '0';
+// NOTE: Seems to be not needed when using `ignoreHostHttpsErrors` in mockttp, but leaving it here just in case
+// process.env.NODE_TLS_REJECT_UNAUTHORIZED = '0';
 
 describe('E2E router', () => {
   let targetServerA: Mockttp;

tsconfig.test.json

@@ -2,7 +2,8 @@
   "extends": "./tsconfig.json",
   "compilerOptions": {
     "rootDir": ".",
+    "incremental": false,
     "noEmit": true
   },
-  "include": ["./src/**/*.ts", "./test/**/*.ts"]
+  "include": ["./src/**/*.ts", "./test/**/*.ts", "./vitest.config.ts"]
 }