Fix undefined descriptor error in property formatting

[siaeyy]

Fixes: #60717

[BridgeAR]

The change and test is LGTM.

After giving it some more thought, I just lean on not printing these when the properties are not own properties. That aligns more with other inspected parts.

@ljharb since you often have an opinion about inspect, what is your take on it?

[BridgeAR]

@siaeyy do you mind changing the code as follows instead:

diff --git a/lib/internal/util/inspect.js b/lib/internal/util/inspect.js
index 83c254c3d6c..813c7eec659 100644
--- a/lib/internal/util/inspect.js
+++ b/lib/internal/util/inspect.js
@@ -1955,14 +1955,14 @@ function formatError(err, constructor, tag, ctx, keys) {
   }
   name ??= 'Error';
 
-  if ('cause' in err &&
+  if (Object.hasOwn(err, 'cause') &&
       (keys.length === 0 || !ArrayPrototypeIncludes(keys, 'cause'))) {
     ArrayPrototypePush(keys, 'cause');
   }
 
   // Print errors aggregated into AggregateError
   try {
-    const errors = err.errors;
+    const errors = Object.hasOwn(err, 'errors') ? err.errors : undefined;
     if (ArrayIsArray(errors) &&
       (keys.length === 0 || !ArrayPrototypeIncludes(keys, 'errors'))) {
       ArrayPrototypePush(keys, 'errors');

That way only own properties are handled as with other objects. The test should still pass with the new change.

[ljharb]

I think that makes sense, since both cause and errors will only ever be own properties.

[siaeyy]

There is already another pr that fixes the issue.