Skip to content

[refactor/#379] Post summary pipeline 책임 정리#390

Merged
Dimo-2562 merged 8 commits into
developfrom
refactor/#379
May 10, 2026
Merged

[refactor/#379] Post summary pipeline 책임 정리#390
Dimo-2562 merged 8 commits into
developfrom
refactor/#379

Conversation

@Dimo-2562
Copy link
Copy Markdown
Contributor

❤️ 기능 설명

  • Post summary pipeline의 책임을 Processor / Reader / Writer / SummaryExtractionService / Post aggregate 기준으로 다시 정리했습니다.
  • Post가 게시글 키워드 재구성 책임을 가지도록 캡슐화했습니다.
  • malformed LLM JSON 응답은 더 이상 빈 DTO로 삼키지 않고 fail-fast 하도록 변경했습니다.
  • summary reader/writer contract를 H2 @DataJpaTest로 보강했습니다.
  • 관련 테스트 갭 문서와 roadmap 문서를 최신화하고, 남아 있는 우려사항도 함께 기록했습니다.

Swagger 테스트 성공 결과 스크린샷 첨부

  • Swagger 변경 없음 (배치/도메인/테스트/문서 리팩터링 범위)

연결된 issue

close #379


🩷 Approve 하기 전 확인해주세요!

  • summary 단계와 embedding 단계 상태가 아직 summary IS NULL OR '', embeddedAt IS NULL 조합으로 암묵적으로 표현되는 점
  • malformed JSON fail-fast 이후에도 response-format 실패와 transport/API 실패가 모두 LlmException 버킷으로 묶이는 점
  • PostSummaryReader가 미요약 backlog를 한 번에 메모리로 올리는 구조라, 데이터 증가 시 paging/streaming reader 후속 전환이 필요할 수 있는 점

✅ 체크리스트

  • PR 제목 규칙 잘 지켰는가?
  • 추가/수정사항을 설명하였는가?
  • 테스트 결과 사진을 넣었는가? (Swagger/API 변경 없음, 아래 테스트 명령으로 대체)
  • 이슈넘버를 적었는가?

테스트

./gradlew test --tests '*SummaryExtractionServiceTest' --tests '*PostSummaryProcessorTest'
./gradlew test --tests '*PostSummaryWriterDataJpaTest'
./gradlew test --tests '*PostSummaryReaderDataJpaTest'
./gradlew test --tests '*PostSummaryProcessorTest' --tests '*PostSummaryReaderTest' --tests '*PostSummaryReaderDataJpaTest' --tests '*PostSummaryWriterTest' --tests '*PostSummaryWriterDataJpaTest' --tests '*SummaryExtractionServiceTest' --tests '*PostTest'

@Dimo-2562 Dimo-2562 added the 🌟 REFACTOR 리팩터링 (기능 변화 X) label May 10, 2026
@Dimo-2562 Dimo-2562 self-assigned this May 10, 2026
@Dimo-2562 Dimo-2562 added the 📝 domain:post 게시글 관련 label May 10, 2026
@Dimo-2562 Dimo-2562 requested review from Joon9750 and rover0811 and removed request for Joon9750 and rover0811 May 10, 2026 04:26
@Dimo-2562 Dimo-2562 merged commit aa84cba into develop May 10, 2026
1 check passed
@Dimo-2562 Dimo-2562 deleted the refactor/#379 branch May 10, 2026 04:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📝 domain:post 게시글 관련 🌟 REFACTOR 리팩터링 (기능 변화 X)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[REFACTOR] Post summary pipeline 책임 정리

1 participant