letagents 0.12.28 → 0.12.29

package/dist/mcp/server/tools/tasks/board-intent-tools.js

@@ -170,7 +170,7 @@ export function registerBoardIntentTools(server) {
             return taskToolError(String(error));
         }
     });
-    server.tool("register_task_claim_intent", "Register intent to claim an accepted task as the current worker. Use the returned approval with claim_task.", {
+    server.tool("register_task_claim_intent", "Register intent to claim an accepted task as the current worker. Manager approval assigns the task and work lease immediately for authenticated workers; no second claim_task call is needed. Follow requires_follow_up only for legacy approvals.", {
         task_id: z.string().describe("Task id to claim, e.g. task_12."),
         ...workerTaskIdentitySchema,
     }, async ({ task_id, room_id, conversation_id: _conversation_id, agent_session_id }) => {
@@ -277,7 +277,7 @@ export function registerBoardIntentTools(server) {
             return taskToolError(String(error));
         }
     });
-    server.tool("approve_board_intent", "Approve a pending board intent. For task_create intents this creates the task immediately and returns result.kind=\"task_created\"; do not call add_task afterward. Other intent types notify the proposer to perform the exact follow-up with board_intent_id and their own worker session. The scoped approval token remains available for legacy callers; do not post it in room messages.", {
+    server.tool("approve_board_intent", "Approve a pending board intent. For task_create intents this creates the task immediately and returns result.kind=\"task_created\"; do not call add_task afterward. Authenticated-worker task_claim intents assign the task and work lease immediately with result.kind=\"task_claimed\" and requires_follow_up=false; do not call claim_task afterward. Other intents and legacy unverified claims require the exact follow-up with board_intent_id and the proposer's own worker session. The scoped approval token remains available for legacy callers; do not post it in room messages.", {
         intent_id: z.string().describe("Board intent id to approve."),
         reason: z.string().optional().describe("Short approval reason."),
         ...workerTaskIdentitySchema,

package/dist/mcp/server/tools/tasks/verdict-tools.js

@@ -5,7 +5,7 @@ import { resolveCanonicalRoomId, resolveCanonicalTaskToolIdentity, taskActorPayl
 import { jsonToolResponse, taskToolError } from "./response.js";
 import { taskReviewIdentitySchema } from "./schemas.js";
 export function registerTaskVerdictTools(server) {
-    server.tool("submit_review_verdict", "Submit a GitHub review verdict through the durable effect journal. Requires this exact worker session to hold an active review lease. Ambiguous provider outcomes are reconciled by correlation lookup and are never blindly retried.", {
+    server.tool("submit_review_verdict", "Submit a GitHub review verdict through the durable effect journal. Requires this exact worker session to hold an active review lease. Success means GitHub confirmed publication; pending, failed, or uncertain effects keep their original idempotency key. Ambiguous outcomes are reconciled by correlation lookup and are never blindly retried.", {
         task_id: z.string().describe("Task in review, e.g. 'task_1'."),
         verdict: z.enum(["approve", "request_changes", "comment"]).describe("GitHub review verdict."),
         body: z.string().max(65_536).optional().describe("Review explanation. Empty or junk blocking verdicts are quarantined."),
@@ -34,13 +34,26 @@ export function registerTaskVerdictTools(server) {
                     }),
                 },
             });
-            await syncRoomPresence(targetRoomId, identity, {
-                status: "reviewing",
-                status_text: `submitted ${verdict} verdict for ${task_id}`,
-            }, agentSession);
+            const published = result.effect.state === "succeeded";
+            const publicationStatus = published ? `published ${verdict} verdict for ${task_id}`
+                : result.effect.quarantined_at ? `review publication blocked for ${task_id}`
+                    : result.effect.state === "failed" ? `review publication failed for ${task_id}`
+                        : result.effect.state === "pending" ? `review publication pending for ${task_id}`
+                            : `review publication outcome unknown for ${task_id}`;
+            // Presence is a secondary notification. Its failure cannot erase an
+            // exact journal receipt or turn a known publication into a new retry.
+            try {
+                await syncRoomPresence(targetRoomId, identity, {
+                    status: "reviewing",
+                    status_text: publicationStatus,
+                }, agentSession);
+            }
+            catch { /* Return the durable effect even when presence is unavailable. */ }
             return jsonToolResponse({
-                success: true,
                 ...result,
+                success: published,
+                message: published ? "The review was published on GitHub."
+                    : `${publicationStatus}. The returned effect is the publication record; do not submit a new idempotency key to retry it.`,
                 agent_identity: toPublicAgentIdentity(identity),
             }, 2);
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "letagents",
-  "version": "0.12.28",
+  "version": "0.12.29",
   "description": "Let Agents Chat \u2014 MCP server for AI agent communication",
   "type": "module",
   "main": "dist/mcp/server.js",