Skip to content

開発環境でSentryの初期化はしない#5371

Merged
TinyKitten merged 1 commit intodevfrom
chore/sentry-local
Feb 20, 2026
Merged

開発環境でSentryの初期化はしない#5371
TinyKitten merged 1 commit intodevfrom
chore/sentry-local

Conversation

@TinyKitten
Copy link
Member

@TinyKitten TinyKitten commented Feb 20, 2026

Summary by CodeRabbit

リリースノート

  • 修正
    • エラートラッキングサービスの初期化が本番環境に限定されるようになりました。これまでは全ての環境で有効でしたが、本番環境でのみ機能するように改善されました。プライバシー設定とリプレイ機能の動作は変更されていません。

@TinyKitten TinyKitten self-assigned this Feb 20, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 20, 2026

No actionable comments were generated in the recent review. 🎉


📝 Walkthrough

ウォークスルー

Sentry初期化をproduction環境専用ガードで囲む変更。NODE_ENVが'production'と等しい場合にのみSentry.init()が実行されるようになり、その他のSentry設定と統合は変更されていない。

変更

コホート / ファイル 概要
Sentry初期化設定
index.js
Sentry.init()をNODE_ENV === 'production'条件で囲み、production環境でのみ初期化を実行するように修正。プライバシーオプションとリプレイ設定は保持。

見積もりコード審査労力

🎯 1 (Trivial) | ⏱️ ~5分

関連の可能性があるPR

  • シミュレータでSentryを使わない #4031: 同じindex.jsのSentry初期化ロジックをproduction環境専用に変更する修正で、本PRと同様のアプローチを採用している(本PR: NODE_ENV === 'production'、該当PR: __DEV__)。

🐰✨ ウサギより

Sentryよ本番のみで、
開発は静かに寝かせて、
環境に優しく、
初期化も賢く選別。
ふんわり軽くなったね!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed タイトルは、Sentryの初期化を本番環境のみに限定するという変更内容を正確に反映しており、変更内容と完全に一致しています。
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 (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/sentry-local

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

@TinyKitten TinyKitten merged commit 525ef75 into dev Feb 20, 2026
5 checks passed
@TinyKitten TinyKitten deleted the chore/sentry-local branch February 20, 2026 09:11
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