Serialize JS booleans

bartblast · Oct 17, 2024 · 74ff796 · 74ff796
1 parent 525d52f
commit 74ff796
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 1 deletion.
diff --git a/assets/js/serializer.mjs b/assets/js/serializer.mjs
@@ -40,7 +40,8 @@ export default class Serializer {
     if (
       !serialized.startsWith('"') &&
       !serialized.startsWith("{") &&
-      !serialized.startsWith("[")
+      !serialized.startsWith("[") &&
+      !["true", "false"].includes(serialized)
     ) {
       // [version, data]
       return `[1,"${serialized}"]`;

diff --git a/test/javascript/serializer_test.mjs b/test/javascript/serializer_test.mjs
@@ -213,6 +213,22 @@ describe("Serializer", () => {
           assert.equal(serialize(term), expected);
         });
       });
+
+      describe("boolean", () => {
+        it("top-level", () => {
+          const term = true;
+          const expected = "[1,true]";
+
+          assert.equal(serialize(term), expected);
+        });
+
+        it("nested", () => {
+          const term = {a: true, b: 2};
+          const expected = '[1,{"a":true,"b":2}]';
+
+          assert.equal(serialize(term), expected);
+        });
+      });
     });
   });
 });