[m-a-king]

한눈에

계약을 proto 로 옮긴 뒤 남은 옛 설명과 안 쓰는 의존성 정리

• 이전: 코드 설명이 지워진 문서·옛 동작(null 모델, 구버전 렌더 서비스)을 가리킴
• 설명을 현행 계약에 맞추고, 사용처가 없어진 검증 라이브러리를 뺌
• 도메인 추출 경로 enum 과 프로브 전용 code 가 계약과 어긋나면 테스트가 잡음
• 머지하면 extractor 가 prod 에 자동 배포됨. 동작 변화 없음

Situation

리뷰에서 드러난 잔재

• 계약 정본이 infra proto 로 옮겨간 뒤 주석이 옛 문서 절·옛 호환 조건을 가리킴
• Bean Validation 은 손으로 쓴 요청 record 와 함께 사라졌는데 starter 만 남음
• 도메인 ExtractionMethod 와 계약 enum 의 이름 대조가 없어 어긋나면 응답 매퍼가 500 으로 샘
• 참조: 렌더 계약 proto 추가 (5단계) infra#119

Task

잔재 정리와 대조 보강

• 주석·의존성 정리
• 계약 대조 테스트 2개 추가

Action

정리

대상	이전	이후
validation starter	사용처 0건인 채 남음	삭제
compress 호환 설명	구버전 renderer 호환 근거	응답 헤더 기준 판별만
model 설명 5곳	없으면 null	없으면 빈 문자열
문서 참조	계약 §2·docs/api-contract.md	infra 계약 문서·proto

대조 테스트

검사	실물	계약
추출 경로	도메인 ExtractionMethod 이름 집합	계약 enum 값 집합
프로브 전용	모델 프로브 예외가 만드는 code	probe_only 옵션

Result

남긴 것

• 실패 code 의미 Javadoc 이 proto 주석과 겹치는 곳은 요청 범위 밖이라 그대로 둠
• springdoc 의존성은 호출자 쪽 소비 여부를 몰라 그대로 둠

---

연관 이슈

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

• Configuration used: Organization UI
• Review profile: CHILL
• Plan: Advanced
• Run ID: a8a643fc-916d-42e6-8a4f-7c0a90772b3d

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[github-actions]

Discord 스레드 연동용 메타데이터입니다. discord-pr-bot 워크플로가 자동 생성하며, 수정·삭제하면 PR 과 Discord 알림 연동이 끊깁니다.