[Feat] #58 법령/내규검색 API 항목 추가

[ledu1017]

개요

• closed [Feat] 법령/내규검색 API 항목 추가 #58 이슈 //이슈 번호는 추적을 위해 필요합니다!

PR 유형

• 새로운 기능 추가
• 버그 수정
• CSS 등 사용자 UI 디자인 변경
• 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
• 코드 리팩토링
• 주석 추가 및 수정
• 문서 수정
• 테스트 추가, 테스트 리팩토링
• 빌드 부분 혹은 패키지 매니저 수정
• 파일 혹은 폴더명 수정
• 파일 혹은 폴더 삭제

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

• 커밋 메시지 컨벤션에 맞게 작성했습니다.
• 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

📣 To Reviewers

• response에 law_name을 추가

• Reviewers : 팀 선택

• Labels : 작업 유형, 자기 자신

Summary by CodeRabbit

개선사항

• 개선사항
  • 규정 검색 결과에 법령명 필드가 추가되었습니다
  • 각 참고문서는 이제 문서 제목과 함께 해당 법령명을 함께 표시합니다
  • 검색 결과를 통해 규정의 정확한 출처와 관련 법령을 더욱 명확하게 파악할 수 있습니다

[coderabbitai]

📝 Walkthrough

Walkthrough

이 변경은 규정 검색 API 응답 데이터 모델에 법령명(lawName) 필드를 추가합니다. 관련 DTO 객체들의 생성자에 새로운 매개변수를 추가하고, 스텁 클라이언트의 호출 사이트를 업데이트하여 일관성을 유지합니다.

Changes

Cohort / File(s)	Summary
DTO 필드 추가 src/main/java/com/lastcommit/piilot/domain/regulation/dto/response/AiRegulationSearchResponseDTO.java, src/main/java/com/lastcommit/piilot/domain/regulation/dto/response/ReferenceDocumentDTO.java	lawName 필드를 DTO 레코드에 추가하여 JSON 직렬화/역직렬화 시 법령명 정보를 포함하도록 변경.
생성자 호출 업데이트 src/main/java/com/lastcommit/piilot/domain/regulation/client/StubRegulationSearchAiClient.java, src/main/java/com/lastcommit/piilot/domain/regulation/dto/response/RegulationSearchResponseDTO.java	AiReferenceDTO와 ReferenceDocumentDTO 생성자 호출 시 새로운 lawName 매개변수를 추가하고, 스텁 클라이언트에서 각 참조 항목의 제목을 전달.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• [Feat] #27 법령/내규 검색 API #29: 이번 PR에서 직접 수정하는 AiReferenceDTO 및 ReferenceDocumentDTO 타입이 PR #29에서 도입되었으며, 생성자 호출 사이트 업데이트와 필드 추가가 연관되어 있습니다.

Suggested reviewers

• chanudevelop
• L-eunbin
• jangyunhyuk

Poem

🐰 법령명 필드를 쏙 넣고,
DTO 레코드를 반짝반짝하게,
생성자 호출도 맞춰 정렬하며,
응답 데이터는 더욱 풍성해지네! ✨
규정 검색의 완성을 향해! 🎉

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목이 변경사항의 주요 내용을 명확하게 요약하고 있습니다. '법령/내규검색 API 항목 추가'는 law_name 필드 추가라는 핵심 변경을 정확히 반영합니다.
Linked Issues check	✅ Passed	이슈 #58의 요구사항인 'response에 law_name 항목 추가'가 완전히 구현되었습니다. 네 개의 DTO 클래스에서 lawName 필드를 추가하여 요구사항을 충족합니다.
Out of Scope Changes check	✅ Passed	모든 변경사항이 #58 이슈의 요구사항과 관련이 있으며, law_name 필드 추가라는 목표에 맞춰 실행되었습니다. 범위 외의 변경사항이 없습니다.

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

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/#58-law-search-api-add

---

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 and usage tips.}

[coderabbitai]

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

src/main/java/com/lastcommit/piilot/domain/regulation/dto/response/ReferenceDocumentDTO.java (1)

3-10: ⚠️ Potential issue | 🟡 Minor

lawName 필드 JSON 직렬화 형식 일관성 확인 필요

lawName 필드가 올바르게 추가되었으나, JSON 직렬화 형식에 일관성 문제가 있습니다.

AiRegulationSearchResponseDTO의 중첩 클래스 AiReferenceDTO는 @JsonProperty("document_title")과 @JsonProperty("law_name")으로 snake_case를 명시하고 있습니다. 반면, ReferenceDocumentDTO는 이러한 어노테이션이 없어 기본 Jackson 동작에 따라 camelCase(lawName)로 직렬화됩니다.

같은 참고 문헌 데이터가 API 응답에서 서로 다른 필드명으로 반환되는 것을 방지하기 위해, ReferenceDocumentDTO에도 @JsonProperty("document_title")과 @JsonProperty("law_name") 어노테이션 추가를 권장합니다.