feat(slack): Nudge per-workspace to update outdated Slack app installs

Detect an outdated Slack app installation by the absence of the
app_mentions:read scope, which is only granted to current installs.
Previously the check keyed on the older 'commands' scope, which no
longer distinguishes outdated installs.

On the integration configurations tab, compute the upgrade prompt
per-workspace so that an outdated Slack workspace no longer flags a
sibling workspace that is already current. Also refresh the nudge copy
to reflect the new capability (tag Sentry to triage and debug issues)
and render the directory 'Update Now' button as a primary action.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Author: alexsohn1126

static/app/utils/integrationUtil.tsx

@@ -282,18 +282,27 @@ export function getCodeOwnerIcon(
       return <IconSentry size={iconSize} />;
   }
 }
+const isIntegrationUpToDate = (integration: Integration): boolean =>
+  integration.provider.key !== 'slack' ||
+  (integration.scopes?.includes('app_mentions:read') ?? false);
+
 const isSlackIntegrationUpToDate = (integrations: Integration[]): boolean => {
-  return integrations.every(
-    integration =>
-      integration.provider.key !== 'slack' || integration.scopes?.includes('commands')
-  );
+  return integrations.every(isIntegrationUpToDate);
 };
 
+/**
+ * Whether a single integration installation is running an outdated app and
+ * should surface an "Update Now" prompt. Checked per-workspace so that, e.g.,
+ * an outdated Slack workspace doesn't flag a sibling workspace that is current.
+ */
+export const integrationRequiresUpgrade = (integration: Integration): boolean =>
+  !isIntegrationUpToDate(integration);
+
 export const getAlertText = (integrations?: Integration[]): string | undefined => {
   return isSlackIntegrationUpToDate(integrations || [])
     ? undefined
     : t(
-        'Update to the latest version of our Slack app to get access to personal and team notifications.'
+        'Update to the latest version of our Slack app to tag Sentry and ask it to triage and debug issues'
       );
 };
 

static/app/views/settings/organizationIntegrations/integrationDetailedView.spec.tsx

@@ -165,6 +165,53 @@ describe('IntegrationDetailedView', () => {
     expect(screen.getByRole('button', {name: 'Configure'})).toBeEnabled();
   });
 
+  it('shows Update Now only for the outdated Slack workspace', async () => {
+    const slackProvider = {
+      aspects: {},
+      canAdd: true,
+      canDisable: false,
+      features: ['alert-rule', 'chat-unfurl'],
+      key: 'slack',
+      name: 'Slack',
+      slug: 'slack',
+    };
+    MockApiClient.addMockResponse({
+      url: `/organizations/${organization.slug}/integrations/`,
+      match: [MockApiClient.matchQuery({provider_key: 'slack', includeConfig: 0})],
+      body: [
+        {
+          id: '10',
+          name: 'Outdated Workspace',
+          domainName: 'outdated.slack.com',
+          provider: slackProvider,
+          status: 'active',
+          // Missing app_mentions:read -> outdated install.
+          scopes: ['commands', 'chat:write'],
+        },
+        {
+          id: '11',
+          name: 'Current Workspace',
+          domainName: 'current.slack.com',
+          provider: slackProvider,
+          status: 'active',
+          scopes: ['commands', 'chat:write', 'app_mentions:read'],
+        },
+      ],
+    });
+
+    render(<IntegrationDetailedView />, {
+      initialRouterConfig: createRouterConfig('slack', {tab: 'configurations'}),
+      organization,
+    });
+
+    expect(await screen.findByText('Outdated Workspace')).toBeInTheDocument();
+    expect(screen.getByText('Current Workspace')).toBeInTheDocument();
+
+    // Only the outdated workspace surfaces an Update Now button, not every row.
+    expect(screen.getByTestId('integration-upgrade-button')).toBeInTheDocument();
+    expect(screen.getAllByTestId('integration-upgrade-button')).toHaveLength(1);
+  });
+
   it('disables configure for members without access', async () => {
     const lowerAccessOrg = OrganizationFixture({access: ['org:read']});
     render(<IntegrationDetailedView />, {

static/app/views/settings/organizationIntegrations/integrationDetailedView.tsx

@@ -25,6 +25,7 @@ import {getApiUrl} from 'sentry/utils/api/getApiUrl';
 import {
   getAlertText,
   getIntegrationStatus,
+  integrationRequiresUpgrade,
   isScmProvider,
   trackIntegrationAnalytics,
 } from 'sentry/utils/integrationUtil';
@@ -406,7 +407,7 @@ export default function IntegrationDetailedView() {
                     organization,
                   });
                 }}
-                requiresUpgrade={!!alertText}
+                requiresUpgrade={integrationRequiresUpgrade(integration)}
               />
             </PanelItem>
           ))}

static/app/views/settings/organizationIntegrations/integrationRow.tsx

@@ -118,8 +118,9 @@ export function IntegrationRow(props: Props) {
             <Alert
               variant="warning"
               trailingItems={
-                <ResolveNowButton
+                <LinkButton
                   href={`${baseUrl}?tab=configurations&referrer=directory_resolve_now`}
+                  variant="primary"
                   size="xs"
                   onClick={() =>
                     trackIntegrationAnalytics('integrations.resolve_now_clicked', {
@@ -130,7 +131,7 @@ export function IntegrationRow(props: Props) {
                   }
                 >
                   {resolveText || t('Resolve Now')}
-                </ResolveNowButton>
+                </LinkButton>
               }
             >
               {alertText}
@@ -195,8 +196,3 @@ const PublishStatus = styled(({status, ...props}: PublishStatusProps) => (
     font-weight: ${p => p.theme.font.weight.sans.regular};
   }
 `;
-
-const ResolveNowButton = styled(LinkButton)`
-  color: ${p => p.theme.tokens.content.secondary};
-  float: right;
-`;