fix: make format match browser console log

Author: ojeytonwilliams

packages/shared/src/format.test.ts

@@ -47,15 +47,16 @@ describe("format", () => {
     expect(format(NaN)).toBe("NaN");
   });
   it("handles Maps", () => {
-    const map = new Map([
+    const map = new Map<string|number,string|number>([
       ["a", 1],
-      ["b", 2],
+      ["b", "2"],
+      [3, "three"],
     ]);
-    expect(format(map)).toBe("Map(2) { 'a' => 1, 'b' => 2 }");
+    expect(format(map)).toBe("Map(3) {'a' => 1, 'b' => '2', 3 => 'three'}");
   });
   it("handles Sets", () => {
     const set = new Set([1, "2", 3]);
-    expect(format(set)).toBe("Set(3) { 1, '2', 3 }");
+    expect(format(set)).toBe("Set(3) {1, '2', 3}");
   });
   it("handles BigInts", () => {
     expect(format(BigInt(10))).toBe("10n");

packages/shared/src/format.ts

@@ -1,19 +1,20 @@
 import { inspect } from "util/util";
 
-const quoteString = (x: unknown) => (typeof x === "string" ? `'${x}'` : x);
+const quoteString = <T>(x: T): T | string =>
+  typeof x === "string" ? `'${x}'` : x;
 
 export function format(x: unknown): string {
   // We're trying to mimic console.log, so we avoid wrapping strings in quotes:
   if (typeof x === "string") return x;
   if (x instanceof Set) {
-    return `Set(${x.size}) { ${Array.from(x, quoteString).join(", ")} }`;
+    return `Set(${x.size}) {${Array.from(x, quoteString).join(", ")}}`;
   }
 
   if (x instanceof Map) {
     return `Map(${x.size}) {${Array.from(
       x.entries(),
-      ([k, v]) => ` '${k}' => ${v}`,
-    ).join(",")} }`;
+      ([k, v]) => `${quoteString(k)} => ${quoteString(v)}`,
+    ).join(", ")}}`;
   }
 
   if (typeof x === "bigint") {