@b9g/crank 0.5.1 → 0.5.3

package/crank.cjs

@@ -862,8 +862,8 @@ const _ContextImpl = Symbol.for("crank.ContextImpl");
  * element tree are connected via contexts. Components can use this tree to
  * communicate data upwards via events and downwards via provisions.
  *
- * @template [TProps=*] - The expected shape of the props passed to the
- * component. Used to strongly type the Context iterator methods.
+ * @template [T=*] - The expected shape of the props passed to the component,
+ * or a component function. Used to strongly type the Context iterator methods.
  * @template [TResult=*] - The readable element value type. It is used in
  * places such as the return value of refresh and the argument passed to
  * schedule and cleanup callbacks.
@@ -1020,6 +1020,11 @@ class Context {
      */
     cleanup(callback) {
         const ctx = this[_ContextImpl];
+        if (ctx.f & IsUnmounted) {
+            const value = ctx.renderer.read(getValue(ctx.ret));
+            callback(value);
+            return;
+        }
         let callbacks = cleanupMap.get(ctx);
         if (!callbacks) {
             callbacks = new Set();
@@ -1430,6 +1435,9 @@ function enqueueComponentRun(ctx, hydrationData) {
         }
         catch (err) {
             if (!(ctx.f & IsUpdating)) {
+                if (!ctx.parent) {
+                    throw err;
+                }
                 return propagateError(ctx.parent, err);
             }
             throw err;
@@ -1453,6 +1461,9 @@ function enqueueComponentRun(ctx, hydrationData) {
             }
             catch (err) {
                 if (!(ctx.f & IsUpdating)) {
+                    if (!ctx.parent) {
+                        throw err;
+                    }
                     return propagateError(ctx.parent, err);
                 }
                 throw err;
@@ -1695,7 +1706,16 @@ async function runAsyncGenComponent(ctx, iterationP, hydrationData) {
                 // children. Sync generator components only resume when their children
                 // have fulfilled so the element’s inflight child values will never be
                 // defined.
-                oldValue = ctx.ret.inflightValue.then((value) => ctx.renderer.read(value), () => ctx.renderer.read(undefined));
+                oldValue = ctx.ret.inflightValue.then((value) => ctx.renderer.read(value));
+                oldValue.catch((err) => {
+                    if (ctx.f & IsUpdating) {
+                        return;
+                    }
+                    if (!ctx.parent) {
+                        throw err;
+                    }
+                    return propagateError(ctx.parent, err);
+                });
             }
             else {
                 oldValue = ctx.renderer.read(getValue(ctx.ret));
@@ -1773,7 +1793,10 @@ function unmountComponent(ctx) {
                         returnComponent(ctx);
                     }
                 }, (err) => {
-                    propagateError(ctx.parent, err);
+                    if (!ctx.parent) {
+                        throw err;
+                    }
+                    return propagateError(ctx.parent, err);
                 });
             }
             else {
@@ -1796,7 +1819,10 @@ function unmountComponent(ctx) {
                         returnComponent(ctx);
                     }
                 }, (err) => {
-                    propagateError(ctx.parent, err);
+                    if (!ctx.parent) {
+                        throw err;
+                    }
+                    return propagateError(ctx.parent, err);
                 });
             }
             else {
@@ -1814,7 +1840,12 @@ function returnComponent(ctx) {
             ctx.f |= IsSyncExecuting;
             const iteration = ctx.iterator.return();
             if (isPromiseLike(iteration)) {
-                iteration.catch((err) => propagateError(ctx.parent, err));
+                iteration.catch((err) => {
+                    if (!ctx.parent) {
+                        throw err;
+                    }
+                    return propagateError(ctx.parent, err);
+                });
             }
         }
         finally {
@@ -1927,18 +1958,23 @@ function handleChildError(ctx, err) {
     return updateComponentChildren(ctx, iteration.value);
 }
 function propagateError(ctx, err) {
-    if (ctx === undefined) {
-        throw err;
-    }
     let result;
     try {
         result = handleChildError(ctx, err);
     }
     catch (err) {
+        if (!ctx.parent) {
+            throw err;
+        }
         return propagateError(ctx.parent, err);
     }
     if (isPromiseLike(result)) {
-        return result.catch((err) => propagateError(ctx.parent, err));
+        return result.catch((err) => {
+            if (!ctx.parent) {
+                throw err;
+            }
+            return propagateError(ctx.parent, err);
+        });
     }
     return result;
 }