git-review-hotfix

Hotfix Review and Validation Checklist

This prompt provides comprehensive validation of hotfix implementation ensuring all critical requirements are met before commit and deployment.

🎯 HOTFIX REVIEW OBJECTIVES:

1. VALIDATE fix implementation meets hotfix standards and scope
2. VERIFY comprehensive testing including edge cases and regression
3. CONFIRM documentation is complete and accurate
4. ASSESS deployment readiness with proper monitoring and rollback plans
5. ENSURE compliance with security, performance, and business requirements

🔍 HOTFIX IMPLEMENTATION REVIEW:

STEP 1: Fix Scope and Implementation Validation

Hotfix Scope Compliance:

# Verify we're on the correct hotfix branch
CURRENT_BRANCH=$(git branch --show-current)
if [[ ! "$CURRENT_BRANCH" =~ ^hotfix/ ]]; then
    echo "❌ ERROR: Not on a hotfix branch. Current: $CURRENT_BRANCH"
    exit 1
fi

echo "✅ Validated hotfix branch: $CURRENT_BRANCH"

# Check commit count since branch creation
COMMIT_COUNT=$(git rev-list --count HEAD ^main)
echo "📊 Commits in hotfix: $COMMIT_COUNT"

if [ $COMMIT_COUNT -gt 5 ]; then
    echo "⚠️  WARNING: High commit count ($COMMIT_COUNT) - ensure minimal scope"
fi

Code Change Analysis:

# Analyze changed files
echo "📁 Files changed in hotfix:"
git diff --name-only main..HEAD

# Show detailed changes
echo "📝 Detailed changes:"
git diff main..HEAD --stat

# Check for large changes
LINES_CHANGED=$(git diff main..HEAD --shortstat | grep -o '[0-9]\+ insertions\|[0-9]\+ deletions' | grep -o '[0-9]\+' | paste -sd+ | bc 2>/dev/null || echo "0")
echo "📊 Total lines changed: $LINES_CHANGED"

if [ $LINES_CHANGED -gt 100 ]; then
    echo "⚠️  WARNING: Large change set ($LINES_CHANGED lines) - validate necessity"
fi

Hotfix Scope Validation Checklist:

• Minimal changes only - Fix addresses only the critical issue
• No feature additions - No new functionality introduced
• No refactoring - Code structure improvements avoided
• Focused scope - Changes limited to affected components
• Existing patterns preserved - Current code patterns maintained
• Configuration minimal - Only essential config changes
• Documentation proportional - Documentation matches change scope