Skip to content

feat: Develop - #439

Merged
yshls merged 11 commits into
masterfrom
develop
Oct 28, 2026
Merged

yshls merged 11 commits into
masterfrom
develop

Conversation

@yshls

@yshls yshls commented Oct 28, 2026

Copy link
Copy Markdown
Contributor

📌 관련 이슈

  • Close #

📝 PR 유형

어떤 변경 사항이 있나요?

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

🛠️ 변경 사항

🔍 테스트 및 검증 결과

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


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

yshls and others added 4 commits October 28, 2026 17:01
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- 이미 ?room= 있으면 현재 URL 그대로 복사 (핀 유지)
- 없으면 새 UUID 생성 → router.replace로 URL 교체 → 복사
- 버튼 여러 번 눌러도 같은 방 공유됨

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…nfo-mode-pins

Feat:collab session badge and info mode pins
@coderabbitai

coderabbitai Bot commented Oct 28, 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 59 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 04a8a1d and 4a7672f.

Files selected for processing: 16
  • changelog.md
  • src/features/map/MapPage.tsx
  • src/features/map/collaboration/CollabParticipants.tsx
  • src/features/map/collaboration/CollabPinsOverlay.tsx
  • src/features/map/collaboration/CollabSessionBadge.tsx
  • src/features/map/collaboration/MapCollabProvider.tsx
  • src/features/map/collaboration/ShareCollabButton.tsx
  • src/features/map/components/PlaceDetail.tsx
  • src/features/map/components/detail/PlaceDetail.styles.ts
  • src/features/visit-review/api/visitReviewApi.test.ts
  • src/features/visit-review/api/visitReviewApi.ts
  • src/features/visit-review/application/ports.ts
  • src/features/visit-review/application/visitReviewUseCases.test.ts
  • src/features/visit-review/application/visitReviewUseCases.ts
  • src/features/visit-review/infrastructure/visitReviewRepository.ts
  • src/private/core-ui/map-warmth/WriteWarmthModal.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.

@github-actions

Copy link
Copy Markdown

✅ PR 검증 통과

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

@github-actions

Copy link
Copy Markdown

📋 코드 리뷰 체크리스트

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

@github-actions

Copy link
Copy Markdown

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

@github-actions

Copy link
Copy Markdown

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

@github-actions

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/00000009.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000021.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
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000017.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000016.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000013.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000006.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000004.sst
11M .next/cache/turbopack/v16.4.3-29bbeab0/00000003.sst
6.1M .next/static/chunks/1x2c7bdxhwxnj.js
6.1M .next/server/chunks/ssr/src_app_dev_icons_page_tsx_0bmgqvoiobm8p._.js
4.5M .next/cache/turbopack/v16.4.3-29bbeab0/00000020.sst
4.2M .next/cache/turbopack/v16.4.3-29bbeab0/00000023.sst
3.5M .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
3.1M .next/cache/turbopack/v16.4.3-29bbeab0/00000001.sst

@github-actions

Copy link
Copy Markdown

📦 번들 크기 분석 요약:

번들 크기 분석 결과

Next.js 빌드 출력 크기

224M .next

가장 큰 파일 상위 20개

13M .next/cache/turbopack/v16.4.3-29bbeab0/00000019.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000018.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000011.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000010.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000009.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000008.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000007.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000006.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000021.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000014.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000004.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000003.sst
6.6M .next/cache/turbopack/v16.4.3-29bbeab0/00000024.sst
6.1M .next/static/chunks/1x2c7bdxhwxnj.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.2M .next/cache/turbopack/v16.4.3-29bbeab0/00000015.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

Copy link
Copy Markdown

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

@github-actions

Copy link
Copy Markdown

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

@github-actions

Copy link
Copy Markdown

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

@github-actions

Copy link
Copy Markdown

📊 테스트 커버리지 리포트

테스트 커버리지 요약

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

yshls and others added 7 commits October 28, 2026 17:29
- 백엔드 Issue #643 신규 외부 장소 등록 엔드포인트(POST /api/v1/visit-reviews) 연동
- 카카오 검색 장소 선택 시 { provider: 'KAKAO', externalId, name, lat, lng } 페이로드 전송
- 정규 카탈로그 장소(p-*)는 기존 POST /api/v1/places/{placeId}/visit-reviews 엔드포인트 유지
- 온기 작성 전 로그인 인증 상태 검사 및 401 발생 시 로그인 안내 토스트 연계
- 클린 아키텍처 포트, 유스케이스, 리포지토리 및 단위 테스트 작성
- MapCollabProvider: 공유 링크로 첫 입장 시 1회 안내 toast (sessionStorage 중복 방지)
- ShareCollabButton: 공유 toast 문구에 "친구가 이 링크로 들어오면 같은 지도를 함께 볼 수 있어요" 추가
- CollabPinsOverlay: pins === null (storage 미로드) 상태에서 addPin/removePin 호출 방지

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
MapChips는 데스크탑 전용(display:none on mobile)이라
MobileTopBar에 ShareCollabButton과 CollabSessionBadge 추가

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- ShareCollabButton: 모바일에서 텍스트 숨김(아이콘만), flex-shrink:0 추가
- CollabSessionBadge: 모바일에서 텍스트 숨김(도트+아이콘만), flex-shrink:0 추가
- CollabParticipants: 데스크탑 top 14→96px (MapChips 아래로 이동)
- MapCollabProvider: 비로그인 시 LiveblocksProvider 마운트 skip → 401 무한 재시도 방지

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- visitReviewApi.test.ts에서 실제 프로덕션 훅이 참조하는 infrastructure/visitReviewRepository 팩토리와 api/visitReviewApi 팩토리를 모두 검증하도록 describe.each로 확장
- POST /visit-reviews 엔드포인트, 외부 장소 페이로드, CSRF 토큰 및 Idempotency-Key 검증 강화
같은 카테고리의 listItems를 최대 4개 표시하고,
카드 클릭 시 지도가 해당 위치로 포커스되며 상세 패널이 전환됨.

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

fix(warmth): 카카오 검색 외부 장소 온기 등록 404 오류 수정 및 신규 API 연동
@github-actions

Copy link
Copy Markdown

✅ PR 검증 통과

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

@github-actions

Copy link
Copy Markdown

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

@github-actions

Copy link
Copy Markdown

📦 번들 크기 분석 요약:

번들 크기 분석 결과

Next.js 빌드 출력 크기

224M .next

가장 큰 파일 상위 20개

14M .next/cache/turbopack/v16.4.3-29bbeab0/00000005.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000018.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000012.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000009.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000007.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000004.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/00000011.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000010.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000006.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000003.sst
6.1M .next/static/chunks/1x2c7bdxhwxnj.js
6.1M .next/server/chunks/ssr/src_app_dev_icons_page_tsx_0bmgqvoiobm8p._.js
4.2M .next/cache/turbopack/v16.4.3-29bbeab0/00000013.sst
3.7M .next/cache/turbopack/v16.4.3-29bbeab0/00000022.sst
3.2M .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

@github-actions

Copy link
Copy Markdown

📊 테스트 커버리지 리포트

테스트 커버리지 요약

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

@github-actions

Copy link
Copy Markdown

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

@github-actions

Copy link
Copy Markdown

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

@yshls
yshls merged commit 338fb9e into master Oct 28, 2026
27 checks passed
@github-actions

Copy link
Copy Markdown

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

@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/00000003.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000019.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000013.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000011.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000009.sst
13M .next/cache/turbopack/v16.4.3-29bbeab0/00000004.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/00000008.sst
12M .next/cache/turbopack/v16.4.3-29bbeab0/00000007.sst
9.5M .next/cache/turbopack/v16.4.3-29bbeab0/00000005.sst
6.1M .next/static/chunks/1x2c7bdxhwxnj.js
6.1M .next/server/chunks/ssr/src_app_dev_icons_page_tsx_0bmgqvoiobm8p._.js
3.9M .next/cache/turbopack/v16.4.3-29bbeab0/00000022.sst
3.6M .next/cache/turbopack/v16.4.3-29bbeab0/00000018.sst
3.2M .next/cache/turbopack/v16.4.3-29bbeab0/00000023.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

✅ 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