@grknbyk/agent-wire 0.13.1 → 0.13.2

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@grknbyk/agent-wire",
-    "version": "0.13.1",
+    "version": "0.13.2",
     "description": "Let AI coding agents message each other through a shared Slack channel, over MCP.",
     "type": "module",
     "license": "MIT",

package/src/inbox.mjs

@@ -160,9 +160,13 @@ export const findByTs = (ts) => readInbox().find((item) => item.ts === ts) ?? nu
 
 // Newest first: a ref is short enough to collide eventually, and the one a person
 // just read off the channel is the recent one.
-export const findByRef = (ref) => readInbox()
-    .filter((item) => item.ref === String(ref).replace(/^@/, '').toLowerCase())
-    .at(-1) ?? null;
+export function findByRef(handle) {
+    const [channel, ref] = String(handle).toLowerCase().split('@');
+    const wanted = ref ?? channel;
+    return readInbox()
+        .filter((item) => item.ref === wanted && (!ref || !channel || item.channel === channel))
+        .at(-1) ?? null;
+}
 
 export const readCursor = (channelId) => readJsonCached(paths.cursors, {})[channelId] ?? null;
 

package/src/mcp.mjs

@@ -262,7 +262,7 @@ async function sendText(config, { to, text, channel, replyTo }) {
 
     const client = slackClient(config.bot_token);
     const ref = mintRef();
-    const rendered = formatMessage({ mark: config.mark, from: config.nickname, to, text, ref });
+    const rendered = formatMessage({ mark: config.mark, from: config.nickname, to, text, ref, channel: target.name });
     const signature = signMessage(config.private_key, {
         channel: target.id, from: config.nickname, to, conv: chain.conv, hop: chain.hop, text,
     });
@@ -279,7 +279,7 @@ async function sendText(config, { to, text, channel, replyTo }) {
     if (!posted.ok) return `Slack rejected it (${posted.reason})`;
 
     recordOwnMessage(config, { ts: posted.ts, target, to, text, chain, ref });
-    return `delivered to ${to} in #${target.name} as @${ref}`;
+    return `delivered to ${to} in #${target.name} as ${target.name}@${ref}`;
 }
 
 // Our own sent messages go into the local log too, so the log is a complete
@@ -319,7 +319,7 @@ async function postFile(config, { to, path, note, target, chain, logText }) {
     });
     const posted = await postMessage(client, {
         channel: target.id,
-        rendered: formatMessage({ mark: config.mark, from: config.nickname, to, text, ref }),
+        rendered: formatMessage({ mark: config.mark, from: config.nickname, to, text, ref, channel: target.name }),
         signature,
         publicKey: config.public_key,
         from: config.nickname,
@@ -351,7 +351,7 @@ async function sendLongText(config, { to, text, target, chain }) {
     unlinkSync(path);
     if (!result.ok) return result.message;
 
-    return `delivered to ${to} in #${result.channelName} as @${result.ref} — ${text.length} characters, sent as a file`;
+    return `delivered to ${to} in #${result.channelName} as ${result.channelName}@${result.ref} — ${text.length} characters, sent as a file`;
 }
 
 async function call(name, args, session) {
@@ -454,7 +454,7 @@ async function call(name, args, session) {
         });
         if (!result.ok) return result.message;
 
-        return `sent ${result.name} to ${args.to} in #${result.channelName} as @${result.ref}`;
+        return `sent ${result.name} to ${args.to} in #${result.channelName} as ${result.channelName}@${result.ref}`;
     }
 
     if (name === 'archive') return `archived ${archive(args.ts)} message(s)`;

package/src/protocol.mjs

@@ -72,7 +72,11 @@ export function formatMessage({ mark, from, to, text, ref, channel }) {
     const left = `${mark ? `${mark} ` : ''}${from} => ${to}`;
     if (!ref) return `${left}\n${toSlackText(text)}\n`;
 
-    const handle = `${channel ?? ''}@${ref}`;
+    // Shipped once without this: a call site forgot the channel, the handle went out
+    // bare, and every test still passed because they all called this directly.
+    if (!channel) throw new TypeError(`formatMessage: ref ${ref} with no channel to put in front of it`);
+
+    const handle = `${channel}@${ref}`;
     const gap = Math.max(1, HEADER_WIDTH - displayWidth(left) - handle.length);
     return `${left}${' '.repeat(gap)}${handle}\n${toSlackText(text)}\n`;
 }