Skip to content

docs: CLAUDE.md를 현재 코드 구조에 맞춘다 - #37

Open
seongwon030 wants to merge 1 commit into
mainfrom
docs/sync-claude-md
Open

seongwon030 wants to merge 1 commit into
mainfrom
docs/sync-claude-md

Conversation

@seongwon030

@seongwon030 seongwon030 commented Sep 20, 2026 •

Copy link
Copy Markdown
Member

왜

홈이 웹뷰로 전환되면서(feature/webview-shell-migration) 라우팅이 바뀌었는데 CLAUDE.md가 따라가지 않았다. 문서를 믿고 app/(tabs)/more.tsx를 찾다가 파일이 없어서 발견했다.

코드로 확인한 불일치만 고쳤다. 추측으로 채운 문장은 없다.

무엇

문서에 적혀 있던 것 실제
(tabs)/index.tsx, (tabs)/more.tsx 하단 탭 네비게이터 (tabs)/ 자체가 없음. 홈은 app/index.tsx, HomeWebViewScreen 우선 + 네이티브 폴백
clubDetail/[id].tsx = 동아리 상세 (네이티브) club/[id].tsx와 동일한 ClubDetailScreen 재export. 둘 다 WebView. FCM 딥링크가 이 경로로 들어옴
부트스트랩 3→4→5→6 순차 토큰 발급만 순차. 구독 동기화·Mixpanel identify·FCM 등록은 Promise.all 병렬이라 순서가 없음
ATT 권한 요청이 2번째 부트스트랩 성공 후, 스플래시가 내려간 뒤. 사실상 마지막
<Text type="..."> components/moa-text.tsx가 export하는 이름은 MoaText
프로바이더 2개 HomeWebViewPreloadProvider 누락. 스플래시 종료 시점을 결정함
환경변수 EXPO_PUBLIC_BASE_URL만 EXPO_PUBLIC_WEBVIEW_URL, EXPO_PUBLIC_MIXPANEL_TOKEN 추가
ui/ 하위 폴더 일반론 현재 ui/home·ui/club-detail 둘. hook/·model/은 ui/home에만 존재

개요 문단도 고쳤다. 기존 문장은 "WebView로 동아리 상세 페이지를 볼 수 있다"고 읽혀서 상세만 웹뷰인 것처럼 보였는데, 홈까지 웹뷰이고 네이티브는 부트스트랩·푸시·구독·딥링크를 담당하는 셸이라는 게 이 레포를 읽을 때 가장 먼저 필요한 정보다.

확인 방법

문서 전용 변경이라 코드 영향은 없다. 각 항목은 아래로 확인된다.

find app -name "*.tsx"                      # (tabs)/ 없음
cat app/clubDetail/\[id\].tsx               # ClubDetailScreen 재export 3줄
sed -n '41,64p' services/app-bootstrap.service.ts   # Promise.all
grep -n "^export" components/moa-text.tsx   # MoaText
cat contexts/index.ts                       # 프로바이더 3개
find ui -type f                             # home, club-detail

Summary by CodeRabbit

  • 문서
    • 프로젝트 아키텍처 안내를 업데이트했습니다. 홈 및 동아리 상세 화면의 WebView 기반 구조와 네이티브 폴백 동작을 확인할 수 있습니다.
    • 실험 기능으로 typed routes를 추가하고, 주요 환경 변수 설정 방법을 정리했습니다.
    • 라우팅 구조, 앱 초기화 순서, Mixpanel·푸시 알림 초기화 방식에 대한 설명을 최신화했습니다.
    • 상태 관리 제공자와 UI 폴더 구조, 텍스트 스타일 지정자 안내를 현재 구현에 맞게 수정했습니다.

홈이 웹뷰로 전환되면서(feature/webview-shell-migration) 라우팅이 바뀌었는데
문서가 따라가지 않았다. 코드로 확인한 불일치만 고친다.

- (tabs)/ 하단 탭 네비게이터는 존재하지 않는다. 홈은 app/index.tsx이고
  HomeWebViewScreen 우선, 로드 실패 시 네이티브 home-screen 폴백이다.
- clubDetail/[id].tsx는 "네이티브 상세"가 아니라 club/[id].tsx와 동일한
  ClubDetailScreen 재export다. FCM 딥링크가 이 경로로 들어온다.
- 부트스트랩은 토큰 발급만 순차고, 구독 동기화·Mixpanel identify·FCM 등록은
  Promise.all 병렬이라 순서가 없다. ATT는 2번째가 아니라 부트스트랩 성공 후
  스플래시가 내려간 뒤에 요청한다.
- moa-text가 export하는 이름은 Text가 아니라 MoaText다.
- HomeWebViewPreloadProvider가 빠져 있었다. 스플래시 종료 시점을 결정한다.
- 환경 변수에 EXPO_PUBLIC_WEBVIEW_URL, EXPO_PUBLIC_MIXPANEL_TOKEN 추가.
- ui/ 하위는 현재 home과 club-detail 둘이고 hook/·model/은 home에만 있다.

개요 문단도 고쳤다. 동아리 상세만 웹뷰인 것처럼 읽혔는데, 홈까지 웹뷰이고
네이티브가 셸이라는 게 이 레포를 읽을 때 가장 먼저 알아야 할 구조다.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 20, 2026 •

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: b00f2320-defe-49f3-9a3b-ac132c7dd1a1

📥 Commits

Reviewing files that changed from the base of the PR and between 16ae7da and 9e767a4.

📒 Files selected for processing (1)
  • CLAUDE.md

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


Walkthrough

CLAUDE.md가 WebView 기반 구조, typed routes, 환경 변수, 라우팅, 부트스트랩, 상태 관리, UI 패턴, MoaText 사용법을 현재 구현에 맞게 갱신했습니다.

Changes

프로젝트 문서 갱신

Layer / File(s) Summary
아키텍처 및 개발 지침 갱신
CLAUDE.md
WebView 셸 구조와 네이티브 폴백을 문서화했습니다. typed routes와 환경 변수를 추가했습니다. 라우팅 및 부트스트랩 동작을 갱신했습니다. MixpanelProvider, HomeWebViewPreloadProvider, UI 폴더와 훅 예시를 수정했습니다. MoaText 사용 예시를 반영했습니다.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 9e767

The documentation update has no verified merge-blocking issue and is ready to merge after normal checks.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 제목은 CLAUDE.md 문서를 현재 코드 구조에 맞게 업데이트하는 주요 변경 사항을 정확하고 간결하게 설명합니다.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

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.

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