slicejs-cli 2.2.3 → 2.2.5

package/commands/buildProduction/buildProduction.js

@@ -11,29 +11,50 @@ import Print from '../Print.js';
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
 
 /**
- * Opciones de minificación para diferentes tipos de archivos
+ * Opciones de minificación para diferentes tipos de archivos - CORREGIDAS
  */
 const getMinificationOptions = () => ({
   js: {
     compress: {
       dead_code: true,
-      drop_console: true, // Remover console.log en producción
+      drop_console: false, // NO remover console.log para evitar problemas
       drop_debugger: true,
-      pure_funcs: ['console.log', 'console.info', 'console.warn'],
-      passes: 2
+      pure_funcs: [], // NO remover funciones específicas
+      passes: 1, // Reducir pasadas para ser menos agresivo
+      keep_classnames: true, // IMPORTANTE: Preservar nombres de clases
+      keep_fnames: true // IMPORTANTE: Preservar nombres de funciones
     },
     mangle: {
-      toplevel: true,
-      reserved: ['Slice', 'Controller', 'StylesManager'] // Preservar clases principales
+      toplevel: false, // NO hacer mangle a nivel superior
+      keep_classnames: true, // Preservar nombres de clases
+      keep_fnames: true, // Preservar nombres de funciones
+      reserved: [
+        // Framework core
+        'Slice', 'Controller', 'StylesManager', 'ThemeManager', 'Logger',
+        // Métodos importantes
+        'slice', 'build', 'init', 'attachTemplate', 'getComponent',
+        // Eventos y propiedades de componentes
+        'constructor', 'connectedCallback', 'disconnectedCallback',
+        'attributeChangedCallback', 'adoptedCallback',
+        // Variables comunes en componentes
+        'componentName', 'props', 'options', 'value', 'disabled',
+        // HTML Elements y DOM
+        'HTMLElement', 'customElements', 'define', 'querySelector',
+        'querySelectorAll', 'addEventListener', 'removeEventListener',
+        // Métodos de componentes Slice.js
+        'setComponentProps', 'componentCategories', 'templates',
+        'activeComponents', 'classes', 'requestedStyles'
+      ]
     },
     output: {
       comments: false,
-      beautify: false
+      beautify: false,
+      keep_quoted_props: true // Preservar propiedades entre comillas
     },
-    toplevel: true
+    toplevel: false // NO optimizar a nivel superior
   },
   css: {
-    level: 2, // Optimización agresiva
+    level: 1, // Optimización moderada en lugar de agresiva
     returnPromise: false
   },
   html: {
@@ -41,20 +62,38 @@ const getMinificationOptions = () => ({
     removeComments: true,
     removeRedundantAttributes: true,
     removeEmptyAttributes: true,
-    minifyCSS: true,
-    minifyJS: true,
+    minifyCSS: false, // NO minificar CSS inline para evitar problemas
+    minifyJS: false, // NO minificar JS inline para evitar problemas
     useShortDoctype: true,
-    removeAttributeQuotes: true,
-    removeOptionalTags: true
+    removeAttributeQuotes: false, // Mantener comillas en atributos
+    removeOptionalTags: false // Mantener tags opcionales
   }
 });
 
 /**
- * Minifica un archivo JavaScript
+ * Minifica un archivo JavaScript de forma segura
  */
 async function minifyJavaScript(content, filename) {
   try {
-    const options = getMinificationOptions().js;
+    // Para archivos de componentes, ser menos agresivo
+    const isComponentFile = filename.includes('/Components/') || filename.includes('\\Components\\');
+    
+    let options = getMinificationOptions().js;
+    
+    if (isComponentFile) {
+      // Configuración especial para archivos de componentes
+      options = {
+        ...options,
+        compress: {
+          ...options.compress,
+          passes: 1,
+          keep_classnames: true,
+          keep_fnames: true
+        },
+        mangle: false // NO hacer mangle en archivos de componentes
+      };
+    }
+    
     const result = await terserMinify(content, options);
     
     if (result.error) {
@@ -70,7 +109,9 @@ async function minifyJavaScript(content, filename) {
     return result.code;
   } catch (error) {
     Print.error(`Error minifying ${filename}: ${error.message}`);
-    throw error;
+    // En caso de error, devolver contenido original
+    Print.warning(`Using original content for ${filename}`);
+    return content;
   }
 }
 
@@ -377,7 +418,7 @@ export async function serveProductionBuild(port = 3001) {
 }
 
 /**
- * Comando build con opciones
+ * Comando build con opciones - CORREGIDO
  */
 export async function buildCommand(options = {}) {
   // Verificar dependencias necesarias
@@ -400,17 +441,18 @@ export async function buildCommand(options = {}) {
   // Build completo
   const success = await buildProduction(options);
   
+  // Solo mostrar mensaje informativo, no ejecutar servidor automáticamente
   if (success && options.preview) {
     Print.newLine();
-    Print.info('Starting preview server...');
-    setTimeout(() => {
-      serveProductionBuild(options.port);
-    }, 1000);
+    Print.info('✨ Build completed successfully!');
+    Print.info('💡 Use "slice build --serve" to preview the production build');
+    Print.info('💡 Or "slice start" to start production server');
   }
   
   return success;
 }
 
+
 /**
  * Verifica que las dependencias de build estén instaladas en el CLI
  */

package/commands/startServer/startServer.js

@@ -1,4 +1,4 @@
-// commands/startServer/startServer.js
+// commands/startServer/startServer.js - VERSIÓN SIMPLIFICADA
 
 import fs from 'fs-extra';
 import path from 'path';
@@ -27,66 +27,7 @@ async function checkDevelopmentStructure() {
 }
 
 /**
- * Modifica temporalmente el servidor Express para modo producción
- */
-async function createProductionIndexFile() {
-  try {
-    const apiDir = path.join(__dirname, '../../../../api');
-    const originalIndexPath = path.join(apiDir, 'index.js');
-    const backupIndexPath = path.join(apiDir, 'index.dev.js');
-    
-    // Crear backup del index original si no existe
-    if (!await fs.pathExists(backupIndexPath)) {
-      await fs.copy(originalIndexPath, backupIndexPath);
-    }
-    
-    // Leer el contenido original
-    const originalContent = await fs.readFile(originalIndexPath, 'utf8');
-    
-    // Modificar para servir desde /dist en lugar de /src
-    const productionContent = originalContent.replace(
-      /express\.static\(['"`]src['"`]\)/g,
-      "express.static('dist')"
-    ).replace(
-      /express\.static\(path\.join\(__dirname,\s*['"`]\.\.\/src['"`]\)\)/g,
-      "express.static(path.join(__dirname, '../dist'))"
-    );
-    
-    // Escribir la versión modificada directamente
-    await fs.writeFile(originalIndexPath, productionContent, 'utf8');
-    
-    Print.success('Express server configured for production mode');
-    
-    return true;
-  } catch (error) {
-    Print.error(`Error configuring production server: ${error.message}`);
-    return false;
-  }
-}
-
-/**
- * Restaura el servidor Express al modo desarrollo
- */
-async function restoreDevelopmentIndexFile() {
-  try {
-    const apiDir = path.join(__dirname, '../../../../api');
-    const originalIndexPath = path.join(apiDir, 'index.js');
-    const backupIndexPath = path.join(apiDir, 'index.dev.js');
-    
-    if (await fs.pathExists(backupIndexPath)) {
-      await fs.copy(backupIndexPath, originalIndexPath);
-      Print.success('Express server restored to development mode');
-    }
-    
-    return true;
-  } catch (error) {
-    Print.error(`Error restoring development server: ${error.message}`);
-    return false;
-  }
-}
-
-/**
- * Inicia el servidor Node.js
+ * Inicia el servidor Node.js - SIMPLIFICADO
  */
 function startNodeServer(port, mode) {
   return new Promise((resolve, reject) => {
@@ -99,7 +40,8 @@ function startNodeServer(port, mode) {
       env: {
         ...process.env,
         PORT: port,
-        NODE_ENV: mode === 'production' ? 'production' : 'development'
+        NODE_ENV: mode === 'production' ? 'production' : 'development',
+        SLICE_CLI_MODE: 'true' // Flag para que api/index.js sepa que viene del CLI
       }
     });
 
@@ -108,39 +50,26 @@ function startNodeServer(port, mode) {
       reject(error);
     });
 
-    // Manejar Ctrl+C para limpiar archivos temporales
-    process.on('SIGINT', async () => {
+    // Manejar Ctrl+C - SIMPLIFICADO
+    process.on('SIGINT', () => {
       Print.info('Shutting down server...');
-      
-      if (mode === 'production') {
-        await restoreDevelopmentIndexFile();
-      }
-      
       serverProcess.kill('SIGINT');
       process.exit(0);
     });
 
-    // Manejar cierre del proceso
-    process.on('SIGTERM', async () => {
-      if (mode === 'production') {
-        await restoreDevelopmentIndexFile();
-      }
-      
+    process.on('SIGTERM', () => {
       serverProcess.kill('SIGTERM');
     });
 
-    // El servidor se considera iniciado exitosamente después de un breve delay
+    // NO mostrar mensajes duplicados - el api/index.js ya se encarga
     setTimeout(() => {
-      Print.success(`${mode === 'production' ? 'Production' : 'Development'} server running at http://localhost:${port}`);
-      Print.info(`Serving files from /${mode === 'production' ? 'dist' : 'src'} directory`);
-      Print.info('Press Ctrl+C to stop the server');
       resolve(serverProcess);
-    }, 1000);
+    }, 500);
   });
 }
 
 /**
- * Función principal para iniciar servidor
+ * Función principal para iniciar servidor - ULTRA SIMPLIFICADA
  */
 export default async function startServer(options = {}) {
   const { mode = 'development', port = 3000 } = options;
@@ -155,35 +84,20 @@ export default async function startServer(options = {}) {
     }
     
     if (mode === 'production') {
-      // Modo producción: verificar build y configurar servidor
+      // Verificar que existe build de producción
       if (!await checkProductionBuild()) {
         throw new Error('No production build found. Run "slice build" first.');
       }
-      
-      // Configurar Express para modo producción (modifica api/index.js temporalmente)
-      const configSuccess = await createProductionIndexFile();
-      if (!configSuccess) {
-        throw new Error('Failed to configure production server');
-      }
-      
       Print.info('Production mode: serving optimized files from /dist');
     } else {
-      // Modo desarrollo: asegurar que está en modo desarrollo
-      await restoreDevelopmentIndexFile();
       Print.info('Development mode: serving files from /src with hot reload');
     }
     
-    // Iniciar el servidor (solo uno)
+    // Iniciar el servidor - api/index.js maneja todo automáticamente
     await startNodeServer(port, mode);
     
   } catch (error) {
     Print.error(`Failed to start server: ${error.message}`);
-    
-    // Limpiar en caso de error
-    if (mode === 'production') {
-      await restoreDevelopmentIndexFile();
-    }
-    
     throw error;
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "slicejs-cli",
-  "version": "2.2.3",
+  "version": "2.2.5",
   "description": "Command client for developing web applications with Slice.js framework",
   "main": "client.js",
   "scripts": {