@qwik.dev/core 2.0.0-beta.42 → 2.0.0-beta.43

package/dist/core.mjs

@@ -1,6 +1,6 @@
 /**
  * @license
- * @qwik.dev/core 2.0.0-beta.42-dev+3473a49
+ * @qwik.dev/core 2.0.0-beta.43-dev+e478dfb
  * Copyright QwikDev. All Rights Reserved.
  * Use of this source code is governed by an MIT-style license that can be
  * found in the LICENSE file at https://github.com/QwikDev/qwik/blob/main/LICENSE
@@ -189,9 +189,6 @@ const isString = (v) => {
 const isFunction = (v) => {
     return typeof v === 'function';
 };
-const isPrimitiveOrNullUndefined = (v) => {
-    return (typeof v !== 'object' && typeof v !== 'function') || v === null || v === undefined;
-};
 
 const baseUrl = 'https://qwikdev-build-v2.qwik-8nx.pages.dev/docs/errors/#q';
 const codeToText = (code, ...parts) => {
@@ -475,6 +472,7 @@ const QDefaultSlot = '';
 const ELEMENT_ID = 'q:id';
 const ELEMENT_KEY = 'q:key';
 const ELEMENT_PROPS = 'q:props';
+const USE_ON_PLACEHOLDER_KEY = ':useOn';
 /** @internal */
 const ELEMENT_SEQ = 'q:seq';
 const ELEMENT_SEQ_IDX = 'q:seqIdx';
@@ -1281,7 +1279,7 @@ const COMMA = ',';
  *
  * @public
  */
-const version = "2.0.0-beta.42-dev+3473a49";
+const version = "2.0.0-beta.43-dev+e478dfb";
 
 const isNode = (value) => {
     return value && typeof value.nodeType === 'number';
@@ -5528,11 +5526,14 @@ function cleanup(container, journal, vNode, cursorRoot = null) {
                         const obj = seq[i];
                         if (isObject(obj)) {
                             const objIsTask = isTask(obj);
-                            if (objIsTask && obj.$flags$ & 1 /* TaskFlags.VISIBLE_TASK */) {
-                                obj.$flags$ |= 16 /* TaskFlags.NEEDS_CLEANUP */;
-                                markVNodeDirty(container, vCursor, 128 /* ChoreBits.CLEANUP */, cursorRoot);
-                                // don't call cleanupDestroyable yet, do it by the scheduler
-                                continue;
+                            if (objIsTask) {
+                                clearAllEffects(container, obj);
+                                if (obj.$flags$ & 1 /* TaskFlags.VISIBLE_TASK */) {
+                                    obj.$flags$ |= 16 /* TaskFlags.NEEDS_CLEANUP */;
+                                    markVNodeDirty(container, vCursor, 128 /* ChoreBits.CLEANUP */, cursorRoot);
+                                    // don't call cleanupDestroyable yet, do it by the scheduler
+                                    continue;
+                                }
                             }
                             // Stores and plain signals are only producers; their subscriptions are removed
                             // when cleaning the consumers that read them. They don't own reactive backrefs.
@@ -5988,43 +5989,16 @@ function findFirstElementNode(jsx) {
  * This is necessary for headless components (components that don't render a real DOM element) to
  * have an anchor point for `useOn` event listeners that target the document or window.
  *
- * @param jsx The JSX output to modify.
+ * @param jsx The JSX output of the component.
  * @param placeholder The placeholder element to inject.
- * @returns The modified JSX output.
+ * @returns The JSX output with the placeholder as its last sibling.
  */
 function injectPlaceholderElement(jsx, placeholder) {
-    // For regular JSX nodes, we can append the placeholder to its children.
-    if (isJSXNode(jsx)) {
-        // Inline components may ignore children, while component$ children become projections.
-        if (jsx.type !== Fragment) {
-            return _jsxSorted(Fragment, null, null, [jsx, placeholder], 0, null);
-        }
-        if (jsx.children == null) {
-            jsx.children = placeholder;
-        }
-        else if (isArray(jsx.children)) {
-            jsx.children.push(placeholder);
-        }
-        else {
-            jsx.children = [jsx.children, placeholder];
-        }
-        return jsx;
-    }
-    // For primitives, we can't add children, so we wrap them in a fragment.
-    if (isPrimitiveOrNullUndefined(jsx)) {
-        return _jsxSorted(Fragment, null, null, [jsx, placeholder], 0, null);
-    }
-    // For an array of nodes, we inject the placeholder into the first element.
-    if (isArray(jsx) && jsx.length > 0) {
-        injectPlaceholderElement(jsx[0], placeholder);
-        return jsx;
-    }
-    // For anything else we do nothing.
-    return jsx;
+    return isArray(jsx) ? [...jsx, placeholder] : [jsx, placeholder];
 }
 /** @returns An empty <script> element for adding qwik metadata attributes to */
 function createPlaceholderScriptNode() {
-    return new JSXNodeImpl('script', null, { hidden: '' }, null, 0, null);
+    return new JSXNodeImpl('script', null, { hidden: '' }, null, 0, USE_ON_PLACEHOLDER_KEY);
 }
 
 function collectChildren(first, last = null) {
@@ -15196,7 +15170,8 @@ class DomContainer extends _SharedContainer {
         if (!__EXPERIMENTAL__.errorBoundary) {
             const errorStore = host && this.resolveContext(host, ERROR_CONTEXT);
             if (!errorStore) {
-                throw err;
+                logErrorAndThrowAsync(err);
+                return;
             }
             errorStore.error = err;
             return;