Skip to content

[공통] Sentry 설정 수정#1219

Merged
ff1451 merged 1 commit intodevelopfrom
chore/#1218/sentry-setting
Mar 30, 2026
Merged

[공통] Sentry 설정 수정#1219
ff1451 merged 1 commit intodevelopfrom
chore/#1218/sentry-setting

Conversation

@ff1451
Copy link
Copy Markdown
Contributor

@ff1451 ff1451 commented Mar 30, 2026

What is this PR? 🔍

Changes 📝

  • disableLogger 옵션이 deprecated되어 새롭게 추가된 bundleSizeOptimizations.excludeDebugStatements: true로 변경했습니다.
  • widenClientFileUpload: false → true : 처음 설정 시 빌드 시간 증가를 막기 위해 false로 설정했으나 정확한 에러 추적에 어려움이 있어 모든 sourceMap을 Sentry에 업로드하도록 변경했습니다.
  • sourcemaps.deleteSourcemapsAfterUpload: true 추가하여 프로덕션 클라이언트에는 .map 파일이 노출되지 않아 소스 코드 보안이
    유지되도록 했습니다.

✔️ Please check if the PR fulfills these requirements

  • It's submitted to the correct branch, not the develop branch unconditionally?
  • If on a hotfix branch, ensure it targets main?
  • There are no warning message when you run yarn lint

Summary by CodeRabbit

릴리스 노트

  • Chores
    • 에러 추적 인프라 설정 개선으로 더욱 효율적인 문제 모니터링 구성
    • 빌드 최적화 및 자동 정리 기능 추가로 배포 프로세스 향상

@ff1451 ff1451 self-assigned this Mar 30, 2026
@ff1451 ff1451 added the ⚙ Setting 개발 환경 세팅 label Mar 30, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 30, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 964d1d4f-cf7e-4be4-b7db-68b89a4a7803

📥 Commits

Reviewing files that changed from the base of the PR and between be79e9c and 67af3e1.

📒 Files selected for processing (1)
  • next.config.mjs

요약

Sentry Next.js 설정을 업데이트하여 클라이언트 파일 업로드 범위를 확장하고, 빌드 최적화 및 소스맵 관리 옵션을 추가했습니다.

변경 사항

코호트 / 파일 요약
Sentry 설정 업데이트
next.config.mjs
widenClientFileUploadtrue로 활성화하고, 폐기된 disableLogger 옵션을 제거한 후 bundleSizeOptimizations 블록(디버그 구문 제외)과 sourcemaps 설정(업로드 후 소스맵 삭제)을 추가했습니다.

예상 코드 리뷰 노력

🎯 2 (Simple) | ⏱️ ~10 분

관련 가능성 있는 PR

  • [공통] Sentry 연결 #1199: Next.js Sentry 설정의 widenClientFileUpload 옵션을 포함하여 동일한 파일을 수정하고 있어 직접적인 연관이 있습니다.

추천 검토자

  • hyejun0228
  • ParkSungju01
  • dooohun
🚥 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 풀 리퀘스트 제목은 Sentry 설정 수정이라는 주요 변경사항을 명확하게 나타내고 있으며, 간결하고 구체적입니다.
Linked Issues check ✅ Passed 모든 코드 변경사항이 연결된 이슈 #1218의 요구사항을 충족합니다: deprecated된 disableLogger 제거, widenClientFileUpload 활성화, sourcemaps 설정 추가.
Out of Scope Changes check ✅ Passed 모든 변경사항은 Sentry 설정 수정 범위 내에 있으며, next.config.mjs 파일에서만 설정 관련 변경이 이루어졌습니다.
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
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/#1218/sentry-setting

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.

Copy link
Copy Markdown
Contributor

@ParkSungju01 ParkSungju01 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다!

@ff1451 ff1451 merged commit 44eeab9 into develop Mar 30, 2026
4 checks passed
@github-actions github-actions Bot deleted the chore/#1218/sentry-setting branch March 30, 2026 06:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⚙ Setting 개발 환경 세팅

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[공통] Sentry 설정 수정

2 participants