bertui 0.4.4 → 0.4.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bertui",
-  "version": "0.4.4",
+  "version": "0.4.5",
   "description": "Lightning-fast React dev server powered by Bun and Elysia",
   "type": "module",
   "main": "./index.js",

package/src/build/image-optimizer.js

@@ -1,4 +1,4 @@
-// bertui/src/build/image-optimizer.js - WASM-POWERED VERSION 🚀
+// bertui/src/build/image-optimizer.js - FIXED VERSION
 import { join, extname } from 'path';
 import { existsSync, mkdirSync, readdirSync, cpSync } from 'fs';
 import logger from '../logger/logger.js';
@@ -46,7 +46,18 @@ export async function optimizeImages(srcDir, outDir) {
   let optimized = 0;
   let totalSaved = 0;
 
-  logger.info('🖼️  Optimizing images with WASM codecs...');
+  logger.info(`🖼️  Optimizing images from ${srcDir} to ${outDir}...`);
+
+  // Check if source directory exists
+  if (!existsSync(srcDir)) {
+    logger.warn(`⚠️  Source directory not found: ${srcDir}`);
+    return { optimized: 0, saved: 0 };
+  }
+
+  // Create output directory if it doesn't exist
+  if (!existsSync(outDir)) {
+    mkdirSync(outDir, { recursive: true });
+  }
 
   async function processDirectory(dir, targetDir) {
     const entries = readdirSync(dir, { withFileTypes: true });
@@ -56,10 +67,12 @@ export async function optimizeImages(srcDir, outDir) {
       const destPath = join(targetDir, entry.name);
 
       if (entry.isDirectory()) {
-        if (!existsSync(destPath)) {
-          mkdirSync(destPath, { recursive: true });
+        // Create subdirectory in target
+        const subDestPath = join(targetDir, entry.name);
+        if (!existsSync(subDestPath)) {
+          mkdirSync(subDestPath, { recursive: true });
         }
-        await processDirectory(srcPath, destPath);
+        await processDirectory(srcPath, subDestPath);
       } else if (entry.isFile()) {
         const ext = extname(entry.name).toLowerCase();
 
@@ -77,7 +90,12 @@ export async function optimizeImages(srcDir, outDir) {
           } catch (error) {
             logger.warn(`⚠️  Failed to optimize ${entry.name}: ${error.message}`);
             // Fallback: just copy the file
-            cpSync(srcPath, destPath);
+            try {
+              cpSync(srcPath, destPath);
+              logger.debug(`📋 Copied ${entry.name} (fallback)`);
+            } catch (copyError) {
+              logger.error(`❌ Failed to copy ${entry.name}: ${copyError.message}`);
+            }
           }
         }
       }
@@ -90,17 +108,45 @@ export async function optimizeImages(srcDir, outDir) {
     logger.success(
       `✅ Optimized ${optimized} images (saved ${(totalSaved / 1024).toFixed(2)}KB total)`
     );
+  } else {
+    logger.info(`📋 No images optimized (copied ${countFilesInDir(outDir)} files)`);
   }
 
   return { optimized, saved: totalSaved };
 }
 
+/**
+ * Count files in directory (for logging)
+ */
+function countFilesInDir(dir) {
+  if (!existsSync(dir)) return 0;
+  
+  let count = 0;
+  const entries = readdirSync(dir, { withFileTypes: true });
+  
+  for (const entry of entries) {
+    if (entry.isFile()) {
+      count++;
+    } else if (entry.isDirectory()) {
+      count += countFilesInDir(join(dir, entry.name));
+    }
+  }
+  
+  return count;
+}
+
 /**
  * Optimize a single image using WASM codecs
  */
 async function optimizeImage(srcPath, destPath) {
   const ext = extname(srcPath).toLowerCase();
   const originalFile = Bun.file(srcPath);
+  
+  // Check if file exists
+  if (!await originalFile.exists()) {
+    throw new Error(`File not found: ${srcPath}`);
+  }
+  
   const originalSize = originalFile.size;
 
   try {
@@ -140,6 +186,10 @@ async function optimizeImage(srcPath, destPath) {
       // WebP optimization
       const imageData = await webpDecode(originalBuffer);
       optimizedBuffer = await webpEncode(imageData, { quality: 85 });
+    } else {
+      // Unsupported format, just copy
+      cpSync(srcPath, destPath);
+      return null;
     }
 
     // Only save if we actually reduced the size
@@ -184,9 +234,20 @@ export async function checkOptimizationTools() {
  * Copy images without optimization (fallback)
  */
 export function copyImages(srcDir, outDir) {
-  const imageExtensions = ['.png', '.jpg', '.jpeg', '.webp', '.gif', '.svg', '.avif'];
+  const imageExtensions = ['.png', '.jpg', '.jpeg', '.webp', '.gif', '.svg', '.avif', '.ico'];
   let copied = 0;
 
+  // Check if source directory exists
+  if (!existsSync(srcDir)) {
+    logger.warn(`⚠️  Source directory not found: ${srcDir}`);
+    return 0;
+  }
+
+  // Create output directory if it doesn't exist
+  if (!existsSync(outDir)) {
+    mkdirSync(outDir, { recursive: true });
+  }
+
   function processDirectory(dir, targetDir) {
     const entries = readdirSync(dir, { withFileTypes: true });
 
@@ -195,22 +256,36 @@ export function copyImages(srcDir, outDir) {
       const destPath = join(targetDir, entry.name);
 
       if (entry.isDirectory()) {
-        if (!existsSync(destPath)) {
-          mkdirSync(destPath, { recursive: true });
+        // Create subdirectory in target
+        const subDestPath = join(targetDir, entry.name);
+        if (!existsSync(subDestPath)) {
+          mkdirSync(subDestPath, { recursive: true });
         }
-        processDirectory(srcPath, targetDir);
+        // FIXED: Use destPath, not targetDir
+        processDirectory(srcPath, subDestPath);
       } else if (entry.isFile()) {
         const ext = extname(entry.name).toLowerCase();
 
         if (imageExtensions.includes(ext)) {
-          cpSync(srcPath, destPath);
-          copied++;
+          try {
+            cpSync(srcPath, destPath);
+            copied++;
+            logger.debug(`📋 Copied ${entry.name}`);
+          } catch (error) {
+            logger.warn(`Failed to copy ${entry.name}: ${error.message}`);
+          }
         }
       }
     }
   }
 
   processDirectory(srcDir, outDir);
-  logger.info(`📋 Copied ${copied} images without optimization`);
+  
+  if (copied > 0) {
+    logger.info(`📋 Copied ${copied} images without optimization`);
+  } else {
+    logger.warn(`⚠️  No images found in ${srcDir}`);
+  }
+  
   return copied;
 }

package/src/build.js

@@ -124,6 +124,7 @@ export async function buildProduction(options = {}) {
   }
 }
 
+// ✅ FIX 3: Enhanced asset copying with proper directory structure
 // ✅ FIX 3: Enhanced asset copying with proper directory structure
 async function copyAllStaticAssets(root, outDir, optimize = true) {
   const publicDir = join(root, 'public');
@@ -132,9 +133,15 @@ async function copyAllStaticAssets(root, outDir, optimize = true) {
   let assetsCopied = 0;
   let assetsOptimized = 0;
   
+  logger.info(`🔍 Checking source directories...`);
+  logger.info(`  public/: ${existsSync(publicDir) ? '✅ exists' : '❌ not found'}`);
+  logger.info(`  src/images/: ${existsSync(srcImagesDir) ? '✅ exists' : '❌ not found'}`);
+  
   // Create images directory in dist/
   const distImagesDir = join(outDir, 'images');
-  mkdirSync(distImagesDir, { recursive: true });
+  if (!existsSync(distImagesDir)) {
+    mkdirSync(distImagesDir, { recursive: true });
+  }
   
   // Copy from public/ to root of dist/
   if (existsSync(publicDir)) {
@@ -145,26 +152,36 @@ async function copyAllStaticAssets(root, outDir, optimize = true) {
     } else {
       assetsCopied += copyImages(publicDir, outDir);
     }
+  } else {
+    logger.info('No public/ directory found, skipping...');
   }
   
   // ✅ FIX: Copy from src/images/ to dist/images/
   if (existsSync(srcImagesDir)) {
-    logger.info('Copying src/images/ to dist/images/...');
+    logger.info(`Copying src/images/ to dist/images/...`);
+    
+    // Debug: List files in src/images/
+    const files = readdirSync(srcImagesDir);
+    logger.info(`Found ${files.length} items in src/images/: ${files.join(', ')}`);
+    
     if (optimize) {
       const result = await optimizeImages(srcImagesDir, distImagesDir);
       assetsOptimized += result.optimized;
     } else {
       assetsCopied += copyImages(srcImagesDir, distImagesDir);
     }
+  } else {
+    logger.info('No src/images/ directory found, skipping...');
   }
   
   if (optimize && assetsOptimized > 0) {
     logger.success(`🎨 Optimized ${assetsOptimized} images with WASM codecs`);
-  } else {
+  } else if (assetsCopied > 0) {
     logger.success(`📋 Copied ${assetsCopied} static assets`);
+  } else {
+    logger.warn(`⚠️  No static assets found or copied`);
   }
 }
-
 async function buildAllCSS(root, outDir) {
   const srcStylesDir = join(root, 'src', 'styles');
   const stylesOutDir = join(outDir, 'styles');