fix: strings properly mapped to YText

src/index.spec.ts

@@ -619,6 +619,41 @@ describe("Yjs middleware", () =>
       }).not.toThrow();
     });
   });
+
+  // See issue #49
+  describe("When nesting strings into arrays and objects", () =>
+  {
+    it("Does not crash", () =>
+    {
+      type Store =
+      {
+        foo: { bar: string }
+        updateFoo: (s: string) => void
+      };
+
+      const doc = new Y.Doc();
+
+      const api = createVanilla<Store>(yjs(
+        doc,
+        "hello",
+        (set) =>
+          ({
+            "foo": {
+              "bar": "baz",
+            },
+            "updateFoo": (s: string) =>
+              set((state) =>
+                ({ ...state, "foo": { "bar": s, }, })),
+          })
+      ));
+
+      expect(() =>
+      {
+        api.getState().updateFoo("bingo");
+        api.getState().updateFoo("bango"); // Always on subsequent update
+      }).not.toThrow();
+    });
+  });
 });
 
 describe("Yjs middleware with network provider", () =>

src/mapping.spec.ts

@@ -67,6 +67,16 @@ describe("arrayToYArray", () =>
     expect(ymap.get("array").get(0)
       .get("foo")).toBe(1);
   });
+
+  it("Creates YText nested in YArrays from strings nested in arrays.", () =>
+  {
+    ymap.set("array", arrayToYArray([ "hello" ]));
+
+    expect(ymap.get("array").toJSON()).toEqual([ "hello" ]);
+    expect(ymap.get("array").get(0)).toBeInstanceOf(Y.Text);
+    expect(ymap.get("array").get(0)
+      .toString()).toBe("hello");
+  });
 });
 
 describe("yArrayToArray", () =>
@@ -167,6 +177,17 @@ describe("objectToYMap", () =>
     expect((ymap.get("map") as Y.Map<any>).get("foo").get(0)).toBe(1);
     expect((ymap.get("map") as Y.Map<any>).get("foo").get(1)).toBe(2);
   });
+
+  it("Converts strings into YText.", () =>
+  {
+    const ydoc = new Y.Doc();
+    const ymap = ydoc.getMap("tmp");
+
+    ymap.set("map", objectToYMap({ "foo": "bar", }));
+
+    expect((ymap.get("map") as Y.Map<any>).get("foo")).toBeInstanceOf(Y.Text);
+    expect((ymap.get("map") as Y.Map<any>).get("foo").toString()).toBe("bar");
+  });
 });
 
 describe("yMapToObject", () =>

src/mapping.ts

@@ -28,6 +28,9 @@ export const arrayToYArray = (array: any[]): Y.Array<any> =>
     else if (value instanceof Object)
       yarray.push([ objectToYMap(value) ]);
 
+    else if (typeof value === "string")
+      yarray.push([ stringToYText(value) ]);
+
     else
       yarray.push([ value ]);
   });
@@ -102,6 +105,9 @@ export const objectToYMap = (object: any): Y.Map<any> =>
     else if (value instanceof Object)
       ymap.set(property, objectToYMap(value));
 
+    else if (typeof value === "string")
+      ymap.set(property, stringToYText(value));
+
     else
       ymap.set(property, value);
   });