claude-git-hooks 1.2.1 → 1.2.4

package/CHANGELOG.md

@@ -5,6 +5,40 @@ Todos los cambios notables en este proyecto se documentarán en este archivo.
 El formato está basado en [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 y este proyecto adhiere a [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.2.4] - 2025-08-22
+
+### Fixed
+- 🐛 Corregido el análisis de SonarQube para mostrar correctamente el formato detallado
+- 📊 Arreglado el parsing de JSON para buscar `QUALITY_GATE` en mayúsculas según las pautas
+- 🔧 Actualizado el mapeo de campos JSON para coincidir con la estructura esperada
+
+### Changed
+- 📈 Añadidas métricas de coverage, duplications y complexity al formato SonarQube
+- 📝 Actualizado el archivo de pautas SonarQube para incluir todas las métricas
+
+## [1.2.3] - 2025-08-22
+
+### Added
+- 🚀 Actualización automática de `.gitignore` durante la instalación
+- 📝 Claude Hooks ahora agrega automáticamente las entradas necesarias a `.gitignore`
+- 🔍 El comando `status` ahora muestra el estado de las entradas en `.gitignore`
+
+### Changed
+- 🎯 Mejorado el proceso de instalación para ser más completo y automatizado
+- 📊 El comando `status` ahora proporciona información más detallada sobre la configuración
+
+## [1.2.2] - 2025-08-22
+
+### Fixed
+- 🐛 Corregido problema donde los archivos markdown de pautas no se instalaban correctamente desde npm
+- 📁 Los archivos de pautas ahora se instalan en el directorio `.claude/` en lugar de la raíz del proyecto
+- 🔧 Actualizado el hook pre-commit para buscar los archivos de pautas en la nueva ubicación `.claude/`
+
+### Changed
+- 📂 Los archivos `CLAUDE_PRE_COMMIT.md` y `CLAUDE_PRE_COMMIT_SONAR.md` ahora se almacenan en `.claude/`
+- 🎯 Mejorada la organización del proyecto manteniendo los archivos de configuración separados del código fuente
+- 📝 Actualizada la documentación para reflejar la nueva estructura de directorios
+
 ## [1.2.1] - 2024-07-24
 
 ### Fixed

package/README.md

@@ -8,8 +8,8 @@ Este directorio contiene un pre-commit hook que utiliza Claude CLI para revisar
 - **`pre-commit`** - Hook principal para análisis de código (con auto-actualización)
 - **`prepare-commit-msg`** - Hook para generar mensajes de commit automáticos
 - **`.gitattributes`** - Configuración para mantener line endings correctos
-- **`CLAUDE_PRE_COMMIT.md`** - Pautas de evaluación formato estándar
-- **`CLAUDE_PRE_COMMIT_SONAR.md`** - Pautas de evaluación formato SonarQube
+- **`.claude/CLAUDE_PRE_COMMIT.md`** - Pautas de evaluación formato estándar
+- **`.claude/CLAUDE_PRE_COMMIT_SONAR.md`** - Pautas de evaluación formato SonarQube
 
 ## 🔧 Configuración Previa Importante
 
@@ -69,6 +69,7 @@ El comando `claude-hooks install` ahora incluye:
 - ✅ Configuración de Git (line endings WSL/Windows)
 - ✅ Verificación de autenticación Claude con entretenimiento
 - ✅ Instalación de hooks y archivos de pautas
+- ✅ Actualización automática de .gitignore con archivos de Claude
 
 #### Añadir como Dependencia de Desarrollo
 
@@ -99,6 +100,37 @@ Luego añade esto a tu `package.json`:
 - `pre-commit`: Análisis de código con Claude (solo archivos Java/config)
 - `prepare-commit-msg`: Generación automática de mensajes de commit
 
+## 📁 Gestión de Archivos
+
+### Archivos creados durante la instalación
+
+El comando `claude-hooks install` crea los siguientes archivos y directorios:
+
+1. **`.git/hooks/pre-commit`** - Hook de análisis de código
+2. **`.git/hooks/prepare-commit-msg`** - Hook de generación de mensajes
+3. **`.claude/`** - Directorio para archivos de configuración
+   - `CLAUDE_PRE_COMMIT.md` - Pautas de evaluación estándar
+   - `CLAUDE_PRE_COMMIT_SONAR.md` - Pautas de evaluación SonarQube
+4. **`.claude-analysis-mode`** - Archivo de preferencia de modo (creado al primer uso)
+
+### Actualización automática de .gitignore
+
+Durante la instalación, Claude Hooks actualiza automáticamente tu `.gitignore` para incluir:
+
+```gitignore
+# Claude Git Hooks
+.claude/
+.claude-analysis-mode
+debug-claude-response.json
+```
+
+Esto asegura que:
+- Los archivos de configuración de Claude específicos del proyecto no se suban al repositorio
+- Los archivos de debug temporales se ignoren
+- Cada desarrollador pueda tener sus propias preferencias de análisis
+
+Si no existe un `.gitignore`, se creará uno nuevo. Si ya existe, las entradas se agregarán al final solo si no están presentes.
+
 ## 🎯 Funcionamiento
 
 ### Hook pre-commit (Análisis de código)
@@ -112,7 +144,7 @@ Luego añade esto a tu `package.json`:
    - **Estándar**: Formato clásico con score y recomendaciones detalladas
    - **SonarQube**: Simula salida de SonarQube con Quality Gate, métricas y clasificación de issues
 4. **Construye prompt inteligente**:
-   - Lee las pautas desde `CLAUDE_PRE_COMMIT.md` o `CLAUDE_PRE_COMMIT_SONAR.md`
+   - Lee las pautas desde `.claude/CLAUDE_PRE_COMMIT.md` o `.claude/CLAUDE_PRE_COMMIT_SONAR.md`
    - Incluye el diff completo para archivos nuevos
    - Muestra solo cambios para archivos existentes
 5. **Envía a Claude CLI para revisión**
@@ -284,8 +316,8 @@ En el archivo `pre-commit`:
 - **`MAX_FILE_SIZE`**: Tamaño máximo de archivo a analizar (default: 100KB)
 - **`MAX_FILES`**: Número máximo de archivos por commit (default: 10)
 - **`CLAUDE_CLI`**: Comando de Claude CLI (default: "claude")
-- **`GUIDELINES_FILE`**: Archivo de pautas para modo estándar (default: "CLAUDE_PRE_COMMIT.md")
-- **`GUIDELINES_FILE_SONAR`**: Archivo de pautas para modo SonarQube (default: "CLAUDE_PRE_COMMIT_SONAR.md")
+- **`GUIDELINES_FILE`**: Archivo de pautas para modo estándar (default: ".claude/CLAUDE_PRE_COMMIT.md")
+- **`GUIDELINES_FILE_SONAR`**: Archivo de pautas para modo SonarQube (default: ".claude/CLAUDE_PRE_COMMIT_SONAR.md")
 
 En el archivo `prepare-commit-msg`:
 
@@ -351,7 +383,11 @@ Este problema suele ocurrir por conflictos de configuración de line endings:
 
 ## 📝 Personalización
 
-Puedes modificar las pautas de evaluación editando `CLAUDE_PRE_COMMIT.md` para adaptarlas a los estándares de tu equipo.
+Puedes modificar las pautas de evaluación editando los archivos en el directorio `.claude/`:
+- `.claude/CLAUDE_PRE_COMMIT.md` - Para el modo estándar
+- `.claude/CLAUDE_PRE_COMMIT_SONAR.md` - Para el modo SonarQube
+
+Estos archivos son específicos de tu proyecto y puedes adaptarlos a los estándares de tu equipo.
 
 ## 🔄 Arquitectura del Sistema
 
@@ -394,6 +430,9 @@ claude-git-hooks/
 │   ├── prepare-commit-msg    # Hook de generación de mensajes
 │   ├── CLAUDE_PRE_COMMIT.md  # Pautas estándar
 │   └── CLAUDE_PRE_COMMIT_SONAR.md # Pautas SonarQube
+├── .claude/                  # Directorio creado en el repo del usuario
+│   ├── CLAUDE_PRE_COMMIT.md  # Pautas copiadas aquí durante la instalación
+│   └── CLAUDE_PRE_COMMIT_SONAR.md
 ├── package.json              # Configuración NPM
 ├── README.md                 # Este archivo
 ├── CHANGELOG.md              # Historial de versiones

package/bin/claude-hooks

@@ -308,14 +308,22 @@ async function install(args) {
     success(`${hook} instalado`);
   });
 
-  // Copiar archivos de pautas si no existen
+  // Crear directorio .claude si no existe
+  const claudeDir = '.claude';
+  if (!fs.existsSync(claudeDir)) {
+    fs.mkdirSync(claudeDir, { recursive: true });
+    success('.claude directory created');
+  }
+
+  // Copiar archivos de pautas a .claude si no existen
   const guidelines = ['CLAUDE_PRE_COMMIT.md', 'CLAUDE_PRE_COMMIT_SONAR.md'];
   guidelines.forEach(guideline => {
-    if (!fs.existsSync(guideline)) {
+    const destPath = path.join(claudeDir, guideline);
+    if (!fs.existsSync(destPath)) {
       const sourcePath = path.join(templatesPath, guideline);
       if (fs.existsSync(sourcePath)) {
-        fs.copyFileSync(sourcePath, guideline);
-        success(`${guideline} creado`);
+        fs.copyFileSync(sourcePath, destPath);
+        success(`${guideline} creado en .claude/`);
       }
     }
   });
@@ -323,6 +331,9 @@ async function install(args) {
   // Configurar Git
   configureGit();
 
+  // Actualizar .gitignore
+  updateGitignore();
+
   success('¡Claude Git Hooks instalado exitosamente! 🎉');
   console.log('\nUso:');
   console.log('  git commit -m "auto"     # Genera mensaje automáticamente');
@@ -503,6 +514,79 @@ async function checkClaudeAuth() {
   }
 }
 
+// Actualizar .gitignore con las entradas de Claude
+function updateGitignore() {
+  info('Actualizando .gitignore...');
+  
+  const gitignorePath = '.gitignore';
+  const claudeEntries = [
+    '# Claude Git Hooks',
+    '.claude/',
+    '.claude-analysis-mode',
+    'debug-claude-response.json'
+  ];
+  
+  let gitignoreContent = '';
+  let fileExists = false;
+  
+  // Leer .gitignore existente si existe
+  if (fs.existsSync(gitignorePath)) {
+    gitignoreContent = fs.readFileSync(gitignorePath, 'utf8');
+    fileExists = true;
+  }
+  
+  // Verificar qué entradas faltan
+  const missingEntries = [];
+  claudeEntries.forEach(entry => {
+    if (entry.startsWith('#')) {
+      // Para comentarios, verificar si ya existe algún comentario de Claude
+      if (!gitignoreContent.includes('# Claude')) {
+        missingEntries.push(entry);
+      }
+    } else {
+      // Para entradas normales, verificar si ya están presentes
+      const regex = new RegExp(`^${entry.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')}$`, 'm');
+      if (!regex.test(gitignoreContent)) {
+        missingEntries.push(entry);
+      }
+    }
+  });
+  
+  // Si hay entradas faltantes, agregarlas
+  if (missingEntries.length > 0) {
+    // Asegurar que hay una nueva línea al final si el archivo existe y no está vacío
+    if (fileExists && gitignoreContent.length > 0 && !gitignoreContent.endsWith('\n')) {
+      gitignoreContent += '\n';
+    }
+    
+    // Si el archivo no está vacío, agregar una línea en blanco antes
+    if (gitignoreContent.length > 0) {
+      gitignoreContent += '\n';
+    }
+    
+    // Agregar las entradas faltantes
+    gitignoreContent += missingEntries.join('\n') + '\n';
+    
+    // Escribir el archivo actualizado
+    fs.writeFileSync(gitignorePath, gitignoreContent);
+    
+    if (fileExists) {
+      success('.gitignore actualizado con entradas de Claude');
+    } else {
+      success('.gitignore creado con entradas de Claude');
+    }
+    
+    // Mostrar qué se agregó
+    missingEntries.forEach(entry => {
+      if (!entry.startsWith('#')) {
+        info(`  + ${entry}`);
+      }
+    });
+  } else {
+    info('.gitignore ya contiene todas las entradas necesarias');
+  }
+}
+
 // Configurar Git (line endings, etc.)
 function configureGit() {
   info('Configurando Git...');
@@ -631,12 +715,40 @@ function status() {
   console.log('\nArchivos de pautas:');
   const guidelines = ['CLAUDE_PRE_COMMIT.md', 'CLAUDE_PRE_COMMIT_SONAR.md'];
   guidelines.forEach(guideline => {
-    if (fs.existsSync(guideline)) {
-      success(`${guideline}: presente`);
+    const claudePath = path.join('.claude', guideline);
+    if (fs.existsSync(claudePath)) {
+      success(`${guideline}: presente en .claude/`);
+    } else if (fs.existsSync(guideline)) {
+      warning(`${guideline}: presente en raíz (debería estar en .claude/)`);
     } else {
       warning(`${guideline}: faltante`);
     }
   });
+
+  // Verificar entradas en .gitignore
+  console.log('\n.gitignore:');
+  const gitignorePath = '.gitignore';
+  if (fs.existsSync(gitignorePath)) {
+    const gitignoreContent = fs.readFileSync(gitignorePath, 'utf8');
+    const claudeIgnores = ['.claude/', '.claude-analysis-mode', 'debug-claude-response.json'];
+    let allPresent = true;
+    
+    claudeIgnores.forEach(entry => {
+      const regex = new RegExp(`^${entry.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')}$`, 'm');
+      if (regex.test(gitignoreContent)) {
+        success(`${entry}: incluido`);
+      } else {
+        warning(`${entry}: faltante`);
+        allPresent = false;
+      }
+    });
+    
+    if (!allPresent) {
+      info('\nEjecuta "claude-hooks install" para actualizar .gitignore');
+    }
+  } else {
+    warning('.gitignore no existe');
+  }
 }
 
 // Comando set-mode

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-git-hooks",
-  "version": "1.2.1",
+  "version": "1.2.4",
   "description": "Git hooks con Claude CLI para análisis de código y generación automática de mensajes de commit",
   "main": "lib/index.js",
   "bin": {

package/templates/CLAUDE_PRE_COMMIT.md

@@ -44,7 +44,8 @@ Responde ÚNICAMENTE con un JSON válido siguiendo esta estructura:
   ],
   "blockingIssues": [
     "Problema crítico 1 (si existe)"
-  ]
+  ],
+  "details": "Información adicional detallada (opcional)"
 }
 ```
 

package/templates/CLAUDE_PRE_COMMIT_SONAR.md

@@ -23,6 +23,11 @@ Evaluar el código modificado siguiendo métricas similares a SonarQube para ase
 - Duplicación de código
 - Deuda técnica
 
+### Métricas Adicionales
+- **Coverage**: Porcentaje estimado de cobertura de pruebas (0-100)
+- **Duplications**: Porcentaje estimado de código duplicado (0-100)
+- **Complexity**: Complejidad ciclomática promedio del código
+
 ## Clasificación de Issues
 
 ### Severidad
@@ -42,7 +47,10 @@ Responde ÚNICAMENTE con un JSON válido siguiendo esta estructura:
   "metrics": {
     "reliability": "A/B/C/D/E",
     "security": "A/B/C/D/E",
-    "maintainability": "A/B/C/D/E"
+    "maintainability": "A/B/C/D/E",
+    "coverage": 75,
+    "duplications": 5,
+    "complexity": 15
   },
   "issues": {
     "blocker": 0,
@@ -60,7 +68,9 @@ Responde ÚNICAMENTE con un JSON válido siguiendo esta estructura:
       "line": 42
     }
   ],
-  "securityHotspots": 0
+  "securityHotspots": 0,
+  "approved": true/false,
+  "blockingIssues": ["Lista de problemas bloqueantes si existen"]
 }
 ```
 

package/templates/pre-commit

@@ -64,10 +64,10 @@ if [ "$ANALYSIS_MODE" != "standard" ] && [ "$ANALYSIS_MODE" != "sonar" ]; then
     
     if [ "$REPLY" = "2" ]; then
         ANALYSIS_MODE="sonar"
-        GUIDELINES_FILE="CLAUDE_PRE_COMMIT_SONAR.md"
+        GUIDELINES_FILE=".claude/CLAUDE_PRE_COMMIT_SONAR.md"
     else
         ANALYSIS_MODE="standard"
-        GUIDELINES_FILE="CLAUDE_PRE_COMMIT.md"
+        GUIDELINES_FILE=".claude/CLAUDE_PRE_COMMIT.md"
     fi
     
     # Guardar preferencia para futuros commits
@@ -78,10 +78,10 @@ if [ "$ANALYSIS_MODE" != "standard" ] && [ "$ANALYSIS_MODE" != "sonar" ]; then
 else
     # Usar modo guardado
     if [ "$ANALYSIS_MODE" = "sonar" ]; then
-        GUIDELINES_FILE="CLAUDE_PRE_COMMIT_SONAR.md"
+        GUIDELINES_FILE=".claude/CLAUDE_PRE_COMMIT_SONAR.md"
         log "Usando modo de análisis: SonarQube"
     else
-        GUIDELINES_FILE="CLAUDE_PRE_COMMIT.md"
+        GUIDELINES_FILE=".claude/CLAUDE_PRE_COMMIT.md"
         log "Usando modo de análisis: Estándar"
     fi
 fi
@@ -233,7 +233,7 @@ if $CLAUDE_CLI < "$PROMPT_FILE" > "$RESPONSE_FILE" 2>&1; then
     BLOCKING_ISSUES=$(echo "$JSON_RESPONSE" | jq -r '.blockingIssues[]?' 2>/dev/null | sed '/^$/d')
     
     # Verificar si estamos en modo SonarQube
-    QUALITY_GATE=$(echo "$JSON_RESPONSE" | jq -r '.qualityGate // ""' 2>/dev/null)
+    QUALITY_GATE=$(echo "$JSON_RESPONSE" | jq -r '.QUALITY_GATE // ""' 2>/dev/null)
     
     if [ "$ANALYSIS_MODE" = "sonar" ] && [ -n "$QUALITY_GATE" ] && [ "$QUALITY_GATE" != "null" ]; then
         # Mostrar resultados estilo SonarQube
@@ -265,15 +265,15 @@ if $CLAUDE_CLI < "$PROMPT_FILE" > "$RESPONSE_FILE" 2>&1; then
         fi
         
         # Issues Summary
-        SUMMARY=$(echo "$JSON_RESPONSE" | jq -r '.summary // {}' 2>/dev/null)
-        if [ "$SUMMARY" != "{}" ] && [ "$SUMMARY" != "null" ]; then
+        ISSUES=$(echo "$JSON_RESPONSE" | jq -r '.issues // {}' 2>/dev/null)
+        if [ "$ISSUES" != "{}" ] && [ "$ISSUES" != "null" ]; then
             echo "📋 ISSUES SUMMARY"
-            TOTAL=$(echo "$SUMMARY" | jq -r '.total // 0' 2>/dev/null)
-            BLOCKER=$(echo "$SUMMARY" | jq -r '.blocker // 0' 2>/dev/null)
-            CRITICAL=$(echo "$SUMMARY" | jq -r '.critical // 0' 2>/dev/null)
-            MAJOR=$(echo "$SUMMARY" | jq -r '.major // 0' 2>/dev/null)
-            MINOR=$(echo "$SUMMARY" | jq -r '.minor // 0' 2>/dev/null)
-            INFO=$(echo "$SUMMARY" | jq -r '.info // 0' 2>/dev/null)
+            BLOCKER=$(echo "$ISSUES" | jq -r '.blocker // 0' 2>/dev/null)
+            CRITICAL=$(echo "$ISSUES" | jq -r '.critical // 0' 2>/dev/null)
+            MAJOR=$(echo "$ISSUES" | jq -r '.major // 0' 2>/dev/null)
+            MINOR=$(echo "$ISSUES" | jq -r '.minor // 0' 2>/dev/null)
+            INFO=$(echo "$ISSUES" | jq -r '.info // 0' 2>/dev/null)
+            TOTAL=$((BLOCKER + CRITICAL + MAJOR + MINOR + INFO))
             
             echo "Total: $TOTAL issues found"
             [ "$BLOCKER" -gt 0 ] && echo -e "  ${RED}🔴 Blocker: $BLOCKER${NC}"
@@ -285,18 +285,19 @@ if $CLAUDE_CLI < "$PROMPT_FILE" > "$RESPONSE_FILE" 2>&1; then
         fi
         
         # Detailed Issues
-        ISSUES_COUNT=$(echo "$JSON_RESPONSE" | jq -r '.issues | length' 2>/dev/null)
-        if [ "$ISSUES_COUNT" -gt 0 ] 2>/dev/null; then
+        DETAILS_COUNT=$(echo "$JSON_RESPONSE" | jq -r '.details | length' 2>/dev/null)
+        if [ "$DETAILS_COUNT" -gt 0 ] 2>/dev/null; then
             echo "🔍 DETAILED ISSUES"
-            echo "$JSON_RESPONSE" | jq -r '.issues[]? | 
-                "[\(.severity)] \(.type) in \(.file):\(.line // "?")\n   \(.message)\n   Rule: \(.rule // "N/A") | Effort: \(.effort // "?")\n"' 2>/dev/null
+            echo "$JSON_RESPONSE" | jq -r '.details[]? | 
+                "[\(.severity)] \(.type) in \(.file):\(.line // "?")\n   \(.message)\n"' 2>/dev/null
         fi
         
         # Security Hotspots
-        HOTSPOTS_COUNT=$(echo "$JSON_RESPONSE" | jq -r '.hotspots | length' 2>/dev/null)
-        if [ "$HOTSPOTS_COUNT" -gt 0 ] 2>/dev/null; then
-            echo "🔥 SECURITY HOTSPOTS"
-            echo "$JSON_RESPONSE" | jq -r '.hotspots[]? | "  • \(.file): \(.message)"' 2>/dev/null
+        HOTSPOTS=$(echo "$JSON_RESPONSE" | jq -r '.securityHotspots // 0' 2>/dev/null)
+        if [ "$HOTSPOTS" -gt 0 ] 2>/dev/null; then
+            echo "🔥 SECURITY HOTSPOTS: $HOTSPOTS found"
+            echo "   Review security-sensitive code carefully"
+
             echo
         fi
         
@@ -332,11 +333,18 @@ if $CLAUDE_CLI < "$PROMPT_FILE" > "$RESPONSE_FILE" 2>&1; then
             exit 1
         fi
 
-        DETAILS=$(echo "$JSON_RESPONSE" | jq -r '.details // {}')
-        if [ -n "$DETAILS" ] && [ "$DETAILS" != "{}" ] && [ "$DETAILS" != "null" ]; then
+        # Mostrar detalles adicionales si existen
+        DETAILS=$(echo "$JSON_RESPONSE" | jq -r '.details // null')
+        if [ -n "$DETAILS" ] && [ "$DETAILS" != "null" ]; then
             echo
-            echo "=== DETALLES POR CATEGORÍA ==="
-            echo "$DETAILS" | jq
+            echo "=== DETALLES ADICIONALES ==="
+            # Si details es un string, imprimirlo directamente
+            if echo "$DETAILS" | jq -e 'type == "string"' >/dev/null 2>&1; then
+                echo "$DETAILS" | jq -r '.'
+            # Si es un objeto o array, formatearlo
+            else
+                echo "$DETAILS" | jq '.'
+            fi
         fi
         
         log "✅ Revisión completada. Commit aprobado (Score: $SCORE/10)"