@imenam/mcp-messenger 1.0.3 → 1.0.5

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
package/README.md CHANGED
@@ -96,6 +96,7 @@ la session MCP démarre quand même et l'enregistrement se fera au premier appel
96
96
 
97
97
  L'agent A appelle `send_message` avec `to: ["coder"]`, l'agent B voit le message via
98
98
  `list_messages` puis `read_message`, et répond avec `reply_to` pour rester dans le fil.
99
+ Le fil se nomme au passage (`thread_title` sur `send_message`, ou `set_label` plus tard).
99
100
  `wait_for_message` permet d'attendre une réponse sans faire tourner une boucle d'appels : il ne
100
101
  rend la main que sur un message arrivé *après* le début de l'attente, les non-lus déjà en boîte
101
102
  ne l'interrompent pas.
@@ -143,11 +144,25 @@ n'hérite pas du bloc `env` du `.mcp.json`, donc ni de `MESSENGER_SERVER_URL` ni
143
144
  Le fichier est nommé par hachage de l'adresse, supprimé à la fin de la session, et purgé au
144
145
  bout de sept jours si la session s'est arrêtée brutalement.
145
146
 
147
+ **Savoir si c'est vraiment armé.** L'armement dépend de l'agent : rien ne peut le déclencher
148
+ à sa place, et un agent qui saute l'étape se croirait joignable sans l'être. Le serveur suit
149
+ donc les attentes tenues par un veilleur, et `whoami` le dit :
150
+
151
+ ```
152
+ Notifications: armed — incoming messages will interrupt you (watcher since 2026-09-07T13:31:00.203Z).
153
+ Notifications: NOT armed — no incoming message will reach you. Call enable_notifications.
154
+ ```
155
+
156
+ C'est une présence de connexion, pas une donnée stockée : elle vit en mémoire du serveur et
157
+ disparaît avec lui. Un `wait_for_message` ordinaire ne compte pas — il s'arrête au premier
158
+ message, un veilleur reste. Entre deux attentes longues, un délai de grâce de deux minutes
159
+ évite que le signalement clignote. `enable_notifications` s'en sert aussi : si un veilleur est
160
+ déjà en place, il le dit et n'en fait pas armer un second, qui doublerait chaque notification.
161
+
146
162
  **Ce que ça ne couvre pas.** Le veilleur n'annonce que les messages arrivés **après** son
147
163
  démarrage : `enable_notifications` signale le nombre de non-lus déjà en boîte, à traiter avec
148
- `list_messages`. Et l'armement dépend de l'agent : s'il saute l'étape `Monitor`, il ne sera
149
- jamais notifié et rien côté serveur ne le détecte. C'est pourquoi `--claude-setup-mcp` écrit
150
- un rappel dans le `CLAUDE.md` du projet.
164
+ `list_messages`. Et `--claude-setup-mcp` écrit un rappel d'armement dans le `CLAUDE.md` du
165
+ projet, pour que l'étape ne soit pas oubliée en début de session.
151
166
 
152
167
  **Sur un autre hôte que Claude Code.** `--watch` est une commande ordinaire : n'importe quel
153
168
  superviseur capable de lire les lignes d'un processus peut la brancher. Sans relais, elle lit
@@ -159,11 +174,12 @@ superviseur capable de lire les lignes d'un processus peut la brancher. Sans rel
159
174
 
160
175
  | Outil | Rôle |
161
176
  |---|---|
162
- | `whoami` | Adresse de cette session, serveur connecté, compteurs par dossier |
177
+ | `whoami` | Adresse de cette session, serveur connecté, veilleur armé ou non, compteurs par dossier |
163
178
  | `attach_to_mailbox` | Se brancher sur une autre boîte via son adresse ; sans argument, retour à la sienne |
164
- | `set_mailbox_label` | Nommer sa boîte, pour la reconnaître dans l'annuaire et la console |
165
- | `list_agents` | Annuaire : qui est joignable, avec quelle description |
166
- | `send_message` | Envoyer à un ou plusieurs agents (`to`, `cc`, `priority`, `reply_to`) |
179
+ | `set_label` | Nommer sa boîte, ou un fil (`thread_id`), en langue naturelle |
180
+ | `list_threads` | Carte des conversations : nom, participants, non-lus, dernière activité |
181
+ | `list_agents` | Annuaire : qui est joignable, du plus récemment actif au plus ancien |
182
+ | `send_message` | Envoyer à un ou plusieurs agents (`to`, `cc`, `priority`, `reply_to`, `thread_title`) |
167
183
  | `list_messages` | Lister la boîte (filtres : dossier, non lus, expéditeur, fil, texte libre) |
168
184
  | `read_message` | Lire un message en entier ; le marque lu par défaut |
169
185
  | `get_thread` | Relire toute une conversation, du plus ancien au plus récent |
@@ -175,6 +191,8 @@ superviseur capable de lire les lignes d'un processus peut la brancher. Sans rel
175
191
  | `enable_notifications` | Prépare la notification passive et rend la commande `Monitor` à armer ; ne bloque rien |
176
192
  | `wait_for_message` | Attente longue (jusqu'à 300 s) jusqu'à l'arrivée d'un message *postérieur* à l'appel |
177
193
 
194
+ **Noms lisibles** — voir « Nommer les boîtes et les fils » ci-dessous.
195
+
178
196
  **Dossiers système** — toujours présents, non supprimables : `inbox`, `sent`, `archive`,
179
197
  `trash`. Les dossiers personnalisés sont propres à chaque boîte.
180
198
 
@@ -202,9 +220,35 @@ ont deux boîtes distinctes et ne se volent pas leur courrier.
202
220
  2. Transmets-la à l'agent B, qui l'utilise dans `send_message`.
203
221
 
204
222
  Comme les adresses sont opaques, demande aux agents de nommer leur boîte
205
- (`set_mailbox_label`) : le libellé apparaît dans `list_agents` et dans la console web, ce
223
+ (`set_label`) : le libellé apparaît dans `list_agents` et dans la console web, ce
206
224
  qui permet de retrouver une adresse perdue.
207
225
 
226
+ ### Nommer les boîtes et les fils
227
+
228
+ Les identifiants (`box_…`, `thr_…`) ne disent rien de ce qui se passe. Deux noms en langue
229
+ naturelle viennent par-dessus, et un seul outil les pose :
230
+
231
+ ```
232
+ set_label(label: "Développement des tests unitaires du parser")
233
+ → nomme la boîte de la session
234
+
235
+ set_label(thread_id: "thr_…", label: "Le test échoue sur un bug applicatif")
236
+ → nomme le fil
237
+ ```
238
+
239
+ Le plus simple reste de nommer un fil au moment où on l'ouvre, via `thread_title` sur
240
+ `send_message` : l'agent qui écrit le premier message sait déjà de quoi il parle.
241
+
242
+ Un fil n'a **qu'un seul nom, partagé par tous ses participants** — ils discutent du même
243
+ sujet, ils doivent lire le même intitulé. N'importe quel participant peut le renommer, et
244
+ le dernier a le dernier mot ; le serveur retient qui (`named_by`). Un agent extérieur au
245
+ fil ne peut pas le renommer.
246
+
247
+ Tant qu'un fil n'a pas été nommé, l'affichage retombe sur le sujet de son message
248
+ d'ouverture : rien n'est jamais illisible, mais rien n'est aussi parlant qu'un nom choisi.
249
+ Ces noms apparaissent dans `list_threads`, `list_messages`, `get_thread`, les
250
+ notifications du veilleur et la console web.
251
+
208
252
  ### Reprendre le travail d'une session terminée
209
253
 
210
254
  Les boîtes survivent aux sessions. Après un redémarrage :
@@ -298,16 +342,18 @@ cet agent lorsque `MESSENGER_AGENT_TOKENS` est utilisé.
298
342
  | `GET` | `/api/agents` | Annuaire des boîtes |
299
343
  | `GET` | `/api/agents/:id` | Une boîte, ou 404 |
300
344
  | `PATCH` | `/api/agents/me` | Renommer la boîte courante |
301
- | `GET` | `/api/mailbox` | Compteurs de la boîte |
345
+ | `GET` | `/api/mailbox` | Compteurs de la boîte, et présence d'un veilleur (`watcher`) |
302
346
  | `POST` | `/api/messages` | Envoyer un message |
303
347
  | `GET` | `/api/messages` | Lister (`folder`, `unread_only`, `from`, `thread_id`, `search`, `limit`, `offset`) |
304
348
  | `GET` | `/api/messages/:id` | Lire (`?mark_read=false` pour ne pas marquer) |
305
349
  | `POST` | `/api/messages/mark` | Marquer lu / suivi |
306
350
  | `POST` | `/api/messages/move` | Déplacer vers un dossier |
307
351
  | `POST` | `/api/messages/delete` | Corbeille ou suppression définitive |
308
- | `GET` | `/api/threads/:id` | Fil complet |
352
+ | `GET` | `/api/threads` | Fils de la boîte (`unread_only`, `search`, `limit`, `offset`) |
353
+ | `GET` | `/api/threads/:id` | Fil complet, avec son intitulé |
354
+ | `PATCH` | `/api/threads/:id` | Nommer le fil (`title`, `description`) — participants uniquement |
309
355
  | `GET` `POST` `DELETE` | `/api/folders[/:name]` | Gérer les dossiers |
310
- | `GET` | `/api/wait` | Attente longue (`timeout_seconds`, `folder`) |
356
+ | `GET` | `/api/wait` | Attente longue (`timeout_seconds`, `folder`, `watcher`) |
311
357
  | `GET` | `/admin/overview` · `/admin/messages[/:id]` · `/admin/threads/:id` | Supervision, lecture seule |
312
358
 
313
359
  ---
@@ -355,6 +401,7 @@ src/
355
401
  logger.ts # journalisation sur stderr (stdout est réservé au JSON-RPC)
356
402
  core/store.ts # SQLite : agents, messages, livraisons, dossiers
357
403
  server/app.ts # API HTTP, authentification, supervision
404
+ server/watchers.ts # présence des veilleurs, en mémoire : état de connexion
358
405
  server/start.ts # démarrage, arrêt propre, purge de la corbeille
359
406
  server/ui/ # console web (HTML/CSS/JS sans dépendance)
360
407
  cli/watch.ts # mode veilleur : une ligne sur stdout par message entrant
@@ -366,7 +413,7 @@ test/ # store, config, API HTTP, outils MCP, binaire réel
366
413
 
367
414
  ### Tests
368
415
 
369
- 104 tests couvrant : le stockage et l'isolation des boîtes, la validation de configuration,
416
+ 112 tests couvrant : le stockage et l'isolation des boîtes, la validation de configuration,
370
417
  l'API HTTP et son authentification (refus systématique sans jeton, jeton partagé, jetons par
371
418
  agent, jeton de console distinct), les outils MCP de bout en bout (deux agents qui
372
419
  conversent à travers un vrai serveur), l'allocation des boîtes par session — deux processus
@@ -376,7 +423,8 @@ troisième session reprend la boîte de l'une d'elles — et le binaire publié
376
423
  La notification passive est couverte de bout en bout : le relais vers le veilleur (isolation
377
424
  entre boîtes, adresse jamais transformée en chemin, nettoyage), le format d'évènement (une
378
425
  ligne, quel que soit le sujet), et un veilleur réellement lancé comme processus séparé, sans
379
- jeton dans son environnement, qui émet sa ligne à l'arrivée d'un message.
426
+ jeton dans son environnement, qui émet sa ligne à l'arrivée d'un message — et `whoami` qui
427
+ passe de « NOT armed » à « armed » quand ce veilleur prend son poste.
380
428
 
381
429
  ---
382
430
 
@@ -54,7 +54,9 @@ export function resolveWatchConfig(mailbox, env = process.env) {
54
54
  export function formatEvent(message) {
55
55
  const subject = message.subject.replace(/\s+/g, " ").trim();
56
56
  const flag = message.priority === "high" ? "[high] " : "";
57
- return `[messenger] ${flag}${message.from} — ${subject} (${message.message_id})`;
57
+ // Le nom du fil situe le message d'un coup d'œil : c'est tout ce que la notification montre.
58
+ const thread = message.thread_title ? ` · ${message.thread_title.replace(/\s+/g, " ").trim()}` : "";
59
+ return `[messenger] ${flag}${message.from} — ${subject}${thread} (${message.message_id})`;
58
60
  }
59
61
  export async function runWatchMode(mailbox) {
60
62
  const { config, mailbox: target, source } = resolveWatchConfig(mailbox);
@@ -63,7 +65,7 @@ export async function runWatchMode(mailbox) {
63
65
  let backoff = BACKOFF_MIN_MS;
64
66
  for (;;) {
65
67
  try {
66
- const result = await client.waitForMessage(POLL_SECONDS);
68
+ const result = await client.waitForMessage(POLL_SECONDS, undefined, { watcher: true });
67
69
  backoff = BACKOFF_MIN_MS;
68
70
  if (result.timed_out || !result.message)
69
71
  continue;
@@ -1,5 +1,5 @@
1
1
  import type { AgentConfig } from "../config.js";
2
- import type { Agent, FolderInfo, ListMessagesOptions, MailboxStats, Message, MessageSummary, Priority, SendMessageResult } from "../core/types.js";
2
+ import type { Agent, AgentDirectoryEntry, FolderInfo, ListMessagesOptions, MailboxStats, Message, MessageSummary, Priority, SendMessageResult, ThreadInfo, ThreadSummary } from "../core/types.js";
3
3
  /** Échec d'un appel au serveur boîte aux lettres (réseau, authentification ou métier). */
4
4
  export declare class MessengerClientError extends Error {
5
5
  readonly status?: number | undefined;
@@ -58,7 +58,7 @@ export declare class MessengerClient {
58
58
  description?: string | null;
59
59
  }): Promise<Agent>;
60
60
  getAgent(agentId: string): Promise<Agent>;
61
- listAgents(): Promise<Agent[]>;
61
+ listAgents(): Promise<AgentDirectoryEntry[]>;
62
62
  stats(): Promise<MailboxStats>;
63
63
  sendMessage(input: {
64
64
  to: string[];
@@ -67,6 +67,8 @@ export declare class MessengerClient {
67
67
  body: string;
68
68
  priority?: Priority;
69
69
  reply_to?: string | null;
70
+ thread_title?: string | null;
71
+ thread_description?: string | null;
70
72
  }): Promise<SendMessageResult>;
71
73
  listMessages(options?: ListMessagesOptions): Promise<MessageSummary[]>;
72
74
  getMessage(messageId: string, markRead?: boolean): Promise<Message>;
@@ -76,10 +78,30 @@ export declare class MessengerClient {
76
78
  }): Promise<number>;
77
79
  moveMessages(messageIds: string[], folder: string): Promise<number>;
78
80
  deleteMessages(messageIds: string[], permanent?: boolean): Promise<number>;
79
- getThread(threadId: string): Promise<Message[]>;
81
+ getThread(threadId: string): Promise<{
82
+ thread: ThreadInfo | null;
83
+ messages: Message[];
84
+ }>;
85
+ listThreads(options?: {
86
+ unread_only?: boolean;
87
+ search?: string;
88
+ limit?: number;
89
+ }): Promise<ThreadSummary[]>;
90
+ /** Renomme un fil. Le titre est partagé : il change pour tous ses participants. */
91
+ setThreadLabel(threadId: string, label: {
92
+ title?: string | null;
93
+ description?: string | null;
94
+ }): Promise<ThreadInfo>;
80
95
  listFolders(): Promise<FolderInfo[]>;
81
96
  createFolder(name: string): Promise<FolderInfo>;
82
97
  deleteFolder(name: string): Promise<number>;
83
- waitForMessage(timeoutSeconds: number, folder?: string): Promise<WaitResult>;
98
+ /**
99
+ * `watcher` signale une attente tenue par un veilleur `--watch`, et non par un
100
+ * `wait_for_message` ponctuel : c'est ce qui permet au serveur de dire si la boîte est
101
+ * réellement surveillée.
102
+ */
103
+ waitForMessage(timeoutSeconds: number, folder?: string, options?: {
104
+ watcher?: boolean;
105
+ }): Promise<WaitResult>;
84
106
  private request;
85
107
  }
@@ -124,8 +124,16 @@ export class MessengerClient {
124
124
  return deleted;
125
125
  }
126
126
  async getThread(threadId) {
127
- const { messages } = await this.request("GET", `/api/threads/${encodeURIComponent(threadId)}`);
128
- return messages;
127
+ return this.request("GET", `/api/threads/${encodeURIComponent(threadId)}`);
128
+ }
129
+ async listThreads(options = {}) {
130
+ const { threads } = await this.request("GET", "/api/threads", { query: options });
131
+ return threads;
132
+ }
133
+ /** Renomme un fil. Le titre est partagé : il change pour tous ses participants. */
134
+ async setThreadLabel(threadId, label) {
135
+ const { thread } = await this.request("PATCH", `/api/threads/${encodeURIComponent(threadId)}`, { body: label });
136
+ return thread;
129
137
  }
130
138
  async listFolders() {
131
139
  const { folders } = await this.request("GET", "/api/folders");
@@ -139,10 +147,15 @@ export class MessengerClient {
139
147
  const { moved } = await this.request("DELETE", `/api/folders/${encodeURIComponent(name)}`);
140
148
  return moved;
141
149
  }
142
- async waitForMessage(timeoutSeconds, folder) {
150
+ /**
151
+ * `watcher` signale une attente tenue par un veilleur `--watch`, et non par un
152
+ * `wait_for_message` ponctuel : c'est ce qui permet au serveur de dire si la boîte est
153
+ * réellement surveillée.
154
+ */
155
+ async waitForMessage(timeoutSeconds, folder, options = {}) {
143
156
  // L'attente longue peut dépasser le délai HTTP par défaut : on l'étend d'une marge.
144
157
  return this.request("GET", "/api/wait", {
145
- query: { timeout_seconds: timeoutSeconds, folder },
158
+ query: { timeout_seconds: timeoutSeconds, folder, watcher: options.watcher ? "true" : undefined },
146
159
  timeoutMs: timeoutSeconds * 1000 + this.config.timeoutMs,
147
160
  });
148
161
  }
@@ -1,5 +1,5 @@
1
1
  import { EventEmitter } from "node:events";
2
- import { type Agent, type DeliveryRole, type FolderInfo, type ListMessagesOptions, type MailboxStats, type Message, type MessageSummary, type SendMessageInput, type SendMessageResult } from "./types.js";
2
+ import { type Agent, type AgentDirectoryEntry, type DeliveryRole, type FolderInfo, type ListMessagesOptions, type ListThreadsOptions, type MailboxStats, type Message, type MessageSummary, type SendMessageInput, type SendMessageResult, type ThreadInfo, type ThreadSummary } from "./types.js";
3
3
  /** Erreur métier : traduite en 4xx par la couche HTTP, en message d'outil côté MCP. */
4
4
  export declare class MessengerError extends Error {
5
5
  readonly status: number;
@@ -34,12 +34,31 @@ export declare class MessageStore extends EventEmitter<StoreEvents> {
34
34
  }): Agent;
35
35
  touchAgent(id: string): void;
36
36
  getAgent(id: string): Agent | null;
37
- listAgents(): Agent[];
37
+ /** Annuaire classé par activité décroissante : ce qui bouge est en tête, l'id ne trie rien. */
38
+ listAgents(): AgentDirectoryEntry[];
38
39
  sendMessage(input: SendMessageInput): SendMessageResult;
39
40
  listMessages(agentId: string, opts?: ListMessagesOptions): MessageSummary[];
40
41
  getMessage(agentId: string, messageId: string, markRead?: boolean): Message;
41
42
  /** Tous les messages d'un fil visibles par l'agent, du plus ancien au plus récent. */
42
43
  getThread(agentId: string, threadId: string): Message[];
44
+ getThreadInfo(threadId: string): ThreadInfo | null;
45
+ /**
46
+ * Renomme un fil. Le titre est unique et partagé : les participants parlent du même
47
+ * sujet, ils lisent donc le même nom. Le dernier appelant a le dernier mot, et
48
+ * `named_by` retient qui c'était.
49
+ *
50
+ * Seul un participant peut renommer : le fil n'est pas nommable depuis l'extérieur.
51
+ */
52
+ setThreadLabel(agentId: string, threadId: string, label: {
53
+ title?: string | null;
54
+ description?: string | null;
55
+ }): ThreadInfo;
56
+ /** Fils visibles depuis cette boîte, le plus récemment actif en tête. */
57
+ listThreads(agentId: string, opts?: ListThreadsOptions): ThreadSummary[];
58
+ /** Écriture brute du nom : un champ absent (`null`) laisse l'ancienne valeur en place. */
59
+ private renameThreadRow;
60
+ private assertThreadVisible;
61
+ private participantsFor;
43
62
  markMessages(agentId: string, messageIds: string[], state: {
44
63
  read?: boolean;
45
64
  starred?: boolean;
@@ -74,7 +93,7 @@ export declare class MessageStore extends EventEmitter<StoreEvents> {
74
93
  overview(): {
75
94
  messages: number;
76
95
  threads: number;
77
- agents: (Agent & {
96
+ agents: (AgentDirectoryEntry & {
78
97
  sent: number;
79
98
  received: number;
80
99
  unread: number;
@@ -16,6 +16,19 @@ export class MessengerError extends Error {
16
16
  const PREVIEW_LENGTH = 160;
17
17
  const MAX_LIMIT = 100;
18
18
  const FOLDER_NAME_RE = /^[a-z0-9][a-z0-9 _-]{0,39}$/;
19
+ const THREAD_TITLE_MAX = 200;
20
+ const THREAD_DESCRIPTION_MAX = 2000;
21
+ /**
22
+ * Dernière activité d'une boîte : sa dernière connexion, ou le dernier message qui l'a
23
+ * touchée. Une boîte qui ne fait que recevoir reste vivante, alors que `last_seen_at`
24
+ * seul la ferait paraître abandonnée.
25
+ */
26
+ const LAST_ACTIVITY = `MAX(
27
+ COALESCE(a.last_seen_at, ''),
28
+ COALESCE((SELECT MAX(d.created_at) FROM deliveries d WHERE d.agent_id = a.id), '')
29
+ )`;
30
+ /** Colonnes du fil jointes à tout message : le titre voyage avec chaque livraison. */
31
+ const THREAD_JOIN = "LEFT JOIN threads t ON t.id = m.thread_id";
19
32
  /**
20
33
  * Boîte aux lettres persistée en SQLite.
21
34
  *
@@ -70,6 +83,15 @@ export class MessageStore extends EventEmitter {
70
83
  UNIQUE (message_id, agent_id, role)
71
84
  );
72
85
 
86
+ CREATE TABLE IF NOT EXISTS threads (
87
+ id TEXT PRIMARY KEY,
88
+ title TEXT,
89
+ description TEXT,
90
+ named_by TEXT,
91
+ created_at TEXT NOT NULL,
92
+ updated_at TEXT
93
+ );
94
+
73
95
  CREATE TABLE IF NOT EXISTS folders (
74
96
  agent_id TEXT NOT NULL,
75
97
  name TEXT NOT NULL,
@@ -80,6 +102,15 @@ export class MessageStore extends EventEmitter {
80
102
  CREATE INDEX IF NOT EXISTS idx_deliveries_mailbox ON deliveries (agent_id, folder, created_at DESC);
81
103
  CREATE INDEX IF NOT EXISTS idx_deliveries_message ON deliveries (message_id);
82
104
  CREATE INDEX IF NOT EXISTS idx_messages_thread ON messages (thread_id, created_at);
105
+ `);
106
+ // Bases antérieures aux fils nommés : chaque fil déjà présent reçoit sa ligne, sans
107
+ // titre. L'affichage retombe alors sur le sujet d'ouverture, comme avant.
108
+ this.db.exec(`
109
+ INSERT INTO threads (id, title, description, named_by, created_at, updated_at)
110
+ SELECT m.thread_id, NULL, NULL, NULL, MIN(m.created_at), NULL
111
+ FROM messages m
112
+ WHERE NOT EXISTS (SELECT 1 FROM threads t WHERE t.id = m.thread_id)
113
+ GROUP BY m.thread_id;
83
114
  `);
84
115
  }
85
116
  close() {
@@ -115,9 +146,18 @@ export class MessageStore extends EventEmitter {
115
146
  const row = this.db.prepare("SELECT * FROM agents WHERE id = ?").get(id);
116
147
  return row ? { ...row, registered: Boolean(row.registered) } : null;
117
148
  }
149
+ /** Annuaire classé par activité décroissante : ce qui bouge est en tête, l'id ne trie rien. */
118
150
  listAgents() {
119
- const rows = this.db.prepare("SELECT * FROM agents ORDER BY id").all();
120
- return rows.map((r) => ({ ...r, registered: Boolean(r.registered) }));
151
+ const rows = this.db
152
+ .prepare(`SELECT a.*, ${LAST_ACTIVITY} AS last_activity_at
153
+ FROM agents a
154
+ ORDER BY last_activity_at DESC, a.id`)
155
+ .all();
156
+ return rows.map((r) => ({
157
+ ...r,
158
+ registered: Boolean(r.registered),
159
+ last_activity_at: r.last_activity_at || null,
160
+ }));
121
161
  }
122
162
  // --- Envoi --------------------------------------------------------------
123
163
  sendMessage(input) {
@@ -134,6 +174,8 @@ export class MessageStore extends EventEmitter {
134
174
  const cc = dedupe((input.cc ?? []).map(normalizeAgentId).filter(Boolean)).filter((id) => !to.includes(id));
135
175
  if (to.length === 0)
136
176
  throw new MessengerError("at least one recipient is required in `to`");
177
+ const title = normalizeLabel(input.thread_title, THREAD_TITLE_MAX, "thread_title");
178
+ const threadDescription = normalizeLabel(input.thread_description, THREAD_DESCRIPTION_MAX, "thread_description");
137
179
  let threadId;
138
180
  let inReplyTo = null;
139
181
  if (input.reply_to) {
@@ -161,6 +203,16 @@ export class MessageStore extends EventEmitter {
161
203
  this.touchAgent(from);
162
204
  for (const id of [...to, ...cc])
163
205
  this.ensureAgent(id);
206
+ // Le fil naît avec son premier message ; sur une réponse, la ligne existe déjà et
207
+ // n'est touchée que si l'expéditeur fournit un nouveau titre.
208
+ this.db
209
+ .prepare(`INSERT INTO threads (id, title, description, named_by, created_at, updated_at)
210
+ VALUES (?, NULL, NULL, NULL, ?, NULL)
211
+ ON CONFLICT (id) DO NOTHING`)
212
+ .run(threadId, now);
213
+ if (title !== null || threadDescription !== null) {
214
+ this.renameThreadRow(threadId, from, title, threadDescription, now);
215
+ }
164
216
  this.db
165
217
  .prepare(`INSERT INTO messages (id, thread_id, sender_id, subject, body, priority, in_reply_to, created_at)
166
218
  VALUES (?, ?, ?, ?, ?, ?, ?, ?)`)
@@ -177,6 +229,7 @@ export class MessageStore extends EventEmitter {
177
229
  return {
178
230
  message_id: messageId,
179
231
  thread_id: threadId,
232
+ thread_title: this.getThreadInfo(threadId)?.title ?? null,
180
233
  created_at: now,
181
234
  delivered_to: [...to, ...cc],
182
235
  unknown_recipients: unknownRecipients,
@@ -209,10 +262,11 @@ export class MessageStore extends EventEmitter {
209
262
  params.push(like, like);
210
263
  }
211
264
  const rows = this.db
212
- .prepare(`SELECT m.id AS message_id, m.thread_id, m.sender_id, m.subject, m.body, m.priority,
213
- m.in_reply_to, m.created_at, d.folder, d.role, d.read_at, d.starred
265
+ .prepare(`SELECT m.id AS message_id, m.thread_id, t.title AS thread_title, m.sender_id, m.subject, m.body,
266
+ m.priority, m.in_reply_to, m.created_at, d.folder, d.role, d.read_at, d.starred
214
267
  FROM deliveries d
215
268
  JOIN messages m ON m.id = d.message_id
269
+ ${THREAD_JOIN}
216
270
  WHERE ${where.join(" AND ")}
217
271
  ORDER BY m.created_at DESC, m.id DESC
218
272
  LIMIT ? OFFSET ?`)
@@ -222,10 +276,11 @@ export class MessageStore extends EventEmitter {
222
276
  }
223
277
  getMessage(agentId, messageId, markRead = false) {
224
278
  const row = this.db
225
- .prepare(`SELECT m.id AS message_id, m.thread_id, m.sender_id, m.subject, m.body, m.priority,
226
- m.in_reply_to, m.created_at, d.folder, d.role, d.read_at, d.starred
279
+ .prepare(`SELECT m.id AS message_id, m.thread_id, t.title AS thread_title, m.sender_id, m.subject, m.body,
280
+ m.priority, m.in_reply_to, m.created_at, d.folder, d.role, d.read_at, d.starred
227
281
  FROM deliveries d
228
282
  JOIN messages m ON m.id = d.message_id
283
+ ${THREAD_JOIN}
229
284
  WHERE d.agent_id = ? AND d.message_id = ?
230
285
  ORDER BY CASE d.role WHEN 'to' THEN 0 WHEN 'cc' THEN 1 ELSE 2 END
231
286
  LIMIT 1`)
@@ -245,11 +300,12 @@ export class MessageStore extends EventEmitter {
245
300
  /** Tous les messages d'un fil visibles par l'agent, du plus ancien au plus récent. */
246
301
  getThread(agentId, threadId) {
247
302
  const rows = this.db
248
- .prepare(`SELECT m.id AS message_id, m.thread_id, m.sender_id, m.subject, m.body, m.priority,
249
- m.in_reply_to, m.created_at,
303
+ .prepare(`SELECT m.id AS message_id, m.thread_id, t.title AS thread_title, m.sender_id, m.subject, m.body,
304
+ m.priority, m.in_reply_to, m.created_at,
250
305
  MIN(d.folder) AS folder, MIN(d.role) AS role, MIN(d.read_at) AS read_at, MAX(d.starred) AS starred
251
306
  FROM deliveries d
252
307
  JOIN messages m ON m.id = d.message_id
308
+ ${THREAD_JOIN}
253
309
  WHERE d.agent_id = ? AND m.thread_id = ?
254
310
  GROUP BY m.id
255
311
  ORDER BY m.created_at ASC`)
@@ -259,6 +315,118 @@ export class MessageStore extends EventEmitter {
259
315
  const recipients = this.recipientsFor(rows.map((r) => r.message_id));
260
316
  return rows.map((row) => this.toMessage(row, recipients));
261
317
  }
318
+ // --- Fils ---------------------------------------------------------------
319
+ getThreadInfo(threadId) {
320
+ const row = this.db.prepare("SELECT * FROM threads WHERE id = ?").get(threadId);
321
+ if (!row)
322
+ return null;
323
+ const { id, ...rest } = row;
324
+ return { thread_id: id, ...rest };
325
+ }
326
+ /**
327
+ * Renomme un fil. Le titre est unique et partagé : les participants parlent du même
328
+ * sujet, ils lisent donc le même nom. Le dernier appelant a le dernier mot, et
329
+ * `named_by` retient qui c'était.
330
+ *
331
+ * Seul un participant peut renommer : le fil n'est pas nommable depuis l'extérieur.
332
+ */
333
+ setThreadLabel(agentId, threadId, label) {
334
+ const title = normalizeLabel(label.title, THREAD_TITLE_MAX, "title");
335
+ const description = normalizeLabel(label.description, THREAD_DESCRIPTION_MAX, "description");
336
+ if (title === null && description === null) {
337
+ throw new MessengerError("nothing to update: provide `title` and/or `description`");
338
+ }
339
+ this.assertThreadVisible(agentId, threadId);
340
+ this.renameThreadRow(threadId, agentId, title, description, new Date().toISOString());
341
+ return this.getThreadInfo(threadId);
342
+ }
343
+ /** Fils visibles depuis cette boîte, le plus récemment actif en tête. */
344
+ listThreads(agentId, opts = {}) {
345
+ const limit = clamp(opts.limit ?? 25, 1, MAX_LIMIT);
346
+ const offset = Math.max(0, opts.offset ?? 0);
347
+ const having = [];
348
+ const params = [agentId];
349
+ if (opts.unread_only)
350
+ having.push("unread > 0");
351
+ if (opts.search) {
352
+ // Le titre prime, mais un fil non nommé ne doit pas devenir introuvable :
353
+ // la recherche couvre aussi le sujet d'ouverture.
354
+ const matches = `LIKE ? ESCAPE '\\'`;
355
+ having.push(`(COALESCE(t.title, '') ${matches} OR COALESCE(t.description, '') ${matches} OR subject ${matches})`);
356
+ const like = `%${escapeLike(opts.search)}%`;
357
+ params.push(like, like, like);
358
+ }
359
+ const rows = this.db
360
+ .prepare(`SELECT m.thread_id,
361
+ t.title, t.description, t.named_by, t.created_at, t.updated_at,
362
+ (SELECT r.subject FROM messages r WHERE r.thread_id = m.thread_id
363
+ ORDER BY r.created_at ASC, r.id ASC LIMIT 1) AS subject,
364
+ COUNT(DISTINCT m.id) AS messages,
365
+ COUNT(DISTINCT CASE WHEN d.read_at IS NULL AND d.role != 'sender' THEN m.id END) AS unread,
366
+ MAX(m.created_at) AS last_message_at
367
+ FROM deliveries d
368
+ JOIN messages m ON m.id = d.message_id
369
+ ${THREAD_JOIN}
370
+ WHERE d.agent_id = ? AND d.folder != 'trash'
371
+ GROUP BY m.thread_id
372
+ ${having.length ? `HAVING ${having.join(" AND ")}` : ""}
373
+ ORDER BY last_message_at DESC
374
+ LIMIT ? OFFSET ?`)
375
+ .all(...params, limit, offset);
376
+ const participants = this.participantsFor(rows.map((r) => r.thread_id));
377
+ return rows.map((row) => ({
378
+ thread_id: row.thread_id,
379
+ title: row.title,
380
+ description: row.description,
381
+ named_by: row.named_by,
382
+ // `created_at` du fil manque si la ligne n'a jamais été écrite : le premier
383
+ // message fait alors foi.
384
+ created_at: row.created_at ?? row.last_message_at,
385
+ updated_at: row.updated_at,
386
+ subject: row.subject,
387
+ messages: row.messages,
388
+ unread: row.unread,
389
+ participants: participants.get(row.thread_id) ?? [],
390
+ last_message_at: row.last_message_at,
391
+ }));
392
+ }
393
+ /** Écriture brute du nom : un champ absent (`null`) laisse l'ancienne valeur en place. */
394
+ renameThreadRow(threadId, agentId, title, description, now) {
395
+ this.db
396
+ .prepare(`INSERT INTO threads (id, title, description, named_by, created_at, updated_at)
397
+ VALUES (?, ?, ?, ?, ?, ?)
398
+ ON CONFLICT (id) DO UPDATE SET
399
+ title = COALESCE(excluded.title, threads.title),
400
+ description = COALESCE(excluded.description, threads.description),
401
+ named_by = excluded.named_by,
402
+ updated_at = excluded.updated_at`)
403
+ .run(threadId, title, description, agentId, now, now);
404
+ }
405
+ assertThreadVisible(agentId, threadId) {
406
+ const row = this.db
407
+ .prepare(`SELECT 1 FROM deliveries d JOIN messages m ON m.id = d.message_id
408
+ WHERE d.agent_id = ? AND m.thread_id = ? LIMIT 1`)
409
+ .get(agentId, threadId);
410
+ if (!row)
411
+ throw new MessengerError(`thread not found in this mailbox: ${threadId}`, 404);
412
+ }
413
+ participantsFor(threadIds) {
414
+ const map = new Map();
415
+ const ids = dedupe(threadIds);
416
+ if (ids.length === 0)
417
+ return map;
418
+ const placeholders = ids.map(() => "?").join(",");
419
+ const rows = this.db
420
+ .prepare(`SELECT DISTINCT m.thread_id, d.agent_id
421
+ FROM deliveries d JOIN messages m ON m.id = d.message_id
422
+ WHERE m.thread_id IN (${placeholders})
423
+ ORDER BY d.agent_id`)
424
+ .all(...ids);
425
+ for (const row of rows) {
426
+ map.set(row.thread_id, [...(map.get(row.thread_id) ?? []), row.agent_id]);
427
+ }
428
+ return map;
429
+ }
262
430
  // --- Organisation -------------------------------------------------------
263
431
  markMessages(agentId, messageIds, state) {
264
432
  if (state.read === undefined && state.starred === undefined) {
@@ -380,9 +548,10 @@ export class MessageStore extends EventEmitter {
380
548
  params.push(opts.thread_id);
381
549
  }
382
550
  const rows = this.db
383
- .prepare(`SELECT m.id AS message_id, m.thread_id, m.sender_id, m.subject, m.body, m.priority,
384
- m.in_reply_to, m.created_at, 'inbox' AS folder, 'to' AS role, NULL AS read_at, 0 AS starred
551
+ .prepare(`SELECT m.id AS message_id, m.thread_id, t.title AS thread_title, m.sender_id, m.subject, m.body,
552
+ m.priority, m.in_reply_to, m.created_at, 'inbox' AS folder, 'to' AS role, NULL AS read_at, 0 AS starred
385
553
  FROM messages m
554
+ ${THREAD_JOIN}
386
555
  WHERE ${where.join(" AND ")}
387
556
  ORDER BY m.created_at DESC, m.id DESC
388
557
  LIMIT ? OFFSET ?`)
@@ -393,9 +562,9 @@ export class MessageStore extends EventEmitter {
393
562
  /** Message complet vu du serveur, avec l'état de lecture de chaque destinataire. */
394
563
  getMessageAdmin(messageId) {
395
564
  const row = this.db
396
- .prepare(`SELECT m.id AS message_id, m.thread_id, m.sender_id, m.subject, m.body, m.priority,
397
- m.in_reply_to, m.created_at, 'inbox' AS folder, 'to' AS role, NULL AS read_at, 0 AS starred
398
- FROM messages m WHERE m.id = ?`)
565
+ .prepare(`SELECT m.id AS message_id, m.thread_id, t.title AS thread_title, m.sender_id, m.subject, m.body,
566
+ m.priority, m.in_reply_to, m.created_at, 'inbox' AS folder, 'to' AS role, NULL AS read_at, 0 AS starred
567
+ FROM messages m ${THREAD_JOIN} WHERE m.id = ?`)
399
568
  .get(messageId);
400
569
  if (!row)
401
570
  throw new MessengerError(`message not found: ${messageId}`, 404);
@@ -485,6 +654,7 @@ export class MessageStore extends EventEmitter {
485
654
  return {
486
655
  message_id: row.message_id,
487
656
  thread_id: row.thread_id,
657
+ thread_title: row.thread_title,
488
658
  from: row.sender_id,
489
659
  to: rcpt.to,
490
660
  cc: rcpt.cc,
@@ -506,11 +676,29 @@ export class MessageStore extends EventEmitter {
506
676
  /** Un message sans aucune livraison n'est plus lisible par personne : on le supprime. */
507
677
  dropOrphanMessages() {
508
678
  this.db.exec("DELETE FROM messages WHERE NOT EXISTS (SELECT 1 FROM deliveries d WHERE d.message_id = messages.id)");
679
+ // Le nom d'un fil vidé de ses messages ne désigne plus rien.
680
+ this.db.exec("DELETE FROM threads WHERE NOT EXISTS (SELECT 1 FROM messages m WHERE m.thread_id = threads.id)");
509
681
  }
510
682
  }
511
683
  export function normalizeAgentId(id) {
512
684
  return String(id ?? "").trim();
513
685
  }
686
+ /**
687
+ * Nettoie un intitulé fourni par un agent.
688
+ *
689
+ * `undefined` et `null` valent « ne touche pas » ; une chaîne vide est une erreur, car
690
+ * elle trahit plus souvent un bug d'appel qu'une volonté d'effacer le nom.
691
+ */
692
+ function normalizeLabel(value, max, field) {
693
+ if (value === undefined || value === null)
694
+ return null;
695
+ const trimmed = String(value).trim();
696
+ if (!trimmed)
697
+ throw new MessengerError(`${field} cannot be empty`);
698
+ if (trimmed.length > max)
699
+ throw new MessengerError(`${field} must be at most ${max} characters`);
700
+ return trimmed;
701
+ }
514
702
  function dedupe(values) {
515
703
  return [...new Set(values)];
516
704
  }