diff options
Diffstat (limited to 'frameworks/nextjs-15/.claude/hooks/pre-commit-validation.sh')
| -rw-r--r-- | frameworks/nextjs-15/.claude/hooks/pre-commit-validation.sh | 93 |
1 files changed, 93 insertions, 0 deletions
diff --git a/frameworks/nextjs-15/.claude/hooks/pre-commit-validation.sh b/frameworks/nextjs-15/.claude/hooks/pre-commit-validation.sh new file mode 100644 index 0000000..c005611 --- /dev/null +++ b/frameworks/nextjs-15/.claude/hooks/pre-commit-validation.sh @@ -0,0 +1,93 @@ +#!/bin/bash + +# Next.js 15 Pre-commit Validation Hook +# This script validates Next.js code before allowing commits + +set -e + +echo "๐ Running Next.js pre-commit validation..." + +# Check for common Next.js 15 issues +check_nextjs_patterns() { + local file="$1" + local errors=0 + + # Check for incorrect async params/searchParams usage + if grep -q "params\." "$file" 2>/dev/null || grep -q "searchParams\." "$file" 2>/dev/null; then + if ! grep -q "await params" "$file" 2>/dev/null && ! grep -q "await searchParams" "$file" 2>/dev/null; then + echo "โ ๏ธ Warning in $file: params and searchParams are Promises in Next.js 15 - use await" + errors=$((errors + 1)) + fi + fi + + # Check for useState/useEffect in files without 'use client' + if grep -E "(useState|useEffect|useReducer|useCallback|useMemo)" "$file" >/dev/null 2>&1; then + if ! grep -q "^'use client'" "$file" && ! grep -q '^"use client"' "$file"; then + echo "โ ๏ธ Warning in $file: Client hooks found without 'use client' directive" + errors=$((errors + 1)) + fi + fi + + # Check for 'use server' at file level vs function level + if grep -q "'use server'" "$file" 2>/dev/null; then + line_num=$(grep -n "'use server'" "$file" | head -1 | cut -d: -f1) + if [ "$line_num" -ne 1 ]; then + echo "โน๏ธ Info in $file: 'use server' found inside file - consider file-level directive" + fi + fi + + # Check for process.env usage in Client Components + if grep -q "'use client'" "$file" 2>/dev/null || grep -q '"use client"' "$file" 2>/dev/null; then + if grep -q "process\.env\." "$file" 2>/dev/null; then + if ! grep -q "process\.env\.NEXT_PUBLIC_" "$file" 2>/dev/null; then + echo "โ ๏ธ Warning in $file: Non-public env vars in Client Component" + errors=$((errors + 1)) + fi + fi + fi + + return $errors +} + +# Find all TypeScript/JavaScript files in app directory +total_errors=0 +for file in $(find app -type f \( -name "*.tsx" -o -name "*.ts" -o -name "*.jsx" -o -name "*.js" \) 2>/dev/null || true); do + if [ -f "$file" ]; then + check_nextjs_patterns "$file" || total_errors=$((total_errors + $?)) + fi +done + +# Check for missing error boundaries +if [ -d "app" ]; then + routes=$(find app -type f -name "page.tsx" -o -name "page.jsx" -o -name "page.ts" -o -name "page.js" | xargs -I {} dirname {}) + for route in $routes; do + if [ ! -f "$route/error.tsx" ] && [ ! -f "$route/error.jsx" ] && [ ! -f "$route/error.ts" ] && [ ! -f "$route/error.js" ]; then + echo "โน๏ธ Info: No error boundary in $route/" + fi + done +fi + +# Run type checking if TypeScript is configured +if [ -f "tsconfig.json" ]; then + echo "๐ Running TypeScript type check..." + npx tsc --noEmit || { + echo "โ TypeScript errors found" + exit 1 + } +fi + +# Run Next.js linting if configured +if [ -f ".eslintrc.json" ] || [ -f ".eslintrc.js" ]; then + echo "๐งน Running Next.js ESLint..." + npm run lint || { + echo "โ ESLint errors found" + exit 1 + } +fi + +if [ $total_errors -gt 0 ]; then + echo "โ Found $total_errors potential issues. Please review before committing." + exit 1 +else + echo "โ
Next.js validation passed!" +fi
\ No newline at end of file |