Skip to content

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

Merged
AFOJ merged 5 commits into
mainfrom
chore/sc-65330/improve-error-reporting-for-apps-sentry
Jun 20, 2025
Merged

chore(sc-65330): Improve error reporting for apps by reporting to Sentry#66
AFOJ merged 5 commits into
mainfrom
chore/sc-65330/improve-error-reporting-for-apps-sentry

Conversation

@HappyPaul55

Copy link
Copy Markdown
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:20

Copilot AI left a comment

Copy link
Copy Markdown

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 improves error reporting by integrating Sentry into the application’s configuration, initialization, and build process.

  • Adds conditional Sentry integration in the Vite configuration
  • Integrates Sentry initialization in the application startup and a new instrument file
  • Updates dependencies and CI build workflow to support Sentry reporting

Reviewed Changes

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

Show a summary per file
File Description
vite.config.ts Adds conditional Sentry Vite plugin and removes the WindiCSS plugin
src/main.tsx Incorporates Sentry import and instrument module integration
src/instrument.ts Introduces Sentry initialization based on URL parameters
package.json Adds Sentry dependencies while removing react-error-boundary dependency
.github/workflows/subworkflow-build.yml Updates CI configuration to include Sentry environment variables and logic
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported
Comments suppressed due to low confidence (2)

vite.config.ts:5

  • [nitpick] The removal of the WindiCSS plugin may affect styling if it is used elsewhere in the project. Confirm that this change is intentional and update documentation if necessary.
import WindiCSS from "vite-plugin-windicss";

package.json:36

  • [nitpick] Removal of the 'react-error-boundary' dependency should be verified to ensure that any required error boundary functionality is handled by Sentry or other means.
"react-error-boundary": "^4.1.2",

Comment thread .github/workflows/subworkflow-build.yml Outdated
@github-actions

Copy link
Copy Markdown

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

4 participants