@joshuahhh/pretty-print 0.0.4 → 0.0.6

package/dist/pretty-print.d.ts

@@ -4,7 +4,7 @@ export interface PrettyPrintOptions {
     useColor?: boolean;
     niceId?: boolean;
     niceType?: boolean;
-    /** Number of digits after the decimal point for floating-point numbers (uses toFixed). */
+    /** Maximum number of digits after the decimal point for floating-point numbers. */
     precision?: number;
     /** Number of significant digits for floating-point numbers (uses toPrecision). Ignored if precision is set. */
     significantDigits?: number;

package/dist/pretty-print.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"pretty-print.d.ts","sourceRoot":"","sources":["../src/pretty-print.tsx"],"names":[],"mappings":"AACA,OAAO,KAAK,MAAM,OAAO,CAAC;AAO1B,MAAM,WAAW,kBAAkB;IACjC,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB,0FAA0F;IAC1F,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,+GAA+G;IAC/G,iBAAiB,CAAC,EAAE,MAAM,CAAC;CAC5B;AA2BD;;GAEG;AACH,wBAAgB,qBAAqB,CACnC,KAAK,EAAE,OAAO,EACd,OAAO,GAAE,kBAAuB,GAC/B,CAAC,MAAM,EAAE,GAAG,MAAM,EAAE,CAAC,CA2BvB;AAED;;;GAGG;AACH,wBAAgB,SAAS,CACvB,KAAK,EAAE,OAAO,EACd,EAAE,KAAK,EAAE,GAAG,OAAO,EAAE,GAAE,kBAAkB,GAAG;IAAE,KAAK,CAAC,EAAE,MAAM,CAAA;CAAO,GAClE,IAAI,CAQN;AAED;;;GAGG;AACH,wBAAgB,WAAW,CAAC,EAC1B,KAAK,EACL,KAAK,EACL,SAAS,EACT,GAAG,OAAO,EACX,EAAE,kBAAkB,GAAG;IACtB,KAAK,EAAE,OAAO,CAAC;IACf,KAAK,CAAC,EAAE,KAAK,CAAC,aAAa,CAAC;IAC5B,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB,2CAyFA;AAiUD,wBAAgB,mBAAmB,CACjC,KAAK,EAAE,OAAO,EACd,OAAO,GAAE,kBAAuB,GAC/B,MAAM,CAWR;AAGD,eAAO,MAAM,QAAQ;;;;;;;;;;;;;;;;;;;;;;;;CAYpB,CAAC"}
+{"version":3,"file":"pretty-print.d.ts","sourceRoot":"","sources":["../src/pretty-print.tsx"],"names":[],"mappings":"AACA,OAAO,KAAK,MAAM,OAAO,CAAC;AAO1B,MAAM,WAAW,kBAAkB;IACjC,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB,mFAAmF;IACnF,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,+GAA+G;IAC/G,iBAAiB,CAAC,EAAE,MAAM,CAAC;CAC5B;AA2BD;;GAEG;AACH,wBAAgB,qBAAqB,CACnC,KAAK,EAAE,OAAO,EACd,OAAO,GAAE,kBAAuB,GAC/B,CAAC,MAAM,EAAE,GAAG,MAAM,EAAE,CAAC,CA2BvB;AAED;;;GAGG;AACH,wBAAgB,SAAS,CACvB,KAAK,EAAE,OAAO,EACd,EAAE,KAAK,EAAE,GAAG,OAAO,EAAE,GAAE,kBAAkB,GAAG;IAAE,KAAK,CAAC,EAAE,MAAM,CAAA;CAAO,GAClE,IAAI,CAQN;AAED;;;GAGG;AACH,wBAAgB,WAAW,CAAC,EAC1B,KAAK,EACL,KAAK,EACL,SAAS,EACT,GAAG,OAAO,EACX,EAAE,kBAAkB,GAAG;IACtB,KAAK,EAAE,OAAO,CAAC;IACf,KAAK,CAAC,EAAE,KAAK,CAAC,aAAa,CAAC;IAC5B,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB,2CAyFA;AAmUD,wBAAgB,mBAAmB,CACjC,KAAK,EAAE,OAAO,EACd,OAAO,GAAE,kBAAuB,GAC/B,MAAM,CAaR;AAGD,eAAO,MAAM,QAAQ;;;;;;;;;;;;;;;;;;;;;;;;CAYpB,CAAC"}

package/dist/pretty-print.js

@@ -141,8 +141,6 @@ function prettyPrintToDoc(value, tagger, options, visited = new Set(), path = []
             const props = element.props;
             const { children, ...otherProps } = props;
             const openTag = colorize(`<${type}`, "keyword", [...path, 0]);
-            const closeTag = colorize(`</${type}>`, "keyword", [...path, 4]);
-            const selfCloseTag = colorize("/>", "keyword", [...path, 2]);
             // Format props
             const propEntries = Object.entries(otherProps);
             const propDocs = [];
@@ -166,9 +164,11 @@ function prettyPrintToDoc(value, tagger, options, visited = new Set(), path = []
             const childrenArray = React.Children.toArray(children);
             const hasChildren = childrenArray.length > 0;
             if (!hasChildren && propEntries.length === 0) {
+                const selfCloseTag = colorize("/>", "keyword", [...path, 2]);
                 return [openTag, " ", selfCloseTag];
             }
             if (!hasChildren) {
+                const selfCloseTag = colorize("/>", "keyword", [...path, 2]);
                 return group([
                     openTag,
                     indent(propDocs),
@@ -176,6 +176,7 @@ function prettyPrintToDoc(value, tagger, options, visited = new Set(), path = []
                     selfCloseTag,
                 ]);
             }
+            const closeTag = colorize(`</${type}>`, "keyword", [...path, 4]);
             const childDocs = childrenArray.map((child, i) => typeof child === "string" || typeof child === "number"
                 ? String(child)
                 : prettyPrintToDoc(child, tagger, options, visited, [...path, 3, i]));
@@ -211,7 +212,7 @@ function prettyPrintToDoc(value, tagger, options, visited = new Set(), path = []
     if (typeof value === "number") {
         let numStr;
         if (Number.isFinite(value) && options.precision != null) {
-            numStr = value.toFixed(options.precision);
+            numStr = String(parseFloat(value.toFixed(options.precision)));
         }
         else if (Number.isFinite(value) && options.significantDigits != null) {
             numStr = value.toPrecision(options.significantDigits);
@@ -381,6 +382,7 @@ function prettyPrintToDoc(value, tagger, options, visited = new Set(), path = []
     return "[Unknown]";
 }
 export function prettyPrintToString(value, options = {}) {
+    console.log("value", value);
     const { width = 80, useColor = true } = options;
     const tagger = useColor ? new StringTagger() : null;
     const doc = prettyPrintToDoc(value, tagger, options);

package/dist/pretty-print.test.js

@@ -4,8 +4,14 @@ import { prettyPrintToString } from "./pretty-print.js";
 describe("prettyPrintToString", () => {
     it("should format with and without ANSI codes", () => {
         const longArray = Array.from({ length: 20 }, (_, i) => i);
-        const withoutAnsi = prettyPrintToString(longArray, { width: 200, useColor: false });
-        const withAnsi = prettyPrintToString(longArray, { width: 200, useColor: true });
+        const withoutAnsi = prettyPrintToString(longArray, {
+            width: 200,
+            useColor: false,
+        });
+        const withAnsi = prettyPrintToString(longArray, {
+            width: 200,
+            useColor: true,
+        });
         // Count ANSI escape sequences
         const ansiMatches = withAnsi.match(/\x1b\[\d+m/g);
         expect(ansiMatches).toBeTruthy();
@@ -13,13 +19,19 @@ describe("prettyPrintToString", () => {
     });
     it("should format long arrays inline with wide printWidth", () => {
         const longArray = Array.from({ length: 20 }, (_, i) => i);
-        const result = prettyPrintToString(longArray, { width: 200, useColor: false });
+        const result = prettyPrintToString(longArray, {
+            width: 200,
+            useColor: false,
+        });
         // With width 200, this should be all on one line
         expect(result).not.toContain("\n");
     });
     it("should format long arrays with line breaks when narrow", () => {
         const longArray = Array.from({ length: 20 }, (_, i) => i);
-        const result = prettyPrintToString(longArray, { width: 40, useColor: false });
+        const result = prettyPrintToString(longArray, {
+            width: 40,
+            useColor: false,
+        });
         // With width 40, this should break across multiple lines
         expect(result).toContain("\n");
     });
@@ -53,8 +65,14 @@ describe("prettyPrintToString", () => {
     });
     it("should handle very long arrays", () => {
         const veryLongArray = Array.from({ length: 50 }, (_, i) => i + 1);
-        const wide = prettyPrintToString(veryLongArray, { width: 300, useColor: false });
-        const narrow = prettyPrintToString(veryLongArray, { width: 60, useColor: false });
+        const wide = prettyPrintToString(veryLongArray, {
+            width: 300,
+            useColor: false,
+        });
+        const narrow = prettyPrintToString(veryLongArray, {
+            width: 60,
+            useColor: false,
+        });
         // Both should contain all elements
         expect(wide).toContain("49");
         expect(narrow).toContain("49");
@@ -126,7 +144,11 @@ describe("prettyPrintToString", () => {
     });
     it("should keep type as normal property when niceType is false", () => {
         const obj = { type: "person", name: "Alice" };
-        const result = prettyPrintToString(obj, { width: 80, useColor: false, niceType: false });
+        const result = prettyPrintToString(obj, {
+            width: 80,
+            useColor: false,
+            niceType: false,
+        });
         expect(result).toBe('{type: "person", name: "Alice"}');
     });
     it("should extract id field as prefix by default", () => {
@@ -136,7 +158,11 @@ describe("prettyPrintToString", () => {
     });
     it("should keep id as normal property when niceId is false", () => {
         const obj = { id: 42, name: "Alice" };
-        const result = prettyPrintToString(obj, { width: 80, useColor: false, niceId: false });
+        const result = prettyPrintToString(obj, {
+            width: 80,
+            useColor: false,
+            niceId: false,
+        });
         expect(result).toBe('{id: 42, name: "Alice"}');
     });
     it("should extract both type and id as prefixes by default", () => {
@@ -154,24 +180,33 @@ describe("prettyPrintToString", () => {
         });
         expect(result).toBe('{type: "user", id: 1, name: "Alice"}');
     });
-    it("should format numbers with precision (toFixed)", () => {
-        const result = prettyPrintToString(3.14159, { useColor: false, precision: 2 });
+    it("should limit decimal places with precision", () => {
+        const result = prettyPrintToString(3.14159, {
+            useColor: false,
+            precision: 2,
+        });
         expect(result).toBe("3.14");
     });
     it("should format numbers with precision 0", () => {
         const result = prettyPrintToString(3.7, { useColor: false, precision: 0 });
         expect(result).toBe("4");
     });
-    it("should pad with zeros when precision exceeds decimal places", () => {
+    it("should not pad with zeros when precision exceeds decimal places", () => {
         const result = prettyPrintToString(1.5, { useColor: false, precision: 4 });
-        expect(result).toBe("1.5000");
+        expect(result).toBe("1.5");
     });
     it("should format numbers with significantDigits (toPrecision)", () => {
-        const result = prettyPrintToString(3.14159, { useColor: false, significantDigits: 4 });
+        const result = prettyPrintToString(3.14159, {
+            useColor: false,
+            significantDigits: 4,
+        });
         expect(result).toBe("3.142");
     });
     it("should format large numbers with significantDigits", () => {
-        const result = prettyPrintToString(123456, { useColor: false, significantDigits: 3 });
+        const result = prettyPrintToString(123456, {
+            useColor: false,
+            significantDigits: 3,
+        });
         expect(result).toBe("1.23e+5");
     });
     it("precision should take precedence over significantDigits", () => {
@@ -183,7 +218,10 @@ describe("prettyPrintToString", () => {
         expect(result).toBe("3.1");
     });
     it("should not apply precision to Infinity", () => {
-        const result = prettyPrintToString(Infinity, { useColor: false, precision: 2 });
+        const result = prettyPrintToString(Infinity, {
+            useColor: false,
+            precision: 2,
+        });
         expect(result).toBe("Infinity");
     });
     it("should not apply precision to NaN", () => {
@@ -191,17 +229,24 @@ describe("prettyPrintToString", () => {
         expect(result).toBe("NaN");
     });
     it("should not apply significantDigits to -Infinity", () => {
-        const result = prettyPrintToString(-Infinity, { useColor: false, significantDigits: 3 });
+        const result = prettyPrintToString(-Infinity, {
+            useColor: false,
+            significantDigits: 3,
+        });
         expect(result).toBe("-Infinity");
     });
     it("should apply precision to numbers inside objects and arrays", () => {
         const data = { values: [1.111, 2.222, 3.333] };
-        const result = prettyPrintToString(data, { width: 80, useColor: false, precision: 1 });
+        const result = prettyPrintToString(data, {
+            width: 80,
+            useColor: false,
+            precision: 1,
+        });
         expect(result).toBe("{values: [1.1, 2.2, 3.3]}");
     });
-    it("should format integers with precision", () => {
+    it("should not pad integers with precision", () => {
         const result = prettyPrintToString(42, { useColor: false, precision: 2 });
-        expect(result).toBe("42.00");
+        expect(result).toBe("42");
     });
     it("should leave numbers unchanged when neither option is set", () => {
         const result = prettyPrintToString(3.14159, { useColor: false });
@@ -218,4 +263,11 @@ describe("prettyPrintToString", () => {
   />
 </g>`);
     });
+    it("should print nested JSX elements with color enabled", () => {
+        // Nested JSX with children triggers the bug: phantom selfCloseTag/closeTag
+        // spans get registered in tagger.tag() but never appear in the formatted
+        // output, corrupting path-based marker matching in expand().
+        const element = React.createElement("g", { transform: "translate(0,0)" }, React.createElement("rect", { width: 10, height: 20 }), React.createElement("text", { x: 5 }, "hello"));
+        expect(() => prettyPrintToString(element, { width: 60, useColor: true })).not.toThrow();
+    });
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@joshuahhh/pretty-print",
-  "version": "0.0.4",
+  "version": "0.0.6",
   "type": "module",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",