Skip to content

fix(map): CollabErrorBoundary fallback에서 Liveblocks 훅 재호출 방지 - #426

Merged
yshls merged 1 commit into
developfrom
fix/collab-error-boundary-double-crash
Oct 26, 2026
Merged

yshls merged 1 commit into
developfrom
fix/collab-error-boundary-double-crash

Conversation

@yshls

@yshls yshls commented Oct 26, 2026

Copy link
Copy Markdown
Contributor

에러 경계가 fallback(=MapPage)을 렌더할 때 URL에 ?room=이 남아있으면 CollabPinsOverlay 등이 RoomProvider 없이 훅을 호출해 앱 레벨 크래시 발생.

CollabActiveContext를 추가해 MapPage가 실제 RoomProvider 안에 있을 때만 collabRoom을 truthy로 처리하도록 수정.

📌 관련 이슈

  • Close #

📝 PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가 (Feature)
  • 버그 수정 (Fix)
  • 코드 리팩토링 (Refactor)
  • 문서 수정 (Docs)
  • 테스트 추가/수정 (Test)
  • 빌드/설정/의존성/기타 변경 (Chore)

🛠️ 변경 사항

🔍 테스트 및 검증 결과

💬 리뷰어에게 전달할 사항 (선택)


🐰 아래부터는 CodeRabbit 봇이 자동으로 PR 요약을 작성하는 공간입니다.

에러 경계가 fallback(=MapPage)을 렌더할 때 URL에 ?room=이 남아있으면
CollabPinsOverlay 등이 RoomProvider 없이 훅을 호출해 앱 레벨 크래시 발생.

CollabActiveContext를 추가해 MapPage가 실제 RoomProvider 안에 있을 때만
collabRoom을 truthy로 처리하도록 수정.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 26, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 14 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

Run configuration
Commits

Reviewing files that changed from the base of the PR and between 33099c2 and 8621291.

Files selected for processing: 2
  • src/features/map/MapPage.tsx
  • src/features/map/collaboration/MapCollabProvider.tsx
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

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

@yshls
yshls merged commit 73ce868 into develop Oct 26, 2026
29 of 30 checks passed
@github-actions

Copy link
Copy Markdown

📦 번들 크기 분석 요약:

번들 크기 분석 결과

Next.js 빌드 출력 크기

224M .next

가장 큰 파일 상위 20개

14M .next/cache/turbopack/v16.4.3-29bbeab0/00000009.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000016.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000010.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000008.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000005.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000003.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000020.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000014.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000013.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000011.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000007.sst
8.8M .next/cache/turbopack/v16.4.3-29bbeab0/00000004.sst
6.3M .next/cache/turbopack/v16.4.3-29bbeab0/00000024.sst
6.1M .next/static/chunks/3b7eb-xkdokyo.js
6.1M .next/server/chunks/ssr/src_app_dev_icons_page_tsx_0bmgqvoiobm8p._.js
5.0M .next/cache/turbopack/v16.4.3-29bbeab0/00000012.sst
3.6M .next/cache/turbopack/v16.4.3-29bbeab0/00000015.sst
3.1M .next/cache/turbopack/v16.4.3-29bbeab0/00000006.sst
3.1M .next/cache/turbopack/v16.4.3-29bbeab0/00000002.sst
3.1M .next/cache/turbopack/v16.4.3-29bbeab0/00000001.sst

@github-actions

Copy link
Copy Markdown

📋 코드 리뷰 체크리스트

  • 테스트 추가/업데이트
  • 문서 업데이트
  • 콘솔 로그/디버거 없음
  • 하드코딩된 값 없음
  • 성능 고려됨
  • 보안 검토됨
  • 에러 처리 추가됨

@github-actions

Copy link
Copy Markdown

✅ PR 검증 통과

  • 제목 형식: ✓
  • 설명: ✓
  • 파일 수정: ✓

@github-actions

Copy link
Copy Markdown

📦 번들 크기 분석 요약:

번들 크기 분석 결과

Next.js 빌드 출력 크기

224M .next

가장 큰 파일 상위 20개

16M .next/cache/turbopack/v16.4.3-29bbeab0/00000006.sst
15M .next/cache/turbopack/v16.4.3-29bbeab0/00000005.sst
14M .next/cache/turbopack/v16.4.3-29bbeab0/00000011.sst
14M .next/cache/turbopack/v16.4.3-29bbeab0/00000004.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000014.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000010.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000020.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000016.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000012.sst
11M .next/cache/turbopack/v16.4.3-29bbeab0/00000009.sst
11M .next/cache/turbopack/v16.4.3-29bbeab0/00000007.sst
11M .next/cache/turbopack/v16.4.3-29bbeab0/00000003.sst
6.1M .next/static/chunks/3b7eb-xkdokyo.js
6.1M .next/server/chunks/ssr/src_app_dev_icons_page_tsx_0bmgqvoiobm8p._.js
4.0M .next/cache/turbopack/v16.4.3-29bbeab0/00000019.sst
3.5M .next/cache/turbopack/v16.4.3-29bbeab0/00000024.sst
3.1M .next/cache/turbopack/v16.4.3-29bbeab0/00000008.sst
3.1M .next/cache/turbopack/v16.4.3-29bbeab0/00000002.sst
3.1M .next/cache/turbopack/v16.4.3-29bbeab0/00000001.sst
2.8M .next/cache/turbopack/v16.4.3-29bbeab0/00000022.sst

@github-actions

Copy link
Copy Markdown

⚡ Lighthouse 성능 감사가 완료되었습니다. 아티팩트에서 상세 리포트를 확인하세요.

@github-actions

Copy link
Copy Markdown

♿ 접근성 감사가 완료되었습니다. 아티팩트의 a11y-report를 확인하세요.

@github-actions

Copy link
Copy Markdown

✅ 코드 품질 검사가 완료되었습니다. 위 출력 결과에서 이슈를 확인하세요.

@github-actions

Copy link
Copy Markdown

🔍 CodeQL 보안 분석이 완료되었습니다. 세부 검출 결과는 리포지토리의 Security 탭을 확인하세요.

@github-actions

Copy link
Copy Markdown

📊 테스트 커버리지 리포트

테스트 커버리지 요약

사용 가능한 커버리지 데이터가 없습니다.

@github-actions

Copy link
Copy Markdown

✅ E2E 테스트가 완료되었습니다. 아티팩트에서 Playwright 리포트를 확인하세요.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant