chore: remove Sentry from the entire monorepo#14336
Merged
Merged
Conversation
|
Removes the Sentry SDK (web + mobile + identity-service), the
redux-sentry-middleware, the @sentry/cli release-cut job, and every
call site of reportToSentry / ErrorLevel / Feature.
Most reportToSentry({...}) calls become console.error(...) with the
same context payload. The ErrorReporting model and the sentry +
reportToSentry properties on CommonStoreContext / QueryContext are
deleted. SENTRY_DSN is dropped from web and mobile env + the Env type,
toucan-js is removed from the SSR worker, and the iOS dSYM upload
script, sentry.properties files, and CI sentry-cli steps go too.
verify (tsc + eslint) passes clean for common, web, and mobile after
the SDK and harmony are built locally. The two pre-existing
identity-service tsc errors are unrelated to this change.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
dylanjeffers
force-pushed
the
claude/lucid-darwin-a77873
branch
from
e5029bf to
ca6119f
Compare
Contributor
🌐 Web preview readyPreview URL: https://audius-web-preview-pr-14336.audius.workers.dev Unique preview for this PR (deployed from this branch). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
@sentry/browser,@sentry/node,@sentry/cli,redux-sentry-middleware,@types/redux-sentry-middleware,toucan-js) and theredux-sentry-middlewarewiring in the web store.packages/web/src/services/sentry.ts,packages/web/src/store/errors/reportToSentry.ts,packages/mobile/src/app/sentry.ts,packages/mobile/src/utils/reportToSentry.ts), the iOS dSYM upload script andsentry.propertiesfor both platforms.reportToSentry({...})call sites withconsole.error(...)(or removes them entirely when they were the only line in a catch). Removes thesentry/reportToSentryproperties fromCommonStoreContextandQueryContext, theFeature/ErrorLevel/ReportToSentryArgstypes, and theSENTRY_DSNfield fromEnv+ the web/mobile env files..github/workflows/{web,mobile}.ymland the matching.circleci/src/*.ymlsources (the generated.circleci/config.ymlwas already clean). Removes thesentryDSNconvict knob andconfigureSentry()call fromidentity-service.Test plan
tsc --noEmitpasses clean forpackages/common,packages/web, andpackages/mobile(2 pre-existing identity-service errors are unrelated to this change).eslint --ext=ts,tsx srcpasses clean forpackages/common,packages/web, andpackages/mobile.console.errorrather than a Sentry capture, with no runtime errors.console.errorstill fire.web-deploy-sentry-sourcemapsjob no longer runs and that the mobile build no longer attempts to installsentry-clior upload dSYMs.🤖 Generated with Claude Code