paperclip-plugin-mattermost 0.18.1 → 0.18.3

package/README.md

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

package/dist/manifest.js

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

@@ -45365,6 +45365,8 @@ var STATUS_WORDS = {
 };
 var WorkLinks = class {
   d;
+  /** One task's updates, one at a time: Paperclip sends several at once and each would report the change. */
+  queue = /* @__PURE__ */ new Map();
   constructor(deps) {
     this.d = deps;
   }
@@ -45383,33 +45385,60 @@ var WorkLinks = class {
     const c = (await this.d.comments(issueId)).find((x) => x.id === commentId);
     if (!c?.body?.trim()) return;
     const body = c.body.trim().slice(0, 12e3);
-    const own2 = c.authorAgentId && this.d.hasBot(c.authorAgentId);
-    const text2 = own2 ? body : `**${c.authorAgentId ? this.d.agentName(c.authorAgentId) : "A team member"}** commented on ${this.label(hit.link)}:
+    const quoted = `**${c.authorAgentId ? this.d.agentName(c.authorAgentId) : "A team member"}** commented on ${this.label(hit.link)}:
 
 ${body}`;
-    await this.d.post(own2 ? c.authorAgentId : hit.link.agentId, hit.channelId, "", text2);
+    if (c.authorAgentId && c.authorAgentId !== hit.link.agentId && this.d.hasBot(c.authorAgentId)) {
+      try {
+        return await this.d.post(c.authorAgentId, hit.channelId, "", body);
+      } catch {
+      }
+    }
+    await this.d.post(hit.link.agentId, hit.channelId, "", c.authorAgentId === hit.link.agentId ? body : quoted);
+  }
+  serial(key, run) {
+    const next = (this.queue.get(key) ?? Promise.resolve()).catch(() => {
+    }).then(run);
+    this.queue.set(key, next);
+    void next.catch(() => {
+    }).finally(() => {
+      if (this.queue.get(key) === next) this.queue.delete(key);
+    });
+    return next;
   }
   /**
    * A task asked for in a Mattermost thread (created by the agent while answering there): when it is
    * done, cancelled or blocked, its outcome (the last comment) is posted back in that thread, so the
    * result always reaches the conversation it came from.
    */
-  async threadUpdate(issueId) {
+  threadUpdate(issueId) {
+    return this.serial(`thread:${issueId}`, () => this.reportToThread(issueId));
+  }
+  async reportToThread(issueId) {
     const at = await this.d.store.get(`issuethread:${issueId}`);
     if (!at) return;
     const task2 = await this.d.task(issueId);
     if (!task2 || task2.status === at.status || !THREAD_REPORTS[task2.status]) return;
+    const by = [at.agentId, task2.assigneeAgentId].find((a) => Boolean(a && this.d.hasBot(a)));
+    if (!by) return;
     await this.d.store.set(`issuethread:${issueId}`, { ...at, status: task2.status });
     const last = (await this.d.comments(issueId)).at(-1)?.body?.trim();
     const outcome = last ? `
 
 ${last.length > 3e3 ? last.slice(0, 3e3) + "\u2026" : last}` : "";
-    const by = [task2.assigneeAgentId, at.agentId].find((a) => Boolean(a && this.d.hasBot(a)));
-    if (!by) return;
-    await this.d.post(by, at.channelId, at.rootId, `${THREAD_REPORTS[task2.status]} **${task2.identifier ?? "The task"}** ${task2.title}${outcome}`);
+    const doer = task2.assigneeAgentId && task2.assigneeAgentId !== by ? ` _(by ${this.d.agentName(task2.assigneeAgentId)})_` : "";
+    try {
+      await this.d.post(by, at.channelId, at.rootId, `${THREAD_REPORTS[task2.status]} **${task2.identifier ?? "The task"}** ${task2.title}${doer}${outcome}`);
+    } catch (err) {
+      await this.d.store.set(`issuethread:${issueId}`, at);
+      throw err;
+    }
   }
   /** The task changed: note a new status; on done or cancelled, archive and report back. */
-  async updated(issueId) {
+  updated(issueId) {
+    return this.serial(`link:${issueId}`, () => this.followLink(issueId));
+  }
+  async followLink(issueId) {
     const hit = await this.linked(issueId);
     if (!hit) return;
     const task2 = await this.d.task(issueId);
@@ -46365,7 +46394,7 @@ var route = (routeKey, method, path3) => ({ routeKey, method, path: path3, auth:
 var manifest = {
   id: "mattermost-bridge",
   apiVersion: 1,
-  version: "0.18.1",
+  version: "0.18.3",
   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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "paperclip-plugin-mattermost",
-  "version": "0.18.1",
+  "version": "0.18.3",
   "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",