Skip to content

chore: 모바일 앱 분리#610

Open
ej070961 wants to merge 2 commits into
developfrom
chore/app
Open

chore: 모바일 앱 분리#610
ej070961 wants to merge 2 commits into
developfrom
chore/app

Conversation

@ej070961

@ej070961 ej070961 commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

릴리스 노트

  • Chores
    • Mobile 애플리케이션 및 관련 개발 환경 설정이 제거되었습니다.
    • Mobile 개발 스크립트(dev, lint, build 등)가 더 이상 사용할 수 없습니다.
    • Mobile 앱 관련 문서와 구성 파일이 제거되었습니다.

@ej070961 ej070961 self-assigned this Jun 10, 2026
@vercel

vercel Bot commented Jun 10, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
surf-admin Ready Ready Preview, Comment Jun 10, 2026 12:05pm
surf-web Ready Ready Preview, Comment Jun 10, 2026 12:05pm

@coderabbitai

coderabbitai Bot commented Jun 10, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

@ej070961, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 43 minutes and 28 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 066ca44e-c9a9-4c9e-97e7-f7670637424e

📥 Commits

Reviewing files that changed from the base of the PR and between 49d32a4 and 9dbb9a3.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (3)
  • apps/mobile/app.json
  • apps/mobile/package.json
  • apps/mobile/src/app/index.tsx
📝 Walkthrough

Walkthrough

이 PR은 모노레포에서 모바일 앱(앱/모바일)을 완전히 제거합니다. 모바일 애플리케이션 관련 모든 추적을 삭제하며, 루트 README와 문서에서 모바일 항목을 제거하고, package.json에서 모바일 개발/빌드 스크립트(dev:mobile, lint:mobile, android:mobile, ios:mobile, web:mobile)와 lint-staged 설정을 정리합니다. 또한 VS Code 확장 추천에서 expo.vscode-expo-tools를 제거합니다.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10분


주요 검토 포인트

1. 문서 일관성 확인

  • README.md: 모바일 항목과 "## Mobile" 섹션 제거로 남은 섹션들의 구조가 명확한가요?
  • docs/overview.md: 모노레포 다이어그램에서 mobile/ 제거 후 구조가 정확하게 표현되었는가요?

2. 빌드 환경 검증

  • package.json scripts 검토:

    • 제거된 모바일 스크립트(dev:mobile, lint:mobile, check-types:mobile, android:mobile, ios:mobile, web:mobile) 이후 남은 스크립트들이 정상 작동하는가요?
    • 예: build:admin, lint:admin, lint:fix:admin 호출 시 예상 동작이 보장되는가요?
  • lint-staged 설정:

    • apps/mobile/**/*.{js,jsx,ts,tsx} 제거로 인해 다른 앱(admin)의 파일 린팅에 영향이 없는가요?
    • 글로벌 prettier 규칙(*.{json,md,css,scss,html})이 정상 적용되는가요?

3. CI/CD 영향도

  • 제거된 모바일 빌드 스크립트에 의존하는 CI/CD 파이프라인이 있다면, 해당 설정도 함께 업데이트했는가요? (이 PR에서 보이지 않으면 별도 확인 필요)
  • 팀 문서나 온보딩 가이드에서 모바일 개발 절차를 제거했는가요?

4. 작은 것 같지만 중요한 확인

  • 타사 서비스(예: 배포 자동화, 모니터링 대시보드)에서 모바일 앱 빌드를 추적하고 있었다면, 해당 워크플로우도 정리되었는가요?
  • apps/mobile 내 남은 node_modules, .next, dist 같은 빌드 산출물이 없는지 확인 (기존 .gitignore 제거 후)

Clean Architecture 관점: 이는 제거 작업이므로, 남은 시스템(admin 앱)이 모바일 앱 존재를 가정하지 않도록 의존성이 정리되었는지 확인하세요. 특히 공유 유틸, 타입, 설정이 모바일 관련 임포트를 남겨두지 않았는지 검증이 권장됩니다.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목은 모바일 앱 분리라는 실질적인 변화를 명확히 요약하고 있으며, 삭제된 apps/mobile 디렉토리와 관련 설정들의 제거라는 주요 변경사항을 정확히 반영합니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/app

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.

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