blob: c005611c490f171cb876edece66e48e0f419e512 (
plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
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
|
