@freelancercom/phabricator-mcp 2.0.14 → 2.0.16

package/dist/tools/differential.js

@@ -12,7 +12,6 @@ export function registerDifferentialTools(server, client) {
             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, draft'),
             responsiblePHIDs: z.array(z.string()).optional().describe('User PHIDs who are responsible (as author or reviewer)'),
-            affectedPaths: z.array(z.string()).optional().describe('File paths affected by the revision'),
             createdStart: z.coerce.number().optional().describe('Created after (epoch timestamp)'),
             createdEnd: z.coerce.number().optional().describe('Created before (epoch timestamp)'),
             modifiedStart: z.coerce.number().optional().describe('Modified after (epoch timestamp)'),

package/dist/tools/diffusion.js

@@ -11,8 +11,6 @@ export function registerDiffusionTools(server, client) {
             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'),
-            status: z.string().optional().describe('Repository status: "open" (active) or "closed" (inactive)'),
-            hosted: z.string().optional().describe('Hosting: "phabricator" (hosted) or "remote"'),
             projects: z.array(z.string()).optional().describe('Project PHIDs'),
             spaces: z.array(z.string()).optional().describe('Space PHIDs (for multi-space installations)'),
             query: z.string().optional().describe('Full-text search query'),
@@ -53,7 +51,6 @@ export function registerDiffusionTools(server, client) {
         attachments: jsonCoerce(z.object({
             projects: z.boolean().optional().describe('Include projects'),
             subscribers: z.boolean().optional().describe('Include subscribers'),
-            auditors: z.boolean().optional().describe('Include auditor info'),
         })).optional().describe('Data attachments'),
         order: z.string().optional().describe('Result order'),
         limit: z.coerce.number().max(100).optional().describe('Maximum results (max 100)'),
@@ -64,11 +61,10 @@ export function registerDiffusionTools(server, client) {
         return { content: [{ type: 'text', text: JSON.stringify(result, null, 2) }] };
     });
     // Browse repository file tree
-    server.tool('phabricator_repository_browse', 'Browse a repository directory tree at a given path and commit/branch', {
+    server.tool('phabricator_repository_browse', 'Browse a repository directory tree at a given path and commit/branch. Pass a branch name via the commit parameter.', {
         path: z.string().optional().describe('Path to browse (default: "/")'),
         repository: z.string().optional().describe('Repository callsign, short name, or PHID'),
-        commit: z.string().optional().describe('Commit hash or branch name (default: HEAD)'),
-        branch: z.string().optional().describe('Branch name'),
+        commit: z.string().optional().describe('Commit hash or branch name (default: HEAD). Pass branch names here.'),
         needValidityOnly: z.boolean().optional().describe('Only check path validity without loading the full tree'),
         limit: z.coerce.number().optional().describe('Maximum entries to return'),
         offset: z.coerce.number().optional().describe('Result offset for pagination'),
@@ -158,7 +154,7 @@ export function registerDiffusionTools(server, client) {
         offset: z.coerce.number().optional().describe('Result offset for pagination'),
     }, async (params) => {
         const result = await client.call('diffusion.searchquery', {
-            path: params.path ?? '/',
+            path: params.path ?? '',
             repository: params.repository,
             grep: params.query,
             commit: params.commit,

package/dist/tools/harbormaster.js

@@ -49,13 +49,6 @@ export function registerHarbormasterTools(server, client) {
             ids: z.array(z.coerce.number()).optional().describe('Target IDs'),
             phids: z.array(z.string()).optional().describe('Target PHIDs'),
             buildPHIDs: z.array(z.string()).optional().describe('Build PHIDs'),
-            statuses: z.array(z.string()).optional().describe('Target statuses'),
-            createdStart: z.coerce.number().optional().describe('Created after (epoch timestamp)'),
-            createdEnd: z.coerce.number().optional().describe('Created before (epoch timestamp)'),
-            startedStart: z.coerce.number().optional().describe('Started executing after (epoch timestamp)'),
-            startedEnd: z.coerce.number().optional().describe('Started executing before (epoch timestamp)'),
-            completedStart: z.coerce.number().optional().describe('Completed after (epoch timestamp)'),
-            completedEnd: z.coerce.number().optional().describe('Completed before (epoch timestamp)'),
         })).optional().describe('Search constraints'),
         order: z.string().optional().describe('Result order'),
         limit: z.coerce.number().max(100).optional().describe('Maximum results (max 100)'),
@@ -82,9 +75,9 @@ export function registerHarbormasterTools(server, client) {
         return { content: [{ type: 'text', text: JSON.stringify(result, null, 2) }] };
     });
     // Send build command
-    server.tool('phabricator_build_command', 'Send a command to Harbormaster. For build targets: report status (pass/fail/work) with optional unit/lint results. For builds/buildables: send control commands (pause/resume/abort/restart).', {
-        receiver: z.string().describe('PHID of build target, build, or buildable to send the message to'),
-        type: z.enum(['pass', 'fail', 'work', 'pause', 'resume', 'abort', 'restart']).describe('Message type: "pass"/"fail"/"work" for build targets; "pause"/"resume"/"abort"/"restart" for builds/buildables'),
+    server.tool('phabricator_build_command', 'Send a command to a Harbormaster build target. Report build status (pass/fail/work) with optional unit test and lint results.', {
+        buildTargetPHID: z.string().describe('PHID of the build target (PHID-HMBT-...) to send the message to'),
+        type: z.enum(['pass', 'fail', 'work']).describe('Message type: "pass" (target passed), "fail" (target failed), "work" (target still working, resets timeout)'),
         unit: jsonCoerce(z.array(z.object({
             name: z.string().describe('Test name'),
             result: z.string().describe('Result: "pass", "fail", "skip", "broken", "unsound"'),
@@ -107,7 +100,7 @@ export function registerHarbormasterTools(server, client) {
         }))).optional().describe('Lint results to report'),
     }, async (params) => {
         const apiParams = {
-            receiver: params.receiver,
+            buildTargetPHID: params.buildTargetPHID,
             type: params.type,
         };
         if (params.unit !== undefined) {

package/dist/tools/phame.js

@@ -35,7 +35,6 @@ export function registerPhameTools(server, client) {
         status: z.string().optional().describe('Blog status'),
         addSubscriberPHIDs: z.array(z.string()).optional().describe('Subscriber PHIDs to add'),
         removeSubscriberPHIDs: z.array(z.string()).optional().describe('Subscriber PHIDs to remove'),
-        comment: z.string().optional().describe('Add a comment alongside the edit (supports Remarkup)'),
     }, async (params) => {
         const transactions = [];
         if (params.name !== undefined) {
@@ -65,9 +64,6 @@ export function registerPhameTools(server, client) {
         if (params.removeSubscriberPHIDs !== undefined) {
             transactions.push({ type: 'subscribers.remove', value: params.removeSubscriberPHIDs });
         }
-        if (params.comment !== undefined) {
-            transactions.push({ type: 'comment', value: params.comment });
-        }
         if (transactions.length === 0) {
             return { content: [{ type: 'text', text: 'No changes specified' }] };
         }

package/dist/tools/project.js

@@ -3,7 +3,7 @@ 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", "watching"'),
+        queryKey: z.string().optional().describe('Built-in query: "all", "active" (non-archived), "joined", "watching"'),
         constraints: jsonCoerce(z.object({
             ids: z.array(z.coerce.number()).optional().describe('Project IDs'),
             phids: z.array(z.string()).optional().describe('Project PHIDs'),
@@ -13,7 +13,6 @@ export function registerProjectTools(server, client) {
             watchers: z.array(z.string()).optional().describe('Watcher user PHIDs'),
             ancestors: z.array(z.string()).optional().describe('Ancestor project PHIDs'),
             parents: z.array(z.string()).optional().describe('Parent project PHIDs (find subprojects)'),
-            status: z.string().optional().describe('Project status: "active" (default), "archived"'),
             icons: z.array(z.string()).optional().describe('Filter by project icon'),
             isMilestone: z.boolean().optional().describe('Filter milestones'),
             isRoot: z.boolean().optional().describe('Filter root projects'),
@@ -41,6 +40,7 @@ export function registerProjectTools(server, client) {
     server.tool('phabricator_project_edit', 'Create or edit a Phabricator project. Omit objectIdentifier to create a new project (name is required for creation).', {
         objectIdentifier: z.string().optional().describe('Project PHID or ID. Omit to create a new project.'),
         name: z.string().optional().describe('New name'),
+        description: z.string().optional().describe('Project description (supports Remarkup)'),
         icon: z.string().optional().describe('New icon'),
         color: z.string().optional().describe('New color'),
         addMemberPHIDs: z.array(z.string()).optional().describe('Add members'),
@@ -49,12 +49,14 @@ export function registerProjectTools(server, client) {
         parent: z.string().optional().describe('Parent project PHID (to create as a subproject)'),
         milestone: z.string().optional().describe('Parent project PHID (to create as a milestone of that project)'),
         slug: z.string().optional().describe('Project URL slug (replaces ALL existing slugs with this one)'),
-        comment: z.string().optional().describe('Add a comment alongside the edit (supports Remarkup)'),
     }, async (params) => {
         const transactions = [];
         if (params.name !== undefined) {
             transactions.push({ type: 'name', value: params.name });
         }
+        if (params.description !== undefined) {
+            transactions.push({ type: 'description', value: params.description });
+        }
         if (params.space !== undefined) {
             transactions.push({ type: 'space', value: params.space });
         }
@@ -79,9 +81,6 @@ export function registerProjectTools(server, client) {
         if (params.removeMemberPHIDs !== undefined) {
             transactions.push({ type: 'members.remove', value: params.removeMemberPHIDs });
         }
-        if (params.comment !== undefined) {
-            transactions.push({ type: 'comment', value: params.comment });
-        }
         if (transactions.length === 0) {
             return { content: [{ type: 'text', text: 'No changes specified' }] };
         }

package/dist/tools/transaction.js

@@ -1,8 +1,8 @@
 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'),
+    server.tool('phabricator_transaction_search', 'Search transactions (comments, status changes, etc.) on a Phabricator object (e.g., "D123", "T456")', {
+        objectIdentifier: z.string().describe('Object ID (e.g., "D123", "T456") or PHID to get transactions for'),
         constraints: jsonCoerce(z.object({
             phids: z.array(z.string()).optional().describe('Transaction PHIDs'),
             authorPHIDs: z.array(z.string()).optional().describe('Author PHIDs'),
@@ -11,11 +11,7 @@ export function registerTransactionTools(server, client) {
         after: z.string().optional().describe('Cursor for next-page pagination'),
         before: z.string().optional().describe('Cursor for previous-page pagination'),
     }, async (params) => {
-        const { objectIdentifier, ...searchParams } = params;
-        const result = await client.call('transaction.search', {
-            objectIdentifier,
-            ...searchParams,
-        });
+        const result = await client.call('transaction.search', params);
         return { content: [{ type: 'text', text: JSON.stringify(result, null, 2) }] };
     });
 }

package/dist/tools/user.js

@@ -19,6 +19,7 @@ export function registerUserTools(server, client) {
             isBot: z.boolean().optional().describe('Filter by bot status'),
             isMailingList: z.boolean().optional().describe('Filter by mailing list status'),
             needsApproval: z.boolean().optional().describe('Filter to users awaiting admin approval'),
+            mfa: z.boolean().optional().describe('Filter by MFA enrollment status (admin-only)'),
             createdStart: z.coerce.number().optional().describe('Created after (epoch timestamp)'),
             createdEnd: z.coerce.number().optional().describe('Created before (epoch timestamp)'),
             query: z.string().optional().describe('Full-text search query'),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@freelancercom/phabricator-mcp",
-  "version": "2.0.14",
+  "version": "2.0.16",
   "description": "MCP server for Phabricator Conduit API - manage tasks, code reviews, repositories, and more",
   "type": "module",
   "main": "dist/index.js",