mnfst 0.5.208 → 0.5.210

package/lib/manifest.appwrite.data.js

@@ -2,7 +2,7 @@
 
 (function () {
 
-const MANIFEST_BUILD_VERSION = '0.5.208';
+const MANIFEST_BUILD_VERSION = '0.5.210';
 
 /* Manifest Data Sources - Appwrite Integration */
 

package/lib/manifest.charts.js

@@ -124,6 +124,11 @@
 		const _registry = Alpine.reactive ? Alpine.reactive({}) : {};
 		let _uid = 0;
 
+		const isDevelopment = window.location.hostname === 'localhost' ||
+			window.location.hostname === '127.0.0.1' ||
+			window.location.hostname.includes('dev') ||
+			window.location.search.includes('debug=true');
+
 		const _nullApi = { type: '', series: [], update() { }, redraw() { }, toString() { return ''; }, valueOf() { return ''; } };
 
 		function findAncestorState(el) { let n = el; while (n) { if (n._chartState) return n._chartState; n = n.parentElement; } return null; }
@@ -228,6 +233,30 @@
 			return cfg;
 		}
 
+		// Deterministic structural signature of a normalized config, for the redraw
+		// short-circuit below: sorted object keys (key order doesn't matter), arrays
+		// keep position, Dates by epoch ms, typed arrays by value. Functions are
+		// omitted — identity isn't a meaningful "value" for a config comparison and
+		// authors sometimes pass fresh closures that are behaviorally identical.
+		function stableStringify(v, seen) {
+			if (v === null || typeof v !== 'object') return typeof v === 'function' ? undefined : JSON.stringify(v);
+			if (v instanceof Date) return String(v.getTime());
+			if (ArrayBuffer.isView(v)) return '[' + Array.prototype.join.call(v, ',') + ']';
+			seen = seen || new Set();
+			if (seen.has(v)) return '"[Circular]"'; // config is author data; don't trust it's acyclic
+			seen.add(v);
+			let out;
+			if (Array.isArray(v)) {
+				out = '[' + v.map(x => { const s = stableStringify(x, seen); return s === undefined ? 'null' : s; }).join(',') + ']';
+			} else {
+				const parts = [];
+				Object.keys(v).sort().forEach(k => { const s = stableStringify(v[k], seen); if (s !== undefined) parts.push(JSON.stringify(k) + ':' + s); });
+				out = '{' + parts.join(',') + '}';
+			}
+			seen.delete(v);
+			return out;
+		}
+
 		/* ---- SVG helpers -------------------------------------------- */
 		function svg(tag, attrs, parent) {
 			const node = document.createElementNS(SVGNS, tag);
@@ -287,7 +316,7 @@
 					return {
 						get type() { return self.config ? self.config.type : ''; },
 						get series() { return self.config ? self.config.series : []; },
-						update(cfg) { self.config = normalize(cfg, self.el, self.typeFromModifier); self.draw(); },
+						update(cfg) { if (self.applyConfig(normalize(cfg, self.el, self.typeFromModifier))) self.draw(); },
 						redraw() { self.draw(); },
 						toString() { return self.config ? self.config.type : ''; }
 					};
@@ -306,10 +335,17 @@
 					Alpine.effect(() => {
 						// Subscribe to the data store heartbeat so $x loads/locale reloads re-run.
 						try { void Alpine.store('data')?._dataVersion; } catch (_) { }
-						if (getCfg) getCfg(raw => { self.config = normalize(raw, self.el, self.typeFromModifier); self.schedule(); });
-						else { self.config = normalize(configFromDom(self.el), self.el, self.typeFromModifier); self.schedule(); }
+						// Every effect fire re-evaluates the expression (it may read live reactive
+						// state — realtime rows, presence, staged $x hydration — that changes far
+						// more often than the resulting config). applyConfig() short-circuits a
+						// schedule() when the normalized result is value-identical to last time;
+						// the first call always applies (self._sig starts undefined).
+						if (getCfg) getCfg(raw => { if (self.applyConfig(normalize(raw, self.el, self.typeFromModifier))) self.schedule(); });
+						else if (self.applyConfig(normalize(configFromDom(self.el), self.el, self.typeFromModifier))) self.schedule();
 					});
-					// Re-render on locale (axis number/date formatting) and container resize.
+					// Re-render on locale (axis number/date formatting) and container resize —
+					// these change rendering without changing the config, so they bypass the
+					// signature check and always schedule.
 					self._onLocale = () => self.schedule();
 					window.addEventListener('localechange', self._onLocale);
 					self._ro = new ResizeObserver(() => self.schedule());
@@ -317,9 +353,35 @@
 					if (self.el.id) _registry[self.el.id] = self.api;
 				},
 
+				// Applies a normalized config, returning whether it actually changed (by
+				// structural signature, not reference) — see stableStringify. Always
+				// "changes" the first time (this._sig starts undefined) so the first draw
+				// is unconditional.
+				applyConfig(next) {
+					const sig = stableStringify(next);
+					const changed = this._sig === undefined || sig !== this._sig;
+					if (changed) { this._sig = sig; this.config = next; }
+					return changed;
+				},
+
 				// One draw per tick. setTimeout (not rAF) so backgrounded tabs still draw;
 				// doesn't reset a pending timer, so a high-frequency trigger can't starve it.
-				schedule() { if (this._t) return; this._t = setTimeout(() => { this._t = null; this.draw(); }, 0); },
+				schedule() {
+					if (this._t) return;
+					// Runaway-redraw guard: a chart that schedules dozens of times a second
+					// (usually a config-signature miss, or a map's heavy geometry rebuild —
+					// see drawMap) pegs the main thread and shows up to the user as a hung
+					// tab, not a chart bug. Warn once per element so it's self-diagnosing.
+					if (isDevelopment && !this._warnedRedraw) {
+						const now = Date.now();
+						if (!this._scheduleWindowStart || now - this._scheduleWindowStart > 1000) { this._scheduleWindowStart = now; this._scheduleWindowCount = 0; }
+						if (++this._scheduleWindowCount > 20) {
+							this._warnedRedraw = true;
+							try { console.warn('[Manifest Charts] "' + (this.el.id || this.expression || 'chart') + '" redrew ' + this._scheduleWindowCount + '+ times within one second — likely a reactive dependency changing on every read (or, for a map, a legitimately busy data feed hitting its per-draw geometry cost).'); } catch (_) { }
+						}
+					}
+					this._t = setTimeout(() => { this._t = null; this.draw(); }, 0);
+				},
 
 				renderError(msg) { this.el.innerHTML = ''; const d = document.createElement('small'); d.textContent = msg; this.el.appendChild(d); },
 
@@ -921,9 +983,18 @@
 		// (numeric values → heat ramp) or categorically (string values → palette).
 		function drawMap(state, root, width, height, geo) {
 			const cfg = state.config;
-			const fc = geo.topojson.feature(geo.world, geo.world.objects.countries);
-			const projection = geo.geoNaturalEarth1().fitSize([width, height], fc);
-			const path = geo.geoPath(projection);
+			// The feature collection, fitted projection, and path generator depend only
+			// on the atlas (`geo`) and the element's [width, height] — not on cfg.data —
+			// so a data-only redraw (the common case: realtime values ticking in) reuses
+			// them instead of re-running topojson.feature/fitSize over ~175 polygons.
+			// Cached per element, invalidated when the atlas or size changes.
+			let geom = state._mapGeom;
+			if (!geom || geom.geo !== geo || geom.width !== width || geom.height !== height) {
+				const fc = geo.topojson.feature(geo.world, geo.world.objects.countries);
+				const projection = geo.geoNaturalEarth1().fitSize([width, height], fc);
+				geom = state._mapGeom = { geo, width, height, fc, projection, path: geo.geoPath(projection) };
+			}
+			const { fc, projection, path } = geom;
 
 			// Author data: an object { key: value } or an array of { id|code|name, value } / [key, value].
 			let entries = [];

package/lib/manifest.combobox.js

@@ -168,8 +168,41 @@ function initializeComboboxPlugin() {
         setTimeout(() => build(el, modifiers, expression || '', cleanup), 0);
     });
 
+    // Resolve the menu's owning top-layer context fresh, every time — never cache it.
+    // A [popover]/dialog ancestor can be mid-restash (manifest.defer's idle prewarm/evict
+    // cycle moving a still-closed container's subtree back into its <template> stash)
+    // at any point after build(), so the target must be re-read at each attach, not
+    // assumed stable from build time.
+    // The menu belongs in its field's top-layer context, but that ancestor can be
+    // mid-restash (defer moving a closed container's subtree back into its <template>),
+    // so re-resolve at every attach — and never attach into a detached tree, or
+    // showPopover() throws InvalidStateError and leaves the field dead.
+    let warnedDetached = false;
+    const attachTarget = (el) => {
+        const owner = el.closest('[popover]');
+        if (owner && owner.isConnected) return owner;
+        if (owner && !warnedDetached) {
+            warnedDetached = true;
+            console.warn('[Manifest Combobox] the field\'s popover ancestor was detached when its menu was attached; ' +
+                'using document.body so the field keeps working. Please report this with the field id:', el.id || el);
+        }
+        return document.body;
+    };
+    const reattach = (el, menu) => {
+        const target = attachTarget(el);
+        if (menu.parentNode !== target) target.appendChild(menu);
+    };
+
     function build(el, modifiers, expression, cleanup) {
-        if (el.__mnfstCombobox) return;
+        if (el.__mnfstCombobox) {
+            // Already built. If manifest.defer's idle prewarm/evict cycle raced this very
+            // build (stashing the owning [popover] between directive init and the setTimeout
+            // below), the generated menu was appended into a since-detached tree — reattach
+            // it into wherever the owner lives now. Harmless no-op otherwise.
+            const m = el.__mnfstComboboxMenu;
+            if (m) reattach(el, m);
+            return;
+        }
         el.__mnfstCombobox = true;
 
         // If a mount path bypassed the strip (x-if/x-for via Alpine's internal initTree)
@@ -409,8 +442,10 @@ function initializeComboboxPlugin() {
             }
             // Nearest popover host, so an outer auto popover isn't light-dismissed
             // by listbox clicks; body otherwise (out of overflow contexts either way).
-            (el.closest('[popover]') || document.body).appendChild(menu);
+            // Re-resolved (not just attached once) at every open — see reattach().
+            reattach(el, menu);
             generatedMenu = menu;               // tracked so cleanup() can remove it on re-render
+            el.__mnfstComboboxMenu = menu;      // tracked so a re-entrant build() can reattach it
             if (src) src.style.setProperty('display', 'none', 'important');
             menu.setAttribute('popover', 'manual');
             menu.setAttribute('x-defer.off', ''); // options are built here, not by directives
@@ -459,8 +494,22 @@ function initializeComboboxPlugin() {
             if (!menu || menu.matches(':popover-open') || atCap()) return;
             // Re-stashed since build (x-defer.discard) — re-hydrate before showing.
             if (src && window.ManifestDefer?.isPending(src)) { options = readOptions(src); setOptions(options); }
+            // Lazy re-resolve: the owning [popover]/dialog may have been (re-)stashed and
+            // rendered since build(), or build() itself raced a defer restash and attached
+            // into a tree that's since gone disconnected. A real open only ever happens
+            // once the field itself is genuinely connected/visible, so this always has a
+            // valid target.
+            reattach(el, menu);
             menu.style.minWidth = wrap.offsetWidth + 'px';
-            menu.showPopover();
+            try {
+                menu.showPopover();
+            } catch (_) {
+                // Belt-and-suspenders: resolve once more and retry, then give up quietly —
+                // an uncaught error here would abort whatever handler called openMenu() and
+                // can strand the containing dialog's own dismiss handling mid-flight.
+                reattach(el, menu);
+                try { menu.showPopover(); } catch (_) { return; }
+            }
             el.setAttribute('aria-expanded', 'true');
         }
         function closeMenu() {

package/lib/manifest.data.js

@@ -2,7 +2,7 @@
 
 (function () {
 
-const MANIFEST_BUILD_VERSION = '0.5.208';
+const MANIFEST_BUILD_VERSION = '0.5.210';
 
 /* Manifest Data Sources - Configuration */
 

package/lib/manifest.integrity.json

@@ -1,17 +1,17 @@
 {
   "manifest.appwrite.auth.js": "sha384-HMJmRXnvdDBWyuteXL+DTxWM2llSCdjl4EDamEI7iHNAVlqeOCE4GDTm8ZHipPRj",
-  "manifest.appwrite.data.js": "sha384-thWjsEQ6dGC4VT58G7CPD3bLFBVlGc3oDB0yq02y47nFvQgcntv7en0HJ7rrexZ/",
+  "manifest.appwrite.data.js": "sha384-kmsw1sYLqTxnkRfh7dt0urLSFU+LOkgHofsL1jYGndFvCizo7wsbQix6vV022/ov",
   "manifest.appwrite.presences.js": "sha384-JyVDWFT69b7Svq8J8p9tNWe2v9PZNQBY5UyhtOl1vnuHFAX0CRQ3MA9SiKq5wzNh",
   "manifest.bindings.js": "sha384-lPtMf8Ykaic9gNVZKq8Scia1881ZeMAyxer3WCIkUNwwrgzGBWto1OXADUwjvORF",
-  "manifest.charts.js": "sha384-qQrjhWkIDi4oU5uQrpy8DdlMjY+gfiivBuX6mssNiFdkawgkVBvKneGgp+nd3PYJ",
+  "manifest.charts.js": "sha384-WE0pnHK4LBAkFyTw4+R6gheB9NFVJg81ZAG9GwMnbKI5gY3zlJIKM5neMlBUhTso",
   "manifest.chat.js": "sha384-88DfMfJLoi2oxclVD2iIMDkIGAExMXq5k47mekUredlicLaP3oZY5bwtwqs+TqDF",
   "manifest.code.js": "sha384-aoQ4E26vrhNe/+tL7K0h90c8/0ies7e8oIR6Ay92he2X3dnWZ+OPcn2Dna8fmc+c",
   "manifest.color.js": "sha384-6Rv3LxyTcZNjrhtayQfqRdCx0uSZ4BiEbgEI98I62eTvp8Aw7LBIoNJ0Je1oktwL",
   "manifest.colorpicker.js": "sha384-1+ZV5Hhjf7WnGP7b71Cqa5h3BKW8UgGT5V14i8Q+aF5zH8Xxwowt/ibSUVsiIHSm",
-  "manifest.combobox.js": "sha384-9/wrX1kOe/8/ESSM6k45SjkM/YNZyGySrR9XxQCw1nRdBstYM33TyPGs2bN5QHn5",
+  "manifest.combobox.js": "sha384-5DaKDWkWF7zxS80h7G1AcZG9hr+uGqJ1+B09aaVIVF+z0Qn4j8+9KEg7gys/ABQG",
   "manifest.components.js": "sha384-DLGSqCbnGkS87AWJ8GpKoAFP4nCosgC5SFSg6C10hH/nWu/MxI+0ntHlnFEfs++u",
   "manifest.computed.js": "sha384-GYML/e81gQfN+iDbqhPUgny8bINCyL+SrbTofoGk4hlCnLLjv51mCv42JMqq+ku5",
-  "manifest.data.js": "sha384-wD+/0DfyBzpM/4SzA6JirXgINqOuQLysEvYJqz84i+gugP/EDyu8nHG3k1H9kHCt",
+  "manifest.data.js": "sha384-U4/sSXZI/DL0X7ymE9KrwAKhIv31pJoEo0YnZBgJTAhHYwUZxhaC7nrhWVj/5a/q",
   "manifest.datepicker.js": "sha384-8vTJYkLqqvgXgm1w1becuaRR/jUEpiLBjBJ6RsRbUr+ZkypvspD6EsssuDrX4SQq",
   "manifest.defer.js": "sha384-wXa60M2OHigi5rBurXQ7Oa8eyr5WTKs7EUn2GOru62KMdfs/LDyNegxCc9MNJFSe",
   "manifest.device.js": "sha384-g+txmrEszYtuOvRelTp/c1wSNR2AtllVI9rx5NPPD7DzhVUnMNughonZykv2xlBW",
@@ -28,7 +28,7 @@
   "manifest.slides.js": "sha384-3uRTkyK9XPLmnxI2+igZlpi4EyPlU/7IHj5j3BZJJ2KN455vXyk99fiXV3feO/XY",
   "manifest.status.js": "sha384-nwvUSbT+gpRMqhYx6MeBYswUMH72rY92mjgxv0vrSG7Ht7l83XYpEB6fCTbXjjfK",
   "manifest.svg.js": "sha384-qr/8H9+2cIhSXNbRWRtIdrDlTOxKlNJqwEefHvCYoiBNW/kKhPTFa109Orv/EBnb",
-  "manifest.sw.js": "sha384-+hD/KjRLCTRpOQ2FYepewamzFVPst9mkMoLPB0n7pV8DI/o3OPirEcCSkWimMAuJ",
+  "manifest.sw.js": "sha384-uUl8PEc1B0HXLOMpjQqF0w+4ybSeyHAAOSXmWQkHH4eO7ZiyU43yIlnEWKpsXlgb",
   "manifest.tabs.js": "sha384-OIIcyaQnGS0eNYk4zgAFEDboNYect7THvUd9D0V6nqCeDOxQcoiqLrBEmmWVTTet",
   "manifest.tailwind.js": "sha384-r/Jvqb+Piz21jUts10BbVAJoKkShOdJRVFpywyi4HPIi/OfOJq5CtPyWAPbdkMgt",
   "manifest.text.edit.js": "sha384-pfhc4IPKNJokj6ldlgD+KwGLN4UoM21CNeVf2sVq/34X1Un2PUhEuyTg5itmtsXS",

package/lib/manifest.sw.js

@@ -16,7 +16,7 @@
 
 // Build stamps the published version here; unstamped (dev) falls back to the
 // registration URL's `v`, then 'dev'.
-const BUILD_VERSION = '0.5.208';
+const BUILD_VERSION = '0.5.210';
 const CACHE_PREFIX = 'mnfst-sw:';
 const META_CACHE = CACHE_PREFIX + 'meta';
 const SWR_CAP = 200;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "mnfst",
-    "version": "0.5.208",
+    "version": "0.5.210",
     "private": false,
     "workspaces": [
         "templates/starter",