mirror of
https://github.com/comfyanonymous/ComfyUI.git
synced 2026-02-28 23:07:33 +08:00
b6cb30bab5
* chore: tune CodeRabbit config to limit review scope and disable for drafts - Add tone_instructions to focus only on newly introduced issues - Add global path_instructions entry to ignore pre-existing issues in moved/reformatted code - Disable draft PR reviews (drafts: false) and add WIP title keywords - Disable ruff tool to prevent linter-based outside-diff-range comments Addresses feedback from maintainers about CodeRabbit flagging pre-existing issues in code that was merely moved or de-indented (e.g., PR #12557), which can discourage community contributions and cause scope creep. Amp-Thread-ID: https://ampcode.com/threads/T-019c82de-0481-7253-ad42-20cb595bb1ba * chore: add 'DO NOT MERGE' to ignore_title_keywords Amp-Thread-ID: https://ampcode.com/threads/T-019c82de-0481-7253-ad42-20cb595bb1ba
128 lines
3.8 KiB
YAML
128 lines
3.8 KiB
YAML
# yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json
|
|
language: "en-US"
|
|
early_access: false
|
|
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."
|
|
|
|
reviews:
|
|
profile: "chill"
|
|
request_changes_workflow: false
|
|
high_level_summary: false
|
|
poem: false
|
|
review_status: false
|
|
review_details: false
|
|
commit_status: true
|
|
collapse_walkthrough: true
|
|
changed_files_summary: false
|
|
sequence_diagrams: false
|
|
estimate_code_review_effort: false
|
|
assess_linked_issues: false
|
|
related_issues: false
|
|
related_prs: false
|
|
suggested_labels: false
|
|
auto_apply_labels: false
|
|
suggested_reviewers: false
|
|
auto_assign_reviewers: false
|
|
in_progress_fortune: false
|
|
enable_prompt_for_ai_agents: true
|
|
|
|
path_filters:
|
|
- "!comfy_api_nodes/apis/**"
|
|
- "!**/generated/*.pyi"
|
|
- "!.ci/**"
|
|
- "!script_examples/**"
|
|
- "!**/__pycache__/**"
|
|
- "!**/*.ipynb"
|
|
- "!**/*.png"
|
|
- "!**/*.bat"
|
|
|
|
path_instructions:
|
|
- path: "**"
|
|
instructions: |
|
|
IMPORTANT: Only comment on issues directly introduced by this PR's code changes.
|
|
Do NOT flag pre-existing issues in code that was merely moved, re-indented,
|
|
de-indented, or reformatted without logic changes. If code appears in the diff
|
|
only due to whitespace or structural reformatting (e.g., removing a `with:` block),
|
|
treat it as unchanged. Contributors should not feel obligated to address
|
|
pre-existing issues outside the scope of their contribution.
|
|
- path: "comfy/**"
|
|
instructions: |
|
|
Core ML/diffusion engine. Focus on:
|
|
- Backward compatibility (breaking changes affect all custom nodes)
|
|
- Memory management and GPU resource handling
|
|
- Performance implications in hot paths
|
|
- Thread safety for concurrent execution
|
|
- path: "comfy_api_nodes/**"
|
|
instructions: |
|
|
Third-party API integration nodes. Focus on:
|
|
- No hardcoded API keys or secrets
|
|
- Proper error handling for API failures (timeouts, rate limits, auth errors)
|
|
- Correct Pydantic model usage
|
|
- Security of user data passed to external APIs
|
|
- path: "comfy_extras/**"
|
|
instructions: |
|
|
Community-contributed extra nodes. Focus on:
|
|
- Consistency with node patterns (INPUT_TYPES, RETURN_TYPES, FUNCTION, CATEGORY)
|
|
- No breaking changes to existing node interfaces
|
|
- path: "comfy_execution/**"
|
|
instructions: |
|
|
Execution engine (graph execution, caching, jobs). Focus on:
|
|
- Caching correctness
|
|
- Concurrent execution safety
|
|
- Graph validation edge cases
|
|
- path: "nodes.py"
|
|
instructions: |
|
|
Core node definitions (2500+ lines). Focus on:
|
|
- Backward compatibility of NODE_CLASS_MAPPINGS
|
|
- Consistency of INPUT_TYPES return format
|
|
- path: "alembic_db/**"
|
|
instructions: |
|
|
Database migrations. Focus on:
|
|
- Migration safety and rollback support
|
|
- Data preservation during schema changes
|
|
|
|
auto_review:
|
|
enabled: true
|
|
auto_incremental_review: true
|
|
drafts: false
|
|
ignore_title_keywords:
|
|
- "WIP"
|
|
- "DO NOT REVIEW"
|
|
- "DO NOT MERGE"
|
|
|
|
finishing_touches:
|
|
docstrings:
|
|
enabled: false
|
|
unit_tests:
|
|
enabled: false
|
|
|
|
tools:
|
|
ruff:
|
|
enabled: false
|
|
pylint:
|
|
enabled: false
|
|
flake8:
|
|
enabled: false
|
|
gitleaks:
|
|
enabled: true
|
|
shellcheck:
|
|
enabled: false
|
|
markdownlint:
|
|
enabled: false
|
|
yamllint:
|
|
enabled: false
|
|
languagetool:
|
|
enabled: false
|
|
github-checks:
|
|
enabled: true
|
|
timeout_ms: 90000
|
|
ast-grep:
|
|
essential_rules: true
|
|
|
|
chat:
|
|
auto_reply: true
|
|
|
|
knowledge_base:
|
|
opt_out: false
|
|
learnings:
|
|
scope: "auto"
|