Skip to content

chore: configure CodeRabbit reviews#215

Merged
YoungJinJung merged 3 commits into
mainfrom
chore/coderabbit-config
May 12, 2026
Merged

chore: configure CodeRabbit reviews#215
YoungJinJung merged 3 commits into
mainfrom
chore/coderabbit-config

Conversation

@YoungJinJung
Copy link
Copy Markdown
Contributor

@YoungJinJung YoungJinJung commented May 12, 2026

Summary

  • Add repository-root .coderabbit.yaml so CodeRabbit can use branch-local review settings
  • Enable automatic reviews for PRs targeting main, including incremental reviews on push
  • Add unic-specific review guidance for Go, Bubble Tea TUI, AWS service mapping, tests, and docs
  • Enable CodeRabbit knowledge-base guidance from AGENTS.md and repo workflow docs

Validation

  • ruby YAML parse for .coderabbit.yaml
  • yamllint .coderabbit.yaml

Related Issues

  • N/A: repository workflow configuration only

Checklist

  • CodeRabbit configuration is located at the repository root
  • Auto review is enabled for PRs targeting main
  • Repo-specific path instructions are configured
  • YAML syntax and lint validation passed

Note: this is a repository workflow configuration change without a linked issue.

Summary by CodeRabbit

  • 설정
    • CodeRabbit 통합 구성 추가: 검토 언어를 en-US로 설정하고 assertive 검토 프로필 및 상세 리뷰 섹션 활성화.
    • main 브랜치 대상 자동·증분 자동리뷰 활성화, 제목 키워드(wip, draft) 무시.
    • 특정 경로 제외 규칙 및 경로별 맞춤 검토 체크리스트 지정.
    • 다수 정적분석 도구 및 사전 병합 검사(타이틀/설명/이슈)를 경고 수준으로 설정.
    • 채팅 자동응답·지식베이스 검색 활성화, 코드 가이드라인 출처 범위 제한.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 12, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: d6ca5960-0340-4370-8345-a49bb90d08fe

📥 Commits

Reviewing files that changed from the base of the PR and between 05c6065 and 2ee10b6.

📒 Files selected for processing (1)
  • .coderabbit.yaml
📜 Recent review details
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: test
🔇 Additional comments (1)
.coderabbit.yaml (1)

5-102: Config looks consistent and ready to merge.

I don’t see blocking issues here. The settings are coherent with the stated PR goals, and the previously problematic filter is now correctly scoped as !unic/**.


Walkthrough

Adds .coderabbit.yaml to configure CodeRabbit: sets review language to en-US, enables the assertive profile, configures auto-reviews on main, path exclusions and per-path review checklists, pre-merge checks, multiple analysis tools, and AI/KB settings.

전체 개요

새로운 .coderabbit.yaml 파일이 추가되어 리포지토리의 CodeRabbit 자동 리뷰 동작을 정의합니다. 주요 항목으로는 리뷰 언어(en-US), assertive 프로필, 자동 증분 리뷰 설정, 경로별 검토 체크리스트 및 제외 규칙, 사전 병합 검사(경고 모드), 정적/보안 분석 도구 활성화, 채팅 자동응답 및 지식 기반 웹 검색 설정이 포함됩니다.

변경사항

CodeRabbit 통합 설정

계층 / 파일 요약
스키마/헤더
.coderabbit.yaml
YAML lint suppression(행 길이) 및 CodeRabbit 스키마 선언을 추가합니다.
리뷰 언어·프로필·자동 리뷰
.coderabbit.yaml
리뷰 언어를 en-US로 설정하고 assertive 프로필을 구성하며, 자동 리뷰(auto_review)를 main 브랜치에 대해 증분 모드로 설정하고 wip/draft 제목 키워드를 무시하도록 구성합니다.
경로 제외 및 경로별 지침
.coderabbit.yaml
dist/**, tmp/**, unic/**를 제외하도록 설정하고 Go 소스, internal/app/**(Bubble Tea UI), internal/services/aws/**(AWS 통합), 테스트(**/*_test.go), README.md, docs/**에 대한 경로별 검토 체크리스트를 추가합니다.
사전 병합 검사
.coderabbit.yaml
PR 제목, 설명, 이슈 평가에 대해 pre_merge_checkswarning 모드로 구성합니다.
분석 도구 활성화
.coderabbit.yaml
golangci-lint, markdownlint, yamllint, actionlint, shellcheck, checkmake, gitleaks, osvScanner 등 다양한 정적/보안 분석 도구를 활성화합니다.
AI/지식 기반 설정
.coderabbit.yaml
채팅 자동응답(art: false, auto_reply 활성화)과 지식 기반 웹 검색을 활성화하고 code_guidelines 소스 패턴을 AGENTS.md 및 특정 docs/ 파일로 제한합니다.

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title uses the required conventional prefix 'chore:' and clearly describes the change (configure CodeRabbit reviews), making it fully relevant to the changeset.
Description check ✅ Passed The description includes Summary, Related Issues, Validation, and Checklist sections that align with the template. All required information about the configuration change is provided and well-documented.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/coderabbit-config

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@amazon-q-developer amazon-q-developer Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR adds a CodeRabbit configuration file with comprehensive review automation settings. The YAML structure is valid, all referenced files exist in the repository (AGENTS.md, docs/documentation-harness.md, docs/branch-naming-harness.md), and the configuration appropriately enables multiple linting tools and Korean language support. The validation mentioned in the PR description (Ruby YAML parse and yamllint) confirms the file's correctness. No blocking issues identified.


You can now have the agent implement changes and create commits directly on your pull request's source branch. Simply comment with /q followed by your request in natural language to ask the agent to make changes.

The repository now has CodeRabbit installed, so review behavior should be codified in the repo instead of relying on UI defaults. The config enables automatic reviews for main-targeted PRs, uses Korean feedback, turns on focused review guidance for Go, Bubble Tea, AWS service code, tests, and docs, and points CodeRabbit at the existing repository guidance files.

Constraint: CodeRabbit requires .coderabbit.yaml at the repository root to apply branch-local review settings

Rejected: Leave defaults implicit | defaults do not encode unic-specific AWS/TUI/documentation review criteria

Confidence: high

Scope-risk: narrow

Directive: Keep this file aligned with AGENTS.md, documentation-harness.md, and branch-naming-harness.md when workflow rules change

Tested: ruby YAML parse for .coderabbit.yaml

Tested: yamllint .coderabbit.yaml

Not-tested: Live CodeRabbit review execution before opening PR
@YoungJinJung YoungJinJung force-pushed the chore/coderabbit-config branch from 21787f9 to 77b700c Compare May 12, 2026 09:18
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.coderabbit.yaml:
- Around line 26-29: The path_filters entry currently uses the pattern "!unic"
which only matches the exact path "unic" and does not exclude files under the
directory; update the pattern in path_filters from "!unic" to a directory-wide
exclusion such as "!unic/" or "!unic/**" so that all files under the unic
directory are excluded; locate the path_filters list and replace the "!unic"
item accordingly in the configuration.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 372e0efd-ecf1-4919-ba9d-6cb747ae78b0

📥 Commits

Reviewing files that changed from the base of the PR and between 21787f9 and 77b700c.

📒 Files selected for processing (1)
  • .coderabbit.yaml
📜 Review details
🔇 Additional comments (1)
.coderabbit.yaml (1)

8-95: PR 목표와 설정 구성이 전반적으로 잘 맞습니다.

자동/증분 리뷰, 경로별 리뷰 지침, 정적 분석 도구 활성화, 지식 베이스 가이드 파일 제한까지 목적에 맞게 일관되게 구성되었습니다.

Comment thread .coderabbit.yaml Outdated
CodeRabbit correctly pointed out that the unic path filter matched only the exact path and not files below a directory. The exclusion now uses a recursive glob so review sparse-checkout filtering behaves as intended.

Constraint: CodeRabbit path_filters use glob patterns where directory exclusions should include /**

Confidence: high

Scope-risk: narrow

Tested: ruby YAML parse for .coderabbit.yaml

Tested: yamllint .coderabbit.yaml
The CodeRabbit configuration should be readable across the project and produce English review output. This switches the configured review language to en-US and translates all path-specific review guidance from Korean to English without changing the review policy intent.

Constraint: User requested Korean text in the CodeRabbit configuration be changed to English

Confidence: high

Scope-risk: narrow

Tested: ruby YAML parse for .coderabbit.yaml

Tested: yamllint .coderabbit.yaml
@YoungJinJung YoungJinJung merged commit 02ececc into main May 12, 2026
2 checks passed
@YoungJinJung YoungJinJung deleted the chore/coderabbit-config branch May 12, 2026 09:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants