claude-git-hooks 2.1.0 → 2.3.1

package/templates/shared/PRE_COMMIT_GUIDELINES.md

@@ -0,0 +1,145 @@
+# General Code Quality Guidelines
+
+## Security Fundamentals
+
+### Input Validation
+✅ Validate all user input
+✅ Sanitize data before use
+✅ Use parameterized queries
+✅ Escape output properly
+✅ Validate file paths
+
+### Credentials and Secrets
+✅ Never hardcode credentials
+✅ Use environment variables or secure vaults
+✅ Don't log sensitive data
+✅ Rotate keys regularly
+✅ Use secure random generators
+
+### Authentication & Authorization
+✅ Implement proper authentication
+✅ Check permissions before operations
+✅ Use secure session management
+✅ Implement rate limiting
+✅ Log security events
+
+## Reliability
+
+### Error Handling
+✅ Handle all error cases
+✅ Provide meaningful error messages
+✅ Clean up resources in finally blocks
+✅ Don't swallow exceptions silently
+✅ Log errors appropriately
+
+### Null Safety
+✅ Check for null/undefined values
+✅ Use optional chaining where supported
+✅ Provide default values
+✅ Document when null is valid
+✅ Avoid null pointer exceptions
+
+### Resource Management
+✅ Close files and connections
+✅ Free allocated memory
+✅ Use try-with-resources patterns
+✅ Implement proper cleanup
+✅ Avoid resource leaks
+
+## Maintainability
+
+### Code Organization
+✅ Keep functions small and focused
+✅ Use descriptive names
+✅ Follow consistent naming conventions
+✅ Organize code logically
+✅ Limit nesting depth
+
+### Documentation
+✅ Document complex logic
+✅ Explain non-obvious decisions
+✅ Keep comments up to date
+✅ Use inline documentation
+✅ Document public APIs
+
+### Code Duplication
+✅ Extract common code into functions
+✅ Use appropriate design patterns
+✅ Don't copy-paste code
+✅ Refactor when you see duplication
+✅ Keep DRY (Don't Repeat Yourself)
+
+## Performance
+
+### General Guidelines
+✅ Choose appropriate data structures
+✅ Avoid unnecessary loops
+✅ Cache expensive computations
+✅ Use lazy loading when appropriate
+✅ Profile before optimizing
+
+### Resource Usage
+✅ Limit memory allocations
+✅ Use streaming for large data
+✅ Implement pagination
+✅ Clean up resources promptly
+✅ Avoid memory leaks
+
+## Common Issues to Avoid
+
+### Critical Issues (BLOCKER)
+❌ Security vulnerabilities (SQL injection, XSS, etc.)
+❌ Exposed credentials or secrets
+❌ Data loss risks
+❌ Critical bugs that crash the application
+
+### Major Issues (CRITICAL/MAJOR)
+❌ Unhandled exceptions
+❌ Resource leaks
+❌ Missing input validation
+❌ Poor error handling
+❌ Performance bottlenecks
+
+### Minor Issues (MINOR/INFO)
+❌ Code duplication
+❌ Poor naming
+❌ Missing documentation
+❌ Style inconsistencies
+❌ Unused code
+
+## Testing
+
+✅ Write tests for new functionality
+✅ Test error scenarios
+✅ Test edge cases
+✅ Mock external dependencies
+✅ Aim for reasonable coverage
+
+## Version Control
+
+✅ Write clear commit messages
+✅ Keep commits focused and atomic
+✅ Don't commit sensitive data
+✅ Review your own changes first
+✅ Rebase/clean up before pushing
+
+## Language-Specific
+
+Different languages have specific best practices:
+
+- **JavaScript/Node.js**: Use strict mode, handle promises, avoid callback hell
+- **Python**: Follow PEP 8, use virtual environments, handle exceptions
+- **Java**: Follow naming conventions, use proper access modifiers, handle checked exceptions
+- **SQL**: Use parameterized queries, optimize for performance, maintain data integrity
+- **Shell scripts**: Quote variables, check exit codes, handle errors
+
+## Quality Standards
+
+Strive for:
+- **Reliability**: A or B
+- **Security**: A or B
+- **Maintainability**: A or B
+- **Coverage**: 70%+ for new code
+- **Complexity**: Keep cyclomatic complexity low
+
+Remember: The goal is to write code that is secure, reliable, maintainable, and performs well.

package/templates/shared/RESOLUTION_PROMPT.md

@@ -0,0 +1,32 @@
+# FIX_CRITICAL_ISSUES
+
+CONTEXT:
+repo:{{REPO_NAME}}
+branch:{{BRANCH_NAME}}
+commit:{{COMMIT_SHA}}
+files:{{FILE_COUNT}}
+
+CRITICAL_ISSUES:
+{{BLOCKING_ISSUES}}
+
+AFFECTED_FILES:
+{{FILE_CONTENTS}}
+
+INSTRUCTIONS:
+1.Navigate→exact_file:line
+2.Apply_minimal_fix
+3.Maintain_style/conventions
+4.No_unrelated_refactoring
+
+OUTPUT:
+- diff_format
+- one_line_explanation_max
+
+PRIORITY:
+1.security_vulns
+2.null_ptr/runtime
+3.logic_errors
+4.performance
+5.code_quality
+
+START_FIXING:

package/templates/check-version.sh

@@ -1,266 +0,0 @@
-#!/bin/bash
-
-# Script compartido para verificar versión de claude-git-hooks
-# Este script puede ser invocado desde cualquier hook
-
-# Colores para output
-RED='\033[0;31m'
-GREEN='\033[0;32m'
-YELLOW='\033[1;33m'
-BLUE='\033[0;34m'
-NC='\033[0m' # No Color
-
-# Función para logging
-log() {
-    echo -e "${GREEN}[VERSION-CHECK]${NC} $1"
-}
-
-error() {
-    echo -e "${RED}[ERROR]${NC} $1"
-}
-
-warning() {
-    echo -e "${YELLOW}[WARNING]${NC} $1"
-}
-
-info() {
-    echo -e "${BLUE}[INFO]${NC} $1"
-}
-
-# Función compartida para comparar versiones semánticas
-# Retorna: 1 si v1 > v2, 2 si v1 < v2, 0 si son iguales
-compare_versions() {
-    local v1="$1"
-    local v2="$2"
-    
-    if [ -n "$DEBUG_VERSION" ]; then
-        echo "[DEBUG] compare_versions: comparando '$v1' con '$v2'"
-    fi
-    
-    # Si son exactamente iguales
-    if [ "$v1" = "$v2" ]; then
-        if [ -n "$DEBUG_VERSION" ]; then
-            echo "[DEBUG] compare_versions: versiones iguales, retornando 0"
-        fi
-        return 0
-    fi
-    
-    # Usar sort -V que entiende versiones semánticas correctamente
-    local sorted=$(printf "%s\n%s" "$v1" "$v2" | sort -V)
-    local smaller=$(echo "$sorted" | head -1)
-    local larger=$(echo "$sorted" | tail -1)
-    
-    if [ -n "$DEBUG_VERSION" ]; then
-        echo "[DEBUG] compare_versions: sorted='$sorted', smaller='$smaller', larger='$larger'"
-    fi
-    
-    if [ "$v1" = "$larger" ]; then
-        if [ -n "$DEBUG_VERSION" ]; then
-            echo "[DEBUG] compare_versions: v1 es mayor, retornando 1"
-        fi
-        return 1  # v1 es mayor que v2
-    else
-        if [ -n "$DEBUG_VERSION" ]; then
-            echo "[DEBUG] compare_versions: v1 es menor, retornando 2"
-        fi
-        return 2  # v1 es menor que v2 (cambié de 255 a 2 para evitar problemas con set -e)
-    fi
-}
-
-# Función principal de verificación
-check_version() {
-    # Debug mode si está activado
-    if [ -n "$DEBUG_VERSION" ]; then
-        echo "[DEBUG] Iniciando verificación de versión..."
-    fi
-    
-    info "Verificando versión instalada..."
-    
-    # Intentar obtener versión actual desde package.json global
-    CURRENT_VERSION=""
-    NPM_PREFIX=$(npm prefix -g 2>/dev/null || echo "/usr/local")
-    PACKAGE_JSON_PATHS=(
-        "$NPM_PREFIX/lib/node_modules/claude-git-hooks/package.json"
-        "/usr/lib/node_modules/claude-git-hooks/package.json"
-        "/usr/local/lib/node_modules/claude-git-hooks/package.json"
-        "$HOME/.npm-global/lib/node_modules/claude-git-hooks/package.json"
-    )
-    
-    for path in "${PACKAGE_JSON_PATHS[@]}"; do
-        if [ -f "$path" ]; then
-            CURRENT_VERSION=$(grep '"version"' "$path" 2>/dev/null | sed 's/.*"version".*"\(.*\)".*/\1/')
-            break
-        fi
-    done
-    
-    if [ -z "$CURRENT_VERSION" ]; then
-        warning "No se pudo determinar la versión actual"
-        return 0  # Continuar sin bloquear
-    fi
-    
-    if [ -n "$DEBUG_VERSION" ]; then
-        echo "[DEBUG] Versión actual encontrada: $CURRENT_VERSION"
-    fi
-    
-    # Obtener última versión desde NPM (usando npm view que es más confiable)
-    info "Verificando última versión disponible..."
-    
-    # Método principal: usar npm view que es más confiable
-    if [ -n "$DEBUG_VERSION" ]; then
-        echo "[DEBUG] Ejecutando: npm view claude-git-hooks version"
-    fi
-    
-    LATEST_VERSION=$(npm view claude-git-hooks version 2>/dev/null || echo "")
-    
-    if [ -n "$DEBUG_VERSION" ]; then
-        echo "[DEBUG] npm view retornó: '$LATEST_VERSION'"
-        echo "[DEBUG] Longitud de LATEST_VERSION: ${#LATEST_VERSION}"
-    fi
-    
-    # Si npm view falla, intentar con la API de NPM
-    if [ -z "$LATEST_VERSION" ]; then
-        # Obtener el JSON completo y extraer dist-tags.latest con jq si está disponible
-        if command -v jq &> /dev/null; then
-            LATEST_VERSION=$(curl -s https://registry.npmjs.org/claude-git-hooks 2>/dev/null | jq -r '."dist-tags".latest' 2>/dev/null || echo "")
-            if [ -n "$DEBUG_VERSION" ]; then
-                echo "[DEBUG] jq parsing retornó: '$LATEST_VERSION'"
-            fi
-        else
-            # Sin jq, usar método más robusto con sed
-            LATEST_VERSION=$(curl -s https://registry.npmjs.org/claude-git-hooks 2>/dev/null | sed -n 's/.*"dist-tags":{[^}]*"latest":"\([^"]*\)".*/\1/p' | head -1)
-            if [ -n "$DEBUG_VERSION" ]; then
-                echo "[DEBUG] sed parsing retornó: '$LATEST_VERSION'"
-            fi
-        fi
-    fi
-    
-    # Validar que lo obtenido parece una versión (formato X.Y.Z)
-    if [ -n "$DEBUG_VERSION" ]; then
-        echo "[DEBUG] Validando versión: '$LATEST_VERSION'"
-    fi
-    
-    if ! echo "$LATEST_VERSION" | grep -qE '^[0-9]+\.[0-9]+\.[0-9]+' ; then
-        if [ -n "$DEBUG_VERSION" ]; then
-            echo "[DEBUG] Versión obtenida no es válida: '$LATEST_VERSION'"
-        fi
-        warning "No se pudo obtener una versión válida del registro NPM"
-        return 0
-    fi
-    
-    if [ -n "$DEBUG_VERSION" ]; then
-        echo "[DEBUG] Versión válida confirmada: '$LATEST_VERSION'"
-    fi
-    
-    if [ -z "$LATEST_VERSION" ]; then
-        # Si no se puede obtener la versión, continuar sin bloquear
-        return 0
-    fi
-    
-    if [ "$CURRENT_VERSION" = "$LATEST_VERSION" ]; then
-        log "Versión actual ($CURRENT_VERSION) está actualizada ✓"
-        return 0
-    fi
-    
-    # Comparar versiones usando la función compartida
-    if [ -n "$DEBUG_VERSION" ]; then
-        echo "[DEBUG] Comparando versiones: '$CURRENT_VERSION' vs '$LATEST_VERSION'"
-    fi
-    
-    # Deshabilitar set -e temporalmente para capturar el código de retorno
-    set +e
-    compare_versions "$CURRENT_VERSION" "$LATEST_VERSION"
-    comparison_result=$?
-    set -e
-    
-    if [ -n "$DEBUG_VERSION" ]; then
-        echo "[DEBUG] Resultado de comparación: $comparison_result"
-    fi
-    
-    if [ $comparison_result -eq 0 ]; then
-        # Las versiones son iguales
-        if [ -n "$DEBUG_VERSION" ]; then
-            echo "[DEBUG] Las versiones son iguales"
-        fi
-        log "Versión actual ($CURRENT_VERSION) está actualizada ✓"
-        return 0
-    elif [ $comparison_result -eq 1 ]; then
-        # La versión actual es mayor que la publicada (desarrollo local)
-        if [ -n "$DEBUG_VERSION" ]; then
-            echo "[DEBUG] Versión local ($CURRENT_VERSION) es más nueva que la publicada ($LATEST_VERSION)"
-        fi
-        log "Versión de desarrollo ($CURRENT_VERSION) > versión publicada ($LATEST_VERSION) ✓"
-        return 0
-    elif [ $comparison_result -eq 2 ]; then
-        # La versión publicada es mayor, continuar con prompt de actualización
-        if [ -n "$DEBUG_VERSION" ]; then
-            echo "[DEBUG] La versión publicada ($LATEST_VERSION) es más nueva que la local ($CURRENT_VERSION)"
-        fi
-        # Continuar con el prompt de actualización (no return aquí)
-    fi
-    
-    if [ -n "$DEBUG_VERSION" ]; then
-        echo "[DEBUG] La versión publicada es más nueva, procediendo con prompt de actualización"
-    fi
-    
-    # Solo mostrar actualización si la versión publicada es realmente más nueva
-    echo
-    warning "Nueva versión disponible: $LATEST_VERSION (actual: $CURRENT_VERSION)"
-    echo -e "${YELLOW}╭─────────────────────────────────────────────╮${NC}"
-    echo -e "${YELLOW}│${NC} Una nueva versión de claude-git-hooks está ${YELLOW}│${NC}"
-    echo -e "${YELLOW}│${NC} disponible con mejoras y correcciones      ${YELLOW}│${NC}"
-    echo -e "${YELLOW}╰─────────────────────────────────────────────╯${NC}"
-    echo
-    
-    # Leer respuesta del usuario de forma compatible con todas las consolas
-    echo -n "¿Deseas actualizar ahora? (s/n): "
-    
-    # Intentar varios métodos de lectura para máxima compatibilidad
-    REPLY=""
-    
-    # Método 1: Leer desde /dev/tty si existe
-    if [ -c /dev/tty ]; then
-        read -r REPLY </dev/tty 2>/dev/null || REPLY=""
-    fi
-    
-    # Método 2: Si falla, intentar leer desde stdin estándar
-    if [ -z "$REPLY" ]; then
-        read -r REPLY 2>/dev/null || REPLY=""
-    fi
-    
-    # Método 3: Si aún no tenemos respuesta, usar bash -c
-    if [ -z "$REPLY" ]; then
-        REPLY=$(bash -c 'read -r reply && echo "$reply"' 2>/dev/null || echo "")
-    fi
-    
-    # Si ningún método funcionó, informar y continuar
-    if [ -z "$REPLY" ]; then
-        echo
-        info "No se pudo leer la respuesta. Continuando sin actualizar."
-        info "Para actualizar manualmente, ejecuta: claude-hooks update"
-        return 0
-    fi
-    
-    if [ "$REPLY" = "s" ] || [ "$REPLY" = "S" ] || [ "$REPLY" = "si" ] || [ "$REPLY" = "SI" ] || [ "$REPLY" = "sí" ] || [ "$REPLY" = "SÍ" ] || [ "$REPLY" = "y" ] || [ "$REPLY" = "Y" ] || [ "$REPLY" = "yes" ] || [ "$REPLY" = "YES" ]; then
-        echo
-        info "Actualizando claude-git-hooks..."
-        
-        if npm install -g claude-git-hooks@latest; then
-            echo
-            log "✅ Actualización completada exitosamente"
-            info "Por favor, vuelve a ejecutar tu comando git commit"
-            exit 0  # Salir para que el usuario reinicie con la nueva versión
-        else
-            error "Error al actualizar. Intenta manualmente con: npm install -g claude-git-hooks@latest"
-            return 1
-        fi
-    else
-        info "Actualización pospuesta. Puedes actualizar más tarde con: claude-hooks update"
-        echo
-        return 0
-    fi
-}
-
-# Si se ejecuta directamente (no como source), ejecutar la verificación
-if [ "${BASH_SOURCE[0]}" = "${0}" ]; then
-    check_version
-fi