@aimeloic/monkey-tester 2.0.6 → 2.0.8

package/htmlTemplate.js

@@ -296,14 +296,20 @@ function getHtmlTemplate(endpoints) {
     let body = undefined;
     if (['POST', 'PUT', 'PATCH'].includes(ep.method) && ep.fields && ep.fields.length) {
       const payload = {};
+      let hasData = false;
+      
       ep.fields.forEach(function(f) {
         const el = document.getElementById('field-' + f.name);
         if (!el) return;
         let v = el.value.trim();
-        if (f.type === 'number' && v !== '') v = Number(v);
+        if (v === '') return; // Avoid submitting empty strings for unfilled fields
+        
+        if (f.type === 'number') v = Number(v);
         payload[f.name] = v;
+        hasData = true;
       });
-      body = JSON.stringify(payload);
+      
+      if (hasData) body = JSON.stringify(payload);
     }
 
     setResponse(null, 'loading');

package/index.js

@@ -2,4 +2,5 @@
 
 const { endtesterExpress } = require('./monkey.js');
 
-module.exports = { endtesterExpress };
+module.exports = { endtesterExpress };
+module.exports.endtesterExpress = endtesterExpress;

package/monkey.js

@@ -41,7 +41,6 @@ function extractBodyFields(handler) {
     let m;
     while ((m = destructRe.exec(source)) !== null) {
       m[1].split(',').forEach(part => {
-        // handle rename (email: userEmail) and default (name = '')
         const name = part.split(':')[0].split('=')[0].trim();
         if (name && /^[a-zA-Z_$][a-zA-Z0-9_$]*$/.test(name) && !seen.has(name)) {
           seen.set(name, buildField(name));
@@ -84,25 +83,22 @@ function fallbackFields(path) {
   return [];
 }
 
-// ─── Extract router prefix from Express layer regexp ─────────────────────────
+// ─── Extract router prefix safely from Express layer ─────────────────────────
 function extractRouterPrefix(layer) {
   if (!layer.regexp) return '';
-  const src = layer.regexp.source;
-
-  // Express serialises router paths as: ^\/<prefix>(?:\/(?=$))?(?=\/|$)
-  const patterns = [
-    /^\^\\\/([^\\?$]+)/,          // common format
-    /^\^\\\/([a-zA-Z0-9_/-]+)/,  // simple paths
-  ];
-  for (const re of patterns) {
-    const m = re.exec(src);
-    if (m && m[1]) {
-      return '/' + m[1].replace(/\\\//g, '/').replace(/\\/g, '');
-    }
-  }
-  // Fallback: check .regexp property stored by Express (Express 5 style)
-  if (layer.regexp && layer.regexp.fast_slash) return '';
-  return '';
+  if (layer.path) return layer.path; // If explicit paths exist
+
+  let src = layer.regexp.source;
+  
+  // Strip out boilerplate generated patterns from default express mounting structures
+  src = src
+    .replace('^\\/', '/')
+    .replace('\\/?(?=\\/|$)', '')
+    .replace('(?:\\/(?=$))?(?=\\/|$)', '')
+    .replace('\\/', '/');
+
+  const cleanPath = src.split('(?')[0].replace(/\\/g, '');
+  return cleanPath === '/' ? '' : cleanPath;
 }
 
 // ─── Walk the Express router stack recursively ────────────────────────────────
@@ -118,7 +114,6 @@ function parseStack(stack, detectedEndpoints, prefix = '') {
 
       const fullPath = (prefix + rawPath).replace(/\/+/g, '/') || '/';
 
-      // skip the tester's own route
       if (fullPath.startsWith('/api/tester')) continue;
 
       const methods = Object.keys(layer.route.methods || {});
@@ -127,11 +122,12 @@ function parseStack(stack, detectedEndpoints, prefix = '') {
         const httpMethod = method.toUpperCase();
         const key = `${httpMethod}::${fullPath}`;
 
-        // ── Path params (:id, :slug …) ─────────────────────────────────────
+        // ── Path params (:id, :slug …) safely parsed using matchAll ─────────
         const pathParams = [];
         const paramRe = /:([a-zA-Z_$][a-zA-Z0-9_$]*)/g;
-        let pm;
-        while ((pm = paramRe.exec(fullPath)) !== null) {
+        const matches = [...fullPath.matchAll(paramRe)];
+
+        for (const pm of matches) {
           pathParams.push({
             name: pm[1],
             label: pm[1].charAt(0).toUpperCase() + pm[1].slice(1),
@@ -146,14 +142,12 @@ function parseStack(stack, detectedEndpoints, prefix = '') {
           for (const handler of handlers) {
             bodyFields.push(...extractBodyFields(handler));
           }
-          // deduplicate by name
           const seen = new Map();
           bodyFields = bodyFields.filter(f => {
             if (seen.has(f.name)) return false;
             seen.set(f.name, true);
             return true;
           });
-          // fallback if extraction produced nothing
           if (bodyFields.length === 0) {
             bodyFields = fallbackFields(fullPath);
           }
@@ -190,7 +184,7 @@ function endtesterExpress() {
 
     const rootStack =
       (app._router && app._router.stack) ||   // Express 4
-      (app.router  && app.router.stack)  ||   // Express 5 preview
+      (app.router  && app.router.stack)  ||   // Express 5
       [];
 
     parseStack(rootStack, detectedEndpoints);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aimeloic/monkey-tester",
-  "version": "2.0.6",
+  "version": "2.0.8",
   "description": "Auto route scanning visual runner dashboard.",
   "main": "index.js",
   "type": "module",