Skip to content

Develop - #428

Merged
yshls merged 4 commits into
masterfrom
develop
Oct 26, 2026
Merged

Develop#428
yshls merged 4 commits into
masterfrom
develop

Conversation

@yshls

@yshls yshls commented Oct 26, 2026

Copy link
Copy Markdown
Contributor

📌 관련 이슈

  • Close #

📝 PR 유형

어떤 변경 사항이 있나요?

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

🛠️ 변경 사항

🔍 테스트 및 검증 결과

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

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

yshls and others added 4 commits October 27, 2026 00:54
@yshls @claude
fix(map): CollabErrorBoundary fallback에서 Liveblocks 훅 재호출 방지
8621291 
에러 경계가 fallback(=MapPage)을 렌더할 때 URL에 ?room=이 남아있으면
CollabPinsOverlay 등이 RoomProvider 없이 훅을 호출해 앱 레벨 크래시 발생.

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

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@yshls
Merge pull request #426 from YRootLab/fix/collab-error-boundary-doubl…
73ce868 
…e-crash

fix(map): CollabErrorBoundary fallback에서 Liveblocks 훅 재호출 방지
@yshls @claude
chore(liveblocks): migrate to v2 new API (LiveblocksProvider + global…
cb9d258 
… types)

- liveblocks.config.ts: 구버전 createClient + createRoomContext 제거,
  전역 interface Liveblocks 타입 선언으로 교체 (v2 권장 방식)
- MapCollabProvider: LiveblocksProvider 추가 (authEndpoint 함수 인라인),
  RoomProvider와 함께 감싸는 표준 구조로 변경
- CollabCursorsOverlay, CollabParticipants, useCollabPins:
  훅 import를 liveblocks.config에서 @liveblocks/react 직접 import로 변경

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@yshls
Merge pull request #427 from YRootLab/chore/liveblocks-v2-api-migration
e44fc68 
chore(liveblocks): v2 신규 API 마이그레이션 (LiveblocksProvider + 전역 타입)
@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 9 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 a1f38ba and e44fc68.

Files selected for processing: 6
  • src/features/map/MapPage.tsx
  • src/features/map/collaboration/CollabCursorsOverlay.tsx
  • src/features/map/collaboration/CollabParticipants.tsx
  • src/features/map/collaboration/MapCollabProvider.tsx
  • src/features/map/collaboration/liveblocks.config.ts
  • src/features/map/collaboration/useCollabPins.ts
  • 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.

@github-actions

github-actions Bot commented Oct 26, 2026

Copy link
Copy Markdown

📋 코드 리뷰 체크리스트

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

@github-actions

github-actions Bot commented Oct 26, 2026

Copy link
Copy Markdown

❌ 유효하지 않은 PR 제목

PR 제목은 다음 중 하나로 시작해야 합니다:

  • feat: 새 기능
  • fix: 버그 수정
  • refactor: 코드 리팩토링
  • chore: 유지보수
  • docs: 문서
  • test: 테스트
  • perf: 성능
  • style: 스타일
  • ci: CI/CD
  • build: 빌드

현재 제목: Develop

예시: feat: 새로운 기능 추가

@github-actions

github-actions Bot commented Oct 26, 2026

Copy link
Copy Markdown

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

@github-actions

github-actions Bot commented Oct 26, 2026

Copy link
Copy Markdown

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

@github-actions

github-actions Bot commented Oct 26, 2026

Copy link
Copy Markdown

📦 번들 크기 분석 요약:

번들 크기 분석 결과

Next.js 빌드 출력 크기

224M .next

가장 큰 파일 상위 20개

14M .next/cache/turbopack/v16.4.3-29bbeab0/00000009.sst
14M .next/cache/turbopack/v16.4.3-29bbeab0/00000008.sst
14M .next/cache/turbopack/v16.4.3-29bbeab0/00000004.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000020.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000019.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000010.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000007.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000018.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000016.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000011.sst
11M .next/cache/turbopack/v16.4.3-29bbeab0/00000006.sst
9.5M .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.3M .next/cache/turbopack/v16.4.3-29bbeab0/00000024.sst
4.1M .next/cache/turbopack/v16.4.3-29bbeab0/00000017.sst
3.4M .next/cache/turbopack/v16.4.3-29bbeab0/00000012.sst
3.1M .next/cache/turbopack/v16.4.3-29bbeab0/00000005.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

github-actions Bot commented Oct 26, 2026

Copy link
Copy Markdown

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

@github-actions

github-actions Bot commented Oct 26, 2026

Copy link
Copy Markdown

📦 번들 크기 분석 요약:

번들 크기 분석 결과

Next.js 빌드 출력 크기

224M .next

가장 큰 파일 상위 20개

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/00000020.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/00000009.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000019.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000014.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000008.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000006.sst
11M .next/cache/turbopack/v16.4.3-29bbeab0/00000005.sst
8.5M .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
5.4M .next/cache/turbopack/v16.4.3-29bbeab0/00000024.sst
5.2M .next/cache/turbopack/v16.4.3-29bbeab0/00000015.sst
4.9M .next/cache/turbopack/v16.4.3-29bbeab0/00000023.sst
3.9M .next/cache/turbopack/v16.4.3-29bbeab0/00000018.sst
3.1M .next/cache/turbopack/v16.4.3-29bbeab0/00000007.sst
3.1M .next/cache/turbopack/v16.4.3-29bbeab0/00000002.sst

@yshls
yshls merged commit 74df7fc into master Oct 26, 2026
30 of 31 checks passed
@github-actions

github-actions Bot commented Oct 26, 2026

Copy link
Copy Markdown

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

@github-actions

github-actions Bot commented Oct 26, 2026

Copy link
Copy Markdown

📊 테스트 커버리지 리포트

테스트 커버리지 요약

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

@github-actions

github-actions Bot commented Oct 26, 2026

Copy link
Copy Markdown

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

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@yshls