@eggjs/koa 2.19.1 → 2.20.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@eggjs/koa",
-  "version": "2.19.1",
+  "version": "2.20.0",
   "engines": {
     "node": ">= 18.19.0"
   },
@@ -30,18 +30,18 @@
   ],
   "license": "MIT",
   "dependencies": {
-    "accepts": "^1.3.5",
+    "accepts": "^1.3.8",
     "cache-content-type": "^2.0.0",
     "content-disposition": "~0.5.4",
     "content-type": "^1.0.5",
-    "cookies": "~0.8.0",
+    "cookies": "^0.9.1",
     "delegates": "^1.0.0",
     "destroy": "^1.0.4",
     "encodeurl": "^1.0.2",
     "escape-html": "^1.0.3",
     "fresh": "~0.5.2",
     "gals": "^1.0.1",
-    "http-assert": "^1.3.0",
+    "http-assert": "^1.5.0",
     "http-errors": "^2.0.0",
     "is-type-of": "^2.1.0",
     "koa-compose": "^4.1.0",
@@ -52,14 +52,18 @@
     "vary": "^1.1.2"
   },
   "devDependencies": {
-    "@arethetypeswrong/cli": "^0.15.3",
+    "@arethetypeswrong/cli": "^0.17.1",
     "@eggjs/tsconfig": "^1.3.3",
+    "@types/accepts": "^1.3.7",
+    "@types/content-disposition": "^0.5.8",
     "@types/content-type": "^1.1.8",
+    "@types/cookies": "^0.9.0",
     "@types/delegates": "^1.0.3",
     "@types/destroy": "^1.0.3",
     "@types/encodeurl": "^1.0.2",
     "@types/escape-html": "^1.0.4",
     "@types/fresh": "^0.5.2",
+    "@types/http-assert": "^1.5.6",
     "@types/http-errors": "^2.0.4",
     "@types/koa-compose": "^3.2.8",
     "@types/mocha": "^10.0.1",
@@ -76,7 +80,7 @@
     "mm": "^3.3.0",
     "supertest": "^3.1.0",
     "tsd": "^0.31.0",
-    "tshy": "^1.15.1",
+    "tshy": "3",
     "tshy-after": "^1.0.0",
     "typescript": "^5.4.5"
   },
@@ -91,12 +95,10 @@
     "./package.json": "./package.json",
     ".": {
       "import": {
-        "source": "./src/index.ts",
         "types": "./dist/esm/index.d.ts",
         "default": "./dist/esm/index.js"
       },
       "require": {
-        "source": "./src/index.ts",
         "types": "./dist/commonjs/index.d.ts",
         "default": "./dist/commonjs/index.js"
       }

package/src/application.ts

@@ -17,7 +17,7 @@ import { Response } from './response.js';
 import type { ContextDelegation } from './context.js';
 import type { CustomError, AnyProto } from './types.js';
 
-const debug = debuglog('@eggjs/koa:application');
+const debug = debuglog('@eggjs/koa/application');
 
 export type ProtoImplClass<T = object> = new(...args: any[]) => T;
 export type Next = () => Promise<void>;
@@ -29,6 +29,7 @@ export type MiddlewareFunc = _MiddlewareFunc & { _name?: string };
  * Inherits from `Emitter.prototype`.
  */
 export class Application extends Emitter {
+  [key: symbol]: unknown;
   /**
    * Make HttpError available to consumers of the library so that consumers don't
    * have a direct dependency upon `http-errors`

package/src/context.ts

@@ -11,6 +11,7 @@ import type { Response } from './response.js';
 import type { CustomError, AnyProto } from './types.js';
 
 export class Context {
+  [key: symbol]: unknown;
   app: Application;
   req: IncomingMessage;
   res: ServerResponse;
@@ -201,19 +202,20 @@ export class Context {
     res.end(msg);
   }
 
-  #cookies: Cookies;
+  protected _cookies: Cookies | undefined;
+
   get cookies() {
-    if (!this.#cookies) {
-      this.#cookies = new Cookies(this.req, this.res, {
+    if (!this._cookies) {
+      this._cookies = new Cookies(this.req, this.res, {
         keys: this.app.keys,
         secure: this.request.secure,
       });
     }
-    return this.#cookies;
+    return this._cookies;
   }
 
   set cookies(cookies: Cookies) {
-    this.#cookies = cookies;
+    this._cookies = cookies;
   }
 }
 

package/src/request.ts

@@ -1,11 +1,9 @@
-import net from 'node:net';
-import type { Socket } from 'node:net';
+import net, { type Socket } from 'node:net';
 import { format as stringify } from 'node:url';
-import qs from 'node:querystring';
+import qs, { type ParsedUrlQuery } from 'node:querystring';
 import util from 'node:util';
-import type { ParsedUrlQuery } from 'node:querystring';
 import type { IncomingMessage, ServerResponse } from 'node:http';
-import accepts from 'accepts';
+import accepts, { type Accepts } from 'accepts';
 import contentType from 'content-type';
 import parse from 'parseurl';
 import typeis from 'type-is';
@@ -15,6 +13,7 @@ import type { ContextDelegation } from './context.js';
 import type { Response } from './response.js';
 
 export class Request {
+  [key: symbol]: unknown;
   app: Application;
   req: IncomingMessage;
   res: ServerResponse;
@@ -134,19 +133,19 @@ export class Request {
     this.url = stringify(url);
   }
 
-  #parsedUrlQueryCache: Record<string, ParsedUrlQuery>;
+  protected _parsedUrlQueryCache: Record<string, ParsedUrlQuery> | undefined;
 
   /**
    * Get parsed query string.
    */
   get query() {
     const str = this.querystring;
-    if (!this.#parsedUrlQueryCache) {
-      this.#parsedUrlQueryCache = {};
+    if (!this._parsedUrlQueryCache) {
+      this._parsedUrlQueryCache = {};
     }
-    let parsedUrlQuery = this.#parsedUrlQueryCache[str];
+    let parsedUrlQuery = this._parsedUrlQueryCache[str];
     if (!parsedUrlQuery) {
-      parsedUrlQuery = this.#parsedUrlQueryCache[str] = qs.parse(str);
+      parsedUrlQuery = this._parsedUrlQueryCache[str] = qs.parse(str);
     }
     return parsedUrlQuery;
   }
@@ -210,10 +209,16 @@ export class Request {
     const proxy = this.app.proxy;
     let host = proxy ? this.get<string>('X-Forwarded-Host') : '';
     if (!host) {
-      if (this.req.httpVersionMajor >= 2) host = this.get(':authority');
-      if (!host) host = this.get('Host');
+      if (this.req.httpVersionMajor >= 2) {
+        host = this.get(':authority');
+      }
+      if (!host) {
+        host = this.get('Host');
+      }
+    }
+    if (!host) {
+      return '';
     }
-    if (!host) return '';
     return host.split(/\s*,\s*/, 1)[0];
   }
 
@@ -224,27 +229,31 @@ export class Request {
    */
   get hostname() {
     const host = this.host;
-    if (!host) return '';
-    if (host[0] === '[') return this.URL.hostname || ''; // IPv6
+    if (!host) {
+      return '';
+    }
+    if (host[0] === '[') {
+      return this.URL.hostname || ''; // IPv6
+    }
     return host.split(':', 1)[0];
   }
 
-  #memoizedURL: URL;
+  protected _memoizedURL: URL | undefined;
 
   /**
    * Get WHATWG parsed URL.
    * Lazily memoized.
    */
   get URL() {
-    if (!this.#memoizedURL) {
+    if (!this._memoizedURL) {
       const originalUrl = this.originalUrl || ''; // avoid undefined in template string
       try {
-        this.#memoizedURL = new URL(`${this.origin}${originalUrl}`);
+        this._memoizedURL = new URL(`${this.origin}${originalUrl}`);
       } catch {
-        this.#memoizedURL = Object.create(null);
+        this._memoizedURL = Object.create(null);
       }
     }
-    return this.#memoizedURL;
+    return this._memoizedURL!;
   }
 
   /**
@@ -257,7 +266,9 @@ export class Request {
     const status = this.response.status;
 
     // GET or HEAD for weak freshness validation only
-    if (method !== 'GET' && method !== 'HEAD') return false;
+    if (method !== 'GET' && method !== 'HEAD') {
+      return false;
+    }
 
     // 2xx or 304 as per rfc2616 14.26
     if ((status >= 200 && status < 300) || status === 304) {
@@ -308,7 +319,9 @@ export class Request {
    */
   get length() {
     const len = this.get<string>('Content-Length');
-    if (len === '') return;
+    if (len === '') {
+      return;
+    }
     return parseInt(len);
   }
 
@@ -321,8 +334,12 @@ export class Request {
    * may be enabled.
    */
   get protocol() {
-    if (this.socket.encrypted) return 'https';
-    if (!this.app.proxy) return 'http';
+    if (this.socket.encrypted) {
+      return 'https';
+    }
+    if (!this.app.proxy) {
+      return 'http';
+    }
     const proto = this.get<string>('X-Forwarded-Proto');
     return proto ? proto.split(/\s*,\s*/, 1)[0] : 'http';
   }
@@ -356,21 +373,21 @@ export class Request {
     return ips;
   }
 
-  #ip: string;
+  protected _ip: string;
   /**
    * Return request's remote address
    * When `app.proxy` is `true`, parse
    * the "X-Forwarded-For" ip address list and return the first one
    */
   get ip() {
-    if (!this.#ip) {
-      this.#ip = this.ips[0] || this.socket.remoteAddress || '';
+    if (!this._ip) {
+      this._ip = this.ips[0] || this.socket.remoteAddress || '';
     }
-    return this.#ip;
+    return this._ip;
   }
 
   set ip(ip: string) {
-    this.#ip = ip;
+    this._ip = ip;
   }
 
   /**
@@ -395,20 +412,20 @@ export class Request {
       .slice(offset);
   }
 
-  #accept: any;
+  protected _accept: Accepts;
   /**
    * Get accept object.
    * Lazily memoized.
    */
   get accept() {
-    return this.#accept || (this.#accept = accepts(this.req));
+    return this._accept || (this._accept = accepts(this.req));
   }
 
   /**
    * Set accept object.
    */
-  set accept(obj) {
-    this.#accept = obj;
+  set accept(obj: Accepts) {
+    this._accept = obj;
   }
 
   /**
@@ -459,8 +476,19 @@ export class Request {
    *
    *     ['gzip', 'deflate']
    */
-  acceptsEncodings(...args: any[]): string | string[] {
-    return this.accept.encodings(...args);
+  acceptsEncodings(): string[];
+  acceptsEncodings(encodings: string[]): string | false;
+  acceptsEncodings(...encodings: string[]): string | false;
+  acceptsEncodings(encodings?: string | string[], ...others: string[]): string[] | string | false {
+    if (!encodings) {
+      return this.accept.encodings();
+    }
+    if (Array.isArray(encodings)) {
+      encodings = [ ...encodings, ...others ];
+    } else {
+      encodings = [ encodings, ...others ];
+    }
+    return this.accept.encodings(...encodings);
   }
 
   /**
@@ -471,8 +499,19 @@ export class Request {
    *
    *     ['utf-8', 'utf-7', 'iso-8859-1']
    */
-  acceptsCharsets(...args: any[]): string | string[] {
-    return this.accept.charsets(...args);
+  acceptsCharsets(): string[];
+  acceptsCharsets(charsets: string[]): string | false;
+  acceptsCharsets(...charsets: string[]): string | false;
+  acceptsCharsets(charsets?: string | string[], ...others: string[]): string[] | string | false {
+    if (!charsets) {
+      return this.accept.charsets();
+    }
+    if (Array.isArray(charsets)) {
+      charsets = [ ...charsets, ...others ];
+    } else {
+      charsets = [ charsets, ...others ];
+    }
+    return this.accept.charsets(...charsets);
   }
 
   /**
@@ -483,8 +522,19 @@ export class Request {
    *
    *     ['es', 'pt', 'en']
    */
-  acceptsLanguages(...args: any[]): string | string[] {
-    return this.accept.languages(...args);
+  acceptsLanguages(): string[];
+  acceptsLanguages(languages: string[]): string | false;
+  acceptsLanguages(...languages: string[]): string | false;
+  acceptsLanguages(languages?: string | string[], ...others: string[]): string | string[] | false {
+    if (!languages) {
+      return this.accept.languages();
+    }
+    if (Array.isArray(languages)) {
+      languages = [ ...languages, ...others ];
+    } else {
+      languages = [ languages, ...others ];
+    }
+    return this.accept.languages(...languages);
   }
 
   /**

package/src/response.ts

@@ -4,7 +4,7 @@ import util from 'node:util';
 import Stream from 'node:stream';
 import type { IncomingMessage, ServerResponse } from 'node:http';
 import contentDisposition from 'content-disposition';
-import getType from 'cache-content-type';
+import { getType } from 'cache-content-type';
 import onFinish from 'on-finished';
 import escape from 'escape-html';
 import { is as typeis } from 'type-is';
@@ -17,6 +17,7 @@ import type { ContextDelegation } from './context.js';
 import type { Request } from './request.js';
 
 export class Response {
+  [key: symbol]: unknown;
   app: Application;
   req: IncomingMessage;
   res: ServerResponse;