module: print amount of load time of a cjs module

Author: H4ad

lib/internal/modules/cjs/loader.js

@@ -359,7 +359,19 @@ function setModuleParent(value) {
   moduleParentCache.set(this, value);
 }
 
-let debug = require('internal/util/debuglog').debuglog('module', (fn) => {
+const { debuglog, isDebugEnabled } = require('internal/util/debuglog');
+
+let debug = debuglog('module', (fn) => {
+  debug = fn;
+});
+
+let hrtimeBigIntTimingFn = () => {
+  hrtimeBigIntTimingFn = isDebugEnabled('timing_module_cjs') ? process.hrtime.bigint : () => 0
+
+  return hrtimeBigIntTimingFn()
+};
+
+let debugTiming = debuglog('timing_module_cjs', (fn) => {
   debug = fn;
 });
 
@@ -958,6 +970,10 @@ function getExportsForCircularRequire(module) {
   return module.exports;
 }
 
+function logTiming(request, parent, start) {
+  debugTiming('[%s] [%s]: %d ms', parent?.id || '', request, Number((hrtimeBigIntTimingFn() - start)) / 1e6);
+}
+
 /**
  * Load a module from cache if it exists, otherwise create a new module instance.
  * 1. If a module already exists in the cache: return its exports object.
@@ -966,10 +982,12 @@ function getExportsForCircularRequire(module) {
  * 3. Otherwise, create a new module for the file and save it to the cache.
  *    Then have it load the file contents before returning its exports object.
  * @param {string} request Specifier of module to load via `require`
- * @param {string} parent Absolute path of the module importing the child
+ * @param {Module} parent Absolute path of the module importing the child
  * @param {boolean} isMain Whether the module is the main entry point
  */
 Module._load = function(request, parent, isMain) {
+  const start = hrtimeBigIntTimingFn();
+
   let relResolveCacheIdentifier;
   if (parent) {
     debug('Module._load REQUEST %s parent: %s', request, parent.id);
@@ -984,8 +1002,14 @@ Module._load = function(request, parent, isMain) {
       if (cachedModule !== undefined) {
         updateChildren(parent, cachedModule, true);
         if (!cachedModule.loaded) {
-          return getExportsForCircularRequire(cachedModule);
+          const result = getExportsForCircularRequire(cachedModule);
+
+          logTiming(request, parent, start);
+
+          return result;
         }
+
+        logTiming(request, parent, start);
         return cachedModule.exports;
       }
       delete relativeResolveCache[relResolveCacheIdentifier];
@@ -1001,6 +1025,8 @@ Module._load = function(request, parent, isMain) {
     }
 
     const module = loadBuiltinModule(id, request);
+
+    logTiming(request, parent, start);
     return module.exports;
   }
 
@@ -1011,16 +1037,24 @@ Module._load = function(request, parent, isMain) {
     if (!cachedModule.loaded) {
       const parseCachedModule = cjsSourceCache.get(cachedModule);
       if (!parseCachedModule || parseCachedModule.loaded) {
-        return getExportsForCircularRequire(cachedModule);
+        const result = getExportsForCircularRequire(cachedModule);
+
+        logTiming(request, parent, start);
+
+        return result;
       }
       parseCachedModule.loaded = true;
     } else {
+      logTiming(request, parent, start);
       return cachedModule.exports;
     }
   }
 
   if (BuiltinModule.canBeRequiredWithoutScheme(filename)) {
     const mod = loadBuiltinModule(filename, request);
+
+    logTiming(request, parent, start);
+
     return mod.exports;
   }
 
@@ -1068,6 +1102,8 @@ Module._load = function(request, parent, isMain) {
     }
   }
 
+  logTiming(request, parent, start);
+
   return module.exports;
 };
 

lib/internal/util/debuglog.js

@@ -115,7 +115,12 @@ function debuglog(set, cb) {
   return logger;
 }
 
+function isDebugEnabled(set) {
+  return testEnabled(set);
+}
+
 module.exports = {
   debuglog,
+  isDebugEnabled,
   initializeDebugEnv,
 };

test/parallel/test-module-print-timing.mjs

@@ -0,0 +1,24 @@
+import { spawnPromisified } from '../common/index.mjs';
+import assert from 'node:assert';
+import { execPath } from 'node:process';
+import { it } from 'node:test';
+
+it('should print the timing information for cjs', async () => {
+  process.env.NODE_DEBUG = 'timing_module_cjs';
+  const result = await spawnPromisified(execPath, ['--eval', 'require("url");'], {
+    env: {
+      NODE_DEBUG: 'timing_module_cjs',
+    },
+  });
+
+  assert.strictEqual(result.code, 0);
+  assert.strictEqual(result.signal, null);
+  assert.strictEqual(result.stdout, '');
+
+  const firstLine = result.stderr.split('\n').find(line => line.includes('[url]'));
+
+  assert.notStrictEqual(firstLine, undefined);
+  assert.ok(firstLine.includes('TIMING_MODULE_CJS'), `Not found TIMING_MODULE_CJS on ${firstLine}`);
+  assert.ok(firstLine.includes('[url]:'), `Not found [url]: on ${firstLine}`);
+  assert.ok(firstLine.endsWith('ms'), `Not found ms on ${firstLine}`);
+});