Skip to content

feat: 라우트 코드 스플리팅 적용 (#387) - #388

Merged
kcleverp merged 2 commits into
devfrom
fix/kcleverp-route-code-splitting
Aug 12, 2026
Merged

feat: 라우트 코드 스플리팅 적용 (#387)#388
kcleverp merged 2 commits into
devfrom
fix/kcleverp-route-code-splitting

Conversation

@kcleverp

Copy link
Copy Markdown
Collaborator

이슈

Closes #387

변경 사항

  • App, fullscreen, 워크스페이스 라우트의 페이지 import를 lazy loading으로 전환했습니다.
  • 공통 Suspense 로딩 UI와 청크 로딩 실패 시 새로고침 가능한 오류 경계를 추가했습니다.
  • 게스트 공유 링크(/share/:token)를 공개 경로로 등록했습니다.

스크린샷

리뷰 포인트

  • 기존 직접 URL과 인증 가드 흐름을 유지하면서 초기 번들에 페이지 코드를 포함하지 않는지 확인 부탁드립니다.

체크리스트

  • 셀프 리뷰 완료
  • 동작 확인

@vercel

vercel Bot commented Aug 12, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
slate-to-fe Ready Ready Preview Aug 12, 2026 7:45am

@kcleverp

Copy link
Copy Markdown
Collaborator Author

코드 리뷰

Must: RouteLoadingBoundary가 MainLayout 전체의 Routes를 감싸지만 경로 변경 시 hasError를 초기화하지 않습니다. 한 페이지의 lazy import가 일시적으로 실패하면 이후 다른 메뉴로 이동해도 boundary가 계속 오류 화면만 렌더링해 앱을 복구할 수 없습니다. key={pathname}으로 경계를 재마운트하거나 location 변경 시 상태를 초기화해 주세요.

Should: 없음

@kcleverp

Copy link
Copy Markdown
Collaborator Author

리뷰 반영 완료

라우트별 오류 경계에 key={pathname}을 적용했습니다. 특정 lazy 청크 로딩이 실패해도 다른 경로로 이동하면 오류 상태가 초기화되어 화면을 다시 복구할 수 있습니다.

검증: diff --check 통과. 로컬 worktree의 의존성 연결 문제로 lint·타입·빌드는 실행하지 못했으며, PR CI에서 재검증됩니다.

@kcleverp kcleverp changed the title perf: 라우트 코드 스플리팅 적용 (#387) feat: 라우트 코드 스플리팅 적용 (#387) Aug 12, 2026
@kcleverp
kcleverp merged commit 1edeca3 into dev Aug 12, 2026
3 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.

1 participant