Page MenuHomePhabricator

Don't subscribe bots implicitly when they act on objects, or when they are mentioned
ClosedPublic

Authored by epriestley on Mar 13 2019, 6:48 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 1, 7:41 AM
Unknown Object (File)
Sat, Nov 30, 12:04 PM
Unknown Object (File)
Thu, Nov 28, 12:31 PM
Unknown Object (File)
Thu, Nov 28, 1:21 AM
Unknown Object (File)
Tue, Nov 26, 10:20 PM
Unknown Object (File)
Tue, Nov 26, 3:44 PM
Unknown Object (File)
Mon, Nov 25, 8:23 PM
Unknown Object (File)
Sat, Nov 23, 7:26 AM
Subscribers
None

Details

Summary

See PHI1098. When users comment on objects, they are automatically subscribed. And when @alice mentions @bailey on a task, that usually subscribes @bailey.

These rules make less sense if the user is a bot. There's generally no reason for a bot to automatically subscribe to objects it acts on (it's not going to read email and follow up later), and it can always subscribe itself pretty easily if it wants (since everything is *.edit now and supports subscribe transactions).

Also, don't subscribe bots when they're mentioned for similar reasons. If users really want to subscribe bots, they can do so explicitly.

These rules seem slightly like "bad implicit magic" since it's not immediately obvious why @abc subscribes that user but @xyz may not, but some of these rules are fairly complicated already (e.g., @xyz doesn't subscribe them if they unsubscribed or are implicitly subscribed) and this feels like it gets the right/desired result almost-always.

Test Plan

On a fresh task:

  • Mentioned a bot in a comment with @bot.
    • Before patch: bot got CC'd.
    • After patch: no CC.
  • Called maniphest.edit via the API to add a comment as a bot.
    • Before patch: bot got CC'd.
    • After patch: no CC.

Diff Detail

Repository
rP Phabricator
Branch
sub1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 22282
Build 30471: Run Core Tests
Build 30470: arc lint + arc unit