@oclif/core 4.0.28 → 4.0.29

package/lib/ux/colorize-json.d.ts

@@ -2,7 +2,7 @@ type Options = {
     pretty?: boolean | undefined;
     theme?: Record<string, string> | undefined;
 };
-export declare function removeCycles(object: unknown): unknown;
+export declare function stringifyInput(json?: unknown, options?: Options): string;
 export declare function tokenize(json?: unknown, options?: Options): {
     type: string;
     value: string;

package/lib/ux/colorize-json.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.removeCycles = removeCycles;
+exports.stringifyInput = stringifyInput;
 exports.tokenize = tokenize;
 exports.default = colorizeJson;
 const theme_1 = require("./theme");
@@ -16,46 +16,48 @@ const tokenTypes = [
     { regex: /^true|^false/, tokenType: 'boolean' },
     { regex: /^null/, tokenType: 'null' },
 ];
-function removeCycles(object) {
-    // Keep track of seen objects.
-    const seenObjects = new WeakMap();
-    const _removeCycles = (obj) => {
-        // Use object prototype to get around type and null checks
-        if (Object.prototype.toString.call(obj) === '[object Object]') {
-            // We know it is a "Record<string, unknown>" because of the conditional
-            const dictionary = obj;
-            // Seen, return undefined to remove.
-            if (seenObjects.has(dictionary))
-                return;
-            seenObjects.set(dictionary, undefined);
-            for (const key in dictionary) {
-                // Delete the duplicate object if cycle found.
-                if (_removeCycles(dictionary[key]) === undefined) {
-                    delete dictionary[key];
-                }
-            }
-        }
-        else if (Array.isArray(obj)) {
-            for (const i in obj) {
-                if (_removeCycles(obj[i]) === undefined) {
-                    // We don't want to delete the array, but we can replace the element with null.
-                    obj[i] = null;
-                }
-            }
+function stringify(value, replacer, spaces) {
+    return JSON.stringify(value, serializer(replacer, replacer), spaces);
+}
+// Inspired by https://github.com/moll/json-stringify-safe
+function serializer(replacer, cycleReplacer) {
+    const stack = [];
+    const keys = [];
+    if (!cycleReplacer)
+        cycleReplacer = function (key, value) {
+            if (stack[0] === value)
+                return '[Circular ~]';
+            return '[Circular ~.' + keys.slice(0, stack.indexOf(value)).join('.') + ']';
+        };
+    return function (key, value) {
+        if (stack.length > 0) {
+            // @ts-expect-error because `this` is not typed
+            const thisPos = stack.indexOf(this);
+            // @ts-expect-error because `this` is not typed
+            // eslint-disable-next-line no-bitwise
+            ~thisPos ? stack.splice(thisPos + 1) : stack.push(this);
+            // eslint-disable-next-line no-bitwise
+            ~thisPos ? keys.splice(thisPos, Number.POSITIVE_INFINITY, key) : keys.push(key);
+            // @ts-expect-error because `this` is not typed
+            if (stack.includes(value))
+                value = cycleReplacer.call(this, key, value);
         }
-        return obj;
+        else
+            stack.push(value);
+        // @ts-expect-error because `this` is not typed
+        return replacer ? replacer.call(this, key, value) : value;
     };
-    return _removeCycles(object);
 }
-function formatInput(json, options) {
-    return options?.pretty
-        ? JSON.stringify(typeof json === 'string' ? JSON.parse(json) : json, null, 2)
+function stringifyInput(json, options) {
+    const str = options?.pretty
+        ? stringify(typeof json === 'string' ? JSON.parse(json) : json, undefined, 2)
         : typeof json === 'string'
             ? json
-            : JSON.stringify(json);
+            : stringify(json);
+    return str;
 }
 function tokenize(json, options) {
-    let input = formatInput(removeCycles(json), options);
+    let input = stringifyInput(json, options);
     const tokens = [];
     let foundToken = false;
     do {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@oclif/core",
   "description": "base library for oclif CLIs",
-  "version": "4.0.28",
+  "version": "4.0.29",
   "author": "Salesforce",
   "bugs": "https://github.com/oclif/core/issues",
   "dependencies": {
@@ -37,7 +37,7 @@
     "@types/debug": "^4.1.10",
     "@types/ejs": "^3.1.5",
     "@types/indent-string": "^4.0.1",
-    "@types/mocha": "^10.0.8",
+    "@types/mocha": "^10.0.9",
     "@types/node": "^18",
     "@types/pnpapi": "^0.0.5",
     "@types/sinon": "^17.0.3",