Skip to content

fix: 홈에서 보낸 첫 대화의 순차 노출 딜레이가 사라지는 문제 수정 - #39

Open
soyeonLee126 wants to merge 3 commits into
developfrom
fix/add-chat-delay
Open

fix: 홈에서 보낸 첫 대화의 순차 노출 딜레이가 사라지는 문제 수정#39
soyeonLee126 wants to merge 3 commits into
developfrom
fix/add-chat-delay

Conversation

@soyeonLee126

@soyeonLee126 soyeonLee126 commented Aug 15, 2026

Copy link
Copy Markdown
Collaborator

작업 개요

홈에서 첫 메시지를 보내고 채팅방으로 넘어가면, 서버가 이미 만든 댓글 전체를 재조회해 한번에 표시되면서 원래 있던 1~3초 랜덤 순차 노출 딜레이가 사라지는 문제를 수정했다.

작업 유형

  • fix: 버그 수정
  • test: 테스트 추가/수정

변경 사항

  • ConversationSession이 Hilt에서 무스코프라 홈/채팅방이 서로 다른 인스턴스를 받는다는 걸 확인 — 홈에서 저장한 첫 교환을 채팅방이 못 받고 있었음
  • PendingConversationReveal(싱글턴 우편함)을 새로 만들어 홈이 저장한 첫 교환을 채팅방이 한 번 소비하게 하고, 기존 순차 노출 로직(launchCommentReveal)을 그대로 타도록 함
  • 캐시 소비 시 채팅방 쪽 요약·감정 상태도 함께 시드해 다음 전송 문맥과 대화 종료 시 감정 결과가 비지 않게 함
  • 소비되지 못한 값은 30초 뒤 만료시켜, 나중에 같은 대화를 다시 열 때 오래된 캐시가 잘못 쓰이지 않게 함
  • 첫 댓글도 즉시 붙지 않고 나머지와 동일하게 지연 후 노출되도록 변경
  • 노출 간격을 13초에서 0.53초로 조정

관련 이슈

관련 작업 (Notion)

스크린샷 / 동작 화면

체크리스트

  • base 브랜치가 develop으로 설정되어 있다
  • 로컬 빌드 및 실행을 확인했다
  • 커밋 메시지가 컨벤션(feat:, fix: 등)을 따른다
  • 불필요한 로그/주석/디버그 코드를 제거했다
  • 리뷰어가 이해할 수 있도록 설명을 작성했다

리뷰 요청 사항

  • PendingConversationReveal@Singleton으로 둔 판단(요약/감정 상태를 들고 있는 ConversationSession과 달리, 소비되면 비워지는 일회성 우편함이라 인스턴스 간 공유해도 안전하다고 봄)이 타당한지 봐주시면 좋겠습니다.

홈에서 새 대화를 보내면 채팅방으로 넘어가며 ChatRoomViewModel이 서버 메시지를
통째로 재조회해 딜레이 없이 한번에 표시했다. ConversationSession(무스코프)이
갖고 있던 서버 응답을 PendingConversationReveal(싱글턴 우편함)에 담아 채팅방이
한 번 소비하게 하고, 그 자리에서 기존 순차 노출 로직을 그대로 타도록 했다.
소비 시 채팅방 쪽 요약·감정 상태도 함께 시드해 다음 전송 문맥이 비지 않게 하고,
소비되지 못한 값은 30초 뒤 만료시켜 나중에 다른 화면이 잘못 집어가지 않게 했다.

첫 댓글도 도착 즉시 붙지 않고 나머지와 동일하게 지연 후 노출되도록 바꾸고,
노출 간격 하한을 1초에서 0.5초로 낮췄다.
@soyeonLee126
soyeonLee126 marked this pull request as ready for review August 15, 2026 08:05
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