From f277d758f1f6efa6375d45c009b8edc0913db7ed Mon Sep 17 00:00:00 2001
From: cjihrig <cjihrig@gmail.com>
Date: Sun, 19 Sep 2021 17:44:33 -0400
Subject: [PATCH] util: expose stripVTControlCharacters()

This commit exposes the existing stripVTControlCharacters()
method with docs and some additional input validation.
---
 doc/api/util.md              | 15 +++++++++++++++
 lib/internal/util/inspect.js |  4 ++++
 lib/util.js                  |  4 +++-
 test/parallel/test-util.js   | 10 +++++++++-
 4 files changed, 31 insertions(+), 2 deletions(-)

diff --git a/doc/api/util.md b/doc/api/util.md
index ac768c38c7eb39..29dddd805ee6b3 100644
--- a/doc/api/util.md
+++ b/doc/api/util.md
@@ -1111,6 +1111,21 @@ doSomething[kCustomPromisifiedSymbol] = (foo) => {
 };
 ```
 
+## `util.stripVTControlCharacters(str)`
+<!-- YAML
+added: REPLACEME
+-->
+
+* `str` {string}
+* Returns: {string}
+
+Returns `str` with any ANSI escape codes removed.
+
+```js
+console.log(util.stripVTControlCharacters('\u001B[4mvalue\u001B[0m'));
+// Prints "value"
+```
+
 ## Class: `util.TextDecoder`
 <!-- YAML
 added: v8.3.0
diff --git a/lib/internal/util/inspect.js b/lib/internal/util/inspect.js
index 6f8eb710a9255f..a3b7b8677d37d4 100644
--- a/lib/internal/util/inspect.js
+++ b/lib/internal/util/inspect.js
@@ -2113,6 +2113,10 @@ if (internalBinding('config').hasIntl) {
  * Remove all VT control characters. Use to estimate displayed string width.
  */
 function stripVTControlCharacters(str) {
+  if (typeof str !== 'string') {
+    throw new ERR_INVALID_ARG_TYPE('str', 'string', str);
+  }
+
   return str.replace(ansi, '');
 }
 
diff --git a/lib/util.js b/lib/util.js
index e3f151541ee2fb..fbd8c550318650 100644
--- a/lib/util.js
+++ b/lib/util.js
@@ -57,7 +57,8 @@ const {
 const {
   format,
   formatWithOptions,
-  inspect
+  inspect,
+  stripVTControlCharacters,
 } = require('internal/util/inspect');
 const { debuglog } = require('internal/util/debuglog');
 const {
@@ -369,6 +370,7 @@ module.exports = {
   isPrimitive,
   log,
   promisify,
+  stripVTControlCharacters,
   toUSVString,
   TextDecoder,
   TextEncoder,
diff --git a/test/parallel/test-util.js b/test/parallel/test-util.js
index 42218be8581579..6b26f3398fa0b7 100644
--- a/test/parallel/test-util.js
+++ b/test/parallel/test-util.js
@@ -21,7 +21,7 @@
 
 'use strict';
 // Flags: --expose-internals
-require('../common');
+const common = require('../common');
 const assert = require('assert');
 const util = require('util');
 const errors = require('internal/errors');
@@ -178,3 +178,11 @@ assert.strictEqual(util.toUSVString('string\ud801'), 'string\ufffd');
     true
   );
 }
+
+assert.throws(() => {
+  util.stripVTControlCharacters({});
+}, {
+  code: 'ERR_INVALID_ARG_TYPE',
+  message: 'The "str" argument must be of type string.' +
+           common.invalidArgTypeHelper({})
+});