@songsid/agend 2.0.11-beta.15 → 2.0.11-beta.17

package/dist/fleet-manager.d.ts

@@ -185,7 +185,7 @@ export declare class FleetManager implements FleetContext, LifecycleContext, Arc
     saveFleetConfig(): void;
     removeInstance(name: string): Promise<void>;
     startStatuslineWatcher(name: string): void;
-    reactMessageStatus(chatId: string, messageId: string, emoji: string): void;
+    reactMessageStatus(instanceName: string, chatId: string, messageId: string, emoji: string): void;
     private static FAILOVER_TRIGGER_PCT;
     private static FAILOVER_RECOVER_PCT;
     checkModelFailover(name: string, fiveHourPct: number): void;

package/dist/fleet-manager.js

@@ -516,8 +516,9 @@ export class FleetManager {
         // Rotate classic channel chat logs daily (piggyback on daily summary timer)
         this.classicChannels?.rotateLogs();
         this.rotateInboxes();
-        // Auto-create general instance(s) — one per adapter that lacks a general
+        // Auto-create/adopt a general dispatcher — ONLY for the primary adapter.
         const channelConfigs = fleet.channels ?? (fleet.channel ? [fleet.channel] : []);
+        const primaryAdapterId = channelConfigs[0] ? (channelConfigs[0].id ?? channelConfigs[0].type) : undefined;
         const generalInstances = Object.entries(fleet.instances).filter(([, inst]) => inst.general_topic === true);
         let generalsCreated = false;
         // Collect unbound generals (no channel_id set) for auto-assignment
@@ -526,6 +527,14 @@ export class FleetManager {
         const needsGeneral = [];
         for (const ch of channelConfigs) {
             const adapterId = ch.id ?? ch.type;
+            // Only the primary adapter gets an auto-general. Secondary (persona) bots
+            // answer for their explicitly-bound instances only — they don't need or
+            // auto-claim a general dispatcher, and must never adopt the primary's
+            // unbound general. A general a user manually bound to a secondary
+            // (channel_id: <persona>) is left untouched — the auto logic just won't
+            // create or reassign bindings for non-primary adapters.
+            if (adapterId !== primaryAdapterId)
+                continue;
             // Check if any general is explicitly bound to this adapter
             if (generalInstances.some(([, inst]) => inst.channel_id === adapterId))
                 continue;
@@ -2609,15 +2618,17 @@ export class FleetManager {
     startStatuslineWatcher(name) {
         this.statuslineWatcher.watch(name);
     }
-    reactMessageStatus(chatId, messageId, emoji) {
-        // Find the adapter that owns this chatId (check all adapters, not just primary)
-        for (const [, w] of this.worlds) {
-            if (w.type === "discord") {
-                w.react(chatId, messageId, emoji)
-                    .catch(e => this.logger.debug({ err: e.message }, "Message status react failed"));
-                return;
-            }
-        }
+    reactMessageStatus(instanceName, chatId, messageId, emoji) {
+        // React via the adapter BOUND to this instance — NOT the first discord world.
+        // Otherwise, in a same-channel/same-guild multi-bot setup, the inbound 👀
+        // (bound bot) and the delivery/confirm reactions (some other bot) come from
+        // different bots, leaving a duplicate 👀 that never turns into ✅.
+        const adapter = this.getAdapterForInstance(instanceName) ?? this.adapter;
+        // Status reactions are Discord-only (TG/others use the inbound react path).
+        if (!adapter || adapter.type !== "discord")
+            return;
+        adapter.react(chatId, messageId, emoji)
+            .catch(e => this.logger.debug({ err: e.message }, "Message status react failed"));
     }
     // ── Model failover ──────────────────────────────────────────────────────
     static FAILOVER_TRIGGER_PCT = 90;