@masterteam/properties 0.0.65 → 0.0.66

package/fesm2022/masterteam-properties.mjs

@@ -1,5 +1,5 @@
 import * as i0 from '@angular/core';
-import { inject, Injectable, computed, viewChild, input, linkedSignal, signal, Component, DestroyRef, effect, forwardRef, ChangeDetectionStrategy, EnvironmentInjector, ViewContainerRef, ViewChild } from '@angular/core';
+import { inject, Injectable, computed, viewChild, input, linkedSignal, signal, Component, DestroyRef, effect, forwardRef, ChangeDetectionStrategy, EnvironmentInjector, untracked, ViewContainerRef, ViewChild } from '@angular/core';
 import { CommonModule } from '@angular/common';
 import { Table } from '@masterteam/components/table';
 import { Router, ActivatedRoute } from '@angular/router';
@@ -2657,7 +2657,10 @@ class PropertyForm {
         effect(() => {
             const defaultViewType = this.facade.defaultViewType();
             if (!this.propertyId() && defaultViewType) {
-                this.mainControl.patchValue({ ...this.mainControl.value, viewType: defaultViewType }, { emitEvent: true });
+                // untracked: see syncFormWithSelectedProperty effect — patchValue runs
+                // DynamicForm.writeValue → evaluateRelations (reads+writes its signals),
+                // which would otherwise make this effect self-dependent and loop.
+                untracked(() => this.mainControl.patchValue({ ...this.mainControl.value, viewType: defaultViewType }, { emitEvent: true }));
             }
         });
         // Track previous property type
@@ -2668,7 +2671,10 @@ class PropertyForm {
                 prev = this.propertyType();
                 return;
             }
-            this.configurationControl.reset(null);
+            // untracked: reset can reach a bound config DynamicForm.writeValue →
+            // evaluateRelations (reads+writes its signals); keep this effect bound
+            // only to propertyType so it doesn't self-trigger.
+            untracked(() => this.configurationControl.reset(null));
         });
         effect(() => {
             const currentPropertyType = this.propertyType();
@@ -2684,7 +2690,12 @@ class PropertyForm {
         });
         effect(() => {
             const property = this.facade.selected();
-            this.syncFormWithSelectedProperty(property);
+            // Patch the form imperatively, untracked: mainControl.patchValue triggers
+            // DynamicForm.writeValue → evaluateRelations, which both reads and writes
+            // the DynamicForm's signals (fieldStates, etc.). Without untracked those
+            // reads register as dependencies of THIS effect, which the same call then
+            // mutates — so the effect re-runs forever (the freeze-on-save loop).
+            untracked(() => this.syncFormWithSelectedProperty(property));
         });
         effect(() => {
             this.resolveConfigurationComponent(this.selectedPropertyTypeConfiguration());