paperclip-plugin-mattermost 0.16.0 → 0.16.1

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.16.1 -m v0.16.1 && git push origin main v0.16.1
 ```
 
 GitHub Actions builds, tests and publishes to npm through npm trusted publishing, so no npm token is involved.

package/dist/manifest.js

@@ -82,7 +82,7 @@ var route = (routeKey, method, path) => ({ routeKey, method, path, auth: "agent"
 var manifest = {
   id: "mattermost-bridge",
   apiVersion: 1,
-  version: "0.16.0",
+  version: "0.16.1",
   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",

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,7 +45186,7 @@ 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));
@@ -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() {
@@ -46272,7 +46275,7 @@ var route = (routeKey, method, path3) => ({ routeKey, method, path: path3, auth:
 var manifest = {
   id: "mattermost-bridge",
   apiVersion: 1,
-  version: "0.16.0",
+  version: "0.16.1",
   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",
@@ -46598,6 +46601,14 @@ 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),
+      // 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 +46679,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 +46837,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 +46884,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.16.1",
   "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",