-
Notifications
You must be signed in to change notification settings - Fork 0
Expand file tree
/
Copy path.coderabbit.yaml
More file actions
98 lines (80 loc) · 2.23 KB
/
Copy path.coderabbit.yaml
File metadata and controls
98 lines (80 loc) · 2.23 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
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
language: "ja-JP"
early_access: false
reviews:
profile: "chill"
high_level_summary: true
high_level_summary_placeholder: "@coderabbitai summary"
review_status: true
poem: false
auto_review:
enabled: true
drafts: false
auto_review_base_branches:
- main
- develop
request_changes_workflow: true
collapse_walkthrough: true
path_instructions:
- path: "frontend/**"
instructions: |
React / Next.js / TypeScript のベストプラクティスでレビューしてください。
優先順位:
1. バグになる可能性
2. パフォーマンス
3. 保守性
特に確認してください。
- Hooksの依存配列
- useMemo/useCallbackの不要・不足
- 再レンダリング
- keyの付け忘れ
- Suspense/Server Componentの利用
- anyの使用
- null/undefined安全性
- Tailwindの冗長なclass
- 命名規則
- 重複コード
- path: "backend/**"
instructions: |
NestJS / TypeScript のベストプラクティスでレビューしてください。
優先順位:
1. バグ
2. セキュリティ
3. 保守性
特に確認してください。
- Exception Filterの利用
- エラーハンドリング
- Promise漏れ
- await漏れ
- return漏れ
- Transaction漏れ
- SQL Injection
- DTO Validation
- Logger利用
- DIの適切性
- 重複コード
- path: "**/*.test.*"
instructions: |
テストコードでは以下を確認してください。
- テストケース不足
- 境界値
- Happy Pathだけになっていないか
- テスト名が分かりやすいか
- path: ".github/workflows/**"
instructions: |
GitHub Actionsでは
- Secret漏洩
- 権限(permission)
- Cache利用
- 不要なJob
を確認してください。
chat:
auto_reply: true
knowledge_base:
issues:
scope: "auto"
pull_requests:
scope: "auto"
code_guidelines:
enabled: true
filePatterns:
- ".coderabbit/code-guidelines.md"