fix: 홈에서 보낸 첫 대화의 순차 노출 딜레이가 사라지는 문제 수정 - #39
Open
soyeonLee126 wants to merge 3 commits into
Open
Conversation
홈에서 새 대화를 보내면 채팅방으로 넘어가며 ChatRoomViewModel이 서버 메시지를 통째로 재조회해 딜레이 없이 한번에 표시했다. ConversationSession(무스코프)이 갖고 있던 서버 응답을 PendingConversationReveal(싱글턴 우편함)에 담아 채팅방이 한 번 소비하게 하고, 그 자리에서 기존 순차 노출 로직을 그대로 타도록 했다. 소비 시 채팅방 쪽 요약·감정 상태도 함께 시드해 다음 전송 문맥이 비지 않게 하고, 소비되지 못한 값은 30초 뒤 만료시켜 나중에 다른 화면이 잘못 집어가지 않게 했다. 첫 댓글도 도착 즉시 붙지 않고 나머지와 동일하게 지연 후 노출되도록 바꾸고, 노출 간격 하한을 1초에서 0.5초로 낮췄다.
soyeonLee126
marked this pull request as ready for review
August 15, 2026 08:05
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
작업 개요
홈에서 첫 메시지를 보내고 채팅방으로 넘어가면, 서버가 이미 만든 댓글 전체를 재조회해 한번에 표시되면서 원래 있던 1~3초 랜덤 순차 노출 딜레이가 사라지는 문제를 수정했다.
작업 유형
변경 사항
ConversationSession이 Hilt에서 무스코프라 홈/채팅방이 서로 다른 인스턴스를 받는다는 걸 확인 — 홈에서 저장한 첫 교환을 채팅방이 못 받고 있었음PendingConversationReveal(싱글턴 우편함)을 새로 만들어 홈이 저장한 첫 교환을 채팅방이 한 번 소비하게 하고, 기존 순차 노출 로직(launchCommentReveal)을 그대로 타도록 함3초에서 0.53초로 조정관련 이슈
관련 작업 (Notion)
스크린샷 / 동작 화면
체크리스트
develop으로 설정되어 있다feat:,fix:등)을 따른다리뷰 요청 사항
PendingConversationReveal을@Singleton으로 둔 판단(요약/감정 상태를 들고 있는ConversationSession과 달리, 소비되면 비워지는 일회성 우편함이라 인스턴스 간 공유해도 안전하다고 봄)이 타당한지 봐주시면 좋겠습니다.