Skip to content

Add Sentry integration - #32

Merged
Dmytro-Fedoryshyn merged 15 commits into
mainfrom
sentry-integration
Jul 8, 2026
Merged

Add Sentry integration#32
Dmytro-Fedoryshyn merged 15 commits into
mainfrom
sentry-integration

Conversation

@Dmytro-Fedoryshyn

@Dmytro-Fedoryshyn Dmytro-Fedoryshyn commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

In this changeset, we have integrated Sentry into MōVoice to track basic launch and crash telemetry.

Issue: #29

@Dmytro-Fedoryshyn Dmytro-Fedoryshyn self-assigned this Jul 7, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Integrates Sentry into MōVoice’s main and renderer processes to capture basic launch/crash telemetry, and updates the Vite main-process bundle configuration to account for additional runtime externals introduced by the Sentry Node SDK.

Changes:

  • Add Sentry initialization for the renderer (@sentry/react) and wire React root error callbacks to Sentry.
  • Add Sentry initialization for the main process (@sentry/node) and capture a launch event plus initialization failures.
  • Expand Vite main-process Rollup externals to include Node built-ins and Sentry’s hook-related dependencies; add Sentry dependencies to package.json/lockfile.

Reviewed changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
vite.config.ts Expands main-process Rollup externals to include Node built-ins and Sentry-related runtime deps.
src/renderer/instrument.ts Initializes Sentry for the renderer process and tags events with process=renderer.
src/renderer/index.tsx Imports renderer instrumentation and routes React root errors to Sentry handlers.
src/main/instrument.ts Initializes Sentry for the main process, tags events, and emits launch/session telemetry.
src/main/index.ts Imports main instrumentation and flushes Sentry on initialization failure.
package.json Adds @sentry/node and @sentry/react dependencies.
package-lock.json Locks Sentry and transitive dependencies.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/renderer/instrument.ts Outdated
Comment thread src/renderer/instrument.ts Outdated
Comment thread src/renderer/index.tsx
Comment thread src/main/instrument.ts Outdated
Comment thread src/main/instrument.ts Outdated
Comment thread src/main/instrument.ts Outdated
@Dmytro-Fedoryshyn
Dmytro-Fedoryshyn marked this pull request as ready for review July 8, 2026 09:48
Comment thread src/main/instrument.ts Outdated
Comment thread src/main/instrument.ts Outdated
Comment thread src/main/instrument.ts Outdated
Comment thread src/main/instrument.ts Outdated
Comment thread src/main/instrument.ts Outdated
Comment thread src/renderer/instrument.ts Outdated
Comment thread src/renderer/instrument.ts Outdated
Comment thread src/renderer/instrument.ts Outdated
Comment thread vite.config.ts Outdated
Comment thread src/main/sentry.ts Outdated
Comment thread vite.config.ts Outdated
Comment thread vite.config.ts Outdated
Comment thread vite.config.ts Outdated
@Dmytro-Fedoryshyn
Dmytro-Fedoryshyn merged commit 6b73c4e into main Jul 8, 2026
2 checks passed
@Dmytro-Fedoryshyn
Dmytro-Fedoryshyn deleted the sentry-integration branch July 8, 2026 16:30
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.

3 participants