Skip to content

feat(ui): Add LLM issue detection toggle to performance settings#112711

Open
roggenkemper wants to merge 16 commits intomasterfrom
roggenkemper/feat/llm-detection-enable-option-fe
Open

feat(ui): Add LLM issue detection toggle to performance settings#112711
roggenkemper wants to merge 16 commits intomasterfrom
roggenkemper/feat/llm-detection-enable-option-fe

Conversation

@roggenkemper
Copy link
Copy Markdown
Member

@roggenkemper roggenkemper commented Apr 10, 2026

Add an LLM Issue Detection toggle to the detector threshold settings on the project performance page, using the same performance issues endpoint as other detector toggles. Gated behind gen-ai-features and ai-issue-detection flags

Screenshot 2026-04-15 at 3 51 45 PM

Add an LLM Issue Detection toggle to the detector threshold settings
on the project performance page, using the same performance issues
endpoint as other detector toggles. Gated behind gen-ai-features.

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 is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit add55a0. Configure here.

Comment thread static/app/views/settings/projectPerformance/projectPerformance.tsx
Each AI detected issue category (HTTP, Database, Runtime Performance,
Security, Code Health, General) gets its own toggle as a sub-option
under the main AI Issue Detection setting. Sub-options are disabled
when the parent toggle is off.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@sentry
Copy link
Copy Markdown
Contributor

sentry bot commented Apr 15, 2026

Sentry Snapshot Testing

Name Added Removed Modified Renamed Unchanged Status
sentry-frontend
sentry-frontend
0 0 0 0 204 ✅ Unchanged

⚙️ sentry-frontend Snapshot Settings

…rontend

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…2 title

Replace IssueTitle.LLM_DETECTED_EXPERIMENTAL_V2 references with plain
'AI Issue Detection' string for the settings section title and admin
field key.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Each per-category toggle now has a subtitle explaining what it controls
(e.g., "Allow HTTP-related issues to be created").

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@roggenkemper roggenkemper force-pushed the roggenkemper/feat/llm-detection-enable-option-fe branch from 0a9c7c2 to a474637 Compare April 15, 2026 19:23
…toggle

The admin field map key must match the section title for the parent toggle
to render in the section header. Also removed the General Issues toggle
since it doesn't need a separate setting.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
roggenkemper and others added 2 commits April 15, 2026 15:28
Use plain string for section title to match admin field map key so the
parent toggle renders in the section header. Remove hyphens from help text.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
All fields in the AI Issue Detection section now share the same visibility
condition so the entire section is hidden when the feature flags are off.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Use AI-specific description instead of the generic "detect this type of
issue" text. Reorder the spread so field-level help takes precedence over
the generic default.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Use a shared constant for the title so the admin field map key, label,
and section title all reference the same translated string.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
roggenkemper and others added 2 commits April 15, 2026 15:45
…title

Match the pattern used by other detection sections instead of a local variable.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…TECTED_GENERAL

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Section title stays as IssueTitle.AI_DETECTED_GENERAL ('AI Detected Issue'),
parent toggle label is 'AI Issue Detection'.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@roggenkemper roggenkemper marked this pull request as ready for review April 15, 2026 19:54
@roggenkemper roggenkemper requested a review from a team as a code owner April 15, 2026 19:54
@roggenkemper roggenkemper requested review from a team and removed request for a team April 15, 2026 19:54
Comment thread static/app/views/settings/projectPerformance/projectPerformance.tsx
Comment thread static/app/views/settings/projectPerformance/projectPerformance.tsx
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