claude-git-hooks 2.0.0 → 2.1.0

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).
 
+## [2.1.0] - 2025-11-04
+
+### Changed
+
+- 🔄 Renombrado marcador de exclusión de línea única: `// SKIP-ANALYSIS` → `// SKIP_ANALYSIS_LINE` para consistencia de nomenclatura
+- 🐛 Corregido bug crítico en orden de detección de patrones que impedía funcionamiento de `SKIP_ANALYSIS_BLOCK`
+
+### Known Issues
+
+- ⚠️ **SKIP_ANALYSIS marcado como EXPERIMENTAL/BROKEN**: Los marcadores de exclusión no funcionan correctamente porque el análisis se realiza sobre git diff en lugar del archivo completo. Marcadores agregados en commits anteriores no son detectados en cambios subsecuentes.
+
+### Migration Guide
+
+Si utilizabas `// SKIP-ANALYSIS` en tu código, reemplázalo por `// SKIP_ANALYSIS_LINE`:
+
+```java
+// Antes (deprecated):
+// SKIP-ANALYSIS
+private String legacyCode;
+
+// Ahora:
+// SKIP_ANALYSIS_LINE
+private String legacyCode;
+```
+
+El marcador de bloque `// SKIP_ANALYSIS_BLOCK` permanece sin cambios.
+
+## [2.0.1] - 2025-11-04
+
+### Fixed
+
+- 🔧 Compatibilidad completa en Windows: corregidos line endings CRLF, validación WSL, y comparación de versiones sin dependencia bash
+- 🔧 Operaciones de rutas utilizan módulo `path` nativo para mejor portabilidad multiplataforma
+
 ## [2.0.0] - 2025-10-30
 
 ### 🚀 BREAKING CHANGE: Migration from Bash to Node.js Hooks

package/README.md

@@ -16,7 +16,7 @@
 - 🔍 **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
 - 📋 **Generación de PRs**: Título, descripción y tests sugeridos con un solo comando
-- 🚫 **Skip inteligente**: Excluye código con comentarios SKIP-ANALYSIS
+- ⚠️ **Skip inteligente** (EXPERIMENTAL/BROKEN): Exclusión de código con comentarios SKIP_ANALYSIS - no funciona correctamente con git diff
 - 🕵️‍♂️ **Paralelización**: Análisis asincrónico de archivos con sub agentes
 - 🔄 **Auto-actualización**: Se mantiene actualizado automáticamente
 - 🌍 **Cross-platform**: Windows, WSL, macOS, Linux sin configuración especial
@@ -85,10 +85,13 @@ git commit --no-verify -m "hotfix: corrección urgente"
 git commit --amend
 ```
 
-### 🚫 Exclusión de Código del Análisis
+### ⚠️ Exclusión de Código del Análisis (EXPERIMENTAL/BROKEN)
 
 ```java
-// SKIP-ANALYSIS
+// NOTA: Este feature NO funciona correctamente
+// Los marcadores no son detectados si fueron agregados en commits anteriores
+
+// SKIP_ANALYSIS_LINE
 private String legacyCode = "no analizar siguiente línea";
 
 // SKIP_ANALYSIS_BLOCK
@@ -347,21 +350,19 @@ Si no existe un `.gitignore`, se creará uno nuevo. Si ya existe, las entradas s
 - **Modo debug**: `DEBUG=1 git commit` guarda respuestas en `debug-claude-response.json`
 - **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
-- **Exclusión de código del análisis con SKIP-ANALYSIS**: Puedes excluir código específico del análisis usando comentarios `// SKIP-ANALYSIS`:
+- **⚠️ Exclusión de código del análisis (EXPERIMENTAL/BROKEN)**: Los marcadores `// SKIP_ANALYSIS_LINE` y `// SKIP_ANALYSIS_BLOCK` no funcionan correctamente. El hook analiza git diff en lugar del archivo completo, por lo que marcadores agregados en commits anteriores no son detectados en cambios subsecuentes.
 
 ```java
-// Excluir una sola línea
-// SKIP-ANALYSIS
-@Autowired private LegacyService legacyService;  // Esta línea no será analizada
+// NOTA: Este feature actualmente NO funciona de forma confiable
+// SKIP_ANALYSIS_LINE
+@Autowired private LegacyService legacyService;
 
-// Excluir un bloque de código
-// SKIP-ANALYSIS_BLOCK
+// SKIP_ANALYSIS_BLOCK
 @Deprecated
 public void methodWithKnownIssues() {
-    // Código legacy que no queremos que sea analizado
-    System.out.println("Debug temporal");
+    System.out.println("Legacy code");
 }
-// SKIP-ANALYSIS_BLOCK
+// SKIP_ANALYSIS_BLOCK
 ```
 
 ### Desactivar/Activar hooks
@@ -437,7 +438,7 @@ claude-git-hooks/
 │   └── utils/
 │       ├── logger.js                         # Sistema de logging centralizado
 │       ├── git-operations.js                 # Operaciones git abstractas
-│       ├── file-operations.js                # I/O con filtro SKIP-ANALYSIS
+│       ├── file-operations.js                # I/O con filtro SKIP_ANALYSIS_LINE
 │       ├── claude-client.js                  # Cliente Claude CLI
 │       ├── prompt-builder.js                 # Constructor de prompts
 │       └── resolution-prompt.js              # Generador de resolution prompts

package/bin/claude-hooks

@@ -429,11 +429,10 @@ async function install(args) {
   console.log('  git commit -m "auto"     # Generate message automatically');
   console.log('  git commit -m "message"  # Analyze code before commit');
   console.log('  git commit --no-verify   # Skip analysis completely');
-  console.log('\nExclude code from analysis:');
-  console.log('  // SKIP-ANALYSIS         # Exclude the next line');
-  console.log('  // SKIP_ANALYSIS_BLOCK  # Exclude block until finding another equal one');
-  console.log('  ...excluded code...');
-  console.log('  // SKIP_ANALYSIS_BLOCK');
+  console.log('\n⚠️  Exclude code from analysis (EXPERIMENTAL/BROKEN):');
+  console.log('  // SKIP_ANALYSIS_LINE    # Does NOT work reliably');
+  console.log('  // SKIP_ANALYSIS_BLOCK   # Does NOT work reliably');
+  console.log('  // Reason: Analyzes git diff, not full file');
   console.log('\nNEW: Parallel analysis for faster multi-file commits:');
   console.log('  export CLAUDE_USE_SUBAGENTS=true       # Enable subagents');
   console.log('  export CLAUDE_SUBAGENT_MODEL=haiku     # haiku/sonnet/opus');
@@ -1064,36 +1063,24 @@ function setMode(mode) {
   }
 }
 
-// Función para comparar versiones usando el script compartido
+// Cross-platform version comparison (semver)
+// Why: Pure JavaScript, no bash dependency
+// Returns: 0 if equal, 1 if v1 > v2, -1 if v1 < v2
 function compareVersions(v1, v2) {
-  try {
-    // Usar el script compartido para mantener consistencia
-    const result = execSync(`bash -c 'source "${getTemplatesPath()}/check-version.sh" && compare_versions "${v1}" "${v2}"; echo $?'`, { encoding: 'utf8' }).trim();
-    const exitCode = parseInt(result);
-
-    // Convertir los códigos de retorno del script bash a valores JS
-    if (exitCode === 0) return 0;      // iguales
-    if (exitCode === 1) return 1;      // v1 > v2  
-    if (exitCode === 2) return -1;     // v1 < v2
-
-    // Fallback: comparación simple si el script falla
-    if (v1 === v2) return 0;
-    const sorted = [v1, v2].sort((a, b) => {
-      const aParts = a.split('.').map(Number);
-      const bParts = b.split('.').map(Number);
-      for (let i = 0; i < Math.max(aParts.length, bParts.length); i++) {
-        const aPart = aParts[i] || 0;
-        const bPart = bParts[i] || 0;
-        if (aPart !== bPart) return aPart - bPart;
-      }
-      return 0;
-    });
-    return v1 === sorted[1] ? 1 : -1;
-  } catch (e) {
-    // Si falla todo, usar comparación simple
-    if (v1 === v2) return 0;
-    return v1 > v2 ? 1 : -1;
+  if (v1 === v2) return 0;
+
+  const v1Parts = v1.split('.').map(Number);
+  const v2Parts = v2.split('.').map(Number);
+
+  for (let i = 0; i < Math.max(v1Parts.length, v2Parts.length); i++) {
+    const v1Part = v1Parts[i] || 0;
+    const v2Part = v2Parts[i] || 0;
+
+    if (v1Part > v2Part) return 1;
+    if (v1Part < v2Part) return -1;
   }
+
+  return 0;
 }
 
 // Update command - update to the latest version
@@ -1189,11 +1176,10 @@ Analyze-diff use case:
     → PR Description: "## Summary\n- Added JWT authentication..." 
     → Suggested branch: "feature/user-authentication"
 
-Exclude code from analysis:
-  // SKIP-ANALYSIS                  # Exclude the next line from analysis
-  // SKIP_ANALYSIS_BLOCK           # Exclude block until finding another equal one
-  ...excluded code...
-  // SKIP_ANALYSIS_BLOCK
+⚠️  Exclude code from analysis (EXPERIMENTAL/BROKEN):
+  // SKIP_ANALYSIS_LINE    # Does NOT work reliably
+  // SKIP_ANALYSIS_BLOCK   # Does NOT work reliably
+  Reason: Analyzes git diff, not full file
 
 Performance optimization (NEW in v1.5.5):
   export CLAUDE_USE_SUBAGENTS=true        # Enable parallel analysis for 3+ files

package/lib/hooks/pre-commit.js

@@ -205,7 +205,7 @@ const main = async () => {
                 // Get diff
                 let diff = getFileDiff(filePath);
 
-                // Apply SKIP-ANALYSIS filtering
+                // Apply SKIP_ANALYSIS filtering
                 diff = filterSkipAnalysis(diff);
 
                 // Check if new file

package/lib/utils/file-operations.js

@@ -121,11 +121,21 @@ const readFile = async (filePath, { maxSize = 100000, encoding = 'utf8' } = {})
 };
 
 /**
- * Filters SKIP-ANALYSIS patterns from code content
+ * Filters SKIP_ANALYSIS patterns from code content
  * Why: Allows developers to exclude specific code from analysis
  *
+ * ⚠️ KNOWN ISSUE (EXPERIMENTAL/BROKEN):
+ * This feature does NOT work reliably. The pre-commit hook analyzes git diff
+ * instead of full file content. Markers added in previous commits are NOT
+ * present in subsequent diffs, so they are not detected.
+ *
+ * Example of failure:
+ * - Commit 1: Add // SKIP_ANALYSIS_BLOCK markers
+ * - Commit 2: Modify line inside block
+ * - Result: Diff only shows modified line, NOT the markers → filter fails
+ *
  * Supports two patterns:
- * 1. Single line: // SKIP-ANALYSIS (excludes next line)
+ * 1. Single line: // SKIP_ANALYSIS_LINE (excludes next line)
  * 2. Block: // SKIP_ANALYSIS_BLOCK ... // SKIP_ANALYSIS_BLOCK (excludes block)
  *
  * @param {string} content - File content to filter
@@ -134,7 +144,7 @@ const readFile = async (filePath, { maxSize = 100000, encoding = 'utf8' } = {})
 const filterSkipAnalysis = (content) => {
     logger.debug(
         'file-operations - filterSkipAnalysis',
-        'Filtering SKIP-ANALYSIS patterns',
+        'Filtering SKIP_ANALYSIS patterns',
         { originalLength: content.length }
     );
 
@@ -145,11 +155,8 @@ const filterSkipAnalysis = (content) => {
     // Why: Use map instead of filter to preserve line numbers for error reporting
     // Empty lines maintain original line number mapping
     const filteredLines = lines.map((line) => {
-        // Detect single-line SKIP-ANALYSIS
-        if (line.includes('// SKIP-ANALYSIS')) {
-            skipNext = true;
-            return ''; // Preserve line number
-        }
+        // IMPORTANT: Check specific pattern first (SKIP_ANALYSIS_BLOCK)
+        // before general pattern (SKIP_ANALYSIS_LINE) to avoid substring match issues
 
         // Detect SKIP_ANALYSIS_BLOCK (toggle block state)
         if (line.includes('// SKIP_ANALYSIS_BLOCK')) {
@@ -157,12 +164,18 @@ const filterSkipAnalysis = (content) => {
             return ''; // Preserve line number
         }
 
+        // Detect single-line SKIP_ANALYSIS_LINE (must check AFTER block check)
+        if (line.includes('// SKIP_ANALYSIS_LINE')) {
+            skipNext = true;
+            return ''; // Preserve line number
+        }
+
         // Skip lines inside block
         if (inSkipBlock) {
             return '';
         }
 
-        // Skip next line after single SKIP-ANALYSIS
+        // Skip next line after single SKIP_ANALYSIS_LINE
         if (skipNext) {
             skipNext = false;
             return '';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-git-hooks",
-  "version": "2.0.0",
+  "version": "2.1.0",
   "description": "Git hooks with Claude CLI for code analysis and automatic commit messages",
   "type": "module",
   "main": "lib/index.js",