inspect: 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: nodejs#50909
Ch3nYuY · Mar 18, 2024 · baf1b8c · baf1b8c
1 parent 454d080
commit baf1b8c
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,