offhands 0.1.5 → 0.1.7

package/approval-mcp.mjs

@@ -1,15 +1,20 @@
-// offhand approval MCP server (plain JS, zero deps — spawned BY claude).
-// Newline-delimited JSON-RPC 2.0 over stdio. Exposes one tool,
-// `approval_prompt`, named via --permission-prompt-tool
-// mcp__offhand__approval_prompt. Each call is forwarded to the daemon's
-// local HTTP endpoint (OFFHAND_APPROVAL_URL) which long-polls the phone's
-// verdict, then this returns allow/deny to claude.
+// offhand MCP toolkit (plain JS, zero deps — spawned BY the agent CLI).
+// Newline-delimited JSON-RPC 2.0 over stdio. Started life as a single tool
+// (`approval_prompt`, still the one named via --permission-prompt-tool
+// mcp__offhand__approval_prompt) — Execution Plan 5 adds `offhand_status_update`
+// alongside it, since a CLI wired in via --mcp-config sees every tool this
+// server exposes, not just the one used as the permission hook. Each call
+// is forwarded to a daemon-local HTTP endpoint (localhost-only by
+// construction); the daemon does the real work, this file is just the wire
+// adapter between JSON-RPC-over-stdio and plain HTTP.
 
 import { createInterface } from 'node:readline';
 
 const APPROVAL_URL = process.env.OFFHAND_APPROVAL_URL ?? 'http://127.0.0.1:4317/approval';
+const STATUS_URL = process.env.OFFHAND_STATUS_URL ?? 'http://127.0.0.1:4317/status';
+const SESSION_ID = process.env.OFFHAND_SESSION_ID ?? '';
 
-const TOOL = {
+const APPROVAL_TOOL = {
   name: 'approval_prompt',
   description: 'Forwards a permission request to the offhand phone client and waits for the verdict.',
   inputSchema: {
@@ -23,6 +28,21 @@ const TOOL = {
   },
 };
 
+const STATUS_TOOL = {
+  name: 'offhand_status_update',
+  description:
+    "Sends a short plain-English progress note to the person's phone, outside the normal response — use it at meaningful checkpoints (starting a long step, hitting a snag, finishing a phase) so they see progress without waiting for the whole run to finish.",
+  inputSchema: {
+    type: 'object',
+    properties: {
+      text: { type: 'string', description: 'A short, human-readable status note.' },
+    },
+    required: ['text'],
+  },
+};
+
+const TOOLS = [APPROVAL_TOOL, STATUS_TOOL];
+
 function send(msg) {
   process.stdout.write(JSON.stringify(msg) + '\n');
 }
@@ -45,6 +65,14 @@ async function callDaemon(args) {
   return res.json();
 }
 
+async function sendStatus(text) {
+  await fetch(STATUS_URL, {
+    method: 'POST',
+    headers: { 'content-type': 'application/json' },
+    body: JSON.stringify({ sessionId: SESSION_ID, text }),
+  });
+}
+
 const rl = createInterface({ input: process.stdin });
 rl.on('line', (line) => {
   if (!line.trim()) return;
@@ -66,14 +94,26 @@ async function handle(msg) {
       reply(id, {
         protocolVersion: params?.protocolVersion ?? '2024-11-05',
         capabilities: { tools: {} },
-        serverInfo: { name: 'offhand-approvals', version: '0.0.1' },
+        serverInfo: { name: 'offhand', version: '0.1.0' },
       });
       return;
     case 'tools/list':
-      reply(id, { tools: [TOOL] });
+      reply(id, { tools: TOOLS });
       return;
     case 'tools/call': {
-      if (params?.name !== TOOL.name) {
+      if (params?.name === STATUS_TOOL.name) {
+        const text = String(params.arguments?.text ?? '').trim();
+        try {
+          if (text) await sendStatus(text);
+          reply(id, { content: [{ type: 'text', text: text ? 'sent' : 'nothing to send (empty text)' }] });
+        } catch (e) {
+          // A dropped status note is never fatal to the run — it's a nice-
+          // to-have channel, not a control one like approvals.
+          reply(id, { content: [{ type: 'text', text: `status channel error: ${e?.message ?? e}` }] });
+        }
+        return;
+      }
+      if (params?.name !== APPROVAL_TOOL.name) {
         replyError(id, -32602, `unknown tool: ${params?.name}`);
         return;
       }