nodejs
diff --git a/Collapse file
‎lib/internal/util/inspect.js‎
Copy file name to clipboardExpand all lines: lib/internal/util/inspect.js
+3Lines changed: 3 additions & 0 deletions b/Collapse file
‎lib/internal/util/inspect.js‎
Copy file name to clipboardExpand all lines: lib/internal/util/inspect.js
+3Lines changed: 3 additions & 0 deletions
diff --git a/Collapse file
‎test/parallel/test-util-inspect-getters-accessing-this.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-util-inspect-getters-accessing-this.js
+52-15Lines changed: 52 additions & 15 deletions b/Collapse file
‎test/parallel/test-util-inspect-getters-accessing-this.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-util-inspect-getters-accessing-this.js
+52-15Lines changed: 52 additions & 15 deletions
@@ -5,6 +5,7 @@ const {
   ArrayIsArray,
   ArrayPrototypeFilter,
   ArrayPrototypeForEach,
+  ArrayPrototypePop,
   ArrayPrototypePush,
   ArrayPrototypePushApply,
   ArrayPrototypeSort,
@@ -620,6 +621,7 @@ function addPrototypeProperties(ctx, main, obj, recurseTimes, output) {
     }
     // Get all own property names and symbols.
     keys = ReflectOwnKeys(obj);
+    ArrayPrototypePush(ctx.seen, main);
     for (const key of keys) {
       // Ignore the `constructor` property and keys that exist on layers above.
       if (key === 'constructor' ||
@@ -640,6 +642,7 @@ function addPrototypeProperties(ctx, main, obj, recurseTimes, output) {
         ArrayPrototypePush(output, value);
       }
     }
+    ArrayPrototypePop(ctx.seen);
   // Limit the inspection to up to three prototype layers. Using `recurseTimes`
   // is not a good choice here, because it's as if the properties are declared
   // on the current object from the users perspective.
 
@@ -7,24 +7,61 @@ require('../common');
 
 const assert = require('assert');
 
-const util = require('util');
+const { inspect } = require('util');
 
-class X {
-  constructor() {
-    this._y = 123;
-  }
+{
+  class X {
+    constructor() {
+      this._y = 123;
+    }
 
-  get y() {
-    return this._y;
+    get y() {
+      return this._y;
+    }
   }
+
+  const result = inspect(new X(), {
+    getters: true,
+    showHidden: true
+  });
+
+  assert.strictEqual(
+    result,
+    'X { _y: 123, [y]: [Getter: 123] }'
+  );
 }
 
-const result = util.inspect(new X(), {
-  getters: true,
-  showHidden: true
-});
+// Regression test for https://github.com/nodejs/node/issues/37054
+{
+  class A {
+    constructor(B) {
+      this.B = B;
+    }
+    get b() {
+      return this.B;
+    }
+  }
+
+  class B {
+    constructor() {
+      this.A = new A(this);
+    }
+    get a() {
+      return this.A;
+    }
+  }
+
+  const result = inspect(new B(), {
+    depth: 1,
+    getters: true,
+    showHidden: true
+  });
 
-assert.strictEqual(
-  result,
-  'X { _y: 123, [y]: [Getter: 123] }'
-);
+  assert.strictEqual(
+    result,
+    '<ref *1> B {\n' +
+    '  A: A { B: [Circular *1], [b]: [Getter] [Circular *1] },\n' +
+    '  [a]: [Getter] A { B: [Circular *1], [b]: [Getter] [Circular *1] }\n' +
+    '}',
+  );
+}