paperclip-plugin-mattermost 0.16.0 → 0.17.0

package/README.md

@@ -84,7 +84,7 @@ PAPERCLIP_REF=<commit> node build.mjs   # build for another Paperclip version
 Bump `version` in `package.json` and `src/manifest.ts`, commit, then tag and push:
 
 ```bash
-git tag -a v0.16.0 -m v0.16.0 && git push origin main v0.16.0
+git tag -a v0.17.0 -m v0.17.0 && git push origin main v0.17.0
 ```
 
 GitHub Actions builds, tests and publishes to npm through npm trusted publishing, so no npm token is involved.

package/dist/manifest.js

@@ -82,13 +82,14 @@ var route = (routeKey, method, path) => ({ routeKey, method, path, auth: "agent"
 var manifest = {
   id: "mattermost-bridge",
   apiVersion: 1,
-  version: "0.16.0",
+  version: "0.17.0",
   displayName: "Mattermost",
   description: "Every agent becomes its own Mattermost bot. DM an agent or mention it in a channel and it answers in the thread, showing its steps as it works; follow-ups continue the conversation. Paperclip approvals arrive with Approve / Reject buttons.",
   author: "Haseeb Azhar",
   categories: ["connector"],
   capabilities: [
     "agents.read",
+    "companies.read",
     "agent.sessions.create",
     "agent.sessions.send",
     "agent.sessions.list",

package/dist/worker.js

@@ -45032,7 +45032,9 @@ var Actions = class {
     this.d = deps;
   }
   /** One route call. `params` are the route's path parameters, `input` its query or body. */
-  async handle(agentId, routeKey, params, input2) {
+  runId = null;
+  async handle(agentId, routeKey, params, input2, runId = null) {
+    this.runId = runId;
     const bot = this.d.bot(agentId);
     if (!bot) return bad(409, "You have no Mattermost bot in this company yet (your agent may be switched off in the plugin's settings).");
     try {
@@ -45184,13 +45186,13 @@ var Actions = class {
     const text2 = str3(input2.text);
     if (!text2) return bad(400, "text is empty");
     const ch = await this.channel(bot, str3(input2.channel));
-    const threadId = str3(input2.threadId);
+    const threadId = str3(input2.threadId) || await this.d.taskThread?.(agentId, this.runId, ch.id).catch(() => null) || "";
     const waiting = await this.gate(agentId, canPost(text2), "post", `post in ~${ch.name}: "${text2.slice(0, 200)}"`, { channelId: ch.id, threadId, text: text2 });
     if (waiting) return waiting;
     return ok(await this.doPost(bot, ch.id, threadId, text2));
   }
   async doPost(bot, channelId, threadId, text2) {
-    const p = await bot.request("POST", "/posts", { channel_id: channelId, root_id: threadId, message: text2 });
+    const p = await bot.request("POST", "/posts", { channel_id: channelId, root_id: threadId, message: this.d.format ? this.d.format(text2) : text2 });
     return { posted: p.id, threadId: p.root_id || p.id };
   }
   async react(bot, input2) {
@@ -45207,7 +45209,8 @@ var Actions = class {
     const verdict = canDm(await this.d.canTalk(user2.id, agentId));
     if (!verdict.ok) return bad(403, `Not allowed: ${verdict.why}.`);
     const channel2 = await bot.request("POST", "/channels/direct", [bot.botUserId, user2.id]);
-    return ok(await this.doPost(bot, channel2.id, "", text2));
+    const threadId = str3(input2.threadId) || await this.d.taskThread?.(agentId, this.runId, channel2.id).catch(() => null) || "";
+    return ok(await this.doPost(bot, channel2.id, threadId, text2));
   }
   // ---- channels ----------------------------------------------------------------------
   async created() {
@@ -45526,6 +45529,13 @@ function absoluteLinks(text2, base) {
   const root = base.replace(/\/+$/, "");
   return text2.replace(/\]\((\/(?!\/)[^)\s]*)\)/g, (_m, path3) => `](${root}${path3})`);
 }
+function linkTasks(text2, base, prefix) {
+  if (!base || !/^[A-Z][A-Z0-9]{0,15}$/.test(prefix)) return text2;
+  const root = base.replace(/\/+$/, "");
+  const skip = /(```[\s\S]*?```|`[^`\n]*`|\[[^\]]*\]\([^)]*\)|<[^>\s]+>|https?:\/\/[^\s)]+)/;
+  const id = new RegExp(`(?<![\\w/-])(${prefix}-\\d+)(?![\\w-])`, "g");
+  return text2.split(skip).map((part, i) => i % 2 ? part : part.replace(id, (m) => `[${m}](${root}/${prefix}/issues/${m})`)).join("");
+}
 function splitMessage(text2, max = MAX_POST) {
   const parts = [];
   let rest = text2;
@@ -45551,8 +45561,9 @@ var Bridge = class {
   choiceButtons;
   canTalk;
   turnLimit;
-  /** Paperclip's public URL, for agents' relative links. */
+  /** Paperclip's public URL, for agents' relative links, and the company's task prefix (BAD). */
   linkBase = "";
+  taskPrefix = "";
   /**
    * Steering: messages people send in a thread while its agent is mid-run, kept for that agent to
    * read in the same run (`takeInbox`). One it read is not run again afterwards.
@@ -45774,7 +45785,7 @@ To send files back with your reply, save them in ${outbox}` : "");
     }
     if (this.active.get(member.agentId) === here) this.active.delete(member.agentId);
     this.inbox.delete(member.agentId);
-    const { text: body, choices } = extractChoices(absoluteLinks(reply ?? "", this.linkBase));
+    const { text: body, choices } = extractChoices(linkTasks(absoluteLinks(reply ?? "", this.linkBase), this.linkBase, this.taskPrefix));
     const origin = { channelId: post.channel_id, rootId, channelType: evt.channelType, channelName: evt.channelName };
     const buttons = choices.length && this.choiceButtons ? this.choiceButtons(origin, choices) : null;
     const shown = choices.length && !buttons ? `${body}
@@ -46272,13 +46283,14 @@ var route = (routeKey, method, path3) => ({ routeKey, method, path: path3, auth:
 var manifest = {
   id: "mattermost-bridge",
   apiVersion: 1,
-  version: "0.16.0",
+  version: "0.17.0",
   displayName: "Mattermost",
   description: "Every agent becomes its own Mattermost bot. DM an agent or mention it in a channel and it answers in the thread, showing its steps as it works; follow-ups continue the conversation. Paperclip approvals arrive with Approve / Reject buttons.",
   author: "Haseeb Azhar",
   categories: ["connector"],
   capabilities: [
     "agents.read",
+    "companies.read",
     "agent.sessions.create",
     "agent.sessions.send",
     "agent.sessions.list",
@@ -46552,6 +46564,8 @@ var Tenant = class {
       void bridge.handle(evt);
     };
     bridge.linkBase = paperclipUrl;
+    const company = await ctx.companies.get(companyId).catch(() => null);
+    bridge.taskPrefix = company?.issuePrefix ?? "";
     bridge.onSteer = (agentId, postId) => {
       void ctx.events.emit("person-wrote", companyId, { agentId, postId }).catch(() => {
       });
@@ -46598,6 +46612,15 @@ var Tenant = class {
       agents: () => roster.all().map((m) => ({ agentId: m.agentId, agentName: m.agentName, username: m.username, botUserId: m.botUserId })),
       canTalk: async (userId, agentId) => !canTalk || canTalk(userId, agentId),
       origin: (agentId) => bridge.activeThread(agentId),
+      format: (text2) => linkTasks(absoluteLinks(text2, paperclipUrl), paperclipUrl, bridge.taskPrefix),
+      // The task this run works on (its execution or checkout run), and the thread it was asked for in.
+      taskThread: async (agentId, runId, channelId) => {
+        if (!runId) return null;
+        const issues = await ctx.issues.list({ companyId, assigneeAgentId: agentId, limit: 200 });
+        const issue2 = issues.find((i) => i.executionRunId === runId || i.checkoutRunId === runId);
+        const at = issue2 ? await store.get(`issuethread:${issue2.id}`) : null;
+        return at && at.channelId === channelId ? at.rootId : null;
+      },
       task: async (taskId) => await ctx.issues.get(taskId, companyId),
       askApproval: async (req) => {
         if (!paperclipUrl || !approvers.size) throw Object.assign(new Error("this needs a person's approval, and nobody can approve from Mattermost yet (the plugin needs its Paperclip public URL and approvers set)"), { status: 403 });
@@ -46668,9 +46691,9 @@ var Tenant = class {
     return null;
   }
   /** An agent's call to one of the plugin's `/mm/...` routes. */
-  async api(agentId, routeKey, params, input2) {
+  async api(agentId, routeKey, params, input2, runId = null) {
     if (!this.actions) return { status: 503, body: { error: `Mattermost is not connected for this company (${this.status})` } };
-    return this.actions.handle(agentId, routeKey, params, input2);
+    return this.actions.handle(agentId, routeKey, params, input2, runId);
   }
   /** The Agent computer plugin: an agent wants a person at its computer. Posted where the agent is working. */
   async computerNeedsPerson(p) {
@@ -46826,6 +46849,14 @@ var plugin = definePlugin({
     ctx.events.on("plugin.agent-computer.needs-person", (e) => quietly("could not post a computer help request", async () => {
       await tenants.get(e.companyId)?.computerNeedsPerson(e.payload ?? {});
     }));
+    ctx.events.on("issue.created", (e) => quietly("could not link a task to its thread", async () => {
+      const tenant = tenants.get(e.companyId);
+      if (!tenant?.bridge || !e.entityId) return;
+      const issue2 = await ctx.issues.get(e.entityId, e.companyId);
+      const by = issue2?.createdByAgentId;
+      const at = by ? tenant.bridge.activeThread(by) : null;
+      if (at) await tenant.store().set(`issuethread:${e.entityId}`, at);
+    }));
     ctx.events.on("approval.decided", (e) => quietly("could not update approval", async () => {
       if (e.entityId) await tenants.get(e.companyId)?.approvals?.decided(e.entityId);
     }));
@@ -46865,7 +46896,7 @@ var plugin = definePlugin({
       const messages = tenant.bridge ? await tenant.bridge.takeInbox(agentId) : [];
       return { status: 200, body: { messages, note: messages.length ? "Written to you while you were working. Act on them now: they are not sent to you again." : "Nothing new." } };
     }
-    const res = await tenant.api(agentId, input2.routeKey, input2.params, { ...query, ...body });
+    const res = await tenant.api(agentId, input2.routeKey, input2.params, { ...query, ...body }, input2.actor.runId ?? null);
     if (tenant.bridge?.hasInbox(agentId) && res.body && typeof res.body === "object" && !Array.isArray(res.body)) {
       res.body = { ...res.body, newMessages: await tenant.bridge.takeInbox(agentId) };
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "paperclip-plugin-mattermost",
-  "version": "0.16.0",
+  "version": "0.17.0",
   "description": "Paperclip agents in Mattermost: a bot per agent, live progress, files, approval and decision buttons, and per-person access, for each Paperclip company.",
   "type": "module",
   "license": "MIT",