Repository navigation
ci: 자동 리뷰는 diff만, cross-service 분석은 @claude 수동으로 - #87
Merged
Merged
Conversation
common-module 변경 PR(#85)의 자동 리뷰가 9분 넘게 걸렸다. CLAUDE.md 의 "서비스 경계를 넘는 변경은 하위 호환성을 특히 주의해서 본다" 지침을 따라 diff 밖의 다른 서비스 소비 코드까지 찾아 읽었기 때문으로 보인다 - diff 자체는 403줄로 크지 않았다. 자동 리뷰(claude-code-review.yml)는 diff 에 나온 코드만 보게 프롬프트로 범위를 좁히고, common-module/내부 API/Kafka 스키마 변경이 보이면 실제 분석 대신 "@claude 로 수동 리뷰를 요청하라"는 코멘트만 남기도록 한다. cross-service 하위 호환성 분석은 claude.yml(@claude 수동 호출)의 역할로 문서화한다. allowedTools 는 건드리지 않았다 - #79 에서 도구를 뺐다가 리뷰가 통째로 증발한 사고가 있어서, 속도 문제는 프롬프트로만 좁힌다.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
claude-code-review.yml)는 diff에 나온 코드만 보도록 프롬프트로 범위를 좁힘. common-module/내부 API/Kafka 스키마 변경이 있으면 실제 분석 대신 "@claude로 수동 리뷰 요청" 코멘트만 남기게 함.claude.yml(@claude수동 호출)의 역할로 문서화.allowedTools는 그대로 둠 — #79에서 도구를 뺐다가 리뷰 코멘트가 통째로 증발한 사고가 있어서, 속도 문제는 도구 제한이 아니라 프롬프트 범위 조정으로만 해결.Test plan
@claude 이 PR의 common-module 변경이 다른 서비스에 미치는 영향 봐줘로 수동 호출했을 때 실제 cross-service 분석이 나오는지 확인🤖 Generated with Claude Code