Skip to content

[feat] 백오피스 LLM 설정에 카드 감정 프롬프트 탭 추가 - #137

Open
theminjunchoi wants to merge 4 commits into
feat/134-card-emotion-extractionfrom
feat/135-backoffice-card-emotion-tab
Open

[feat] 백오피스 LLM 설정에 카드 감정 프롬프트 탭 추가#137
theminjunchoi wants to merge 4 commits into
feat/134-card-emotion-extractionfrom
feat/135-backoffice-card-emotion-tab

Conversation

@theminjunchoi

Copy link
Copy Markdown
Collaborator

🔗 연관 이슈

📌 개요

PR ①(#136)에서 PromptType.CARD_EMOTION이 신설돼 백오피스 API로는 이미 조회·수정·리비전이 되지만, 화면에 탭이 없어 관리자가 편집할 수 없다. 탭을 추가하고, "댓글·답글·카드"로 타입을 나열한 하드코딩 안내 문구를 실제 조립 규칙과 맞춘다.

🔧 주요 변경사항

  • 탭별 문구를 탭 정의로 정리(선행 리팩터링) — 라벨·주의사항이 JSX 안 3중 중첩 삼항으로 흩어져 있어 타입을 하나 늘릴 때마다 분기가 깊어지는 구조였다. PromptTabeditorLabel·editorNote·standalone을 두고 JSX는 탭 정의를 읽기만 하도록 바꿈 — 조립 배너 조건도 type !== 'EONGTTUNG_TOPIC' 하드코딩에서 standalone 플래그로
  • '카드 감정' 탭 추가 — 조회·수정·리비전 이력·복원은 기존 컴포넌트가 타입 제네릭이라 그대로 동작
  • 안내 문구 갱신 — ① 모델 카드: "댓글·답글·카드 생성과 카드 감정 분류가 모두 이 모델을 씁니다" ② 공통 탭: 공통 프롬프트가 반영되는 범위에서 카드 감정·엉뚱이 소재가 제외됨을 명시. 감정 분류는 COMMON과 조립되지 않는 단독 프롬프트라, 이 문구가 없으면 "공통을 바꾸면 감정 추출에도 반영된다"는 오해가 생긴다
  • Swagger 태그 설명CARD_EMOTIONEONGTTUNG_TOPIC처럼 조립에 쓰이지 않는 타입임을 명시
  • 통합 테스트 — 백오피스 API가 신규 타입을 조회·수정하고 리비전 기록자가 남는지 검증

🌐 API · DB 영향

  • API 변경 없음 (기존 promptType enum에 이미 추가된 값을 화면에서 쓸 뿐)
  • DB 마이그레이션 없음 (시딩은 PR ①의 V28)
  • 하위 호환: 앱 클라이언트 영향 없음

💬 리뷰 포인트

  • 첫 커밋의 리팩터링 범위 — 탭 하나 추가하려면 삼항 분기를 4중으로 늘려야 해서 먼저 정리했습니다. 기존 5개 탭의 화면 출력 문자열은 그대로입니다(공통 탭 문구만 의도적으로 갱신)
  • 프롬프트 실험실(prompt-playground.tsx)은 현재 COMMENT/REPLY만 지원해 이번 범위에서 제외했습니다 — 카드 계열 미리보기 지원은 별도 이슈가 필요합니다

@github-actions

Copy link
Copy Markdown

Test Results

533 tests  +1   533 ✅ +1   1m 55s ⏱️ ±0s
 75 suites ±0     0 💤 ±0 
 75 files   ±0     0 ❌ ±0 

Results for commit 0a094e8. ± Comparison against base commit 31aea24.

@github-actions

Copy link
Copy Markdown

Test Coverage

Overall Project 79.81% 🍏
File Coverage
AdminLlmSettingsController.kt 64.6% 🍏

@theminjunchoi theminjunchoi linked an issue Aug 13, 2026 that may be closed by this pull request
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[feat] 백오피스 LLM 설정에 카드 감정 프롬프트 탭 추가

1 participant