claude-git-hooks 1.5.2 → 1.5.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.5.4] - 2024-12-22
+
+### Fixed
+
+- 🔧 **Generación de prompt de resolución en modo SonarQube**
+  - Ahora se genera correctamente `claude_resolution_prompt.md` cuando el Quality Gate falla
+  - El archivo se genera cuando hay `blockingIssues`, independiente del tipo de fallo
+  - Corregido el flujo para que siempre muestre los issues críticos antes de generar el prompt
+
+- 🎯 **Unificación de lógica de análisis**
+  - Eliminada lógica duplicada entre modo "clásico" y SonarQube
+  - Ahora siempre se usa modo SonarQube (como se especificó en v1.4.1)
+  - Simplificación del flujo de decisión en el pre-commit hook
+
+### Changed
+
+- 📝 Refactorización del hook pre-commit para mayor claridad y mantenibilidad
+- 🔄 La función `generate_resolution_prompt` ahora se llama consistentemente cuando hay issues bloqueantes
+
+## [1.5.3] - 2025-09-12
+
+### Fixed
+
+- 🔧 Comando `analyze-diff` ahora siempre compara con ramas origin
+  - Si se especifica rama: compara con `origin/rama-especificada`
+  - Si no se especifica: compara con `origin/rama-actual`
+  - Fallback automático a `origin/develop` y luego `origin/main` si no existe la rama origin
+
+### Changed
+
+- 📝 Documentación mejorada con intro atractiva y características principales
+- ⚠️ Aclaración importante sobre uso exclusivo en WSL/Ubuntu (no PowerShell/CMD)
+- 🔗 Enlaces directos a configuración previa en sección de instalación
+
 ## [1.5.2] - 2025-09-11
 
 ### Changed

package/README.md

@@ -1,10 +1,15 @@
 # Pre-commit Hook con Claude CLI
 
-Este directorio contiene un pre-commit hook que utiliza Claude CLI para revisar automáticamente el código antes de cada commit.
-Otras funciones de interés:
+🚀 **Transforma tu flujo de desarrollo con IA**: análisis de código instantáneo, mensajes de commit automáticos y generación de PRs perfectas. Claude revisa tu código antes de cada commit, detecta issues críticos al estilo SonarQube, y genera toda la documentación que necesitas. ¿Lo mejor? Se ejecuta localmente sin contaminar tu CI/CD.
 
-- Generación de mensaje de commit.
-- Generación de información de PR y tests de validación de PR - Utiliza análisis de diferencias entre contenido local y origin.
+## 🎯 Características principales
+
+- 🔍 **Análisis de código pre-commit**: Detecta issues críticos antes de que lleguen al repo
+- 💬 **Mensajes de commit automáticos**: Escribe "auto" y Claude genera el mensaje perfecto
+- 📋 **Generación de PRs**: Título, descripción y tests sugeridos con un solo comando
+- 🎨 **Modo SonarQube**: Métricas de calidad y Quality Gate integrados
+- 🚫 **Skip inteligente**: Excluye código legacy con comentarios SKIP-ANALYSIS
+- 🔄 **Auto-actualización**: Se mantiene actualizado automáticamente
 
 ## 📋 CHEATSHEET
 
@@ -14,10 +19,10 @@ Otras funciones de interés:
 # Commit rápido sin análisis + mensaje automático
 git commit --no-verify -m "auto"
 
-# Analizar diferencias con rama origin actual
+# Analizar diferencias con origin de la rama actual
 claude-hooks analyze-diff
 
-# Analizar diferencias para PR (comparar con develop LOCAL)
+# Analizar diferencias para PR (comparar con origin/develop)
 claude-hooks analyze-diff develop
 
 # Reinstalar durante desarrollo (después de npm link)
@@ -26,8 +31,16 @@ claude-hooks install --force --skip-auth
 
 ### 📦 Instalación y Gestión
 
+⚠️ **IMPORTANTE**: Todo debe ejecutarse desde consola WSL/Ubuntu (no PowerShell ni CMD). Ver [Configuración Previa](#-configuración-previa-importante) antes de comenzar.
+
+Se debe instalar el paquete globalmente, para luego gestionarlo localmente en cada repositorio.
+
 ```bash
-# Instalación inicial
+# En consola WSL/Ubuntu - Instalar globalmente
+npm install -g claude-git-hooks
+
+# Luego en cada proyecto (también desde WSL/Ubuntu)
+cd /tu/proyecto
 claude-hooks install
 
 # Actualizar a última versión
@@ -123,28 +136,7 @@ git commit -m "fix: resolver issues"
 
 ## 🔧 Configuración Previa Importante
 
-### 1. Armonización de Line Endings (EOL) -- Automatizada
-
-Debido a que Claude CLI corre en WSL y el desarrollo puede hacerse en Windows, es crucial configurar correctamente los line endings para evitar que los archivos de hooks se corrompan:
-
-```bash
-# IMPORTANTE: Usar la misma configuración en WSL y Windows
-# Recomendación: usar 'input' en ambos entornos
-
-# En WSL
-git config core.autocrlf input
-
-# En PowerShell/Windows
-git config core.autocrlf input
-
-# Verificar configuración actual
-git config --local core.autocrlf
-git config --global core.autocrlf
-```
-
-**⚠️ ADVERTENCIA**: Si tienes `core.autocrlf = true` en local e `input` en global, esto puede causar que los archivos de los hooks se vacíen. Asegúrate de que ambas configuraciones sean consistentes.
-
-### 2. Credenciales Git en WSL
+### Credenciales Git en WSL
 
 Debes configurar tus credenciales de git nuevamente en WSL:
 
@@ -264,7 +256,7 @@ Si no existe un `.gitignore`, se creará uno nuevo. Si ya existe, las entradas s
 
 ### Características adicionales
 
-- **Generación de información para Pull Requests**: `claude-hooks analyze-diffs {branch}` para comparar rama local con otra rama, propone nombre para rama actual, título y detalles para pull request, da tips para verificar trabajo. Este comando genera automáticamente:
+- **Generación de información para Pull Requests**: `claude-hooks analyze-diff [branch]` para comparar rama local con rama origin, propone nombre para rama actual, título y detalles para pull request, da tips para verificar trabajo. Si se especifica branch, compara con origin/branch. Si no, compara con origin de la rama actual. Este comando genera automáticamente:
   - 📝 Título de PR conciso (máx. 72 caracteres)
   - 📄 Descripción detallada con markdown
   - 🌿 Nombre de rama sugerido (formato: tipo/descripcion)

package/bin/claude-hooks

@@ -808,15 +808,19 @@ function analyzeDiff(args) {
     return;
   }
 
+  // Update remote references
+  execSync('git fetch', { stdio: 'ignore' });
+
   let baseBranch, compareWith, contextDescription;
 
   if (args[0]) {
-    // Case with argument: compare current branch vs specified branch
-    baseBranch = args[0];
+    // Case with argument: compare current branch vs origin/specified-branch
+    const targetBranch = args[0];
+    baseBranch = `origin/${targetBranch}`;
     compareWith = `${baseBranch}...HEAD`;
     contextDescription = `${currentBranch} vs ${baseBranch}`;
     
-    // Check that the base branch exists
+    // Check that the origin branch exists
     try {
       execSync(`git rev-parse --verify ${baseBranch}`, { stdio: 'ignore' });
     } catch (e) {
@@ -824,38 +828,34 @@ function analyzeDiff(args) {
       return;
     }
   } else {
-    // Case without argument: compare local vs remote
+    // Case without argument: compare current branch vs origin/current-branch
+    baseBranch = `origin/${currentBranch}`;
+    compareWith = `${baseBranch}...HEAD`;
+    contextDescription = `${currentBranch} vs ${baseBranch}`;
+    
+    // Check that the origin branch exists
     try {
-      // Get the tracked remote branch
-      const remoteBranch = execSync(`git rev-parse --abbrev-ref ${currentBranch}@{upstream}`, { encoding: 'utf8' }).trim();
-      
-      // Update remote references
-      execSync('git fetch', { stdio: 'ignore' });
-      
-      baseBranch = remoteBranch;
-      compareWith = `HEAD..${remoteBranch}`;
-      contextDescription = `local changes without push in ${currentBranch}`;
-      
-      info(`Comparing local changes vs remote: ${remoteBranch}`);
+      execSync(`git rev-parse --verify ${baseBranch}`, { stdio: 'ignore' });
     } catch (e) {
-      // If there's no remote branch, use develop as fallback
-      baseBranch = 'develop';
+      // Try fallback to origin/develop
+      baseBranch = 'origin/develop';
       compareWith = `${baseBranch}...HEAD`;
-      contextDescription = `${currentBranch} vs ${baseBranch} (sin rama remota)`;
+      contextDescription = `${currentBranch} vs ${baseBranch} (fallback)`;
       
       try {
         execSync(`git rev-parse --verify ${baseBranch}`, { stdio: 'ignore' });
-        warning(`No remote branch configured. Comparing with ${baseBranch}.`);
+        warning(`Branch origin/${currentBranch} does not exist. Using ${baseBranch} as fallback.`);
       } catch (e2) {
-        baseBranch = 'main';
+        // Try fallback to origin/main
+        baseBranch = 'origin/main';
         compareWith = `${baseBranch}...HEAD`;
         contextDescription = `${currentBranch} vs ${baseBranch} (fallback)`;
         
         try {
           execSync(`git rev-parse --verify ${baseBranch}`, { stdio: 'ignore' });
-          warning(`No develop or remote branch. Comparing with main.`);
+          warning(`No origin/develop branch. Using ${baseBranch} as fallback.`);
         } catch (e3) {
-          error('Could not find a valid comparison branch.');
+          error('Could not find a valid comparison branch (tried origin/current, origin/develop, origin/main).');
           return;
         }
       }
@@ -867,12 +867,7 @@ function analyzeDiff(args) {
   // Get modified files
   let diffFiles;
   try {
-    // For local changes without push, invert the comparison
-    if (!args[0] && compareWith.includes('HEAD..')) {
-      diffFiles = execSync(`git diff ${compareWith} --name-only`, { encoding: 'utf8' }).trim();
-    } else {
-      diffFiles = execSync(`git diff ${compareWith} --name-only`, { encoding: 'utf8' }).trim();
-    }
+    diffFiles = execSync(`git diff ${compareWith} --name-only`, { encoding: 'utf8' }).trim();
     
     if (!diffFiles) {
       // Check if there are staged or unstaged changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-git-hooks",
-  "version": "1.5.2",
+  "version": "1.5.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": {
@@ -28,7 +28,6 @@
   "engines": {
     "node": ">=14.0.0"
   },
-  "dependencies": {},
   "preferGlobal": true,
   "files": [
     "bin/",

package/templates/CLAUDE_RESOLUTION_PROMPT.md

@@ -5,7 +5,6 @@ repo:{{REPO_NAME}}
 branch:{{BRANCH_NAME}}
 commit:{{COMMIT_SHA}}
 files:{{FILE_COUNT}}
-mode:{{ANALYSIS_MODE}}
 
 CRITICAL_ISSUES:
 {{BLOCKING_ISSUES}}

package/templates/pre-commit

@@ -332,125 +332,95 @@ if $CLAUDE_CLI < "$PROMPT_FILE" > "$RESPONSE_FILE" 2>&1; then
         BLOCKING_ISSUES=$(echo "$JSON_RESPONSE" | jq -r '.blockingIssues[].description' 2>/dev/null | sed '/^$/d')
     fi
     
+    # Always use SonarQube mode (as per v1.4.1)
     QUALITY_GATE=$(echo "$JSON_RESPONSE" | jq -r '.QUALITY_GATE // ""' 2>/dev/null)
-    
-    if [ -n "$QUALITY_GATE" ] && [ "$QUALITY_GATE" != "null" ]; then
-        # Show SonarQube style results
-        echo
-        echo "╔════════════════════════════════════════════════════════════════════╗"
-        echo "║                     CODE QUALITY ANALYSIS                           ║"
-        echo "╚════════════════════════════════════════════════════════════════════╝"
+
+    # Show SonarQube style results
+    echo
+    echo "╔════════════════════════════════════════════════════════════════════╗"
+    echo "║                     CODE QUALITY ANALYSIS                           ║"
+    echo "╚════════════════════════════════════════════════════════════════════╝"
+    echo
+
+    # Quality Gate Status
+    if [ "$QUALITY_GATE" = "PASSED" ]; then
+        echo -e "${GREEN}✓ Quality Gate: PASSED${NC}"
+    else
+        echo -e "${RED}✗ Quality Gate: FAILED${NC}"
+    fi
+    echo
+
+    # Metrics
+    METRICS=$(echo "$JSON_RESPONSE" | jq -r '.metrics // {}' 2>/dev/null)
+    if [ "$METRICS" != "{}" ] && [ "$METRICS" != "null" ]; then
+        echo "📊 METRICS"
+        echo "├─ Reliability: $(echo "$METRICS" | jq -r '.reliability // "?"' 2>/dev/null)"
+        echo "├─ Security: $(echo "$METRICS" | jq -r '.security // "?"' 2>/dev/null)"
+        echo "├─ Maintainability: $(echo "$METRICS" | jq -r '.maintainability // "?"' 2>/dev/null)"
+        echo "├─ Coverage: $(echo "$METRICS" | jq -r '.coverage // "?"' 2>/dev/null)%"
+        echo "├─ Duplications: $(echo "$METRICS" | jq -r '.duplications // "?"' 2>/dev/null)%"
+        echo "└─ Complexity: $(echo "$METRICS" | jq -r '.complexity // "?"' 2>/dev/null)"
         echo
-        
-        # Quality Gate Status
-        if [ "$QUALITY_GATE" = "PASSED" ]; then
-            echo -e "${GREEN}✓ Quality Gate: PASSED${NC}"
-        else
-            echo -e "${RED}✗ Quality Gate: FAILED${NC}"
-        fi
+    fi
+
+    # Issues Summary
+    ISSUES=$(echo "$JSON_RESPONSE" | jq -r '.issues // {}' 2>/dev/null)
+    if [ "$ISSUES" != "{}" ] && [ "$ISSUES" != "null" ]; then
+        echo "📋 ISSUES SUMMARY"
+        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}"
+        [ "$CRITICAL" -gt 0 ] && echo -e "  ${RED}🟠 Critical: $CRITICAL${NC}"
+        [ "$MAJOR" -gt 0 ] && echo -e "  ${YELLOW}🟡 Major: $MAJOR${NC}"
+        [ "$MINOR" -gt 0 ] && echo "  🔵 Minor: $MINOR"
+        [ "$INFO" -gt 0 ] && echo "  ⚪ Info: $INFO"
         echo
-        
-        # Metrics
-        METRICS=$(echo "$JSON_RESPONSE" | jq -r '.metrics // {}' 2>/dev/null)
-        if [ "$METRICS" != "{}" ] && [ "$METRICS" != "null" ]; then
-            echo "📊 METRICS"
-            echo "├─ Reliability: $(echo "$METRICS" | jq -r '.reliability // "?"' 2>/dev/null)"
-            echo "├─ Security: $(echo "$METRICS" | jq -r '.security // "?"' 2>/dev/null)"
-            echo "├─ Maintainability: $(echo "$METRICS" | jq -r '.maintainability // "?"' 2>/dev/null)"
-            echo "├─ Coverage: $(echo "$METRICS" | jq -r '.coverage // "?"' 2>/dev/null)%"
-            echo "├─ Duplications: $(echo "$METRICS" | jq -r '.duplications // "?"' 2>/dev/null)%"
-            echo "└─ Complexity: $(echo "$METRICS" | jq -r '.complexity // "?"' 2>/dev/null)"
-            echo
-        fi
-        
-        # Issues Summary
-        ISSUES=$(echo "$JSON_RESPONSE" | jq -r '.issues // {}' 2>/dev/null)
-        if [ "$ISSUES" != "{}" ] && [ "$ISSUES" != "null" ]; then
-            echo "📋 ISSUES SUMMARY"
-            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}"
-            [ "$CRITICAL" -gt 0 ] && echo -e "  ${RED}🟠 Critical: $CRITICAL${NC}"
-            [ "$MAJOR" -gt 0 ] && echo -e "  ${YELLOW}🟡 Major: $MAJOR${NC}"
-            [ "$MINOR" -gt 0 ] && echo "  🔵 Minor: $MINOR"
-            [ "$INFO" -gt 0 ] && echo "  ⚪ Info: $INFO"
-            echo
-        fi
-        
-        # Detailed Issues
-        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 '.details[]? | 
-                "[\(.severity)] \(.type) in \(.file):\(.line // "?")\n   \(.message)\n"' 2>/dev/null
-        fi
-        
-        # Security Hotspots
-        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"
+    fi
 
-            echo
-        fi
-        
-        # Check if commit should be blocked
-        if [ "$QUALITY_GATE" = "FAILED" ] || [ "$APPROVED" = "false" ]; then
-            echo
-            error "❌ Commit blocked due to quality gate failure"
-            exit 1
-        fi
-        
+    # Detailed Issues
+    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 '.details[]? |
+            "[\(.severity)] \(.type) in \(.file):\(.line // "?")\n   \(.message)\n"' 2>/dev/null
+    fi
+
+    # Security Hotspots
+    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
-        log "✅ Code analysis completed. Quality gate passed."
-    else
-        # Show results in classic format
+    fi
+
+    # Check if commit should be blocked
+    if [ "$QUALITY_GATE" = "FAILED" ] || [ "$APPROVED" = "false" ]; then
         echo
-        echo "=== REVIEW RESULTS ==="
-        echo "Score: $SCORE/10"
-        
-        if [ -n "$RECOMMENDATIONS" ] && [ "$RECOMMENDATIONS" != "null" ]; then
+        error "❌ Commit blocked due to quality gate failure"
+
+        # Show blocking issues if they exist
+        if [ -n "$BLOCKING_ISSUES" ] && [ "$BLOCKING_ISSUES" != "null" ]; then
             echo
-            echo "=== RECOMMENDATIONS ==="
-            echo "$RECOMMENDATIONS" | sed 's/^/- /'
-        fi
-        
-        # Check if the commit should be blocked
-        if [ "$APPROVED" = "false" ]; then
-            error "❌ Commit rejected due to critical issues"
-            if [ -n "$BLOCKING_ISSUES" ] && [ "$BLOCKING_ISSUES" != "null" ]; then
-                echo
-                echo "=== CRITICAL ISSUES ==="
-                echo "$BLOCKING_ISSUES" | sed 's/^/- /'
-                
-                # Generate AI-friendly resolution prompt
-                generate_resolution_prompt
-            fi
-            exit 1
+            echo "=== CRITICAL ISSUES ==="
+            echo "$BLOCKING_ISSUES" | sed 's/^/- /'
         fi
 
-        # Show additional details if they exist
-        DETAILS=$(echo "$JSON_RESPONSE" | jq -r '.details // null')
-        if [ -n "$DETAILS" ] && [ "$DETAILS" != "null" ]; then
-            echo
-            echo "=== ADDITIONAL DETAILS ==="
-            # If details is a string, print it directly
-            if echo "$DETAILS" | jq -e 'type == "string"' >/dev/null 2>&1; then
-                echo "$DETAILS" | jq -r '.'
-            # If it's an object or array, format it
-            else
-                echo "$DETAILS" | jq '.'
-            fi
+        # Generate AI-friendly resolution prompt if there are blocking issues
+        if [ "$BLOCKING_COUNT" -gt 0 ]; then
+            generate_resolution_prompt
         fi
-        
-        log "✅ Review completed. Commit approved (Score: $SCORE/10)"
+
+        exit 1
     fi
+
+    echo
+    log "✅ Code analysis completed. Quality gate passed."
     
 else
     error "Error executing Claude CLI"