@commonlyai/mcp 0.3.18 → 0.3.20

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@commonlyai/mcp",
   "mcpName": "io.github.Team-Commonly/commonly",
-  "version": "0.3.18",
+  "version": "0.3.20",
   "license": "Apache-2.0",
   "repository": {
     "type": "git",

package/src/tools.js

@@ -58,6 +58,14 @@ const reqWith = (props, requiredKeys) => ({ ...required(props), required: requir
 
 const STRING = { type: 'string' };
 const INT = { type: 'integer' };
+const normalizePullRequestUrl = (value) => {
+  const url = typeof value === 'string' ? value.split(/[?#]/, 1)[0] : '';
+  const match = /^https:\/\/github\.com\/[A-Za-z0-9_.-]+\/[A-Za-z0-9_.-]+\/pull\/[1-9]\d*$/.exec(url);
+  if (!match || match[0] !== url || new URL(url).href !== url) {
+    throw new Error('prUrl must be https://github.com/<owner>/<repo>/pull/<n>');
+  }
+  return url;
+};
 const CLAIM_OUTCOME = { type: 'string', enum: ['declined', 'completed'] };
 const DECISION_CLASS = { type: 'string', enum: ['strategy', 'implementation', 'prioritization'] };
 const DECISION_OPTION = {
@@ -207,6 +215,22 @@ export const buildTools = (config) => {
         body: outcome === undefined ? undefined : { outcome },
       })),
     },
+    {
+      name: 'commonly_checkpoint_event',
+      description: 'Checkpoint ONE inbox event only after tool-side work is committed (including a reply posted through a tool), or a deliberate NO_REPLY. Never checkpoint an answer held for final CLI output: the wrapper must post that reply before acknowledging the item. This durably acknowledges that event now, so a later turn timeout does not replay it. Use only the eventId and deliveryId supplied together in the current inbox item. Never checkpoint unseen or unfinished work. Release any message claim with its correct completed/declined outcome first. A failed call is not a checkpoint; leave the item pending. This posts no chat message and does not complete a task.',
+      inputSchema: reqWith({ eventId: { type: 'string', minLength: 1 }, deliveryId: { type: 'string', minLength: 1 } }, ['eventId', 'deliveryId']),
+      call: wrap(async ({ eventId, deliveryId }) => {
+        if (typeof eventId !== 'string' || !eventId.trim()
+          || typeof deliveryId !== 'string' || !deliveryId.trim()) {
+          throw new Error('eventId and deliveryId from the current inbox item are required');
+        }
+        return request(config, {
+          method: 'POST',
+          path: `/api/agents/runtime/events/${encodeURIComponent(eventId)}/ack`,
+          body: { deliveryId, result: { outcome: 'acknowledged', reason: 'handled-checkpoint' } },
+        });
+      }),
+    },
     {
       name: 'commonly_get_messages',
       description: 'Read chat messages from a pod. `limit` is clamped server-side to [1, 50] (default 20). To page into older history, pass the `createdAt` timestamp of the oldest message as `before`; the response `hasMore` flag distinguishes another page from the end of history. Pass `threadRootId` (a message id) to read one thread — returns the root message plus every reply in that thread, oldest first.',
@@ -409,19 +433,27 @@ export const buildTools = (config) => {
     },
     {
       name: 'commonly_update_task',
-      description: 'Append an update note and optionally set priority (P0 highest through P3), without changing status — visible in the task drawer history. If a 409 reports unseen messages, read them and retry, or pass continueAnyway:true to update despite them.',
+      description: 'Append an update note and optionally set priority (P0 highest through P3), without changing status — visible in the task drawer history. Optionally set prUrl to a GitHub pull request URL before appending the note. If a 409 reports unseen messages, read them and retry, or pass continueAnyway:true to update despite them.',
       inputSchema: reqWith({
         podId: STRING,
         taskId: STRING,
         text: STRING,
         priority: { type: 'string', enum: ['P0', 'P1', 'P2', 'P3'] },
+        prUrl: STRING,
         continueAnyway: { type: 'boolean' },
       }, ['podId', 'taskId', 'text']),
-      call: wrap(async ({ podId, taskId, text, priority, continueAnyway }) => request(config, {
-        method: 'POST',
-        path: `/api/v1/tasks/${encodeURIComponent(podId)}/${encodeURIComponent(taskId)}/updates`,
-        body: { text, priority, continueAnyway },
-      })),
+      call: wrap(async ({ podId, taskId, text, priority, prUrl, continueAnyway }) => {
+        const taskPath = `/api/v1/tasks/${encodeURIComponent(podId)}/${encodeURIComponent(taskId)}`;
+        if (prUrl !== undefined) {
+          const normalized = normalizePullRequestUrl(prUrl);
+          await request(config, { method: 'PATCH', path: taskPath, body: { prUrl: normalized } });
+        }
+        return request(config, {
+          method: 'POST',
+          path: `${taskPath}/updates`,
+          body: { text, priority, continueAnyway },
+        });
+      }),
     },
     {
       name: 'commonly_create_pod',