claude-git-hooks 1.5.2 → 1.5.3

package/CHANGELOG.md

@@ -5,6 +5,21 @@ 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.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.3",
   "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/",