@imenam/mcp-messenger 1.0.9 → 1.0.11

package/dist/src/core/store.js

@@ -23,12 +23,26 @@ const THREAD_DESCRIPTION_MAX = 2000;
  * touchée. Une boîte qui ne fait que recevoir reste vivante, alors que `last_seen_at`
  * seul la ferait paraître abandonnée.
  */
-const LAST_ACTIVITY = `MAX(
-  COALESCE(a.last_seen_at, ''),
-  COALESCE((SELECT MAX(d.created_at) FROM deliveries d WHERE d.agent_id = a.id), '')
+const LAST_ACTIVITY = `MAX(
+  COALESCE(a.last_seen_at, ''),
+  COALESCE((SELECT MAX(d.created_at) FROM deliveries d WHERE d.agent_id = a.id), '')
 )`;
 /** Colonnes du fil jointes à tout message : le titre voyage avec chaque livraison. */
 const THREAD_JOIN = "LEFT JOIN threads t ON t.id = m.thread_id";
+/**
+ * Colonnes de livraison pour les vues de supervision.
+ *
+ * La console ne lit aucune boîte en particulier : elle n'a donc pas de copie « à elle »
+ * d'où tirer l'état du message. C'est la copie du destinataire qui fait foi — « ce message
+ * a-t-il été lu par celui à qui il était adressé ? » est la seule question que la console
+ * pose. Un message n'ayant qu'une copie d'expéditeur (destinataire jamais enregistré) n'a
+ * pas de ligne ici : `read_at` reste alors nul, donc non lu.
+ */
+const ADMIN_DELIVERY = `'to' AS role,
+                (SELECT dr.read_at FROM deliveries dr WHERE dr.message_id = m.id AND dr.role != 'sender'
+                  ORDER BY dr.agent_id LIMIT 1) AS read_at,
+                COALESCE((SELECT dr.folder FROM deliveries dr WHERE dr.message_id = m.id AND dr.role != 'sender'
+                  ORDER BY dr.agent_id LIMIT 1), 'inbox') AS folder`;
 /**
  * Boîte aux lettres persistée en SQLite.
  *
@@ -50,84 +64,84 @@ export class MessageStore extends EventEmitter {
         this.migrate();
     }
     migrate() {
-        this.db.exec(`
-      CREATE TABLE IF NOT EXISTS agents (
-        id                TEXT PRIMARY KEY,
-        agent_name        TEXT,
-        agent_description TEXT,
-        display_name      TEXT,
-        description       TEXT,
-        created_at   TEXT NOT NULL,
-        last_seen_at TEXT,
-        registered   INTEGER NOT NULL DEFAULT 0
-      );
-
-      CREATE TABLE IF NOT EXISTS messages (
-        id          TEXT PRIMARY KEY,
-        thread_id   TEXT NOT NULL,
-        sender_id   TEXT NOT NULL,
-        subject     TEXT NOT NULL,
-        body        TEXT NOT NULL,
-        priority    TEXT NOT NULL DEFAULT 'normal',
-        in_reply_to TEXT,
-        created_at  TEXT NOT NULL
-      );
-
-      CREATE TABLE IF NOT EXISTS deliveries (
-        id         TEXT PRIMARY KEY,
-        message_id TEXT NOT NULL REFERENCES messages(id) ON DELETE CASCADE,
-        agent_id   TEXT NOT NULL,
-        role       TEXT NOT NULL,
-        folder     TEXT NOT NULL,
-        read_at    TEXT,
-        starred    INTEGER NOT NULL DEFAULT 0,
-        created_at TEXT NOT NULL,
-        UNIQUE (message_id, agent_id, role)
-      );
-
-      CREATE TABLE IF NOT EXISTS threads (
-        id          TEXT PRIMARY KEY,
-        title       TEXT,
-        description TEXT,
-        named_by    TEXT,
-        created_at  TEXT NOT NULL,
-        updated_at  TEXT
-      );
-
-      CREATE TABLE IF NOT EXISTS folders (
-        agent_id   TEXT NOT NULL,
-        name       TEXT NOT NULL,
-        created_at TEXT NOT NULL,
-        PRIMARY KEY (agent_id, name)
-      );
-
-      CREATE INDEX IF NOT EXISTS idx_deliveries_mailbox ON deliveries (agent_id, folder, created_at DESC);
-      CREATE INDEX IF NOT EXISTS idx_deliveries_message ON deliveries (message_id);
-      CREATE INDEX IF NOT EXISTS idx_messages_thread ON messages (thread_id, created_at);
+        this.db.exec(`
+      CREATE TABLE IF NOT EXISTS agents (
+        id                TEXT PRIMARY KEY,
+        agent_name        TEXT,
+        agent_description TEXT,
+        display_name      TEXT,
+        description       TEXT,
+        created_at   TEXT NOT NULL,
+        last_seen_at TEXT,
+        registered   INTEGER NOT NULL DEFAULT 0
+      );
+
+      CREATE TABLE IF NOT EXISTS messages (
+        id          TEXT PRIMARY KEY,
+        thread_id   TEXT NOT NULL,
+        sender_id   TEXT NOT NULL,
+        subject     TEXT NOT NULL,
+        body        TEXT NOT NULL,
+        priority    TEXT NOT NULL DEFAULT 'normal',
+        in_reply_to TEXT,
+        created_at  TEXT NOT NULL
+      );
+
+      CREATE TABLE IF NOT EXISTS deliveries (
+        id         TEXT PRIMARY KEY,
+        message_id TEXT NOT NULL REFERENCES messages(id) ON DELETE CASCADE,
+        agent_id   TEXT NOT NULL,
+        role       TEXT NOT NULL,
+        folder     TEXT NOT NULL,
+        read_at    TEXT,
+        starred    INTEGER NOT NULL DEFAULT 0,
+        created_at TEXT NOT NULL,
+        UNIQUE (message_id, agent_id, role)
+      );
+
+      CREATE TABLE IF NOT EXISTS threads (
+        id          TEXT PRIMARY KEY,
+        title       TEXT,
+        description TEXT,
+        named_by    TEXT,
+        created_at  TEXT NOT NULL,
+        updated_at  TEXT
+      );
+
+      CREATE TABLE IF NOT EXISTS folders (
+        agent_id   TEXT NOT NULL,
+        name       TEXT NOT NULL,
+        created_at TEXT NOT NULL,
+        PRIMARY KEY (agent_id, name)
+      );
+
+      CREATE INDEX IF NOT EXISTS idx_deliveries_mailbox ON deliveries (agent_id, folder, created_at DESC);
+      CREATE INDEX IF NOT EXISTS idx_deliveries_message ON deliveries (message_id);
+      CREATE INDEX IF NOT EXISTS idx_messages_thread ON messages (thread_id, created_at);
     `);
         // Bases antérieures à la séparation agent / boîte : `display_name` y tenait les deux
         // rôles, et venait presque toujours de la configuration (le nom du projet). On le
         // déplace vers l'identité de l'agent ; la boîte repart sans nom, à poser par l'agent.
         const agentColumns = this.db.prepare("PRAGMA table_info(agents)").all().map((c) => c.name);
         if (!agentColumns.includes("agent_name")) {
-            this.db.exec(`
-        ALTER TABLE agents ADD COLUMN agent_name TEXT;
-        ALTER TABLE agents ADD COLUMN agent_description TEXT;
-        UPDATE agents
-           SET agent_name = display_name,
-               agent_description = description,
-               display_name = NULL,
-               description = NULL;
+            this.db.exec(`
+        ALTER TABLE agents ADD COLUMN agent_name TEXT;
+        ALTER TABLE agents ADD COLUMN agent_description TEXT;
+        UPDATE agents
+           SET agent_name = display_name,
+               agent_description = description,
+               display_name = NULL,
+               description = NULL;
       `);
         }
         // Bases antérieures aux fils nommés : chaque fil déjà présent reçoit sa ligne, sans
         // titre. L'affichage retombe alors sur le sujet d'ouverture, comme avant.
-        this.db.exec(`
-      INSERT INTO threads (id, title, description, named_by, created_at, updated_at)
-      SELECT m.thread_id, NULL, NULL, NULL, MIN(m.created_at), NULL
-        FROM messages m
-       WHERE NOT EXISTS (SELECT 1 FROM threads t WHERE t.id = m.thread_id)
-       GROUP BY m.thread_id;
+        this.db.exec(`
+      INSERT INTO threads (id, title, description, named_by, created_at, updated_at)
+      SELECT m.thread_id, NULL, NULL, NULL, MIN(m.created_at), NULL
+        FROM messages m
+       WHERE NOT EXISTS (SELECT 1 FROM threads t WHERE t.id = m.thread_id)
+       GROUP BY m.thread_id;
     `);
     }
     close() {
@@ -144,19 +158,19 @@ export class MessageStore extends EventEmitter {
         const existing = this.db.prepare("SELECT id FROM agents WHERE id = ?").get(id);
         if (!existing) {
             this.db
-                .prepare(`INSERT INTO agents (id, agent_name, agent_description, display_name, description, created_at, last_seen_at, registered)
+                .prepare(`INSERT INTO agents (id, agent_name, agent_description, display_name, description, created_at, last_seen_at, registered)
            VALUES (?, ?, ?, ?, ?, ?, ?, ?)`)
                 .run(id, opts.agent_name ?? null, opts.agent_description ?? null, opts.display_name ?? null, opts.description ?? null, now, opts.registered ? now : null, opts.registered ? 1 : 0);
         }
         else {
             this.db
-                .prepare(`UPDATE agents
-              SET agent_name        = COALESCE(?, agent_name),
-                  agent_description = COALESCE(?, agent_description),
-                  display_name      = COALESCE(?, display_name),
-                  description       = COALESCE(?, description),
-                  registered        = MAX(registered, ?),
-                  last_seen_at      = CASE WHEN ? = 1 THEN ? ELSE last_seen_at END
+                .prepare(`UPDATE agents
+              SET agent_name        = COALESCE(?, agent_name),
+                  agent_description = COALESCE(?, agent_description),
+                  display_name      = COALESCE(?, display_name),
+                  description       = COALESCE(?, description),
+                  registered        = MAX(registered, ?),
+                  last_seen_at      = CASE WHEN ? = 1 THEN ? ELSE last_seen_at END
             WHERE id = ?`)
                 .run(opts.agent_name ?? null, opts.agent_description ?? null, opts.display_name ?? null, opts.description ?? null, opts.registered ? 1 : 0, opts.registered ? 1 : 0, now, id);
         }
@@ -178,9 +192,9 @@ export class MessageStore extends EventEmitter {
     listAgents(exclude) {
         const excluded = exclude ? normalizeAgentId(exclude) : "";
         const rows = this.db
-            .prepare(`SELECT a.*, ${LAST_ACTIVITY} AS last_activity_at
-           FROM agents a
-          WHERE a.id != ?
+            .prepare(`SELECT a.*, ${LAST_ACTIVITY} AS last_activity_at
+           FROM agents a
+          WHERE a.id != ?
           ORDER BY last_activity_at DESC, a.id`)
             .all(excluded);
         return rows.map((r) => ({
@@ -237,18 +251,18 @@ export class MessageStore extends EventEmitter {
             // Le fil naît avec son premier message ; sur une réponse, la ligne existe déjà et
             // n'est touchée que si l'expéditeur fournit un nouveau titre.
             this.db
-                .prepare(`INSERT INTO threads (id, title, description, named_by, created_at, updated_at)
-           VALUES (?, NULL, NULL, NULL, ?, NULL)
+                .prepare(`INSERT INTO threads (id, title, description, named_by, created_at, updated_at)
+           VALUES (?, NULL, NULL, NULL, ?, NULL)
            ON CONFLICT (id) DO NOTHING`)
                 .run(threadId, now);
             if (title !== null || threadDescription !== null) {
                 this.renameThreadRow(threadId, from, title, threadDescription, now);
             }
             this.db
-                .prepare(`INSERT INTO messages (id, thread_id, sender_id, subject, body, priority, in_reply_to, created_at)
+                .prepare(`INSERT INTO messages (id, thread_id, sender_id, subject, body, priority, in_reply_to, created_at)
            VALUES (?, ?, ?, ?, ?, ?, ?, ?)`)
                 .run(messageId, threadId, from, subject, input.body, priority, inReplyTo, now);
-            const insert = this.db.prepare(`INSERT INTO deliveries (id, message_id, agent_id, role, folder, read_at, starred, created_at)
+            const insert = this.db.prepare(`INSERT INTO deliveries (id, message_id, agent_id, role, folder, read_at, starred, created_at)
          VALUES (?, ?, ?, ?, ?, ?, 0, ?)`);
             for (const d of deliveries) {
                 insert.run(newId("dlv"), messageId, d.agent_id, d.role, d.folder, d.read_at, now);
@@ -293,13 +307,13 @@ export class MessageStore extends EventEmitter {
             params.push(like, like);
         }
         const rows = this.db
-            .prepare(`SELECT m.id AS message_id, m.thread_id, t.title AS thread_title, m.sender_id, m.subject, m.body,
-                m.priority, m.in_reply_to, m.created_at, d.folder, d.role, d.read_at, d.starred
-           FROM deliveries d
-           JOIN messages m ON m.id = d.message_id
-           ${THREAD_JOIN}
-          WHERE ${where.join(" AND ")}
-          ORDER BY m.created_at DESC, m.id DESC
+            .prepare(`SELECT m.id AS message_id, m.thread_id, t.title AS thread_title, m.sender_id, m.subject, m.body,
+                m.priority, m.in_reply_to, m.created_at, d.folder, d.role, d.read_at, d.starred
+           FROM deliveries d
+           JOIN messages m ON m.id = d.message_id
+           ${THREAD_JOIN}
+          WHERE ${where.join(" AND ")}
+          ORDER BY m.created_at DESC, m.id DESC
           LIMIT ? OFFSET ?`)
             .all(...params, limit, offset);
         const recipients = this.recipientsFor(rows.map((r) => r.message_id));
@@ -307,13 +321,13 @@ export class MessageStore extends EventEmitter {
     }
     getMessage(agentId, messageId, markRead = false) {
         const row = this.db
-            .prepare(`SELECT m.id AS message_id, m.thread_id, t.title AS thread_title, m.sender_id, m.subject, m.body,
-                m.priority, m.in_reply_to, m.created_at, d.folder, d.role, d.read_at, d.starred
-           FROM deliveries d
-           JOIN messages m ON m.id = d.message_id
-           ${THREAD_JOIN}
-          WHERE d.agent_id = ? AND d.message_id = ?
-          ORDER BY CASE d.role WHEN 'to' THEN 0 ELSE 1 END
+            .prepare(`SELECT m.id AS message_id, m.thread_id, t.title AS thread_title, m.sender_id, m.subject, m.body,
+                m.priority, m.in_reply_to, m.created_at, d.folder, d.role, d.read_at, d.starred
+           FROM deliveries d
+           JOIN messages m ON m.id = d.message_id
+           ${THREAD_JOIN}
+          WHERE d.agent_id = ? AND d.message_id = ?
+          ORDER BY CASE d.role WHEN 'to' THEN 0 ELSE 1 END
           LIMIT 1`)
             .get(agentId, messageId);
         if (!row)
@@ -331,14 +345,14 @@ export class MessageStore extends EventEmitter {
     /** Tous les messages d'un fil visibles par l'agent, du plus ancien au plus récent. */
     getThread(agentId, threadId) {
         const rows = this.db
-            .prepare(`SELECT m.id AS message_id, m.thread_id, t.title AS thread_title, m.sender_id, m.subject, m.body,
-                m.priority, m.in_reply_to, m.created_at,
-                MIN(d.folder) AS folder, MIN(d.role) AS role, MIN(d.read_at) AS read_at, MAX(d.starred) AS starred
-           FROM deliveries d
-           JOIN messages m ON m.id = d.message_id
-           ${THREAD_JOIN}
-          WHERE d.agent_id = ? AND m.thread_id = ?
-          GROUP BY m.id
+            .prepare(`SELECT m.id AS message_id, m.thread_id, t.title AS thread_title, m.sender_id, m.subject, m.body,
+                m.priority, m.in_reply_to, m.created_at,
+                MIN(d.folder) AS folder, MIN(d.role) AS role, MIN(d.read_at) AS read_at, MAX(d.starred) AS starred
+           FROM deliveries d
+           JOIN messages m ON m.id = d.message_id
+           ${THREAD_JOIN}
+          WHERE d.agent_id = ? AND m.thread_id = ?
+          GROUP BY m.id
           ORDER BY m.created_at ASC`)
             .all(agentId, threadId);
         if (rows.length === 0)
@@ -376,9 +390,18 @@ export class MessageStore extends EventEmitter {
         const limit = clamp(opts.limit ?? 25, 1, MAX_LIMIT);
         const offset = Math.max(0, opts.offset ?? 0);
         const having = [];
+        const where = ["d.agent_id = ?", "d.folder != 'trash'"];
         const params = [agentId];
         if (opts.unread_only)
             having.push("unread > 0");
+        // Correspondance avec une boîte donnée : le fil compte dès qu'elle y a une livraison,
+        // qu'elle ait écrit ou seulement reçu. Les paramètres du WHERE se lient avant ceux du
+        // HAVING : cette condition doit donc être poussée avant celles de la recherche.
+        if (opts.with_agent) {
+            where.push(`EXISTS (SELECT 1 FROM deliveries d2 JOIN messages m2 ON m2.id = d2.message_id
+                  WHERE m2.thread_id = m.thread_id AND d2.agent_id = ?)`);
+            params.push(normalizeAgentId(opts.with_agent));
+        }
         if (opts.search) {
             // Le titre prime, mais un fil non nommé ne doit pas devenir introuvable :
             // la recherche couvre aussi le sujet d'ouverture.
@@ -388,20 +411,20 @@ export class MessageStore extends EventEmitter {
             params.push(like, like, like);
         }
         const rows = this.db
-            .prepare(`SELECT m.thread_id,
-                t.title, t.description, t.named_by, t.created_at, t.updated_at,
-                (SELECT r.subject FROM messages r WHERE r.thread_id = m.thread_id
-                  ORDER BY r.created_at ASC, r.id ASC LIMIT 1) AS subject,
-                COUNT(DISTINCT m.id) AS messages,
-                COUNT(DISTINCT CASE WHEN d.read_at IS NULL AND d.role != 'sender' THEN m.id END) AS unread,
-                MAX(m.created_at) AS last_message_at
-           FROM deliveries d
-           JOIN messages m ON m.id = d.message_id
-           ${THREAD_JOIN}
-          WHERE d.agent_id = ? AND d.folder != 'trash'
-          GROUP BY m.thread_id
-          ${having.length ? `HAVING ${having.join(" AND ")}` : ""}
-          ORDER BY last_message_at DESC
+            .prepare(`SELECT m.thread_id,
+                t.title, t.description, t.named_by, t.created_at, t.updated_at,
+                (SELECT r.subject FROM messages r WHERE r.thread_id = m.thread_id
+                  ORDER BY r.created_at ASC, r.id ASC LIMIT 1) AS subject,
+                COUNT(DISTINCT m.id) AS messages,
+                COUNT(DISTINCT CASE WHEN d.read_at IS NULL AND d.role != 'sender' THEN m.id END) AS unread,
+                MAX(m.created_at) AS last_message_at
+           FROM deliveries d
+           JOIN messages m ON m.id = d.message_id
+           ${THREAD_JOIN}
+          WHERE ${where.join(" AND ")}
+          GROUP BY m.thread_id
+          ${having.length ? `HAVING ${having.join(" AND ")}` : ""}
+          ORDER BY last_message_at DESC
           LIMIT ? OFFSET ?`)
             .all(...params, limit, offset);
         const participants = this.participantsFor(rows.map((r) => r.thread_id));
@@ -424,18 +447,18 @@ export class MessageStore extends EventEmitter {
     /** Écriture brute du nom : un champ absent (`null`) laisse l'ancienne valeur en place. */
     renameThreadRow(threadId, agentId, title, description, now) {
         this.db
-            .prepare(`INSERT INTO threads (id, title, description, named_by, created_at, updated_at)
-         VALUES (?, ?, ?, ?, ?, ?)
-         ON CONFLICT (id) DO UPDATE SET
-           title       = COALESCE(excluded.title, threads.title),
-           description = COALESCE(excluded.description, threads.description),
-           named_by    = excluded.named_by,
+            .prepare(`INSERT INTO threads (id, title, description, named_by, created_at, updated_at)
+         VALUES (?, ?, ?, ?, ?, ?)
+         ON CONFLICT (id) DO UPDATE SET
+           title       = COALESCE(excluded.title, threads.title),
+           description = COALESCE(excluded.description, threads.description),
+           named_by    = excluded.named_by,
            updated_at  = excluded.updated_at`)
             .run(threadId, title, description, agentId, now, now);
     }
     assertThreadVisible(agentId, threadId) {
         const row = this.db
-            .prepare(`SELECT 1 FROM deliveries d JOIN messages m ON m.id = d.message_id
+            .prepare(`SELECT 1 FROM deliveries d JOIN messages m ON m.id = d.message_id
           WHERE d.agent_id = ? AND m.thread_id = ? LIMIT 1`)
             .get(agentId, threadId);
         if (!row)
@@ -448,9 +471,9 @@ export class MessageStore extends EventEmitter {
             return map;
         const placeholders = ids.map(() => "?").join(",");
         const rows = this.db
-            .prepare(`SELECT DISTINCT m.thread_id, d.agent_id
-           FROM deliveries d JOIN messages m ON m.id = d.message_id
-          WHERE m.thread_id IN (${placeholders})
+            .prepare(`SELECT DISTINCT m.thread_id, d.agent_id
+           FROM deliveries d JOIN messages m ON m.id = d.message_id
+          WHERE m.thread_id IN (${placeholders})
           ORDER BY d.agent_id`)
             .all(...ids);
         for (const row of rows) {
@@ -496,7 +519,7 @@ export class MessageStore extends EventEmitter {
     listFolders(agentId) {
         const custom = this.db.prepare("SELECT name FROM folders WHERE agent_id = ? ORDER BY name").all(agentId);
         const counts = this.db
-            .prepare(`SELECT folder, COUNT(*) AS total, SUM(CASE WHEN read_at IS NULL THEN 1 ELSE 0 END) AS unread
+            .prepare(`SELECT folder, COUNT(*) AS total, SUM(CASE WHEN read_at IS NULL THEN 1 ELSE 0 END) AS unread
            FROM deliveries WHERE agent_id = ? GROUP BY folder`)
             .all(agentId);
         const byName = new Map(counts.map((c) => [c.folder, c]));
@@ -535,7 +558,7 @@ export class MessageStore extends EventEmitter {
         if (!exists)
             throw new MessengerError(`folder not found: ${folder}`, 404);
         const moved = this.db
-            .prepare(`UPDATE deliveries SET folder = CASE role WHEN 'sender' THEN 'sent' ELSE 'inbox' END
+            .prepare(`UPDATE deliveries SET folder = CASE role WHEN 'sender' THEN 'sent' ELSE 'inbox' END
           WHERE agent_id = ? AND folder = ?`)
             .run(agentId, folder).changes;
         this.db.prepare("DELETE FROM folders WHERE agent_id = ? AND name = ?").run(agentId, folder);
@@ -579,22 +602,117 @@ export class MessageStore extends EventEmitter {
             params.push(opts.thread_id);
         }
         const rows = this.db
-            .prepare(`SELECT m.id AS message_id, m.thread_id, t.title AS thread_title, m.sender_id, m.subject, m.body,
-                m.priority, m.in_reply_to, m.created_at, 'inbox' AS folder, 'to' AS role, NULL AS read_at, 0 AS starred
-           FROM messages m
-           ${THREAD_JOIN}
-          WHERE ${where.join(" AND ")}
-          ORDER BY m.created_at DESC, m.id DESC
+            .prepare(`SELECT m.id AS message_id, m.thread_id, t.title AS thread_title, m.sender_id, m.subject, m.body,
+                m.priority, m.in_reply_to, m.created_at, 0 AS starred, ${ADMIN_DELIVERY}
+           FROM messages m
+           ${THREAD_JOIN}
+          WHERE ${where.join(" AND ")}
+          ORDER BY m.created_at DESC, m.id DESC
           LIMIT ? OFFSET ?`)
             .all(...params, limit, offset);
         const recipients = this.recipientsFor(rows.map((r) => r.message_id));
         return rows.map((row) => this.toSummary(row, recipients));
     }
+    /**
+     * Fils du serveur entier, le plus récemment actif en tête.
+     *
+     * La console est organisée autour du fil : c'est lui qui porte le sujet, les deux
+     * correspondants et l'état de lecture. Chaque ligne est autonome — dernier message,
+     * aperçu, priorité la plus haute du fil — pour que la liste s'affiche d'une requête.
+     */
+    listAllThreadsAdmin(opts = {}) {
+        const limit = clamp(opts.limit ?? 50, 1, MAX_LIMIT);
+        const offset = Math.max(0, opts.offset ?? 0);
+        const where = ["1 = 1"];
+        const having = [];
+        const params = [];
+        // Un agent « participe » au fil dès qu'il y a une livraison, qu'il ait écrit ou reçu :
+        // c'est la correspondance qui est filtrée, pas les messages qu'il a signés.
+        if (opts.agent) {
+            where.push(`EXISTS (SELECT 1 FROM deliveries d JOIN messages mp ON mp.id = d.message_id
+                  WHERE mp.thread_id = m.thread_id AND d.agent_id = ?)`);
+            params.push(opts.agent);
+        }
+        // La recherche porte sur le fil entier : son nom, sa description, et le texte de
+        // n'importe lequel de ses messages. Un fil non nommé reste ainsi trouvable.
+        if (opts.search) {
+            const like = `%${escapeLike(opts.search)}%`;
+            where.push(`(COALESCE(t.title, '') LIKE ? ESCAPE '\\' OR COALESCE(t.description, '') LIKE ? ESCAPE '\\'
+          OR EXISTS (SELECT 1 FROM messages ms WHERE ms.thread_id = m.thread_id
+                      AND (ms.subject LIKE ? ESCAPE '\\' OR ms.body LIKE ? ESCAPE '\\')))`);
+            params.push(like, like, like, like);
+        }
+        if (opts.unread_only)
+            having.push("unread > 0");
+        if (opts.priority)
+            having.push("top_priority = ?");
+        const rows = this.db
+            .prepare(`SELECT m.thread_id,
+                t.title, t.description, t.named_by, t.created_at, t.updated_at,
+                (SELECT r.subject FROM messages r WHERE r.thread_id = m.thread_id
+                  ORDER BY r.created_at ASC, r.id ASC LIMIT 1) AS subject,
+                COUNT(*) AS messages,
+                MAX(m.created_at) AS last_message_at,
+                (SELECT r.sender_id FROM messages r WHERE r.thread_id = m.thread_id
+                  ORDER BY r.created_at DESC, r.id DESC LIMIT 1) AS last_message_from,
+                (SELECT r.body FROM messages r WHERE r.thread_id = m.thread_id
+                  ORDER BY r.created_at DESC, r.id DESC LIMIT 1) AS last_body,
+                (SELECT COUNT(*) FROM deliveries d JOIN messages mu ON mu.id = d.message_id
+                  WHERE mu.thread_id = m.thread_id AND d.role != 'sender'
+                    AND d.read_at IS NULL AND d.folder != 'trash') AS unread,
+                CASE WHEN SUM(m.priority = 'high') > 0 THEN 'high'
+                     WHEN SUM(m.priority = 'normal') > 0 THEN 'normal'
+                     ELSE 'low' END AS top_priority
+           FROM messages m
+           ${THREAD_JOIN}
+          WHERE ${where.join(" AND ")}
+          GROUP BY m.thread_id
+          ${having.length ? `HAVING ${having.join(" AND ")}` : ""}
+          ORDER BY last_message_at DESC
+          LIMIT ? OFFSET ?`)
+            .all(...params, ...(opts.priority ? [opts.priority] : []), limit, offset);
+        const participants = this.participantsFor(rows.map((r) => r.thread_id));
+        return rows.map((row) => ({
+            thread_id: row.thread_id,
+            title: row.title,
+            description: row.description,
+            named_by: row.named_by,
+            created_at: row.created_at ?? row.last_message_at,
+            updated_at: row.updated_at,
+            subject: row.subject,
+            messages: row.messages,
+            unread: row.unread,
+            participants: participants.get(row.thread_id) ?? [],
+            last_message_at: row.last_message_at,
+            last_message_from: row.last_message_from,
+            last_message_preview: preview(row.last_body ?? ""),
+            priority: row.top_priority,
+        }));
+    }
+    /**
+     * Fil complet vu du serveur, du plus ancien au plus récent, corps inclus.
+     *
+     * La console affiche les échanges antérieurs sous le message ouvert : elle a besoin du
+     * texte, pas seulement de l'aperçu. Les deux sont renvoyés, l'aperçu servant aux vues
+     * qui ne déplient rien.
+     */
+    listThreadAdmin(threadId) {
+        const rows = this.db
+            .prepare(`SELECT m.id AS message_id, m.thread_id, t.title AS thread_title, m.sender_id, m.subject, m.body,
+                m.priority, m.in_reply_to, m.created_at, 0 AS starred, ${ADMIN_DELIVERY}
+           FROM messages m
+           ${THREAD_JOIN}
+          WHERE m.thread_id = ?
+          ORDER BY m.created_at ASC, m.id ASC`)
+            .all(threadId);
+        const recipients = this.recipientsFor(rows.map((r) => r.message_id));
+        return rows.map((row) => ({ ...this.toSummary(row, recipients), body: row.body }));
+    }
     /** Message complet vu du serveur, avec l'état de lecture de chaque destinataire. */
     getMessageAdmin(messageId) {
         const row = this.db
-            .prepare(`SELECT m.id AS message_id, m.thread_id, t.title AS thread_title, m.sender_id, m.subject, m.body,
-                m.priority, m.in_reply_to, m.created_at, 'inbox' AS folder, 'to' AS role, NULL AS read_at, 0 AS starred
+            .prepare(`SELECT m.id AS message_id, m.thread_id, t.title AS thread_title, m.sender_id, m.subject, m.body,
+                m.priority, m.in_reply_to, m.created_at, 0 AS starred, ${ADMIN_DELIVERY}
            FROM messages m ${THREAD_JOIN} WHERE m.id = ?`)
             .get(messageId);
         if (!row)
@@ -618,9 +736,9 @@ export class MessageStore extends EventEmitter {
         const threads = this.db.prepare("SELECT COUNT(DISTINCT thread_id) AS n FROM messages").get().n;
         const sentByAgent = new Map(this.db.prepare("SELECT sender_id, COUNT(*) AS n FROM messages GROUP BY sender_id").all().map((r) => [r.sender_id, r.n]));
         const receivedByAgent = new Map(this.db
-            .prepare(`SELECT agent_id,
-                    COUNT(*) AS received,
-                    SUM(CASE WHEN read_at IS NULL AND folder != 'trash' THEN 1 ELSE 0 END) AS unread
+            .prepare(`SELECT agent_id,
+                    COUNT(*) AS received,
+                    SUM(CASE WHEN read_at IS NULL AND folder != 'trash' THEN 1 ELSE 0 END) AS unread
                FROM deliveries WHERE role != 'sender' GROUP BY agent_id`)
             .all().map((r) => [r.agent_id, r]));
         const agents = this.listAgents().map((agent) => ({
@@ -667,7 +785,7 @@ export class MessageStore extends EventEmitter {
             return map;
         const placeholders = ids.map(() => "?").join(",");
         const rows = this.db
-            .prepare(`SELECT message_id, agent_id FROM deliveries
+            .prepare(`SELECT message_id, agent_id FROM deliveries
           WHERE message_id IN (${placeholders}) AND role = 'to'`)
             .all(...ids);
         for (const row of rows)

package/dist/src/core/types.d.ts

@@ -111,6 +111,13 @@ export interface FolderInfo {
 export interface ListThreadsOptions {
     /** Ne garder que les fils ayant au moins un message non lu. */
     unread_only?: boolean;
+    /**
+     * Adresse d'un correspondant : ne garder que les fils qu'il partage avec cette boîte.
+     *
+     * L'unité d'une correspondance est le fil, pas le message : « ce que j'ai échangé avec
+     * cette boîte » se lit ici, et non en filtrant les messages sur leur expéditeur.
+     */
+    with_agent?: string;
     /** Texte libre confronté au titre, à la description et au sujet d'ouverture. */
     search?: string;
     limit?: number;
@@ -145,3 +152,25 @@ export interface MailboxStats {
     /** Absent des réponses du store : ajouté par le serveur, qui seul voit les connexions. */
     watcher?: WatcherStatus;
 }
+/**
+ * Fil vu depuis la console de supervision, toutes boîtes confondues.
+ *
+ * La console lit le serveur, pas une boîte : elle n'a donc pas de point de vue depuis
+ * lequel compter les non-lus d'un fil. `unread` compte ici toutes les livraisons non lues
+ * du fil, quel que soit le destinataire, et `participants` liste les deux côtés.
+ *
+ * Le dernier message est résumé sur place : la liste des fils doit tenir en une requête,
+ * sans un aller-retour par fil pour afficher son aperçu.
+ */
+export interface AdminThreadSummary extends ThreadInfo {
+    /** Sujet du message d'ouverture : intitulé de repli tant que le fil n'est pas nommé. */
+    subject: string;
+    messages: number;
+    unread: number;
+    participants: string[];
+    last_message_at: string;
+    last_message_from: string;
+    last_message_preview: string;
+    /** Priorité la plus haute rencontrée dans le fil : ce qui doit sauter aux yeux. */
+    priority: Priority;
+}

package/dist/src/mcp/start.js

@@ -18,14 +18,19 @@ import { TOOLS, callTool } from "./tools.js";
  */
 const INSTRUCTIONS = `mcp-messenger gives this session a mailbox and lets it exchange messages with other agent sessions.
 
+Two words, two different things, used consistently by every tool here:
+- a **mailbox** is an address (\`box_…\`). It identifies exactly one session, and it is what you write to.
+- an **agent name** is the project a session works for. It is a label, never an address, and every session of that project carries it.
+
 Who you are, and who the others are — read this before writing to anyone:
 - A session is identified by its mailbox address, and by nothing else. \`whoami\` gives you yours.
 - The agent name (e.g. "Ai Corrector Dev") comes from the configuration and says which project the agent works on. It is NOT unique: every session started from that configuration carries it.
-- So several mailboxes in \`list_agents\` can carry the same agent name as yours. Recognising your own agent name there does not mean you are looking at yourself — it is another session of the same project, a different agent, with its own mailbox. Compare addresses, never names.
+- \`list_agents\` groups mailboxes under their agent name: one group per project, one line per session. A group holding several lines is several live sessions of that project, each with its own address — pick one of those addresses, never the group name.
+- One of those groups may carry the same agent name as yours. Recognising your own agent name there does not mean you are looking at yourself — those are other sessions of the same project, each a different agent with its own mailbox. Compare addresses, never names.
 - Your own mailbox is never listed in \`list_agents\`: every address you see there belongs to someone else.
 
 Writing:
-- One message goes to exactly one agent, and never to yourself. To reach several agents, send each one its own message.
+- One message goes to exactly one mailbox, and never to your own. To reach several correspondents, send each one its own message.
 - Never write a mailbox address in the body of a message. The server attaches the sender and the recipient to every message, and the reader sees them: an address copied by hand adds nothing and misleads as soon as it is stale or mistaken.
 - Call \`enable_notifications\` once at the start of the session and arm the watcher it hands you. Until then, no incoming message reaches you.`;
 /** Serveur MCP prêt à être branché sur n'importe quel transport (stdio en production, mémoire en test). */