Skip to content

Commit b7ba504

Browse files
Try to make coderabbit enforce AGENTS.md (#14759)
1 parent 6c62ca0 commit b7ba504

1 file changed

Lines changed: 16 additions & 3 deletions

File tree

.coderabbit.yaml

Lines changed: 16 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,12 +4,12 @@ early_access: false
44
tone_instructions: "Only comment on issues introduced by this PR's changes. Do not flag pre-existing problems in moved, re-indented, or reformatted code."
55

66
reviews:
7-
profile: "chill"
8-
request_changes_workflow: false
7+
profile: "assertive"
8+
request_changes_workflow: true
99
high_level_summary: false
1010
poem: false
1111
review_status: false
12-
review_details: false
12+
review_details: true
1313
commit_status: true
1414
collapse_walkthrough: true
1515
changed_files_summary: false
@@ -39,6 +39,14 @@ reviews:
3939
- path: "**"
4040
instructions: |
4141
IMPORTANT: Only comment on issues directly introduced by this PR's code changes.
42+
Treat AGENTS.md as mandatory repository policy, not optional style guidance.
43+
Flag PR changes that violate AGENTS.md even when the code is otherwise functional.
44+
In particular, enforce architecture boundaries, dtype/device/memory rules,
45+
interface contracts, import style, no unnecessary try/except blocks, no inline
46+
imports, no outbound internet paths in core ComfyUI, and narrow scoped fixes.
47+
Prefer direct findings over suggestions when a rule is violated. Only ignore
48+
AGENTS.md when it clearly conflicts with a newer explicit maintainer instruction
49+
in the PR.
4250
Do NOT flag pre-existing issues in code that was merely moved, re-indented,
4351
de-indented, or reformatted without logic changes. If code appears in the diff
4452
only due to whitespace or structural reformatting (e.g., removing a `with:` block),
@@ -123,5 +131,10 @@ chat:
123131

124132
knowledge_base:
125133
opt_out: false
134+
code_guidelines:
135+
enabled: true
136+
filePatterns:
137+
- files: "AGENTS.md"
138+
applyTo: "**"
126139
learnings:
127140
scope: "auto"

0 commit comments

Comments
 (0)