Skip to content

[Fix] 소재 프롬프트 수정#148

Merged
marshmallowing merged 1 commit intodevelopfrom
fix/147
Feb 19, 2026
Merged

[Fix] 소재 프롬프트 수정#148
marshmallowing merged 1 commit intodevelopfrom
fix/147

Conversation

@marshmallowing
Copy link
Copy Markdown
Member

@marshmallowing marshmallowing commented Feb 19, 2026

🔍️ 작업 내용

✨ 상세 설명

두께감(선택지 전체)만 단독으로 선택했을 때 보풀, 비침 등 관련 없는 소재 속성 문장이 생성되는 문제

  • 원인
    • 기존 코드에서 두께감:선택지전체는 continue로 건너뛰어 선택한 소재 특징: 목록이 비어있는 상태가 됨. 그런데 그 아래에 "위 리스트에 나열된 각 특징마다 2문장씩 생성할 것" 규칙이 먼저 등장해, LLM이 리스트가 비어있음에도 시스템 프롬프트의 소재 속성 예시(보풀, 비침 등)를 참고해 스스로 채워버리는 문제 발생
  • 변경 내용
    • 두께감 단독 선택 케이스를 별도 분기로 분리

🛠️ 추후 리팩토링 및 고도화 계획

📸 스크린샷 (선택)

💬 리뷰 요구사항

Summary by CodeRabbit

릴리스 노트

  • 개선 사항
    • 소재 리뷰 생성 로직을 최적화하여 선택한 특징에 따라 더욱 정교한 프롬프트를 생성합니다.
    • 두께감 기능 선택 시 전용 처리 방식을 도입하여 더 나은 리뷰 품질을 제공합니다.
    • 다양한 소재 특징 조합에 대응하는 향상된 구조를 적용했습니다.

@marshmallowing marshmallowing self-assigned this Feb 19, 2026
@marshmallowing marshmallowing added the 🐞 Fix 버그 수정 label Feb 19, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Feb 19, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

소재 프롬프트 생성 로직을 재구성하여 두께감 단독 선택 시 독립적인 프롬프트를 생성하고, 다른 특징과 함께 선택될 경우 통합된 규칙을 적용하는 조건부 분기 구조로 변경했습니다.

Changes

Cohort / File(s) Summary
소재 프롬프트 조건부 분기 로직
src/main/java/com/ongil/backend/domain/review/service/prompter/MaterialReviewPrompter.java
두께감 단독 선택 여부(hasThicknessAll && otherFeatures.isEmpty())에 따라 세 가지 분기로 구분: (1) 두께감만 선택 시 세로바 구분자를 사용한 전용 프롬프트, (2) 다른 특징이 있을 경우 통합 규칙 적용, (3) 두께감 병행 시 특수 지시 섹션 추가. 기존 반복문 기반 처리를 조건 기반 접근으로 전환.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

✨ 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 fix/147

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

Comment @coderabbitai help to get the list of available commands and usage tips.

@marshmallowing marshmallowing merged commit a9b5c9b into develop Feb 19, 2026
1 check was pending
@noeyhgnas
Copy link
Copy Markdown

바쁘네

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 Fix 버그 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Fix] 소재 프롬프트 수정

2 participants