apiClient 코드 중복 제거 및 네이티브 브릿지 인증 동기화 중앙화

[ff1451]

✨ 요약

• sendRequest / sendRequestWithoutRetry 중복 함수를 allowRetry 파라미터로 통합하고, 공통 로직을 buildUrl, buildFetchOptions, executeFetch로 분리
• client.ts의 모듈 스코프 refreshPromise 제거 (auth/index.ts의 dedup에 위임)
• postNativeMessage 유틸을 추출하여 모든 네이티브 브릿지 호출(TOKEN_REFRESH, LOGIN_COMPLETE, LOGOUT)을 타입 안전하게 중앙화
• clearAuth() 내부에서 LOGOUT 브릿지 전송 → 회원탈퇴, 알림 스트림 등 모든 인증 해제 경로에서 네이티브 동기화 보장
• initialize() refresh 실패 경로도 clearAuth() 경유하도록 통일
• parseResponse JSON 파싱 실패 시 ApiError 형태로 래핑 (status, statusText, url 포함), 204 No Content 명시 처리

😎 해결한 이슈

close #243

Summary by CodeRabbit

릴리스 노트

• 버그 수정

  • API 응답 파싱 실패 시 오류 처리 개선

• 리팩토링

  • 인증 토큰 갱신 메커니즘 최적화로 성능 향상
  • API 요청 생성 및 실행 프로세스 개선
  • 인증 요청 재시도 로직 정교화
  • 네이티브 애플리케이션 간 메시지 통신 표준화

[coderabbitai]

Warning

Rate limit exceeded

@ff1451 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 9 minutes and 29 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 9 minutes and 29 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: ea19b547-0bce-4ba3-b747-35cfc162ba6b

📥 Commits

Reviewing files that changed from the base of the PR and between c8a5cfc and 7ab1637.

📒 Files selected for processing (1)

• src/apis/client.ts

Walkthrough

apiClient의 인증 로직을 리팩토링하고 네이티브 브릿지 메시지 전송을 통합했습니다. sendRequest에 allowRetry 파라미터를 추가하여 sendRequestWithoutRetry 제거, 요청 구성 로직을 buildUrl, buildFetchOptions, executeFetch 헬퍼로 분리했습니다. refreshPromise 캐싱을 제거하고, parseResponse에서 JSON 파싱 실패 시 ApiError를 throw하도록 개선했습니다. nativeBridge.ts의 새로운 postNativeMessage 유틸로 TOKEN_REFRESH, LOGIN_COMPLETE, LOGOUT 메시지 전송을 중앙화했습니다.

Possibly related PRs

• \[fix] apiClient 로직 수정 및 ErrorBoundary 추가 #150: handleUnauthorized의 재시도 로직 및 네이티브 브릿지 토큰 포스팅 플로우를 동시에 변경합니다.
• \refactor: 푸시 알림 토큰 로직 브릿지 방식으로 수정 #134: authStore, useLogout 등 여러 파일에서 ReactNativeWebView 호출을 postNativeMessage로 통합하는 동일한 리팩토링이 포함됩니다.

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목이 주요 변경사항(apiClient 중복 제거, 네이티브 브릿지 중앙화)을 명확하게 요약하고 있습니다.
Linked Issues check	✅ Passed	코드 변경사항이 #243의 모든 핵심 목표(sendRequest 통합, refreshPromise 제거, postNativeMessage 중앙화, clearAuth 내 LOGOUT 동기화, parseResponse 에러 처리)를 충족합니다.
Out of Scope Changes check	✅ Passed	모든 변경사항이 #243의 정의된 범위 내에 있으며, 명시된 리팩토링 목표에 직접 관련되어 있습니다.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch 243-refactor-apiClient-네이티브-브릿지-인증-개선

---

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 and usage tips.}

[coderabbitai]

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/apis/client.ts`:
- Around line 146-173: The request body handling currently JSON.stringifys
everything except Blob and FormData, which wrongly serializes other valid
BodyInit types (URLSearchParams, ArrayBuffer, TypedArray, DataView, etc.);
update the guard around JSON.stringify (the block setting fetchOpts.body and the
earlier isJsonBody/header logic) to only stringify plain objects and arrays
(e.g., Array.isArray(body) || (typeof body === 'object' && body !== null &&
body.constructor === Object)), and explicitly exclude Blob, FormData,
URLSearchParams, ArrayBuffer, ArrayBuffer views (ArrayBuffer.isView), and
DataView so URLSearchParams, ArrayBuffer/typed arrays, DataView, Blob, and
FormData are passed through unchanged while plain objects/arrays are
JSON.stringified. Ensure the Content-Type header (isJsonBody/h) aligns with this
narrower check.

In `@src/stores/authStore.ts`:
- Around line 90-91: The current clearAuth() always sends the native LOGOUT
bridge event, but it's being used for transient refresh failures as well; change
clearAuth() so it only clears in-memory/state tokens and does NOT send the
bridge event, and create a separate explicit method (e.g.,
sendNativeLogoutBridge() or clearAuthAndNotify()) that both clears auth and
sends the LOGOUT bridge; update callers that represent transient failures (those
in client refresh failure paths and useInboxNotificationStream) to call the
plain clearAuth() (no bridge) while explicit user actions (logout/unregister
flows) call the new combined method to send the bridge.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 9eeb0fde-30d5-40f5-bc8d-cef6d943d290

📥 Commits

Reviewing files that changed from the base of the PR and between 78e1960 and c8a5cfc.

📒 Files selected for processing (5)

• src/apis/client.ts
• src/components/notification/hooks/useInboxNotificationStream.ts
• src/pages/User/MyPage/hooks/useLogout.ts
• src/stores/authStore.ts
• src/utils/ts/nativeBridge.ts

💤 Files with no reviewable changes (1)

• src/pages/User/MyPage/hooks/useLogout.ts

[coderabbitai]

⚠️ Potential issue | 🟠 Major

clearAuth()가 일시적 refresh 실패까지 네이티브 로그아웃으로 확장됩니다.

이제 초기화 실패도 clearAuth()를 타고, clearAuth()는 항상 LOGOUT을 전송합니다. 그런데 이 메서드는 src/apis/client.ts와 src/components/notification/hooks/useInboxNotificationStream.ts의 refresh 실패 경로에서도 호출되므로, 네트워크/5xx 같은 일시적 실패까지 네이티브 로그아웃으로 처리될 수 있습니다. 명시적 로그아웃/회원탈퇴에서만 브릿지를 보내도록 분리하는 편이 안전합니다.

Also applies to: 114-118

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/stores/authStore.ts` around lines 90 - 91, The current clearAuth() always
sends the native LOGOUT bridge event, but it's being used for transient refresh
failures as well; change clearAuth() so it only clears in-memory/state tokens
and does NOT send the bridge event, and create a separate explicit method (e.g.,
sendNativeLogoutBridge() or clearAuthAndNotify()) that both clears auth and
sends the LOGOUT bridge; update callers that represent transient failures (those
in client refresh failure paths and useInboxNotificationStream) to call the
plain clearAuth() (no bridge) while explicit user actions (logout/unregister
flows) call the new combined method to send the bridge.

[Copilot]

Pull request overview

apiClient의 중복 로직을 정리하고, WebView ↔ 네이티브 브릿지 인증 동기화를 postNativeMessage/clearAuth() 중심으로 중앙화하여 인증 상태 불일치를 줄이려는 PR입니다.

Changes:

• sendRequest/sendRequestWithoutRetry를 allowRetry 파라미터 기반으로 통합하고 URL/옵션/실행 로직을 함수로 분리
• 네이티브 브릿지 메시지 전송을 postNativeMessage 유틸로 추출하고 호출 지점을 통일
• clearAuth()에서 LOGOUT 브릿지 전송 및 parseResponse의 JSON 파싱 실패 처리 개선(204 처리 포함)

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
src/utils/ts/nativeBridge.ts	타입 안전한 postNativeMessage 유틸 추가
src/stores/authStore.ts	인증 실패 경로를 clearAuth()로 통일하고 LOGOUT 브릿지 동기화 보장
src/pages/User/MyPage/hooks/useLogout.ts	개별 LOGOUT 브릿지 호출 제거(스토어 clearAuth()에 위임)
src/components/notification/hooks/useInboxNotificationStream.ts	TOKEN_REFRESH 브릿지 호출을 postNativeMessage로 중앙화
src/apis/client.ts	요청/재시도/응답 파싱 리팩터링 및 TOKEN_REFRESH 브릿지 동기화 추가

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[ff1451]

@CodeRabbit review