Merge pull request #24 from ty-ras/issue/17-fix-state-optionality

#17 Forgot crucial 'optional' call.

state/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ty-ras/state-zod",
-  "version": "2.0.0",
+  "version": "2.0.1",
   "author": {
     "name": "Stanislav Muhametsin",
     "email": "[email protected]",

state/src/__test__/state-validator-factory.spec.ts

@@ -70,6 +70,22 @@ test("Verify that giving wrong input to state validator factory throws an error"
   });
 });
 
+test("Verify that optional authenticated property validation behaves as expected", (c) => {
+  c.plan(1);
+
+  const stateValidatorFactory = createStateValidatorFactory(
+    { userId: t.string() },
+    {},
+  );
+
+  const optional = stateValidatorFactory({ userId: false });
+  c.deepEqual(optional.validator({}), {
+    // Important that this it NOT protocol-error
+    error: "none",
+    data: {},
+  });
+});
+
 const createStateValidatorFactory = <
   TAuthenticated extends Record<string, data.AnyDecoder>,
   TOther extends Record<string, data.AnyDecoder>,

state/src/state-validator-factory.ts

@@ -31,7 +31,10 @@ export const createStateValidatorFactory = <
         ...Object.fromEntries(
           optional.map(
             (optionalProp) =>
-              [optionalProp, validation[optionalProp].validation] as const,
+              [
+                optionalProp,
+                validation[optionalProp].validation.optional(),
+              ] as const,
           ),
         ),
       });