fix(slack): complete setup remediation

Author: waleedlatif1

packages/sim-setup/src/checks.test.ts

@@ -42,7 +42,7 @@ describe('setup coherence checks', () => {
       status: 'fail',
       message:
         'SLACK_EXTENDED_SCOPES is on but SLACK_SIGNING_SECRET is not set — native Slack triggers will fail at runtime',
-      fix: 'set SLACK_SIGNING_SECRET or remove SLACK_EXTENDED_SCOPES',
+      fix: 'set SLACK_SIGNING_SECRET or remove SLACK_EXTENDED_SCOPES and NEXT_PUBLIC_SLACK_EXTENDED_SCOPES',
     })
   })
 

packages/sim-setup/src/checks.ts

@@ -461,7 +461,12 @@ function checkCoherence(ctx: CheckContext): Finding[] {
     })
   }
 
-  const featureRules: Array<{ flag: string; needs: string[]; label: string }> = [
+  const featureRules: Array<{
+    flag: string
+    needs: string[]
+    label: string
+    disableFields?: string[]
+  }> = [
     {
       flag: 'BILLING_ENABLED',
       needs: ['STRIPE_SECRET_KEY'],
@@ -471,6 +476,7 @@ function checkCoherence(ctx: CheckContext): Finding[] {
       flag: 'SLACK_EXTENDED_SCOPES',
       needs: ['SLACK_SIGNING_SECRET'],
       label: 'native Slack triggers',
+      disableFields: ['SLACK_EXTENDED_SCOPES', 'NEXT_PUBLIC_SLACK_EXTENDED_SCOPES'],
     },
     { flag: 'SSO_ENABLED', needs: ['SSO_ISSUER'], label: 'SSO' },
   ]
@@ -482,7 +488,7 @@ function checkCoherence(ctx: CheckContext): Finding[] {
         group: 'coherence',
         status: 'fail',
         message: `${rule.flag} is on but ${missing.join(', ')} is not set — ${rule.label} will fail at runtime`,
-        fix: `set ${missing.join(', ')} or remove ${rule.flag}`,
+        fix: `set ${missing.join(', ')} or remove ${(rule.disableFields ?? [rule.flag]).join(' and ')}`,
       })
     }
   }