@mintjamsinc/ichigojs 0.1.88 → 0.1.89

package/dist/ichigo.esm.js

@@ -10795,8 +10795,7 @@ class VNode {
             const changed = this.bindings.isAffected(changes, this.#textEvaluator.identifiers, this.#textEvaluator.trackedPaths);
             // If the text node has changed, update its content
             if (changed) {
-                const text = this.#node;
-                text.data = this.#textEvaluator.evaluate(this.bindings);
+                this.#renderText();
             }
         }
         else if (this.#nodeType === Node.ELEMENT_NODE) {
@@ -10817,7 +10816,7 @@ class VNode {
                 for (const preparer of this.#directiveManager.bindingsPreparers) {
                     const changed = this.bindings.isAffected(changes, preparer.dependentIdentifiers, preparer.trackedPaths);
                     if (changed) {
-                        preparer.prepareBindings();
+                        this.#renderSafely(() => preparer.prepareBindings());
                     }
                 }
             }
@@ -10826,7 +10825,7 @@ class VNode {
                 for (const updater of this.#directiveManager.domUpdaters) {
                     const changed = this.bindings.isAffected(changes, updater.dependentIdentifiers, updater.trackedPaths);
                     if (changed) {
-                        updater.applyToDOM();
+                        this.#renderSafely(() => updater.applyToDOM());
                     }
                 }
             }
@@ -10864,8 +10863,7 @@ class VNode {
     forceUpdate() {
         if (this.#nodeType === Node.TEXT_NODE && this.#textEvaluator) {
             // If this is a text node with a text evaluator, update its content if needed
-            const text = this.#node;
-            text.data = this.#textEvaluator.evaluate(this.bindings);
+            this.#renderText();
         }
         else if (this.#nodeType === Node.ELEMENT_NODE) {
             // If this is an element node, update directives and child nodes
@@ -10883,13 +10881,13 @@ class VNode {
                 }
                 // Prepare bindings for each preparer if relevant identifiers have changed
                 for (const preparer of this.#directiveManager.bindingsPreparers) {
-                    preparer.prepareBindings();
+                    this.#renderSafely(() => preparer.prepareBindings());
                 }
             }
             // Apply DOM updaters from directives, if any
             if (this.#directiveManager?.domUpdaters) {
                 for (const updater of this.#directiveManager.domUpdaters) {
-                    updater.applyToDOM();
+                    this.#renderSafely(() => updater.applyToDOM());
                 }
             }
             // Recursively update child virtual nodes. A copy: an update can
@@ -10911,6 +10909,38 @@ class VNode {
             });
         }
     }
+    /**
+     * Evaluates this text node's expressions into its content.
+     */
+    #renderText() {
+        const text = this.#node;
+        this.#renderSafely(() => {
+            text.data = this.#textEvaluator.evaluate(this.bindings);
+        });
+    }
+    /**
+     * Runs one binding's render step. A binding whose expression throws (for example one that
+     * reads a member of a value that is still undefined) is logged and left as it was, so that
+     * it does not stop the rest of the tree from rendering.
+     * @param render The render step.
+     */
+    #renderSafely(render) {
+        try {
+            render();
+        }
+        catch (error) {
+            this.#vApplication.logManager.getLogger(this.constructor.name).error(`Error rendering ${this.#describe()}: ${error}`);
+        }
+    }
+    /**
+     * Describes this node for a log message: a text node by its text, an element by its tag.
+     */
+    #describe() {
+        if (this.#nodeType === Node.TEXT_NODE) {
+            return `text "${(this.#node.textContent ?? '').trim()}"`;
+        }
+        return `<${this.#nodeName.toLowerCase()}>`;
+    }
     /**
      * Adds a child virtual node to this virtual node.
      * @param child The child virtual node to add.
@@ -15637,8 +15667,11 @@ class VApplication {
             vApplication: this,
             bindings: this.#bindings
         });
-        // Initial rendering
-        this.#vNode.update();
+        // Initial rendering. Every binding is applied, not only those whose identifiers were
+        // recorded as changed while the data was set up: a value that starts undefined is never
+        // recorded (undefined -> undefined is no change), and an expression over globals only
+        // has no identifier to record at all.
+        this.#vNode.forceUpdate();
         // Expand the components in the tree. Last, so every component element
         // has been compiled and has had its prop values delivered before its
         // template is cloned in. See VNode.expandComponents.