pi-mcp-adapter 1.4.0 → 1.5.0

package/CHANGELOG.md

@@ -5,6 +5,30 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.5.0] - 2026-01-22
+
+### Changed
+- **BREAKING: `args` parameter is now a JSON string** - The `args` parameter which previously accepted an object now accepts a JSON string. This change was required for compatibility with Claude's Vertex AI API (`google-antigravity` provider) which rejects `patternProperties` in JSON schemas (generated by `Type.Record()`).
+
+### Added
+- **Type validation for args** - Parsed args are now validated to ensure they're a JSON object (not null, array, or primitive). Clear error messages for invalid input.
+- **`isError: true` on error responses** - JSON parse errors and type validation errors now properly set `isError: true` to indicate failure to the LLM.
+
+### Migration
+```typescript
+// Before (1.4.x)
+mcp({ tool: "my_tool", args: { key: "value" } })
+
+// After (1.5.0)
+mcp({ tool: "my_tool", args: '{"key": "value"}' })
+```
+
+## [1.4.1] - 2026-01-19
+
+### Changed
+
+- Status bar shows server count instead of tool count ("MCP: 5 servers")
+
 ## [1.4.0] - 2026-01-19
 
 ### Changed

package/README.md

@@ -6,11 +6,11 @@ https://github.com/user-attachments/assets/4b7c66ff-e27e-4639-b195-22c3db406a5a
 
 ## Why This Exists
 
-Mario (Pi's creator) wrote about [why you might not need MCP](https://mariozechner.at/posts/2025-11-02-what-if-you-dont-need-mcp/). The core issue: MCP servers ship dozens of tools with verbose descriptions. Playwright MCP has 21 tools eating 13.7k tokens. Chrome DevTools MCP has 26 tools eating 18k tokens. Connect a few servers and you've burned 30-50k tokens before the conversation starts. That's context bloat, slower responses, higher costs, and confused agents drowning in tool options.
+Mario wrote about [why you might not need MCP](https://mariozechner.at/posts/2025-11-02-what-if-you-dont-need-mcp/). The problem: tool definitions are verbose. A single MCP server can burn 10k+ tokens, and you're paying that cost whether you use those tools or not. Connect a few servers and you've burned half your context window before the conversation starts.
 
-His solution: skip MCP, write simple CLI tools.
+His take: skip MCP entirely, write simple CLI tools instead.
 
-But there's an active ecosystem of well-maintained MCP servers for databases, browsers, APIs, file systems, and more. This adapter lets you tap into that ecosystem without the context cost. One proxy tool (~200 tokens) instead of hundreds of individual tool definitions. The LLM discovers what it needs on-demand.
+But the MCP ecosystem has useful stuff - databases, browsers, APIs. This adapter gives you access without the bloat. One proxy tool (~200 tokens) instead of hundreds. The agent discovers what it needs on-demand.
 
 ## Install
 
@@ -50,9 +50,11 @@ chrome_devtools_take_screenshot
     fullPage (boolean) - Full page instead of viewport
 ```
 ```
-mcp({ tool: "chrome_devtools_take_screenshot", args: { format: "png" } })
+mcp({ tool: "chrome_devtools_take_screenshot", args: '{"format": "png"}' })
 ```
 
+Note: `args` is a JSON string, not an object.
+
 Two calls instead of 26 tools cluttering the context.
 
 ## Config
@@ -102,7 +104,7 @@ Supported: `cursor`, `claude-code`, `claude-desktop`, `vscode`, `windsurf`, `cod
 |------|---------|
 | Search | `mcp({ search: "screenshot navigate" })` |
 | Describe | `mcp({ describe: "tool_name" })` |
-| Call | `mcp({ tool: "...", args: {...} })` |
+| Call | `mcp({ tool: "...", args: '{"key": "value"}' })` |
 | Status | `mcp({ })` or `mcp({ server: "name" })` |
 
 Search includes parameter schemas by default. Space-separated words are OR'd.

package/index.ts

@@ -71,9 +71,10 @@ export default function mcpAdapter(pi: ExtensionAPI) {
       
       // Update status bar when ready
       if (ctx.hasUI) {
-        const totalTools = [...s.registeredTools.values()].flat().length;
-        if (totalTools > 0) {
-          ctx.ui.setStatus("mcp", ctx.ui.theme.fg("accent", `MCP: ${totalTools} tools`));
+        const serverCount = s.registeredTools.size;
+        if (serverCount > 0) {
+          const label = serverCount === 1 ? "server" : "servers";
+          ctx.ui.setStatus("mcp", ctx.ui.theme.fg("accent", `MCP: ${serverCount} ${label}`));
         } else {
           ctx.ui.setStatus("mcp", "");
         }
@@ -174,13 +175,13 @@ Usage:
   mcp({ server: "name" })               → List tools from server
   mcp({ search: "query" })              → Search for tools (includes schemas, space-separated words OR'd)
   mcp({ describe: "tool_name" })        → Show tool details and parameters
-  mcp({ tool: "name", args: {...} })    → Call a tool
+  mcp({ tool: "name", args: '{"key": "value"}' })    → Call a tool (args is JSON string)
 
 Mode: tool (call) > describe > search > server (list) > nothing (status)`,
     parameters: Type.Object({
       // Call mode
       tool: Type.Optional(Type.String({ description: "Tool name to call (e.g., 'xcodebuild_list_sims')" })),
-      args: Type.Optional(Type.Record(Type.String(), Type.Unknown(), { description: "Arguments for tool call" })),
+      args: Type.Optional(Type.String({ description: "Arguments as JSON string (e.g., '{\"key\": \"value\"}')" })),
       // Describe mode
       describe: Type.Optional(Type.String({ description: "Tool name to describe (shows parameters)" })),
       // Search mode
@@ -192,13 +193,35 @@ Mode: tool (call) > describe > search > server (list) > nothing (status)`,
     }),
     async execute(_toolCallId, params: {
       tool?: string;
-      args?: Record<string, unknown>;
+      args?: string;
       describe?: string;
       search?: string;
       regex?: boolean;
       includeSchemas?: boolean;
       server?: string;
     }) {
+      // Parse args from JSON string if provided
+      let parsedArgs: Record<string, unknown> | undefined;
+      if (params.args) {
+        try {
+          parsedArgs = JSON.parse(params.args);
+          if (typeof parsedArgs !== "object" || parsedArgs === null || Array.isArray(parsedArgs)) {
+            const gotType = Array.isArray(parsedArgs) ? "array" : parsedArgs === null ? "null" : typeof parsedArgs;
+            return {
+              content: [{ type: "text", text: `Invalid args: expected a JSON object, got ${gotType}` }],
+              isError: true,
+              details: { error: "invalid_args_type" },
+            };
+          }
+        } catch (e) {
+          return {
+            content: [{ type: "text", text: `Invalid args JSON: ${e instanceof Error ? e.message : e}` }],
+            isError: true,
+            details: { error: "invalid_args" },
+          };
+        }
+      }
+      
       // Wait for init if still in progress
       if (!state && initPromise) {
         try {
@@ -219,7 +242,7 @@ Mode: tool (call) > describe > search > server (list) > nothing (status)`,
       
       // Mode resolution: tool > describe > search > server > status
       if (params.tool) {
-        return executeCall(state, params.tool, params.args);
+        return executeCall(state, params.tool, parsedArgs);
       }
       if (params.describe) {
         return executeDescribe(state, params.describe);
@@ -901,11 +924,12 @@ async function reconnectServers(
     }
   }
   
-  // Update status bar with new tool count
+  // Update status bar with server count
   if (ctx.hasUI) {
-    const totalTools = [...state.registeredTools.values()].flat().length;
-    if (totalTools > 0) {
-      ctx.ui.setStatus("mcp", ctx.ui.theme.fg("accent", `MCP: ${totalTools} tools`));
+    const serverCount = state.registeredTools.size;
+    if (serverCount > 0) {
+      const label = serverCount === 1 ? "server" : "servers";
+      ctx.ui.setStatus("mcp", ctx.ui.theme.fg("accent", `MCP: ${serverCount} ${label}`));
     } else {
       ctx.ui.setStatus("mcp", "");
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-mcp-adapter",
-  "version": "1.4.0",
+  "version": "1.5.0",
   "description": "MCP (Model Context Protocol) adapter extension for Pi coding agent",
   "type": "module",
   "license": "MIT",