@imenam/mcp-messenger 1.0.10 → 1.0.11

package/dist/src/mcp/tools.js

@@ -55,14 +55,14 @@ export const TOOLS = [
     },
     {
         name: "list_threads",
-        description: "List the conversations visible from this mailbox, most recently active first, with their name, participants and unread count. This is the readable map of the ongoing work: prefer it over list_messages when you want to know what is being discussed rather than what arrived last. Set with_agent to see everything exchanged with one mailbox. A thread with no name falls back to the subject of its opening message — name it with rename_thread.",
+        description: "List the conversations visible from this mailbox, most recently active first, with their name, participants and unread count. This is the readable map of the ongoing work: prefer it over list_messages when you want to know what is being discussed rather than what arrived last. Set with_mailbox to see everything exchanged with one correspondent. A thread with no name falls back to the subject of its opening message — name it with rename_thread.",
         inputSchema: {
             type: "object",
             properties: {
                 unread_only: { type: "boolean", description: "Keep only threads with unread messages." },
-                with_agent: {
+                with_mailbox: {
                     type: "string",
-                    description: "Mailbox address of a correspondent: keep only the threads shared with that mailbox. This is how you review what you have exchanged with someone — a conversation is a thread, not a stack of messages.",
+                    description: "A correspondent's mailbox address, as listed by list_agents: keep only the threads shared with that mailbox. This is how you review what you have exchanged with someone — a conversation is a thread, not a stack of messages.",
                 },
                 search: { type: "string", description: "Free text matched against thread name, description and subject." },
                 limit: { type: "number", description: "1-100, defaults to 25." },
@@ -72,7 +72,7 @@ export const TOOLS = [
     },
     {
         name: "list_agents",
-        description: "List the other agents known to the mailbox server, grouped by agent name — one group per project, one line per session mailbox inside it, most recently active first. An agent name is shared by every session of the same project, so a group with several lines means several live sessions, each with its own address. Your own mailbox is never listed — you cannot write to yourself. Use it to find the address of the agent you want to write to; pass agent_name to expand a group whose sessions were not all listed.",
+        description: "List the mailboxes known to the server, grouped by agent name — one group per project, one line per session mailbox inside it, most recently active first. An agent name is shared by every session of the same project, so a group with several lines means several live sessions, each with its own mailbox address. Your own mailbox is never listed — you cannot write to yourself. Use it to find the mailbox address you want to write to; pass agent_name to expand a group whose sessions were not all listed.",
         inputSchema: {
             type: "object",
             properties: {
@@ -86,11 +86,11 @@ export const TOOLS = [
     },
     {
         name: "send_message",
-        description: "Send a message to one agent, identified by its mailbox address. A message has a single recipient: to reach several agents, send each one its own message. Set reply_to to answer an existing message and keep the thread together. Never write a mailbox address in the body: the server attaches the sender and the recipient to every message, so an address copied by hand adds nothing and misleads as soon as it is wrong.",
+        description: "Send a message to one mailbox, identified by its address. A message has a single recipient: to reach several correspondents, send each one its own message. Set reply_to to answer an existing message and keep the thread together. Never write a mailbox address in the body: the server attaches the sender and the recipient to every message, so an address copied by hand adds nothing and misleads as soon as it is wrong.",
         inputSchema: {
             type: "object",
             properties: {
-                to: { type: "string", description: "Recipient agent id (mailbox address). Exactly one recipient." },
+                to: { type: "string", description: "Recipient mailbox address, as listed by list_agents. Exactly one recipient." },
                 subject: { type: "string", description: "Short subject line." },
                 body: {
                     type: "string",
@@ -110,13 +110,13 @@ export const TOOLS = [
     },
     {
         name: "list_messages",
-        description: "List messages in this agent's mailbox, newest first, with a short preview. Filter by folder, unread state, sender, thread or free text.",
+        description: "List messages in this mailbox, newest first, with a short preview. Filter by folder, unread state, sender, thread or free text.",
         inputSchema: {
             type: "object",
             properties: {
                 folder: { type: "string", description: FOLDER_HINT + " Defaults to inbox." },
                 unread_only: { type: "boolean", description: "Keep only unread messages." },
-                from: { type: "string", description: "Only messages sent by this agent id." },
+                from: { type: "string", description: "Only messages sent by this mailbox address." },
                 thread_id: { type: "string", description: "Only messages of this thread." },
                 search: { type: "string", description: "Free text matched against subject and body." },
                 limit: { type: "number", description: "1-100, defaults to 25." },
@@ -140,7 +140,7 @@ export const TOOLS = [
     },
     {
         name: "get_thread",
-        description: "Read every message of a thread visible to this agent, oldest first — the full conversation.",
+        description: "Read every message of a thread visible to this mailbox, oldest first — the full conversation.",
         inputSchema: {
             type: "object",
             properties: { thread_id: { type: "string", description: "Thread id shown on any message." } },
@@ -177,7 +177,7 @@ export const TOOLS = [
     },
     {
         name: "delete_messages",
-        description: "Send messages to the trash folder, or erase them from this mailbox for good when permanent is true. Other recipients keep their own copy.",
+        description: "Send messages to the trash folder, or erase them from this mailbox for good when permanent is true. The other party keeps its own copy in its own mailbox.",
         inputSchema: {
             type: "object",
             properties: {
@@ -289,6 +289,10 @@ const schemas = {
     }),
     list_threads: z.object({
         unread_only: z.boolean().optional(),
+        with_mailbox: z.string().trim().min(1).optional(),
+        // Ancien nom du même filtre, plus publié mais toujours accepté : une session l'a pu
+        // apprendre avant la mise à jour. Sans cette entrée, zod l'écarterait en silence et
+        // l'agent recevrait tous ses fils en croyant les avoir filtrés.
         with_agent: z.string().trim().min(1).optional(),
         search: z.string().trim().min(1).optional(),
         limit: z.number().int().optional(),
@@ -386,8 +390,17 @@ export async function callTool(client, name, rawArgs) {
         }
         case "list_threads": {
             const input = schemas.list_threads.parse(args);
-            const threads = await client.listThreads(input);
-            const scope = input.with_agent ? ` with ${input.with_agent}` : "";
+            // Une adresse désigne une boîte : les outils disent `with_mailbox`. Le protocole HTTP
+            // et le stockage gardent `with_agent`, du nom de la colonne, comme `thread_title`
+            // garde son nom face au `thread_name` des outils.
+            const correspondent = input.with_mailbox ?? input.with_agent;
+            const threads = await client.listThreads({
+                unread_only: input.unread_only,
+                search: input.search,
+                limit: input.limit,
+                with_agent: correspondent,
+            });
+            const scope = correspondent ? ` with ${correspondent}` : "";
             if (threads.length === 0)
                 return `No thread in this mailbox${scope} matching these filters.`;
             return [`${threads.length} thread(s)${scope}:`, "", ...threads.map(formatThread)].join("\n");
@@ -396,7 +409,7 @@ export async function callTool(client, name, rawArgs) {
             const input = schemas.list_agents.parse(args);
             const agents = await client.listAgents();
             if (agents.length === 0)
-                return "No other agent on this server yet.";
+                return "No other mailbox on this server yet.";
             const groups = groupByAgentName(agents);
             return input.agent_name ? formatAgentGroup(groups, input.agent_name) : formatDirectory(groups);
         }
@@ -561,7 +574,7 @@ function groupByAgentName(agents) {
     const groups = new Map();
     for (const agent of agents) {
         const name = agent.agent_name?.trim() ?? "";
-        const key = name ? name.toLowerCase() : `${agent.id}`;
+        const key = name ? name.toLowerCase() : `\u0000${agent.id}`;
         const group = groups.get(key);
         if (group) {
             group.mailboxes.push(agent);
@@ -597,8 +610,8 @@ function formatMailboxLine(agent) {
 const DIRECTORY_FOOTER = [
     "Addresses identify sessions; agent names do not. Several mailboxes above may carry the same",
     "agent name as yours — they are other sessions of the same project, never you: your own mailbox",
-    "is never listed here. Always write to an address taken from this list, never to a name.",
-    "To see what you have already exchanged with one of them, call list_threads with its address in with_agent.",
+    "is never listed here. Always write to a mailbox address taken from this list, never to a name.",
+    "To see what you have already exchanged with one of them, call list_threads with its address in with_mailbox.",
 ];
 function formatDirectory(groups) {
     const mailboxes = groups.reduce((total, group) => total + group.mailboxes.length, 0);

package/dist/src/server/app.js

@@ -79,6 +79,14 @@ const waitQuerySchema = z.object({
     watcher: z.enum(["true", "1"]).optional(),
 });
 const loginSchema = z.object({ token: z.string().min(1).max(1024) });
+const adminThreadsSchema = z.object({
+    search: z.string().trim().min(1).optional(),
+    agent: z.string().trim().min(1).optional(),
+    unread_only: z.enum(["true", "1"]).optional(),
+    priority: z.enum(PRIORITIES).optional(),
+    limit: z.coerce.number().int().min(1).max(100).optional(),
+    offset: z.coerce.number().int().min(0).optional(),
+});
 const adminListSchema = z.object({
     search: z.string().trim().min(1).optional(),
     agent: z.string().trim().min(1).optional(),
@@ -162,6 +170,20 @@ export function createApp({ store, config }) {
     admin.get("/messages/:id", (req, res) => {
         res.json(store.getMessageAdmin(String(req.params.id)));
     });
+    // Déclarée avant `/threads/:id` : la console lit d'abord les fils, le détail ensuite.
+    admin.get("/threads", (req, res) => {
+        const query = adminThreadsSchema.parse(req.query);
+        res.json({
+            threads: store.listAllThreadsAdmin({
+                search: query.search,
+                agent: query.agent,
+                unread_only: query.unread_only !== undefined,
+                priority: query.priority,
+                limit: query.limit,
+                offset: query.offset,
+            }),
+        });
+    });
     admin.get("/threads/:id", (req, res) => {
         const threadId = String(req.params.id);
         const messages = store.listThreadAdmin(threadId);