@mintjamsinc/ichigojs 0.1.88 → 0.1.89

package/dist/ichigo.cjs

@@ -10801,8 +10801,7 @@
                 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) {
@@ -10823,7 +10822,7 @@
                     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());
                         }
                     }
                 }
@@ -10832,7 +10831,7 @@
                     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());
                         }
                     }
                 }
@@ -10870,8 +10869,7 @@
         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
@@ -10889,13 +10887,13 @@
                     }
                     // 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
@@ -10917,6 +10915,38 @@
                 });
             }
         }
+        /**
+         * 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.
@@ -15643,8 +15673,11 @@
                 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.