diff options
Diffstat (limited to 'frameworks/nextjs-15/.claude/hooks')
| -rw-r--r-- | frameworks/nextjs-15/.claude/hooks/hooks.json | 55 | ||||
| -rw-r--r-- | frameworks/nextjs-15/.claude/hooks/pre-commit-validation.sh | 93 |
2 files changed, 148 insertions, 0 deletions
diff --git a/frameworks/nextjs-15/.claude/hooks/hooks.json b/frameworks/nextjs-15/.claude/hooks/hooks.json new file mode 100644 index 0000000..a93954c --- /dev/null +++ b/frameworks/nextjs-15/.claude/hooks/hooks.json @@ -0,0 +1,55 @@ +{ + "hooks": { + "PreToolUse": [ + { + "matcher": "Edit|MultiEdit|Write", + "hooks": [ + { + "type": "command", + "command": "sh -c 'file=\"$(echo \"$STDIN\" | jq -r .tool_input.file_path)\"; if [[ \"$file\" == *.tsx ]] || [[ \"$file\" == *.jsx ]]; then ext=\"${file##*.}\"; if grep -q \"useState\\|useEffect\\|useReducer\\|useCallback\\|useMemo\" \"$file\" 2>/dev/null; then if ! grep -q \"^['\\\"]use client['\\\"]\" \"$file\" 2>/dev/null; then echo \"โ ๏ธ Warning: Client hooks detected. Add \\'use client\\' directive if needed.\"; fi; fi; fi'" + } + ] + }, + { + "matcher": "Bash", + "hooks": [ + { + "type": "command", + "command": "sh -c 'cmd=\"$(echo \"$STDIN\" | jq -r .tool_input.command)\"; if echo \"$cmd\" | grep -q \"^npm install\\|^yarn add\\|^pnpm add\"; then echo \"๐ฆ Installing dependencies - checking for Next.js compatibility...\"; fi'" + } + ] + } + ], + "PostToolUse": [ + { + "matcher": "Write|MultiEdit", + "hooks": [ + { + "type": "command", + "command": "sh -c 'file=\"$(echo \"$STDIN\" | jq -r .tool_input.file_path)\"; if [[ \"$file\" == app/**/page.tsx ]] || [[ \"$file\" == app/**/page.jsx ]]; then dir=\"$(dirname \"$file\")\"; if [ ! -f \"$dir/loading.tsx\" ] && [ ! -f \"$dir/loading.jsx\" ]; then echo \"๐ก Tip: Consider adding a loading.tsx for better UX\"; fi; if [ ! -f \"$dir/error.tsx\" ] && [ ! -f \"$dir/error.jsx\" ]; then echo \"๐ก Tip: Consider adding an error.tsx for error handling\"; fi; fi'" + } + ] + }, + { + "matcher": "Write|MultiEdit", + "hooks": [ + { + "type": "command", + "command": "sh -c 'file=\"$(echo \"$STDIN\" | jq -r .tool_input.file_path)\"; if [[ \"$file\" == *.ts ]] || [[ \"$file\" == *.tsx ]]; then if which prettier >/dev/null 2>&1; then prettier --write \"$file\" 2>/dev/null || true; fi; fi'" + } + ] + } + ], + "Stop": [ + { + "matcher": "", + "hooks": [ + { + "type": "command", + "command": "sh -c 'if [ -f \"package.json\" ] && [ -d \"app\" ]; then echo \"๐ Next.js Tip: Run \\`npm run dev\\` to start the development server\"; if [ -f \"tsconfig.json\" ]; then echo \"๐ Run \\`npm run type-check\\` to verify TypeScript types\"; fi; fi'" + } + ] + } + ] + } +}
\ No newline at end of file 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 |