ai-first-cli 1.2.2 → 1.3.0

package/ANALISIS_MEJORAS.md

@@ -1,327 +0,0 @@
-# 📋 ANÁLISIS COMPLETO DE MEJORAS - ai-first-cli
-## Fecha: 2026-03-17
-## Versión analizada: 1.1.5
-
----
-
-## ✅ PUNTO 1 COMPLETADO: Release v1.1.5
-
-- ✅ Versión actualizada a 1.1.5
-- ✅ CHANGELOG.md actualizado
-- ✅ BUGS.md actualizado con nuevos issues
-- ✅ Build exitoso
-- ✅ Commit realizado (f983ba8)
-- ✅ Push a GitHub completado
-- ⏳ NPM publish (pendiente de confirmación del usuario)
-
-**Commit:** https://github.com/julianperezpesce/ai-first/commit/f983ba8
-
----
-
-## 🔍 ANÁLISIS DE PROBLEMAS Y MEJORAS IDENTIFICADAS
-
-### 1. COBERTURA DE TESTING (Alta Prioridad)
-
-**Estado actual:**
-- ✅ 5 proyectos de test funcionando (Express, NestJS, Python CLI, React, Salesforce)
-- ❌ 11+ frameworks sin testing
-
-**Frameworks soportados pero NO testeados:**
-| Framework | Tipo | Prioridad |
-|-----------|------|-----------|
-| Laravel | PHP Web | Alta |
-| Django | Python Web | Alta |
-| Flask/FastAPI | Python API | Alta |
-| Rails | Ruby Web | Media |
-| Spring Boot | Java Enterprise | Media |
-| ASP.NET Core | C# Web | Media |
-| Blazor | .NET Frontend | Baja |
-| Phoenix | Elixir Web | Baja |
-| FastAPI | Python API | Alta |
-| Ruby | Ruby general | Baja |
-
-**Impacto:** Los usuarios de estos frameworks pueden encontrar errores no detectados.
-
-**Solución recomendada:**
-1. Crear un test project para cada framework soportado
-2. Implementar test automatizados que validen:
-   - Generación correcta de símbolos
-   - Detección de entrypoints
-   - Extracción de dependencias
-   - Generación de features/flows
-3. Ejecutar tests en CI/CD antes de cada release
-
----
-
-### 2. ARCHIVOS DUPLICADOS (Media Prioridad)
-
-**Problema detectado:**
-En `ai/` se generan archivos duplicados con diferentes convenciones:
-
-```
-ai/
-├── repo-map.json      ← kebab-case
-├── repo_map.json      ← snake_case
-├── repo_map.md        ← snake_case
-├── module-graph.json  ← kebab-case
-├── symbol-graph.json  ← kebab-case
-└── ...
-```
-
-**Archivos duplicados encontrados:**
-1. `repo-map.json` vs `repo_map.json`
-2. Posiblemente otros casos similares
-
-**Impacto:**
-- Confusión para usuarios (¿cuál usar?)
-- Doble espacio en disco
-- Inconsistencia percibida
-
-**Solución recomendada:**
-1. Elegir UNA convención (recomendado: snake_case)
-2. Mantener solo `repo_map.json`
-3. Eliminar `repo-map.json` o marcarlo como deprecated
-4. Actualizar toda la documentación
-
-**Archivos a modificar:**
-- `src/commands/ai-first.ts` (líneas que generan repo-map)
-- `src/core/aiContextGenerator.ts`
-- `src/core/ccp.ts`
-
----
-
-### 3. INCONSISTENCIAS DE NOMENCLATURA (Media Prioridad)
-
-**Problema detectado:**
-Mezcla de convenciones en archivos generados:
-
-| Archivo | Convención Actual | Convención Recomendada |
-|---------|-------------------|------------------------|
-| `repo-map.json` | kebab-case | `repo_map.json` |
-| `module-graph.json` | kebab-case | `module_graph.json` |
-| `symbol-graph.json` | kebab-case | `symbol_graph.json` |
-| `knowledge-graph.json` | kebab-case | `knowledge_graph.json` |
-| `symbol-references.json` | kebab-case | `symbol_references.json` |
-| `ai_context.md` | snake_case | ✅ Correcto |
-| `tech_stack.md` | snake_case | ✅ Correcto |
-| `repo_map.md` | snake_case | ✅ Correcto |
-
-**Impacto:**
-- Difícil de recordar nombres
-- Inconsistencia visual
-- No sigue estándares de Node.js (que usa snake_case o camelCase)
-
-**Solución recomendada:**
-1. Estandarizar TODO a snake_case
-2. Crear función helper `toSnakeCase()` para consistencia
-3. Actualizar todos los generadores de archivos
-4. Agregar test que valide convenciones
-
-**Ejemplo de cambio:**
-```typescript
-// Antes
-const outputFile = path.join(outputPath, 'module-graph.json');
-
-// Después  
-const outputFile = path.join(outputPath, 'module_graph.json');
-```
-
----
-
-### 4. ORGANIZACIÓN DE CARPETA ai/ (Alta Prioridad)
-
-**Problema detectado:**
-La carpeta `ai/` en el repositorio contiene archivos que NO son generados por el CLI:
-
-```
-ai/ (en el repo, no generado)
-├── ai_context.md          ← Generado por CLI ✅
-├── ai_rules.md            ← Generado por CLI ✅
-├── cache.json             ← Cache (temporal) ⚠️
-├── embeddings.json        ← Muy grande (706KB) ⚠️
-├── git/                   ← Datos git temporales ⚠️
-├── hierarchy.json         ← Temporal ⚠️
-├── index-state.json       ← Estado interno ⚠️
-└── ...
-```
-
-**Problemas:**
-1. Los usuarios no saben qué archivos versionar
-2. Archivos temporales/cache ocupan espacio
-3. Mezcla de archivos de ejemplo con archivos reales
-4. Los test projects tienen archivos `ai/` versionados
-
-**Solución recomendada:**
-
-#### Opción A: Estructura limpia (Recomendada)
-```
-ai/
-├── ai_context.md          ← ✅ Mantener (principal)
-├── ai_rules.md            ← ✅ Mantener (reglas)
-├── architecture.md        ← ✅ Mantener
-├── conventions.md         ← ✅ Mantener
-├── dependencies.json      ← ✅ Mantener
-├── entrypoints.md         ← ✅ Mantener
-├── repo_map.md            ← ✅ Mantener
-├── symbols.json           ← ✅ Mantener
-├── tech_stack.md          ← ✅ Mantener
-├── context/               ← ✅ Mantener (features, flows)
-├── graph/                 ← ✅ Mantener (graphs estáticos)
-├── .cache/                ← 🆕 Nuevo (archivos temporales)
-│   ├── cache.json
-│   ├── index-state.json
-│   └── embeddings.json
-└── .git/                  ← 🆕 Mover desde raíz
-    └── ...
-```
-
-#### Opción B: Separación completa
-```
-repo/
-├── ai/                    ← Solo archivos esenciales
-│   ├── ai_context.md
-│   ├── symbols.json
-│   └── ...
-├── .ai-cache/             ← Temporales (en .gitignore)
-│   └── ...
-└── .ai-git/               ← Datos git (en .gitignore)
-    └── ...
-```
-
-**Archivos a modificar:**
-- `.gitignore` (agregar `ai/.cache/`, `ai/.git/`)
-- `src/commands/ai-first.ts` (cambiar rutas de salida)
-- `src/core/indexer.ts` (cambiar ruta de index.db)
-- Documentación
-
----
-
-### 5. MANEJO DE ERRORES GIT (Baja Prioridad)
-
-**Problema actual:**
-```bash
-$ ai-first git --root test-projects/express-api
-❌ Not a git repository
-EXIT_CODE: 1
-```
-
-**Mejora recomendada:**
-```bash
-$ ai-first git --root test-projects/express-api
-⚠️  No git repository found in test-projects/express-api
-
-💡 To enable git analysis:
-   1. cd test-projects/express-api
-   2. git init
-   3. git add .
-   4. git commit -m "Initial commit"
-
-📚 Learn more: https://ai-first-cli.netlify.app/docs/git-command
-```
-
-**Beneficios:**
-- Mejor UX para nuevos usuarios
-- Reduce tickets de soporte
-- Educación inline
-
----
-
-### 6. MEJORAS ADICIONALES IDENTIFICADAS
-
-#### 6.1 Validación de archivos generados
-**Problema:** No hay validación automática de que los archivos generados sean válidos.
-
-**Solución:** Agregar validación:
-```typescript
-// Después de generar JSON
-const validateJson = (content: string) => {
-  try {
-    JSON.parse(content);
-    return true;
-  } catch {
-    return false;
-  }
-};
-```
-
-#### 6.2 Tamaño de archivos
-**Problema:** `embeddings.json` puede ser muy grande (706KB en ejemplo).
-
-**Solución:** 
-- Agregar compresión opcional
-- O dividir en chunks
-- O guardar en SQLite en lugar de JSON
-
-#### 6.3 Documentación de archivos generados
-**Falta:** Tabla clara que explique cada archivo generado.
-
-**Solución:** Crear `docs/generated-files.md`:
-| Archivo | Propósito | ¿Versionar? | Tamaño típico |
-|---------|-----------|-------------|---------------|
-| ai_context.md | Contexto principal para AI | Sí | ~2KB |
-| symbols.json | Símbolos del proyecto | Sí | ~30KB |
-| index.db | Base de datos SQLite | Opcional | ~45KB |
-| cache.json | Cache temporal | No | Variable |
-
-#### 6.4 Cleanup automático
-**Problema:** Archivos viejos no se limpian automáticamente.
-
-**Solución:** Agregar `ai-first clean`:
-```bash
-ai-first clean              # Limpiar cache
-ai-first clean --all        # Limpiar todo excepto esenciales
-ai-first clean --cache      # Solo cache
-```
-
-#### 6.5 Configuración global
-**Falta:** Archivo de configuración para defaults.
-
-**Solución:** Soportar `ai-first.config.json`:
-```json
-{
-  "outputDir": "./ai",
-  "exclude": ["*.test.js", "node_modules"],
-  "cacheEnabled": true,
-  "maxFileSize": "1MB"
-}
-```
-
----
-
-## 📊 RESUMEN DE PRIORIDADES
-
-| # | Issue | Prioridad | Esfuerzo | Impacto |
-|---|-------|-----------|----------|---------|
-| 1 | Testing de frameworks | 🔴 Alta | Alto | Alto |
-| 2 | Organización de ai/ | 🔴 Alta | Medio | Alto |
-| 3 | Archivos duplicados | 🟡 Media | Bajo | Medio |
-| 4 | Inconsistencias de nombres | 🟡 Media | Medio | Medio |
-| 5 | Manejo de errores git | 🟢 Baja | Bajo | Bajo |
-| 6 | Validación de archivos | 🟡 Media | Medio | Medio |
-| 7 | Documentación de archivos | 🟢 Baja | Bajo | Medio |
-| 8 | Cleanup automático | 🟢 Baja | Medio | Bajo |
-| 9 | Configuración global | 🟢 Baja | Medio | Medio |
-
----
-
-## 🎯 RECOMENDACIONES INMEDIATAS
-
-### Para v1.1.6 (Próximo release):
-1. ✅ Testing completo de Laravel y Django
-2. ✅ Documentar estructura de ai/ claramente
-3. ✅ Agregar mensaje amigable para error de git
-
-### Para v1.2.0 (Release mayor):
-1. Reorganizar estructura de ai/ (breaking change)
-2. Consolidar archivos duplicados (breaking change)
-3. Estandarizar nomenclatura (breaking change)
-
-### Para v2.0.0 (Futuro):
-1. Sistema de plugins
-2. Configuración global
-3. UI web para visualizar graphs
-
----
-
-*Análisis completado: 2026-03-17*
-*Issues registrados en: BUGS.md (sección "Nuevos Issues Identificados")*