@adaas/are-html 0.0.23 → 0.0.24

package/dist/browser/index.d.mts

@@ -295,7 +295,23 @@ declare class AreDirectiveContext extends A_ExecutionContext {
 declare class AreDirectiveIf extends AreDirective {
     transform(attribute: AreDirectiveAttribute, scope: A_Scope, store: AreStore, scene: AreScene, logger: A_Logger, ...args: any[]): void;
     compile(attribute: AreDirectiveAttribute, store: AreStore, scene: AreScene, syntax: AreSyntax, directiveContext?: AreDirectiveContext, ...args: any[]): void;
-    update(attribute: AreDirectiveAttribute, store: AreStore, scope: A_Scope, syntax: AreSyntax, scene: AreScene, ...args: any[]): void;
+    update(attribute: AreDirectiveAttribute, store: AreStore, scope: A_Scope, syntax: AreSyntax, scene: AreScene, directiveContext?: AreDirectiveContext, ...args: any[]): void;
+    /**
+     * Evaluates the `$if` condition defensively.
+     *
+     * A condition can reference data that is momentarily unavailable — most
+     * commonly a nested `$if` (e.g. `$if="selected.fields.length"`) living
+     * inside a parent `$if="selected"` whose object has just become `null`.
+     * Because the nested directive is still subscribed to the store, its
+     * update fires on that same change and the raw expression would throw
+     * `Cannot read properties of null`, crashing the whole update pipeline.
+     *
+     * Treating an evaluation error as `false` is the correct contract for a
+     * conditional: if the condition cannot be resolved, the subtree simply
+     * stays hidden until the referenced data is present again (at which point
+     * the parent `$if` re-activates and re-evaluates this one).
+     */
+    private evaluateCondition;
 }
 
 /**
@@ -777,7 +793,7 @@ declare class AreHTMLCompiler extends AreCompiler {
     compileStaticAttribute(attribute: AreStaticAttribute, scene: AreScene, ...args: any[]): void;
     compileDirectiveAttribute(directive: AreDirectiveAttribute, store: AreStore, feature: A_Feature, logger?: A_Logger, ...args: any[]): void;
     compileEventAttribute(attribute: AreEventAttribute, scene: AreScene, ...args: any[]): void;
-    compileBindingAttribute(attribute: AreBindingAttribute, scene: AreScene, parentStore: AreStore, store: AreStore, syntax: AreSyntax, ...args: any[]): void;
+    compileBindingAttribute(attribute: AreBindingAttribute, scene: AreScene, parentStore: AreStore, store: AreStore, syntax: AreSyntax, directiveContext?: AreDirectiveContext, ...args: any[]): void;
 }
 
 /**

package/dist/browser/index.mjs

@@ -952,9 +952,7 @@ var AreDirectiveIf = class extends AreDirective {
     attribute.template = ifTemplate;
   }
   compile(attribute, store, scene, syntax, directiveContext, ...args) {
-    attribute.value = syntax.evaluate(attribute.content, store, {
-      ...directiveContext?.scope || {}
-    });
+    attribute.value = this.evaluateCondition(syntax, attribute, store, directiveContext);
     const hostInstruction = scene.host;
     const commentIdentifier = ` --- if: ${attribute.template.id} --- `;
     const declaration = new AddCommentInstruction({ content: commentIdentifier });
@@ -966,9 +964,9 @@ var AreDirectiveIf = class extends AreDirective {
     else
       attribute.template.scene.deactivate();
   }
-  update(attribute, store, scope, syntax, scene, ...args) {
+  update(attribute, store, scope, syntax, scene, directiveContext, ...args) {
     const previous = !!attribute.value;
-    const next = !!syntax.evaluate(attribute.content, store);
+    const next = this.evaluateCondition(syntax, attribute, store, directiveContext);
     attribute.value = next;
     if (previous === next) return;
     if (next) {
@@ -979,6 +977,30 @@ var AreDirectiveIf = class extends AreDirective {
       attribute.template.scene.deactivate();
     }
   }
+  /**
+   * Evaluates the `$if` condition defensively.
+   *
+   * A condition can reference data that is momentarily unavailable — most
+   * commonly a nested `$if` (e.g. `$if="selected.fields.length"`) living
+   * inside a parent `$if="selected"` whose object has just become `null`.
+   * Because the nested directive is still subscribed to the store, its
+   * update fires on that same change and the raw expression would throw
+   * `Cannot read properties of null`, crashing the whole update pipeline.
+   *
+   * Treating an evaluation error as `false` is the correct contract for a
+   * conditional: if the condition cannot be resolved, the subtree simply
+   * stays hidden until the referenced data is present again (at which point
+   * the parent `$if` re-activates and re-evaluates this one).
+   */
+  evaluateCondition(syntax, attribute, store, directiveContext) {
+    try {
+      return !!syntax.evaluate(attribute.content, store, {
+        ...directiveContext?.scope || {}
+      });
+    } catch {
+      return false;
+    }
+  }
 };
 __decorateClass([
   AreDirective.Transform,
@@ -1002,7 +1024,8 @@ __decorateClass([
   __decorateParam(1, A_Inject(AreStore)),
   __decorateParam(2, A_Inject(A_Scope)),
   __decorateParam(3, A_Inject(AreSyntax)),
-  __decorateParam(4, A_Inject(AreScene))
+  __decorateParam(4, A_Inject(AreScene)),
+  __decorateParam(5, A_Inject(AreDirectiveContext))
 ], AreDirectiveIf.prototype, "update", 1);
 AreDirectiveIf = __decorateClass([
   A_Frame.Define({
@@ -1789,7 +1812,7 @@ var AreHTMLCompiler = class extends AreCompiler {
       handler: attribute.content
     }));
   }
-  compileBindingAttribute(attribute, scene, parentStore, store, syntax, ...args) {
+  compileBindingAttribute(attribute, scene, parentStore, store, syntax, directiveContext, ...args) {
     if (!scene.host)
       throw new AreCompilerError({
         title: "Scene Host Not Found",
@@ -1825,11 +1848,12 @@ var AreHTMLCompiler = class extends AreCompiler {
         }
         return value;
       };
+      const directiveScope = () => directiveContext?.scope ?? {};
       const watcher = {
         update: () => {
           try {
             parentStore.watch(watcher);
-            const next = coerce(syntax.evaluate(attribute.content, parentStore));
+            const next = coerce(syntax.evaluate(attribute.content, parentStore, directiveScope()));
             parentStore.unwatch(watcher);
             store.set(propName, next);
           } catch (e) {
@@ -1838,7 +1862,7 @@ var AreHTMLCompiler = class extends AreCompiler {
         }
       };
       parentStore.watch(watcher);
-      const initial = coerce(syntax.evaluate(attribute.content, parentStore));
+      const initial = coerce(syntax.evaluate(attribute.content, parentStore, directiveScope()));
       parentStore.unwatch(watcher);
       store.set(propName, initial);
       return;
@@ -1894,7 +1918,8 @@ __decorateClass([
   __decorateParam(2, A_Dependency.Parent()),
   __decorateParam(2, A_Inject(AreStore)),
   __decorateParam(3, A_Inject(AreStore)),
-  __decorateParam(4, A_Inject(AreSyntax))
+  __decorateParam(4, A_Inject(AreSyntax)),
+  __decorateParam(5, A_Inject(AreDirectiveContext))
 ], AreHTMLCompiler.prototype, "compileBindingAttribute", 1);
 AreHTMLCompiler = __decorateClass([
   A_Frame.Define({