feat(manager): use Sentry User Feedback API for manager feedback#2743
Open
feat(manager): use Sentry User Feedback API for manager feedback#2743
Conversation
…ion for Electron 30
Migrate from Sentry.captureEvent to Sentry.captureFeedback in both the contact-us dialog and the legacy feedback path in app.ts. Also removes the preload-based URL redaction workaround (now done inline in app.ts) and fixes the Chromium version test to resolve electron from server_manager's own node_modules, so it works when the client and server_manager use different Electron versions. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
fortuna
changed the title
ohnorobo
approved these changes
Apr 13, 2026
Contributor
ohnorobo
left a comment
ohnorobo
left a comment
There was a problem hiding this comment.
This LGTM. (minus tiny lint fixes) Thank you for breaking it out from the client change.
Nevertheless please DO NOT MERGE this until I've finished a clean 1.20.0 release across platforms. I want to shake out any remaining unknown-unknowns from the transfer process before we add an electron upgrade on top.
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
2 participants
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.captureEvent→Sentry.captureFeedbackinoutline-contact-us-dialog.tsandapp.tsto use the Sentry User Feedback APIapp.tswhere it belongs@sentry/electronto v7,electron-to-chromiumto ^1.5.331,PUPPETEER_CHROMIUM_REVISIONto 1274542)electron/package.jsonfromserver_manager/node_modulesso it picks up Electron 30 even when the client remains on Electron 19Test plan
npm run action server_manager/test— all 18 jasmine specs pass, including the Chromium version checknpm run action server_manager/www/test— all 59 Karma tests pass🤖 Generated with Claude Code