@freelancercom/phabricator-mcp 1.0.2 → 1.0.3

package/dist/tools/coerce.d.ts

@@ -0,0 +1,9 @@
+import { z } from 'zod';
+/**
+ * Coerce a JSON string into an object before Zod validation.
+ *
+ * Some MCP clients may send object parameters as JSON strings when they
+ * haven't loaded the tool schema. This wrapper gracefully handles that
+ * by parsing the string before validation.
+ */
+export declare function jsonCoerce<T extends z.ZodTypeAny>(schema: T): z.ZodEffects<T, T["_output"], unknown>;

package/dist/tools/coerce.js

@@ -0,0 +1,21 @@
+import { z } from 'zod';
+/**
+ * Coerce a JSON string into an object before Zod validation.
+ *
+ * Some MCP clients may send object parameters as JSON strings when they
+ * haven't loaded the tool schema. This wrapper gracefully handles that
+ * by parsing the string before validation.
+ */
+export function jsonCoerce(schema) {
+    return z.preprocess((val) => {
+        if (typeof val === 'string') {
+            try {
+                return JSON.parse(val);
+            }
+            catch {
+                return val;
+            }
+        }
+        return val;
+    }, schema);
+}

package/dist/tools/differential.js

@@ -1,23 +1,24 @@
 import { z } from 'zod';
+import { jsonCoerce } from './coerce.js';
 export function registerDifferentialTools(server, client) {
     // Search revisions
     server.tool('phabricator_revision_search', 'Search Differential revisions (code reviews)', {
         queryKey: z.string().optional().describe('Built-in query: "all", "active", "authored", "waiting"'),
-        constraints: z.object({
-            ids: z.array(z.number()).optional().describe('Revision IDs'),
+        constraints: jsonCoerce(z.object({
+            ids: z.array(z.coerce.number()).optional().describe('Revision IDs'),
             phids: z.array(z.string()).optional().describe('Revision PHIDs'),
             authorPHIDs: z.array(z.string()).optional().describe('Author PHIDs'),
             reviewerPHIDs: z.array(z.string()).optional().describe('Reviewer PHIDs'),
             repositoryPHIDs: z.array(z.string()).optional().describe('Repository PHIDs'),
             statuses: z.array(z.string()).optional().describe('Statuses: needs-review, needs-revision, accepted, published, abandoned, changes-planned'),
-        }).optional().describe('Search constraints'),
-        attachments: z.object({
+        })).optional().describe('Search constraints'),
+        attachments: jsonCoerce(z.object({
             reviewers: z.boolean().optional().describe('Include reviewers'),
             subscribers: z.boolean().optional().describe('Include subscribers'),
             projects: z.boolean().optional().describe('Include projects'),
-        }).optional().describe('Data attachments'),
+        })).optional().describe('Data attachments'),
         order: z.string().optional().describe('Result order'),
-        limit: z.number().max(100).optional().describe('Maximum results'),
+        limit: z.coerce.number().max(100).optional().describe('Maximum results'),
         after: z.string().optional().describe('Pagination cursor'),
     }, async (params) => {
         const result = await client.call('differential.revision.search', params);
@@ -71,7 +72,7 @@ export function registerDifferentialTools(server, client) {
     });
     // Get raw diff content
     server.tool('phabricator_get_raw_diff', 'Get the raw diff/patch content for a Differential diff by diff ID. Use phabricator_diff_search to find the diff ID from a revision PHID first.', {
-        diffID: z.number().describe('The diff ID (numeric, e.g., 1392561). Use phabricator_diff_search to find this from a revision.'),
+        diffID: z.coerce.number().describe('The diff ID (numeric, e.g., 1392561). Use phabricator_diff_search to find this from a revision.'),
     }, async (params) => {
         const result = await client.call('differential.getrawdiff', {
             diffID: params.diffID,
@@ -80,15 +81,15 @@ export function registerDifferentialTools(server, client) {
     });
     // Search diffs
     server.tool('phabricator_diff_search', 'Search Differential diffs', {
-        constraints: z.object({
-            ids: z.array(z.number()).optional().describe('Diff IDs'),
+        constraints: jsonCoerce(z.object({
+            ids: z.array(z.coerce.number()).optional().describe('Diff IDs'),
             phids: z.array(z.string()).optional().describe('Diff PHIDs'),
             revisionPHIDs: z.array(z.string()).optional().describe('Revision PHIDs'),
-        }).optional().describe('Search constraints'),
-        attachments: z.object({
+        })).optional().describe('Search constraints'),
+        attachments: jsonCoerce(z.object({
             commits: z.boolean().optional().describe('Include commit info'),
-        }).optional().describe('Data attachments'),
-        limit: z.number().max(100).optional().describe('Maximum results'),
+        })).optional().describe('Data attachments'),
+        limit: z.coerce.number().max(100).optional().describe('Maximum results'),
         after: z.string().optional().describe('Pagination cursor'),
     }, async (params) => {
         const result = await client.call('differential.diff.search', params);

package/dist/tools/diffusion.js

@@ -1,24 +1,25 @@
 import { z } from 'zod';
+import { jsonCoerce } from './coerce.js';
 export function registerDiffusionTools(server, client) {
     // Search repositories
     server.tool('phabricator_repository_search', 'Search Diffusion repositories', {
         queryKey: z.string().optional().describe('Built-in query: "all", "active"'),
-        constraints: z.object({
-            ids: z.array(z.number()).optional().describe('Repository IDs'),
+        constraints: jsonCoerce(z.object({
+            ids: z.array(z.coerce.number()).optional().describe('Repository IDs'),
             phids: z.array(z.string()).optional().describe('Repository PHIDs'),
             callsigns: z.array(z.string()).optional().describe('Repository callsigns'),
             shortNames: z.array(z.string()).optional().describe('Repository short names'),
             types: z.array(z.string()).optional().describe('VCS types: git, hg, svn'),
             uris: z.array(z.string()).optional().describe('Repository URIs'),
             query: z.string().optional().describe('Full-text search query'),
-        }).optional().describe('Search constraints'),
-        attachments: z.object({
+        })).optional().describe('Search constraints'),
+        attachments: jsonCoerce(z.object({
             uris: z.boolean().optional().describe('Include repository URIs'),
             metrics: z.boolean().optional().describe('Include metrics'),
             projects: z.boolean().optional().describe('Include projects'),
-        }).optional().describe('Data attachments'),
+        })).optional().describe('Data attachments'),
         order: z.string().optional().describe('Result order'),
-        limit: z.number().max(100).optional().describe('Maximum results'),
+        limit: z.coerce.number().max(100).optional().describe('Maximum results'),
         after: z.string().optional().describe('Pagination cursor'),
     }, async (params) => {
         const result = await client.call('diffusion.repository.search', params);
@@ -26,20 +27,20 @@ export function registerDiffusionTools(server, client) {
     });
     // Search commits
     server.tool('phabricator_commit_search', 'Search Diffusion commits', {
-        constraints: z.object({
-            ids: z.array(z.number()).optional().describe('Commit IDs'),
+        constraints: jsonCoerce(z.object({
+            ids: z.array(z.coerce.number()).optional().describe('Commit IDs'),
             phids: z.array(z.string()).optional().describe('Commit PHIDs'),
             repositoryPHIDs: z.array(z.string()).optional().describe('Repository PHIDs'),
             identifiers: z.array(z.string()).optional().describe('Commit identifiers (hashes)'),
             authorPHIDs: z.array(z.string()).optional().describe('Author PHIDs'),
             query: z.string().optional().describe('Full-text search query'),
-        }).optional().describe('Search constraints'),
-        attachments: z.object({
+        })).optional().describe('Search constraints'),
+        attachments: jsonCoerce(z.object({
             projects: z.boolean().optional().describe('Include projects'),
             subscribers: z.boolean().optional().describe('Include subscribers'),
-        }).optional().describe('Data attachments'),
+        })).optional().describe('Data attachments'),
         order: z.string().optional().describe('Result order'),
-        limit: z.number().max(100).optional().describe('Maximum results'),
+        limit: z.coerce.number().max(100).optional().describe('Maximum results'),
         after: z.string().optional().describe('Pagination cursor'),
     }, async (params) => {
         const result = await client.call('diffusion.commit.search', params);

package/dist/tools/maniphest.js

@@ -1,27 +1,28 @@
 import { z } from 'zod';
+import { jsonCoerce } from './coerce.js';
 export function registerManiphestTools(server, client) {
     // Search tasks
     server.tool('phabricator_task_search', 'Search Maniphest tasks with optional filters', {
         queryKey: z.string().optional().describe('Built-in query: "all", "open", "authored", "assigned"'),
-        constraints: z.object({
-            ids: z.array(z.number()).optional().describe('Task IDs to search for'),
+        constraints: jsonCoerce(z.object({
+            ids: z.array(z.coerce.number()).optional().describe('Task IDs to search for'),
             phids: z.array(z.string()).optional().describe('Task PHIDs to search for'),
             assigned: z.array(z.string()).optional().describe('Assigned user PHIDs'),
             authorPHIDs: z.array(z.string()).optional().describe('Author PHIDs'),
             statuses: z.array(z.string()).optional().describe('Task statuses: open, resolved, wontfix, invalid, spite, duplicate'),
-            priorities: z.array(z.number()).optional().describe('Priority levels'),
+            priorities: z.array(z.coerce.number()).optional().describe('Priority levels'),
             subtypes: z.array(z.string()).optional().describe('Task subtypes'),
             columnPHIDs: z.array(z.string()).optional().describe('Workboard column PHIDs'),
             projectPHIDs: z.array(z.string()).optional().describe('Project PHIDs (tasks tagged with these projects)'),
             query: z.string().optional().describe('Full-text search query'),
-        }).optional().describe('Search constraints'),
-        attachments: z.object({
+        })).optional().describe('Search constraints'),
+        attachments: jsonCoerce(z.object({
             columns: z.boolean().optional().describe('Include workboard column info'),
             projects: z.boolean().optional().describe('Include project info'),
             subscribers: z.boolean().optional().describe('Include subscriber info'),
-        }).optional().describe('Data attachments to include'),
+        })).optional().describe('Data attachments to include'),
         order: z.string().optional().describe('Result order: "priority", "updated", "newest", "oldest"'),
-        limit: z.number().max(100).optional().describe('Maximum results (max 100)'),
+        limit: z.coerce.number().max(100).optional().describe('Maximum results (max 100)'),
         after: z.string().optional().describe('Cursor for pagination'),
     }, async (params) => {
         const result = await client.call('maniphest.search', params);

package/dist/tools/paste.js

@@ -1,20 +1,21 @@
 import { z } from 'zod';
+import { jsonCoerce } from './coerce.js';
 export function registerPasteTools(server, client) {
     // Search pastes
     server.tool('phabricator_paste_search', 'Search Phabricator pastes', {
         queryKey: z.string().optional().describe('Built-in query: "all", "authored"'),
-        constraints: z.object({
-            ids: z.array(z.number()).optional().describe('Paste IDs'),
+        constraints: jsonCoerce(z.object({
+            ids: z.array(z.coerce.number()).optional().describe('Paste IDs'),
             phids: z.array(z.string()).optional().describe('Paste PHIDs'),
             authorPHIDs: z.array(z.string()).optional().describe('Author PHIDs'),
             languages: z.array(z.string()).optional().describe('Languages'),
             query: z.string().optional().describe('Full-text search query'),
-        }).optional().describe('Search constraints'),
-        attachments: z.object({
+        })).optional().describe('Search constraints'),
+        attachments: jsonCoerce(z.object({
             content: z.boolean().optional().describe('Include paste content'),
-        }).optional().describe('Data attachments'),
+        })).optional().describe('Data attachments'),
         order: z.string().optional().describe('Result order'),
-        limit: z.number().max(100).optional().describe('Maximum results'),
+        limit: z.coerce.number().max(100).optional().describe('Maximum results'),
         after: z.string().optional().describe('Pagination cursor'),
     }, async (params) => {
         const result = await client.call('paste.search', params);

package/dist/tools/phame.js

@@ -1,15 +1,16 @@
 import { z } from 'zod';
+import { jsonCoerce } from './coerce.js';
 export function registerPhameTools(server, client) {
     // Search blogs
     server.tool('phabricator_blog_search', 'Search Phame blogs', {
         queryKey: z.string().optional().describe('Built-in query: "all", "active"'),
-        constraints: z.object({
-            ids: z.array(z.number()).optional().describe('Blog IDs'),
+        constraints: jsonCoerce(z.object({
+            ids: z.array(z.coerce.number()).optional().describe('Blog IDs'),
             phids: z.array(z.string()).optional().describe('Blog PHIDs'),
             query: z.string().optional().describe('Full-text search query'),
-        }).optional().describe('Search constraints'),
+        })).optional().describe('Search constraints'),
         order: z.string().optional().describe('Result order'),
-        limit: z.number().max(100).optional().describe('Maximum results (max 100)'),
+        limit: z.coerce.number().max(100).optional().describe('Maximum results (max 100)'),
         after: z.string().optional().describe('Cursor for pagination'),
     }, async (params) => {
         const result = await client.call('phame.blog.search', params);
@@ -18,15 +19,15 @@ export function registerPhameTools(server, client) {
     // Search blog posts
     server.tool('phabricator_blog_post_search', 'Search Phame blog posts', {
         queryKey: z.string().optional().describe('Built-in query: "all", "live"'),
-        constraints: z.object({
-            ids: z.array(z.number()).optional().describe('Post IDs'),
+        constraints: jsonCoerce(z.object({
+            ids: z.array(z.coerce.number()).optional().describe('Post IDs'),
             phids: z.array(z.string()).optional().describe('Post PHIDs'),
             blogPHIDs: z.array(z.string()).optional().describe('Filter by blog PHIDs'),
             visibility: z.array(z.string()).optional().describe('Visibility: "published", "draft", "archived"'),
             query: z.string().optional().describe('Full-text search query'),
-        }).optional().describe('Search constraints'),
+        })).optional().describe('Search constraints'),
         order: z.string().optional().describe('Result order'),
-        limit: z.number().max(100).optional().describe('Maximum results (max 100)'),
+        limit: z.coerce.number().max(100).optional().describe('Maximum results (max 100)'),
         after: z.string().optional().describe('Cursor for pagination'),
     }, async (params) => {
         const result = await client.call('phame.post.search', params);

package/dist/tools/phriction.js

@@ -1,21 +1,22 @@
 import { z } from 'zod';
+import { jsonCoerce } from './coerce.js';
 export function registerPhrictionTools(server, client) {
     // Search wiki documents
     server.tool('phabricator_document_search', 'Search Phriction wiki documents', {
         queryKey: z.string().optional().describe('Built-in query: "all", "active"'),
-        constraints: z.object({
-            ids: z.array(z.number()).optional().describe('Document IDs'),
+        constraints: jsonCoerce(z.object({
+            ids: z.array(z.coerce.number()).optional().describe('Document IDs'),
             phids: z.array(z.string()).optional().describe('Document PHIDs'),
             paths: z.array(z.string()).optional().describe('Document paths'),
             ancestorPaths: z.array(z.string()).optional().describe('Ancestor paths to search under'),
             statuses: z.array(z.string()).optional().describe('Document statuses'),
             query: z.string().optional().describe('Full-text search query'),
-        }).optional().describe('Search constraints'),
-        attachments: z.object({
+        })).optional().describe('Search constraints'),
+        attachments: jsonCoerce(z.object({
             content: z.boolean().optional().describe('Include document content'),
-        }).optional().describe('Data attachments'),
+        })).optional().describe('Data attachments'),
         order: z.string().optional().describe('Result order'),
-        limit: z.number().max(100).optional().describe('Maximum results'),
+        limit: z.coerce.number().max(100).optional().describe('Maximum results'),
         after: z.string().optional().describe('Pagination cursor'),
     }, async (params) => {
         const result = await client.call('phriction.document.search', params);

package/dist/tools/project.js

@@ -1,10 +1,11 @@
 import { z } from 'zod';
+import { jsonCoerce } from './coerce.js';
 export function registerProjectTools(server, client) {
     // Search projects
     server.tool('phabricator_project_search', 'Search Phabricator projects', {
         queryKey: z.string().optional().describe('Built-in query: "all", "active", "joined"'),
-        constraints: z.object({
-            ids: z.array(z.number()).optional().describe('Project IDs'),
+        constraints: jsonCoerce(z.object({
+            ids: z.array(z.coerce.number()).optional().describe('Project IDs'),
             phids: z.array(z.string()).optional().describe('Project PHIDs'),
             slugs: z.array(z.string()).optional().describe('Project slugs'),
             name: z.string().optional().describe('Exact name match'),
@@ -14,14 +15,14 @@ export function registerProjectTools(server, client) {
             isMilestone: z.boolean().optional().describe('Filter milestones'),
             isRoot: z.boolean().optional().describe('Filter root projects'),
             query: z.string().optional().describe('Full-text search query'),
-        }).optional().describe('Search constraints'),
-        attachments: z.object({
+        })).optional().describe('Search constraints'),
+        attachments: jsonCoerce(z.object({
             members: z.boolean().optional().describe('Include members'),
             watchers: z.boolean().optional().describe('Include watchers'),
             ancestors: z.boolean().optional().describe('Include ancestors'),
-        }).optional().describe('Data attachments'),
+        })).optional().describe('Data attachments'),
         order: z.string().optional().describe('Result order'),
-        limit: z.number().max(100).optional().describe('Maximum results'),
+        limit: z.coerce.number().max(100).optional().describe('Maximum results'),
         after: z.string().optional().describe('Pagination cursor'),
     }, async (params) => {
         const result = await client.call('project.search', params);
@@ -67,13 +68,13 @@ export function registerProjectTools(server, client) {
     });
     // Search workboard columns
     server.tool('phabricator_column_search', 'Search project workboard columns', {
-        constraints: z.object({
-            ids: z.array(z.number()).optional().describe('Column IDs'),
+        constraints: jsonCoerce(z.object({
+            ids: z.array(z.coerce.number()).optional().describe('Column IDs'),
             phids: z.array(z.string()).optional().describe('Column PHIDs'),
             projects: z.array(z.string()).optional().describe('Project PHIDs'),
-        }).optional().describe('Search constraints'),
+        })).optional().describe('Search constraints'),
         order: z.string().optional().describe('Result order'),
-        limit: z.number().max(100).optional().describe('Maximum results'),
+        limit: z.coerce.number().max(100).optional().describe('Maximum results'),
         after: z.string().optional().describe('Pagination cursor'),
     }, async (params) => {
         const result = await client.call('project.column.search', params);

package/dist/tools/transaction.js

@@ -1,12 +1,13 @@
 import { z } from 'zod';
+import { jsonCoerce } from './coerce.js';
 export function registerTransactionTools(server, client) {
     server.tool('phabricator_transaction_search', 'Search transactions (comments, status changes, etc.) on any Phabricator object (e.g., "D123", "T456")', {
         objectIdentifier: z.string().describe('Object ID (e.g., "D123", "T456") or PHID'),
-        constraints: z.object({
+        constraints: jsonCoerce(z.object({
             phids: z.array(z.string()).optional().describe('Transaction PHIDs'),
             authorPHIDs: z.array(z.string()).optional().describe('Author PHIDs'),
-        }).optional().describe('Search constraints'),
-        limit: z.number().max(100).optional().describe('Maximum results (max 100)'),
+        })).optional().describe('Search constraints'),
+        limit: z.coerce.number().max(100).optional().describe('Maximum results (max 100)'),
         after: z.string().optional().describe('Pagination cursor'),
     }, async (params) => {
         const { objectIdentifier, ...searchParams } = params;

package/dist/tools/user.js

@@ -1,4 +1,5 @@
 import { z } from 'zod';
+import { jsonCoerce } from './coerce.js';
 export function registerUserTools(server, client) {
     // Get current user
     server.tool('phabricator_user_whoami', 'Get information about the current authenticated user', {}, async () => {
@@ -8,8 +9,8 @@ export function registerUserTools(server, client) {
     // Search users
     server.tool('phabricator_user_search', 'Search Phabricator users', {
         queryKey: z.string().optional().describe('Built-in query: "all", "active", "approval"'),
-        constraints: z.object({
-            ids: z.array(z.number()).optional().describe('User IDs'),
+        constraints: jsonCoerce(z.object({
+            ids: z.array(z.coerce.number()).optional().describe('User IDs'),
             phids: z.array(z.string()).optional().describe('User PHIDs'),
             usernames: z.array(z.string()).optional().describe('Usernames'),
             nameLike: z.string().optional().describe('Name prefix search'),
@@ -18,12 +19,12 @@ export function registerUserTools(server, client) {
             isBot: z.boolean().optional().describe('Filter by bot status'),
             isMailingList: z.boolean().optional().describe('Filter by mailing list status'),
             query: z.string().optional().describe('Full-text search query'),
-        }).optional().describe('Search constraints'),
-        attachments: z.object({
+        })).optional().describe('Search constraints'),
+        attachments: jsonCoerce(z.object({
             availability: z.boolean().optional().describe('Include availability info'),
-        }).optional().describe('Data attachments'),
+        })).optional().describe('Data attachments'),
         order: z.string().optional().describe('Result order'),
-        limit: z.number().max(100).optional().describe('Maximum results'),
+        limit: z.coerce.number().max(100).optional().describe('Maximum results'),
         after: z.string().optional().describe('Pagination cursor'),
     }, async (params) => {
         const result = await client.call('user.search', params);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@freelancercom/phabricator-mcp",
-  "version": "1.0.2",
+  "version": "1.0.3",
   "description": "MCP server for Phabricator Conduit API - manage tasks, code reviews, repositories, and more",
   "type": "module",
   "main": "dist/index.js",
@@ -15,8 +15,7 @@
     "prepare": "npm run build",
     "start": "node dist/index.js",
     "dev": "tsx --watch src/index.ts",
-    "typecheck": "tsc --noEmit",
-    "test": "node --test --import tsx 'src/**/*.test.ts'"
+    "typecheck": "tsc --noEmit"
   },
   "keywords": [
     "mcp",