chore: 호스트가 처리하지 못한 웹뷰 메시지를 관측한다

[seongwon030]

#34 위에 쌓은 PR입니다. 같은 파일 3개를 건드려 base를 fix/club-detail-subscribe-toggle로 뒀습니다. #34 머지 후 base를 main으로 바꾸겠습니다.

왜

같은 웹 화면이 홈 웹셸, ClubDetailScreen, webview/[slug] 세 WebView에서 도는데 메시지 처리 구현은 두 벌(공용 훅 + 홈 인라인 switch)입니다. 호스트마다 아는 메시지가 다른데, 어긋나도 아무 신호가 없습니다. 공용 훅은 console.warn만 남기고 홈은 default도 없이 바깥 catch가 삼킵니다.

그래서 웹이 6월(Moadong/moadong#1665)부터 보내던 SUBSCRIBE_TOGGLE을 상세가 버리고 있다는 걸 12월에 제보로 알았습니다(#34).

무엇을

각 호스트 switch의 default에서 리포터를 부릅니다. dev는 console.warn, prod는 Mixpanel Bridge UnknownMessage.

파일
utils/webview.ts	reportUnknownBridgeMessage 추가
constants/eventname.ts	DIAGNOSTIC_EVENT 그룹
hooks/use-webview-message-handler.ts	host 필수화, default에서 리포트
ui/home/home-webview-screen.tsx	switch에 default 추가
ui/club-detail/club-detail-screen.tsx, app/webview/[slug].tsx	host 전달

+41 / -3

판별을 타입 집합 대조가 아니라 default로 한 이유 — 홈만 처리하는 REQUEST_APP_VERSION이 WebViewMessageTypes에 없어서, 집합 대조는 홈이 정상 처리한 메시지를 오탐합니다.

문자열 type만 취급 — 서드파티 비정형 메시지 필터. JSON 파싱 실패는 대상 아닙니다(이미 로그로 드러남).

안전성

리포터는 void를 반환하고 예외를 밖으로 내보내지 않습니다. default는 원래 아무것도 안 하던 자리라 기존 9종 처리 경로는 그대로입니다.

검증

• npx tsc --noEmit
• npm run lint — 에러 0 (경고 5개는 기존 styled 임포트, 무관)
• 실기기 확인 아직 안 했습니다 — 웹 콘솔에서 window.ReactNativeWebView.postMessage(JSON.stringify({ type: 'TEST_UNKNOWN' }))

범위 밖

구현을 한 벌로 합치는 건 별건입니다(REQUEST_APP_VERSION, 호스트마다 다른 OPEN_EXTERNAL_URL·NAVIGATE_BACK 흡수 필요). 죽은 코드(NOTIFICATION_SUBSCRIBE/UNSUBSCRIBE — 웹 호출처 0개)도 그대로 뒀습니다.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: e7f8c1d8-ad1e-4c50-8afa-0d92a6a187d8

📥 Commits

Reviewing files that changed from the base of the PR and between ffb53d0 and 430b177.

📒 Files selected for processing (6)

• app/webview/[slug].tsx
• constants/eventname.ts
• hooks/use-webview-message-handler.ts
• ui/club-detail/club-detail-screen.tsx
• ui/home/home-webview-screen.tsx
• utils/webview.ts

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

---

Walkthrough

WebView 브리지 메시지 핸들러에 호스트 식별자를 추가했습니다. 알 수 없는 메시지는 호스트, 앱 버전, 플랫폼과 함께 진단 이벤트로 보고합니다.

Changes

브리지 메시지 진단 보고

Layer / File(s)	Summary
진단 이벤트와 보고 유틸리티 constants/eventname.ts, utils/webview.ts	DIAGNOSTIC_EVENT.BRIDGE_UNKNOWN_MESSAGE와 reportUnknownBridgeMessage를 추가했습니다. 문자열 메시지만 보고하며, 보고 실패는 콘솔에 기록합니다.
메시지 핸들러 호스트 연결 hooks/use-webview-message-handler.ts, app/webview/[slug].tsx, ui/club-detail/club-detail-screen.tsx	Hook 옵션에 필수 host를 추가했습니다. WebView 화면은 각각 webview_slug와 club_detail을 전달합니다. 알 수 없는 메시지는 진단 유틸리티로 전달합니다.
홈 화면의 알 수 없는 메시지 처리 ui/home/home-webview-screen.tsx	홈 화면의 기본 분기에서 알 수 없는 메시지를 reportUnknownBridgeMessage(type, 'home')로 보고합니다.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Feature

Sequence Diagram(s)

sequenceDiagram
  participant WebView
  participant MessageHandler
  participant reportUnknownBridgeMessage
  participant Mixpanel
  WebView->>MessageHandler: 브리지 메시지 전달
  MessageHandler->>reportUnknownBridgeMessage: 알 수 없는 type과 host 전달
  reportUnknownBridgeMessage->>Mixpanel: BRIDGE_UNKNOWN_MESSAGE 이벤트 전송

Loading

Merge Risk: ⚪ Minimal · up to 430b1

The change adds best-effort host-scoped diagnostics for unknown WebView messages without disrupting normal message handling, so no merge-blocking risk is established.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 3 functions across 6 files.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목은 처리되지 않은 WebView 메시지의 관측을 추가하는 주요 변경 사항을 정확하게 설명합니다.

• Fix all pre-merge checks with AI

✨ Finishing Touches 💡 1

📝 Generate docstrings 💡

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch chore/bridge-unknown-message-reporting

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}