Skip to content

feat(llm-detector): Add beta badge for AI detected issues#112686

Merged
roggenkemper merged 1 commit intomasterfrom
roggenkemper/feat/llm-v2-beta-badge
Apr 14, 2026
Merged

feat(llm-detector): Add beta badge for AI detected issues#112686
roggenkemper merged 1 commit intomasterfrom
roggenkemper/feat/llm-v2-beta-badge

Conversation

@roggenkemper
Copy link
Copy Markdown
Member

@roggenkemper roggenkemper commented Apr 10, 2026

re-adds that beta tag for certain issue types that was removed in #99550

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 10, 2026
Copy link
Copy Markdown
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

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

Fix All in Cursor

Bugbot Autofix prepared a fix for the issue found in the latest run.

  • ✅ Fixed: Grid gap applied even when badge is absent
    • Changed Title component from CSS Grid to Flexbox, which naturally collapses the gap when the FeatureBadge is not rendered, ensuring full title space for non-LLM issues.

Create PR

Or push these changes by commenting:

@cursor push a2985ade4a
Preview (a2985ade4a)
diff --git a/static/app/views/issueDetails/streamline/header/header.tsx b/static/app/views/issueDetails/streamline/header/header.tsx
--- a/static/app/views/issueDetails/streamline/header/header.tsx
+++ b/static/app/views/issueDetails/streamline/header/header.tsx
@@ -326,7 +326,8 @@
   white-space: nowrap;
   font-size: 20px;
   font-weight: ${p => p.theme.font.weight.sans.medium};
-  flex-shrink: 0;
+  flex-shrink: 1;
+  min-width: 0;
 `;
 
 const StatTitle = styled('div')`
@@ -410,10 +411,10 @@
 `;
 
 const Title = styled('div')`
-  display: grid;
-  grid-template-columns: minmax(0, max-content) min-content;
+  display: flex;
   align-items: center;
-  column-gap: ${p => p.theme.space.sm};
+  gap: ${p => p.theme.space.sm};
+  min-width: 0;
 `;
 
 const StyledBreadcrumbs = styled(Breadcrumbs)`

This Bugbot Autofix run was free. To enable autofix for future PRs, go to the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 2e72334. Configure here.

Comment thread static/app/views/issueDetails/streamline/header/header.tsx
@roggenkemper roggenkemper marked this pull request as ready for review April 13, 2026 18:11
@roggenkemper roggenkemper requested a review from a team as a code owner April 13, 2026 18:11
@roggenkemper roggenkemper requested a review from scttcper April 13, 2026 18:11
<PrimaryTitle>{primaryTitle}</PrimaryTitle>
</Tooltip>
{group.issueType === IssueType.LLM_DETECTED_EXPERIMENTAL_V2 && (
<FeatureBadge type="beta" />
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Bug: The Title component's grid layout defines two columns with a gap, but the second column's content is conditional. This creates unwanted whitespace for most issue types.
Severity: LOW

Suggested Fix

Make the grid template conditional, adding the second column only when the badge is rendered. Alternatively, use display: flex with conditional rendering of the badge, or use grid-auto-columns instead of grid-template-columns to avoid creating an explicit empty track.

Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent.
Verify if this is a real issue. If it is, propose a fix; if not, explain why it's not
valid.

Location: static/app/views/issueDetails/streamline/header/header.tsx#L193

Potential issue: The `Title` component's CSS was changed to a two-column grid layout
(`grid-template-columns: minmax(0, max-content) min-content;`) with a `column-gap`.
However, the content for the second column, the `FeatureBadge`, is only rendered for
`LLM_DETECTED_EXPERIMENTAL_V2` issues. For all other issue types, the second column is
empty but the `column-gap` still applies. This results in an unwanted whitespace gap to
the right of the title, creating inconsistent spacing for the vast majority of issues.

Did we get this right? 👍 / 👎 to inform future reviews.

@roggenkemper roggenkemper merged commit 14ed490 into master Apr 14, 2026
68 checks passed
@roggenkemper roggenkemper deleted the roggenkemper/feat/llm-v2-beta-badge branch April 14, 2026 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants