Skip to content

Commit b254cec

Browse files
chore: add CodeRabbit configuration for automated code review (Comfy-Org#12539)
Amp-Thread-ID: https://ampcode.com/threads/T-019c7915-2abf-743c-9c74-b93d87d63055 Co-authored-by: Jedrzej Kosinski <kosinkadink1@gmail.com>
1 parent 1bb956f commit b254cec

File tree

1 file changed

+114
-0
lines changed

1 file changed

+114
-0
lines changed

.coderabbit.yaml

Lines changed: 114 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,114 @@
1+
# yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json
2+
language: "en-US"
3+
early_access: false
4+
5+
reviews:
6+
profile: "chill"
7+
request_changes_workflow: false
8+
high_level_summary: false
9+
poem: false
10+
review_status: false
11+
review_details: false
12+
commit_status: true
13+
collapse_walkthrough: true
14+
changed_files_summary: false
15+
sequence_diagrams: false
16+
estimate_code_review_effort: false
17+
assess_linked_issues: false
18+
related_issues: false
19+
related_prs: false
20+
suggested_labels: false
21+
auto_apply_labels: false
22+
suggested_reviewers: false
23+
auto_assign_reviewers: false
24+
in_progress_fortune: false
25+
enable_prompt_for_ai_agents: true
26+
27+
path_filters:
28+
- "!comfy_api_nodes/apis/**"
29+
- "!**/generated/*.pyi"
30+
- "!.ci/**"
31+
- "!script_examples/**"
32+
- "!**/__pycache__/**"
33+
- "!**/*.ipynb"
34+
- "!**/*.png"
35+
- "!**/*.bat"
36+
37+
path_instructions:
38+
- path: "comfy/**"
39+
instructions: |
40+
Core ML/diffusion engine. Focus on:
41+
- Backward compatibility (breaking changes affect all custom nodes)
42+
- Memory management and GPU resource handling
43+
- Performance implications in hot paths
44+
- Thread safety for concurrent execution
45+
- path: "comfy_api_nodes/**"
46+
instructions: |
47+
Third-party API integration nodes. Focus on:
48+
- No hardcoded API keys or secrets
49+
- Proper error handling for API failures (timeouts, rate limits, auth errors)
50+
- Correct Pydantic model usage
51+
- Security of user data passed to external APIs
52+
- path: "comfy_extras/**"
53+
instructions: |
54+
Community-contributed extra nodes. Focus on:
55+
- Consistency with node patterns (INPUT_TYPES, RETURN_TYPES, FUNCTION, CATEGORY)
56+
- No breaking changes to existing node interfaces
57+
- path: "comfy_execution/**"
58+
instructions: |
59+
Execution engine (graph execution, caching, jobs). Focus on:
60+
- Caching correctness
61+
- Concurrent execution safety
62+
- Graph validation edge cases
63+
- path: "nodes.py"
64+
instructions: |
65+
Core node definitions (2500+ lines). Focus on:
66+
- Backward compatibility of NODE_CLASS_MAPPINGS
67+
- Consistency of INPUT_TYPES return format
68+
- path: "alembic_db/**"
69+
instructions: |
70+
Database migrations. Focus on:
71+
- Migration safety and rollback support
72+
- Data preservation during schema changes
73+
74+
auto_review:
75+
enabled: true
76+
auto_incremental_review: true
77+
drafts: true
78+
79+
finishing_touches:
80+
docstrings:
81+
enabled: false
82+
unit_tests:
83+
enabled: false
84+
85+
tools:
86+
ruff:
87+
enabled: true
88+
pylint:
89+
enabled: false
90+
flake8:
91+
enabled: false
92+
gitleaks:
93+
enabled: true
94+
shellcheck:
95+
enabled: false
96+
markdownlint:
97+
enabled: false
98+
yamllint:
99+
enabled: false
100+
languagetool:
101+
enabled: false
102+
github-checks:
103+
enabled: true
104+
timeout_ms: 90000
105+
ast-grep:
106+
essential_rules: true
107+
108+
chat:
109+
auto_reply: true
110+
111+
knowledge_base:
112+
opt_out: false
113+
learnings:
114+
scope: "auto"

0 commit comments

Comments
 (0)