Skip to content

Conversation

@manNomi
Copy link
Contributor

@manNomi manNomi commented Jan 25, 2026

관련 이슈

작업 내용

멘토 페이지 레이지로드 적용 (2fdbb7e)
멘토/멘티 페이지 분리 (a97e439)
rounded-lg 스타일 수정 (798d026)

특이 사항

리뷰 요구사항 (선택)

@vercel
Copy link

vercel bot commented Jan 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
solid-connection-web Ready Ready Preview, Comment Jan 25, 2026 3:57pm

@coderabbitai
Copy link

coderabbitai bot commented Jan 25, 2026

Walkthrough

1. MenteePage 컴포넌트 추가: MenteePageTabsMentorFindSection을 합성하는 클라이언트 컴포넌트가 apps/web/src/app/mentor/_ui/MentorClient/_ui/MenteePage/index.tsx에 새로 생성되었습니다.
2. MentorClient 로직 변경: Mentor/Mentee 페이지를 dynamic import로 lazy 로드하고 Suspense로 감싼 렌더링 구조로 전환했습니다.
3. 토큰 재발급 흐름 추가: 초기화 시 액세스 토큰 재발급을 시도하는 클라이언트 사이드 로직(refreshStatus, postReissueToken 사용)이 apps/web/src/app/mentor/_ui/MentorClient/index.tsx에 도입되었습니다.
4. 인증 상태 확장: RefreshStatus 타입과 refreshStatus 상태 및 세터가 apps/web/src/lib/zustand/useAuthStore.ts에 추가되어 상태 관리가 확장되었습니다.
5. 빌드/스타일 설정 변경: Tailwind의 borderRadius 값이 CSS 변수 기반에서 고정 rem 값으로 변경되었고(apps/web/tailwind.config.ts), Biome 스키마 버전과 일부 linter 규칙이 조정되었습니다(apps/web/biome.json).

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Suggested reviewers

  • wibaek
  • khwww
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목은 '멘토 페이지 버그 개선 및 border 버그 수정'으로 변경 내용의 주요 포인트(멘토 페이지 개선 및 border 스타일 수정)를 잘 반영하고 있습니다.
Description check ✅ Passed PR 설명은 필수 섹션(관련 이슈, 작업 내용)을 포함하고 있으며, 커밋별 작업 내용이 명확하게 기재되어 있습니다.
Linked Issues check ✅ Passed PR의 변경사항들이 #367 이슈의 border-radius 버그 수정 요구사항을 충족합니다. tailwind.config.ts에서 border-radius 값을 CSS 변수에서 고정된 rem 값으로 변경하여 border 스타일이 올바르게 적용되도록 수정했습니다.
Out of Scope Changes check ✅ Passed 모든 변경사항이 #367 이슈와 PR 제목에 명시된 범위 내에 있습니다. 멘토 페이지 개선(레이지로드, 페이지 분리), border 스타일 수정, 설정 파일 업데이트 모두 PR 목표와 일치합니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
  • 📝 Generate docstrings

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 and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@apps/web/src/app/mentor/_ui/MentorClient/index.tsx`:
- Around line 23-41: The token-refresh effect currently re-runs even when
refreshStatus is "failed", causing repeated redirect loops; update the guard in
the attemptTokenRefresh logic (inside the useEffect) to abort if refreshStatus
=== "failed" (or add an explicit check like if (refreshStatus === "failed")
return), ensuring you only proceed when refreshStatus is not "failed" and
isInitialized is true and no other refresh is in progress; keep using the
existing symbols attemptTokenRefresh, refreshStatus, setRefreshStatus,
isRefreshing, setIsRefreshing, postReissueToken and router.push so the flow sets
refreshStatus to "failed" once and prevents further automatic retries until an
explicit retry is triggered.

- refreshStatus가 'failed'일 때 재시도하지 않도록 가드 추가
- 토큰 재발급 실패 후 반복적인 리다이렉트 방지
@manNomi manNomi merged commit 90dedc3 into main Jan 25, 2026
5 checks passed
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.

border radius 적용 안되는 버그

2 participants