chore(anomaly detection): Remove unused code (#89998)

An attempt at the FE counterpart to
https://github.com/getsentry/sentry/pull/89406/files to remove code
around a feature flag that we never ended up using.

Author: ceorourke

static/app/views/alerts/rules/metric/details/index.tsx

@@ -24,10 +24,9 @@ import {
   AlertRuleComparisonType,
   TimePeriod,
 } from 'sentry/views/alerts/rules/metric/types';
-import type {Anomaly, Incident} from 'sentry/views/alerts/types';
+import type {Incident} from 'sentry/views/alerts/types';
 import {
   fetchAlertRule,
-  fetchAnomaliesForRule,
   fetchIncident,
   fetchIncidentsForRule,
 } from 'sentry/views/alerts/utils/apiCalls';
@@ -51,7 +50,6 @@ interface State {
   hasError: boolean;
   isLoading: boolean;
   selectedIncident: Incident | null;
-  anomalies?: Anomaly[];
   incidents?: Incident[];
   rule?: MetricRule;
   warning?: string;
@@ -217,24 +215,16 @@ class MetricAlertDetails extends Component<Props, State> {
     const timePeriod = this.getTimePeriod(selectedIncident);
     const {start, end} = timePeriod;
     try {
-      const [incidents, rule, anomalies] = await Promise.all([
+      const [incidents, rule] = await Promise.all([
         fetchIncidentsForRule(organization.slug, ruleId, start, end),
         rulePromise,
-        organization.features.includes('anomaly-detection-alerts-charts')
-          ? fetchAnomaliesForRule(organization.slug, ruleId, start, end)
-          : undefined, // NOTE: there's no way for us to determine the alert rule detection type here.
-        // proxy API will need to determine whether to fetch anomalies or not
       ]);
-      // NOTE: 'anomaly-detection-alerts-charts' flag does not exist
-      // Flag can be enabled IF we want to enable marked lines/areas for anomalies in the future
-      // For now, we defer to incident lines as indicators for anomalies
       let warning: any;
       if (rule.status === ALERT_RULE_STATUS.NOT_ENOUGH_DATA) {
         warning =
           'Insufficient data for anomaly detection. This feature will enable automatically when more data is available.';
       }
       this.setState({
-        anomalies,
         incidents,
         rule,
         warning,
@@ -264,7 +254,7 @@ class MetricAlertDetails extends Component<Props, State> {
   }
 
   render() {
-    const {rule, incidents, hasError, selectedIncident, anomalies, warning} = this.state;
+    const {rule, incidents, hasError, selectedIncident, warning} = this.state;
     const {organization, projects, loadingProjects} = this.props;
     const timePeriod = this.getTimePeriod(selectedIncident);
 
@@ -302,7 +292,6 @@ class MetricAlertDetails extends Component<Props, State> {
           rule={rule}
           project={project}
           incidents={incidents}
-          anomalies={anomalies}
           timePeriod={timePeriod}
           selectedIncident={selectedIncident}
         />

static/app/views/alerts/utils/apiCalls.tsx

@@ -1,6 +1,6 @@
 import {Client} from 'sentry/api';
 import type {MetricRule} from 'sentry/views/alerts/rules/metric/types';
-import type {Anomaly, Incident} from 'sentry/views/alerts/types';
+import type {Incident} from 'sentry/views/alerts/types';
 
 // Use this api for requests that are getting cancelled
 const uncancellableApi = new Client();
@@ -41,20 +41,3 @@ export function fetchIncident(
 ): Promise<Incident> {
   return api.requestPromise(`/organizations/${orgId}/incidents/${alertId}/`);
 }
-
-export function fetchAnomaliesForRule(
-  orgId: string,
-  ruleId: string,
-  start: string,
-  end: string
-): Promise<Anomaly[]> {
-  return uncancellableApi.requestPromise(
-    `/organizations/${orgId}/alert-rules/${ruleId}/anomalies/`,
-    {
-      query: {
-        start,
-        end,
-      },
-    }
-  );
-}

static/app/views/issueDetails/metricIssues/metricIssueChart.tsx

@@ -12,8 +12,7 @@ import useOrganization from 'sentry/utils/useOrganization';
 import type {TimePeriodType} from 'sentry/views/alerts/rules/metric/details/constants';
 import type {MetricRule} from 'sentry/views/alerts/rules/metric/types';
 import {useMetricRule} from 'sentry/views/alerts/rules/metric/utils/useMetricRule';
-import type {Anomaly, Incident} from 'sentry/views/alerts/types';
-import {useMetricAnomalies} from 'sentry/views/issueDetails/metricIssues/useMetricAnomalies';
+import type {Incident} from 'sentry/views/alerts/types';
 import {useMetricIncidents} from 'sentry/views/issueDetails/metricIssues/useMetricIncidents';
 import {useMetricStatsChart} from 'sentry/views/issueDetails/metricIssues/useMetricStatsChart';
 import {
@@ -49,20 +48,7 @@ export function MetricIssueChart({group, project}: MetricIssueChartProps) {
       enabled: !!ruleId,
     }
   );
-  const {data: anomalies = [], isLoading: isAnomaliesLoading} = useMetricAnomalies(
-    {
-      orgSlug: organization.slug,
-      ruleId: ruleId ?? '',
-      query: {
-        start: timePeriod.start,
-        end: timePeriod.end,
-      },
-    },
-    {
-      enabled:
-        !!ruleId && organization.features.includes('anomaly-detection-alerts-charts'),
-    }
-  );
+
   const {data: incidents = [], isLoading: isIncidentsLoading} = useMetricIncidents(
     {
       orgSlug: organization.slug,
@@ -78,7 +64,7 @@ export function MetricIssueChart({group, project}: MetricIssueChartProps) {
     }
   );
 
-  if (isRuleLoading || isAnomaliesLoading || isIncidentsLoading || !rule) {
+  if (isRuleLoading || isIncidentsLoading || !rule) {
     return (
       <MetricChartSection>
         <MetricIssuePlaceholder type="loading" />
@@ -96,7 +82,6 @@ export function MetricIssueChart({group, project}: MetricIssueChartProps) {
         rule={rule}
         timePeriod={timePeriod}
         project={project}
-        anomalies={anomalies}
         incidents={incidents}
       />
     </MetricChartSection>
@@ -110,21 +95,18 @@ function MetricIssueChartContent({
   rule,
   timePeriod,
   project,
-  anomalies,
   incidents,
 }: {
   project: Project;
   rule: MetricRule;
   timePeriod: TimePeriodType;
-  anomalies?: Anomaly[];
   incidents?: Incident[];
 }) {
   const chartZoomProps = useChartZoom({saveOnZoom: true});
   const {chartProps, queryResult} = useMetricStatsChart({
     project,
     rule,
     timePeriod,
-    anomalies,
     incidents,
     referrer: 'metric-issue-chart',
   });