Add GitHub workflows and fix Vercel build issue

[Rice-Cameron]

Summary

• Add comprehensive GitHub workflows for automated PR quality gates
• Fix ESLint build error that was causing Vercel deployment failures
• Implement CI/CD pipeline with database testing, linting, and security checks

🚀 GitHub Workflows Added

pr-checks.yml - Essential PR Validation

• Automatic linting, testing, and build verification on all PRs to main
• PostgreSQL service integration for isolated database testing
• Ensures code quality before merge

ci.yml - Comprehensive CI Pipeline

• Quality Assurance: ESLint, TypeScript checking, full test suite
• Security Audit: npm vulnerability scanning
• Database Safety: Validates DATABASE_MODE system functionality
• Runs on both main branch pushes and pull requests

quality-gate.yml - Enhanced PR Experience

• Visual feedback with emojis and organized output
• Automatic quality summary reports in PR comments
• Comprehensive validation with clear pass/fail indicators

🐛 Critical Fix: Vercel Build Issue

Problem: Vercel builds were failing with ESLint error:

./src/app/api/problems/route.test.ts
2:32  Error: 'vi' is defined but never used.  @typescript-eslint/no-unused-vars

Root Cause: After consolidating auth mocking utilities, the vi import became unused in route.test.ts

Solution: Removed unused vi import while preserving necessary Vitest imports (describe, it, expect)

🔧 Workflow Features

Database Safety

• PostgreSQL service with proper DATABASE_MODE=3 for testing
• Isolated test database prevents production contamination
• Automated schema setup via npm run test:db

Code Quality Assurance

• ESLint enforcement for consistent code style
• TypeScript type checking for type safety
• Complete test suite execution (unit + integration tests)
• Production build verification

Security & Reliability

• npm security audit scanning for vulnerabilities
• Dummy environment variables for safe CI builds
• No sensitive data exposure in workflow logs

Developer Experience

• Clear visual feedback with status indicators
• Organized output with collapsible sections
• Automatic quality summaries in PR descriptions

Test Plan

• Local build passes without ESLint errors
• All existing tests continue to pass
• Workflows properly validate PR requirements
• PostgreSQL service integration works correctly
• Security audit completes successfully
• Production build succeeds with dummy credentials

Impact

• Prevents Build Failures: Catches issues before they reach Vercel
• Enforces Quality: Automated linting and testing on every PR
• Improves Security: Regular vulnerability scanning
• Enhances DX: Clear feedback and validation status

🤖 Generated with Claude Code