agents-relay 1.0.116 → 1.0.117

package/dist/store-github.js

@@ -1,6 +1,7 @@
 import { parseAgent, parseJob, parseTask, renderAgentComment, renderJobBody, renderTaskComment, JOB_MARKER } from './markers.js';
 import { isTaskForJob, sameTaskDefinition } from './store-task-definition.js';
 import { resolveBlockerDependencies } from './blockers.js';
+const TASK_WRITE_OVERLAY = new Map();
 export class GitHubStore {
     client;
     repository;
@@ -12,6 +13,29 @@ export class GitHubStore {
         this.prNumber = prNumber;
         this.trustedAuthors = trustedAuthors;
     }
+    taskKey(task) { return `${this.repository}#${this.prNumber}:${task.jobId ?? ''}/${task.id}`; }
+    taskKeyFromBody(body) { const task = parseTask(body); return task ? this.taskKey(task) : null; }
+    overlayTaskWrites(comments) {
+        if (TASK_WRITE_OVERLAY.size === 0)
+            return comments;
+        const result = [...comments];
+        for (const [key, overlay] of [...TASK_WRITE_OVERLAY]) {
+            const index = result.findIndex(comment => this.taskKeyFromBody(comment.body) === key);
+            if (index >= 0) {
+                const remote = result[index];
+                if (remote.updatedAt && overlay.updatedAt && remote.updatedAt >= overlay.updatedAt) {
+                    TASK_WRITE_OVERLAY.delete(key);
+                    continue;
+                }
+                result[index] = { ...remote, ...overlay, id: remote.id || overlay.id, nodeId: remote.nodeId ?? overlay.nodeId, url: remote.url ?? overlay.url, authorLogin: remote.authorLogin ?? overlay.authorLogin };
+            }
+            else {
+                TASK_WRITE_OVERLAY.delete(key);
+            }
+        }
+        return result;
+    }
+    rememberTaskWrite(task, comment, body) { TASK_WRITE_OVERLAY.set(this.taskKey(task), { ...comment, body }); }
     isTrusted(comment) { return comment.authorLogin !== null && this.trustedAuthors.has(comment.authorLogin); }
     repoParts() {
         const [owner, name] = this.repository.split('/', 2);
@@ -61,7 +85,7 @@ export class GitHubStore {
             const pageInfo = comments?.pageInfo;
             cursor = pageInfo?.hasNextPage === true && typeof pageInfo.endCursor === 'string' ? pageInfo.endCursor : null;
         } while (cursor);
-        return all;
+        return this.overlayTaskWrites(all);
     }
     async comments() { return this.list(); }
     async pullRequestState() {
@@ -151,12 +175,31 @@ export class GitHubStore {
     async listAgents() { return (await this.list()).filter(comment => this.isTrusted(comment)).map(x => parseAgent(x.body)).filter((x) => x !== null); }
     async comment(body) {
         const pr = await this.prNode();
-        await this.graphql('mutation($subjectId:ID!,$body:String!){addComment(input:{subjectId:$subjectId,body:$body}){commentEdge{node{id}}}}', { subjectId: pr.id, body });
+        const data = await this.graphql('mutation($subjectId:ID!,$body:String!){addComment(input:{subjectId:$subjectId,body:$body}){commentEdge{node{id databaseId url body updatedAt author{login}}}}}', { subjectId: pr.id, body });
+        const edge = data.addComment?.commentEdge;
+        const node = edge?.node;
+        return {
+            id: Number(node?.databaseId ?? 0),
+            nodeId: typeof node?.id === 'string' ? node.id : undefined,
+            url: typeof node?.url === 'string' ? node.url : undefined,
+            body: typeof node?.body === 'string' ? node.body : body,
+            updatedAt: typeof node?.updatedAt === 'string' ? node.updatedAt : new Date().toISOString(),
+            authorLogin: typeof node?.author?.login === 'string' ? String((node?.author).login) : [...this.trustedAuthors][0] ?? null,
+        };
     }
     async updateComment(comment, body) {
         if (!comment.nodeId)
             throw new Error('GitHub comment node id is required for GraphQL update');
-        await this.graphql('mutation($id:ID!,$body:String!){updateIssueComment(input:{id:$id,body:$body}){issueComment{id}}}', { id: comment.nodeId, body });
+        const data = await this.graphql('mutation($id:ID!,$body:String!){updateIssueComment(input:{id:$id,body:$body}){issueComment{id databaseId url body updatedAt author{login}}}}', { id: comment.nodeId, body });
+        const node = data.updateIssueComment?.issueComment;
+        return {
+            id: Number(node?.databaseId ?? comment.id),
+            nodeId: typeof node?.id === 'string' ? node.id : comment.nodeId,
+            url: typeof node?.url === 'string' ? node.url : comment.url,
+            body: typeof node?.body === 'string' ? node.body : body,
+            updatedAt: typeof node?.updatedAt === 'string' ? node.updatedAt : new Date().toISOString(),
+            authorLogin: typeof node?.author?.login === 'string' ? String((node?.author).login) : comment.authorLogin,
+        };
     }
     async deleteComment(comment) {
         if (!comment.nodeId)
@@ -190,7 +233,9 @@ export class GitHubStore {
             return existing !== null && existing.jobId === task.jobId && existing.id === task.id;
         });
         if (matches.length === 0) {
-            await this.comment(renderTaskComment(task));
+            const body = renderTaskComment(task);
+            const persisted = await this.comment(body);
+            this.rememberTaskWrite(task, persisted, body);
             return;
         }
         const parsed = matches.map(comment => ({ comment, task: parseTask(comment.body) }));
@@ -203,7 +248,9 @@ export class GitHubStore {
         const comments = (await this.list()).filter(comment => this.isTrusted(comment));
         const existing = comments.filter(comment => { const markerTask = parseTask(comment.body); return markerTask !== null && markerTask.jobId === task.jobId && markerTask.id === task.id; });
         if (existing.length > 0) {
-            await this.updateComment(existing[0], renderTaskComment(task));
+            const body = renderTaskComment(task);
+            const persisted = await this.updateComment(existing[0], body);
+            this.rememberTaskWrite(task, persisted, body);
             for (const duplicate of existing.slice(1))
                 await this.deleteComment(duplicate);
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agents-relay",
-  "version": "1.0.116",
+  "version": "1.0.117",
   "description": "Durable async agent jobs coordinated through GitHub pull requests",
   "type": "module",
   "bin": {