lib: remove unnecessary error formatting

Author: miguelmarcondesf

lib/internal/util/inspect.js

@@ -1902,13 +1902,6 @@ function formatError(err, constructor, tag, ctx, keys) {
   const name = err.name != null ? err.name : 'Error';
   let stack = getStackString(ctx, err);
 
-  if (tag !== '' && constructor !== null &&
-      StringPrototypeIncludes(constructor, 'Error') &&
-      !StringPrototypeIncludes(tag, 'Error') &&
-      StringPrototypeStartsWith(constructor, tag)) {
-    return `[${tag}: ${err.message}]`;
-  }
-
   removeDuplicateErrorKeys(ctx, keys, err, stack);
 
   if ('cause' in err &&

test/parallel/test-util-inspect.js

@@ -1432,12 +1432,7 @@ if (typeof Symbol !== 'undefined') {
   );
 
   class MiddleErrorPart extends Error {}
-  Object.defineProperty(MiddleErrorPart.prototype, Symbol.toStringTag, {
-    value: 'Middle',
-    configurable: true
-  });
-  assert.strictEqual(util.inspect(new MiddleErrorPart('foo')),
-                     '[Middle: foo]');
+  assert(util.inspect(new MiddleErrorPart('foo')).includes('MiddleErrorPart: foo'));
 
   class MapClass extends Map {}
   assert.strictEqual(util.inspect(new MapClass([['key', 'value']])),