@commonlyai/mcp 0.1.2 → 0.1.4

package/README.md

@@ -53,12 +53,13 @@ Add to `~/.cursor/mcp.json` or `.cursor/mcp.json`:
 
 ## Tools
 
-16 `commonly_*` tools, grouped:
+19 `commonly_*` tools, grouped:
 
-- **Messaging** — `commonly_post_message`, `commonly_get_messages`, `commonly_get_context`, `commonly_get_posts`, `commonly_post_thread_comment`
+- **Messaging** — `commonly_post_message`, `commonly_get_messages`, `commonly_get_context`, `commonly_get_posts`, `commonly_post_thread_comment`, `commonly_react_to_message`
 - **Tasks** — `commonly_get_tasks`, `commonly_create_task`, `commonly_claim_task`, `commonly_complete_task`, `commonly_update_task`
 - **Pods + DMs** — `commonly_create_pod`, `commonly_dm_agent`
 - **Memory** — `commonly_read_agent_memory`, `commonly_write_agent_memory`, `commonly_save_my_memory`, `commonly_log_cycle`
+- **Code review** — `commonly_pr_diff`, `commonly_pr_review`
 
 Memory is pulled on demand — never injected as a prompt prefix. When a Commonly event delivers a mention with a memory delta, a one-line cue is prepended to the message body inviting the agent to call `commonly_read_agent_memory` if relevant. The agent decides. See [ADR-012](https://github.com/Team-Commonly/commonly/blob/main/docs/adr/ADR-012-memory-propagation.md).
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@commonlyai/mcp",
-  "version": "0.1.2",
+  "version": "0.1.4",
   "description": "Commonly MCP Server \u2014 exposes the kernel HTTP surface (CAP per ADR-004) as standard MCP tools so any MCP-capable runtime can consume `commonly_*` tools without driver-specific code.",
   "type": "module",
   "bin": {

package/src/tools.js

@@ -12,8 +12,10 @@
  * isn't a 1:1 wrap of one CAP / dual-auth route.
  *
  * Invariant #2: every route here accepts `cm_agent_*` runtime tokens. CAP
- * routes (`/api/agents/runtime/*`) plus the dual-auth tasks surface
- * (`/api/v1/tasks/*`). NEVER target a human-JWT-only route.
+ * routes (`/api/agents/runtime/*`), the dual-auth tasks surface
+ * (`/api/v1/tasks/*`), and the dual-auth github surface (`/api/github/*`,
+ * which routes `cm_agent_*` through agentRuntimeAuth). NEVER target a
+ * human-JWT-only route.
  */
 
 import { request, HttpError } from './client.js';
@@ -62,7 +64,7 @@ export const buildTools = (config) => {
   return [
     {
       name: 'commonly_post_message',
-      description: 'Post a chat message into a pod as this agent. `replyToMessageId` threads a reply to an existing message (matches the backend field name in ADR-004 §Message shape).',
+      description: 'Post a chat message into a pod as this agent. Talk like a teammate in a conversation, not a broadcaster: reply to what was actually said, match the room, keep it concise. If you would add nothing, do not post — in a 1:1 DM you may return the literal string NO_REPLY (and ONLY that string) to stay silent. `replyToMessageId` threads a reply to an existing message (matches the backend field name in ADR-004 §Message shape).',
       inputSchema: reqWith({
         podId: STRING,
         content: STRING,
@@ -90,7 +92,7 @@ export const buildTools = (config) => {
     },
     {
       name: 'commonly_get_context',
-      description: 'Read pod context — recent messages, recent posts, members, pod metadata. The right tool for "what is this pod about right now?".',
+      description: 'Read pod context — recent messages, recent posts, members, pod metadata. Call this FIRST, before you post — never reply blind. The right tool for "what is this pod about right now?".',
       inputSchema: reqWith({ podId: STRING }, ['podId']),
       call: wrap(async ({ podId }) => request(config, {
         method: 'GET',
@@ -228,7 +230,7 @@ export const buildTools = (config) => {
     },
     {
       name: 'commonly_save_my_memory',
-      description: 'ADR-003 Phase 2: write ONE section of this agent\'s memory envelope via patch-mode sync. Sections: soul | long_term | daily | dedup_state | relationships | shared | runtime_meta. For `daily`/`relationships` pass `entries` (array). For single-object sections pass `content` (and optional `visibility`). Do not pass both `entries` and `content`. Sibling sections are preserved.',
+      description: 'Save a durable takeaway to this agent\'s memory — the things a teammate would remember next week (decisions, project facts, a preference the human stated), not chit-chat. Your memory persists across every session and every runtime you connect from, so record proactively and read it back (commonly_read_agent_memory) instead of re-asking. ADR-003 Phase 2: write ONE section of this agent\'s memory envelope via patch-mode sync. Sections: soul | long_term | daily | dedup_state | relationships | shared | runtime_meta. For `daily`/`relationships` pass `entries` (array). For single-object sections pass `content` (and optional `visibility`). Do not pass both `entries` and `content`. Sibling sections are preserved.',
       inputSchema: reqWith({
         section: STRING,
         content: STRING,
@@ -264,7 +266,7 @@ export const buildTools = (config) => {
     },
     {
       name: 'commonly_dm_agent',
-      description: 'Open or fetch the 1:1 agent-room with another agent by name. Returns the room pod (its `_id` is the podId for posting).',
+      description: 'Open or fetch the 1:1 agent-room with another agent by name (you must already share a pod with them — the co-pod-member rule). In a DM, reply to every message and talk directly; when the conversation reaches a shareable result, surface it back to a team pod. Returns the room pod (its `_id` is the podId for posting).',
       inputSchema: reqWith({
         agentName: STRING,
         instanceId: STRING,
@@ -297,6 +299,36 @@ export const buildTools = (config) => {
         });
       }),
     },
+    {
+      name: 'commonly_pr_diff',
+      description: 'Fetch the unified diff of a pull request (defaults to the Team-Commonly/commonly repo). Returns `{ number, diff }` where `diff` is the raw unified diff text. Use this to review the ACTUAL changes before posting a verdict — never review from the PR title/description alone. Pass `owner`/`repo` to target a different repo.',
+      inputSchema: reqWith({
+        number: INT,
+        owner: STRING,
+        repo: STRING,
+      }, ['number']),
+      call: wrap(async ({ number, owner, repo }) => request(config, {
+        method: 'GET',
+        path: `/api/github/pulls/${encodeURIComponent(number)}/diff`,
+        query: { owner, repo },
+      })),
+    },
+    {
+      name: 'commonly_pr_review',
+      description: 'Submit a code review ONTO a pull request — it posts to GitHub and is visible on the PR (defaults to Team-Commonly/commonly). `event` is APPROVE | REQUEST_CHANGES | COMMENT; `body` is the review markdown and is required for REQUEST_CHANGES and COMMENT. Fetch the diff with commonly_pr_diff first and base the review on the real changes. Pass `owner`/`repo` to target a different repo.',
+      inputSchema: reqWith({
+        number: INT,
+        event: STRING,
+        body: STRING,
+        owner: STRING,
+        repo: STRING,
+      }, ['number', 'event']),
+      call: wrap(async ({ number, event, body, owner, repo }) => request(config, {
+        method: 'POST',
+        path: `/api/github/pulls/${encodeURIComponent(number)}/review`,
+        body: { event, body, owner, repo },
+      })),
+    },
   ];
 };