diff --git a/.coderabbit.yaml b/.coderabbit.yaml index f69abfed..351958b2 100644 --- a/.coderabbit.yaml +++ b/.coderabbit.yaml @@ -79,10 +79,6 @@ reviews: # Exclude these paths from reviews (build artifacts and dependencies) path_filters: - "!**/node_modules/**" # npm dependencies - - "!**/android/**" # Native Android build files - - "!**/ios/**" # Native iOS build files - - "!**/.expo/**" # Expo build cache - - "!**/.expo-shared/**" # Expo shared config - "!**/dist/**" # Build output # Use the following tools when reviewing @@ -105,9 +101,9 @@ reviews: hadolint: enabled: true swiftlint: - enabled: true + enabled: false phpstan: - enabled: true + enabled: false level: default golangci-lint: enabled: true @@ -118,7 +114,7 @@ reviews: checkov: enabled: true detekt: - enabled: true + enabled: false eslint: enabled: true @@ -166,24 +162,17 @@ reviews: # Custom review instructions for specific file patterns path_instructions: - # TypeScript/JavaScript files - - path: "**/*.{ts,tsx,js,jsx}" + # JavaScript/React files + - path: "**/*.{js,jsx}" instructions: | - NextJS: - - Ensure that "use client" is being used - - Ensure that only features that allow pure client-side rendering are used - - NextJS best practices (including file structure, API routes, and static generation methods) are used. - - TypeScript: - - Avoid 'any', use explicit types - - Prefer 'import type' for type imports - - Review for significant deviations from Google JavaScript style guide. Minor style issues are not a priority - - The code adheres to best practices associated with React - - The code adheres to best practices associated with React PWA - - The code adheres to best practices associated with SPA - - The code adheres to best practices recommended by lighthouse or similar tools for performance - - The code adheres to best practices associated with Node.js - - The code adheres to best practices recommended for performance + React/Vite SPA: + - The code is a Vite/React Single Page Application (SPA). Do not enforce or suggest Next.js directives like "use client". + - Do not enforce TypeScript rules (like avoiding 'any' or 'import type') as this codebase uses standard JavaScript. + - Review for significant deviations from Google JavaScript style guide. Minor style issues are not a priority. + - The code adheres to best practices associated with React. + - The code adheres to best practices associated with React PWA and SPA. + - The code adheres to best practices recommended by lighthouse or similar tools for performance. + - The code adheres to best practices associated with Node.js. Security: - No exposed API keys or sensitive data @@ -245,12 +234,12 @@ reviews: - Front-running risks where applicable - # Javascript/Typescript test files - - path: "**/*.test.{ts,tsx,js,jsx}" + # Javascript test files + - path: "**/*.test.{js,jsx}" instructions: | Review test files for: - Comprehensive coverage of component behavior - - Proper use of @testing-library/react-native + - Proper use of Jest and standard React testing libraries like @testing-library/react - Async behavior is properly tested - Accessibility testing is included - Test descriptions are sufficiently detailed to clarify the purpose of each test