Skip to content

Fix/vercel build#381

Merged
manNomi merged 2 commits intomainfrom
fix/vercel-build
Jan 25, 2026
Merged

Fix/vercel build#381
manNomi merged 2 commits intomainfrom
fix/vercel-build

Conversation

@manNomi
Copy link
Contributor

@manNomi manNomi commented Jan 25, 2026

관련 이슈

  • resolves: #이슈 번호

작업 내용

특이 사항

리뷰 요구사항 (선택)

@manNomi
chore: remove build.yml workflow again
14f6352 
- Vercel Integration handles all automatic deployments
- Only keep release.yml for manual production releases
@manNomi
feat: restore .env.development and .env.preview files
99d7d08 
- 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 •
edited
Loading

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 •
edited
Loading

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
coderabbitai[bot]
coderabbitai bot reviewed Jan 25, 2026
View reviewed changes
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

Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] left review comments

@wibaek wibaek Awaiting requested review from wibaek wibaek is a code owner

@enunsnv enunsnv Awaiting requested review from enunsnv enunsnv is a code owner

@khwww khwww Awaiting requested review from khwww khwww is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@manNomi