|
| 1 | +# .coderabbit.yaml |
| 2 | +language: 'en-US' |
| 3 | +early_access: false |
| 4 | + |
| 5 | +chat: |
| 6 | + auto_reply: true |
| 7 | + |
| 8 | +reviews: |
| 9 | + poem: false |
| 10 | + profile: 'chill' # not too strict |
| 11 | + high_level_summary: true |
| 12 | + review_status: true |
| 13 | + commit_status: true |
| 14 | + collapse_walkthrough: false |
| 15 | + |
| 16 | + # Scope: review code, skip docs/config/assets |
| 17 | + path_filters: |
| 18 | + - '**/*' |
| 19 | + - '!.github/**' |
| 20 | + - '!**/*.svg' |
| 21 | + |
| 22 | + # Repo-specific guidance |
| 23 | + instructions: | |
| 24 | + Focus on correctness, readability, and maintainability over cosmetic nitpicks. |
| 25 | + - Avoid reviewing README, config, or asset files. |
| 26 | + - Prioritize identifying bugs, performance issues, and architectural concerns. |
| 27 | + - For Angular, check for OnPush strategy, unsubscribe patterns, proper RxJS usage, |
| 28 | + and template accessibility (labels, roles, tab order). |
| 29 | + - For TypeScript, enforce strong typing and minimal use of `any`. |
| 30 | + - Point out missing tests only when changes are medium/high risk. |
| 31 | +
|
| 32 | + # Use path + instructions |
| 33 | + path_instructions: |
| 34 | + - path: 'src/app/**' |
| 35 | + instructions: | |
| 36 | + For Angular code: verify component separation, trackBy on *ngFor, |
| 37 | + strict type safety, and clean observable patterns. |
| 38 | + - path: '**/*.spec.ts' |
| 39 | + instructions: | |
| 40 | + For tests: encourage clear Arrange-Act-Assert structure, |
| 41 | + stable selectors, and minimal brittle timing dependencies. |
| 42 | +
|
| 43 | + auto_review: |
| 44 | + enabled: true |
| 45 | + auto_incremental_review: true |
| 46 | + drafts: false |
| 47 | + base_branches: ['dev'] |
| 48 | + ignore_title_keywords: ['WIP', 'Draft'] |
| 49 | + labels: ['!skip-ai-review'] |
| 50 | + ignore_usernames: ['dependabot', 'renovate[bot]'] |
| 51 | + |
| 52 | + finishing_touches: |
| 53 | + docstrings: |
| 54 | + enabled: true |
| 55 | + unit_tests: |
| 56 | + enabled: true |
| 57 | + |
| 58 | +# Valid schema: turn off gates entirely |
| 59 | +pre_merge_checks: |
| 60 | + mode: 'off' |
0 commit comments