slicejs-cli 3.4.0 → 3.4.1

package/commands/buildProduction/buildProduction.js

@@ -1,18 +1,15 @@
-// commands/buildProduction/buildProduction.js - VERSIÓN LIMPIA
+// commands/buildProduction/buildProduction.js - CLEAN VERSION
 
 import fs from 'fs-extra';
 import path from 'path';
-import { fileURLToPath } from 'url';
 import { minify as terserMinify } from 'terser';
 import { minify } from 'html-minifier-terser';
 import CleanCSS from 'clean-css';
 import Print from '../Print.js';
 import { getSrcPath, getDistPath, getConfigPath } from '../utils/PathHelper.js';
 
-const __dirname = path.dirname(fileURLToPath(import.meta.url));
-
 /**
- * Carga la configuración desde sliceConfig.json
+ * Loads configuration from sliceConfig.json
  */
 const loadConfig = () => {
   try {
@@ -26,7 +23,7 @@ const loadConfig = () => {
 };
 
 /**
- * Verifica dependencias necesarias para el build
+ * Checks necessary build dependencies
  */
 async function checkBuildDependencies() {
   const srcDir = getSrcPath(import.meta.url);
@@ -50,7 +47,7 @@ async function checkBuildDependencies() {
 }
 
 /**
- * Verifica que existan los archivos críticos para Slice.js
+ * Verifies that critical Slice.js files exist
  */
 async function verifySliceFiles(srcDir) {
   Print.info('Verifying Slice.js critical files...');
@@ -72,7 +69,7 @@ async function verifySliceFiles(srcDir) {
 }
 
 /**
- * Verifica la integridad del build para Slice.js
+ * Verifies build integrity for Slice.js
  */
 async function verifyBuildIntegrity(distDir) {
   Print.info('Verifying build integrity for Slice.js...');
@@ -101,7 +98,7 @@ async function verifyBuildIntegrity(distDir) {
 }
 
 /**
- * Copia sliceConfig.json al directorio dist
+ * Copies sliceConfig.json to the dist directory
  */
 async function copySliceConfig() {
   const srcConfig = getConfigPath(import.meta.url);
@@ -114,7 +111,7 @@ async function copySliceConfig() {
 }
 
 /**
- * Procesa un directorio completo
+ * Processes a complete directory
  */
 async function processDirectory(srcPath, distPath, baseSrcPath, options) {
   const items = await fs.readdir(srcPath);
@@ -134,7 +131,7 @@ async function processDirectory(srcPath, distPath, baseSrcPath, options) {
 }
 
 /**
- * Procesa un archivo individual
+ * Processes an individual file
  */
 async function processFile(srcFilePath, distFilePath, options) {
   const ext = path.extname(srcFilePath).toLowerCase();
@@ -200,7 +197,7 @@ async function processFile(srcFilePath, distFilePath, options) {
 }
 
 /**
- * Procesa el archivo components.js de forma especial
+ * Processes the components.js file in a special way
  */
 async function processComponentsFile(srcPath, distPath) {
   const content = await fs.readFile(srcPath, 'utf8');
@@ -229,7 +226,7 @@ async function processComponentsFile(srcPath, distPath) {
 }
 
 /**
- * Minifica archivos JavaScript preservando la arquitectura de Slice.js
+ * Minifies JavaScript files preserving Slice.js architecture
  */
 async function minifyJavaScript(srcPath, distPath) {
   const content = await fs.readFile(srcPath, 'utf8');
@@ -250,9 +247,9 @@ async function minifyJavaScript(srcPath, distPath) {
       reserved: [
         // Core Slice
         'slice', 'Slice', 'SliceJS', 'window', 'document',
-        // Clases principales
+        // Main classes
         'Controller', 'StylesManager', 'Router', 'Logger', 'Debugger',
-        // Métodos de Slice
+        // Slice methods
         'getClass', 'isProduction', 'getComponent', 'build', 'setTheme', 'attachTemplate',
         // Controller
         'componentCategories', 'templates', 'classes', 'requestedStyles', 'activeComponents',
@@ -269,7 +266,7 @@ async function minifyJavaScript(srcPath, distPath) {
         'onClickCallback', 'props',
         // Custom Elements
         'customElements', 'define', 'HTMLElement',
-        // DOM APIs críticas
+        // Critical DOM APIs
         'addEventListener', 'removeEventListener', 'querySelector', 'querySelectorAll',
         'appendChild', 'removeChild', 'innerHTML', 'textContent', 'style', 'classList',
         // Lifecycle
@@ -308,7 +305,7 @@ async function minifyJavaScript(srcPath, distPath) {
 }
 
 /**
- * Minifica archivos CSS
+ * Minifies CSS files
  */
 async function minifyCSS(srcPath, distPath) {
   const content = await fs.readFile(srcPath, 'utf8');
@@ -334,7 +331,7 @@ async function minifyCSS(srcPath, distPath) {
 }
 
 /**
- * Minifica archivos HTML
+ * Minifies HTML files
  */
 async function minifyHTML(srcPath, distPath) {
   const content = await fs.readFile(srcPath, 'utf8');
@@ -367,7 +364,7 @@ async function minifyHTML(srcPath, distPath) {
 }
 
 /**
- * Crea un bundle optimizado del archivo principal
+ * Creates an optimized bundle of the main file
  */
 async function createOptimizedBundle() {
   Print.buildProgress('Creating optimized bundle...');
@@ -382,7 +379,7 @@ async function createOptimizedBundle() {
 }
 
 /**
- * Genera estadísticas del build
+ * Generates build statistics
  */
 async function generateBuildStats(srcDir, distDir) {
   Print.buildProgress('Generating build statistics...');
@@ -422,7 +419,7 @@ async function generateBuildStats(srcDir, distDir) {
 }
 
 /**
- * Analiza el build sin construir
+ * Analyzes the build without building
  */
 async function analyzeBuild() {
   const distDir = getDistPath(import.meta.url);
@@ -440,7 +437,7 @@ async function analyzeBuild() {
 }
 
 /**
- * FUNCIÓN PRINCIPAL DE BUILD
+ * MAIN BUILD FUNCTION
  */
 export default async function buildProduction(options = {}) {
   const startTime = Date.now();
@@ -458,7 +455,7 @@ export default async function buildProduction(options = {}) {
 
     await verifySliceFiles(srcDir);
 
-    // Limpiar directorio dist
+    // Clean dist directory
     if (await fs.pathExists(distDir)) {
       if (!options.skipClean) {
         Print.info('Cleaning previous build...');
@@ -470,7 +467,7 @@ export default async function buildProduction(options = {}) {
     await fs.ensureDir(distDir);
     await copySliceConfig();
 
-    // Procesar archivos
+    // Process files
     Print.info('Processing and optimizing source files for Slice.js...');
     await processDirectory(srcDir, distDir, srcDir, options);
     Print.success('All source files processed and optimized');
@@ -498,7 +495,7 @@ export default async function buildProduction(options = {}) {
 }
 
 /**
- * Servidor de preview para testing del build de producción
+ * Preview server for testing the production build
  */
 export async function serveProductionBuild(port) {
   try {
@@ -546,7 +543,7 @@ export async function serveProductionBuild(port) {
 }
 
 /**
- * Comando build con opciones
+ * Build command with options
  */
 export async function buildCommand(options = {}) {
   const config = loadConfig();

package/commands/bundle/bundle.js

@@ -4,13 +4,14 @@ import fs from 'fs-extra';
 import DependencyAnalyzer from '../utils/bundling/DependencyAnalyzer.js';
 import BundleGenerator from '../utils/bundling/BundleGenerator.js';
 import Print from '..//Print.js';
+import { getProjectRoot, getSrcPath } from '../utils/PathHelper.js';
 
 /**
  * Main bundling command
  */
 export default async function bundle(options = {}) {
   const startTime = Date.now();
-  const projectRoot = process.cwd();
+  const projectRoot = getProjectRoot(import.meta.url);
 
   try {
     Print.title('📦 Slice.js Bundle Generator');
@@ -82,14 +83,14 @@ export default async function bundle(options = {}) {
  */
 async function validateProject(projectRoot) {
   const requiredPaths = [
-    'src/Components/components.js',
-    'src/routes.js'
+    getSrcPath(import.meta.url, 'Components', 'components.js'),
+    getSrcPath(import.meta.url, 'routes.js')
   ];
 
-  for (const reqPath of requiredPaths) {
-    const fullPath = path.join(projectRoot, reqPath);
+  for (const fullPath of requiredPaths) {
     if (!await fs.pathExists(fullPath)) {
-      throw new Error(`Required file not found: ${reqPath}`);
+      const relativePath = path.relative(projectRoot, fullPath);
+      throw new Error(`Required file not found: ${relativePath}`);
     }
   }
 }
@@ -146,8 +147,7 @@ function printSummary(result, startTime) {
  * Subcommand: Clean bundles
  */
 export async function cleanBundles() {
-  const projectRoot = process.cwd();
-  const srcPath = path.join(projectRoot, 'src');
+  const srcPath = getSrcPath(import.meta.url);
 
   try {
     Print.title('🧹 Cleaning bundles...');
@@ -166,7 +166,7 @@ export async function cleanBundles() {
     }
 
     // Remove config
-    const configPath = path.join(srcPath, 'bundle.config.json');
+    const configPath = getSrcPath(import.meta.url, 'bundle.config.json');
     if (await fs.pathExists(configPath)) {
       await fs.remove(configPath);
       console.log(`   ✓ Deleted: bundle.config.json`);
@@ -185,8 +185,7 @@ export async function cleanBundles() {
  * Subcommand: Bundle information
  */
 export async function bundleInfo() {
-  const projectRoot = process.cwd();
-  const configPath = path.join(projectRoot, 'src/bundle.config.json');
+  const configPath = getSrcPath(import.meta.url, 'bundle.config.json');
 
   try {
     if (!await fs.pathExists(configPath)) {

package/commands/createComponent/createComponent.js

@@ -2,21 +2,19 @@
 import componentTemplates from './VisualComponentTemplate.js';
 import fs from 'fs-extra';
 import path from 'path';
-import { fileURLToPath } from 'url';
 import Validations from '../Validations.js';
 import Print from '../Print.js';
 import { getSrcPath } from '../utils/PathHelper.js';
-const __dirname = path.dirname(fileURLToPath(import.meta.url));
 
 function createComponent(componentName, category) {
-    // Validación: Nombre de componente requerido
+    // Validation: Component name is required
     if (!componentName) {
         Print.error('Component name is required');
         Print.commandExample("Create a component", "slice component create");
         return false;
     }
 
-    // Validación: Nombre de componente válido
+    // Validation: Valid component name
     if (!Validations.isValidComponentName(componentName)) {
         Print.error(`Invalid component name: '${componentName}'`);
         Print.info('Component name must start with a letter and contain only alphanumeric characters');
@@ -25,7 +23,7 @@ function createComponent(componentName, category) {
         return false;
     }
 
-    // Validación: Componente ya existe
+    // Validation: Component already exists
     if(Validations.componentExists(componentName)){
         Print.error(`Component '${componentName}' already exists in your project`);
         Print.info('Please use a different name or delete the existing component first');
@@ -33,7 +31,7 @@ function createComponent(componentName, category) {
         return false;
     }
 
-    // Validación: Categoría válida
+    // Validation: Valid category
     let flagCategory = Validations.isValidCategory(category);
 
     if (!flagCategory.isValid) {
@@ -44,14 +42,14 @@ function createComponent(componentName, category) {
     }
     category = flagCategory.category;
 
-    // Crear el nombre de la clase y del archivo
+    // Create class name and file name
     const className = componentName.charAt(0).toUpperCase() + componentName.slice(1);
     const fileName = `${className}.js`;
     let template;
 
     const type = Validations.getCategoryType(category);
 
-    // Generar template según el tipo
+    // Generate template based on type
     if(type === 'Visual'){
        template = componentTemplates.visual(className);
     } else if(type === 'Service'){
@@ -67,7 +65,7 @@ function createComponent(componentName, category) {
     const componentDir = getSrcPath(import.meta.url, categoryPathClean, className);
     
     try {
-        // Crear directorio del componente
+        // Create component directory
         fs.ensureDirSync(componentDir);
     } catch (error) {
         Print.error(`Failed to create component directory: '${componentDir}'`);
@@ -75,10 +73,10 @@ function createComponent(componentName, category) {
         return false;
     }
 
-    // Determinar la ruta del archivo
+    // Determine the file path
     let componentPath = path.join(componentDir, fileName);
 
-    // Verificar si el archivo ya existe (doble verificación)
+    // Verify if the file already exists (double check)
     if (fs.existsSync(componentPath)) {
         Print.error(`Component file already exists at: '${componentPath}'`);
         Print.info('This component may have been created outside the CLI');
@@ -86,13 +84,13 @@ function createComponent(componentName, category) {
     }
 
     try {
-        // Escribir el código del componente en el archivo
+        // Write component code to file
         fs.writeFileSync(componentPath, template);
 
-        // Si es Visual, crear archivos adicionales (CSS y HTML)
+        // If Visual, create additional files (CSS and HTML)
         if(type === 'Visual'){
-            const cssPath = `${componentDir}/${className}.css`;
-            const htmlPath = `${componentDir}/${className}.html`;
+            const cssPath = path.join(componentDir, `${className}.css`);
+            const htmlPath = path.join(componentDir, `${className}.html`);
             
             fs.writeFileSync(cssPath, '/* Styles for ' + componentName + ' component */\n');
             fs.writeFileSync(htmlPath, `<div class="${componentName.toLowerCase()}">\n  ${componentName}\n</div>`);
@@ -107,7 +105,7 @@ function createComponent(componentName, category) {
         Print.error(`Failed to create component files`);
         Print.info(`Error details: ${error.message}`);
         
-        // Intentar limpiar archivos parcialmente creados
+        // Try to clean up partially created files
         try {
             if (fs.existsSync(componentDir)) {
                 fs.removeSync(componentDir);

package/commands/deleteComponent/deleteComponent.js

@@ -7,21 +7,21 @@ import { getSrcPath } from '../utils/PathHelper.js';
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
 
 function deleteComponent(componentName, category) {
-    // Validación: Nombre de componente requerido
+    // Validation: Component name is required
     if (!componentName) {
         Print.error('Component name is required to delete');
         Print.commandExample("Delete a component", "slice component delete");
         return false;
     }
 
-    // Validación: Nombre de componente válido
+    // Validation: Valid component name
     if (!Validations.isValidComponentName(componentName)) {
         Print.error(`Invalid component name: '${componentName}'`);
         Print.info('Component name must start with a letter and contain only alphanumeric characters');
         return false;
     }
 
-    // Validación: Categoría válida
+    // Validation: Valid category
     let flagCategory = Validations.isValidCategory(category);
 
     if (!flagCategory.isValid) {
@@ -36,7 +36,7 @@ function deleteComponent(componentName, category) {
     const categoryPathClean = categoryPath ? categoryPath.replace(/^[/\\]+/, '') : '';
     const componentDir = getSrcPath(import.meta.url, categoryPathClean, componentName);
 
-    // Verificar si el directorio del componente existe
+    // Check if component directory exists
     if (!fs.existsSync(componentDir)) {
         Print.error(`Component '${componentName}' does not exist in category '${category}'`);
         Print.info('Make sure you selected the correct category');
@@ -44,7 +44,7 @@ function deleteComponent(componentName, category) {
         return false;
     }
 
-    // Verificar si es un directorio
+    // Verify it's a directory
     try {
         const stats = fs.statSync(componentDir);
         if (!stats.isDirectory()) {
@@ -58,7 +58,7 @@ function deleteComponent(componentName, category) {
         return false;
     }
 
-    // Intentar eliminar el directorio del componente y su contenido
+    // Try to delete the component directory and its contents
     try {
         const files = fs.readdirSync(componentDir);
         Print.info(`Deleting ${files.length} file(s) from component directory...`);

package/commands/doctor/doctor.js

@@ -1,6 +1,5 @@
 import fs from 'fs-extra';
 import path from 'path';
-import { fileURLToPath } from 'url';
 import { createServer } from 'net';
 import chalk from 'chalk';
 import Table from 'cli-table3';
@@ -11,10 +10,8 @@ import { promisify } from 'util';
 import { getProjectRoot, getSrcPath, getApiPath, getConfigPath, getPath } from '../utils/PathHelper.js';
 import updateManager from '../utils/updateManager.js';
 
-const __dirname = path.dirname(fileURLToPath(import.meta.url));
-
 /**
- * Verifica la versión de Node.js
+ * Checks the Node.js version
  */
 async function checkNodeVersion() {
     const currentVersion = process.version;
@@ -36,7 +33,7 @@ async function checkNodeVersion() {
 }
 
 /**
- * Verifica la estructura de directorios
+ * Checks the directory structure
  */
 async function checkDirectoryStructure() {
     const srcPath = getSrcPath(import.meta.url);
@@ -64,7 +61,7 @@ async function checkDirectoryStructure() {
 }
 
 /**
- * Verifica sliceConfig.json
+ * Checks sliceConfig.json
  */
 async function checkConfig() {
     const configPath = getConfigPath(import.meta.url);
@@ -102,7 +99,7 @@ async function checkConfig() {
 }
 
 /**
- * Verifica disponibilidad del puerto
+ * Checks port availability
  */
 async function checkPort() {
     const configPath = getConfigPath(import.meta.url);
@@ -113,7 +110,7 @@ async function checkPort() {
             const config = await fs.readJson(configPath);
             port = config.server?.port || 3000;
         }
-    } catch { }
+    } catch { /* config missing or unreadable — use default port */ }
 
     return new Promise((resolve) => {
         const server = createServer();
@@ -146,11 +143,11 @@ async function checkPort() {
 }
 
 /**
- * Verifica dependencias en package.json
+ * Checks dependencies in package.json
  */
 async function checkDependencies() {
     const projectRoot = getProjectRoot(import.meta.url);
-    const packagePath = path.join(projectRoot, 'package.json');
+    const packagePath = getPath(import.meta.url, 'package.json');
 
     if (!await fs.pathExists(packagePath)) {
         return {
@@ -163,7 +160,7 @@ async function checkDependencies() {
     try {
         const pkg = await fs.readJson(packagePath);
         const hasFrameworkDep = pkg.dependencies?.['slicejs-web-framework'] || pkg.devDependencies?.['slicejs-web-framework'];
-        const frameworkNodePath = path.join(projectRoot, 'node_modules', 'slicejs-web-framework', 'package.json');
+        const frameworkNodePath = getPath(import.meta.url, 'node_modules', 'slicejs-web-framework', 'package.json');
         const hasFrameworkNode = await fs.pathExists(frameworkNodePath);
         const hasFramework = !!(hasFrameworkDep || hasFrameworkNode);
 
@@ -194,7 +191,7 @@ async function checkDependencies() {
 }
 
 /**
- * Verifica integridad de componentes
+ * Checks component integrity
  */
 async function checkComponents() {
     const configPath = getConfigPath(import.meta.url);
@@ -228,7 +225,7 @@ async function checkComponents() {
                     if (stat.isDirectory()) {
                         totalComponents++;
 
-                        // Verificar archivos JS
+                        // Check JS files
                         const jsFile = path.join(itemPath, `${item}.js`);
                         if (!await fs.pathExists(jsFile)) {
                             componentIssues++;
@@ -260,7 +257,7 @@ async function checkComponents() {
 }
 
 /**
- * Comando principal de diagnóstico
+ * Main diagnostic command
  */
 export default async function runDiagnostics() {
     Print.newLine();