Skip to content

Conversation

@hyxklee
Copy link
Collaborator

@hyxklee hyxklee commented Jan 16, 2026

작업 내용 💻

  • 협업을 위한 클로드 코드 설정을 추가했습니다. 개발에 용이한 서브 에이전트도 추가해뒀으니 유용하게 사용 바랍니다. (다른 AI Agent에도 컨텍스트만 넣어주면 유사하게 동작 할겁니다.)
  • 코드 컨벤션을 위한 Ktlint를 추가 설정했습니다. 작업시 Agent에게 시키거나, 직접 명령어를 쳐서 체크를 해도 됩니다.
스타일 검사: ./gradlew ktlintCheck

자동 포맷: ./gradlew ktlintFormat

스크린샷 📷

같이 얘기해보고 싶은 내용이 있다면 작성 📢

  • 현재 Ktlint 설정에 맞게 포맷팅이 되어있는데 약간 어색할 수도 있습니다. 너무 가독성이 좋지 않은 부분은 관련 설정 말씀해주시면 수정해보겠습니덩
  • ktlint의 옵션 중 trailing comma를 활성화하는 경우 IDE의 플러그인과 동작이 틀어지는 현상이 있어 우선 제거했습니당

Summary by CodeRabbit

  • Documentation

    • 프로젝트 가이드 및 여러 AI 에이전트(코드리뷰/디버거/Kotlin 마이그레이션) 워크플로 문서 추가
  • Chores

    • 코드 스타일 검사 도구(ktlint) 통합 및 에디터 설정 추가
    • CI에 코드 스타일 검사 단계 추가
    • 전반적 개발 설정/권한 구성 파일 추가
  • Tests

    • 단위/통합 테스트 추가 및 테스트 코드 포맷 정리 및 리팩터링

✏️ Tip: You can customize this high-level summary in your review settings.

@hyxklee hyxklee self-assigned this Jan 16, 2026
@hyxklee hyxklee added ⚙️Chore ✨Feat 새로운 기능 개발 labels Jan 16, 2026
@coderabbitai
Copy link

coderabbitai bot commented Jan 16, 2026

Walkthrough

클로드 에이전트용 설명서 3개와 권한 설정을 추가하고, ktlint 플러그인 및 CI 검사 통합, 테스트 코드 스타일 정리(표현식 바디·후행 쉼표 등)와 일부 테스트 파일 이동·추가/삭제가 이루어졌습니다.

Changes

Cohort / File(s) Summary
Claude 에이전트 문서 및 설정
.claude/agents/code-review-agent.md, .claude/agents/debugger-agent.md, .claude/agents/kotlin-migration-agent.md, .claude/settings.json
Claude용 에이전트 문서 3건 추가(워크플로우, 출력포맷, 규칙 등) 및 에이전트 권한/도구 설정 파일 추가
프로젝트 문서 및 에디터 설정
CLAUDE.md, .editorconfig
프로젝트 가이드/아키텍처 문서 추가 및 에디터 포맷 규칙(kotlin/yaml 등) 정의
빌드 및 CI
build.gradle.kts, .github/workflows/dev.yml
ktlint 플러그인 추가(14.0.1) 및 ktlintCheck 실행 단계 CI에 추가, ktlint 설정(버전·필터 등) 적용
테스트 컨테이너 설정
src/test/kotlin/.../config/MongoTestConfig.kt, .../MysqlTestConfig.kt, .../TestContainersTest.kt
TestContainers 관련 테스트 설정을 식표현식·메서드 체이닝으로 정리, 후행 쉼표 및 경미한 서식 수정
FeedGetService 테스트 변경
src/test/kotlin/.../domain/feed/application/domain/service/FeedGetServiceTest.kt
기존 테스트 파일 삭제 후(또는 대체) PessimisticLockException → ResourceLockedException 동작을 검증하는 테스트 추가(테스트 파일 교체)
통합/유스케이스 테스트 리포맷
src/test/kotlin/.../domain/feed/application/usecase/FeedUsecaseIntegrationTest.kt, .../leenk/application/usecase/LeenkUsecaseTest.kt
Given/When/Then 구조 재배치, 서식·후행 쉼표 정리 — 기능/로직 변경 없음
테스트 픽스처 리팩토링
src/test/kotlin/.../domain/feed/test/fixture/FeedTestFixture.kt, .../leenk/test/fixture/LeenkTestFixture.kt, .../LeenkParticipantsTestFixture.kt, .../LocationTestFixture.kt, .../user/test/fixture/UserTestFixture.kt
표현식 본문으로 변환, 후행 쉼표 추가, 여러 라인 포맷팅 조정 — 동작 불변

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

Suggested reviewers

  • jj0526
  • 1winhyun

Poem

🐰 깡총 뛰며 코드를 다듬었지,
쉼표 하나, 들여쓰기 한 줄,
에이전트 문서 우르르 추가하고,
ktlint도 챙겨 빌드에 넣었네,
테스트는 정돈되어 안심이야.

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 5.26% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목은 Ktlint 설정과 Claude 코드 설정 추가라는 변경 내용을 명확하게 반영하고 있으며, 변경 사항의 핵심을 요약하고 있습니다.
Description check ✅ Passed PR 설명은 제공된 템플릿의 주요 섹션(작업 내용, 스크린샷, 토의 사항)을 포함하고 있으며, 구체적인 변경 사항과 명령어, 관련 고려사항을 상세히 기재하고 있습니다.

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


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.

❤️ Share

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

Copy link
Collaborator

@soo0711 soo0711 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!
자동 포맷으로 인해 약간 어색한 부분은 있지만 가독성은 나쁘지 않은 것 같습니당
사용해보면서 불편한 점이 있으면 추후에 보완해도 좋을 것 같아요 👍

Copy link
Collaborator

@jj0526 jj0526 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

Comment on lines +21 to +32
): Leenk {
val builder =
Leenk
.builder()
.author(author)
.location(location)
.title(title)
.content(content)
.startTime(startTime)
.maxParticipants(maxParticipants)
.currentParticipants(currentParticipants)
.status(status)
Copy link
Collaborator

Choose a reason for hiding this comment

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

음 약간 빌더패턴에서는 줄바꿈이 많아져서 가독성이 떨어진 느낌인데 이게 더 코틀린 스러운 방식인가요??

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⚙️Chore ✨Feat 새로운 기능 개발

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants