@hono/node-server 1.3.2 → 1.3.4

package/dist/vercel.mjs

@@ -1,5 +1,68 @@
-// src/globals.ts
-import crypto from "crypto";
+// src/request.ts
+import { Readable } from "stream";
+var newRequestFromIncoming = (method, url, incoming) => {
+  const headerRecord = [];
+  const len = incoming.rawHeaders.length;
+  for (let i = 0; i < len; i += 2) {
+    headerRecord.push([incoming.rawHeaders[i], incoming.rawHeaders[i + 1]]);
+  }
+  const init = {
+    method,
+    headers: headerRecord
+  };
+  if (!(method === "GET" || method === "HEAD")) {
+    init.body = Readable.toWeb(incoming);
+    init.duplex = "half";
+  }
+  return new Request(url, init);
+};
+var getRequestCache = Symbol("getRequestCache");
+var requestCache = Symbol("requestCache");
+var incomingKey = Symbol("incomingKey");
+var requestPrototype = {
+  get method() {
+    return this[incomingKey].method || "GET";
+  },
+  get url() {
+    return `http://${this[incomingKey].headers.host}${this[incomingKey].url}`;
+  },
+  [getRequestCache]() {
+    return this[requestCache] ||= newRequestFromIncoming(this.method, this.url, this[incomingKey]);
+  }
+};
+[
+  "body",
+  "bodyUsed",
+  "cache",
+  "credentials",
+  "destination",
+  "headers",
+  "integrity",
+  "mode",
+  "redirect",
+  "referrer",
+  "referrerPolicy",
+  "signal"
+].forEach((k) => {
+  Object.defineProperty(requestPrototype, k, {
+    get() {
+      return this[getRequestCache]()[k];
+    }
+  });
+});
+["arrayBuffer", "blob", "clone", "formData", "json", "text"].forEach((k) => {
+  Object.defineProperty(requestPrototype, k, {
+    value: function() {
+      return this[getRequestCache]()[k]();
+    }
+  });
+});
+Object.setPrototypeOf(requestPrototype, global.Request.prototype);
+var newRequest = (incoming) => {
+  const req = Object.create(requestPrototype);
+  req[incomingKey] = incoming;
+  return req;
+};
 
 // src/utils.ts
 function writeFromReadableStream(stream, writable) {
@@ -59,31 +122,35 @@ var buildOutgoingHttpHeaders = (headers) => {
 
 // src/response.ts
 var responseCache = Symbol("responseCache");
-var newGlobalResponseKey = Symbol("newGlobalResponse");
 var cacheKey = Symbol("cache");
 var GlobalResponse = global.Response;
 var Response2 = class _Response {
   #body;
   #init;
-  [newGlobalResponseKey]() {
-    return new GlobalResponse(
-      this.#body,
-      this.#init instanceof _Response ? this.#init[newGlobalResponseKey]() : this.#init
-    );
-  }
-  // @ts-ignore
   get cache() {
     delete this[cacheKey];
-    return this[responseCache] ||= this[newGlobalResponseKey]();
+    return this[responseCache] ||= new GlobalResponse(this.#body, this.#init);
   }
   constructor(body, init) {
     this.#body = body;
-    this.#init = init;
+    if (init instanceof _Response) {
+      const cachedGlobalResponse = init[responseCache];
+      if (cachedGlobalResponse) {
+        this.#init = cachedGlobalResponse;
+        this.cache;
+        return;
+      } else {
+        this.#init = init.#init;
+      }
+    } else {
+      this.#init = init;
+    }
     if (typeof body === "string" || body instanceof ReadableStream) {
       let headers = init?.headers || { "content-type": "text/plain;charset=UTF-8" };
       if (headers instanceof Headers) {
         headers = buildOutgoingHttpHeaders(headers);
       }
+      ;
       this[cacheKey] = [init?.status || 200, body, headers];
     }
   }
@@ -120,6 +187,7 @@ Object.defineProperty(global, "Response", {
 });
 
 // src/globals.ts
+import crypto from "crypto";
 Object.defineProperty(global, "Response", {
   value: Response2
 });
@@ -137,72 +205,6 @@ global.fetch = (info, init) => {
   return webFetch(info, init);
 };
 
-// src/request.ts
-import { Readable } from "stream";
-var newRequestFromIncoming = (method, url, incoming) => {
-  const headerRecord = [];
-  const len = incoming.rawHeaders.length;
-  for (let i = 0; i < len; i += 2) {
-    headerRecord.push([incoming.rawHeaders[i], incoming.rawHeaders[i + 1]]);
-  }
-  const init = {
-    method,
-    headers: headerRecord
-  };
-  if (!(method === "GET" || method === "HEAD")) {
-    init.body = Readable.toWeb(incoming);
-    init.duplex = "half";
-  }
-  return new Request(url, init);
-};
-var getRequestCache = Symbol("getRequestCache");
-var requestCache = Symbol("requestCache");
-var incomingKey = Symbol("incomingKey");
-var requestPrototype = {
-  get method() {
-    return this[incomingKey].method || "GET";
-  },
-  get url() {
-    return `http://${this[incomingKey].headers.host}${this[incomingKey].url}`;
-  },
-  [getRequestCache]() {
-    return this[requestCache] ||= newRequestFromIncoming(this.method, this.url, this[incomingKey]);
-  }
-};
-[
-  "body",
-  "bodyUsed",
-  "cache",
-  "credentials",
-  "destination",
-  "headers",
-  "integrity",
-  "mode",
-  "redirect",
-  "referrer",
-  "referrerPolicy",
-  "signal"
-].forEach((k) => {
-  Object.defineProperty(requestPrototype, k, {
-    get() {
-      return this[getRequestCache]()[k];
-    }
-  });
-});
-["arrayBuffer", "blob", "clone", "formData", "json", "text"].forEach((k) => {
-  Object.defineProperty(requestPrototype, k, {
-    value: function() {
-      return this[getRequestCache]()[k]();
-    }
-  });
-});
-Object.setPrototypeOf(requestPrototype, global.Request.prototype);
-var newRequest = (incoming) => {
-  const req = Object.create(requestPrototype);
-  req[incomingKey] = incoming;
-  return req;
-};
-
 // src/listener.ts
 var regBuffer = /^no$/i;
 var regContentType = /^(application\/json\b|text\/(?!event-stream\b))/i;
@@ -215,6 +217,9 @@ var handleResponseError = (e, outgoing) => {
     console.info("The user aborted a request.");
   } else {
     console.error(e);
+    if (!outgoing.headersSent)
+      outgoing.writeHead(500, { "Content-Type": "text/plain" });
+    outgoing.end(`Error: ${err.message}`);
     outgoing.destroy(err);
   }
 };
@@ -235,6 +240,13 @@ var responseViaResponseObject = async (res, outgoing) => {
   if (res instanceof Promise) {
     res = await res.catch(handleFetchError);
   }
+  if (!(res instanceof Response)) {
+    return handleResponseError(
+      // @ts-expect-error the object must have `toString()`
+      new Error(`The response is not an instance of Response, but ${res.toString()}`),
+      outgoing
+    );
+  }
   if (cacheKey in res) {
     try {
       return responseViaCache(res, outgoing);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hono/node-server",
-  "version": "1.3.2",
+  "version": "1.3.4",
   "description": "Node.js Adapter for Hono",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -43,7 +43,9 @@
     "watch": "tsup --watch",
     "postbuild": "publint",
     "prerelease": "yarn build && yarn test",
-    "release": "np"
+    "release": "np",
+    "lint": "eslint --ext js,ts src test",
+    "lint:fix": "eslint --ext js,ts src test --fix"
   },
   "license": "MIT",
   "repository": {
@@ -60,10 +62,12 @@
     "node": ">=18.14.1"
   },
   "devDependencies": {
+    "@hono/eslint-config": "^0.0.2",
     "@types/jest": "^29.5.3",
     "@types/node": "^20.10.0",
     "@types/supertest": "^2.0.12",
-    "hono": "^3.9.2",
+    "eslint": "^8.55.0",
+    "hono": "^3.11.7",
     "jest": "^29.6.1",
     "np": "^7.7.0",
     "publint": "^0.1.16",