Skip to content

feat(ai): add transcript-aware chat prompts and TranscriptChatRequest#44

Open
rivkode wants to merge 1 commit into
devfrom
feat/ai-video-transcript-chat
Open

feat(ai): add transcript-aware chat prompts and TranscriptChatRequest#44
rivkode wants to merge 1 commit into
devfrom
feat/ai-video-transcript-chat

Conversation

@rivkode
Copy link
Copy Markdown
Owner

@rivkode rivkode commented Apr 26, 2026

Summary

  • AiChatCommand.AiRequest.TranscriptChatRequest 추가 (conversation history + video transcript context 운반)
  • AiChatService.greetingTranscriptChat / generateTranscriptChat 추가 — transcript 컨텍스트를 받아 AI 응답을 persist
  • AiPromptService가 두 개의 신규 prompt 리소스를 로드하고 persona/history/transcript_context placeholder를 렌더링
  • 신규 프롬프트 리소스
    • prompts/system-transcript-chat-message.st — "자모" 페르소나, transcript에 grounded
    • prompts/user-transcript-chat-message.sttranscript_context/history placeholder만 포함

Notes

  • 시크릿 없음 (프롬프트는 일반 가이드 텍스트)
  • 이 PR 단독으로는 호출자가 없음. chat 도메인 PR에서 ChatServiceImpl이 transcript 분기를 추가하면서 사용

Test plan

  • ./gradlew compileKotlin compileTestKotlin 통과
  • AiPromptService 단위 테스트가 있다면 통과
  • 후속 chat 도메인 PR과 함께 머지 후 video transcript chatroom flow가 정상 동작하는지 확인

🤖 Generated with Claude Code

- Add AiChatCommand.AiRequest.TranscriptChatRequest carrying conversation
  history plus video transcript context
- AiChatService gains greetingTranscriptChat / generateTranscriptChat that
  use the new request type and persist the AI response with the right
  ChatRoom and SenderType.AI
- AiPromptService loads two new prompt resources and renders the
  system/user messages with persona, history, and transcript_context
- Add prompt resources:
  - prompts/system-transcript-chat-message.st (Korean "자모" persona that
    must stay grounded in the transcript)
  - prompts/user-transcript-chat-message.st (transcript_context + history
    placeholders only)

No secret values introduced; prompt resources contain only generic
guidance text.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant