bimba-cli 0.7.5 → 0.7.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bimba-cli",
-  "version": "0.7.5",
+  "version": "0.7.7",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/HeapVoid/bimba.git"

package/serve.js

@@ -113,6 +113,8 @@ const hmrClient = `
 			_collector = prev;
 		}
 
+		console.log('[bimba HMR]', file, 'slots=' + slots, 'tags:', collected.join(', '));
+
 		// Sync _ns_ (CSS namespace) from the new classes. imba_defineTag sets
 		// _ns_ on NewClass.prototype AFTER register$ calls customElements.define,
 		// so _patchClass missed it. Now that import is done, all _ns_ values are set.
@@ -144,6 +146,7 @@ const hmrClient = `
 		if (slots !== 'stable') {
 			for (const tag of collected) {
 				const els = document.querySelectorAll(tag);
+				console.log('[bimba HMR] destructive:', tag, 'instances:', els.length);
 				els.forEach(el => {
 					const state = {};
 					for (const k of Object.keys(el)) state[k] = el[k];
@@ -159,10 +162,36 @@ const hmrClient = `
 					try { el.mount && el.mount(); } catch(_) {}
 				});
 			}
+		} else {
+			for (const tag of collected) {
+				const els = document.querySelectorAll(tag);
+				console.log('[bimba HMR] stable:', tag, 'instances:', els.length);
+			}
+		}
+
+		// Snapshot DOM before commit to detect duplication
+		const _domBefore = {};
+		for (const tag of collected) {
+			_domBefore[tag] = {
+				count: document.querySelectorAll(tag).length,
+				bodyChildren: document.body.children.length,
+			};
 		}
 
 		if (typeof imba !== 'undefined') imba.commit();
 
+		// Check DOM after commit (delayed to let rAF fire)
+		requestAnimationFrame(() => {
+			for (const tag of collected) {
+				const after = document.querySelectorAll(tag).length;
+				const before = _domBefore[tag]?.count || 0;
+				if (after !== before) {
+					console.warn('[bimba HMR] DUPLICATION:', tag, before, '->', after);
+				}
+				console.log('[bimba HMR] after commit:', tag, 'count:', after, 'body children:', document.body.children.length);
+			}
+		});
+
 		// Patch className on ALL custom elements: replace old CSS namespace
 		// hashes with new ones. Must be global because subclass elements
 		// (e.g. panel-agent < basic-panel) inherit the parent's _ns_ hash
@@ -307,21 +336,6 @@ function updateImportGraph(fromAbs, newDeps) {
 	_imports.set(fromAbs, newDeps)
 }
 
-function transitiveImporters(absFile, skip) {
-	const out = new Set()
-	const stack = [absFile]
-	while (stack.length) {
-		const cur = stack.pop()
-		const ups = _importers.get(cur)
-		if (!ups) continue
-		for (const u of ups) {
-			if (out.has(u) || (skip && skip.has(u))) continue
-			out.add(u)
-			stack.push(u)
-		}
-	}
-	return out
-}
 
 // Imba compiles tag render-cache slots as anonymous local Symbols at module top
 // level: `var $4 = Symbol(), $11 = Symbol(), ...; let c$0 = Symbol();`. Each
@@ -497,7 +511,6 @@ export function serve(entrypoint, flags) {
 	const htmlDir  = path.dirname(htmlPath)
 	const srcDir   = path.dirname(entrypoint)
 	const sockets  = new Set()
-	const entryAbs = path.resolve(entrypoint)
 	let importMapTag = null
 
 	// ── Status line (prints current compile result, fades out on success) ──────
@@ -597,18 +610,6 @@ export function serve(entrypoint, flags) {
 			printStatus(rel, 'ok')
 			broadcast({ type: 'clear-error' })
 			broadcast({ type: 'update', file: rel, slots: out.slots || 'shifted' })
-
-			// Cascade: re-import modules that transitively import this file.
-			// Skip the entry point — re-importing it re-runs imba.mount and
-			// recreates global services, and traversing through it would
-			// cascade to the entire project.
-			const ups = transitiveImporters(path.resolve(filepath), new Set([entryAbs]))
-			for (const upAbs of ups) {
-				const upRel = path.relative('.', upAbs).replaceAll('\\', '/')
-				const cached = _compileCache.get(upAbs)
-				const slots = cached?.result?.slots || 'shifted'
-				broadcast({ type: 'update', file: upRel, slots })
-			}
 		} catch(e) {
 			printStatus(rel, 'fail', [{ message: e.message }])
 			broadcast({ type: 'error', file: rel, errors: [{ message: e.message, snippet: e.stack || e.message }] })