-
Notifications
You must be signed in to change notification settings - Fork 0
Expand file tree
/
Copy path.coderabbit.yaml
More file actions
55 lines (55 loc) · 2.16 KB
/
.coderabbit.yaml
File metadata and controls
55 lines (55 loc) · 2.16 KB
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
# yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json
language: 'en'
reviews:
profile: 'assertive'
high_level_summary: false
changed_files_summary: true
poem: false
review_status: false
collapse_walkthrough: true
request_changes_workflow: true
sequence_diagrams: false
related_issues: false
related_prs: false
suggested_labels: false
assess_linked_issues: false
path_instructions:
- path: '**/*'
instructions: |
Read AGENTS.md at the repo root first. It defines our architecture,
testing approach (AAA pattern, plain-language behavior descriptions,
test every .ts logic file, skip pure components), component-should-be-dumb
rule (logic in hooks), folder placement rules, env-var policy (all vars
in .env, accessed via ConfigService), dependency-justification policy,
and code quality principles. Enforce every rule in AGENTS.md during review.
- path: '**/*.ts,**/*.tsx'
instructions: |
Review the TypeScript/React code for:
- Possible bugs and unhandled edge cases
- Performance issues and unnecessary re-renders
- Type safety improvements
- Clean code principles and readability
- Security vulnerabilities
- Components holding logic that should live in a hook (see AGENTS.md "Components must be dumb")
- Direct `import.meta.env` usage outside ConfigService
- Hardcoded user-facing strings (must go through i18n)
- New dependencies without justification in the PR description
- path: '**/*.test.ts,**/*.test.tsx'
instructions: |
Review tests for:
- Missing edge case coverage
- Proper assertion usage
- Test isolation and mocking correctness
- AAA structure (Arrange / Act / Assert) as defined in AGENTS.md
- Descriptions written in plain behavior language ("When X, then Y"),
without variable/function/type names
- Tests that verify purpose/behavior, not implementation details
auto_review:
enabled: true
drafts: false
tools:
eslint:
enabled: true
github-checks:
enabled: true
timeout_ms: 90000