fix: replace npmlog with simple inline logger

Julusian · Julusian · commit 818bece26820 · 2026-01-27T19:01:36.000Z
diff --git a/bin/cmake-js b/bin/cmake-js
@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 'use strict'
 
-const log = require('npmlog')
+const log = require('../lib/logger')
 const BuildSystem = require('../').BuildSystem
 const util = require('util')
 const version = require('../package').version
@@ -186,7 +186,6 @@ if (argv.h) {
 
 if (argv.l && logLevels.includes(argv.l)) {
 	log.level = argv.l
-	log.resume()
 }
 
 log.silly('CON', 'argv:')
diff --git a/lib/cmLog.js b/lib/cmLog.js
@@ -1,5 +1,5 @@
 'use strict'
-const log = require('npmlog')
+const log = require('./logger')
 
 class CMLog {
 	get level() {
diff --git a/lib/import/find-visualstudio.js b/lib/import/find-visualstudio.js
@@ -1,6 +1,6 @@
 'use strict'
 
-const log = require('npmlog')
+const log = require('../logger')
 const { existsSync } = require('fs')
 const { win32: path } = require('path')
 const { regSearchKeys, execFile, logWithPrefix } = require('./util')
diff --git a/lib/import/util.js b/lib/import/util.js
@@ -1,6 +1,6 @@
 'use strict'
 
-const log = require('npmlog')
+const log = require('../logger')
 const cp = require('child_process')
 const path = require('path')
 
diff --git a/lib/logger.js b/lib/logger.js
@@ -0,0 +1,59 @@
+const util = require('util')
+
+const levels = {
+	silly: -Infinity,
+	verbose: 1000,
+	info: 2000,
+	http: 3000,
+	warn: 4000,
+	error: 5000,
+	silent: Infinity,
+}
+
+const colors = {
+	silly: 'inverse',
+	verbose: 'blue',
+	info: 'green',
+	http: 'green',
+	warn: 'yellow',
+	error: 'red',
+}
+
+let currentLevel = levels.info
+
+function log(level, prefix, message, ...args) {
+	if (currentLevel <= levels[level]) {
+		const stream = level === 'error' ? process.stderr : process.stdout
+		const color = colors[level]
+		let levelStr = level.toUpperCase()
+		if (process.stdout.isTTY && util.styleText) {
+			// util.styleText is available in Node.js >= 20.12.0
+			levelStr = util.styleText(color, levelStr)
+			if (prefix) prefix = util.styleText('magenta', prefix)
+		}
+
+		const formattedMessage = util.format(message, ...args)
+		const line = prefix
+			? util.format('%s %s %s', levelStr, prefix, formattedMessage)
+			: util.format('%s %s', levelStr, formattedMessage)
+
+		stream.write(line + '\n')
+	}
+}
+
+const logger = {
+	get level() {
+		return Object.keys(levels).find((key) => levels[key] === currentLevel)
+	},
+	set level(newLevel) {
+		if (levels[newLevel] !== undefined) {
+			currentLevel = levels[newLevel]
+		}
+	},
+}
+
+for (const level of Object.keys(colors)) {
+	logger[level] = (prefix, message, ...args) => log(level, prefix, message, ...args)
+}
+
+module.exports = logger
diff --git a/package.json b/package.json
@@ -41,12 +41,11 @@
 		"node": "^20.17.0 || >=22.9.0"
 	},
 	"dependencies": {
-		"debug": "^4",
+		"debug": "^4.4.3",
 		"fs-extra": "^11.3.3",
 		"node-api-headers": "^1.8.0",
-		"npmlog": "^6.0.2",
-		"rc": "^1.2.7",
-		"semver": "^7.5.4",
+		"rc": "1.2.8",
+		"semver": "^7.7.3",
 		"tar": "^7.5.6",
 		"url-join": "^4.0.1",
 		"which": "^6.0.0",
diff --git a/tests/es6/buildSystem.js b/tests/es6/buildSystem.js
@@ -5,7 +5,7 @@ const lib = require('../../')
 const locateNAN = require('../../lib/locateNAN')
 const CMake = lib.CMake
 const path = require('path')
-const log = require('npmlog')
+const log = require('../../lib/logger')
 const testRunner = require('./testRunner')
 const testCases = require('./testCases')
 
@@ -15,7 +15,6 @@ describe('BuildSystem', function () {
 	before(function () {
 		if (process.env.UT_LOG_LEVEL) {
 			log.level = process.env.UT_LOG_LEVEL
-			log.resume()
 		}
 		locateNAN.__projectRoot = path.resolve(path.join(__dirname, '../../'))
 	})
diff --git a/tests/es6/testRunner.js b/tests/es6/testRunner.js
@@ -2,7 +2,7 @@
 
 const lib = require('../../')
 const environment = lib.environment
-const log = require('npmlog')
+const log = require('../../lib/logger')
 const util = require('util')
 
 function* generateRuntimeOptions() {
