Fix: login error 및 사이드바 조건 추가

[jjangminii]

📌 Related Issues

관련된 Issue를 태그해주세요. (e.g. - close #25)

• close [Fix] 무한로딩 에러 #234

📄 Tasks

⭐ PR Point (To Reviewer)

📷 Screenshot

Summary by CodeRabbit

릴리스 노트

• 버그 수정
  • 온보딩 페이지의 라우팅 경로를 표준화했습니다.
  • 특정 페이지에서 사이드바 표시 동작을 조정했습니다.
  • 인증 실패 시 리다이렉트 경로를 수정했습니다.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[vercel]

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

Project	Deployment	Review	Updated (UTC)
pinback-client-client	Ready	Preview, Comment	Jan 7, 2026 4:24pm
pinback-client-landing	Ready	Preview, Comment	Jan 7, 2026 4:24pm

[coderabbitai]

워크스루

온보딩 라우트의 경로 표기를 /onboarding에서 /onBoarding으로 일관되게 변경하고, Layout 컴포넌트에서 onBoardingCallback 경로를 사이드바 숨김 조건에 추가했습니다. 제어 흐름 변경 없이 경로 표준화만 수행됩니다.

변경 사항

코호트 / 파일	변경 요약
라우팅 설정 apps/client/src/routes/routesConfig.ts	onBoardingCallback 라우트 경로를 /onboarding/callback에서 /onBoarding/callback으로 변경
온보딩 페이지 apps/client/src/pages/onBoarding/GoogleCallback.tsx, apps/client/src/pages/onBoarding/components/funnel/MainCard.tsx	세 곳의 네비게이션 경로를 /onboarding?step=...에서 /onBoarding?step=...으로 일관되게 수정
레이아웃 및 인증 apps/client/src/layout/Layout.tsx, apps/client/src/shared/apis/setting/axiosInstance.ts	사이드바 숨김 조건에 onBoardingCallback 추가, 401/403 에러 시 리다이렉트 경로를 /onBoarding?step=SOCIAL_LOGIN으로 업데이트

예상 코드 리뷰 소요 시간

🎯 1 (Trivial) | ⏱️ ~5분

관련 가능성이 있는 PR

• PR #235: 동일한 온보딩 라우트 문자열과 네비게이션 핸들러(GoogleCallback, MainCard, routesConfig, axiosInstance)를 수정하므로 관련됨
• PR #233: Layout.tsx의 사이드바 가시성 로직을 변경하므로 관련됨
• PR #204: Google OAuth 콜백 라우팅과 GoogleCallback 사용처를 변경하므로 관련됨

제안 라벨

fix

제안 리뷰어

• jllee000
• constantly-dev

래빗의 축시

🐰 경로를 고르게 다듬으니, 반짝반짝 반사되는 온보딩!
/onBoarding으로 통일되어 우아하게 흘러
사이드바는 쏙 숨어 매끄러운 흐름을 그리네 ✨

🚥 Pre-merge checks | ✅ 1 | ❌ 4

❌ Failed checks (3 warnings, 1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	PR 설명은 템플릿 구조를 유지하지만, '관련 Issue', '작업 내용', '리뷰어 안내', '스크린샷' 섹션이 모두 비어 있거나 미작성 상태입니다.	Related Issues 섹션에 #234를 명시하고, Tasks 섹션에 작업 내용(경로 대소문자 변경, 사이드바 조건 추가)을 작성해주세요.
Linked Issues check	⚠️ Warning	PR이 연결된 Issue #234와 관계없이 Issue #25(Progress Bar 구현)와만 연결되어 있으며, 실제 변경사항은 경로 정정과 사이드바 조건 추가로 #25 요구사항과 불일치합니다.	PR이 실제로 해결하는 Issue #234를 명확히 링크하고, Issue #25 요구사항(Progress 컴포넌트 구현)과의 관련성을 명확히 해주세요.
Out of Scope Changes check	⚠️ Warning	PR의 모든 변경사항(경로 대소문자 정정, onBoardingCallback 추가)은 로그인 오류 및 사이드바 조건 수정과 관련 있으나, Issue #25(Progress Bar 구현)와는 전혀 무관한 변경입니다.	현재 PR의 목표와 Issue #25 요구사항이 맞지 않습니다. 올바른 Issue #234와 연결하거나 PR 범위를 명확히 해주세요.
Title check	❓ Inconclusive	PR 제목 'Fix/#234/login error'는 변경 사항의 핵심 내용인 경로(path) 대소문자 수정 및 사이드바 숨김 조건 업데이트와 직접적으로 관련이 없으며, 구체적인 기술적 세부사항을 전달하지 못하는 모호한 표현입니다.	제목을 'Fix onBoarding route casing and sidebar visibility' 같이 변경하거나, 주요 변경 사항(경로 대소문자 정정, 사이드바 조건 추가)을 명확히 반영하는 제목으로 수정해주세요.

✅ Passed checks (1 passed)

Check name	Status	Explanation
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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[github-actions]

✅ Storybook chromatic 배포 확인:
🐿️ storybook

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

apps/client/src/layout/Layout.tsx (1)

12-16: 중복 조건 제거 필요

Line 15의 onBoardingCallback 경로 체크는 불필요합니다. Line 13에서 이미 ROUTES_CONFIG.onBoarding.path ('/onBoarding')로 시작하는지 확인하고 있기 때문에, /onBoarding/callback 경로도 자동으로 포함됩니다.

♻️ 제안하는 수정사항

  const isSidebarHidden =
    location.pathname.startsWith(ROUTES_CONFIG.onBoarding.path) ||
    location.pathname.startsWith(ROUTES_CONFIG.login.path) ||
-    location.pathname.startsWith(ROUTES_CONFIG.onBoardingCallback.path) ||
    isPolicyPage;

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c689234 and e965e5d.

📒 Files selected for processing (5)

• apps/client/src/layout/Layout.tsx
• apps/client/src/pages/onBoarding/GoogleCallback.tsx
• apps/client/src/pages/onBoarding/components/funnel/MainCard.tsx
• apps/client/src/routes/routesConfig.ts
• apps/client/src/shared/apis/setting/axiosInstance.ts

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: storybook

🔇 Additional comments (4)

apps/client/src/routes/routesConfig.ts (1)

24-24: 경로 케이싱 변경 확인 완료

라우트 경로를 /onBoarding/callback로 통일한 것은 적절합니다. 다른 파일들과 일관성을 유지합니다.

apps/client/src/pages/onBoarding/GoogleCallback.tsx (1)

15-15: 경로 업데이트 확인 완료

세 곳의 네비게이션 경로가 모두 /onBoarding으로 일관되게 업데이트되었습니다. 로그인 실패, 신규 사용자, 에러 처리 로직이 올바르게 유지되고 있습니다.

Also applies to: 39-39, 54-54

apps/client/src/pages/onBoarding/components/funnel/MainCard.tsx (1)

167-167: 온보딩 퍼널 경로 업데이트 확인 완료

다음 단계, 이전 단계, Mac 단계 스킵 로직의 모든 네비게이션 경로가 /onBoarding으로 일관되게 업데이트되었습니다. 퍼널 플로우가 올바르게 유지되고 있습니다.

Also applies to: 187-187, 196-196

apps/client/src/shared/apis/setting/axiosInstance.ts (1)

50-50: 인증 실패 리다이렉트 경로 업데이트 확인 완료

401/403 에러 처리 시 리다이렉트 경로가 /onBoarding으로 올바르게 업데이트되었습니다. 인증 인터셉터 로직이 적절하게 유지되고 있습니다.

[constantly-dev]

굿입니다~ 👍

[jllee000]

확인했습니다~!