-
Notifications
You must be signed in to change notification settings - Fork 38
Create .coderabbit.yaml #3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including You can disable this status message by setting the
📝 WalkthroughWalkthrough새로운 Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 4
🤖 Fix all issues with AI agents
In @.coderabbit.yaml:
- Around line 107-118: The top-level "security" section (and keys like
"warn_on_sensitive_logging" and "sensitive_keywords") is not part of the schema
and won't be applied; replace it with a supported configuration such as enabling
a built-in security review tool (e.g., add or update reviews.tools.gitleaks with
equivalent rules/patterns) or rework to use the schema's supported security
options; specifically, migrate the list under "sensitive_keywords" into gitleaks
configuration (or the repository scanner configured under
reviews.tools.gitleaks) and remove the unsupported "security" block so that
sensitive-data detection is executed by the schema-supported tool.
- Around line 98-106: The YAML contains a top-level key "summary" which is not
part of the schema and will be ignored; move that configuration into the
schema-supported keys by replacing "summary" with "reviews.high_level_summary"
(for enabling the section) and "reviews.high_level_summary_instructions" (for
the section contents/instructions), updating the key names and nesting so the PR
summary is preserved under reviews.high_level_summary and the guidance text goes
under reviews.high_level_summary_instructions.
- Around line 7-96: The reviews block fails schema validation because
reviews.auto_review is a boolean but must be an object and many keys
(skip_draft, comment_style, include_summary, severity_focus, limits, paths,
context, focus_areas, noise_control, testing, closing_note) are not allowed by
the schema; update the reviews block to match the schema by replacing
auto_review: true with an object of the expected shape (e.g., auto_review: {
enabled: true, ... } per schema), remove or relocate the disallowed keys
(skip_draft, comment_style, include_summary, severity_focus, limits, paths,
context, focus_areas, noise_control, testing, closing_note) into their correct
top-level sections or drop them if the schema does not support them, and ensure
only schema-approved fields remain under the reviews key so validation passes.
- Around line 120-123: 현재 파일에 있는 하위 섹션 'output'의 'language'와 'tone' 설정은 스키마에서
무시되므로, 해당 키들을 제거하고 상위 최상위 구성으로 이동해 'language'와 'tone_instructions' 최상위 속성으로
설정하세요; 예를 들어 현재의 output.language → 최상위 language, output.tone → 최상위
tone_instructions로 옮기고 'output' 섹션은 삭제해 스키마에 맞게 적용되도록 수정하십시오.
.coderabbit.yaml
Outdated
| reviews: | ||
| # PR이 생성/업데이트될 때 자동 리뷰 | ||
| auto_review: true | ||
|
|
||
| # Draft PR은 리뷰하지 않도록(소음 감소) | ||
| skip_draft: true | ||
|
|
||
| # 리뷰 코멘트 방식 | ||
| comment_style: inline # inline | summary | both 중 선택(지원 여부에 따라 조정) | ||
| include_summary: true | ||
|
|
||
| # 리뷰 강도(교육 + 실무 균형) | ||
| severity_focus: | ||
| - blocker | ||
| - major | ||
|
|
||
| # 중복 코멘트/잔소리 방지 | ||
| limits: | ||
| max_comments_per_file: 8 | ||
| max_total_comments: 40 | ||
| avoid_repeat_comments: true | ||
|
|
||
| # 파일/경로 스코프 | ||
| paths: | ||
| include: | ||
| - "src/**" | ||
| - "build.gradle*" | ||
| - "settings.gradle*" | ||
| - "gradle/**" | ||
| exclude: | ||
| - "**/*.md" | ||
| - "**/*.adoc" | ||
| - "**/*.txt" | ||
| - "**/*.png" | ||
| - "**/*.jpg" | ||
| - "**/*.jpeg" | ||
| - "**/*.gif" | ||
| - "**/*.svg" | ||
| - "**/*.lock" | ||
| - "**/generated/**" | ||
| - "**/build/**" | ||
| - "**/out/**" | ||
| - "**/.gradle/**" | ||
|
|
||
| # 언어/프레임워크 힌트(리뷰 정확도 향상) | ||
| context: | ||
| primary_stack: | ||
| - kotlin | ||
| - java | ||
| - spring-boot | ||
| - jpa | ||
| style_guide: | ||
| kotlin: | ||
| # Kotlin idiom 관련 힌트 | ||
| discourage_force_unwrap: true # '!!' 지양 | ||
| discourage_jpa_entity_data_class: true | ||
| prefer_immutable_collections: true | ||
| java: | ||
| discourage_overcomplex_streams: true | ||
| keep_exception_cause: true | ||
| spring: | ||
| transactional_boundary_check: true | ||
| n_plus_one_check: true | ||
| controller_service_repository_separation: true | ||
| exception_to_http_status_check: true | ||
|
|
||
| # 보안/운영 안정성 우선 | ||
| focus_areas: | ||
| - correctness | ||
| - security | ||
| - reliability | ||
| - performance | ||
| - testing | ||
|
|
||
| # 스타일/컨벤션은 과도하게 잡지 않도록(ktlint/detekt는 CI로 해결 권장) | ||
| noise_control: | ||
| style_comments: low | ||
| naming_comments: low | ||
| formatting_comments: low | ||
|
|
||
| # 테스트 제안 강화(학습용) | ||
| testing: | ||
| suggest_missing_tests: true | ||
| require_edge_case_tests: true | ||
| highlight_flaky_risks: true | ||
|
|
||
| # 리뷰에 포함할 “루퍼스 성장 루프” 문장 | ||
| closing_note: | ||
| enabled: true | ||
| template: "다음 PR에서 더 좋아지기 위한 1가지 루프를 제안한다: {{one_actionable_loop}}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reviews 블록이 스키마와 불일치하여 검증이 실패합니다.
Line 9의 auto_review는 boolean이 아니라 object여야 하며, Line 12~96의 skip_draft, comment_style, include_summary, severity_focus, limits, paths, context, focus_areas, noise_control, testing, closing_note는 스키마에 없는 키라 reviews 전체가 검증 실패합니다(추가 속성 허용 안 됨). 스키마에 맞게 재구성해 주세요.
✅ 스키마에 맞춘 예시 수정안
reviews:
- auto_review: true
- skip_draft: true
+ auto_review:
+ enabled: true
+ drafts: false
@@
- paths:
- include:
- - "src/**"
- - "build.gradle*"
- - "settings.gradle*"
- - "gradle/**"
- exclude:
- - "**/*.md"
- - "**/*.adoc"
- - "**/*.txt"
- - "**/*.png"
- - "**/*.jpg"
- - "**/*.jpeg"
- - "**/*.gif"
- - "**/*.svg"
- - "**/*.lock"
- - "**/generated/**"
- - "**/build/**"
- - "**/out/**"
- - "**/.gradle/**"
+ path_filters:
+ - "src/**"
+ - "build.gradle*"
+ - "settings.gradle*"
+ - "gradle/**"
+ - "!**/*.md"
+ - "!**/*.adoc"
+ - "!**/*.txt"
+ - "!**/*.png"
+ - "!**/*.jpg"
+ - "!**/*.jpeg"
+ - "!**/*.gif"
+ - "!**/*.svg"
+ - "!**/*.lock"
+ - "!**/generated/**"
+ - "!**/build/**"
+ - "!**/out/**"
+ - "!**/.gradle/**"🤖 Prompt for AI Agents
In @.coderabbit.yaml around lines 7 - 96, The reviews block fails schema
validation because reviews.auto_review is a boolean but must be an object and
many keys (skip_draft, comment_style, include_summary, severity_focus, limits,
paths, context, focus_areas, noise_control, testing, closing_note) are not
allowed by the schema; update the reviews block to match the schema by replacing
auto_review: true with an object of the expected shape (e.g., auto_review: {
enabled: true, ... } per schema), remove or relocate the disallowed keys
(skip_draft, comment_style, include_summary, severity_focus, limits, paths,
context, focus_areas, noise_control, testing, closing_note) into their correct
top-level sections or drop them if the schema does not support them, and ensure
only schema-approved fields remain under the reviews key so validation passes.
📌 Summary
💬 Review Points
✅ Checklist
📎 References
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.