[feat] 페이지 접근 권한 전역 관리 구현

[mgYang53]

🚀 풀 리퀘스트 제안

📋 작업 내용

현재 프로젝트는 로그인 여부와 온보딩 완료 여부만 라우트 레벨에서 체크하고 있어, 비멤버가 URL 직접 입력으로 모임 상세/설정/약속 하위 페이지 등에 접근할 수 있는 문제가 있었습니다.

인터셉터에서 권한 에러 코드를 감지하고 커스텀 DOM 이벤트(permission-denied)를 dispatch하면, RootLayout에 등록된 usePermissionRedirect 훅이 에러 토스트와 홈 리다이렉트를 처리하는 방식으로 SPA 내비게이션을 유지하면서 한 곳에서 통합 관리합니다.

🔧 변경 사항

• errors.ts: PAGE_ACCESS_ERROR_CODES Set 추가 — 리다이렉트 대상 에러 코드 6개 정의 (GA002, GA003, M003, M004, M006, R105)
• interceptors.ts: 권한 에러 감지 후 permission-denied 커스텀 이벤트 dispatch
• usePermissionRedirect.ts: 신규 훅 생성 — 이벤트 listen → showErrorToast + navigate(HOME, { replace: true })
• RootLayout.tsx: usePermissionRedirect() 연결 (모든 페이지에 자동 적용)
• MeetingRetrospectiveCreatePage.tsx: useEffect에서 권한 에러 스킵 처리 (이중 내비게이션 방지)

📄 기타

관련 이슈: #99

이슈 체크리스트 변경 사항:

• TopicCreatePage — useMeetingDetail 쿼리 추가는 추후 처리로 미룸. 현재는 mutation 제출 시 권한 에러가 발생하여 인터셉터가 감지하고 홈으로 리다이렉트됨.
• MeetingRetrospectiveCreatePage — useMeetingDetail 추가 대신, 이미 존재하는 useCollectedAnswers 쿼리가 권한 체크 역할을 하며, 기존 useEffect의 이중 내비게이션 충돌을 수정하는 방식으로 처리.

Summary by CodeRabbit

• 개선 사항
  • 권한 부족 상황에 대한 전역 처리 추가: 접근 거부 시 사용자에게 명확한 오류 알림을 표시하고 홈으로 자동 리다이렉트됩니다.
  • 반복되는 접근 거부 알림을 묶어 중복 알림을 줄였습니다.
  • 각 페이지의 기존 로컬 오류 처리 로직은 전역 처리로 위임되어 동일한 오류에 대해 중복된 모달/토스트가 표시되지 않습니다.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 2abaf124-e26e-424e-823e-580574ca3c29

📥 Commits

Reviewing files that changed from the base of the PR and between f79e3eb and 6061c0f.

📒 Files selected for processing (2)

• src/pages/Retrospectives/meeting/MeetingRetrospectiveCreatePage.tsx
• src/pages/Topics/TopicCreatePage.tsx

🚧 Files skipped from review as they are similar to previous changes (1)

• src/pages/Retrospectives/meeting/MeetingRetrospectiveCreatePage.tsx

---

Walkthrough

권한 거부 관련 에러를 전역으로 처리하도록 변경합니다. 인터셉터가 특정 에러 코드를 감지하면 'permission-denied' 커스텀 이벤트를 방출하고, 전역 훅이 이를 받아 토스트를 띄운 뒤 홈으로 리다이렉트합니다. 일부 페이지의 로컬 에러 처리는 건너뜁니다.

Changes

Cohort / File(s)	Summary
에러 코드 정의 src/api/errors.ts	PAGE_ACCESS_ERROR_CODES 상수 추가 (6개 권한 관련 에러 코드). diff에 동일 블록이 중복 삽입된 것으로 보이므로 파일 내 중복 등록 여부 확인 필요.
API 계층 이벤트 발행 src/api/interceptors.ts	응답 에러 처리에서 PAGE_ACCESS_ERROR_CODES에 포함된 코드 발견 시 permission-denied CustomEvent 발행. 이벤트 중복 방지를 위해 1초 디바운스 로직 추가.
전역 권한 거부 훅 src/shared/hooks/usePermissionRedirect.ts, src/shared/hooks/index.ts	usePermissionRedirect 훅 추가 — permission-denied 이벤트 리스닝, 토스트 표시(showErrorToast) 후 ROUTES.HOME로 replace:true 네비게이션. 인덱스에서 export 추가.
RootLayout 통합 src/shared/layout/RootLayout.tsx	RootLayout에서 usePermissionRedirect() 호출 추가로 앱 전역에서 이벤트 처리 활성화.
로컬 처리 건너뜀 src/pages/Retrospectives/...MeetingRetrospectiveCreatePage.tsx, src/pages/Topics/TopicCreatePage.tsx	로컬 에러 핸들러에서 PAGE_ACCESS_ERROR_CODES에 해당하면 기존 토스트/모달/네비게이션 처리를 건너뛰도록 수정 (전역 훅으로 대체).

Sequence Diagram(s)

sequenceDiagram
    participant Client as Client
    participant Interceptor as API 인터셉터
    participant Window as Window
    participant Hook as usePermissionRedirect
    participant Toast as Toast UI
    participant Router as React Router

    Client->>Interceptor: API 요청
    Interceptor-->>Client: 권한 거부 에러 응답

    rect rgba(255, 100, 100, 0.5)
    Interceptor->>Interceptor: PAGE_ACCESS_ERROR_CODES 검사\n(1초 디바운스 로직)
    end

    alt 권한 거부 + 디바운스 통과
        Interceptor->>Window: 'permission-denied' 이벤트 발행 (detail: ErrorMessage)
        Window->>Hook: 이벤트 수신
        Hook->>Toast: showErrorToast(message)
        Hook->>Router: navigate(ROUTES.HOME, {replace:true})
    else 디바운스 차단 또는 기타 에러
        Interceptor->>Client: 기존 ApiError 반환/거부
    end

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related issues

• [feat] 페이지 접근 권한 관리 구현 #99: 동일한 기능(권한 에러 상수 추가, 인터셉터 이벤트 발행, 전역 훅 추가 및 페이지 가드) 구현과 일치합니다.

Suggested reviewers

• choiyoungae
• haruyam15

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 40.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목은 주요 변경 사항인 페이지 접근 권한의 전역 관리 구현을 명확하고 간결하게 요약하고 있습니다.

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

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/page-permission-99

---

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/api/interceptors.ts`:
- Around line 124-129: The permission-denied event is currently dispatched on
every failed response (see PAGE_ACCESS_ERROR_CODES, ErrorMessage lookup and the
CustomEvent('permission-denied') dispatch), causing duplicate toasts/redirects
on concurrent failures; add a short-window duplicate guard (e.g., module-scoped
or window-scoped lastPermissionDeniedAt timestamp) checked before dispatching
and updated after dispatch so that if Date.now() - lastPermissionDeniedAt <
threshold (e.g., 1–2s) you skip emitting the event — keep the existing
errorMessage/detail logic and only suppress duplicate dispatches within that
timeframe.

In `@src/shared/hooks/usePermissionRedirect.ts`:
- Around line 19-21: The event handler in usePermissionRedirect (the handler
function) currently destructures message from (e as CustomEvent).detail without
null checks; change handler to safely extract the payload: verify e is a
CustomEvent and that e.detail exists and that detail.message is a string (or use
optional chaining and a default fallback) before calling showErrorToast, and
pass a default error message when message is missing to avoid runtime crashes in
the global permission hook.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 184af1fc-bd2b-4772-9430-559bb98e39ae

📥 Commits

Reviewing files that changed from the base of the PR and between ff88378 and 15f8d4c.

📒 Files selected for processing (6)

• src/api/errors.ts
• src/api/interceptors.ts
• src/pages/Retrospectives/MeetingRetrospectiveCreatePage.tsx
• src/shared/hooks/index.ts
• src/shared/hooks/usePermissionRedirect.ts
• src/shared/layout/RootLayout.tsx

[coderabbitai]

Actionable comments posted: 1

🤖 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/pages/Retrospectives/meeting/MeetingRetrospectiveCreatePage.tsx`:
- Around line 66-69: Extract a small helper (e.g., isPageAccessError) that
checks PAGE_ACCESS_ERROR_CODES.has(error.code) and use it wherever permission
errors need to be excluded: replace the inline check inside the existing
useEffect that handles useCollectedAnswers errors and add the same early-return
guard at sttMutation.onError so that both flows skip showErrorToast for
PAGE_ACCESS_ERROR_CODES; ensure the helper is imported/defined in the same
module and used by name in both useEffect and sttMutation.onError to avoid
duplicate toasts.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: c84ee131-95b5-4d82-b22e-c6715fc2a82e

📥 Commits

Reviewing files that changed from the base of the PR and between 0737b69 and f79e3eb.

📒 Files selected for processing (1)

• src/pages/Retrospectives/meeting/MeetingRetrospectiveCreatePage.tsx