Skip to content

[CHA-165] 예상하지 못한 오류를 Sentry에 등록한다 - #275

Merged
DongjaJ merged 1 commit into
mainfrom
cha-165-feat-report-unexpected-errors
Aug 22, 2026
Merged

[CHA-165] 예상하지 못한 오류를 Sentry에 등록한다#275
DongjaJ merged 1 commit into
mainfrom
cha-165-feat-report-unexpected-errors

Conversation

@DongjaJ

@DongjaJ DongjaJ commented Aug 22, 2026

Copy link
Copy Markdown
Member

무엇을 변경했나요?

  • API 클라이언트 오류 상태를 보존하고 React Query의 최종 5xx 실패만 Sentry에 등록합니다.
  • 추천 결과와 채널 상세 오류 경계에서 API 오류가 아닌 렌더링 오류를 등록합니다.
  • 인증 BFF 5xx, 파일 업로드 실패, 로그아웃 best-effort 실패를 오류가 처리되기 전에 등록합니다.
  • 4xx, 네트워크 오류, 응답 파싱 실패는 수집하지 않으며 사용자·인증 데이터는 태그에 포함하지 않습니다.

왜 변경했나요?

UI 또는 응답으로 처리되어 기존 전역 오류 수집에서 누락되는 예상 밖 오류를 확인하기 위해 필요합니다. 이미 처리되는 비즈니스 오류는 제외해 Sentry 노이즈를 최소화합니다.

Linear: https://linear.app/chaesozip/issue/CHA-165/feature-에러시-sentry-등록

Closes CHA-165

확인 사항

필수 확인

  • node --run build 통과
    • 로컬 Doppler 설정과 NEXT_PUBLIC_API_BASE_URL이 없어 페이지 데이터 수집 단계에서 실패했습니다. 컴파일과 TypeScript 검사는 통과했습니다.
  • node --run lint 통과
  • node --run test 통과 또는 관련 테스트 확인
  • 관련 이슈를 연결했습니다. 예: Closes #123

기능 검증

  • 변경된 기능을 로컬에서 직접 확인했습니다
  • 사용자 플로우가 의도대로 동작하는지 확인했습니다
  • API 연동 또는 상태 관리 변경 사항을 확인했습니다
  • 에러/로딩/빈 상태 등 예외 케이스를 확인했습니다
  • 변경된 동작에 대한 테스트를 추가하거나 업데이트했습니다

설정 및 문서

  • 사용자에게 노출되는 문서를 업데이트했습니다. 예: README.md, 사용 가이드 등
  • 새로운 의존성이나 설정 변경 사항을 문서화했습니다
  • 환경 변수 추가/변경 사항을 명확히 설명했습니다
  • 민감한 값, 토큰, 인증 정보가 포함되지 않았습니다

해당되는 경우

  • 보안 영향을 검토했습니다
  • 성능 영향을 검토했습니다
  • 기존 동작을 변경하는 내용은 PR 설명에 명확히 작성했습니다
  • 스크린샷 또는 화면 녹화를 첨부했습니다

@linear-code

linear-code Bot commented Aug 22, 2026

Copy link
Copy Markdown

CHA-165

@coderabbitai

coderabbitai Bot commented Aug 22, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: d6193ac1-aab4-4e75-af92-4a31301aab3c


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.

@codecov

codecov Bot commented Aug 22, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 87.50000% with 5 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/shared/lib/sentry/error-reporting.ts 80.00% 1 Missing and 4 partials ⚠️

📢 Thoughts on this report? Let us know!

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@DongjaJ
DongjaJ marked this pull request as ready for review August 22, 2026 04:09
@github-actions
github-actions Bot requested a review from yereong August 22, 2026 04:09
@DongjaJ
DongjaJ merged commit 5b5c3b3 into main Aug 22, 2026
9 checks passed
@DongjaJ
DongjaJ deleted the cha-165-feat-report-unexpected-errors branch August 22, 2026 04:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant