@paulduvall/claude-dev-toolkit 0.0.1-alpha.2 → 0.0.1-alpha.21

package/hooks/on-error-debug.sh

@@ -0,0 +1,226 @@
+#!/usr/bin/env bash
+set -euo pipefail
+
+# Claude Code Hook: Error Handling Trigger
+# 
+# Purpose: Lightweight trigger for automatic debugging assistance on errors
+# Trigger: Manual invocation or PostToolUse (no OnError hook event exists in Claude Code)
+# Approach: Capture error context and delegate to debug-specialist subagent
+#
+# This hook provides immediate debugging help by gathering error context
+# and connecting users with the debug-specialist subagent.
+
+##################################
+# Load Shared Libraries
+##################################
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+LIB_DIR="$SCRIPT_DIR/lib"
+
+# Load essential modules for error context gathering
+source "$LIB_DIR/config-constants.sh" 
+source "$LIB_DIR/context-manager.sh"
+source "$LIB_DIR/error-handler.sh"
+source "$LIB_DIR/file-utils.sh"
+
+##################################
+# Error Context Gathering
+##################################
+gather_error_context() {
+    local error_type="${1:-unknown}"
+    local error_message="${2:-No error message provided}"
+    local failed_command="${3:-unknown}"
+    
+    log_info "Gathering error context for debugging assistance"
+    
+    # Capture comprehensive error context
+    local safe_type safe_msg safe_cmd safe_user safe_wd safe_branch
+    safe_type=$(json_escape "$error_type")
+    safe_msg=$(json_escape "$error_message")
+    safe_cmd=$(json_escape "$failed_command")
+    safe_user=$(json_escape "$USER")
+    safe_wd=$(json_escape "$(pwd)")
+    safe_branch=$(json_escape "$(git branch --show-current 2>/dev/null || echo 'not-in-git')")
+
+    local context_data
+    context_data=$(cat <<EOF
+{
+  "trigger": "on_error_debug",
+  "error_info": {
+    "type": "$safe_type",
+    "message": "$safe_msg",
+    "failed_command": "$safe_cmd",
+    "timestamp": "$(date -u +"%Y-%m-%dT%H:%M:%SZ")"
+  },
+  "environment": {
+    "tool": "${CLAUDE_TOOL:-unknown}",
+    "file": "${CLAUDE_FILE:-none}",
+    "working_directory": "$safe_wd",
+    "user": "$safe_user",
+    "shell": "${SHELL##*/}",
+    "session_id": "${CLAUDE_SESSION_ID:-$$}"
+  },
+  "project_context": {
+    "git_branch": "$safe_branch",
+    "git_status": "$(git status --porcelain 2>/dev/null | wc -l | tr -d ' ') modified files"
+  },
+  "system_context": {
+    "os": "$(uname -s)"
+  }
+}
+EOF
+    )
+    
+    echo "$context_data"
+}
+
+##################################
+# System Diagnostics
+##################################
+gather_diagnostic_info() {
+    local diagnostic_info=""
+    
+    # Check recent error logs if available
+    if [[ -f "$LOG_FILE" ]]; then
+        local recent_errors
+        recent_errors=$(tail -20 "$LOG_FILE" 2>/dev/null | grep -i error || echo "No recent errors in log")
+        diagnostic_info+="\nRecent log errors:\n$recent_errors"
+    fi
+    
+    # Check system resources
+    if command -v df >/dev/null; then
+        local disk_usage
+        disk_usage=$(df -h . | tail -1 | awk '{print "Disk: " $4 " available"}')
+        diagnostic_info+="\n\nSystem resources:\n$disk_usage"
+    fi
+    
+    # Check for common environment issues
+    if [[ -f "package.json" ]] && command -v npm >/dev/null; then
+        diagnostic_info+="\nNode.js project detected"
+        diagnostic_info+="\nNPM version: $(npm --version 2>/dev/null || echo 'npm not available')"
+    fi
+    
+    if [[ -f "requirements.txt" ]] && command -v python >/dev/null; then
+        diagnostic_info+="\nPython project detected"  
+        diagnostic_info+="\nPython version: $(python --version 2>/dev/null || echo 'python not available')"
+    fi
+    
+    echo "$diagnostic_info"
+}
+
+##################################
+# Subagent Delegation  
+##################################
+delegate_to_debug_subagent() {
+    local context="$1"
+    local diagnostics="$2"
+    local error_message="$3"
+    
+    log_info "Delegating error analysis to debug-specialist subagent"
+    
+    echo "🐛 ERROR DETECTED: Automatic debugging assistance activated"
+    echo ""
+    echo "Error Details:"
+    echo "  Message: $error_message"
+    echo "  Tool: ${CLAUDE_TOOL:-unknown}"
+    echo "  File: ${CLAUDE_FILE:-none}"
+    echo "  Time: $(date)"
+    echo ""
+    
+    if [[ -n "$diagnostics" ]]; then
+        echo "System Diagnostics:"
+        echo "$diagnostics"
+        echo ""
+    fi
+    
+    echo "Context for debug-specialist subagent:"
+    echo "$context" | jq . 2>/dev/null || echo "$context"
+    echo ""
+    echo "🔍 DEBUG REQUEST:"
+    echo "Please analyze the error above and provide:"
+    echo "- Root cause analysis"
+    echo "- Step-by-step troubleshooting approach"  
+    echo "- Specific commands or fixes to try"
+    echo "- Prevention strategies for similar issues"
+    echo ""
+    echo "Type your analysis or questions about this error."
+}
+
+##################################
+# Quick Error Classification
+##################################
+classify_error() {
+    local error_message="$1"
+    local error_type="general"
+    
+    # Basic error pattern matching for better context
+    case "$error_message" in
+        *"permission denied"*|*"Permission denied"*)
+            error_type="permission_error"
+            ;;
+        *"command not found"*|*"Command not found"*)
+            error_type="command_not_found"
+            ;;
+        *"No such file"*|*"cannot find"*)
+            error_type="file_not_found" 
+            ;;
+        *"syntax error"*|*"Syntax error"*)
+            error_type="syntax_error"
+            ;;
+        *"connection"*|*"network"*|*"timeout"*)
+            error_type="network_error"
+            ;;
+        *"import"*|*"module"*|*"package"*)
+            error_type="dependency_error"
+            ;;
+        *)
+            error_type="general_error"
+            ;;
+    esac
+    
+    echo "$error_type"
+}
+
+##################################
+# Main Hook Logic  
+##################################
+main() {
+    # Initialize error handling (ironic for an error hook, but important!)
+    initialize_error_handling || {
+        echo "ERROR: Failed to initialize error handling in error hook" >&2
+        exit 1
+    }
+    
+    local error_message="${1:-Unknown error occurred}"
+    local failed_command="${2:-unknown}"
+    
+    log_info "Error hook triggered: $error_message"
+    
+    # Classify the error type
+    local error_type
+    error_type=$(classify_error "$error_message")
+    
+    # Gather comprehensive error context
+    local context
+    context=$(gather_error_context "$error_type" "$error_message" "$failed_command")
+    
+    # Gather additional diagnostic information
+    local diagnostics
+    diagnostics=$(gather_diagnostic_info)
+    
+    # Delegate to debug-specialist subagent
+    delegate_to_debug_subagent "$context" "$diagnostics" "$error_message"
+    
+    log_info "Debug delegation completed for error: $error_type"
+}
+
+##################################
+# Execute Hook
+##################################
+# Handle both direct invocation and error trap scenarios
+if [[ "${1:-}" == "--trap" ]]; then
+    # Called from an error trap
+    main "Script error at line ${2:-unknown}" "${3:-unknown command}"
+else
+    # Called directly with error information
+    main "$@"
+fi

package/hooks/pre-commit-quality.sh

@@ -0,0 +1,204 @@
+#!/usr/bin/env bash
+set -euo pipefail
+
+# Claude Code Hook: Pre-Commit Quality Check
+# 
+# Purpose: Lightweight trigger for code quality validation before git commits
+# Trigger: Custom hook for pre-commit operations
+# Approach: Quick validation with style-enforcer subagent delegation
+#
+# This hook ensures code quality by leveraging AI analysis rather than 
+# running multiple linting tools manually.
+
+##################################
+# Load Shared Libraries  
+##################################
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+LIB_DIR="$SCRIPT_DIR/lib"
+
+# Load essential modules for lightweight operation
+source "$LIB_DIR/config-constants.sh"
+source "$LIB_DIR/context-manager.sh"
+source "$LIB_DIR/error-handler.sh"
+source "$LIB_DIR/file-utils.sh"
+
+##################################
+# Git Context Analysis
+##################################
+analyze_staged_changes() {
+    log_info "Analyzing staged changes for quality check"
+    
+    # Check if we're in a git repository
+    if ! git rev-parse --git-dir >/dev/null 2>&1; then
+        log_debug "Not in a git repository, skipping pre-commit quality check"
+        return 0
+    fi
+    
+    # Get staged files
+    local staged_files
+    staged_files=$(git diff --cached --name-only 2>/dev/null || echo "")
+    
+    if [[ -z "$staged_files" ]]; then
+        log_info "No staged changes found, skipping quality check"
+        return 0  
+    fi
+    
+    echo "$staged_files"
+}
+
+##################################
+# Quality Context Gathering
+##################################
+gather_quality_context() {
+    local staged_files="$1"
+    
+    # Create context for quality analysis
+    local context_data
+    context_data=$(cat <<EOF
+{
+  "trigger": "pre_commit_quality",
+  "staged_files": [$(echo "$staged_files" | sed 's/.*/"&"/' | paste -sd ',' -)],
+  "git_info": {
+    "branch": "$(git branch --show-current 2>/dev/null || echo 'unknown')",
+    "commit_hash": "$(git rev-parse HEAD 2>/dev/null || echo 'initial')",
+    "status": "$(git status --porcelain --cached | wc -l | tr -d ' ') files staged"
+  },
+  "project_info": {
+    "working_directory": "$(pwd)",
+    "user": "$USER", 
+    "timestamp": "$(date -u +"%Y-%m-%dT%H:%M:%SZ")"
+  },
+  "file_analysis": {
+    "total_files": $(echo "$staged_files" | wc -l | tr -d ' '),
+    "file_types": $(echo "$staged_files" | sed 's/.*\.//' | sort | uniq -c | jq -R -s 'split("\n")[:-1] | map(split(" ")[-1]) | unique')
+  }
+}
+EOF
+    )
+    
+    echo "$context_data"
+}
+
+##################################
+# Subagent Delegation
+##################################
+delegate_to_quality_subagent() {
+    local context="$1" 
+    local staged_files="$2"
+    
+    log_info "Delegating quality analysis to style-enforcer subagent"
+    
+    echo "📊 QUALITY CHECK: Analyzing staged changes before commit"
+    echo ""
+    echo "Staged files for analysis:"
+    echo "$staged_files" | sed 's/^/  - /'
+    echo ""
+    echo "Context for style-enforcer subagent:"
+    echo "$context" | jq . 2>/dev/null || echo "$context"
+    echo ""
+    echo "Please analyze the staged changes for:"
+    echo "- Code formatting and style consistency"
+    echo "- Import organization and unused imports" 
+    echo "- Type annotations and documentation"
+    echo "- Best practices and code patterns"
+    echo "- Performance considerations"
+    echo ""
+    echo "Provide specific feedback or type 'approved' if quality standards are met."
+}
+
+##################################
+# Quick File Validation
+##################################
+validate_basic_file_integrity() {
+    local staged_files="$1"
+    
+    log_debug "Performing basic file integrity checks"
+    
+    # Check for common issues that can be caught quickly
+    local issues=()
+    
+    while IFS= read -r file; do
+        [[ -z "$file" ]] && continue
+        
+        # Skip non-existent files (deletions)
+        [[ ! -f "$file" ]] && continue
+        
+        # Check for basic syntax issues in common file types
+        case "$file" in
+            *.json)
+                if ! jq . "$file" >/dev/null 2>&1; then
+                    issues+=("Invalid JSON syntax: $file")
+                fi
+                ;;
+            *.yml|*.yaml) 
+                if command -v yq >/dev/null && ! yq . "$file" >/dev/null 2>&1; then
+                    issues+=("Invalid YAML syntax: $file") 
+                fi
+                ;;
+            *.sh)
+                if ! bash -n "$file" 2>/dev/null; then
+                    issues+=("Invalid shell syntax: $file")
+                fi
+                ;;
+        esac
+        
+        # Check for potential credential patterns (quick scan)
+        if grep -qE "(password|api_key|secret|token)\s*[:=]\s*['\"][^'\"]+['\"]" "$file" 2>/dev/null; then
+            issues+=("Potential credential exposure: $file")
+        fi
+        
+    done <<< "$staged_files"
+    
+    # Report immediate issues
+    if [[ ${#issues[@]} -gt 0 ]]; then
+        echo "⚠️  IMMEDIATE ISSUES FOUND:"
+        printf '%s\n' "${issues[@]}" | sed 's/^/  - /'
+        echo ""
+        echo "Please fix these issues before committing."
+        return 1
+    fi
+    
+    return 0
+}
+
+##################################
+# Main Hook Logic
+##################################
+main() {
+    # Initialize error handling
+    initialize_error_handling || {
+        echo "ERROR: Failed to initialize error handling" >&2
+        exit 1
+    }
+    
+    log_info "Pre-commit quality check initiated"
+    
+    # Analyze what's being committed
+    local staged_files
+    staged_files=$(analyze_staged_changes)
+    
+    # Exit early if no staged changes
+    if [[ -z "$staged_files" ]]; then
+        exit 0
+    fi
+    
+    # Perform quick validation checks
+    if ! validate_basic_file_integrity "$staged_files"; then
+        log_error "Basic file integrity checks failed"
+        exit 1
+    fi
+    
+    # Gather context for subagent analysis
+    local context
+    context=$(gather_quality_context "$staged_files")
+    
+    # Delegate to quality subagent for comprehensive analysis
+    delegate_to_quality_subagent "$context" "$staged_files"
+    
+    log_info "Quality check delegation completed"
+}
+
+##################################
+# Execute Hook
+##################################
+main "$@"

package/hooks/pre-commit-test-runner.sh

@@ -0,0 +1,132 @@
+#!/usr/bin/env bash
+set -euo pipefail
+
+# Claude Code Hook: Pre-Commit Test Runner
+#
+# Description: Auto-detects test framework and runs tests before commits
+# Purpose: Prevent commits with failing tests by validating the full test suite
+# Trigger: PreToolUse
+# Blocking: Yes
+# Tools: Bash
+# Author: Claude Dev Toolkit
+# Version: 1.0.0
+# Category: security
+#
+# This hook validates code quality by running the project's test suite
+# before allowing commit operations to proceed.
+
+##################################
+# Configuration
+##################################
+HOOK_NAME="pre-commit-test-runner"
+LOG_FILE="$HOME/.claude/logs/pre-commit-test-runner.log"
+
+mkdir -p "$(dirname "$LOG_FILE")"
+chmod 700 "$(dirname "$LOG_FILE")"
+touch "$LOG_FILE"
+chmod 600 "$LOG_FILE"
+
+##################################
+# Logging
+##################################
+log() {
+    echo "[$(date +'%Y-%m-%d %H:%M:%S')] [$HOOK_NAME] $*" >> "$LOG_FILE"
+}
+
+##################################
+# Commit Detection
+##################################
+is_commit_command() {
+    local tool="${CLAUDE_TOOL:-}"
+    local content="${CLAUDE_CONTENT:-}"
+
+    if [[ "$tool" != "Bash" ]]; then
+        return 1
+    fi
+
+    echo "$content" | grep -qE 'git\s+commit' || return 1
+}
+
+##################################
+# Framework Detection
+##################################
+detect_framework() {
+    if [[ -f "pytest.ini" ]] || { [[ -f "pyproject.toml" ]] && grep -q '\[tool.pytest' pyproject.toml 2>/dev/null; }; then
+        echo "pytest"
+    elif [[ -f "package.json" ]] && grep -q '"test"' package.json 2>/dev/null; then
+        if grep -qE '"(jest|vitest)' package.json 2>/dev/null; then
+            echo "jest"
+        else
+            echo "npm-test"
+        fi
+    elif [[ -f "go.mod" ]]; then
+        echo "go-test"
+    elif [[ -f "Cargo.toml" ]]; then
+        echo "cargo-test"
+    elif [[ -f "mix.exs" ]]; then
+        echo "mix-test"
+    else
+        echo "none"
+    fi
+}
+
+##################################
+# Test Execution
+##################################
+run_tests() {
+    local framework="$1"
+    log "Running tests with framework: $framework"
+
+    case "$framework" in
+        pytest)     python -m pytest --tb=short 2>&1 ;;
+        jest)       npx jest --ci 2>&1 ;;
+        npm-test)   npm test 2>&1 ;;
+        go-test)    go test ./... 2>&1 ;;
+        cargo-test) cargo test 2>&1 ;;
+        mix-test)   mix test 2>&1 ;;
+        *)          echo "No test framework detected" ;;
+    esac
+}
+
+##################################
+# Main Hook Logic
+##################################
+main() {
+    if ! is_commit_command; then
+        log "Not a commit command, skipping"
+        exit 0
+    fi
+
+    log "Commit detected, checking for test framework"
+    local framework
+    framework=$(detect_framework)
+
+    if [[ "$framework" == "none" ]]; then
+        log "No test framework detected, allowing commit"
+        exit 0
+    fi
+
+    log "Detected framework: $framework"
+    local output
+    output=$(run_tests "$framework") || {
+        local exit_code=$?
+        echo "Test validation failed — commit blocked"
+        echo "Framework: $framework"
+        echo "$output" | tail -10
+        log "Tests FAILED (exit $exit_code)"
+        exit 1
+    }
+
+    log "Security check passed — all tests green"
+    exit 0
+}
+
+##################################
+# Error Handling
+##################################
+trap 'log "Hook failed with error on line $LINENO"' ERR
+
+##################################
+# Execute
+##################################
+main "$@"

package/hooks/pre-write-security.sh

@@ -0,0 +1,115 @@
+#!/usr/bin/env bash
+set -euo pipefail
+
+# Claude Code Hook: Pre-Write Security Check
+# 
+# Purpose: Lightweight trigger for security scanning before file modifications
+# Trigger: PreToolUse for Edit, Write, MultiEdit tools
+# Approach: Gather context and delegate complex logic to security-auditor subagent
+#
+# This hook provides immediate security feedback by leveraging AI reasoning
+# rather than complex bash pattern matching.
+
+##################################
+# Load Shared Libraries
+##################################
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+LIB_DIR="$SCRIPT_DIR/lib"
+
+# Load only essential modules for lightweight operation
+source "$LIB_DIR/config-constants.sh"
+source "$LIB_DIR/file-utils.sh"
+source "$LIB_DIR/context-manager.sh"
+source "$LIB_DIR/error-handler.sh"
+
+##################################
+# Simple Context Gathering
+##################################
+gather_security_context() {
+    local tool="${CLAUDE_TOOL:-unknown}"
+    local file="${CLAUDE_FILE:-none}"
+    
+    log_info "Pre-write security check triggered for: $tool on $file"
+    
+    # Create lightweight context for subagent
+    local safe_tool safe_file safe_user safe_wd safe_branch
+    safe_tool=$(json_escape "$tool")
+    safe_file=$(json_escape "$file")
+    safe_user=$(json_escape "$USER")
+    safe_wd=$(json_escape "$(pwd)")
+    safe_branch=$(json_escape "$(git branch --show-current 2>/dev/null || echo 'not-in-git')")
+
+    local context_data
+    context_data=$(cat <<EOF
+{
+  "trigger": "pre_write_security",
+  "tool": "$safe_tool",
+  "file": "$safe_file",
+  "timestamp": "$(date -u +"%Y-%m-%dT%H:%M:%SZ")",
+  "user": "$safe_user",
+  "working_directory": "$safe_wd",
+  "git_branch": "$safe_branch",
+  "session_id": "${CLAUDE_SESSION_ID:-$$}"
+}
+EOF
+    )
+    
+    echo "$context_data"
+}
+
+##################################
+# Subagent Delegation
+##################################
+delegate_to_security_subagent() {
+    local context="$1"
+    
+    # Log the delegation
+    log_info "Delegating security analysis to security-auditor subagent"
+    
+    # In the hybrid approach, we provide context and let Claude Code 
+    # handle the subagent execution through its native mechanisms
+    echo "🔒 SECURITY CHECK: Analyzing $CLAUDE_TOOL operation on $CLAUDE_FILE"
+    echo ""
+    echo "Context for security-auditor subagent:"
+    echo "$context" | jq . 2>/dev/null || echo "$context"
+    echo ""
+    echo "Please review the operation above for:"
+    echo "- Credential exposure (API keys, passwords, tokens)" 
+    echo "- Security vulnerabilities (SQL injection, XSS, etc.)"
+    echo "- Sensitive data handling"
+    echo "- Access control issues"
+    echo ""
+    echo "Type 'continue' if the operation is secure, or provide specific security concerns."
+}
+
+##################################
+# Main Hook Logic
+##################################
+main() {
+    # Initialize error handling
+    initialize_error_handling || {
+        echo "ERROR: Failed to initialize error handling" >&2
+        exit 1
+    }
+    
+    # Gather context for security analysis
+    local context
+    context=$(gather_security_context)
+    
+    # Check if this is a security-relevant operation
+    case "${CLAUDE_TOOL:-}" in
+        Edit|Write|MultiEdit)
+            log_debug "Security-relevant tool detected: $CLAUDE_TOOL"
+            delegate_to_security_subagent "$context"
+            ;;
+        *)
+            log_debug "Non-security-relevant tool: ${CLAUDE_TOOL:-unknown}"
+            exit 0
+            ;;
+    esac
+}
+
+##################################
+# Execute Hook
+##################################
+main "$@"