feat(prevent): Add logaf sensitivity dropdown to ai config #101040

suejung-sentry · 2025-10-06T22:21:23Z

Add LOGAF sensitivity dropdown for the AI config

Design: https://linear.app/getsentry/issue/PREVENT-315/prevent-ai-logaf-severity-level-confg

codecov · 2025-10-06T23:35:15Z

❌ 2 Tests Failed:

Tests completed	Failed	Passed	Skipped
12126	2	12124	10

View the top 2 failed test(s) by shortest run time

ManageReposPanel disables toggles when loading

Stack Traces | 1.1s run time

Error: Found multiple elements with the text of: /PR Review/i

Here are the matching elements:

Ignored nodes: comments, script, style
...

Ignored nodes: comments, script, style
...

(If this is intentional, then use the `*AllBy*` variant of the query (like `queryAllByText`, `getAllByText`, or `findAllByText`)).

Ignored nodes: comments, script, style
...
    at waitForWrapper (.../sentry/node_modules/.pnpm/@testing-library+dom@10.4.0/node_modules/@.../dom/dist/wait-for.js:163:27)
    at .../sentry/node_modules/.pnpm/@testing-library+dom@10.4.0/node_modules/@.../dom/dist/query-helpers.js:86:33
    at Object.findByLabelText (.../prevent/preventAI/manageReposPanel.spec.tsx:87:25)
    at Promise.finally.completed (.../sentry/node_modules/.pnpm/jest-circus@30.0.4_babel-plugin-macros@3.1..../jest-circus/build/jestAdapterInit.js:1559:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (.../sentry/node_modules/.pnpm/jest-circus@30.0.4_babel-plugin-macros@3.1..../jest-circus/build/jestAdapterInit.js:1499:10)
    at _callCircusTest (.../sentry/node_modules/.pnpm/jest-circus@30.0.4_babel-plugin-macros@3.1..../jest-circus/build/jestAdapterInit.js:1009:40)
    at _runTest (.../sentry/node_modules/.pnpm/jest-circus@30.0.4_babel-plugin-macros@3.1..../jest-circus/build/jestAdapterInit.js:949:3)
    at _runTestsForDescribeBlock (.../sentry/node_modules/.pnpm/jest-circus@30.0.4_babel-plugin-macros@3.1..../jest-circus/build/jestAdapterInit.js:839:13)
    at _runTestsForDescribeBlock (.../sentry/node_modules/.pnpm/jest-circus@30.0.4_babel-plugin-macros@3.1..../jest-circus/build/jestAdapterInit.js:829:11)
    at run (.../sentry/node_modules/.pnpm/jest-circus@30.0.4_babel-plugin-macros@3.1..../jest-circus/build/jestAdapterInit.js:757:3)
    at runAndTransformResultsToJestFormat (.../sentry/node_modules/.pnpm/jest-circus@30.0.4_babel-plugin-macros@3.1..../jest-circus/build/jestAdapterInit.js:1920:21)
    at jestAdapter (.../sentry/node_modules/.pnpm/jest-circus@30.0.4_babel-plugin-macros@3.1..../jest-circus/build/runner.js:101:19)
    at runTestInternal (.../sentry/node_modules/.pnpm/jest-runner@30.0..../jest-runner/build/testWorker.js:272:16)
    at runTest (.../sentry/node_modules/.pnpm/jest-runner@30.0..../jest-runner/build/testWorker.js:340:7)
    at Object.worker (.../sentry/node_modules/.pnpm/jest-runner@30.0..../jest-runner/build/testWorker.js:494:12)

ManageReposPanel shows feature toggles with correct initial state

Stack Traces | 1.12s run time

Error: Found multiple elements with the text of: /PR Review/i

Here are the matching elements:

Ignored nodes: comments, script, style
...

Ignored nodes: comments, script, style
...

(If this is intentional, then use the `*AllBy*` variant of the query (like `queryAllByText`, `getAllByText`, or `findAllByText`)).

Ignored nodes: comments, script, style
...
    at waitForWrapper (.../sentry/node_modules/.pnpm/@testing-library+dom@10.4.0/node_modules/@.../dom/dist/wait-for.js:163:27)
    at .../sentry/node_modules/.pnpm/@testing-library+dom@10.4.0/node_modules/@.../dom/dist/query-helpers.js:86:33
    at Object.findByLabelText (.../prevent/preventAI/manageReposPanel.spec.tsx:72:25)
    at Promise.finally.completed (.../sentry/node_modules/.pnpm/jest-circus@30.0.4_babel-plugin-macros@3.1..../jest-circus/build/jestAdapterInit.js:1559:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (.../sentry/node_modules/.pnpm/jest-circus@30.0.4_babel-plugin-macros@3.1..../jest-circus/build/jestAdapterInit.js:1499:10)
    at _callCircusTest (.../sentry/node_modules/.pnpm/jest-circus@30.0.4_babel-plugin-macros@3.1..../jest-circus/build/jestAdapterInit.js:1009:40)
    at _runTest (.../sentry/node_modules/.pnpm/jest-circus@30.0.4_babel-plugin-macros@3.1..../jest-circus/build/jestAdapterInit.js:949:3)
    at _runTestsForDescribeBlock (.../sentry/node_modules/.pnpm/jest-circus@30.0.4_babel-plugin-macros@3.1..../jest-circus/build/jestAdapterInit.js:839:13)
    at _runTestsForDescribeBlock (.../sentry/node_modules/.pnpm/jest-circus@30.0.4_babel-plugin-macros@3.1..../jest-circus/build/jestAdapterInit.js:829:11)
    at run (.../sentry/node_modules/.pnpm/jest-circus@30.0.4_babel-plugin-macros@3.1..../jest-circus/build/jestAdapterInit.js:757:3)
    at runAndTransformResultsToJestFormat (.../sentry/node_modules/.pnpm/jest-circus@30.0.4_babel-plugin-macros@3.1..../jest-circus/build/jestAdapterInit.js:1920:21)
    at jestAdapter (.../sentry/node_modules/.pnpm/jest-circus@30.0.4_babel-plugin-macros@3.1..../jest-circus/build/runner.js:101:19)
    at runTestInternal (.../sentry/node_modules/.pnpm/jest-runner@30.0..../jest-runner/build/testWorker.js:272:16)
    at runTest (.../sentry/node_modules/.pnpm/jest-runner@30.0..../jest-runner/build/testWorker.js:340:7)
    at Object.worker (.../sentry/node_modules/.pnpm/jest-runner@30.0..../jest-runner/build/testWorker.js:494:12)

To view more test analytics, go to the Test Analytics Dashboard
_{📋 Got 3 mins? Take this short survey to help us improve Test Analytics.}

ajay-sentry

lgtm!

cursor

Bug: Mock Data Configuration Issue

The test_generation feature config in the mock data for `

static/app/views/prevent/preventAI/hooks/useUpdatePreventAIFeature.spec.tsx#L24-L28

sentry/static/app/views/prevent/preventAI/hooks/useUpdatePreventAIFeature.spec.tsx

Lines 24 to 28 in 4330acd

    
           }, 
        
           test_generation: { 
        
             enabled: false, 
        
             triggers: {on_command_phrase: false, on_ready_for_review: false}, 
        
           },

static/app/views/prevent/preventAI/hooks/useUpdatePreventAIFeature.spec.tsx#L44-L48

sentry/static/app/views/prevent/preventAI/hooks/useUpdatePreventAIFeature.spec.tsx

Lines 44 to 48 in 4330acd

    
           }, 
        
           test_generation: { 
        
             enabled: false, 
        
             triggers: {on_command_phrase: false, on_ready_for_review: false}, 
        
           },

(cherry picked from commit 8580194)

suejung-sentry added 7 commits October 2, 2025 21:27

feat(prevent): Hook up config api

8993e57

cleanup

949d46c

fix things

fc1f5a7

fix(prevent): Hook up to prevent ai orgs api

33facf2

fix tests

e04301d

fix test

4275f6d

feat(prevent): Add logaf sensitivity droopdown

48b4bd6

github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Oct 6, 2025

This comment was marked as outdated.

Sign in to view

vercel bot deployed to Preview October 6, 2025 22:38 View deployment

This comment was marked as outdated.

Sign in to view

suejung-sentry force-pushed the sshin/logaf branch 2 times, most recently from f6fbdbc to fbbbcc7 Compare October 6, 2025 23:34

vercel bot deployed to Preview October 6, 2025 23:37 View deployment

suejung-sentry force-pushed the sshin/logaf branch from fbbbcc7 to 57555e9 Compare October 6, 2025 23:44

vercel bot deployed to Preview October 6, 2025 23:46 View deployment

This comment was marked as outdated.

Sign in to view

fix test

ce4cd33

suejung-sentry force-pushed the sshin/logaf branch from 57555e9 to ce4cd33 Compare October 6, 2025 23:56

vercel bot deployed to Preview October 6, 2025 23:58 View deployment

This comment was marked as outdated.

Sign in to view

fix style

045f8c5

vercel bot deployed to Preview October 7, 2025 07:11 View deployment

ajay-sentry approved these changes Oct 7, 2025

View reviewed changes

Base automatically changed from sshin/orgs-api to master October 7, 2025 22:40

Merge remote-tracking branch 'origin/master' into sshin/logaf

fd19096

vercel bot deployed to Preview October 8, 2025 00:05 View deployment

cleanup merge conflicts

4330acd

vercel bot deployed to Preview October 8, 2025 00:18 View deployment

cursor bot reviewed Oct 8, 2025

View reviewed changes

suejung-sentry merged commit 8580194 into master Oct 8, 2025
46 checks passed

suejung-sentry deleted the sshin/logaf branch October 8, 2025 01:16

suejung-sentry added a commit that referenced this pull request Oct 8, 2025

feat(prevent): Add logaf sensitivity dropdown to ai config (#101040)

0a508c6

(cherry picked from commit 8580194)

suejung-sentry added a commit that referenced this pull request Oct 8, 2025

feat(prevent): Add logaf sensitivity dropdown to ai config (#101040)

e92df48

suejung-sentry added a commit that referenced this pull request Oct 8, 2025

feat(prevent): Add logaf sensitivity dropdown to ai config (#101040)

c6f90f1

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

feat(prevent): Add logaf sensitivity dropdown to ai config #101040

feat(prevent): Add logaf sensitivity dropdown to ai config #101040

suejung-sentry commented Oct 6, 2025

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

codecov bot commented Oct 6, 2025 •

edited

Loading

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

ajay-sentry left a comment

Uh oh!

cursor bot left a comment

Uh oh!

Uh oh!

Uh oh!

	},
	test_generation: {
	enabled: false,
	triggers: {on_command_phrase: false, on_ready_for_review: false},
	},

Uh oh!

feat(prevent): Add logaf sensitivity dropdown to ai config #101040

feat(prevent): Add logaf sensitivity dropdown to ai config #101040

Conversation

suejung-sentry commented Oct 6, 2025

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

codecov bot commented Oct 6, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

❌ 2 Tests Failed:

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

ajay-sentry left a comment

Choose a reason for hiding this comment

Uh oh!

cursor bot left a comment

Choose a reason for hiding this comment

Bug: Mock Data Configuration Issue

Uh oh!

Uh oh!

Uh oh!

codecov bot commented Oct 6, 2025 •

edited

Loading