mnfst 0.5.180 → 0.5.181

package/lib/manifest.combobox.js

@@ -70,25 +70,34 @@ function initializeComboboxPlugin() {
     // Alpine binding attributes — never carried onto combobox-owned clones
     const isBinding = (n) => n[0] === ':' || n[0] === '@' || n.indexOf('x-') === 0;
 
+    // Rendered-snapshot clone: strip Alpine bindings (x-/:/@) from the whole subtree —
+    // they reference the source's x-for scope and would throw when Alpine walks the
+    // combobox-owned copy. Computed results (style, class, text) are baked in and survive.
+    function stripClone(root) {
+        Array.from(root.attributes).forEach(a => { if (isBinding(a.name)) root.removeAttribute(a.name); });
+        root.querySelectorAll('*').forEach(n => {
+            Array.from(n.attributes).forEach(a => { if (isBinding(a.name)) n.removeAttribute(a.name); });
+        });
+        return root;
+    }
+
     // Read options from a source element (datalist/select → option, menu → li)
     function readOptions(src) {
         if (!src) return [];
         if (src.tagName === 'MENU') {
             return Array.from(src.querySelectorAll('li')).map(li => {
-                // The clone is a rendered snapshot: strip Alpine bindings (x-/:/@) from the
-                // WHOLE subtree, not just the row — they reference the source's x-for scope
-                // and would throw when Alpine walks the combobox-owned copy. The computed
-                // results (style, class, text) are already baked in and survive.
-                const copy = li.cloneNode(true);
-                copy.querySelectorAll('*').forEach(n => {
-                    Array.from(n.attributes).forEach(a => { if (isBinding(a.name)) n.removeAttribute(a.name); });
-                });
+                const copy = stripClone(li.cloneNode(true));
+                // A [data-chip] element inside the row is the chip's rich content: the chip
+                // shows its rendered clone, and clicks are mirrored back to this live source
+                // fragment so author @click handlers run in their own x-for scope.
+                const chipSrc = li.querySelector('[data-chip]');
                 return {
                     value: li.dataset.value != null ? li.dataset.value : li.textContent.trim(),
                     label: li.dataset.label || li.textContent.trim(),
                     pattern: li.dataset.pattern || null,
                     locked: li.hasAttribute('data-locked'),
                     html: copy.innerHTML,
+                    chip: chipSrc ? (n => ({ node: n, src: chipSrc, sig: n.outerHTML }))(stripClone(chipSrc.cloneNode(true))) : null,
                     // Carry the row's own attributes into the option, minus combobox-managed ones.
                     attrs: Array.from(li.attributes)
                         .filter(a => { const n = a.name; return n !== 'role' && n !== 'id' && n !== 'aria-selected' && !isBinding(n); })
@@ -98,7 +107,9 @@ function initializeComboboxPlugin() {
         }
         return Array.from(src.querySelectorAll('option')).map(o => ({
             value: o.value || o.textContent.trim(),
-            label: o.textContent.trim() || o.value,
+            // o.label is the native `label` IDL prop: the `label` attribute when set, else
+            // the option's text — so <option value="SE" label="Sweden"> works natively.
+            label: (o.label || '').trim() || o.value,
             pattern: o.getAttribute('data-pattern') || null,
             locked: o.hasAttribute('data-locked'),
             html: null
@@ -246,6 +257,17 @@ function initializeComboboxPlugin() {
         if (!editorNone) el.removeAttribute('placeholder');
         if (name) el.removeAttribute('name');   // hidden inputs carry the value(s)
 
+        // A wrapping <label> with no `for` targets its FIRST labelable descendant —
+        // that's the first chip's remove button, not the editor. Hovering the label then
+        // hover-lights that ×, and clicking the label (or a chip's text) activates it,
+        // deleting chips. Bind the label to the editor explicitly; clicks on interactive
+        // chip content (the ×, or any links/handlers) still land on it, not the label.
+        const ownerLabel = wrap.closest('label');
+        if (ownerLabel && !ownerLabel.hasAttribute('for')) {
+            if (!el.id) el.id = 'combobox-editor-' + rand();
+            ownerLabel.setAttribute('for', el.id);
+        }
+
         // Live region for add/remove announcements
         const live = document.createElement('span');
         live.setAttribute('role', 'status');
@@ -572,11 +594,47 @@ function initializeComboboxPlugin() {
             chip.className = 'combobox-chip';
             chip.dataset.value = s.value;
             const label = document.createElement('span');
-            label.textContent = s.label;
+            setChipContent(label, s);
             chip.appendChild(label);
             applyLock(chip, s.value, s.label);
             return chip;
         }
+        // Chip content: the option's [data-chip] fragment when present (rendered clone,
+        // refreshed in place on reread — updating innerHTML never re-inserts the chip
+        // node, so the WebKit focused-insert collapse can't trigger), else the text
+        // label. Links in the clone navigate natively (computed href carries); other
+        // clicks are mirrored onto the same element in the LIVE source fragment, where
+        // the author's @click runs in its own x-for scope.
+        function setChipContent(span, s) {
+            const o = options.find(o => String(o.value).toLowerCase() === String(s.value).toLowerCase());
+            const rich = o && o.chip;
+            const sig = rich ? o.chip.sig : 't:' + s.label;
+            if (span.__cbSig === sig) return;
+            span.__cbSig = sig;
+            if (!rich) { span.textContent = s.label; span.__cbSrc = null; return; }
+            span.innerHTML = '';
+            span.appendChild(o.chip.node.cloneNode(true));
+            span.__cbSrc = o.chip.src;
+            if (span.__cbDelegated) return;
+            span.__cbDelegated = true;
+            span.addEventListener('click', (e) => {
+                const srcRoot = span.__cbSrc;
+                if (!srcRoot || !srcRoot.isConnected) return;
+                if (e.target.closest && e.target.closest('a[href]')) return;   // native nav owns it
+                const cloneRoot = span.firstElementChild;
+                let n = e.target === span ? cloneRoot : e.target;
+                const path = [];
+                while (n && n !== cloneRoot) {
+                    const p = n.parentElement;
+                    if (!p) return;
+                    path.unshift(Array.prototype.indexOf.call(p.children, n));
+                    n = p;
+                }
+                let t = srcRoot;
+                for (const i of path) { t = t.children[i]; if (!t) return; }
+                t.dispatchEvent(new MouseEvent('click', { bubbles: true, cancelable: true }));
+            });
+        }
         // Add/drop the × to match locked state. Re-run from render() so a reactive
         // `locked` change toggles existing chips too.
         function applyLock(chip, value, label) {
@@ -615,7 +673,11 @@ function initializeComboboxPlugin() {
                     const key = norm(s.value);
                     const node = have.get(key);
                     if (!node) { const n = makeChip(s); have.set(key, n); wrap.insertBefore(n, el); }
-                    else applyLock(node, s.value, s.label);
+                    else {
+                        applyLock(node, s.value, s.label);
+                        const sp = node.querySelector(':scope > span');
+                        if (sp) setChipContent(sp, s);   // in-place refresh; node identity kept
+                    }
                 });
                 // Reorder only when order differs — needless re-insertion collapses chips
                 // in WebKit; the common append path (order matches) skips this.
@@ -802,11 +864,11 @@ function initializeComboboxPlugin() {
                 if (menu && menu.matches(':popover-open')) filter();
             };
             const mo = new MutationObserver(reread);
-            // childList/characterData: x-for row edits. attributes: bound data-* toggles.
-            mo.observe(src, {
-                childList: true, subtree: true, characterData: true,
-                attributes: true, attributeFilter: ['data-value', 'data-label', 'data-locked', 'data-pattern']
-            });
+            // childList/characterData: x-for row edits. attributes unfiltered: ANY bound
+            // attribute inside a row (style, class, data-*) changes the rendered snapshot
+            // the menu and rich chips clone, so all must re-trigger. Mutations coalesce
+            // per microtask, so a state flush costs one reread.
+            mo.observe(src, { childList: true, subtree: true, characterData: true, attributes: true });
             if (cleanup) cleanup(() => mo.disconnect());
         }
 

package/lib/manifest.integrity.json

@@ -7,7 +7,7 @@
   "manifest.code.js": "sha384-G3MNfvQRWWY+gm4SGGrSsgQbDeil9MP+ON2DTk6Rcq9Nsex8woHMDt5EJHrFH0Nj",
   "manifest.color.js": "sha384-6Rv3LxyTcZNjrhtayQfqRdCx0uSZ4BiEbgEI98I62eTvp8Aw7LBIoNJ0Je1oktwL",
   "manifest.colorpicker.js": "sha384-N5ezEwgKrDodV4YdURxhdyZKmg1n/QhqvmAC4SEG03mw1pEAZAA8Gphp+NLAOxts",
-  "manifest.combobox.js": "sha384-fE/SS0hPsSoCqn2x3H9u1HLY8dZ38PI/NCzrDpqL7+11QPOehG/f0f5MMGQqhBle",
+  "manifest.combobox.js": "sha384-J1zXXanlzsfoYJsRdz41DJD168S+20oT+lIEpUJ4UOjwjMcy/t+2GwwVjfSSNOlD",
   "manifest.components.js": "sha384-eZDib+RFdwd9mOUdi6o53eNS0LBlEEkZPDegxtkAcE+GrGIFxsuB4O0ymfixGNfM",
   "manifest.data.js": "sha384-3lLvGnqcQr4Ium80MdaQTr8RytjVbs5rnxd0q41BF3ppcoQccMFAcvgFSyZP/0sl",
   "manifest.datepicker.js": "sha384-52BBZdlLN+EATiUorwTEQOWGJZiSNu0voy9q+WQxxWpUeYsGGkuS0QPPXGx2akxG",
@@ -30,5 +30,5 @@
   "manifest.url.parameters.js": "sha384-+ZOsgPgbFmKOZsReC1EiH8PsZwvr2nC/duhzIzFf70xrbRrLsr926sLFW0M+BXtN",
   "manifest.utilities.js": "sha384-yLEqEbd1FncqswrtrcTO7KrIEK4iP0ZvqCi64XLI7D77/L0YiJbjLdLGGg6l7rgN",
   "manifest.virtual.js": "sha384-c194pXD0Ld48QJ+HPVNibvbn54/ETJRuYhUOWesEBjtOIQcQMIKo0DnCyQMvDlLg",
-  "manifest.js": "sha384-FLz68HEE8hEVLlAsoAX2+5cWBrhS5+B24rdJy5obXtP5U5bJKCswjbIQL8MpDUFZ"
+  "manifest.js": "sha384-2XTZDu5M3icGSRvAVLeLQfQqwJQvzeOfW0yMRDqVa8kxhTqfBwA17RBBYOUY3d9a"
 }

package/lib/manifest.js

@@ -466,7 +466,7 @@
 		))) {
 			plugins.push('appwrite-data');
 		}
-		if (manifest.appwrite?.presence) {
+		if (manifest.appwrite?.presence || manifest.appwrite?.presences) {
 			plugins.push('appwrite-presences');
 		}
 		return plugins;
@@ -481,14 +481,54 @@
 	}
 
 	// Detect the chat plugin from manifest.json content.
-	// Opt-in / auto-loaded when an `ai` (or `chat`) config block is present.
+	// Opt-in / auto-loaded when an `ai` (or `chat`) entry is present — an
+	// object block or bare `true` (custom-adapter projects have no config).
 	function detectChatPlugins(manifest) {
 		if (!manifest || typeof manifest !== 'object') return [];
-		if ((manifest.ai && typeof manifest.ai === 'object') ||
-			(manifest.chat && typeof manifest.chat === 'object')) return ['chat'];
+		if (manifest.ai || manifest.chat) return ['chat'];
 		return [];
 	}
 
+	// Detect plugins from markup usage — magics written in Alpine expressions
+	// with no declarative trigger (e.g. $chat driven by custom adapters).
+	// Runs at DCL, before Alpine boots: scans element attributes (recursing
+	// into template content, which the parser detaches) and inline scripts.
+	// Rendered text is deliberately not scanned — code samples on a page
+	// must not pull plugins in.
+	const USAGE_PLUGINS = [
+		{ magic: '$chat', plugin: 'chat' },
+		{ magic: '$presence', plugin: 'appwrite-presences' }
+	];
+	function detectUsagePlugins(alreadyLoaded) {
+		const wanted = USAGE_PLUGINS.filter(u => !alreadyLoaded.includes(u.plugin));
+		if (wanted.length === 0 || typeof document === 'undefined') return [];
+		const found = new Set();
+		const done = () => found.size === wanted.length;
+
+		for (const s of document.querySelectorAll('script:not([src])')) {
+			const t = s.textContent || '';
+			for (const u of wanted) if (!found.has(u.plugin) && t.includes(u.magic)) found.add(u.plugin);
+			if (done()) return [...found];
+		}
+
+		const scan = (root) => {
+			for (const el of root.querySelectorAll('*')) {
+				for (const attr of el.attributes || []) {
+					const v = attr.value;
+					if (!v || v.indexOf('$') === -1) continue;
+					for (const u of wanted) if (!found.has(u.plugin) && v.includes(u.magic)) found.add(u.plugin);
+					if (done()) return;
+				}
+				if (el.tagName === 'TEMPLATE' && el.content) {
+					scan(el.content);
+					if (done()) return;
+				}
+			}
+		};
+		scan(document);
+		return [...found];
+	}
+
 	// Detect the native umbrella plugin. Auto-loaded inside a Capacitor container
 	// (window.Capacitor present) or when a `native` config block opts in on the web.
 	function detectNativePlugins(manifest) {
@@ -520,12 +560,25 @@
 		// Override: custom CDN fallback chain (comma-separated origins).
 		const cdn = script.getAttribute('data-cdn');
 
+		// `data-plugins="a,b"` replaces the default set; a `+` prefix is additive
+		// (`data-plugins="+chat"` = defaults plus chat) — needed for plugins with
+		// no manifest.json trigger, e.g. chat driven by custom adapters only.
 		let pluginList = [];
-		const deriveFromManifest = !plugins;
+		let extraPlugins = [];
+		let deriveFromManifest = !plugins;
 
 		if (plugins) {
-			// Explicit declaration - load only specified plugins (core + Appwrite)
-			pluginList = plugins.split(',').map(p => p.trim()).filter(p => p);
+			const entries = plugins.split(',').map(p => p.trim()).filter(p => p);
+			extraPlugins = entries.filter(p => p.startsWith('+')).map(p => p.slice(1));
+			const explicit = entries.filter(p => !p.startsWith('+'));
+			if (explicit.length > 0) {
+				pluginList = [...explicit, ...extraPlugins];
+			} else {
+				// Only additive entries: keep the default derive-from-manifest
+				// behavior and append the extras once the manifest is inspected.
+				deriveFromManifest = true;
+				pluginList = AVAILABLE_PLUGINS.slice();
+			}
 		} else {
 			// Default: start with all core plugins; loader will trim by manifest when manifest is available
 			pluginList = AVAILABLE_PLUGINS.slice();
@@ -542,6 +595,7 @@
 
 		return {
 			plugins: pluginList,
+			extraPlugins,
 			deriveFromManifest,
 			tailwind,
 			version,
@@ -598,7 +652,7 @@
 
 		const MANIFEST_DEPENDENT_PLUGINS = [
 			'data', 'localization', 'components',
-			'appwrite-auth', 'appwrite-data', 'appwrite-presences', 'payments'
+			'appwrite-auth', 'appwrite-data', 'appwrite-presences', 'payments', 'chat'
 		];
 		const manifestUrl = (document.querySelector('link[rel="manifest"]')?.getAttribute('href')) || '/manifest.json';
 
@@ -664,7 +718,7 @@
 				const paymentsPlugins = detectPaymentsPlugins(manifest);
 				const chatPlugins = detectChatPlugins(manifest);
 				const nativePlugins = detectNativePlugins(manifest);
-				pluginsToLoad = resolveDependencies([...corePlugins, ...appwritePlugins, ...paymentsPlugins, ...chatPlugins, ...nativePlugins]);
+				pluginsToLoad = resolveDependencies([...corePlugins, ...appwritePlugins, ...paymentsPlugins, ...chatPlugins, ...nativePlugins, ...(config.extraPlugins || [])]);
 			} else {
 				const needsManifest = config.plugins.some(p => MANIFEST_DEPENDENT_PLUGINS.includes(p));
 				if (needsManifest) {
@@ -698,7 +752,24 @@
 					window.ManifestComponentsRegistry.manifest = manifest;
 				}
 			}
-			loadAlpine(alpineUrlCandidates(config.alpine));
+			// Usage sniff: Alpine is DCL-gated anyway, so the parsed document can
+			// be checked for magics with no declarative trigger and the missing
+			// plugins fetched before Alpine boots. Derive path only — an explicit
+			// data-plugins list is an intentional constraint, not a default.
+			whenDomReady(async () => {
+				if (config.deriveFromManifest) {
+					try {
+						const used = detectUsagePlugins(pluginsToLoad);
+						if (used.length > 0) {
+							const late = resolveDependencies(used).filter(p => !pluginsToLoad.includes(p));
+							await Promise.all(late.map(p => addScript(p, config.version).catch(error => {
+								console.warn(`[Manifest Loader] Failed to load plugin ${p}:`, error);
+							})));
+						}
+					} catch (_) { /* sniffing must never block Alpine */ }
+				}
+				loadAlpine(alpineUrlCandidates(config.alpine));
+			});
 		};
 
 		loadPlugins();

package/package.json

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