This appears to breaks ConnectFarcasterAlertHandler for non-DC use cases. You chose to reuse an existing alert handler instead of introducing a new one in D15148... now you seem to have broken the existing use case?

D15388

This change modifies this condition from always off to on for staff users. So this diff shouldn't break anything.

The tradeoff between new and existing alerts was described in D15148. Probably the most relevant part from the summary is

Between the supportsFarcasterDCs flag is enabled and releasing this diff, users won't see the modal.