bimba-cli 0.5.12 → 0.5.13

package/package.json

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

package/serve.js

@@ -385,6 +385,19 @@ function stabilizeSymbols(js, filepath) {
 	return { js: bootstrap + out, slotCount: count }
 }
 
+// Imba's compile result puts `errors` on the prototype as a getter, so plain
+// object spread (`{...result}`) silently strips it. We always normalize to a
+// plain shape with `errors` as an own property — otherwise downstream callers
+// see no errors and serve empty 200s for broken files.
+function _normalizeResult(result, extras) {
+	return {
+		js: result.js,
+		errors: result.errors || [],
+		slots: result.slots,
+		...extras,
+	}
+}
+
 async function compileFile(filepath) {
 	const abs = path.resolve(filepath)
 	const file = Bun.file(filepath)
@@ -392,7 +405,7 @@ async function compileFile(filepath) {
 	const mtime = stat.mtime.getTime()
 
 	const cached = _compileCache.get(abs)
-	if (cached && cached.mtime === mtime) return { ...cached.result, changeType: 'cached', slots: cached.slots }
+	if (cached && cached.mtime === mtime) return _normalizeResult(cached.result, { changeType: 'cached' })
 
 	const code = await file.text()
 	const result = compiler.compile(code, {
@@ -401,19 +414,22 @@ async function compileFile(filepath) {
 		sourcemap: 'inline',
 	})
 
-	if (!result.errors?.length && result.js) {
+	const errors = result.errors || []
+	if (!errors.length && result.js) {
 		const { js, slotCount } = stabilizeSymbols(result.js, filepath)
 		result.js = js
-		const prev = _prevSlots.get(filepath)
+		const prev = _prevSlots.get(abs)
 		result.slots = (prev === undefined || prev === slotCount) ? 'stable' : 'shifted'
-		_prevSlots.set(filepath, slotCount)
+		_prevSlots.set(abs, slotCount)
 		updateImportGraph(abs, extractImports(js, abs))
 	}
 
-	const changeType = _prevJs.get(abs) === result.js ? 'none' : 'full'
-	_prevJs.set(abs, result.js)
-	_compileCache.set(abs, { mtime, result, slots: result.slots })
-	return { ...result, changeType }
+	// Bake errors as an own property so caching/spreading preserves them.
+	const baked = { js: result.js, errors, slots: result.slots }
+	const changeType = _prevJs.get(abs) === baked.js ? 'none' : 'full'
+	_prevJs.set(abs, baked.js)
+	_compileCache.set(abs, { mtime, result: baked })
+	return _normalizeResult(baked, { changeType })
 }
 
 // ─── HTML helpers ─────────────────────────────────────────────────────────────
@@ -580,7 +596,7 @@ export function serve(entrypoint, flags) {
 			for (const upAbs of ups) {
 				const upRel = path.relative('.', upAbs).replaceAll('\\', '/')
 				const cached = _compileCache.get(upAbs)
-				const slots = cached?.slots || 'shifted'
+				const slots = cached?.result?.slots || 'shifted'
 				broadcast({ type: 'update', file: upRel, slots })
 			}
 		} catch(e) {