Skip to content

Develop - #431

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

yshls merged 5 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 2 commits October 27, 2026 01:15
Kakao Maps에 getContainer() 메서드가 없어 TypeError 발생.
role=application aria-label으로 맵 컨테이너 요소를 직접 조회.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
fix(map): CollabCursorsOverlay getContainer TypeError 수정
@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 56 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 74df7fc and e07a6ae.

Files selected for processing: 7
  • .agents/AGENTS.md
  • AGENTS.md
  • next.config.ts
  • src/features/map/MapPage.tsx
  • src/features/map/collaboration/CollabCursorsOverlay.tsx
  • src/features/map/collaboration/CollabPinsOverlay.tsx
  • src/features/map/collaboration/CollabSessionBadge.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.

- CollabSessionBadge: room URL 접속 시 '공유된 지도를 보고 있어요' 배지 표시
- CollabPinsOverlay: mode !== 'info' 이면 핀 공유 버튼 미렌더 (정보 모드 전용)

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

Copy link
Copy Markdown

❌ 유효하지 않은 PR 제목

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

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

현재 제목: Develop

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

@github-actions

Copy link
Copy Markdown

📊 테스트 커버리지 리포트

테스트 커버리지 요약

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

@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 리포트를 확인하세요.

yshls and others added 2 commits October 27, 2026 02:30
- CollabCursorsOverlay: 함수 호이스팅으로 인한 container 널 체크 미전파(TS18047) 수정 (화살표 함수로 전환)

- CollabPinsOverlay: mode !== 'info' 시 오버레이 정리 로직 추가 및 의존성 배열에 mode 반영

- CollabPinsOverlay: 핀 등록 시 실제 회원 ID(user.id) 대신 익명 세션 ID(me.presence.userId)를 사용하여 사용자 정보 유출 방지

- next.config.ts: 프로덕션 빌드 시 브라우저 소스맵 생성 및 콘솔 출력 차단 설정 추가

- AGENTS.md: 보안 절대 원칙 및 AI 행동 지침 추가 동기화
…nfo-mode-pins

feat(map): 공유 지도 상태 배지 + 정보 모드 핀 제한
@github-actions

Copy link
Copy Markdown

❌ 유효하지 않은 PR 제목

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

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

현재 제목: Develop

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

@github-actions

Copy link
Copy Markdown

📦 번들 크기 분석 요약:

번들 크기 분석 결과

Next.js 빌드 출력 크기

224M .next

가장 큰 파일 상위 20개

14M .next/cache/turbopack/v16.4.3-29bbeab0/00000010.sst
14M .next/cache/turbopack/v16.4.3-29bbeab0/00000009.sst
14M .next/cache/turbopack/v16.4.3-29bbeab0/00000004.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000021.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000018.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/00000008.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000007.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000005.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000003.sst
6.1M .next/static/chunks/3qed19_qh8ttl.js
6.1M .next/server/chunks/ssr/src_app_dev_icons_page_tsx_0bmgqvoiobm8p._.js
4.2M .next/cache/turbopack/v16.4.3-29bbeab0/00000024.sst
4.0M .next/cache/turbopack/v16.4.3-29bbeab0/00000020.sst
3.9M .next/cache/turbopack/v16.4.3-29bbeab0/00000022.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

📦 번들 크기 분석 요약:

번들 크기 분석 결과

Next.js 빌드 출력 크기

224M .next

가장 큰 파일 상위 20개

14M .next/cache/turbopack/v16.4.3-29bbeab0/00000010.sst
14M .next/cache/turbopack/v16.4.3-29bbeab0/00000004.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000016.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000011.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000008.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000005.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000020.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000019.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000017.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000009.sst
11M .next/cache/turbopack/v16.4.3-29bbeab0/00000003.sst
9.3M .next/cache/turbopack/v16.4.3-29bbeab0/00000006.sst
6.1M .next/static/chunks/3qed19_qh8ttl.js
6.1M .next/server/chunks/ssr/src_app_dev_icons_page_tsx_0bmgqvoiobm8p._.js
4.2M .next/cache/turbopack/v16.4.3-29bbeab0/00000023.sst
4.0M .next/cache/turbopack/v16.4.3-29bbeab0/00000024.sst
3.9M .next/cache/turbopack/v16.4.3-29bbeab0/00000012.sst
3.7M .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 6c58e9c into master Oct 26, 2026
27 of 28 checks passed
@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

✅ 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