ref(issue-details): Remove useHasStreamlinedUI hook and legacy UI code by chromy · Pull Request #112465 · getsentry/sentry

chromy · 2026-04-08T13:49:49Z

Remove the useHasStreamlinedUI hook which always returned true, along with
all legacy (non-streamlined) UI code paths gated behind it.

Changes:

Delete useHasStreamlinedUI from utils.tsx
Remove all call sites and simplify conditional branches in
groupEventDetailsContent, groupEventDetails, and groupDetails
Remove dead imports, styled components, and functions that were only
reachable from the legacy path (renderGroupStatusBanner,
trackTabChanged, StyledLayoutBody, StyledLayoutMain,
GroupStatusBannerWrapper, GroupTabPanels, NotFoundMessage,
StyledDataSection, etc.)
Remove jest.mock overrides of the hook in test files
Simplify inputWithStorage.tsx to no longer gate on the hook

Agent transcript: https://claudescope.sentry.dev/share/N42GnuAIP46Yfs6hbcGKQ4Uybjtnlxsgws23v5IjV_o

static/app/views/issueDetails/groupEventDetails/groupEventDetailsContent.tsx

static/app/views/issueDetails/groupEventDetails/groupEventDetails.tsx

static/app/views/issueDetails/groupEventDetails/groupEventDetailsContent.tsx

static/app/views/issueDetails/groupDetails.tsx

scttcper · 2026-04-08T14:38:16Z

static/app/views/issueDetails/groupDetails.tsx

    enforced_streamlined_ui: user?.options?.prefersIssueDetailsStreamlinedUI === null,
    org_streamline_only: organization.streamlineOnly ?? undefined,
-    has_streamlined_ui: hasStreamlinedUI,
+    has_streamlined_ui: true,


static/app/views/issueDetails/groupDetails.tsx

scttcper · 2026-04-08T15:49:22Z

static/app/components/resolutionBox.tsx

can probably rename this file or combine it with the other one maybe, can't tell if its used in one or two places

renamed to resolutionReason.tsx in a follow up #112509.

scttcper · 2026-04-08T15:50:38Z

static/app/components/events/interfaces/crashContent/exception/actionableItems.tsx

this file looks like it could be renamed or the function relocated as well

Renamed to errorMessages.tsx in a follow up #112509.

malwilley · 2026-04-08T18:01:20Z

static/app/components/events/styles.tsx

now that this just exports DataSection, it probably would be better if the file were renamed to match?

Thanks, renamed to dataSection.tsx in a follow up #112509!

malwilley · 2026-04-08T18:02:21Z

static/app/components/group/inboxBadges/statusBadge.tsx

this no longer exports GroupStatusBadge so the file name no longer matches, maybe we should just inline the function here where it's being used?

static/app/views/issueDetails/groupDetails.tsx

Remove the useHasStreamlinedUI hook which always returned true, and clean up all legacy (non-streamlined) UI code paths that were gated behind it. This removes dead code including unused styled components, the old tab-based layout, and legacy conditional branches across groupEventDetailsContent, groupEventDetails, and groupDetails. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Agent transcript: https://claudescope.sentry.dev/share/GEUTjwTLS-03zoOgJLc4nNOPl6d6kawqWXMe3Bn-fOQ

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Agent transcript: https://claudescope.sentry.dev/share/pxAVht4UjkdydDJ04XfTa8Ow3ZZ8hQtyPe4WbpfpBWs

cursor

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit f4f08a1. Configure here.}

cursor · 2026-04-10T08:18:53Z

static/app/views/issueDetails/groupEventDetails/groupEventDetailsContent.tsx

-      {hasStreamlinedUI && (
-        <EventProcessingErrors event={event} project={project} isShare={false} />
-      )}
+      <EventProcessingErrors event={event} project={project} isShare={false} />


Large exported function is now dead code

Medium Severity

EventDetailsContent is a ~300-line exported function that is now completely dead code. GroupEventDetailsContent was the only caller, and it now delegates directly to EventDetails, bypassing EventDetailsContent entirely. This also leaves behind many unused imports (e.g., ClassNames, GuideAnchor, ScreenshotDataSection, EventTagsDataSection, InterimSection, useCopyIssueDetails, InstrumentationFixSection, and many more) that are only referenced within the dead function.

Additional Locations (1)

static/app/views/issueDetails/groupEventDetails/groupEventDetailsContent.tsx#L410-L417

^{Reviewed by Cursor Bugbot for commit f4f08a1. Configure here.}

Follow-up to #112465. After removing legacy UI components, several files no longer matched their contents. This renames them to reflect what they actually export: resolutionBox.tsx → resolutionReason.tsx (only exports ResolutionReason) events/styles.tsx → events/dataSection.tsx (only exports DataSection) statusBadge.tsx → getBadgeProperties.tsx (only exports getBadgeProperties) actionableItems.tsx → errorMessages.tsx (only exports ErrorMessage type and getErrorMessage) All import paths updated accordingly. Agent transcript: https://claudescope.sentry.dev/share/e2k0R-zMUZqOtcDaAQ4kCZutdhmRwnSTBSqDOHTHtGo

github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 8, 2026

chromy marked this pull request as ready for review April 8, 2026 13:50

chromy requested review from a team as code owners April 8, 2026 13:50

vercel bot deployed to Preview April 8, 2026 13:51 View deployment

cursor bot reviewed Apr 8, 2026

View reviewed changes

static/app/views/issueDetails/groupEventDetails/groupEventDetailsContent.tsx Outdated Show resolved Hide resolved

chromy commented Apr 8, 2026

View reviewed changes

static/app/views/issueDetails/groupEventDetails/groupEventDetails.tsx Show resolved Hide resolved

chromy requested a review from a team as a code owner April 8, 2026 14:02

vercel bot deployed to Preview April 8, 2026 14:04 View deployment

vercel bot deployed to Preview April 8, 2026 14:10 View deployment

cursor bot reviewed Apr 8, 2026

View reviewed changes

static/app/views/issueDetails/groupEventDetails/groupEventDetailsContent.tsx Show resolved Hide resolved

static/app/views/issueDetails/groupDetails.tsx Show resolved Hide resolved

vercel bot deployed to Preview April 8, 2026 14:16 View deployment

scttcper reviewed Apr 8, 2026

View reviewed changes

vercel bot deployed to Preview April 8, 2026 15:06 View deployment

cursor bot reviewed Apr 8, 2026

View reviewed changes

static/app/views/issueDetails/groupDetails.tsx Outdated Show resolved Hide resolved

scttcper reviewed Apr 8, 2026

View reviewed changes

vercel bot deployed to Preview April 8, 2026 17:32 View deployment

chromy force-pushed the chromy/ref/remove-use-has-streamlined-ui branch from d840bc9 to 8c18962 Compare April 8, 2026 17:35

vercel bot deployed to Preview April 8, 2026 17:37 View deployment

malwilley reviewed Apr 8, 2026

View reviewed changes

vercel bot deployed to Preview April 8, 2026 18:05 View deployment

scttcper approved these changes Apr 8, 2026

View reviewed changes

chromy mentioned this pull request Apr 8, 2026

ref(ui): Rename files to match remaining exports #112509

Open

chromy force-pushed the chromy/ref/remove-use-has-streamlined-ui branch from 97c35e6 to 0eff47d Compare April 9, 2026 12:07

vercel bot deployed to Preview April 9, 2026 12:08 View deployment

cursor bot reviewed Apr 9, 2026

View reviewed changes

static/app/views/issueDetails/groupDetails.tsx Show resolved Hide resolved

chromy added 2 commits April 10, 2026 09:16

ref(ui): Remove orphaned files and unused exports

f4f08a1

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Agent transcript: https://claudescope.sentry.dev/share/pxAVht4UjkdydDJ04XfTa8Ow3ZZ8hQtyPe4WbpfpBWs

chromy force-pushed the chromy/ref/remove-use-has-streamlined-ui branch from 0eff47d to f4f08a1 Compare April 10, 2026 08:16

vercel bot deployed to Preview April 10, 2026 08:18 View deployment

cursor bot reviewed Apr 10, 2026

View reviewed changes

chromy merged commit eafc821 into master Apr 10, 2026
65 checks passed

chromy deleted the chromy/ref/remove-use-has-streamlined-ui branch April 10, 2026 10:04

Uh oh!

Conversation

chromy commented Apr 8, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

scttcper Apr 8, 2026

Choose a reason for hiding this comment

Uh oh!

chromy Apr 10, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

scttcper Apr 8, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

chromy Apr 10, 2026

Choose a reason for hiding this comment

Uh oh!

scttcper Apr 8, 2026

Choose a reason for hiding this comment

Uh oh!

chromy Apr 10, 2026

Choose a reason for hiding this comment

Uh oh!

malwilley Apr 8, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

chromy Apr 10, 2026

Choose a reason for hiding this comment

Uh oh!

malwilley Apr 8, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

cursor bot left a comment

Choose a reason for hiding this comment

Uh oh!

cursor bot Apr 10, 2026

Choose a reason for hiding this comment

Large exported function is now dead code

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

chromy commented Apr 8, 2026 •

edited

Loading

scttcper Apr 8, 2026 •

edited

Loading

malwilley Apr 8, 2026 •

edited

Loading

malwilley Apr 8, 2026 •

edited

Loading