← 미러링부스
Browser 콜백 AsyncStream 전환 (PR #287)
2026.02.04적용 완료
클로저를 AsyncStream으로 옮겼습니다. 동료가 준 개선 제안은 근거를 따져 하나는 받아들이고, 하나는 그대로 뒀습니다.
배경
- Browser의 이벤트 처리 방식을 클로저에서 AsyncStream으로 옮기는 후속 작업 (PR #287)
판단
- 콜백 8개를 AsyncStream으로 옮기고, 다른 컴포넌트와 결합도가 높은 onRemoteModeCommand·onSelectedTimerModeCommand 두 개는 이번 PR 범위에서 의도적으로 제외
- 이벤트 성격별로 버퍼 정책을 다르게 설계 (놓치면 안 되는 이벤트는 무제한, 하트비트는 최신 1개만 유지)
- 동료의 '하트비트 처리 로직 공통화' 제안은 화면별로 실제 동작이 달라 복잡도만 늘어난다고 판단해 거절
- 같은 프로젝트의 다른 PR에서는 동료 제안(공유 옵션 캡슐화)을 그 자리에서 수용
결론
- 제안은 무조건 따르거나 거절하지 않고, 매번 근거를 따져 판단