pi-mcp-adapter 1.4.1 → 1.5.1

package/CHANGELOG.md

@@ -5,6 +5,29 @@ 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).
 
+## [Unreleased]
+
+### Changed
+- Added `pi-package` keyword for npm discoverability (pi v0.50.0 package system)
+
+## [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

package/README.md

@@ -6,7 +6,7 @@ https://github.com/user-attachments/assets/4b7c66ff-e27e-4639-b195-22c3db406a5a
 
 ## Why This Exists
 
-Mario wrote about [why you might not need MCP](https://mariozechner.at/posts/2025-11-02-what-if-you-dont-need-mcp/). The problem: each MCP server dumps dozens of tools into your context. Playwright alone eats 13k tokens. Connect a few servers and you've burned half your context window before the conversation starts.
+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 take: skip MCP entirely, write simple CLI tools instead.
 
@@ -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

@@ -175,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
@@ -193,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 {
@@ -220,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);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-mcp-adapter",
-  "version": "1.4.1",
+  "version": "1.5.1",
   "description": "MCP (Model Context Protocol) adapter extension for Pi coding agent",
   "type": "module",
   "license": "MIT",
@@ -13,6 +13,7 @@
     "url": "https://github.com/nicobailon/pi-mcp-adapter"
   },
   "keywords": [
+    "pi-package",
     "pi",
     "mcp",
     "model-context-protocol",
@@ -23,7 +24,9 @@
     "llm"
   ],
   "pi": {
-    "extensions": ["./index.ts"]
+    "extensions": [
+      "./index.ts"
+    ]
   },
   "files": [
     "cli.js",