sibujs 4.1.0 → 4.3.0

package/dist/{chunk-ANGSYNPX.js → chunk-2LSY4WDM.js}

@@ -135,40 +135,42 @@ import {
   use,
   var_,
   video
-} from "./chunk-MRAMKDHQ.js";
+} from "./chunk-7UK6BSAT.js";
 import {
   trustHTML
-} from "./chunk-HYCCIYNS.js";
+} from "./chunk-NIOYEGBQ.js";
 import {
   watch
-} from "./chunk-KSOEJGFO.js";
+} from "./chunk-IYHM4FWD.js";
 import {
   __resetIdCounter,
   createId
 } from "./chunk-CCSJMTRN.js";
 import {
   context
-} from "./chunk-5GJ5RTHA.js";
+} from "./chunk-L4B3Y3OM.js";
 import {
   SVG_NS,
   bindChildNode,
+  captureFocusWithin,
+  restoreFocusWithin,
   tagFactory
-} from "./chunk-I3VI34DP.js";
+} from "./chunk-XGCZ45NH.js";
 import {
   bindAttribute,
   bindDynamic,
   setSafeAttribute
-} from "./chunk-SHDVWATN.js";
+} from "./chunk-7UBG6LDP.js";
 import {
   derived
-} from "./chunk-DCDS2PQ6.js";
+} from "./chunk-3E5YNK2F.js";
 import {
   globalSingleton
 } from "./chunk-7ZHH77QA.js";
 import {
   effect,
   on
-} from "./chunk-PFTDGEPZ.js";
+} from "./chunk-7BJLKDBI.js";
 import {
   disableSSR,
   enableSSR,
@@ -177,7 +179,7 @@ import {
   isSSR,
   runInSSRContext,
   withSSR
-} from "./chunk-TIRZCERI.js";
+} from "./chunk-5DXA2J44.js";
 import {
   MAX_DRAIN_TEARDOWNS,
   checkLeaks,
@@ -186,14 +188,14 @@ import {
   replaceChildrenSafely,
   reportDrainRunaway,
   unregisterDisposer
-} from "./chunk-DHDZ7IVN.js";
+} from "./chunk-QKRPLZ2V.js";
 import {
   batch,
   enqueueBatchedSignal,
   external,
   isBatching,
   signal
-} from "./chunk-UFSQEO7P.js";
+} from "./chunk-MYA5HEK5.js";
 import {
   notifySubscribers,
   reactiveBinding,
@@ -204,17 +206,19 @@ import {
   suspendTracking,
   track,
   untracked
-} from "./chunk-D3SZ4LVS.js";
+} from "./chunk-TSL64O5L.js";
 import {
+  DEV,
   __export,
   devAssert,
   devWarn,
+  devWarnLazy,
   getRuntimeErrorHandler,
   isDev,
   normalizeError,
   reportError,
   setRuntimeErrorHandler
-} from "./chunk-VPP2FONR.js";
+} from "./chunk-2WLZ6757.js";
 
 // index.ts
 var index_exports = {};
@@ -1749,34 +1753,53 @@ function setGlobalErrorHandler(handler) {
 
 // src/core/rendering/directives.ts
 function show(condition, element) {
+  const resolved = typeof element === "function" ? element() : element;
   const update = () => {
-    element.style.display = condition() ? "" : "none";
+    resolved.style.display = condition() ? "" : "none";
   };
-  registerDisposer(element, track(update));
-  return element;
+  registerDisposer(resolved, track(update));
+  return resolved;
+}
+function attachBranch(parent, anchor, branch, attachedOnce, where) {
+  const result = typeof branch === "function" ? branch() : branch;
+  if (result == null || typeof result === "boolean") return { node: null, owned: false };
+  const node = result instanceof Node ? result : document.createTextNode(String(result));
+  const handedIn = node === branch;
+  const seenBefore = attachedOnce.has(node);
+  const owned = !handedIn && !seenBefore;
+  attachedOnce.add(node);
+  if (seenBefore) {
+    devWarnLazy(
+      () => `${where}: ${handedIn ? "a branch was given as an element rather than a function" : "a branch factory returned a node it had already returned"}, so the SAME node is being re-attached \u2014 any state it accumulated while detached (input value, scroll position, classes set imperatively) comes back with it. Its reactive bindings are left intact rather than disposed, because the node is not this directive's to tear down. Return a FRESH node per call (\`() => div(\u2026)\`) for rebuild semantics, or keep this form deliberately when the reuse is what you want.`
+    );
+  }
+  parent.insertBefore(node, anchor.nextSibling);
+  return { node, owned };
 }
 function when(condition, thenBranch, elseBranch) {
   const anchor = document.createComment("when");
   let currentNode = null;
   let lastCondition;
   let initialized = false;
+  const attachedOnce = /* @__PURE__ */ new WeakSet();
+  let currentNodeOwned = false;
   const update = () => {
     const show2 = condition();
     const parent = anchor.parentNode;
     if (!parent) return;
     if (initialized && show2 === lastCondition) return;
     lastCondition = show2;
+    const focused = currentNode ? captureFocusWithin([currentNode]) : null;
     if (currentNode?.parentNode) {
-      dispose(currentNode);
+      if (currentNodeOwned) dispose(currentNode);
       currentNode.parentNode.removeChild(currentNode);
       currentNode = null;
     }
-    const result = show2 ? thenBranch() : elseBranch ? elseBranch() : null;
-    if (result != null) {
-      const node = result instanceof Node ? result : document.createTextNode(String(result));
-      parent.insertBefore(node, anchor.nextSibling);
-      currentNode = node;
-    }
+    const branch = show2 ? thenBranch : elseBranch !== void 0 ? elseBranch : null;
+    const attached = attachBranch(parent, anchor, branch, attachedOnce, "when");
+    currentNode = attached.node;
+    currentNodeOwned = attached.owned;
+    restoreFocusWithin(focused, currentNode ? [currentNode] : [], "when");
     initialized = true;
   };
   registerDisposer(anchor, track(update));
@@ -1792,26 +1815,25 @@ function match(value, cases, fallback) {
   let currentNode = null;
   let lastKey;
   let initialized = false;
+  const attachedOnce = /* @__PURE__ */ new WeakSet();
+  let currentNodeOwned = false;
   const update = () => {
     const key = String(value());
     const parent = anchor.parentNode;
     if (!parent) return;
     if (initialized && key === lastKey) return;
     lastKey = key;
+    const focused = currentNode ? captureFocusWithin([currentNode]) : null;
     if (currentNode?.parentNode) {
-      dispose(currentNode);
+      if (currentNodeOwned) dispose(currentNode);
       currentNode.parentNode.removeChild(currentNode);
       currentNode = null;
     }
-    const renderFn = cases[key] || fallback;
-    if (renderFn) {
-      const result = renderFn();
-      if (result != null) {
-        const node = result instanceof Node ? result : document.createTextNode(String(result));
-        parent.insertBefore(node, anchor.nextSibling);
-        currentNode = node;
-      }
-    }
+    const branch = Object.hasOwn(cases, key) ? cases[key] : fallback;
+    const attached = attachBranch(parent, anchor, branch ?? null, attachedOnce, "match");
+    currentNode = attached.node;
+    currentNodeOwned = attached.owned;
+    restoreFocusWithin(focused, currentNode ? [currentNode] : [], "match");
     initialized = true;
   };
   registerDisposer(anchor, track(update));
@@ -1856,7 +1878,6 @@ function DynamicComponent(is) {
 }
 
 // src/core/rendering/each.ts
-var _isDev = isDev();
 function resolveNodeChild(child) {
   if (typeof child === "function") {
     return resolveNodeChild(child());
@@ -1947,7 +1968,7 @@ function each(getArray, render, options) {
     workMap.clear();
     keyIndexMap.clear();
     for (let i2 = 0; i2 < newLen; i2++) {
-      if (_isDev && keyIndexMap.has(newKeys[i2])) {
+      if (DEV && keyIndexMap.has(newKeys[i2])) {
         devWarn(
           `each: duplicate key "${String(newKeys[i2])}" at index ${i2} (first seen at ${keyIndexMap.get(newKeys[i2])}). Keys must be unique \u2014 duplicates cause rows to be dropped or mis-ordered.`
         );
@@ -2131,7 +2152,6 @@ function resolveChild(child) {
 }
 
 // src/core/rendering/htm.ts
-var _isDev2 = isDev();
 var RAW_TEXT_TAGS = /* @__PURE__ */ new Set(["script", "style"]);
 var VOID_ELEMENTS = /* @__PURE__ */ new Set([
   "area",
@@ -2411,7 +2431,7 @@ function executeElement(tmpl, values) {
         const fn = values[attr.idx];
         if (typeof fn === "function") {
           el.addEventListener(attr.name, fn);
-        } else if (_isDev2) {
+        } else if (DEV) {
           devWarn(
             `html: on:${attr.name} handler is not a function (got ${typeof fn}). Event listener was not attached.`
           );
@@ -3278,7 +3298,16 @@ function enhance(target, setup) {
   const teardowns = [];
   const owner = /* @__PURE__ */ Symbol("sibujs.enhance");
   let disposed = false;
-  const bind = (target_, fn) => {
+  let closed = false;
+  const isClosed = (method) => {
+    if (!closed) return false;
+    devWarnLazy(
+      () => `enhance: ctx.${method}() was called after this enhancement was rolled back or disposed, so it was ignored. The setup is still running past the point where its transaction ended \u2014 usually an async setup continuing after an await, or a callback it queued before it threw. Registrations made now would belong to nothing: the root carries no enhancement marker and the disposer has already run, so nothing could ever release them.`
+    );
+    return true;
+  };
+  const bind = (method, target_, fn) => {
+    if (isClosed(method)) return;
     const el = resolveTarget(root, target_);
     if (!el) {
       if (typeof console !== "undefined") {
@@ -3300,14 +3329,14 @@ function enhance(target, setup) {
       }
     },
     on: (t, event, handler, options) => {
-      bind(t, (el) => {
+      bind("on", t, (el) => {
         const wrapped = (e) => handler(e, el);
         el.addEventListener(event, wrapped, options);
         teardowns.push(() => el.removeEventListener(event, wrapped, options));
       });
     },
     text: (t, value) => {
-      bind(t, (el) => {
+      bind("text", t, (el) => {
         teardowns.push(
           bindNode(el, () => {
             const v = value();
@@ -3318,7 +3347,7 @@ function enhance(target, setup) {
       });
     },
     attr: (t, name, value) => {
-      bind(t, (el) => {
+      bind("attr", t, (el) => {
         teardowns.push(
           bindNode(el, () => {
             const v = value();
@@ -3329,7 +3358,7 @@ function enhance(target, setup) {
       });
     },
     classed: (t, name, on2) => {
-      bind(t, (el) => {
+      bind("classed", t, (el) => {
         teardowns.push(
           bindNode(el, () => {
             el.classList.toggle(name, Boolean(on2()));
@@ -3338,7 +3367,7 @@ function enhance(target, setup) {
       });
     },
     show: (t, when2) => {
-      bind(t, (el) => {
+      bind("show", t, (el) => {
         const prevHidden = el.hidden;
         teardowns.push(
           bindNode(el, () => {
@@ -3351,7 +3380,7 @@ function enhance(target, setup) {
       });
     },
     model: (t, state, options) => {
-      bind(t, (el) => {
+      bind("model", t, (el) => {
         const control = el;
         const [get, set] = state;
         const evt = options?.event ?? (control instanceof HTMLSelectElement || control instanceof HTMLInputElement && (control.type === "checkbox" || control.type === "radio") ? "change" : "input");
@@ -3366,6 +3395,7 @@ function enhance(target, setup) {
       });
     },
     each: (target_, describe) => {
+      if (isClosed("each")) return;
       devAssert(typeof describe === "function", "ctx.each: second argument must be a function.");
       const elements = typeof target_ === "string" ? ctx.refs(target_) : Array.from(target_);
       for (let i2 = 0; i2 < elements.length; i2++) {
@@ -3376,14 +3406,25 @@ function enhance(target, setup) {
       }
     },
     cleanup: (fn) => {
+      if (isClosed("cleanup")) return;
       teardowns.push(fn);
     }
   };
   let extra;
   try {
     extra = setup(ctx);
+    const returned = extra;
+    if (returned && typeof returned.then === "function") {
+      returned.then(void 0, (reason) => {
+        devWarnLazy(() => `enhance: the promise returned by the setup also rejected: ${String(reason)}`);
+      });
+      throw new Error(
+        (typeof __SIBU_DEV__ !== "undefined" ? __SIBU_DEV__ : DEV) ? "[SibuJS enhance] the setup returned a promise, so its work did not complete inside the enhancement transaction. If it is a lazy import, register it as lazyIsland(() => import(\u2026)): an unwrapped loader is called as a setup, so its module is fetched but the setup it resolves to is discarded. If it is an async setup, make it synchronous \u2014 this enhancement has been rolled back, and ctx registrations made after this point are ignored." : "[SibuJS enhance] setup returned a promise"
+      );
+    }
   } catch (err) {
     drainTeardowns(teardowns, "enhance");
+    closed = true;
     throw err;
   }
   if (typeof extra === "function") teardowns.push(extra);
@@ -3396,6 +3437,7 @@ function enhance(target, setup) {
     }
     unregisterDisposer(root, dispose2);
     drainTeardowns(teardowns, "enhance");
+    closed = true;
   };
   registerDisposer(root, dispose2);
   enhancementOwner.set(root, owner);

package/dist/{chunk-VPP2FONR.js → chunk-2WLZ6757.js}

@@ -11,23 +11,28 @@ var __export = (target, all) => {
 };
 
 // src/core/dev.ts
+var DEV = typeof __SIBU_DEV__ !== "undefined" ? !!__SIBU_DEV__ : typeof globalThis.__SIBU_DEV__ !== "undefined" ? !!globalThis.__SIBU_DEV__ : (
+  // safe default: off in browser, on in test/dev Node
+  typeof process !== "undefined" && process.env?.NODE_ENV !== "production"
+);
 function isDev() {
-  return typeof globalThis.__SIBU_DEV__ !== "undefined" ? !!globalThis.__SIBU_DEV__ : (
-    // The bare `__SIBU_DEV__` is a bundler define that only exists in
-    // production builds; under the test runner it is always undefined, so
-    // this branch is unreachable here.
-    /* v8 ignore next 2 */
-    typeof __SIBU_DEV__ !== "undefined" ? __SIBU_DEV__ : typeof process !== "undefined" && process.env?.NODE_ENV !== "production"
-  );
+  if (typeof globalThis.__SIBU_DEV__ !== "undefined") return !!globalThis.__SIBU_DEV__;
+  if (typeof __SIBU_DEV__ !== "undefined") return !!__SIBU_DEV__;
+  return typeof process !== "undefined" && process.env?.NODE_ENV !== "production";
 }
-var _isDev = isDev();
 function devAssert(condition, message) {
-  if (_isDev && !condition) {
+  if ((typeof __SIBU_DEV__ !== "undefined" ? __SIBU_DEV__ : DEV) && !condition) {
     throw new Error(`[SibuJS] ${message}`);
   }
 }
+function devWarnLazy(build) {
+  if (typeof __SIBU_DEV__ !== "undefined" ? __SIBU_DEV__ : DEV) {
+    const message = build();
+    if (message) console.warn(`[SibuJS] ${message}`);
+  }
+}
 function devWarn(message) {
-  if (_isDev) {
+  if (typeof __SIBU_DEV__ !== "undefined" ? __SIBU_DEV__ : DEV) {
     console.warn(`[SibuJS] ${message}`);
   }
 }
@@ -132,8 +137,10 @@ function reportError(error, context) {
 export {
   __require,
   __export,
+  DEV,
   isDev,
   devAssert,
+  devWarnLazy,
   devWarn,
   setRuntimeErrorHandler,
   getRuntimeErrorHandler,

package/dist/{chunk-DCDS2PQ6.js → chunk-3E5YNK2F.js}

@@ -3,10 +3,10 @@ import {
   recordDependency,
   retrack,
   track
-} from "./chunk-D3SZ4LVS.js";
+} from "./chunk-TSL64O5L.js";
 import {
   devAssert
-} from "./chunk-VPP2FONR.js";
+} from "./chunk-2WLZ6757.js";
 
 // src/core/signals/derived.ts
 function derived(getter, options) {

package/dist/{chunk-TIRZCERI.js → chunk-5DXA2J44.js}

@@ -1,7 +1,7 @@
 import {
   __require,
   devWarn
-} from "./chunk-VPP2FONR.js";
+} from "./chunk-2WLZ6757.js";
 
 // src/core/ssr-context.ts
 var SSR_KEY = /* @__PURE__ */ Symbol.for("sibujs.ssr.v1");

package/dist/{chunk-PFTDGEPZ.js → chunk-7BJLKDBI.js}

@@ -1,15 +1,15 @@
 import {
   isSSR
-} from "./chunk-TIRZCERI.js";
+} from "./chunk-5DXA2J44.js";
 import {
   cleanup,
   retrack,
   untracked
-} from "./chunk-D3SZ4LVS.js";
+} from "./chunk-TSL64O5L.js";
 import {
   devAssert,
   reportError
-} from "./chunk-VPP2FONR.js";
+} from "./chunk-2WLZ6757.js";
 
 // src/core/signals/effect.ts
 var _g = globalThis;

package/dist/{chunk-XSZRJW76.js → chunk-7BPN4MXR.js}

@@ -1,25 +1,25 @@
 import {
   context
-} from "./chunk-5GJ5RTHA.js";
+} from "./chunk-L4B3Y3OM.js";
 import {
   derived
-} from "./chunk-DCDS2PQ6.js";
+} from "./chunk-3E5YNK2F.js";
 import {
   sanitizeUrl
-} from "./chunk-5INI7D2L.js";
+} from "./chunk-IXKSNWV5.js";
 import {
   globalSingleton
 } from "./chunk-7ZHH77QA.js";
 import {
   effect
-} from "./chunk-PFTDGEPZ.js";
+} from "./chunk-7BJLKDBI.js";
 import {
   getRequestScopedCache
-} from "./chunk-TIRZCERI.js";
+} from "./chunk-5DXA2J44.js";
 import {
   batch,
   signal
-} from "./chunk-UFSQEO7P.js";
+} from "./chunk-MYA5HEK5.js";
 
 // src/data/abort.ts
 function isAbortError(error) {

package/dist/{chunk-BMSN6NWQ.js → chunk-7GQIRV27.js}

@@ -6,29 +6,29 @@ import {
 import {
   div,
   span
-} from "./chunk-MRAMKDHQ.js";
+} from "./chunk-7UK6BSAT.js";
 import {
   STATIC_VALUE_RESOLVER,
   planHeadElementEntry,
   planMetaEntry
-} from "./chunk-HYCCIYNS.js";
+} from "./chunk-NIOYEGBQ.js";
 import {
   sanitizeUrl
-} from "./chunk-5INI7D2L.js";
+} from "./chunk-IXKSNWV5.js";
 import {
   globalSingleton
 } from "./chunk-7ZHH77QA.js";
 import {
   effect
-} from "./chunk-PFTDGEPZ.js";
+} from "./chunk-7BJLKDBI.js";
 import {
   registerDisposer,
   replaceChildrenSafely
-} from "./chunk-DHDZ7IVN.js";
+} from "./chunk-QKRPLZ2V.js";
 import {
   batch,
   signal
-} from "./chunk-UFSQEO7P.js";
+} from "./chunk-MYA5HEK5.js";
 
 // src/platform/head.ts
 function escapeScriptJsonLocal(json) {

package/dist/{chunk-SHDVWATN.js → chunk-7UBG6LDP.js}

@@ -2,18 +2,17 @@ import {
   isEventHandlerAttr,
   isHtmlContentAttribute,
   sanitizeAttributeString
-} from "./chunk-5INI7D2L.js";
+} from "./chunk-IXKSNWV5.js";
 import {
   reactiveBinding
-} from "./chunk-D3SZ4LVS.js";
+} from "./chunk-TSL64O5L.js";
 import {
+  DEV,
   devWarn,
-  isDev,
   reportError
-} from "./chunk-VPP2FONR.js";
+} from "./chunk-2WLZ6757.js";
 
 // src/utils/setSafeAttribute.ts
-var _isDev = isDev();
 var HTML_NS = "http://www.w3.org/1999/xhtml";
 var SVG_NS = "http://www.w3.org/2000/svg";
 var XLINK_NS = "http://www.w3.org/1999/xlink";
@@ -36,7 +35,7 @@ function setSafeAttribute(el, name, value, options = {}) {
   const ns = namespaceFor(el, name);
   const localName = ns ? name.slice(name.indexOf(":") + 1) : name;
   if (isEventHandlerAttr(name)) {
-    if (_isDev) {
+    if (DEV) {
       devWarn(
         `${options.label ?? "setSafeAttribute"}: refusing to set event-handler attribute "${name}". Its value would be evaluated as JavaScript. Use on:{ ${name.slice(2)}: fn } instead.`
       );
@@ -51,7 +50,7 @@ function setSafeAttribute(el, name, value, options = {}) {
     return true;
   }
   if (isHtmlContentAttribute(name)) {
-    if (_isDev) {
+    if (DEV) {
       devWarn(
         `${options.label ?? "setSafeAttribute"}: refusing to set "${name}". The browser parses this attribute as a nested HTML document, so a generic string value cannot be made safe \u2014 attribute escaping is undone before the parse. Build the frame's content another way.`
       );
@@ -80,7 +79,7 @@ function setSafeAttribute(el, name, value, options = {}) {
     setProp(el, idlName, idlName === "checked" ? Boolean(value) : str);
     return true;
   }
-  const safe = sanitizeAttributeString(name, str);
+  const safe = sanitizeAttributeString(name, str, { element: el });
   const current = ns ? el.getAttributeNS(ns, localName) : el.getAttribute(name);
   if (current === safe) return true;
   if (ns) el.setAttributeNS(ns, name, safe);

package/dist/{chunk-MRAMKDHQ.js → chunk-7UK6BSAT.js}

@@ -1,7 +1,7 @@
 import {
   SVG_NS,
   tagFactory
-} from "./chunk-I3VI34DP.js";
+} from "./chunk-XGCZ45NH.js";
 
 // src/core/rendering/html.ts
 var html = tagFactory("html");

package/dist/{chunk-LOFZHP6V.js → chunk-AAHT6AKT.js}

@@ -1,24 +1,24 @@
 import {
   sanitizeUrl
-} from "./chunk-5INI7D2L.js";
+} from "./chunk-IXKSNWV5.js";
 import {
   globalSingleton
 } from "./chunk-7ZHH77QA.js";
 import {
   effect
-} from "./chunk-PFTDGEPZ.js";
+} from "./chunk-7BJLKDBI.js";
 import {
   dispose,
   replaceChildrenSafely
-} from "./chunk-DHDZ7IVN.js";
+} from "./chunk-QKRPLZ2V.js";
 import {
   signal
-} from "./chunk-UFSQEO7P.js";
+} from "./chunk-MYA5HEK5.js";
 import {
+  DEV,
   devWarn,
-  isDev,
   reportError
-} from "./chunk-VPP2FONR.js";
+} from "./chunk-2WLZ6757.js";
 
 // src/performance/scheduler.ts
 var Priority = {
@@ -228,7 +228,6 @@ function uniqueId(suffix) {
 }
 
 // src/performance/domRecycler.ts
-var _isDev = isDev();
 var DOMPool = class {
   constructor(maxSize = 50) {
     this.pools = /* @__PURE__ */ new Map();
@@ -255,7 +254,7 @@ var DOMPool = class {
    * is almost certainly a bug.
    */
   release(element) {
-    if (_isDev && element.isConnected) {
+    if (DEV && element.isConnected) {
       devWarn(
         "DOMPool.release() called on a still-connected element. Detach it from the DOM first (remove() / dispose())."
       );

package/dist/{chunk-FMHIQNMP.js → chunk-G7GPNB6N.js}

@@ -1,22 +1,22 @@
 import {
   watch
-} from "./chunk-KSOEJGFO.js";
+} from "./chunk-IYHM4FWD.js";
 import {
   createId
 } from "./chunk-CCSJMTRN.js";
 import {
   derived
-} from "./chunk-DCDS2PQ6.js";
+} from "./chunk-3E5YNK2F.js";
 import {
   stripHtml
-} from "./chunk-5INI7D2L.js";
+} from "./chunk-IXKSNWV5.js";
 import {
   effect
-} from "./chunk-PFTDGEPZ.js";
+} from "./chunk-7BJLKDBI.js";
 import {
   batch,
   signal
-} from "./chunk-UFSQEO7P.js";
+} from "./chunk-MYA5HEK5.js";
 
 // src/widgets/Combobox.ts
 var boundComboboxes = /* @__PURE__ */ new WeakMap();