rm unused props

Author: ryan953

static/app/views/settings/seer/seerAgentHooks.spec.tsx

@@ -75,12 +75,9 @@ describe('seerAgentHooks', () => {
 
   describe('useSelectedAgentFromProjectSettings', () => {
     it('returns "none" when project autofixAutomationTuning is off', () => {
-      const p = ProjectFixture({...project, autofixAutomationTuning: 'off'});
-
       const {result} = renderHookWithProviders(useSelectedAgentFromProjectSettings, {
         initialProps: {
           preference: {repositories: []},
-          project: p,
           integrations: [],
         },
         organization,
@@ -93,7 +90,6 @@ describe('seerAgentHooks', () => {
       const {result} = renderHookWithProviders(useSelectedAgentFromProjectSettings, {
         initialProps: {
           preference: {repositories: []},
-          project,
           integrations: [],
         },
         organization,
@@ -117,7 +113,6 @@ describe('seerAgentHooks', () => {
               integration_id: 99,
             },
           },
-          project,
           integrations,
         },
         organization,

static/app/views/settings/seer/seerAgentHooks.tsx

@@ -45,17 +45,11 @@ export function useAgentOptions({
 export function useSelectedAgentFromProjectSettings({
   integrations,
   preference,
-  project,
 }: {
   integrations: CodingAgentIntegration[];
   preference: ProjectSeerPreferences;
-  project: Project;
 }) {
   return useMemo(() => {
-    // If we have autofixAutomationTuning==OFF then 'none' is picked
-    if (project.autofixAutomationTuning === 'off') {
-      return 'none';
-    }
     // If we have nothing in preferences, then we have Seer
     if (!preference?.automation_handoff?.integration_id) {
       return 'seer';
@@ -65,11 +59,7 @@ export function useSelectedAgentFromProjectSettings({
       integration =>
         integration.id === String(preference.automation_handoff?.integration_id)
     );
-  }, [
-    preference?.automation_handoff?.integration_id,
-    project.autofixAutomationTuning,
-    integrations,
-  ]);
+  }, [preference.automation_handoff?.integration_id, integrations]);
 }
 
 export function useSelectedAgentFromBulkSettings({

static/gsApp/views/seerAutomation/components/projectDetails/autofixAgent.tsx

@@ -20,8 +20,8 @@ import {t, tct} from 'sentry/locale';
 import type {Project} from 'sentry/types/project';
 import {useQuery} from 'sentry/utils/queryClient';
 import {useOrganization} from 'sentry/utils/useOrganization';
+import {useFetchAgentOptions} from 'sentry/views/settings/seer/overview/utils/seerPreferredAgent';
 import {
-  useAgentOptions,
   useMutateSelectedAgent,
   useSelectedAgentFromProjectSettings,
 } from 'sentry/views/settings/seer/seerAgentHooks';
@@ -39,14 +39,11 @@ function AgentSpecificFields({
   integration,
   ...props
 }: Props & {
-  integration: 'seer' | 'none' | CodingAgentIntegration;
+  integration: 'seer' | CodingAgentIntegration;
 }) {
   if (integration === 'seer') {
     return <SeerAgentSettings {...props} />;
   }
-  if (integration === 'none') {
-    return null;
-  }
   if (integration.provider === 'cursor' || integration.provider === 'claude_code') {
     return <CodingAgentSettings integration={integration} {...props} />;
   }
@@ -63,10 +60,9 @@ export function AutofixAgent({canWrite, preference, project}: Props) {
     ...organizationIntegrationsCodingAgents(organization),
     select: data => data.json.integrations ?? [],
   });
-  const options = useAgentOptions({integrations: integrations ?? []});
+  const options = useFetchAgentOptions({organization});
   const selected = useSelectedAgentFromProjectSettings({
     preference,
-    project,
     integrations: integrations ?? [],
   });
   const mutateSelectedAgent = useMutateSelectedAgent({project});
@@ -101,37 +97,29 @@ export function AutofixAgent({canWrite, preference, project}: Props) {
                   ),
                 }
               )}
-              options={options}
+              options={options.data ?? []}
               value={selected}
-              onChange={(integration: 'seer' | 'none' | CodingAgentIntegration) => {
+              onChange={(integration: 'seer' | CodingAgentIntegration) => {
                 mutateSelectedAgent(integration, {
                   onSuccess: () =>
                     addSuccessMessage(
-                      integration === 'none'
-                        ? t('Removed coding agent')
-                        : tct('Started using [name] as coding agent', {
-                            name: (
-                              <strong>
-                                {integration === 'seer'
-                                  ? t('Seer Agent')
-                                  : integration.name}
-                              </strong>
-                            ),
-                          })
+                      tct('Started using [name] as coding agent', {
+                        name: (
+                          <strong>
+                            {integration === 'seer' ? t('Seer Agent') : integration.name}
+                          </strong>
+                        ),
+                      })
                     ),
                   onError: () =>
                     addErrorMessage(
-                      integration === 'none'
-                        ? t('Failed to update coding agent')
-                        : tct('Failed to set [name] as coding agent', {
-                            name: (
-                              <strong>
-                                {integration === 'seer'
-                                  ? t('Seer Agent')
-                                  : integration.name}
-                              </strong>
-                            ),
-                          })
+                      tct('Failed to set [name] as coding agent', {
+                        name: (
+                          <strong>
+                            {integration === 'seer' ? t('Seer Agent') : integration.name}
+                          </strong>
+                        ),
+                      })
                     ),
                 });
               }}