util: mark proxied objects as such when inspecting them

Fixes: #60964
PR-URL: #61029
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: LiviaMedeiros <livia@cirno.name>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>

Author: BridgeAR

lib/internal/util/inspect.js

@@ -1180,7 +1180,13 @@ function formatValue(ctx, value, recurseTimes, typedArray) {
 returnctx.stylize(`[Circular *${index}]`,'special');
 }
 
-returnformatRaw(ctx,value,recurseTimes,typedArray);
+constformatted=formatRaw(ctx,value,recurseTimes,typedArray);
+
+if(proxy!==undefined){
+return`${ctx.stylize('Proxy(','special')}${formatted}${ctx.stylize(')','special')}`;
+}
+
+returnformatted;
 }
 
 functionformatRaw(ctx,value,recurseTimes,typedArray){

test/parallel/test-assert-deep.js

@@ -1068,7 +1068,7 @@ test('Check proxies', () => {
 assert.throws(
 ()=>assert.deepStrictEqual(arrProxy,[1,2,3]),
 {message: `${defaultMsgStartFull}\n\n`+
-' [\n 1,\n 2,\n- 3\n ]\n'}
+'+ Proxy([\n- [\n 1,\n 2,\n+ ])\n- 3\n- ]\n'}
 );
 util.inspect.defaultOptions=tmp;
 

test/parallel/test-repl.js

@@ -730,7 +730,7 @@ const errorTests = [
 },
 {
 send: 'repl.writer.options.showProxy = false, new Proxy({x:42}, {});',
-expect: '{ x: 42 }'
+expect: 'Proxy({ x: 42 })'
 },
 
 // Newline within template string maintains whitespace.

test/parallel/test-util-inspect-proxy.js

@@ -116,7 +116,7 @@ const proxy3 = new Proxy(proxy2, proxy1);
 constproxy4=newProxy(proxy1,proxy2);
 constproxy5=newProxy(proxy3,proxy4);
 constproxy6=newProxy(proxy5,proxy5);
-constexpected0='{}';
+constexpected0='Proxy({})';
 constexpected1='Proxy [ {}, {} ]';
 constexpected2='Proxy [ Proxy [ {}, {} ], {} ]';
 constexpected3='Proxy [ Proxy [ Proxy [ {}, {} ], {} ], Proxy [ {}, {} ] ]';
@@ -154,7 +154,7 @@ assert.strictEqual(util.inspect(proxy6), expected0);
 constproxy7=newProxy([],[]);
 constexpected7='Proxy [ [], [] ]';
 assert.strictEqual(util.inspect(proxy7,opts),expected7);
-assert.strictEqual(util.inspect(proxy7),'[]');
+assert.strictEqual(util.inspect(proxy7),'Proxy([])');
 
 // Now we're just getting silly, right?
 constproxy8=newProxy(Date,[]);
@@ -163,8 +163,8 @@ const expected8 = 'Proxy [ [Function: Date], [] ]';
 constexpected9='Proxy [ [Function: Date], [Function: String] ]';
 assert.strictEqual(util.inspect(proxy8,opts),expected8);
 assert.strictEqual(util.inspect(proxy9,opts),expected9);
-assert.strictEqual(util.inspect(proxy8),'[Function: Date]');
-assert.strictEqual(util.inspect(proxy9),'[Function: Date]');
+assert.strictEqual(util.inspect(proxy8),'Proxy([Function: Date])');
+assert.strictEqual(util.inspect(proxy9),'Proxy([Function: Date])');
 
 constproxy10=newProxy(()=>{},{});
 constproxy11=newProxy(()=>{},{
@@ -175,7 +175,16 @@ const proxy11 = new Proxy(() => {}, {
 returnproxy11;
 }
 });
-constexpected10='[Function (anonymous)]';
-constexpected11='[Function (anonymous)]';
+constexpected10='Proxy([Function (anonymous)])';
+constexpected11='Proxy([Function (anonymous)])';
 assert.strictEqual(util.inspect(proxy10),expected10);
 assert.strictEqual(util.inspect(proxy11),expected11);
+
+constproxy12=newProxy([1,2,3],proxy5);
+assert.strictEqual(
+util.inspect(proxy12,{colors: true,breakLength: 1}),
+'\x1B[36mProxy(\x1B[39m'+
+'[\n \x1B[33m1\x1B[39m,\n \x1B[33m2\x1B[39m,\n \x1B[33m3\x1B[39m\n]\x1B[36m'+
+')\x1B[39m'
+);
+assert.strictEqual(util.format('%s',proxy12),'Proxy([ 1, 2, 3 ])');