Set comparison added in 1.1.0 wasn't fully functional.

Additional fix for #194.

Author: ciscoheat

CHANGELOG.md

@@ -5,11 +5,17 @@ Headlines: Added, Changed, Deprecated, Removed, Fixed, Security
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.1.1] - 2023-06-20
+
+### Fixed
+
+- Set comparison added in 1.1.0 wasn't fully functional.
+
 ## [1.1.0] - 2023-06-20
 
 ### Added
 
-- Support for `Set` in schemas, using `z.set()`.
+- Support for `Set` in schemas, using `z.set()`. ([#194](https://github.com/ciscoheat/sveltekit-superforms/issues/194))
 
 ### Fixed
 

src/lib/traversal.ts

@@ -190,6 +190,13 @@ export async function traversePathsAsync<
   }
 }
 
+// Thanks to https://stackoverflow.com/a/31129384/70894
+function eqSet(xs: Set<unknown>, ys: Set<unknown>) {
+  return (
+    xs === ys || (xs.size === ys.size && [...xs].every((x) => ys.has(x)))
+  );
+}
+
 /**
  * Compare two objects and return the differences as paths.
  */
@@ -219,7 +226,7 @@ export function comparePaths(newObj: unknown, oldObj: unknown) {
       exists &&
       data.value instanceof Set &&
       exists.value instanceof Set &&
-      data.value !== exists.value
+      !eqSet(data.value, exists.value)
     ) {
       addDiff();
     }

src/routes/tests/issue-194/+page.svelte

@@ -19,13 +19,10 @@
     <button
       type="button"
       on:click={() =>
-        form.update(
-          ($form) => {
-            $form.things.add(1);
-            return $form;
-          },
-          { taint: { fields: 'things' } }
-        )}>Add 1</button
+        form.update(($form) => {
+          $form.things.add(1);
+          return $form;
+        })}>Add 1</button
     >
     <button
       type="button"