[Fix/#308] CameraPreview 표시 중 Heartbeat 중복 처리를 방지한다.

[YunDaeHyeon]

🔗 연관된 이슈

• closed CameraPreview 표시 중 Heartbeat 중복 처리를 개선한다. #308

📝 작업 내용

📌 요약

CameraPreview의 알럿과 ConnectionCheckView의 pop이 동시에 동작하면서 알럿이 중복으로 출력되며 샤샤샥 사라지는 버그를 해결했습니다.

🔍 상세

문제 상황 재현을 위해 아래와 같이 시나리오를 산정했습니다.

# 사용자 시나리오
BrowsingView -> ConnectionCheckView -> CameraPreview (ConnectionCheckView의 fullScreenCover)

1. 사용자는 촬영기기(iPhone)에서 미러링 기기를 연결하거나 미러링/리모트 기기를 연결한 후 ConnectionCheckView로 이동합니다.
2. 사용자는 하단 "촬영 준비하기" 버튼을 눌러 CameraPreview를 fullScreenCover로 띄웁니다.
3. 그 상태에서 사용자는 미러링기기의 연결을 끊습니다.

• ✅ 이때 기대 동작은 CameraPreview에서 "연결이 끊어졌습니다." 알럿과 함께 홈으로 이동해야합니다.
• ❌ 하지만, 알럿이 잠깐 보였다가 (샤샤샥 하고) BrowsingView로 이동된 뒤 알럿이 다시 발생되며 상태가 꼬이고 있었습니다.

핵심 문제는 CameraPreview를 fullScreenCover로 띄우면 ConnectionCheckView는 dismiss되지 않고 그 아래에 살아있다는 점이었습니다.

그러다보니 Heatbeat 타임아웃 발생 시 브라우저의 모든 이벤트 스트림을 관리하는 BrowserStreamManager가 모든 하트비트 스트림에 .heartbeatTimeout 이벤트를 방출합니다.

이때, ConnectionCheckStore와 CameraPreviewStore가 동시에 이벤트를 수신해서 문제가 발생되고 있었습니다.

• 아래 Store들이 동시에 이벤트를 수신함으로써 발생되는 흐름
  • ConnectionCheckStore는 isMirroringDisconnected를 true로 변경하여 View에서 router.pop() 실행
  • CameraPreviewStores도 isMirroringDisconnected를 true로 변경하여 알럿 표시 후 router.reset() 실행

이렇게 동시에 이벤트를 수신해버리니
CamerePreview의 알럿은 잠깐 보였다가 fullScreenCorver가 dismiss되면서 슈슈슉 사라집니다. 동시에
ConnectionCheckView의 router.pop도 실행되어 BrowsingView로 돌아가버려 네비게이션의 충돌이 발생하고 있었습니다.

이를 해결하기 위해 간단히 guard문을 추가하여 ConnectionCheckView가 CamerePreview를 표시하고 있을 때는 Heartbeat 반응을 건너뛰도록 했습니다.

// ConnectionCheckView.swift
.onChange(of: store.state.isMirroringDisconnected) {
    guard !store.state.showPreview else { return } # ✅ 추가 된 내용~

    store.browser.disconnect(useType: .mirroring)
    router.pop()
}

📸 영상 / 이미지 (Optional)

개선 전

2.mov

개선 후

2.mov

[YunDaeHyeon]

📄 PR Code Review Report

🔍 Summary

isMirroringDisconnected 상태 변화 시 특정 조건(showPreview)에 따라 화면 이탈 및 연결 해제 로직 실행 여부를 제어하는 변경 사항이 있습니다. 이는 특정 사용자 경험 흐름을 위한 의도적인 추가로 보이나, 상태 불일치 및 예기치 않은 동작 가능성에 대한 검토가 필요합니다.

⚠️ Key Issues

1. 상태 불일치 위험: isMirroringDisconnected가 true로 변했지만 showPreview가 true인 상태일 경우 browser.disconnect와 router.pop() 호출이 방지됩니다. 만약 이후 showPreview가 false로 변하더라도 isMirroringDisconnected 값이 다시 변경되지 않으면 연결 해제 로직 및 화면 전환이 누락되어 앱이 미러링이 해제된 상태에서 올바르게 정리되지 않거나 UI가 잘못된 상태를 표시할 수 있습니다. disconnect 호출은 연결 해제 이벤트 발생 시 상태와 무관하게 항상 이루어져야 할 수도 있습니다.

🛠 Improvement Suggestions

1. isMirroringDisconnected가 true가 되었을 때, showPreview 상태와 관계없이 browser.disconnect(useType: .mirroring)가 항상 호출되도록 로직을 분리하는 것을 고려해볼 수 있습니다. router.pop()은 showPreview 상태에 따라 조건부로 실행하더라도, 연결 해제 자체는 항상 처리하는 것이 앱의 안정성을 높일 수 있습니다.
2. 또는 showPreview가 false로 변경될 때 isMirroringDisconnected 상태를 한 번 더 확인하여, 필요한 경우 disconnect 및 pop이 이루어지도록 로직을 추가하여 잠재적인 상태 불일치를 해결하는 것을 고려할 수 있습니다.

✅ Positive Observations

• 특정 조건(showPreview)에서 불필요한 화면 이탈을 방지하려는 의도는 명확하며, 잠재적인 사용자 경험 저하를 막기 위한 노력으로 보입니다.

[sangYuLv]

AI 리뷰의 key issue는 넘어가도 될 듯 합니다!
showPreview가 false가 되는 건 촬영 완료 때만 있으니...

[GRJeon]

가끔 보이던 버근데 이런 문제였군요