Move multiple entity editing inside action

Widen EntityStore type

src/actions/bind-entity-string-deferred.ts

@@ -11,9 +11,11 @@ export function bindEntityStringDeferred<
 >(node: HTMLInputElement, options: BindEntityOptions<U, T>) {
   return inputAction<BindEntityOptions<U, T>>(
     {
-      subscribe: (node, { store, ids, property, hide }) => {
+      subscribe: (node, { store, ids, property }) => {
         return store.subscribe((doc) => {
-          if (!hide) {
+          const values = new Set(ids.map((id) => doc.entities[id]?.[property]));
+
+          if (values.size === 1) {
             node.value = doc.entities[ids[0]]?.[property] || "";
           } else {
             node.value = "";
@@ -29,19 +31,18 @@ export function bindEntityStringDeferred<
           return doc;
         });
       },
-      changeListener: (node, { store, ids, property, title }) => {
+      changeListener: (node, options) => {
+        const { store, ids, property, title } = options;
+
         store.change(
           (doc) => {
-            const lastValue = doc.entities[ids[0]][property as keyof Extend<T>];
-            const patches = getStringPatches(String(lastValue), node.value);
-
-            patches.forEach((p) => {
-              const path = p.path.slice();
-              ids.forEach((id) => {
+            ids.forEach((id) => {
+              const lastValue =
+                doc.entities[id][property as keyof Extend<T>] || "";
+              const patches = getStringPatches(String(lastValue), node.value);
+              patches.forEach((p) => {
                 p.path.unshift(...[property as string]);
-
                 patch(doc.entities[id], p);
-                p.path = path;
               });
             });
           },

src/actions/bind-entity-value-deferred.ts

@@ -8,9 +8,11 @@ export function bindEntityValueDeferred<
 >(node: HTMLInputElement, options: BindEntityOptions<U, T>) {
   return inputAction<BindEntityOptions<U, T>>(
     {
-      subscribe: (node, { store, ids, property, hide }) => {
+      subscribe: (node, { store, ids, property }) => {
         return store.subscribe((doc) => {
-          if (!hide) {
+          const values = new Set(ids.map((id) => doc.entities[id]?.[property]));
+
+          if (values.size === 1) {
             node.value = doc.entities[ids[0]]?.[property] || "";
           } else {
             node.value = "";

src/actions/bind-text-deferred.ts

@@ -1,55 +1,47 @@
-import type { AutomergeSvelteStore } from "../automerge-svelte-store.type";
+import { Extend } from "@automerge/automerge";
+import { patch } from "@onsetsoftware/automerge-patcher";
 import type { Path, PathValue } from "dot-path-value";
 import { getByPath, setByPath } from "dot-path-value";
 import { getTextPatches } from "../diff-to-patches";
-import { patch } from "@onsetsoftware/automerge-patcher";
 import { quickClone } from "../helpers/quick-clone";
-import { Extend } from "@automerge/automerge";
+import { inputAction } from "./input-action";
+import { BindOptions } from "./types/bind-options.type";
 
 export function bindTextDeferred<T extends Record<string, any>>(
   node: HTMLInputElement,
-  {
-    store,
-    path,
-    title,
-  }: { store: AutomergeSvelteStore<T>; path: Path<T>; title?: string },
+  { store, path, title }: BindOptions<T>,
 ) {
-  const subscription = store.subscribe((doc) => {
-    node.value = getByPath(doc, path)?.toString();
-  });
-
-  const inputListener = () => {
-    store.localChange((doc) => {
-      doc = quickClone(doc);
-      setByPath(doc, path, node.value as PathValue<T, Path<T>>);
-
-      return doc;
-    });
-  };
-
-  const changeListener = () => {
-    store.change(
-      (doc) => {
-        const lastValue = getByPath(doc, path as Path<Extend<T>>);
-        const patches = getTextPatches(String(lastValue), node.value);
-
-        patches.forEach((p) => {
-          p.path.unshift(...path.split("."));
-          patch(doc, p);
+  return inputAction<BindOptions<T>>(
+    {
+      subscribe: (node, { store, path }) => {
+        return store.subscribe((doc) => {
+          node.value = getByPath(doc, path)?.toString() || "";
         });
       },
-      title ? { message: `Update ${title}` } : {},
-    );
-  };
+      inputListener: (node, { store, path }) => {
+        store.localChange((doc) => {
+          doc = quickClone(doc);
+          setByPath(doc, path, node.value as PathValue<T, Path<T>>);
 
-  node.addEventListener("input", inputListener);
-  node.addEventListener("change", changeListener);
+          return doc;
+        });
+      },
+      changeListener: (node, { store, path, title }) => {
+        store.change(
+          (doc) => {
+            const lastValue = getByPath(doc, path as Path<Extend<T>>);
+            const patches = getTextPatches(String(lastValue), node.value);
 
-  return {
-    destroy() {
-      subscription();
-      node.removeEventListener("input", inputListener);
-      node.removeEventListener("change", changeListener);
+            patches.forEach((p) => {
+              p.path.unshift(...path.split("."));
+              patch(doc, p);
+            });
+          },
+          title ? { message: `Update ${title}` } : {},
+        );
+      },
     },
-  };
+    node,
+    { store, path, title },
+  );
 }

src/actions/types/bind-entity-options.type.ts

@@ -7,6 +7,5 @@ export type BindEntityOptions<
   store: AutomergeEntityStore<U, T>;
   ids: string[];
   property: keyof T;
-  hide?: boolean;
   title?: string;
 };

src/automerge-entity-store.ts

@@ -15,7 +15,7 @@ type EntityTitles = {
 
 export class AutomergeEntityStore<
   U,
-  T extends { id: string },
+  T extends { id: string; [key: string]: any },
 > extends AutomergeDerivedStore<EntityState<T>, U> {
   readonly #titles: EntityTitles | undefined;
   constructor(