Skip to content

Comments

chore(sc-65330): Improve error reporting for apps by reporting to Sentry#77

Merged
AFOJ merged 4 commits intomainfrom
chore/sc-65330/improve-error-reporting-for-apps-sentry
Jun 6, 2025
Merged

chore(sc-65330): Improve error reporting for apps by reporting to Sentry#77
AFOJ merged 4 commits intomainfrom
chore/sc-65330/improve-error-reporting-for-apps-sentry

Conversation

@HappyPaul55
Copy link
Contributor

This PR improves error reporting for apps by integrating Sentry. It includes the necessary configurations and updates to the main.tsx file to handle errors effectively.

Copilot AI review requested due to automatic review settings May 28, 2025 11:23
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR integrates Sentry error reporting into the application, adding configuration for both development and CI environments.

  • Add and configure the Sentry Vite plugin and enable source maps for builds
  • Initialize Sentry at runtime via a new instrument.ts file and import it in main.tsx
  • Update dependencies and CI workflow to pass Sentry credentials and toggle plugin usage on tagged releases

Reviewed Changes

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

Show a summary per file
File Description
vite.config.ts Inject sentryVitePlugin into Vite plugins and enable sourcemap
src/main.tsx Import Sentry runtime hook (./instrument) and Sentry package
src/instrument.ts New file to initialize Sentry based on URL query parameters
package.json Add @sentry/react and @sentry/vite-plugin; remove unused react-error-boundary
.github/workflows/subworkflow-build.yml Pass Sentry secrets into CI and conditionally disable plugin on non-tag builds
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported
Comments suppressed due to low confidence (1)

src/instrument.ts:1

  • New error instrumentation logic in instrument.ts isn't covered by tests. Please add unit tests to verify DSN and tunnel parameter parsing and ensure Sentry is initialized as expected.
import * as Sentry from "@sentry/react";

@github-actions
Copy link

@AFOJ AFOJ merged commit ed7d0a1 into main Jun 6, 2025
5 checks passed
@AFOJ AFOJ deleted the chore/sc-65330/improve-error-reporting-for-apps-sentry branch June 6, 2025 13:07
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.

2 participants