Skip to content
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
43 changes: 16 additions & 27 deletions .coderabbit.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand All @@ -105,9 +101,9 @@ reviews:
hadolint:
enabled: true
swiftlint:
enabled: true
enabled: false
phpstan:
enabled: true
enabled: false
level: default
golangci-lint:
enabled: true
Expand All @@ -118,7 +114,7 @@ reviews:
checkov:
enabled: true
detekt:
enabled: true
enabled: false
eslint:
enabled: true

Expand Down Expand Up @@ -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
Expand Down Expand Up @@ -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
Expand Down
Loading