inspector: correct %s format behavior with Symbol.toPrimitive

Ensure console.log("%s", obj) correctly invokes obj[Symbol.toPrimitive] for string conversion, fixing unexpected object display issue. Fixes: #50909
nodejs · Mar 18, 2024 · 79df127 · 79df127
1 parent 454d080
commit 79df127
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/lib/internal/util/inspect.js b/lib/internal/util/inspect.js
@@ -151,6 +151,7 @@ const {
 const assert = require('internal/assert');
 
 const { BuiltinModule } = require('internal/bootstrap/realm');
+const { SymbolToPrimitive } = primordials;
 const {
  validateObject,
  validateString,
@@ -2204,6 +2205,10 @@ function formatWithOptionsInternal(inspectOptions, args) {
  tempArg === null ||
  !hasBuiltInToString(tempArg)) {
  tempStr = String(tempArg);
+ } else if (hasBuiltInToString(tempArg) &&
+ getProxyDetails(tempArg, false) === undefined &&
+ tempArg[SymbolToPrimitive] !== undefined) {
+ tempStr = tempArg[SymbolToPrimitive]('string');
  } else {
  tempStr = inspect(tempArg, {
  ...inspectOptions,