Claude 설정 문서의 낡은 서술 정리 - #95
Merged
Merged
Conversation
- 계약 정본 목록에 extraction.proto 추가 - 원인: 요청·응답 모양의 정본이 proto 로 옮겨 감. 빌드가 shared-infra/contracts/ 에서 클래스를 생성 - 모양 변경 시 infra proto 선행 수정 규칙 추가 - 공개 표기 기준을 renderer 이름·역할·호출 관계와 스택 표기까지로 갱신 - 원인: 옛 기준 문구가 현행 기준과 어긋남 - 소비자 서술을 core 하나(파싱 작업 워커·관리자 모델 프로브)로 갱신 - 원인: 관리자 모델 프로브도 /internal/models/probe 호출 - 테스트 사전 조건에 shared-infra/contracts/ 명시 - 원인: 계약 파일이 없으면 proto 생성 단계에서 컴파일 실패 - core 공유 라인 목록을 core build.gradle.kts 기준 규칙으로 교체 - 원인: 목록에 없는 라이브러리 포함·protobuf 누락 - 크기 상한 상수 나열·이력 서술을 Javadoc 포인터 한 줄로 교체 - 원인: 상수 목록이 코드 복제라 낡음 - 통합 stub 목록의 S3 를 ImageStorage 로 교체 - 로깅 절의 중복 불릿 2개 삭제 - 원인: LoggerFactory 금지는 Lombok 절과 중복·placeholder 규칙은 자명 - docs/api-contract.md 의 컨버터 클래스명을 ProtobufJsonConverterConfig 로 교체 - 원인: 옛 클래스명 ContractMessageConverterConfig 가 코드에 없음
|
Discord 스레드 연동용 메타데이터입니다. discord-pr-bot 워크플로가 자동 생성하며, 수정·삭제하면 PR 과 Discord 알림 연동이 끊깁니다. |
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID:
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 |
- CLAUDE.md 계약 정본 문장을 docs/api-contract.md 포인터로 축약 - 원인: 필드 의미의 정본은 extraction.proto 주석. 새 문장이 api.md 로 보내 정본과 어긋남 - api-contract.md 의 proto 역할에 필드 의미 추가 - 테스트 사전 조건의 실패 지점을 컴파일 단계로 교체 - 원인: srcDir 가 비면 generateProto 는 빈 결과로 통과함 - protobuf 버전 맞춤 문장 삭제 - 원인: repo 안 protoc·runtime 은 한 변수로 묶임. repo 간 와이어는 JSON 이라 버전 차이 영향 없음
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.
Situation
설정 문서 감사 결과
Task
낡은 서술 정리
Action
현재 사실로 교체
extraction-api.md·에러 카탈로그 둘contracts/, 파일별 역할은docs/api-contract.md, 모양 변경은 proto 먼저docs/api-contract.mdproto 역할docs/api-contract.md컨버터ContractMessageConverterConfigProtobufJsonConverterConfig./gradlew test가 그냥 돎shared-infra/contracts/필요, 없으면 계약 클래스가 생성되지 않아 컴파일 실패build.gradle.kts에도 있는 라이브러리PruningHtmlParser.MAX_RETAINED_CHARSJavadoc 포인터삭제
LoggerFactory금지 불릿 (언어 절@Slf4j규칙과 같음)Result
범위
InternalHostGuard는 error. 문서와 코드 중 어느 쪽을 맞출지 결정 대기ModelProbeRequest.java주석이 포인터 문서가 된docs/api-contract.md §2를 계약 위치로 가리킴. 코드 주석이라 이 PR 밖연관 이슈