Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions .github/workflows/claude-code-review.yml
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,13 @@ jobs:
리뷰는 이 턴 안에서 끝낸다. 서브에이전트를 백그라운드로 띄우고
완료 알림을 기다리지 마라 - Actions 에는 알림을 받을 루프가 없어
프로세스가 그대로 죽는다.

이 자동 리뷰는 diff 에 나온 코드만 본다. common-module 등
diff 밖의 다른 서비스로 건너가 소비 코드를 찾아 읽는 하위 호환성
분석은 하지 마라 - 시간이 몇 배로 늘어난다. diff 가
common-module 의 DTO, 서비스 간 내부 API, Kafka 이벤트 스키마를
건드린다면, 그 분석 대신 코멘트에 "cross-service 영향 확인이
필요합니다 - `@claude` 로 수동 리뷰를 요청하세요" 한 줄만 남겨라.
# allowedTools 는 화이트리스트다. 목록에 없는 도구는 permissionMode
# default + 비대화형 조합에서 그대로 거부된다.
# - Skill: 없으면 /code-review 스킬 본체가 로드되지 못한다. 모델은
Expand Down
6 changes: 6 additions & 0 deletions .github/workflows/claude.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,12 @@
#
# @claude ChatRoomPresenceTracker 가 동시성 안전한지 봐줘
# @claude 이 테스트 깨지는 원인 찾아줘
#
# common-module/내부 API/Kafka 스키마처럼 서비스 경계를 넘는 변경의 하위
# 호환성 분석은 여기서 한다. claude-code-review.yml 은 diff 만 보는 얕은
# 리뷰라 그 분석을 하지 않고, 대신 이 워크플로를 부르라는 코멘트만 남긴다.
#
# @claude 이 PR 의 common-module 변경이 다른 서비스에 미치는 영향 봐줘
name: Claude Code

on:
Expand Down
6 changes: 6 additions & 0 deletions CLAUDE.md
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,12 @@ Spring 기반 MSA. 서비스는 gateway / user / matching / chat / item / notifi
- **서비스 경계를 넘는 변경은 하위 호환성을 특히 주의해서 본다** —
`common-module` 의 DTO, 서비스 간 내부 API, Kafka 이벤트 스키마는
배포 순서가 어긋나는 순간 깨진다.
- **단, PR 자동 리뷰(`claude-code-review.yml`)는 diff 에 나온 코드만 보고
끝낸다.** 다른 서비스로 건너가 소비 코드를 찾아 읽는 하위 호환성
분석은 턴·시간을 크게 늘린다 — diff 가 `common-module`/내부 API/Kafka
스키마를 건드리면 자동 리뷰는 코멘트에 "cross-service 영향 확인 필요"
한 줄만 남기고, 실제 분석은 PR 코멘트에 `@claude` 로 명시적으로
요청했을 때(`claude.yml`)만 수행한다.
- 운영 설정(`docker-compose.prod.yml`, `application-aws.yml`)은 EC2 단일 호스트에
6개 JVM 이 함께 뜨는 전제로 메모리 한도가 잡혀 있다. 한도를 건드리는 변경은
전체 예산과 함께 본다.
Loading