jerkjs 2.5.2 → 2.5.6

package/lib/mvc/controllerBase.js

@@ -207,57 +207,125 @@ class ControllerBase {
    * Carga un modelo para su uso en el controlador
    * @param {string} modelName - Nombre del modelo a cargar
    * @param {Object} options - Opciones para la creación del modelo
+   * @param {Object} options.adapter - Adaptador de base de datos opcional
+   * @param {string} options.modelDir - Directorio personalizado donde buscar el modelo
    * @returns {ModelBase} - Instancia del modelo cargado
    */
   loadModel(modelName, options = {}) {
     try {
-      // Construir la ruta del modelo
-      // Buscar en diferentes ubicaciones posibles
+      // Obtener la ruta del archivo que llama a este método para calcular rutas relativas correctamente
+      const callerDir = process.cwd();
       let ModelClass;
 
-      // Array de rutas posibles para buscar el modelo
-      // Usamos rutas absolutas basadas en el directorio actual de trabajo
-      const possiblePaths = [
-        `./models/${modelName}`,       // En el subdirectorio models del directorio actual
-        `./models/${modelName}.js`,    // Con extensión explícita
-        `../models/${modelName}`,      // En el directorio models del directorio padre
-        `../models/${modelName}.js`,   // Con extensión explícita
-        `../../models/${modelName}`,   // En el directorio models del directorio abuelo
-        `../../models/${modelName}.js`, // Con extensión explícita
-        `../../../models/${modelName}`, // En el directorio models del directorio bisabuelo
-        `../../../models/${modelName}.js`, // Con extensión explícita
-        `./${modelName}`,              // En el directorio actual sin subdirectorio
-        `./${modelName}.js`            // En el directorio actual con extensión
-      ];
-
-      let lastError = null;
-
-      for (const path of possiblePaths) {
+      // Si se especifica un directorio personalizado, usarlo como prioridad
+      if (options.modelDir) {
+        const customPath = `${options.modelDir}/${modelName}`;
+        const customPathWithJs = `${options.modelDir}/${modelName}.js`;
+        
+        // Intentar cargar desde el directorio personalizado
         try {
-          // Usar require con la ruta absoluta desde el directorio actual
-          ModelClass = require(path);
-          // Si la importación fue exitosa, salir del bucle
-          break;
+          const fullPath = require.resolve(customPath, { paths: [callerDir] });
+          ModelClass = require(fullPath);
         } catch (error) {
-          // Guardar el último error para propósitos de depuración
-          lastError = error;
-          continue;
+          try {
+            const fullPath = require.resolve(customPathWithJs, { paths: [callerDir] });
+            ModelClass = require(fullPath);
+          } catch (error2) {
+            // Si falla en el directorio personalizado, continuar con rutas estándar
+          }
         }
       }
 
-      // Si no se encontró el modelo en ninguna ubicación
+      // Si no se encontró el modelo en el directorio personalizado, buscar en rutas estándar
       if (!ModelClass) {
-        throw new Error(`Modelo '${modelName}' no encontrado en ubicaciones estándar. Ultimo error: ${lastError?.message || 'desconocido'}`);
+        // Array de rutas posibles para buscar el modelo
+        // Usamos rutas relativas al directorio actual de trabajo
+        const possiblePaths = [
+          // Rutas relativas al directorio actual
+          `./models/${modelName}`,       // En el subdirectorio models del directorio actual
+          `./models/${modelName}.js`,    // Con extensión explícita
+          `./${modelName}`,              // En el directorio actual sin subdirectorio
+          `./${modelName}.js`,           // En el directorio actual con extensión
+          
+          // Rutas relativas a directorios superiores
+          `../models/${modelName}`,      // En el directorio models del directorio padre
+          `../models/${modelName}.js`,   // Con extensión explícita
+          `../../models/${modelName}`,   // En el directorio models del directorio abuelo
+          `../../models/${modelName}.js`, // Con extensión explícita
+          `../../../models/${modelName}`, // En el directorio models del directorio bisabuelo
+          `../../../models/${modelName}.js`, // Con extensión explícita
+          
+        ];
+
+        let lastError = null;
+
+        // Intentar cargar el modelo desde cada ruta posible
+        for (const relativePath of possiblePaths) {
+          try {
+            // Resolver la ruta absoluta
+            const fullPath = require.resolve(relativePath, { paths: [callerDir] });
+            ModelClass = require(fullPath);
+            
+            // Si la importación fue exitosa, salir del bucle
+            break;
+          } catch (error) {
+            // Solo registrar el error si es diferente al anterior para evitar spam
+            if (!lastError || lastError.message !== error.message) {
+              lastError = error;
+            }
+            continue;
+          }
+        }
+
+        // Si no se encontró el modelo en ninguna ubicación
+        if (!ModelClass) {
+          throw new Error(`Modelo '${modelName}' no encontrado en ubicaciones estándar. Ultimo error: ${lastError?.message || 'desconocido'}`);
+        }
       }
 
-      // Crear instancia del modelo con las opciones proporcionadas
-      const modelInstance = new ModelClass(options);
+      // Si ModelClass no es una función constructora, verificar si es un objeto con el modelo
+      if (typeof ModelClass !== 'function') {
+        if (ModelClass.default && typeof ModelClass.default === 'function') {
+          ModelClass = ModelClass.default; // Soporte para ES6 modules
+        } else {
+          // Buscar la clase en el objeto exportado
+          const exportedKeys = Object.keys(ModelClass);
+          for (const key of exportedKeys) {
+            if (typeof ModelClass[key] === 'function' && ModelClass[key].prototype) {
+              ModelClass = ModelClass[key];
+              break;
+            }
+          }
+        }
+      }
 
-      // Guardar referencia del modelo en el controlador
+      // Verificar si ya existe una instancia del modelo en este controlador
       if (!this.models) {
         this.models = {};
       }
 
+      // Si ya existe una instancia del modelo, devolverla y actualizar el adaptador si es necesario
+      if (this.models[modelName]) {
+        const existingModel = this.models[modelName];
+        
+        // Si se proporciona un adaptador y el modelo no lo tiene, asignarlo
+        if (!existingModel.adapter && options.adapter) {
+          existingModel.setAdapter(options.adapter);
+        }
+        
+        return existingModel;
+      }
+
+      // Crear nueva instancia del modelo con las opciones proporcionadas
+      const modelOptions = { ...options };
+      const modelInstance = new ModelClass(modelOptions);
+
+      // Si el modelo no tiene un adaptador y se proporcionó uno en las opciones, asignarlo
+      if (!modelInstance.adapter && options.adapter) {
+        modelInstance.setAdapter(options.adapter);
+      }
+
+      // Guardar referencia del modelo en el controlador
       this.models[modelName] = modelInstance;
 
       return modelInstance;

package/lib/router/RouteMatcher.js

@@ -33,8 +33,23 @@ class RouteMatcher {
       };
     }
 
-    // Buscar rutas parametrizadas antes que rutas estáticas (prefijos)
-    // Esto evita que rutas estáticas como /api capturen rutas parametrizadas como /api/users/:id
+    // Buscar rutas estáticas que coincidan exactamente antes que rutas parametrizadas
+    // Esto permite que rutas estáticas como /css/style.css tengan prioridad sobre rutas parametrizadas
+    for (const route of routes) {
+      if (route.method !== method) continue;
+      
+      // Verificar si es una ruta estática que coincide exactamente
+      if (route.isStatic && route.path === pathname) {
+        return {
+          route: route,
+          params: {}
+        };
+      }
+    }
+
+    // Buscar rutas parametrizadas
+    // Pero antes de devolver una ruta parametrizada, verificar si hay una ruta estática más específica
+    const parametrizedMatches = [];
     for (const route of routes) {
       if (route.method !== method) continue;
 
@@ -44,14 +59,15 @@ class RouteMatcher {
 
       if (match) {
         const params = this.extractParams(route.path, pathname);
-        return {
+        parametrizedMatches.push({
           route: route,
-          params
-        };
+          params: params
+        });
       }
     }
 
-    // Buscar rutas estáticas (prefijos) - solo después de buscar rutas parametrizadas
+    // Buscar rutas estáticas (prefijos)
+    const staticMatches = [];
     for (const route of routes) {
       if (route.method !== method) continue;
 
@@ -60,14 +76,33 @@ class RouteMatcher {
         // Verificar que sea exactamente el prefijo o que haya una barra después del prefijo
         const remainingPath = pathname.substring(route.path.length);
         if (route.path === '/' || remainingPath === '' || remainingPath.startsWith('/')) {
-          return {
+          staticMatches.push({
             route: route,
-            params: {}
-          };
+            params: {},
+            specificity: route.path.length  // Agregar medida de especificidad basada en longitud del prefijo
+          });
         }
       }
     }
 
+    // Prioridad: si hay rutas estáticas que coinciden, y la solicitud parece ser para un archivo
+    // (tiene extensión), dar prioridad a la ruta estática más específica
+    const hasFileExtension = /\.[^.]+$/.test(pathname);
+
+    if (hasFileExtension && staticMatches.length > 0) {
+      // Ordenar por especificidad (longitud del prefijo, descendente) y tomar la más específica
+      staticMatches.sort((a, b) => b.specificity - a.specificity);
+      // Para solicitudes de archivos, dar prioridad a rutas estáticas más específicas
+      return staticMatches[0]; // Devolver la coincidencia estática más específica
+    } else if (parametrizedMatches.length > 0) {
+      // Para solicitudes sin extensión o rutas API, dar prioridad a rutas parametrizadas
+      return parametrizedMatches[0]; // Devolver la primera coincidencia parametrizada
+    } else if (staticMatches.length > 0) {
+      // Si no hay coincidencias parametrizadas, usar la estática más específica
+      staticMatches.sort((a, b) => b.specificity - a.specificity);
+      return staticMatches[0];
+    }
+
     return null;
   }
 
@@ -86,7 +121,7 @@ class RouteMatcher {
     let escapedPath = '';
     for (let i = 0; i < path.length; i++) {
       const char = path[i];
-      if (char.match(/[.+?^${}()|[\]\\]/) && !(i > 0 && path[i-1] === ':')) {
+      if (char.match(/[.+?^${}()|[\]\\-]/) && !(i > 0 && path[i-1] === ':')) {
         escapedPath += '\\' + char;
       } else {
         escapedPath += char;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "jerkjs",
-  "version": "2.5.2",
-  "description": "JERK Framework v2.5.2 - A comprehensive framework for building secure and scalable APIs with frontend support, sessions, template engine, integration with qbuilderjs, complete MVC architecture with models, and enhanced route loading from directory",
+  "version": "2.5.6",
+  "description": "JERK Framework v2.5.6 - A comprehensive framework for building secure and scalable APIs with frontend support, sessions, template engine, integration with qbuilderjs, complete MVC architecture with models, enhanced route loading from directory, improved model loading system, and fixed routing issues with static and parametrized routes",
   "main": "index.js",
   "scripts": {
     "test": "echo \"Error: no test specified\" && exit 1",
@@ -39,11 +39,19 @@
   },
   "homepage": "https://jerk.page.gd/",
   "dependencies": {
-    "qbuilderjs":"^1.0.1",
-    "bcrypt": "^6.0.0",
+    "bcrypt": "^5.1.1",
+    "cacache": "^20.0.3",
     "jsonwebtoken": "^9.0.3",
+    "make-fetch-happen": "^14.0.3",
     "mariadb": "^3.4.5",
-    "sqlite3": "^5.1.7"
+    "qbuilderjs": "^1.0.1",
+    "sqlite3": "^5.0.2",
+    "tar": "^7.5.7"
+  },
+  "overrides": {
+    "tar": "^7.5.7",
+    "cacache": "^20.0.3",
+    "make-fetch-happen": "^14.0.3"
   },
   "engines": {
     "node": ">=14.0.0"

package/example-directory-loader.js

@@ -1,46 +0,0 @@
-/**
- * Ejemplo de uso del componente RouteDirectoryLoader
- * Este ejemplo demuestra cómo cargar rutas desde un directorio con múltiples archivos JSON
- */
-
-const { APIServer, RouteDirectoryLoader, hooks } = require('./index.js');
-
-// Crear instancia del servidor
-const server = new APIServer({
-  port: 3000,
-  host: 'localhost'
-});
-
-// Crear instancia del cargador de rutas desde directorio
-const routeDirectoryLoader = new RouteDirectoryLoader();
-
-// Registrar hooks para ver eventos
-hooks.addAction('route_duplicate_detected', (data) => {
-  console.log('Evento: Ruta duplicada detectada');
-  console.log('Ruta sobreescrita:', data.overwrittenRoute);
-  console.log('Nueva ruta:', data.newRoute);
-});
-
-hooks.addAction('route_loaded_from_directory', (data) => {
-  console.log('Evento: Ruta cargada desde directorio');
-  console.log('Ruta:', data.route.path, data.route.method);
-  console.log('Archivo:', data.sourceFile);
-});
-
-// Directorio que contiene los archivos JSON de rutas
-const routesDirectory = './routes';
-
-// Cargar rutas desde el directorio
-routeDirectoryLoader.loadRoutesFromDirectory(server, routesDirectory)
-  .then(routes => {
-    console.log(`${routes.length} rutas cargadas exitosamente desde el directorio`);
-    
-    // Iniciar el servidor
-    server.start();
-  })
-  .catch(error => {
-    console.error('Error cargando rutas desde directorio:', error.message);
-  });
-
-// Opcional: Observar cambios en el directorio de rutas
-// routeDirectoryLoader.watchRoutesDirectory(server, routesDirectory);

package/qa/informe_qa_fix_enrutamiento.md

@@ -1,93 +0,0 @@
-# Informe QA: Sistema BlackCoffee con Fix de Enrutamiento
-
-## Resumen Ejecutivo
-
-El sistema BlackCoffee ha sido sometido a pruebas exhaustivas tras la implementación de un fix crítico para el enrutamiento de rutas parametrizadas. El problema que impedía el funcionamiento correcto de las rutas parametrizadas en modo directorio ha sido completamente resuelto.
-
-## Estado Anterior vs Actual
-
-### Antes del Fix
-- **Modo directorio (predeterminado)**: ❌ Rutas parametrizadas no funcionaban
-- **Modo archivo único**: ✅ Funcionaba correctamente
-- **Impacto**: Sistema de colas de Qwen no operativo en modo directorio
-
-### Después del Fix
-- **Modo directorio**: ✅ **TOTALMENTE OPERATIVO**
-- **Modo archivo único**: ✅ **CONTINÚA FUNCIONANDO CORRECTAMENTE**
-- **Impacto**: Sistema de colas de Qwen completamente funcional
-
-## Cambios Implementados
-
-### Arquitectura de Enrutamiento
-- Se implementó un componente especializado `RouteMatcher.js` para encapsular toda la lógica de enrutamiento
-- Se separaron claramente las responsabilidades entre componentes
-- Se centralizó la lógica de matching de rutas
-
-### Prioridad de Enrutamiento (Fix Principal)
-- **Antes**: Rutas estáticas (prefijos) tenían prioridad sobre rutas parametrizadas
-- **Después**: Rutas parametrizadas tienen prioridad sobre rutas estáticas
-- **Nuevo orden**: 1) Rutas exactas → 2) Rutas parametrizadas → 3) Rutas estáticas
-
-## Resultados de las Pruebas
-
-### Rutas Parametrizadas Funcionales
-| Ruta | Método | Controlador | Estado |
-|------|--------|-------------|---------|
-| `/api/products/:id` | GET | `getProductById` | ✅ Funcional |
-| `/api/qwen/result/:id` | GET | `getQwenResult` | ✅ Funcional |
-| `/api/queue/status/:id` | GET | `getJobStatus` | ✅ Funcional |
-| `/api/queue/jobs/:status` | GET | `getJobsByStatus` | ✅ Funcional |
-
-### Sistema de Colas de Qwen
-| Endpoint | Funcionalidad | Estado |
-|----------|---------------|---------|
-| `POST /api/qwen/queue` | Creación de trabajos | ✅ Funcional |
-| `GET /api/qwen/result/:id` | Consulta de resultados | ✅ Funcional |
-| Procesamiento completo | Trabajo de extremo a extremo | ✅ Funcional |
-
-### Validación de Parámetros
-- Extracción correcta de parámetros de rutas (ej: `:id`, `:status`)
-- Valores de parámetros correctamente pasados a controladores
-- Respuestas coherentes según valores de parámetros
-
-## Cobertura de Pruebas
-
-### Modo Directorio (Principal)
-- ✅ Carga de rutas desde múltiples archivos JSON
-- ✅ Registro correcto de todas las rutas
-- ✅ Prioridad correcta de rutas parametrizadas
-- ✅ Funcionamiento de rutas estáticas y dinámicas
-- ✅ Sistema de colas de Qwen operativo
-
-### Modo Archivo Único (Regresión)
-- ✅ Continuidad del funcionamiento previo
-- ✅ No regresiones introducidas
-- ✅ Compatibilidad hacia atrás mantenida
-
-## Métricas de Desempeño
-
-- **Tiempo de respuesta**: Dentro de rangos normales
-- **Consumo de memoria**: Estable
-- **Procesamiento de rutas**: Correcto en ambos modos
-- **Sistema de hooks**: Operativo sin interrupciones
-
-## Validación de Endpoints Registrados
-
-Se verificó que los 19 endpoints se registran correctamente en modo directorio, incluyendo:
-- 4 rutas parametrizadas funcionales
-- 10 rutas estáticas funcionales  
-- 5 rutas estáticas funcionales
-
-## Conclusión
-
-### ✅ ACEPTACIÓN TOTAL
-
-El fix implementado ha resuelto completamente el problema reportado. El sistema BlackCoffee ahora:
-
-1. **Opera correctamente en modo directorio** con todas las rutas parametrizadas funcionando
-2. **Mantiene compatibilidad** con el modo archivo único
-3. **Soporta completamente** el sistema de colas de Qwen
-4. **Sigue principios de arquitectura limpia** con separación de responsabilidades
-5. **No introduce regresiones** en funcionalidades existentes
-
-**Recomendación**: Aprobar el despliegue del fix a producción.

package/utils/find_file_path.sh

@@ -1,36 +0,0 @@
-#!/bin/bash
-
-# Script to find a file's location using 'which' and show its real path with 'ls -ln'
-
-if [ $# -eq 0 ]; then
-    echo "Usage: $0 <filename>"
-    echo "Example: $0 python"
-    exit 1
-fi
-
-filename="$1"
-
-# Find the file using which
-echo "Finding location of '$filename' using 'which':"
-which_result=$(which "$filename")
-
-if [ $? -eq 0 ]; then
-    echo "Found: $which_result"
-    
-    # Get the directory containing the file
-    file_dir=$(dirname "$which_result")
-    file_basename=$(basename "$which_result")
-    
-    # Change to the directory and use ls -ln to show the real path
-    echo ""
-    echo "Using 'ls -ln' to show detailed info:"
-    (cd "$file_dir" && ls -ln "$file_basename")
-    
-    # If the file is a symlink, show the real path using readlink
-    echo ""
-    echo "Real path (using readlink -f):"
-    readlink -f "$which_result"
-else
-    echo "File '$filename' not found in PATH"
-    exit 1
-fi