@localtech/claude-code-toolkit 1.0.5 → 1.0.7

package/templates/.claude/hooks/pre-commit/code-quality-guardian.sh

@@ -1,257 +1,257 @@
-#!/bin/bash
-# HOOK: pre-commit-code-quality-guardian
-# DESCRIPTION: Comprehensive code quality checks before commits
-# AUTHOR: Claude Code Hooks Master
-# VERSION: 1.0.0
-# DEPENDS: node, npm, eslint, prettier, typescript
-
-set -e
-
-# Colors for output
-RED='\033[0;31m'
-GREEN='\033[0;32m'
-YELLOW='\033[1;33m'
-BLUE='\033[0;34m'
-NC='\033[0m'
-
-# Logging functions
-log_info() {
-    echo -e "${BLUE}[QUALITY]${NC} $1"
-}
-
-log_success() {
-    echo -e "${GREEN}[PASS]${NC} $1"
-}
-
-log_warn() {
-    echo -e "${YELLOW}[WARN]${NC} $1"
-}
-
-log_error() {
-    echo -e "${RED}[FAIL]${NC} $1"
-}
-
-# Timer function
-start_timer() {
-    start_time=$(date +%s)
-}
-
-end_timer() {
-    end_time=$(date +%s)
-    duration=$((end_time - start_time))
-    echo -e "${BLUE}[TIME]${NC} Completed in ${duration}s"
-}
-
-# Check if file exists and is executable
-check_dependency() {
-    if ! command -v "$1" &> /dev/null; then
-        log_error "Required dependency '$1' not found"
-        return 1
-    fi
-    return 0
-}
-
-# Get staged files for analysis
-get_staged_files() {
-    git diff --cached --name-only --diff-filter=ACM | grep -E '\.(js|ts|jsx|tsx|vue|svelte)$' || true
-}
-
-# Lint staged files
-run_linter() {
-    local files=("$@")
-    if [ ${#files[@]} -eq 0 ]; then
-        log_info "No JavaScript/TypeScript files to lint"
-        return 0
-    fi
-
-    log_info "Running ESLint on ${#files[@]} files..."
-    start_timer
-
-    if npx eslint "${files[@]}" --quiet; then
-        log_success "ESLint checks passed"
-        end_timer
-        return 0
-    else
-        log_error "ESLint found issues. Please fix them before committing."
-        log_info "Tip: Run 'npx eslint --fix' to auto-fix some issues"
-        return 1
-    fi
-}
-
-# Format staged files
-run_formatter() {
-    local files=("$@")
-    if [ ${#files[@]} -eq 0 ]; then
-        log_info "No files to format"
-        return 0
-    fi
-
-    log_info "Running Prettier on ${#files[@]} files..."
-    start_timer
-
-    if npx prettier --check "${files[@]}"; then
-        log_success "Code formatting is correct"
-        end_timer
-        return 0
-    else
-        log_error "Code formatting issues found."
-        log_info "Tip: Run 'npx prettier --write' to auto-format"
-        return 1
-    fi
-}
-
-# Type checking for TypeScript
-run_type_check() {
-    if [ -f "tsconfig.json" ]; then
-        log_info "Running TypeScript type checking..."
-        start_timer
-
-        if npx tsc --noEmit; then
-            log_success "TypeScript compilation successful"
-            end_timer
-            return 0
-        else
-            log_error "TypeScript errors found. Please fix type issues."
-            return 1
-        fi
-    else
-        log_info "No tsconfig.json found, skipping TypeScript checks"
-        return 0
-    fi
-}
-
-# Security vulnerability check
-run_security_check() {
-    log_info "Running security vulnerability check..."
-    start_timer
-
-    # Check for common security issues
-    local staged_files
-    mapfile -t staged_files < <(get_staged_files)
-
-    local has_security_issues=false
-
-    for file in "${staged_files[@]}"; do
-        # Check for console.log in production code
-        if grep -q "console\." "$file" && [[ "$file" != *".test."* ]] && [[ "$file" != *".spec."* ]]; then
-            log_warn "console statements found in $file (remove for production)"
-            has_security_issues=true
-        fi
-
-        # Check for debugger statements
-        if grep -q "debugger" "$file"; then
-            log_warn "debugger statement found in $file"
-            has_security_issues=true
-        fi
-
-        # Check for potential secrets
-        if grep -i -q "password\|secret\|token\|key" "$file" && grep -q "=" "$file"; then
-            log_warn "Potential secret exposure in $file - review carefully"
-            has_security_issues=true
-        fi
-    done
-
-    if [ "$has_security_issues" = false ]; then
-        log_success "Security check passed"
-    fi
-
-    end_timer
-    return 0  # Don't fail commit for warnings
-}
-
-# Performance check
-run_performance_check() {
-    log_info "Running basic performance checks..."
-    start_timer
-
-    local staged_files
-    mapfile -t staged_files < <(get_staged_files)
-
-    local has_performance_issues=false
-
-    for file in "${staged_files[@]}"; do
-        # Check bundle size for large files
-        local file_size
-        file_size=$(stat -f%z "$file" 2>/dev/null || stat -c%s "$file" 2>/dev/null || echo "0")
-
-        if [ "$file_size" -gt 1000000 ]; then # 1MB
-            log_warn "Large file detected: $file (${file_size} bytes)"
-            has_performance_issues=true
-        fi
-
-        # Check for inefficient patterns
-        if grep -q "for.*in.*array\|for.*in.*object" "$file"; then
-            log_warn "Potential inefficient loop in $file - consider alternatives"
-            has_performance_issues=true
-        fi
-    done
-
-    if [ "$has_performance_issues" = false ]; then
-        log_success "Performance check passed"
-    fi
-
-    end_timer
-    return 0
-}
-
-# Main execution
-main() {
-    log_info "🛡️  Claude Code Quality Guardian - Pre-commit Hook"
-    echo "=================================================="
-
-    # Check dependencies
-    local deps=("node" "npm" "npx")
-    for dep in "${deps[@]}"; do
-        if ! check_dependency "$dep"; then
-            log_error "Missing required dependency: $dep"
-            exit 1
-        fi
-    done
-
-    # Get staged files
-    local staged_files
-    mapfile -t staged_files < <(get_staged_files)
-
-    if [ ${#staged_files[@]} -eq 0 ]; then
-        log_info "No JavaScript/TypeScript files staged for commit"
-        exit 0
-    fi
-
-    log_info "Checking ${#staged_files[@]} files: ${staged_files[*]}"
-    echo
-
-    # Run all quality checks
-    local checks_passed=true
-
-    if ! run_linter "${staged_files[@]}"; then
-        checks_passed=false
-    fi
-
-    if ! run_formatter "${staged_files[@]}"; then
-        checks_passed=false
-    fi
-
-    if ! run_type_check; then
-        checks_passed=false
-    fi
-
-    run_security_check
-    run_performance_check
-
-    echo
-    if [ "$checks_passed" = true ]; then
-        log_success "🎉 All quality checks passed! Your code is ready for commit."
-        exit 0
-    else
-        log_error "❌ Quality checks failed. Please fix the issues and try again."
-        echo
-        log_info "Quick fixes:"
-        echo "  • Run 'npx eslint --fix' to auto-fix linting issues"
-        echo "  • Run 'npx prettier --write' to auto-format code"
-        echo "  • Check TypeScript errors with 'npx tsc --noEmit'"
-        exit 1
-    fi
-}
-
-# Run main function
-main "$@"
+#!/bin/bash
+# HOOK: pre-commit-code-quality-guardian
+# DESCRIPTION: Comprehensive code quality checks before commits
+# AUTHOR: Claude Code Hooks Master
+# VERSION: 1.0.0
+# DEPENDS: node, npm, eslint, prettier, typescript
+
+set -e
+
+# Colors for output
+RED='\033[0;31m'
+GREEN='\033[0;32m'
+YELLOW='\033[1;33m'
+BLUE='\033[0;34m'
+NC='\033[0m'
+
+# Logging functions
+log_info() {
+    echo -e "${BLUE}[QUALITY]${NC} $1"
+}
+
+log_success() {
+    echo -e "${GREEN}[PASS]${NC} $1"
+}
+
+log_warn() {
+    echo -e "${YELLOW}[WARN]${NC} $1"
+}
+
+log_error() {
+    echo -e "${RED}[FAIL]${NC} $1"
+}
+
+# Timer function
+start_timer() {
+    start_time=$(date +%s)
+}
+
+end_timer() {
+    end_time=$(date +%s)
+    duration=$((end_time - start_time))
+    echo -e "${BLUE}[TIME]${NC} Completed in ${duration}s"
+}
+
+# Check if file exists and is executable
+check_dependency() {
+    if ! command -v "$1" &> /dev/null; then
+        log_error "Required dependency '$1' not found"
+        return 1
+    fi
+    return 0
+}
+
+# Get staged files for analysis
+get_staged_files() {
+    git diff --cached --name-only --diff-filter=ACM | grep -E '\.(js|ts|jsx|tsx|vue|svelte)$' || true
+}
+
+# Lint staged files
+run_linter() {
+    local files=("$@")
+    if [ ${#files[@]} -eq 0 ]; then
+        log_info "No JavaScript/TypeScript files to lint"
+        return 0
+    fi
+
+    log_info "Running ESLint on ${#files[@]} files..."
+    start_timer
+
+    if npx eslint "${files[@]}" --quiet; then
+        log_success "ESLint checks passed"
+        end_timer
+        return 0
+    else
+        log_error "ESLint found issues. Please fix them before committing."
+        log_info "Tip: Run 'npx eslint --fix' to auto-fix some issues"
+        return 1
+    fi
+}
+
+# Format staged files
+run_formatter() {
+    local files=("$@")
+    if [ ${#files[@]} -eq 0 ]; then
+        log_info "No files to format"
+        return 0
+    fi
+
+    log_info "Running Prettier on ${#files[@]} files..."
+    start_timer
+
+    if npx prettier --check "${files[@]}"; then
+        log_success "Code formatting is correct"
+        end_timer
+        return 0
+    else
+        log_error "Code formatting issues found."
+        log_info "Tip: Run 'npx prettier --write' to auto-format"
+        return 1
+    fi
+}
+
+# Type checking for TypeScript
+run_type_check() {
+    if [ -f "tsconfig.json" ]; then
+        log_info "Running TypeScript type checking..."
+        start_timer
+
+        if npx tsc --noEmit; then
+            log_success "TypeScript compilation successful"
+            end_timer
+            return 0
+        else
+            log_error "TypeScript errors found. Please fix type issues."
+            return 1
+        fi
+    else
+        log_info "No tsconfig.json found, skipping TypeScript checks"
+        return 0
+    fi
+}
+
+# Security vulnerability check
+run_security_check() {
+    log_info "Running security vulnerability check..."
+    start_timer
+
+    # Check for common security issues
+    local staged_files
+    mapfile -t staged_files < <(get_staged_files)
+
+    local has_security_issues=false
+
+    for file in "${staged_files[@]}"; do
+        # Check for console.log in production code
+        if grep -q "console\." "$file" && [[ "$file" != *".test."* ]] && [[ "$file" != *".spec."* ]]; then
+            log_warn "console statements found in $file (remove for production)"
+            has_security_issues=true
+        fi
+
+        # Check for debugger statements
+        if grep -q "debugger" "$file"; then
+            log_warn "debugger statement found in $file"
+            has_security_issues=true
+        fi
+
+        # Check for potential secrets
+        if grep -i -q "password\|secret\|token\|key" "$file" && grep -q "=" "$file"; then
+            log_warn "Potential secret exposure in $file - review carefully"
+            has_security_issues=true
+        fi
+    done
+
+    if [ "$has_security_issues" = false ]; then
+        log_success "Security check passed"
+    fi
+
+    end_timer
+    return 0  # Don't fail commit for warnings
+}
+
+# Performance check
+run_performance_check() {
+    log_info "Running basic performance checks..."
+    start_timer
+
+    local staged_files
+    mapfile -t staged_files < <(get_staged_files)
+
+    local has_performance_issues=false
+
+    for file in "${staged_files[@]}"; do
+        # Check bundle size for large files
+        local file_size
+        file_size=$(stat -f%z "$file" 2>/dev/null || stat -c%s "$file" 2>/dev/null || echo "0")
+
+        if [ "$file_size" -gt 1000000 ]; then # 1MB
+            log_warn "Large file detected: $file (${file_size} bytes)"
+            has_performance_issues=true
+        fi
+
+        # Check for inefficient patterns
+        if grep -q "for.*in.*array\|for.*in.*object" "$file"; then
+            log_warn "Potential inefficient loop in $file - consider alternatives"
+            has_performance_issues=true
+        fi
+    done
+
+    if [ "$has_performance_issues" = false ]; then
+        log_success "Performance check passed"
+    fi
+
+    end_timer
+    return 0
+}
+
+# Main execution
+main() {
+    log_info "🛡️  Claude Code Quality Guardian - Pre-commit Hook"
+    echo "=================================================="
+
+    # Check dependencies
+    local deps=("node" "npm" "npx")
+    for dep in "${deps[@]}"; do
+        if ! check_dependency "$dep"; then
+            log_error "Missing required dependency: $dep"
+            exit 1
+        fi
+    done
+
+    # Get staged files
+    local staged_files
+    mapfile -t staged_files < <(get_staged_files)
+
+    if [ ${#staged_files[@]} -eq 0 ]; then
+        log_info "No JavaScript/TypeScript files staged for commit"
+        exit 0
+    fi
+
+    log_info "Checking ${#staged_files[@]} files: ${staged_files[*]}"
+    echo
+
+    # Run all quality checks
+    local checks_passed=true
+
+    if ! run_linter "${staged_files[@]}"; then
+        checks_passed=false
+    fi
+
+    if ! run_formatter "${staged_files[@]}"; then
+        checks_passed=false
+    fi
+
+    if ! run_type_check; then
+        checks_passed=false
+    fi
+
+    run_security_check
+    run_performance_check
+
+    echo
+    if [ "$checks_passed" = true ]; then
+        log_success "🎉 All quality checks passed! Your code is ready for commit."
+        exit 0
+    else
+        log_error "❌ Quality checks failed. Please fix the issues and try again."
+        echo
+        log_info "Quick fixes:"
+        echo "  • Run 'npx eslint --fix' to auto-fix linting issues"
+        echo "  • Run 'npx prettier --write' to auto-format code"
+        echo "  • Check TypeScript errors with 'npx tsc --noEmit'"
+        exit 1
+    fi
+}
+
+# Run main function
+main "$@"