bimba-cli 0.5.19 → 0.6.0

package/index.js

@@ -1,7 +1,7 @@
 #!/usr/bin/env bun
 
 import { parseArgs } from "util";
-import { imbaPlugin, stats, cache } from './plugin.js'
+import { imbaPlugin, stats, cache, setTarget } from './plugin.js'
 import {theme} from './utils.js';
 import fs from 'fs'
 import path from 'path';
@@ -23,6 +23,7 @@ try {
             minify: { type: 'boolean' },
             splitting: { type: 'boolean' },
             target: { type: 'string' },
+            external: { type: 'string', multiple: true },
             sourcemap: { type: 'string' },
             serve: { type: 'boolean' },
             port: { type: 'string' },
@@ -63,7 +64,8 @@ if(flags.help) {
     console.log("   "+theme.flags('--outdir <folder>')+"                     Compile imba files to the specified folder");
     console.log("   "+theme.flags('--minify')+"                              Minify compiled .js files");
     console.log("   "+theme.flags('--sourcemap <inline|external|none>')+"    How should sourcemap files be included in the .js");
-    console.log("   "+theme.flags('--platform <browser|node>')+"             Flag that will be passed to Imba compiler ('node' value does not work in Bun)");
+    console.log("   "+theme.flags('--target <browser|node>')+"               Target platform for both Imba compiler and Bun bundler");
+    console.log("   "+theme.flags('--external <package>')+"                Exclude package from bundle (repeatable, e.g. --external ws --external node-pty)");
     console.log("   "+theme.flags('--watch')+"                               Watch for changes in the entrypoint folder");
     console.log("   "+theme.flags('--clearcache')+"                          Clear cache on exit, works only when in watch mode");
     console.log("");
@@ -137,17 +139,25 @@ async function bundle() {
     console.log(theme.folder("──────────────────────────────────────────────────────────────────────"));
     console.log(theme.start(`Start building the Imba entrypoint: ${theme.filedir(entrypoint)}`));
 
+    // set Imba compiler platform based on target
+    const buildTarget = flags.target || 'browser';
+    setTarget(buildTarget);
+
     let result = undefined
     try {
-        result = await Bun.build({
+        const buildOpts = {
             entrypoints: [entrypoint],
             outdir: flags.outdir,
-            target: flags.target || 'browser',
+            target: buildTarget,
             sourcemap: flags.sourcemap || 'none',
             minify: true,
             splitting: flags.splitting || false,
             plugins: [imbaPlugin]
-        });
+        };
+        if (flags.external?.length) {
+            buildOpts.external = flags.external;
+        }
+        result = await Bun.build(buildOpts);
 
         if(stats.failed)
             console.log(theme.start(theme.failure(" Failure ") + theme.filename(` Imba compiler failed to proceed ${stats.failed} file${stats.failed > 1 ? 's' : ''}`)));

package/package.json

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

package/plugin.js

@@ -18,6 +18,11 @@ export let stats = {
   errors: 0,
 };
 
+// Target platform for the Imba compiler: 'browser' or 'node'
+// Set via setTarget() from the CLI before building
+export let target = 'browser';
+export function setTarget(t) { target = t; }
+
 export const imbaPlugin = {
   name: "imba",
   async setup(build) {
@@ -28,13 +33,11 @@ export const imbaPlugin = {
       const f = dir.parse(path)
       let contents = '';
 
-      // return the cached version if exists
-      const cached = dir.join(cache, Bun.hash(path) + '_' + fs.statSync(path).mtimeMs + '.js');
+      // return the cached version if exists (include target in hash to avoid cross-platform cache hits)
+      const cached = dir.join(cache, Bun.hash(path + ':' + target) + '_' + fs.statSync(path).mtimeMs + '.js');
       if (fs.existsSync(cached)) {
         stats.bundled++;
         stats.cached++;
-        //console.log(theme.action("cached: ") + theme.folder(f.dir + '/') + theme.filename(f.base) + " - " + theme.success("ok"));
-        //console.log(theme.action("compiling: ") + theme.folder(dir.join(f.dir,'/')) + theme.filename(f.base) + " - " + theme.success("from cache"));
         return {
           contents: await Bun.file(cached).text(),
           loader: "js",
@@ -42,14 +45,15 @@ export const imbaPlugin = {
       }
 
       // clear previous cached version
-      const glob = new Glob(Bun.hash(path) + '_' + "*.js");
+      const glob = new Glob(Bun.hash(path + ':' + target) + '_' + "*.js");
       for await (const file of glob.scan(cache)) if (fs.existsSync(dir.join(cache, file))) unlink(dir.join(cache, file));
 
       // if no cached version read and compile it with the imba compiler
       const file = await Bun.file(path).text();
+      const platform = target === 'node' || target === 'bun' ? 'node' : 'browser';
       const out = compiler.compile(file, {
           sourcePath: path,
-          platform: 'browser',
+          platform: platform,
           comments: false
       })
       

package/serve.js

@@ -42,7 +42,9 @@ const hmrClient = `
 			if (target) {
 				// Save old _ns_ before _patchClass overwrites prototype descriptors
 				if (target.prototype._ns_) _oldNs.set(name, target.prototype._ns_);
-				_patchClass(target, cls);
+				// When CSS-only (stable slots), skip patching render/methods —
+				// new render() has new Symbol closures that would cause duplicate DOM.
+				if (!_stableSlots) _patchClass(target, cls);
 			}
 		}
 	};
@@ -51,13 +53,12 @@ const hmrClient = `
 
 	// Copy all own property descriptors from source to target, skipping keys
 	// that match the shouldSkip predicate. Handles both string and symbol keys.
-	function _copyDescriptors(target, source, shouldSkip, skipSymbols) {
+	function _copyDescriptors(target, source, shouldSkip) {
 		for (const key of Object.getOwnPropertyNames(source)) {
 			if (shouldSkip(key)) continue;
 			const d = Object.getOwnPropertyDescriptor(source, key);
 			if (d) try { Object.defineProperty(target, key, d); } catch(_) {}
 		}
-		if (skipSymbols) return; // CSS-only: keep existing Symbol-keyed render caches
 		for (const key of Object.getOwnPropertySymbols(source)) {
 			const d = Object.getOwnPropertyDescriptor(source, key);
 			if (d) try { Object.defineProperty(target, key, d); } catch(_) {}
@@ -65,8 +66,8 @@ const hmrClient = `
 	}
 
 	function _patchClass(target, source) {
-		_copyDescriptors(target.prototype, source.prototype, k => k === 'constructor', _stableSlots);
-		_copyDescriptors(target, source, k => _skipStatics.has(k), _stableSlots);
+		_copyDescriptors(target.prototype, source.prototype, k => k === 'constructor');
+		_copyDescriptors(target, source, k => _skipStatics.has(k));
 	}
 
 	// ── HMR update handler ─────────────────────────────────────────────────────