Skip to content

Conversation

@k80bowman
Copy link
Contributor

Description

Adds Sentry error reporting to the CLI. Also adds scrubbing logic to ensure we are not sending PII or other privileged information to Sentry.

GUS work item: W-19894449

Testing

  1. Go to the CLI project in Sentry
  2. You should see several errors there already from my testing efforts
  3. If you'd like to send your own to test, checkout this branch locally and run yarn && yarn build. Then run IS_DEV_ENVIRONMENT="true" ./bin/run apps:info -a <non-existent-app>. It may take a little time, but you should see the error show up in Sentry.

Copy link
Contributor

@michaelmalave michaelmalave left a comment

Choose a reason for hiding this comment

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

changes look good and accomplish the goal. left a comment before approval.

Copy link
Contributor

@michaelmalave michaelmalave left a comment

Choose a reason for hiding this comment

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

Changes look good to me! Nice implementation.

@k80bowman k80bowman merged commit 15fe316 into main Dec 2, 2025
8 checks passed
@k80bowman k80bowman deleted the k80/sentry-setup branch December 2, 2025 21:39
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