node-loop-detective 1.0.1 → 1.0.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "node-loop-detective",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "description": "Detect event loop blocking & lag in running Node.js apps without code changes or restarts",
   "main": "src/index.js",
   "bin": {

package/src/detective.js

@@ -56,7 +56,7 @@ class Detective extends EventEmitter {
    * We inject a tiny lag-measuring snippet into the target process
    */
   async _startLagDetection() {
-    // Inject a lag detector into the target process
+    // Inject a lag detector that also captures stack traces
     const script = `
       (function() {
         if (globalThis.__loopDetective) {
@@ -66,12 +66,33 @@ class Detective extends EventEmitter {
         let lastTime = Date.now();
         const threshold = ${this.config.threshold};
 
+        // Capture stack trace at the point of lag detection
+        function captureStack() {
+          const orig = Error.stackTraceLimit;
+          Error.stackTraceLimit = 20;
+          const err = new Error();
+          Error.stackTraceLimit = orig;
+          // Parse the stack into structured frames
+          const frames = (err.stack || '').split('\\n').slice(2).map(line => {
+            const m = line.match(/at\\s+(?:(.+?)\\s+\\()?(.+?):(\\d+):(\\d+)\\)?/);
+            if (m) return { fn: m[1] || '(anonymous)', file: m[2], line: +m[3], col: +m[4] };
+            const m2 = line.match(/at\\s+(.+)/);
+            if (m2) return { fn: m2[1], file: '', line: 0, col: 0 };
+            return null;
+          }).filter(Boolean).filter(f =>
+            !f.file.includes('loopDetective') &&
+            !f.fn.includes('Timeout.') &&
+            !f.file.includes('node:internal')
+          );
+          return frames;
+        }
+
         const timer = setInterval(() => {
           const now = Date.now();
           const delta = now - lastTime;
           const lag = delta - ${this.config.interval};
           if (lag > threshold) {
-            lags.push({ lag, timestamp: now });
+            lags.push({ lag, timestamp: now, stack: captureStack() });
             if (lags.length > 100) lags.shift();
           }
           lastTime = now;

package/src/reporter.js

@@ -33,6 +33,11 @@ class Reporter {
     if (this.config.json) return;
     const severity = data.lag > 500 ? COLORS.red : data.lag > 200 ? COLORS.yellow : COLORS.cyan;
     this._print(`${severity}⚠ Event loop lag: ${data.lag}ms${COLORS.reset} ${COLORS.dim}at ${new Date(data.timestamp).toISOString()}${COLORS.reset}`);
+    if (data.stack && data.stack.length > 0) {
+      for (const frame of data.stack.slice(0, 5)) {
+        this._print(`  ${COLORS.dim}  → ${frame.fn} ${frame.file}:${frame.line}:${frame.col}${COLORS.reset}`);
+      }
+    }
   }
 
   onProfile(analysis) {
@@ -143,6 +148,29 @@ class Reporter {
       this._print(`    Events: ${this.lagEvents.length}`);
       this._print(`    Max:    ${maxLag}ms`);
       this._print(`    Avg:    ${avgLag}ms`);
+
+      // Aggregate lag by code location
+      const locationMap = new Map();
+      for (const evt of this.lagEvents) {
+        if (evt.stack && evt.stack.length > 0) {
+          const top = evt.stack[0];
+          const key = `${top.fn} ${top.file}:${top.line}`;
+          const entry = locationMap.get(key) || { count: 0, totalLag: 0, maxLag: 0, frame: top };
+          entry.count++;
+          entry.totalLag += evt.lag;
+          entry.maxLag = Math.max(entry.maxLag, evt.lag);
+          locationMap.set(key, entry);
+        }
+      }
+
+      if (locationMap.size > 0) {
+        this._print(`\n    ${COLORS.bold}Lag by Code Location:${COLORS.reset}`);
+        const sorted = [...locationMap.values()].sort((a, b) => b.totalLag - a.totalLag);
+        for (const loc of sorted.slice(0, 5)) {
+          this._print(`    ${COLORS.yellow}${loc.frame.fn}${COLORS.reset} ${COLORS.dim}${loc.frame.file}:${loc.frame.line}${COLORS.reset}`);
+          this._print(`      ${loc.count} events, total ${loc.totalLag}ms, max ${loc.maxLag}ms`);
+        }
+      }
     }
 
     this._print(`\n${'─'.repeat(60)}\n`);