Skip to content

Conversation

@manNomi
Copy link
Contributor

@manNomi manNomi commented Jan 25, 2026

관련 이슈

  • resolves: #이슈 번호

작업 내용

특이 사항

리뷰 요구사항 (선택)

- Vercel Integration handles all automatic deployments
- Only keep release.yml for manual production releases
- Add .env.development for local dev environment (localhost, localstorage login)
- Add .env.preview for Vercel preview deployments (stage API)
- Keep .env.production for production deployments
- Update .env.guide.md with correct environment variable setup
@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 11:11am

@coderabbitai
Copy link

coderabbitai bot commented Jan 25, 2026

Walkthrough

환경설정 구조를 재구성하는 변경사항입니다.

  1. 공통 기본값 파일 정리: .env 파일에서 환경별 변수들(NEXT_PUBLIC_WEB_URL, NEXT_PUBLIC_API_SERVER_URL, NEXT_PUBLIC_KAKAO_JS_KEY)을 제거하고, 용도를 공통 설정으로 명확히 하는 주석 업데이트를 수행했습니다.

  2. 환경별 설정 파일 신규 추가: 개발 환경용 .env.development와 프리뷰 배포용 .env.preview 파일을 새로 추가하여 각각의 환경별 설정값(Sentry 환경, 웹 주소, API 서버 주소, Kakao 키 등)을 분리했습니다.

  3. 환경 설정 가이드 문서 개선: .env.guide.md 파일을 확장하여 환경별 파일의 용도와 커밋 정책을 명시하고, 배포 프로세스 및 모범 사례를 재구성했습니다.

  4. CI/CD 워크플로우 제거: .github/workflows/build.yml 파일(메인 브랜치 푸시 시 Vercel 프로덕션 배포를 자동화하는 워크플로우)을 삭제했습니다.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Suggested reviewers

  • wibaek
  • enunsnv
  • khwww
🚥 Pre-merge checks | ✅ 1 | ❌ 2
❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Description check ⚠️ Warning PR 설명은 템플릿 구조는 있으나, 관련 이슈 번호, 작업 내용, 특이 사항 등 모든 필수 섹션이 주석 처리되어 있어 실질적인 정보가 없습니다. 템플릿의 각 섹션(관련 이슈, 작업 내용, 특이 사항, 리뷰 요구사항)을 채우고 환경 설정 파일 변경의 목적과 영향을 명확히 설명하세요.
Title check ❓ Inconclusive PR 제목 'Fix/vercel build'는 Vercel 빌드 관련 수정을 나타내지만, 실제 주요 변경사항(환경 설정 파일 구조화 및 워크플로우 재정의)을 충분히 설명하지 못합니다. 제목을 '환경별 .env 파일 분리 및 Vercel 통합 기반 배포 구조로 개선' 또는 유사한 내용으로 더 구체적으로 수정하여 변경사항의 핵심을 명확히 전달하세요.
✅ 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

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

@manNomi manNomi merged commit e68357e into main Jan 25, 2026
4 of 5 checks passed
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: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
.env (1)

16-23: 공개 리포지토리에 실제 API 키가 커밋되어 있어 긴급한 보안 대응이 필요합니다.

  1. 즉시 키 로테이션 필수

    • 현재 노출된 Google Maps, Analytics, Apple 키는 즉시 무효화하고 새 키로 교체해야 합니다.
    • GitHub public 저장소에 노출되었으므로 악용 가능성이 매우 높습니다.
  2. git 이력에서 키 제거

    • .env 파일이 git에 추적 중이며 커밋 이력에 남아있습니다.
    • git filter-branch 또는 bfg-repo-cleaner로 히스토리 정리가 필요합니다.
  3. 향후 방지 방법

    • .env 파일을 .gitignore에 추가하고 (현재는 .env*.local만 있음)
    • Vercel 환경변수 설정으로 실제 값 관리
    • .env.example 파일에만 플레이스홀더 남기기
🔧 권장 수정안
-.env
+.env
+.env.development
+.env.preview
+.env.production

.gitignore에 추가 후, Vercel 대시보드에서 직접 환경변수 설정하세요.

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