isPush 플래그가 만드는 Coordinator 흐름 분기 지옥

어느 날 푸시 알림으로 특정 화면 들어가서 작업 완료하면 본문에 그대로 머무는 버그를 받았다. 일반 진입(리스트 → 상세 → 작업)에선 정상 동작, 푸시 진입에서만 작업 후 리스트로 안 돌아옴. 한참 헤매다 보니 Coordinator 패턴에서 흔히 만들어지는 “진입 경로별 분기 트랩"이었음. 정리해둔다.

Coordinator 체인

문제 화면의 Coordinator 트리는 대략 이런 모양.

HomeCoordinator
└─ ItemTabCoordinator (isPush=true)
    └─ ItemListCoordinator (build 시 isPush 전달)
        └─ ItemDetailCoordinator
            └─ ConfirmCoordinator

푸시 진입과 일반 진입에서 동일한 트리를 빌드하되, 푸시일 때만 isPush=true 플래그가 위에서부터 cascade로 흘러 내려오는 구조.

무엇이 일어났는가

작업 완료 후 종료 흐름은 푸시/일반 모두 동일하게 Detail.shutdown(isSwipe: false) → List.shutdownItem(isSwipe: false)로 진입한다. 그런데 shutdownItem에 이런 분기가 있었다.

1
2
3
4
5
6
7

func shutdownItem(isSwipe: Bool) {
    if isPush {
        sceneFinishDelegate?.finish(to: self)   // ListCoord 자신을 finish
    } else {
        detachChild(type: ItemDetailCoordinator.self)
    }
}

isPush=true 분기는 ListCoord가 자기 자신을 부모(TabCoord)에게 finish 시킨다. TabCoord의 finish도 isPush 분기를 가지고 있어서 더 위로 cascade.

그 결과:

• Detail VC는 단 한 번도 detach/pop되지 않음
• List/TabCoord는 정리되지만 Detail VC는 nav 스택에 그대로 남음

“본문에 머무르는” 증상이 정확히 이거였다. 위쪽 Coordinator는 다 정리됐는데 가장 안쪽 VC만 화면에 살아있는, 잘 보면 좀 무서운 상태 ㅋㅋ.

원래 분기의 의도

코드 짤 당시 의도는 짐작이 됨. “푸시 진입이면 작업 끝나고 화면 자체를 닫고 홈으로” 정책이었을 거다. 근데 실제 요구사항 다시 확인해보면 “푸시든 아니든 작업 후 리스트로 복귀”. 분기 자체가 정책과 어긋난 상태.

그리고 분기 만들 당시엔 한 단계만 보면 됐을 수도 있음. 근데 시간 지나면서 부모 Coordinator에도 같은 isPush 분기가 추가됐고, 그 cascade가 “내가 의도하지 않은 곳"까지 영향을 미친 거.

해결

분기를 제거하고 dismiss 경로를 단일화.

1
2
3
4
5
6
7

func shutdownItem(isSwipe: Bool) {
    if isSwipe {
        detachChildCoordinator(type: ItemDetailCoordinator.self)
    } else {
        detachChild(type: ItemDetailCoordinator.self)
    }
}

isPush 분기 제거. 푸시/일반 모두 동일한 dismiss 경로로 통일. 결과적으로 작업 완료 후 항상 리스트로 복귀.

같은 패턴이 자주 만들어지는 자리

이 함정은 isPush뿐 아니라 진입 경로를 플래그로 추적할 때 어디서나 만들어짐.

• isDeepLink — 딥링크 진입이면 다르게
• isModal — 모달이면 다르게
• isFromOnboarding — 온보딩에서 왔으면 다르게

플래그 받는 순간 dismiss/완료 경로가 두 갈래로 갈리고, 시간 지나면 그 갈래 중 하나가 의도와 다르게 동작하는 모순 발생. 진입 경로 추적은 시작 시점에만 의미가 있는데, 종료 시점의 분기까지 같은 플래그로 표현하면서 문제가 생기는 거다.

교훈

“푸시 진입은 일반 진입이랑 다르다"고 가정하고 isPush 플래그 받는 순간, dismiss 경로가 두 갈래로 갈라짐. 처음엔 둘 다 잘 돌아가지만, 누군가 위에서 cascade finish 호출하면 한쪽이 의도와 정반대로 동작함.

흐름 분기 추가하기 전에 한 번만 더 묻자.

• 이 분기가 진짜 두 가지 다른 동작을 요구하는 비즈니스 요구사항이냐?
• 아니면 그냥 진입 경로만 다른 거냐?

후자라면 분기 만들지 말고 동일한 흐름 유지하자. 진입은 분기, 종료는 통일 — 이거 하나만 지켜도 Coordinator 패턴이 한결 안전해짐.