fix(spans-migration): Disable duplication of transaction alerts (#96432)

As part of the transactions dataset deprecation, we want to also disable
duplication for transactions alerts. Currently the deprecation logic is
under the `performance-transaction-deprecation-alerts` flag. This is in
addition to the disabling logic in the form when editing transaction
widgets.

Closes
[ENG-4641](https://linear.app/getsentry/issue/ENG-4641/prevent-duplicating-transaction-alerts)

Author: nikkikapadia

static/app/views/alerts/list/rules/row.tsx

@@ -35,6 +35,9 @@ import {UptimeMonitorMode} from 'sentry/views/alerts/rules/uptime/types';
 import type {CombinedAlerts} from 'sentry/views/alerts/types';
 import {CombinedAlertType} from 'sentry/views/alerts/types';
 import {isIssueAlert} from 'sentry/views/alerts/utils';
+import {DEPRECATED_TRANSACTION_ALERTS} from 'sentry/views/alerts/wizard/options';
+import {getAlertTypeFromAggregateDataset} from 'sentry/views/alerts/wizard/utils';
+import {deprecateTransactionAlerts} from 'sentry/views/insights/common/utils/hasEAPAlerts';
 
 type Props = {
   hasEditAccess: boolean;
@@ -67,6 +70,21 @@ function RuleListRow({
       ? rule.project.slug
       : rule.projects[0]!;
 
+  const ruleType =
+    rule &&
+    rule.type === CombinedAlertType.METRIC &&
+    getAlertTypeFromAggregateDataset({
+      aggregate: rule.aggregate,
+      dataset: rule.dataset,
+      eventTypes: rule.eventTypes,
+      organization,
+    });
+
+  const deprecateTransactionsAlerts =
+    deprecateTransactionAlerts(organization) &&
+    ruleType &&
+    DEPRECATED_TRANSACTION_ALERTS.includes(ruleType);
+
   const editKey = {
     [CombinedAlertType.ISSUE]: 'rules',
     [CombinedAlertType.METRIC]: 'metric-rules',
@@ -107,7 +125,9 @@ function RuleListRow({
 
   const activeActions = {
     [CombinedAlertType.ISSUE]: ['edit', 'duplicate', 'delete'],
-    [CombinedAlertType.METRIC]: ['edit', 'duplicate', 'delete'],
+    [CombinedAlertType.METRIC]: deprecateTransactionsAlerts
+      ? ['edit', 'delete']
+      : ['edit', 'duplicate', 'delete'],
     [CombinedAlertType.UPTIME]: ['edit', 'delete'],
     [CombinedAlertType.CRONS]: ['edit', 'delete'],
   };

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

@@ -15,8 +15,15 @@ import type {Project} from 'sentry/types/project';
 import {makeAlertsPathname} from 'sentry/views/alerts/pathnames';
 import type {MetricRule} from 'sentry/views/alerts/rules/metric/types';
 import {getAlertRuleActionCategory} from 'sentry/views/alerts/rules/utils';
-import {AlertWizardAlertNames} from 'sentry/views/alerts/wizard/options';
+import {
+  AlertWizardAlertNames,
+  DEPRECATED_TRANSACTION_ALERTS,
+} from 'sentry/views/alerts/wizard/options';
 import {getAlertTypeFromAggregateDataset} from 'sentry/views/alerts/wizard/utils';
+import {
+  deprecateTransactionAlerts,
+  hasEAPAlerts,
+} from 'sentry/views/insights/common/utils/hasEAPAlerts';
 
 type Props = {
   hasMetricRuleDetailsError: boolean;
@@ -70,6 +77,11 @@ function DetailsHeader({
       organization,
     });
 
+  const deprecateTransactionsAlerts =
+    deprecateTransactionAlerts(organization) &&
+    ruleType &&
+    DEPRECATED_TRANSACTION_ALERTS.includes(ruleType);
+
   return (
     <Layout.Header>
       <Layout.HeaderContent>
@@ -118,7 +130,21 @@ function DetailsHeader({
               )}
             </Access>
           )}
-          <LinkButton size="sm" icon={<IconCopy />} to={duplicateLink}>
+          <LinkButton
+            size="sm"
+            icon={<IconCopy />}
+            to={duplicateLink}
+            disabled={deprecateTransactionsAlerts}
+            title={
+              deprecateTransactionsAlerts
+                ? hasEAPAlerts(organization)
+                  ? t(
+                      'Transaction alerts are being deprecated. Please create Span alerts instead.'
+                    )
+                  : t('Transaction alerts are being deprecated.')
+                : undefined
+            }
+          >
             {t('Duplicate')}
           </LinkButton>
           <LinkButton size="sm" icon={<IconEdit />} to={settingsLink}>

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

@@ -10,7 +10,7 @@ import {act, render, screen, userEvent} from 'sentry-test/reactTestingLibrary';
 import ProjectsStore from 'sentry/stores/projectsStore';
 import {trackAnalytics} from 'sentry/utils/analytics';
 import MetricAlertDetails from 'sentry/views/alerts/rules/metric/details';
-import {Dataset} from 'sentry/views/alerts/rules/metric/types';
+import {Dataset, EventTypes} from 'sentry/views/alerts/rules/metric/types';
 
 jest.mock('sentry/utils/analytics');
 
@@ -243,4 +243,51 @@ describe('MetricAlertDetails', () => {
     expect(button).toBeEnabled();
     expect(button).toHaveAttribute('href', expect.stringContaining('dataset=errors'));
   });
+
+  it('disables duplicate button if deprecation flag is on', async () => {
+    const {organization, routerProps} = initializeOrg({
+      organization: {
+        features: ['discover-basic', 'performance-transaction-deprecation-alerts'],
+      },
+    });
+    const rule = MetricRuleFixture({
+      projects: [project.slug],
+      dataset: Dataset.TRANSACTIONS,
+      eventTypes: [EventTypes.TRANSACTION],
+    });
+
+    MockApiClient.addMockResponse({
+      url: '/organizations/org-slug/events/',
+      body: {
+        data: [],
+      },
+    });
+
+    MockApiClient.addMockResponse({
+      url: `/organizations/org-slug/alert-rules/${rule.id}/`,
+      body: rule,
+    });
+    MockApiClient.addMockResponse({
+      url: `/organizations/org-slug/incidents/`,
+      body: [],
+    });
+
+    render(
+      <MetricAlertDetails
+        organization={organization}
+        {...routerProps}
+        params={{ruleId: rule.id}}
+      />,
+      {
+        organization,
+      }
+    );
+
+    expect(await screen.findByText(rule.name)).toBeInTheDocument();
+
+    expect(await screen.findByRole('button', {name: 'Duplicate'})).toHaveAttribute(
+      'aria-disabled',
+      'true'
+    );
+  });
 });