@ndiinginc/router 4.9.0 → 4.10.0

package/README.md

@@ -367,18 +367,6 @@ const server = app.listen(0); // port acak
 console.log(server.address().port);
 ```
 
-## Benchmark
-
-Hello world dengan satu route `GET /`, diuji memakai `autocannon -c 100 -d 40 -p 10` di mesin lokal.
-
-| | Req/sec (avg) | Latency (avg) |
-| --- | --- | --- |
-| Fastify | 31.475 | 31,27 ms |
-| **@ndiinginc/router** | 29.735 | 33,12 ms |
-| Express | 20.211 | 48,93 ms |
-
-Hasil bervariasi tergantung mesin, versi Node.js, dan jumlah route. Benchmark ini hanya menggambarkan overhead dasar, bukan performa aplikasi nyata.
-
 ## Testing
 
 ```bash

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ndiinginc/router",
-  "version": "4.9.0",
+  "version": "4.10.0",
   "description": "",
   "main": "./src/index.js",
   "scripts": {
@@ -21,6 +21,7 @@
     "@ndiinginc/commit": "^1.0.1",
     "@ndiinginc/test": "^1.17.0",
     "@types/jest": "^30.0.0",
-    "@types/node": "^26.6.5"
+    "@types/node": "^26.6.5",
+    "autocannon": "^8.0.0"
   }
 }

package/src/app.js

@@ -45,13 +45,16 @@ class App extends Router {
                 res.statusCode = 404;
                 err = new Error(http.STATUS_CODES[res.statusCode]);
             }
+
             if (res.statusCode < 400) {
                 res.statusCode = 500;
             }
+
             const error = { message: err.message };
             if (process.env.NODE_ENV === "development") {
                 error.stack = err.stack;
             }
+
             res.json({ error });
         }
     }
@@ -72,6 +75,7 @@ class App extends Router {
                     await handler(req, res);
                 }
             };
+
             if (this.requestTimeout) {
                 await this.withTimeout(req, res, handlerPromise);
             } else {
@@ -79,6 +83,7 @@ class App extends Router {
             }
         } catch (error) {
             err = error;
+            
             try {
                 for (const handler of errorHandlers) {
                     if (res.writableEnded) break;

package/src/polyfill.js

@@ -1,3 +1,4 @@
+
 // Polyfill for Map.prototype.getOrInsert
 if (!Map.prototype.getOrInsert) {
     Map.prototype.getOrInsert = function (key, defaultValue) {

package/src/request.js

@@ -21,6 +21,7 @@ class Request extends http.IncomingMessage {
         if (!this._query) {
             this._query = parseQueryString(this._queryString);
         }
+
         return this._query;
     }
 
@@ -31,6 +32,7 @@ class Request extends http.IncomingMessage {
         if (!this._controller) {
             this._controller = new AbortController();
         }
+        
         return this._controller;
     }
 }

package/src/response.js

@@ -5,10 +5,12 @@ class Response extends http.ServerResponse {
         this.statusCode = code;
         return this;
     }
+
     send(chunk) {
         if (this.writableEnded) return;
         this.end(chunk, null, null);
     }
+    
     json(value) {
         if (!this.headersSent) {
             this.setHeader("Content-Type", "application/json; charset=utf-8");

package/src/router.js

@@ -1,14 +1,11 @@
 const { decodeParam } = require("./util");
 const { Node } = require("./node");
-const { Cache } = require("./cache");
 
 class Router {
     constructor(options = {}) {
         this.requestTimeout = options.requestTimeout ?? 0;
-        this.cacheSize = options.cacheSize ?? 1000;
         this.root = new Node();
         this.seq = 0;
-        this.cache = new Cache(this.cacheSize);
     }
 
     split(path) {
@@ -16,7 +13,6 @@ class Router {
     }
 
     add(method, path, ...handlers) {
-        this.cache.clear();
 
         if (typeof path === "function") {
             handlers = [path, ...handlers];
@@ -178,10 +174,6 @@ class Router {
     }
 
     find(method, path) {
-        const key = `${method} ${path}`;
-
-        const cached = this.cache.get(key);
-        if (cached) return cached;
 
         const segments = this.split(path);
 
@@ -208,19 +200,13 @@ class Router {
             this.collectHandler(leaf, method, handlers);
         }
 
-        Object.freeze(finalParams);
-
-        const result = {
+        return {
             matched: res.matched,
             params: finalParams,
             paramsLength: Object.keys(finalParams).length,
             handlers: Object.values(handlers),
             errorHandlers: Object.values(errorHandlers),
         };
-
-        this.cache.set(key, result);
-
-        return result;
     }
 }
 exports.Router = Router;

package/src/util.js

@@ -1,4 +1,5 @@
-const EMPTY_PARAMS = Object.freeze(Object.create(null));
+// const EMPTY_PARAMS = Object.freeze(Object.create(null));
+const EMPTY_PARAMS = Object.create(null)
 exports.EMPTY_PARAMS = EMPTY_PARAMS;
 
 function decodeParam(raw) {
@@ -12,6 +13,7 @@ exports.decodeParam = decodeParam;
 
 function decodeQuery(raw) {
     const normalized = raw.replace(/\+/g, " ");
+
     try {
         return decodeURIComponent(normalized);
     } catch (error) {
@@ -21,18 +23,23 @@ function decodeQuery(raw) {
 
 function parseQueryString(queryString) {
     if (queryString === "") return Object.create(null);
+
     const regex = /([^=&]*)=([^&]*)/g;
-    let match;
     const result = Object.create(null);
+
+    let match;
     while ((match = regex.exec(queryString))) {
         const [, rawName, rawValue] = match;
+
         const name = decodeQuery(rawName);
         const value = decodeQuery(rawValue);
+
         if (Object.hasOwn(result, name)) {
             if (Array.isArray(result[name])) result[name].push(value);
             else result[name] = [result[name], value];
         } else result[name] = value;
     }
+
     return result;
 }
 exports.parseQueryString = parseQueryString;
@@ -41,8 +48,10 @@ function parseUrl(url) {
     if (!(url.includes("?") || url.includes("#"))) {
         return { path: url, queryString: "" };
     }
+
     const regex = /^([^?#]*)(\?([^#]*))?(#.*)?$/;
     const [, path = "/", , queryString = ""] = regex.exec(url);
+    
     return { path, queryString };
 }
 exports.parseUrl = parseUrl;

package/src/cache.js

@@ -1,28 +0,0 @@
-class Cache {
-    constructor(size = 0) {
-        this.size = size;
-        this.map = new Map();
-    }
-
-    get(key) {
-        if (!this.map.has(key)) return undefined;
-        const value = this.map.get(key);
-        this.map.delete(key);
-        this.map.set(key, value);
-        return value;
-    }
-
-    set(key, value) {
-        if (this.size <= 0) return; // disabled
-        if (this.map.has(key)) this.map.delete(key);
-        else if (this.map.size >= this.size) {
-            this.map.delete(this.map.keys().next().value);
-        }
-        this.map.set(key, value);
-    }
-
-    clear() {
-        this.map.clear();
-    }
-}
-exports.Cache = Cache;