@freelancercom/phabricator-mcp 2.0.17 → 2.0.19

package/dist/tools/maniphest.js

@@ -58,6 +58,59 @@ async function linkRevisionsToTask(client, taskIdentifier, revisionPHIDs, action
     }
     return results;
 }
+/**
+ * Phabricator custom fields are only editable when the task's subtype matches
+ * the form that exposes them. For example, `custom.postmortem.*` fields require
+ * subtype "postmortem", while `custom.incident.*` fields require "incident".
+ *
+ * This function groups custom field transactions by required subtype, temporarily
+ * switches the task's subtype to set each group, then restores the original subtype.
+ */
+const SUBTYPE_FIELD_PREFIXES = {
+    'custom.postmortem.': 'postmortem',
+    'custom.incident.': 'incident',
+};
+async function applyCustomFields(client, objectIdentifier, customFields, currentSubtype) {
+    // Group fields by required subtype
+    const groups = new Map();
+    for (const [key, value] of Object.entries(customFields)) {
+        let requiredSubtype = null;
+        for (const [prefix, subtype] of Object.entries(SUBTYPE_FIELD_PREFIXES)) {
+            if (key.startsWith(prefix)) {
+                requiredSubtype = subtype;
+                break;
+            }
+        }
+        const group = groups.get(requiredSubtype) ?? [];
+        group.push({ type: key, value });
+        groups.set(requiredSubtype, group);
+    }
+    const results = {};
+    let lastSubtype = currentSubtype;
+    for (const [requiredSubtype, transactions] of groups) {
+        // Switch subtype if needed
+        if (requiredSubtype !== null && requiredSubtype !== lastSubtype) {
+            await client.call('maniphest.edit', {
+                objectIdentifier,
+                transactions: [{ type: 'subtype', value: requiredSubtype }],
+            });
+            lastSubtype = requiredSubtype;
+        }
+        const result = await client.call('maniphest.edit', {
+            objectIdentifier,
+            transactions,
+        });
+        results[requiredSubtype ?? 'default'] = result;
+    }
+    // Restore original subtype if we changed it
+    if (lastSubtype !== currentSubtype && currentSubtype !== undefined) {
+        await client.call('maniphest.edit', {
+            objectIdentifier,
+            transactions: [{ type: 'subtype', value: currentSubtype }],
+        });
+    }
+    return results;
+}
 export function registerManiphestTools(server, client) {
     // Search tasks
     server.tool('phabricator_task_search', 'Search Maniphest tasks with optional filters', {
@@ -161,25 +214,22 @@ export function registerManiphestTools(server, client) {
         if (params.comment !== undefined) {
             transactions.push({ type: 'comment', value: params.comment });
         }
-        if (params.customFields !== undefined) {
-            for (const [key, value] of Object.entries(params.customFields)) {
-                transactions.push({ type: key, value });
-            }
-        }
         const result = await client.call('maniphest.edit', { transactions });
+        const extras = {};
+        // Custom fields are applied in a second call because Phabricator validates
+        // transaction types against the default subtype during creation. Subtype-specific
+        // custom fields (e.g. postmortem fields) require temporarily switching the subtype.
+        if (params.customFields !== undefined && Object.keys(params.customFields).length > 0) {
+            extras.customFields = await applyCustomFields(client, result.object.phid, params.customFields, params.subtype);
+        }
         // 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),
-                    }],
-            };
+            extras.linkedRevisions = await linkRevisionsToTask(client, taskId, revPHIDs, 'add');
         }
-        return { content: [{ type: 'text', text: JSON.stringify(result, null, 2) }] };
+        const output = Object.keys(extras).length > 0 ? { ...result, ...extras } : result;
+        return { content: [{ type: 'text', text: JSON.stringify(output, null, 2) }] };
     });
     // Edit task
     server.tool('phabricator_task_edit', 'Edit an existing Maniphest task', {
@@ -269,16 +319,13 @@ export function registerManiphestTools(server, client) {
         if (params.comment !== undefined) {
             transactions.push({ type: 'comment', value: params.comment });
         }
-        if (params.customFields !== undefined) {
-            for (const [key, value] of Object.entries(params.customFields)) {
-                transactions.push({ type: key, value });
-            }
-        }
+        const hasCustomFields = params.customFields !== undefined && Object.keys(params.customFields).length > 0;
         const hasRevisionChanges = (params.addRevisionIDs !== undefined && params.addRevisionIDs.length > 0) ||
             (params.removeRevisionIDs !== undefined && params.removeRevisionIDs.length > 0);
-        if (transactions.length === 0 && !hasRevisionChanges) {
+        if (transactions.length === 0 && !hasCustomFields && !hasRevisionChanges) {
             return { content: [{ type: 'text', text: 'No changes specified' }] };
         }
+        const extras = {};
         let result = undefined;
         if (transactions.length > 0) {
             result = await client.call('maniphest.edit', {
@@ -286,18 +333,22 @@ export function registerManiphestTools(server, client) {
                 transactions,
             });
         }
+        // Custom fields may require subtype switching (e.g. postmortem fields
+        // need subtype "postmortem"). applyCustomFields handles this automatically.
+        if (hasCustomFields) {
+            extras.customFields = await applyCustomFields(client, params.objectIdentifier, params.customFields, params.subtype);
+        }
         // 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');
+            extras.addedRevisions = 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');
+            extras.removedRevisions = await linkRevisionsToTask(client, params.objectIdentifier, revPHIDs, 'remove');
         }
-        const output = hasRevisionChanges
-            ? { ...(result !== undefined ? { task: result } : {}), linkedRevisions: revisionResults }
+        const output = Object.keys(extras).length > 0
+            ? { ...(result !== undefined ? { task: result } : {}), ...extras }
             : result;
         return { content: [{ type: 'text', text: JSON.stringify(output, null, 2) }] };
     });

package/package.json

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