claude-git-hooks 1.4.2 → 1.5.0

package/CHANGELOG.md

@@ -5,6 +5,32 @@ 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.0] - 2025-09-10
+
+### Added
+- 🔍 Nuevo comando `claude-hooks analyze-diff [base]` para análisis de diferencias entre ramas
+  - Genera título de PR conciso (máx. 72 caracteres)
+  - Crea descripción detallada del PR con markdown
+  - Sugiere nombre de rama siguiendo formato tipo/descripcion
+  - Identifica tipo de cambio (feature/fix/refactor/docs/test/chore)
+  - Detecta breaking changes
+  - Proporciona notas de testing recomendado
+  - Guarda resultados en `.claude-pr-analysis.json`
+
+### Changed
+- 🎯 Modo SonarQube ahora es el predeterminado (no hay selección interactiva)
+- ⚡ Eliminada lógica de selección de modo del pre-commit hook
+- 🗑️ Removidas referencias a guardado de preferencias `.claude-analysis-mode`
+
+### Fixed
+- 🐛 Eliminado prompt interactivo incompatible con algunas consolas
+- 🔧 Pre-commit ahora va directo al análisis sin solicitar input del usuario
+
+### Removed
+- 🗑️ Eliminada selección interactiva de modo de análisis
+- 🗑️ Removido comando `set-mode` (ya no es necesario)
+- 🗑️ Eliminada variable de entorno `CLAUDE_ANALYSIS_MODE`
+
 ## [1.4.2] - 2025-09-08
 
 ### Added

package/README.md

@@ -201,7 +201,7 @@ Cuando se detectan problemas críticos:
 - **Auto-actualización**: Verificación automática de versiones antes de cada commit con prompt interactivo para actualizar
 - **Comando update**: `claude-hooks update` para actualizar manualmente a la última versión
 - **Modo debug**: `DEBUG=1 git commit` guarda respuestas en `debug-claude-response.json`
-- **Configuración persistente**: Guarda preferencias en `.claude-analysis-mode`
+- **Análisis de PR**: Nuevo comando `analyze-diff` para generar información de Pull Requests
 - **Validación de dependencias**: Verifica que Claude CLI esté autenticado antes de ejecutar
 
 ## 🛠️ Uso
@@ -252,55 +252,27 @@ public void methodWithKnownIssues() {
 
 **Nota**: El código entre comentarios `SKIP-ANALYSIS` no será enviado a Claude para su análisis.
 
-### Modos de análisis 📊
+### Análisis de diferencias entre ramas 🔍
 
-Puedes elegir entre dos formatos de análisis:
-
-#### 🎯 Configurar Modo (Recomendado)
+Nueva funcionalidad para analizar cambios y generar información para Pull Requests:
 
 ```bash
-# Cambiar a modo SonarQube (métricas y quality gate)
-claude-hooks set-mode sonar
-
-# Cambiar a modo estándar (score y recomendaciones)
-claude-hooks set-mode standard
+# Analizar diferencias con la rama main
+claude-hooks analyze-diff
 
-# Ver modo actual
-claude-hooks status
+# Analizar diferencias con otra rama base
+claude-hooks analyze-diff develop
 ```
 
-#### 🔧 Variable de Entorno (Temporal)
-
-```bash
-# Sobrescribir temporalmente el modo configurado
-export CLAUDE_ANALYSIS_MODE=sonar
-git commit -m "mensaje"
-
-# Volver al modo configurado
-unset CLAUDE_ANALYSIS_MODE
-```
-
-#### 📋 Diferencias entre Modos
-
-**Modo Estándar**:
+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)
+- 📋 Tipo de cambio (feature/fix/refactor/docs/test/chore)
+- ⚠️ Indicador de breaking changes
+- 🧪 Notas de testing recomendado
 
-- Análisis con puntuación del 1-10
-- Recomendaciones detalladas por categoría
-- Formato tradicional fácil de leer
-
-**Modo SonarQube**:
-
-- Quality Gate (PASSED/FAILED)
-- Métricas: Reliability, Security, Maintainability
-- Issues clasificados por severidad (Blocker, Critical, Major, Minor, Info)
-- Security hotspots
-
-#### 🔄 Cambio de Modo Interactivo
-
-```bash
-# Sin parámetros muestra ayuda interactiva
-claude-hooks set-mode
-```
+Los resultados se guardan en `.claude-pr-analysis.json` para fácil acceso.
 
 ### Modo debug
 
@@ -440,12 +412,6 @@ claude-hooks status
 
 ### Variables de entorno disponibles
 
-- **`CLAUDE_ANALYSIS_MODE`**: Selecciona el modo de análisis
-
-  - Valores: `"standard"` o `"sonar"`
-  - Ejemplo: `CLAUDE_ANALYSIS_MODE=sonar git commit -m "mensaje"`
-  - Sobrescribe la preferencia guardada en `.claude-analysis-mode`
-
 - **`DEBUG`**: Activa el modo debug
   - Valores: `1` para activar
   - Ejemplo: `DEBUG=1 git commit -m "mensaje"`

package/bin/claude-hooks

@@ -640,7 +640,8 @@ function updateGitignore() {
     '.claude/',
     '.claude-analysis-mode',
     'debug-claude-response.json',
-    'claude_resolution_prompt.md'
+    'claude_resolution_prompt.md',
+    '.claude-pr-analysis.json',
   ];
   
   let gitignoreContent = '';
@@ -794,6 +795,258 @@ function disable(hookName) {
   });
 }
 
+// Comando analyze-diff
+function analyzeDiff(args) {
+  if (!checkGitRepo()) {
+    error('No estás en un repositorio Git.');
+    return;
+  }
+
+  const currentBranch = execSync('git branch --show-current', { encoding: 'utf8' }).trim();
+  
+  if (!currentBranch) {
+    error('No estás en una rama válida.');
+    return;
+  }
+
+  let baseBranch, compareWith, contextDescription;
+
+  if (args[0]) {
+    // Caso con argumento: comparar rama actual vs rama especificada
+    baseBranch = args[0];
+    compareWith = `${baseBranch}...HEAD`;
+    contextDescription = `${currentBranch} vs ${baseBranch}`;
+    
+    // Verificar que la rama base existe
+    try {
+      execSync(`git rev-parse --verify ${baseBranch}`, { stdio: 'ignore' });
+    } catch (e) {
+      error(`La rama ${baseBranch} no existe.`);
+      return;
+    }
+  } else {
+    // Caso sin argumento: comparar local vs remoto
+    try {
+      // Obtener la rama remota tracked
+      const remoteBranch = execSync(`git rev-parse --abbrev-ref ${currentBranch}@{upstream}`, { encoding: 'utf8' }).trim();
+      
+      // Actualizar referencias remotas
+      execSync('git fetch', { stdio: 'ignore' });
+      
+      baseBranch = remoteBranch;
+      compareWith = `HEAD..${remoteBranch}`;
+      contextDescription = `cambios locales sin pushear en ${currentBranch}`;
+      
+      info(`Comparando cambios locales vs remoto: ${remoteBranch}`);
+    } catch (e) {
+      // Si no hay rama remota, usar develop como fallback
+      baseBranch = 'develop';
+      compareWith = `${baseBranch}...HEAD`;
+      contextDescription = `${currentBranch} vs ${baseBranch} (sin rama remota)`;
+      
+      try {
+        execSync(`git rev-parse --verify ${baseBranch}`, { stdio: 'ignore' });
+        warning(`No hay rama remota configurada. Comparando con ${baseBranch}.`);
+      } catch (e2) {
+        baseBranch = 'main';
+        compareWith = `${baseBranch}...HEAD`;
+        contextDescription = `${currentBranch} vs ${baseBranch} (fallback)`;
+        
+        try {
+          execSync(`git rev-parse --verify ${baseBranch}`, { stdio: 'ignore' });
+          warning(`No hay develop ni rama remota. Comparando con main.`);
+        } catch (e3) {
+          error('No se pudo encontrar una rama de comparación válida.');
+          return;
+        }
+      }
+    }
+  }
+
+  info(`Analizando: ${contextDescription}...`);
+
+  // Obtener los archivos modificados
+  let diffFiles;
+  try {
+    // Para cambios locales sin pushear, invertir la comparación
+    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();
+    }
+    
+    if (!diffFiles) {
+      // Verificar si hay cambios staged o unstaged
+      const stagedFiles = execSync('git diff --cached --name-only', { encoding: 'utf8' }).trim();
+      const unstagedFiles = execSync('git diff --name-only', { encoding: 'utf8' }).trim();
+      
+      if (stagedFiles || unstagedFiles) {
+        warning('No hay diferencias con el remoto, pero tienes cambios locales sin commitear.');
+        console.log('Cambios staged:', stagedFiles || 'ninguno');
+        console.log('Cambios unstaged:', unstagedFiles || 'ninguno');
+      } else {
+        success('✅ No hay diferencias. Tu rama está sincronizada.');
+      }
+      return;
+    }
+  } catch (e) {
+    error('Error al obtener las diferencias: ' + e.message);
+    return;
+  }
+
+  // Obtener el diff completo
+  let fullDiff, commits;
+  try {
+    if (!args[0] && compareWith.includes('HEAD..')) {
+      fullDiff = execSync(`git diff ${compareWith}`, { encoding: 'utf8' });
+      commits = execSync(`git log ${baseBranch}..HEAD --oneline`, { encoding: 'utf8' }).trim();
+    } else {
+      fullDiff = execSync(`git diff ${compareWith}`, { encoding: 'utf8' });
+      commits = execSync(`git log ${baseBranch}..HEAD --oneline`, { encoding: 'utf8' }).trim();
+    }
+  } catch (e) {
+    error('Error al obtener diff o commits: ' + e.message);
+    return;
+  }
+  
+  // Crear el prompt para Claude
+  const tempDir = `/tmp/claude-analyze-${Date.now()}`;
+  fs.mkdirSync(tempDir, { recursive: true });
+  
+  const promptFile = path.join(tempDir, 'prompt.txt');
+  const prompt = `Analiza los siguientes cambios. CONTEXTO: ${contextDescription}
+
+Por favor genera:
+1. Un título de PR conciso y descriptivo (máximo 72 caracteres)
+2. Una descripción detallada del PR que incluya:
+   - Resumen de los cambios
+   - Motivación/contexto  
+   - Tipo de cambio (feature/fix/refactor/docs/etc)
+   - Testing recomendado
+3. Un nombre de rama sugerido siguiendo el formato: tipo/descripcion-corta (ejemplo: feature/add-user-auth, fix/memory-leak)
+
+IMPORTANTE: Si estos son cambios locales sin pushear, el nombre de rama sugerido debe ser para crear una nueva rama desde la actual.
+
+Responde EXCLUSIVAMENTE con un JSON válido con esta estructura:
+{
+  "prTitle": "título del PR",
+  "prDescription": "descripción detallada del PR con markdown",
+  "suggestedBranchName": "tipo/nombre-rama-sugerido",
+  "changeType": "feature|fix|refactor|docs|test|chore",
+  "breakingChanges": false,
+  "testingNotes": "notas sobre testing necesario"
+}
+
+=== COMMITS ===
+${commits}
+
+=== ARCHIVOS MODIFICADOS ===
+${diffFiles}
+
+=== DIFF COMPLETO ===
+${fullDiff.substring(0, 50000)} ${fullDiff.length > 50000 ? '\n... (diff truncado)' : ''}`;
+
+  fs.writeFileSync(promptFile, prompt);
+
+  info('Enviando a Claude para análisis...');
+  
+  try {
+    const response = execSync(`claude < "${promptFile}"`, { encoding: 'utf8', maxBuffer: 1024 * 1024 * 10 });
+    
+    // Extraer el JSON de la respuesta
+    const jsonMatch = response.match(/\{[\s\S]*\}/);
+    if (!jsonMatch) {
+      error('No se recibió una respuesta JSON válida de Claude.');
+      console.log('Respuesta completa:', response);
+      return;
+    }
+    
+    let result;
+    try {
+      result = JSON.parse(jsonMatch[0]);
+    } catch (e) {
+      error('Error al parsear la respuesta JSON: ' + e.message);
+      console.log('JSON recibido:', jsonMatch[0]);
+      return;
+    }
+    
+    // Mostrar los resultados
+    console.log('');
+    console.log('═══════════════════════════════════════════════════════════════');
+    console.log('                    ANÁLISIS DE DIFERENCIAS                    ');
+    console.log('═══════════════════════════════════════════════════════════════');
+    console.log('');
+    
+    console.log(`🔍 ${colors.blue}Contexto:${colors.reset} ${contextDescription}`);
+    console.log(`📊 ${colors.blue}Archivos modificados:${colors.reset} ${diffFiles.split('\n').length}`);
+    console.log('');
+    
+    console.log(`📝 ${colors.green}Título del PR:${colors.reset}`);
+    console.log(`   ${result.prTitle}`);
+    console.log('');
+    
+    console.log(`🌿 ${colors.green}Nombre de rama sugerido:${colors.reset}`);
+    console.log(`   ${result.suggestedBranchName}`);
+    console.log('');
+    
+    console.log(`📋 ${colors.green}Tipo de cambio:${colors.reset} ${result.changeType}`);
+    
+    if (result.breakingChanges) {
+      console.log(`⚠️  ${colors.yellow}Breaking Changes: SÍ${colors.reset}`);
+    }
+    
+    console.log('');
+    console.log(`📄 ${colors.green}Descripción del PR:${colors.reset}`);
+    console.log('───────────────────────────────────────────────────────────────');
+    console.log(result.prDescription);
+    console.log('───────────────────────────────────────────────────────────────');
+    
+    if (result.testingNotes) {
+      console.log('');
+      console.log(`🧪 ${colors.green}Notas de Testing:${colors.reset}`);
+      console.log(result.testingNotes);
+    }
+    
+    // Guardar los resultados en un archivo con contexto
+    const outputData = {
+      ...result,
+      context: {
+        currentBranch,
+        baseBranch,
+        contextDescription,
+        filesChanged: diffFiles.split('\n').length,
+        timestamp: new Date().toISOString()
+      }
+    };
+    
+    const outputFile = '.claude-pr-analysis.json';
+    fs.writeFileSync(outputFile, JSON.stringify(outputData, null, 2));
+    console.log('');
+    info(`Resultados guardados en ${outputFile}`);
+    
+    // Sugerencias contextuales
+    console.log('');
+    if (!args[0] && contextDescription.includes('cambios locales sin pushear')) {
+      // Caso de cambios locales sin pushear
+      console.log(`💡 ${colors.yellow}Para crear nueva rama con estos cambios:${colors.reset}`);
+      console.log(`   git checkout -b ${result.suggestedBranchName}`);
+      console.log(`   git push -u origin ${result.suggestedBranchName}`);
+    } else if (currentBranch !== result.suggestedBranchName) {
+      // Caso normal de comparación entre ramas  
+      console.log(`💡 ${colors.yellow}Para renombrar tu rama actual:${colors.reset}`);
+      console.log(`   git branch -m ${result.suggestedBranchName}`);
+    }
+    
+    console.log(`💡 ${colors.yellow}Tip:${colors.reset} Usa esta información para crear tu PR en GitHub.`);
+    
+  } catch (e) {
+    error('Error al ejecutar Claude: ' + e.message);
+  } finally {
+    // Limpiar archivos temporales
+    fs.rmSync(tempDir, { recursive: true, force: true });
+  }
+}
+
 // Comando status
 function status() {
   if (!checkGitRepo()) {
@@ -993,8 +1246,8 @@ Comandos:
   uninstall            Desinstala los hooks del repositorio
   enable [hook]        Habilita hooks (todos o uno específico)
   disable [hook]       Deshabilita hooks (todos o uno específico)
-  set-mode [standard|sonar]  Cambia el modo de análisis
   status               Muestra el estado de los hooks
+  analyze-diff [base]  Analiza diferencias entre ramas y genera PR info
   help                 Muestra esta ayuda
 
 Hooks disponibles:
@@ -1005,11 +1258,10 @@ Ejemplos:
   claude-hooks install              # Instala todos los hooks
   claude-hooks install --skip-auth  # Instala sin verificar autenticación
   claude-hooks update               # Actualiza a la última versión
-  claude-hooks set-mode sonar       # Cambiar a modo SonarQube
-  claude-hooks set-mode standard    # Cambiar a modo estándar
   claude-hooks disable pre-commit   # Deshabilita solo pre-commit
   claude-hooks enable               # Habilita todos los hooks
   claude-hooks status               # Ver estado actual
+  claude-hooks analyze-diff main    # Analiza diferencias con main
 
 Casos de uso de commits:
   git commit -m "mensaje"           # Mensaje manual + análisis bloqueante
@@ -1017,6 +1269,12 @@ Casos de uso de commits:
   git commit --no-verify -m "auto"  # Mensaje automático sin análisis
   git commit --no-verify -m "msg"   # Mensaje manual sin análisis
 
+Caso de uso analyze-diff:
+  claude-hooks analyze-diff main    # Analiza cambios vs main y genera:
+    → Título PR: "feat: add user authentication module"
+    → Descripción PR: "## Summary\n- Added JWT authentication..." 
+    → Rama sugerida: "feature/user-authentication"
+
 Excluir código del análisis:
   // SKIP-ANALYSIS                  # Excluye la siguiente línea del análisis
   // SKIP-ANALYSIS                  # Entre dos comentarios excluye el bloque
@@ -1048,12 +1306,12 @@ async function main() {
     case 'disable':
       disable(args[1]);
       break;
-    case 'set-mode':
-      setMode(args[1]);
-      break;
     case 'status':
       status();
       break;
+    case 'analyze-diff':
+      analyzeDiff(args.slice(1));
+      break;
     case 'help':
     case '--help':
     case '-h':

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-git-hooks",
-  "version": "1.4.2",
+  "version": "1.5.0",
   "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": {