@freelancercom/phabricator-mcp 2.0.15 → 2.0.17

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/maniphest.js

@@ -1,5 +1,63 @@
 import { z } from 'zod';
 import { jsonCoerce } from './coerce.js';
+/**
+ * Resolve revision identifiers (e.g. "D223125", "PHID-DREV-xxx") to PHIDs.
+ * Accepts a mix of D-prefixed IDs and raw PHIDs.
+ */
+async function resolveRevisionPHIDs(client, ids) {
+    const phids = [];
+    const namesToLookup = [];
+    for (const id of ids) {
+        if (id.startsWith('PHID-')) {
+            phids.push(id);
+        }
+        else {
+            // Normalize to D-prefixed format if just a number
+            const name = /^\d+$/.test(id) ? `D${id}` : id;
+            namesToLookup.push(name);
+        }
+    }
+    if (namesToLookup.length > 0) {
+        const lookupResult = await client.call('phid.lookup', { names: namesToLookup });
+        for (const name of namesToLookup) {
+            const entry = lookupResult[name];
+            if (entry) {
+                phids.push(entry.phid);
+            }
+            else {
+                throw new Error(`Could not resolve revision identifier: ${name}`);
+            }
+        }
+    }
+    return phids;
+}
+/**
+ * Link or unlink revisions to/from a task by calling differential.revision.edit
+ * with tasks.add / tasks.remove transactions.
+ */
+async function linkRevisionsToTask(client, taskIdentifier, revisionPHIDs, action) {
+    // Resolve the task identifier to a PHID if it's a T-prefixed ID
+    let taskPHID = taskIdentifier;
+    if (!taskIdentifier.startsWith('PHID-')) {
+        const lookupResult = await client.call('phid.lookup', { names: [taskIdentifier] });
+        const entry = lookupResult[taskIdentifier];
+        if (entry) {
+            taskPHID = entry.phid;
+        }
+        else {
+            throw new Error(`Could not resolve task identifier: ${taskIdentifier}`);
+        }
+    }
+    const results = [];
+    for (const revPHID of revisionPHIDs) {
+        const result = await client.call('differential.revision.edit', {
+            objectIdentifier: revPHID,
+            transactions: [{ type: `tasks.${action}`, value: [taskPHID] }],
+        });
+        results.push(result);
+    }
+    return results;
+}
 export function registerManiphestTools(server, client) {
     // Search tasks
     server.tool('phabricator_task_search', 'Search Maniphest tasks with optional filters', {
@@ -51,9 +109,11 @@ export function registerManiphestTools(server, client) {
         projectPHIDs: z.array(z.string()).optional().describe('Project PHIDs to tag'),
         subscriberPHIDs: z.array(z.string()).optional().describe('Subscriber PHIDs'),
         status: z.string().optional().describe('Initial status'),
+        subtype: z.string().optional().describe('Task subtype (e.g. "default", "incident")'),
         parentPHIDs: z.array(z.string()).optional().describe('Parent task PHIDs'),
         subtaskPHIDs: z.array(z.string()).optional().describe('Subtask PHIDs'),
-        commitPHIDs: z.array(z.string()).optional().describe('Commit PHIDs to associate'),
+        commitPHIDs: z.array(z.string()).optional().describe('Commit PHIDs to associate (for actual commits only, not revisions)'),
+        revisionIDs: z.array(z.string()).optional().describe('Differential revision IDs to link (e.g. ["D223125"] or ["PHID-DREV-xxx"]). Creates a bidirectional link between the revision and the task.'),
         points: z.coerce.number().nullable().optional().describe('Story points value (if points are enabled on this instance)'),
         space: z.string().optional().describe('Space PHID to place the task in (for multi-space installations)'),
         comment: z.string().optional().describe('Initial comment on the task (supports Remarkup)'),
@@ -80,6 +140,9 @@ export function registerManiphestTools(server, client) {
         if (params.status !== undefined) {
             transactions.push({ type: 'status', value: params.status });
         }
+        if (params.subtype !== undefined) {
+            transactions.push({ type: 'subtype', value: params.subtype });
+        }
         if (params.parentPHIDs !== undefined) {
             transactions.push({ type: 'parents.set', value: params.parentPHIDs });
         }
@@ -104,6 +167,18 @@ export function registerManiphestTools(server, client) {
             }
         }
         const result = await client.call('maniphest.edit', { transactions });
+        // Link revisions to the newly created task via differential.revision.edit
+        if (params.revisionIDs !== undefined && params.revisionIDs.length > 0) {
+            const revPHIDs = await resolveRevisionPHIDs(client, params.revisionIDs);
+            const taskId = `T${result.object.id}`;
+            const linkResults = await linkRevisionsToTask(client, taskId, revPHIDs, 'add');
+            return {
+                content: [{
+                        type: 'text',
+                        text: JSON.stringify({ ...result, linkedRevisions: linkResults }, null, 2),
+                    }],
+            };
+        }
         return { content: [{ type: 'text', text: JSON.stringify(result, null, 2) }] };
     });
     // Edit task
@@ -114,6 +189,7 @@ export function registerManiphestTools(server, client) {
         ownerPHID: z.string().nullable().optional().describe('New owner PHID (null to unassign)'),
         priority: z.string().optional().describe('New priority'),
         status: z.string().optional().describe('New status: open, resolved, wontfix, invalid, spite, duplicate'),
+        subtype: z.string().optional().describe('Task subtype (e.g. "default", "incident")'),
         addProjectPHIDs: z.array(z.string()).optional().describe('Project PHIDs to add'),
         removeProjectPHIDs: z.array(z.string()).optional().describe('Project PHIDs to remove'),
         addSubscriberPHIDs: z.array(z.string()).optional().describe('Subscriber PHIDs to add'),
@@ -122,8 +198,10 @@ export function registerManiphestTools(server, client) {
         removeParentPHIDs: z.array(z.string()).optional().describe('Parent task PHIDs to remove'),
         addSubtaskPHIDs: z.array(z.string()).optional().describe('Subtask PHIDs to add'),
         removeSubtaskPHIDs: z.array(z.string()).optional().describe('Subtask PHIDs to remove'),
-        addCommitPHIDs: z.array(z.string()).optional().describe('Commit PHIDs to associate'),
-        removeCommitPHIDs: z.array(z.string()).optional().describe('Commit PHIDs to disassociate'),
+        addCommitPHIDs: z.array(z.string()).optional().describe('Commit PHIDs to associate (for actual commits only, not revisions)'),
+        removeCommitPHIDs: z.array(z.string()).optional().describe('Commit PHIDs to disassociate (for actual commits only, not revisions)'),
+        addRevisionIDs: z.array(z.string()).optional().describe('Differential revision IDs to link (e.g. ["D223125"] or ["PHID-DREV-xxx"]). Creates a bidirectional link between the revision and the task.'),
+        removeRevisionIDs: z.array(z.string()).optional().describe('Differential revision IDs to unlink (e.g. ["D223125"] or ["PHID-DREV-xxx"]).'),
         points: z.coerce.number().nullable().optional().describe('Story points value (null to clear)'),
         columnPHID: z.string().optional().describe('Move to workboard column'),
         space: z.string().optional().describe('Space PHID to move the task to (for multi-space installations)'),
@@ -146,6 +224,9 @@ export function registerManiphestTools(server, client) {
         if (params.status !== undefined) {
             transactions.push({ type: 'status', value: params.status });
         }
+        if (params.subtype !== undefined) {
+            transactions.push({ type: 'subtype', value: params.subtype });
+        }
         if (params.addProjectPHIDs !== undefined) {
             transactions.push({ type: 'projects.add', value: params.addProjectPHIDs });
         }
@@ -193,14 +274,32 @@ export function registerManiphestTools(server, client) {
                 transactions.push({ type: key, value });
             }
         }
-        if (transactions.length === 0) {
+        const hasRevisionChanges = (params.addRevisionIDs !== undefined && params.addRevisionIDs.length > 0) ||
+            (params.removeRevisionIDs !== undefined && params.removeRevisionIDs.length > 0);
+        if (transactions.length === 0 && !hasRevisionChanges) {
             return { content: [{ type: 'text', text: 'No changes specified' }] };
         }
-        const result = await client.call('maniphest.edit', {
-            objectIdentifier: params.objectIdentifier,
-            transactions,
-        });
-        return { content: [{ type: 'text', text: JSON.stringify(result, null, 2) }] };
+        let result = undefined;
+        if (transactions.length > 0) {
+            result = await client.call('maniphest.edit', {
+                objectIdentifier: params.objectIdentifier,
+                transactions,
+            });
+        }
+        // Link/unlink revisions via differential.revision.edit
+        const revisionResults = {};
+        if (params.addRevisionIDs !== undefined && params.addRevisionIDs.length > 0) {
+            const revPHIDs = await resolveRevisionPHIDs(client, params.addRevisionIDs);
+            revisionResults.added = await linkRevisionsToTask(client, params.objectIdentifier, revPHIDs, 'add');
+        }
+        if (params.removeRevisionIDs !== undefined && params.removeRevisionIDs.length > 0) {
+            const revPHIDs = await resolveRevisionPHIDs(client, params.removeRevisionIDs);
+            revisionResults.removed = await linkRevisionsToTask(client, params.objectIdentifier, revPHIDs, 'remove');
+        }
+        const output = hasRevisionChanges
+            ? { ...(result !== undefined ? { task: result } : {}), linkedRevisions: revisionResults }
+            : result;
+        return { content: [{ type: 'text', text: JSON.stringify(output, null, 2) }] };
     });
     // Add comment to task
     server.tool('phabricator_task_add_comment', 'Add a comment to a Maniphest task', {

package/dist/tools/phame.js

@@ -7,7 +7,6 @@ export function registerPhameTools(server, client) {
         constraints: jsonCoerce(z.object({
             ids: z.array(z.coerce.number()).optional().describe('Blog IDs'),
             phids: z.array(z.string()).optional().describe('Blog PHIDs'),
-            statuses: z.string().optional().describe('Blog status: "active" or "archived"'),
             subscribers: z.array(z.string()).optional().describe('Subscriber user/project PHIDs'),
             projects: z.array(z.string()).optional().describe('Project PHIDs'),
             query: z.string().optional().describe('Full-text search query'),
@@ -36,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) {
@@ -66,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,9 +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().optional().describe('Object ID (e.g., "D123", "T456") or PHID. Provide this or objectType, not both.'),
-        objectType: z.string().optional().describe('Object type group to query (e.g., "TASK", "DREV", "CMIT"). Queries all transactions of that type. Provide this or objectIdentifier, not both.'),
+    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'),
@@ -12,12 +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, objectType, ...searchParams } = params;
-        const result = await client.call('transaction.search', {
-            objectIdentifier,
-            objectType,
-            ...searchParams,
-        });
+        const result = await client.call('transaction.search', params);
         return { content: [{ type: 'text', text: JSON.stringify(result, null, 2) }] };
     });
 }

package/package.json

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