Schema transforms weren't applied on the client.

Author: ciscoheat

CHANGELOG.md

@@ -13,6 +13,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Added `FormResult<T>`, which can be used in [onResult](https://superforms.rocks/concepts/events#onresult) to make the ActionResult strongly typed.
 - [SuperDebug](https://superforms.rocks/super-debug) now has a `collapsed` prop, to make it initially collapsed. Use together with `collapsible`. ([#279](https://github.com/ciscoheat/sveltekit-superforms/issues/279))
 
+### Fixed
+
+- Schema `transform` operations weren't applied in [SPA forms](https://superforms.rocks/concepts/spa) and when posting to the server with client-side [validators](https://superforms.rocks/concepts/client-validation#validators).
+
 ## [1.8.0]
 
 ### Fixed

src/lib/client/clientValidation.ts

@@ -110,6 +110,8 @@ async function _clientValidation<T extends AnyZodObject, M = unknown>(
           errorShape(validator)
           // eslint-disable-next-line @typescript-eslint/no-explicit-any
         ) as any;
+      } else {
+        checkData = result.data;
       }
     } else {
       // SuperForms validator

src/lib/client/formEnhance.ts

@@ -165,9 +165,10 @@ export function formEnhance<T extends AnyZodObject, M>(
       setCustomValidity(validityEl as HTMLInputElement, result.errors);
     }
 
-    if (result.data) {
-      data.set(result.data);
-    }
+    // NOTE: Uncomment if Zod transformations should be immediately applied, not just when submitting.
+    // Not enabled because it's not great UX, and it's rare to have transforms, which will just result in
+    // redundant store updates.
+    //if (result.data) data.set(result.data);
   }
 
   /**
@@ -338,25 +339,21 @@ export function formEnhance<T extends AnyZodObject, M>(
         if (options.SPA) {
           cancel();
 
-          const validationResult: SuperValidated<T> = {
-            valid: true,
-            posted: true,
-            errors: {},
-            data: get(data),
-            constraints: get(constraints),
-            message: undefined,
-            id: get(formId)
-          };
+          const validationResult = { ...validation, posted: true };
 
           const result = {
-            type: 'success' as const,
-            status: 200,
+            type: validationResult.valid ? 'success' : 'failure',
+            status: validationResult.valid
+              ? 200
+              : typeof options.SPA == 'object'
+              ? options.SPA?.failStatus
+              : 400 ?? 400,
             data: { form: validationResult }
-          };
+          } as ActionResult;
 
           setTimeout(() => validationResponse({ result }), 0);
         } else if (options.dataType === 'json') {
-          const postData = get(data);
+          const postData = validation.data;
           const chunks = chunkSubstr(
             stringify(postData),
             options.jsonChunkSize ?? 500000