@strictly/react-form 0.0.26 → 0.0.27

package/.turbo/turbo-build.log

@@ -7,12 +7,12 @@ $ tsup
 CLI Target: es6
 CJS Build start
 ESM Build start
-ESM dist/index.js 59.53 KB
-ESM ⚡️ Build success in 105ms
-CJS dist/index.cjs 63.62 KB
-CJS ⚡️ Build success in 110ms
+CJS dist/index.cjs 63.79 KB
+CJS ⚡️ Build success in 126ms
+ESM dist/index.js 59.70 KB
+ESM ⚡️ Build success in 132ms
 DTS Build start
-DTS ⚡️ Build success in 30914ms
-DTS dist/index.d.cts 38.03 KB
-DTS dist/index.d.ts  38.03 KB
-Done in 32.12s.
+DTS ⚡️ Build success in 32334ms
+DTS dist/index.d.cts 38.14 KB
+DTS dist/index.d.ts  38.14 KB
+Done in 33.47s.

package/.turbo/turbo-check-types.log

@@ -1,3 +1,3 @@
 yarn run v1.22.22
 $ tsc -b
-Done in 0.43s.
+Done in 0.40s.

package/core/mobx/form_model.ts

@@ -599,6 +599,8 @@ export abstract class FormModel<
           const validation = this.validation[fromJsonPath]
           delete this.validation[fromJsonPath]
           this.validation[toJsonPath] = validation
+          // eslint-disable-next-line @typescript-eslint/consistent-type-assertions, @typescript-eslint/no-explicit-any
+          this.moveListItem(fromJsonPath as any, toJsonPath as any)
         })
         accessor.set(newList)
         // delete any value overrides so the new list isn't shadowed
@@ -608,6 +610,13 @@ export abstract class FormModel<
     })
   }
 
+  protected moveListItem<K extends keyof FlattenedListTypesOfType<T>>(fromValuePath: K, toValuePath: K) {
+    // do nothing, this is for subclasses to override
+    // put in some nonsense so TS doesn't complain about the parameters not being used
+    fromValuePath satisfies K
+    toValuePath satisfies K
+  }
+
   private internalSetFieldValue<K extends keyof ValuePathsToAdapters>(
     valuePath: K,
     value: ToOfFieldAdapter<ValuePathsToAdapters[K]>,

package/core/mobx/hooks.tsx

@@ -66,8 +66,8 @@ export function useDefaultMobxFormHooks<
       // (e.g. changing a discriminator)
       // TODO debounce?
       setTimeout(function () {
-        // only start validation if the user has changed the field
-        if (model.isValuePathActive(path) && model.isFieldDirty(path)) {
+        // only start validation if the user has changed the field and there isn't already an error visible
+        if (model.isValuePathActive(path) && model.isFieldDirty(path) && model.fields[path].error == null) {
           // further workaround to make sure we don't downgrade the existing validation
           model.validateField(path, Math.max(Validation.Changed, model.getValidation(path)))
         }

package/dist/index.cjs

@@ -720,12 +720,17 @@ var FormModel = class {
           const validation = this.validation[fromJsonPath];
           delete this.validation[fromJsonPath];
           this.validation[toJsonPath] = validation;
+          this.moveListItem(fromJsonPath, toJsonPath);
         });
         accessor.set(newList);
         delete this.fieldOverrides[listValuePath];
       });
     });
   }
+  moveListItem(fromValuePath, toValuePath) {
+    fromValuePath;
+    toValuePath;
+  }
   internalSetFieldValue(valuePath, value, validation) {
     const { revert } = this.getAdapterForValuePath(valuePath);
     (0, import_base2.assertExists)(revert, "setting value not supported {}", valuePath);
@@ -915,7 +920,7 @@ function useDefaultMobxFormHooks(model, {
   const onFieldBlur = (0, import_react.useCallback)(
     function(path) {
       setTimeout(function() {
-        if (model.isValuePathActive(path) && model.isFieldDirty(path)) {
+        if (model.isValuePathActive(path) && model.isFieldDirty(path) && model.fields[path].error == null) {
           model.validateField(path, Math.max(1 /* Changed */, model.getValidation(path)));
         }
       }, 100);

package/dist/index.d.cts

@@ -157,6 +157,7 @@ declare abstract class FormModel<T extends Type, ValueToTypePaths extends Readon
     setFieldValue<K extends keyof ValuePathsToAdapters>(valuePath: K, value: ToOfFieldAdapter<ValuePathsToAdapters[K]>, validation?: Validation): boolean;
     addListItem<K extends keyof FlattenedListTypesOfType<T>>(valuePath: K, elementValue?: Maybe<ElementOfArray<FlattenedValuesOfType<T>[K]>>, index?: number): void;
     removeListItem<K extends keyof FlattenedListTypesOfType<T>>(...elementValuePaths: readonly `${K}.${number}`[]): void;
+    protected moveListItem<K extends keyof FlattenedListTypesOfType<T>>(fromValuePath: K, toValuePath: K): void;
     private internalSetFieldValue;
     /**
      * Forces an error onto a field. Error will be removed if the field value changes

package/dist/index.d.ts

@@ -157,6 +157,7 @@ declare abstract class FormModel<T extends Type, ValueToTypePaths extends Readon
     setFieldValue<K extends keyof ValuePathsToAdapters>(valuePath: K, value: ToOfFieldAdapter<ValuePathsToAdapters[K]>, validation?: Validation): boolean;
     addListItem<K extends keyof FlattenedListTypesOfType<T>>(valuePath: K, elementValue?: Maybe<ElementOfArray<FlattenedValuesOfType<T>[K]>>, index?: number): void;
     removeListItem<K extends keyof FlattenedListTypesOfType<T>>(...elementValuePaths: readonly `${K}.${number}`[]): void;
+    protected moveListItem<K extends keyof FlattenedListTypesOfType<T>>(fromValuePath: K, toValuePath: K): void;
     private internalSetFieldValue;
     /**
      * Forces an error onto a field. Error will be removed if the field value changes

package/dist/index.js

@@ -690,12 +690,17 @@ var FormModel = class {
           const validation = this.validation[fromJsonPath];
           delete this.validation[fromJsonPath];
           this.validation[toJsonPath] = validation;
+          this.moveListItem(fromJsonPath, toJsonPath);
         });
         accessor.set(newList);
         delete this.fieldOverrides[listValuePath];
       });
     });
   }
+  moveListItem(fromValuePath, toValuePath) {
+    fromValuePath;
+    toValuePath;
+  }
   internalSetFieldValue(valuePath, value, validation) {
     const { revert } = this.getAdapterForValuePath(valuePath);
     assertExists(revert, "setting value not supported {}", valuePath);
@@ -887,7 +892,7 @@ function useDefaultMobxFormHooks(model, {
   const onFieldBlur = useCallback(
     function(path) {
       setTimeout(function() {
-        if (model.isValuePathActive(path) && model.isFieldDirty(path)) {
+        if (model.isValuePathActive(path) && model.isFieldDirty(path) && model.fields[path].error == null) {
           model.validateField(path, Math.max(1 /* Changed */, model.getValidation(path)));
         }
       }, 100);

package/package.json

@@ -72,7 +72,7 @@
     "test:watch": "vitest"
   },
   "type": "module",
-  "version": "0.0.26",
+  "version": "0.0.27",
   "exports": {
     ".": {
       "import": {