github: temporarily route detain/sugarcraft + sugarcraft/* to notifications

User-requested temporary routing change — moves both ends of the
sugarcraft upstream→downstream pair from int-dev-announce to
notifications. Will be reverted shortly.

Both sides moved together so the bot's action-triggered push
attribution (notif:wfactive lookup → re-keyed dedup_key) keeps
nesting the sugarcraft/* sync pushes under the parent
detain/sugarcraft commit. The bot's trackable storage is keyed by
room, so splitting the pair across rooms would break the nesting.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: detain

web/github.php

@@ -168,8 +168,13 @@ function addUniversalRepoFields(array $whitelist): array
 
 function pickRoom(string $repo): string
 {
-    if (strpos($repo, 'sugarcraft/') === 0
-        || in_array($repo, ['detain/CandyCore', 'detain/scoop-emulators', 'detain/detain', 'detain/sugarcraft', 'detain/watchable', 'detain/php-dup-finder'], true)) {
+    // TEMP: detain/sugarcraft + sugarcraft/* routed to `notifications` for
+    // now (was `int-dev-announce`). Move back when the experiment is done.
+    // Both sides of the upstream→downstream pair are kept in the same room
+    // so the bot's action-triggered push attribution can still nest the
+    // sugarcraft/* sync pushes under the detain/sugarcraft parent commit
+    // (trackable storage is keyed by room).
+    if (in_array($repo, ['detain/CandyCore', 'detain/scoop-emulators', 'detain/detain', 'detain/watchable', 'detain/php-dup-finder'], true)) {
         return 'int-dev-announce';
     }
     return 'notifications';