@codazen/harmonica-mcp 4.0.1 → 4.1.0

package/dist/index.js

@@ -25533,17 +25533,14 @@ function registerMembershipTools(server, ctx, client) {
   );
   server.tool(
     "add_member",
-    "Add a user to the configured organization. Creates the user record if they do not exist yet.",
+    "Grant a user access to the configured organization. Their name and avatar are filled in when they first sign in.",
     {
-      name: external_exports.string().describe("Full name of the user"),
       email: external_exports.string().email().describe("Email address (used as the user ID)"),
-      title: external_exports.string().optional().describe("Job title"),
-      responsibility: external_exports.string().optional().describe("Primary responsibility or role description"),
       role: external_exports.enum(["member", "admin", "client"]).optional().describe('Organization role (defaults to "member")')
     },
-    async ({ name, email: email2, title, responsibility, role }) => {
+    async ({ email: email2, role }) => {
       try {
-        const member = await client.addOrgMember(ctx.orgId, { name, email: email2, title, responsibility, role });
+        const member = await client.addOrgMember(ctx.orgId, { email: email2, role });
         const lines = [
           "Member added successfully.",
           "",
@@ -25562,17 +25559,14 @@ function registerMembershipTools(server, ctx, client) {
   );
   server.tool(
     "update_member_role",
-    "Update a member's role or profile within the configured organization.",
+    "Change a member's role within the configured organization.",
     {
       userId: external_exports.string().describe("User ID (email address) of the member to update"),
-      role: external_exports.enum(["member", "admin", "client"]).optional().describe("New organization role"),
-      name: external_exports.string().optional().describe("Updated display name"),
-      title: external_exports.string().optional().describe("Updated job title"),
-      responsibility: external_exports.string().optional().describe("Updated responsibility description")
+      role: external_exports.enum(["member", "admin", "client"]).describe("New organization role")
     },
-    async ({ userId, role, name, title, responsibility }) => {
+    async ({ userId, role }) => {
       try {
-        const member = await client.updateOrgMember(ctx.orgId, userId, { role, name, title, responsibility });
+        const member = await client.updateOrgMember(ctx.orgId, userId, { role });
         if (!member) {
           return { content: [{ type: "text", text: `Member not found: "${userId}"` }], isError: true };
         }
@@ -26335,6 +26329,15 @@ var normalizeSystemId = (raw) => {
   if (!id) return raw;
   return { ...b, systemId: id };
 };
+var normalizeSystemIdAndName = (raw) => {
+  if (typeof raw !== "object" || raw === null || Array.isArray(raw)) return raw;
+  const b = raw;
+  const result = { ...b };
+  const id = b["systemId"] || b["projectId"];
+  if (id) result["systemId"] = id;
+  if ((b["systemName"] == null || b["systemName"] === "") && b["projectName"]) result["systemName"] = b["projectName"];
+  return result;
+};
 var normalizeSystemIdAndTitle = (raw) => {
   if (typeof raw !== "object" || raw === null || Array.isArray(raw)) return raw;
   const b = raw;
@@ -26350,15 +26353,17 @@ var BeatApiSchema = external_exports.preprocess(
     if (typeof raw !== "object" || raw === null) return raw;
     const b = raw;
     const id = b["systemId"] || b["projectId"];
-    if (!id) return raw;
-    return { ...b, systemId: id };
+    const result = { ...b };
+    if (id) result["systemId"] = id;
+    if ((b["systemName"] == null || b["systemName"] === "") && b["projectName"]) result["systemName"] = b["projectName"];
+    return result;
   },
   external_exports.object({
     beatId: external_exports.string(),
     systemId: external_exports.string(),
-    /** Source project's title; attached by the flat /api/beats endpoint for
-     *  multi-project (teamspace/org) responses. */
-    projectName: external_exports.string().nullish(),
+    /** Source system's title; attached by the flat /api/beats endpoint for
+     *  multi-system (teamspace/org) responses. */
+    systemName: external_exports.string().nullish(),
     title: external_exports.string(),
     version: external_exports.number().nullish(),
     description: external_exports.string().nullish(),
@@ -26824,13 +26829,13 @@ var NextActionSchema = external_exports.object({
 var NextActionsResponseSchema = external_exports.object({
   actions: external_exports.array(NextActionSchema)
 }).passthrough();
-var SearchResultSchema = external_exports.preprocess(normalizeSystemId, external_exports.object({
+var SearchResultSchema = external_exports.preprocess(normalizeSystemIdAndName, external_exports.object({
   entityId: external_exports.string(),
   entityType: external_exports.enum(["beat", "note", "revision", "beatVersion", "drop", "project", "system"]),
   score: external_exports.number(),
   snippet: external_exports.string(),
   systemId: external_exports.string().optional(),
-  projectName: external_exports.string().optional(),
+  systemName: external_exports.string().optional(),
   teamspaceId: external_exports.string().optional()
 }).passthrough());
 var SearchScopeSchema = external_exports.object({
@@ -27822,7 +27827,7 @@ function registerNoteTools(server, ctx, client) {
           const noteId = await client.getNextNoteId(projectCode, systemId);
           note = await client.createNote({
             noteId,
-            projectId: systemId,
+            systemId,
             beatId,
             revisionId,
             beatVersionId,
@@ -28921,7 +28926,7 @@ Systems (${createdProjects.length}):`);
             const n = await client.createNote({
               noteId: `N-TEMP-${(0, import_node_crypto.randomUUID)()}`,
               // overridden server-side
-              projectId: project.systemId,
+              systemId: project.systemId,
               noteType: note.noteType,
               content: note.content,
               rationale: note.rationale,
@@ -29581,6 +29586,7 @@ function registerPulseReportTools(server, _ctx, client) {
 }
 
 // ../../libs/harmonica-services/src/delivery-policy.decide.ts
+var DELIVERY_STEPS = ["openPullRequest", "requestAgentReview", "watchPullRequest"];
 function decideDeliveryStep(resolution, step) {
   if (!resolution.resolved) return "ask";
   const value = resolution.policy[step];
@@ -29588,6 +29594,19 @@ function decideDeliveryStep(resolution, step) {
   if (value === false) return "prohibited";
   return "ask";
 }
+function mergeAgentDeliveryPolicy(current, patch) {
+  const merged = { ...current };
+  for (const step of DELIVERY_STEPS) {
+    if (!(step in patch)) continue;
+    const value = patch[step];
+    if (value === null) {
+      delete merged[step];
+    } else if (value === true || value === false) {
+      merged[step] = value;
+    }
+  }
+  return merged;
+}
 
 // ../../libs/harmonica-services/src/mcp/tools/revision-lifecycle-tools.ts
 async function checkDeliveryStep(client, revisionId, step) {
@@ -30691,7 +30710,7 @@ function formatImportSummary(result, targetTeamspaceId) {
     lines.push("", `Errors (${result.errors.length}):`);
     result.errors.forEach((e) => lines.push(`  - ${e}`));
   }
-  const hasTeamspaceData = result.projectTeamspaceId !== void 0 || (result.counts.drops ?? 0) + (result.counts.deliverables ?? 0) > 0;
+  const hasTeamspaceData = result.systemTeamspaceId !== void 0 || (result.counts.drops ?? 0) + (result.counts.deliverables ?? 0) > 0;
   if (!targetTeamspaceId && hasTeamspaceData) {
     lines.push(
       "",
@@ -30965,8 +30984,14 @@ function registerSystemTools(server, ctx, client) {
       if (Object.keys(nonEmpty).length === 0) {
         return { content: [{ type: "text", text: "No updates provided." }], isError: true };
       }
-      await assertSystemInOrg(client, systemId, ctx.orgId);
+      const currentSystem = await fetchSystemInOrg(client, systemId, ctx.orgId);
       if (nonEmpty["accountId"] === "") delete nonEmpty["accountId"];
+      if (nonEmpty["agentDeliveryPolicy"] && typeof nonEmpty["agentDeliveryPolicy"] === "object") {
+        nonEmpty["agentDeliveryPolicy"] = mergeAgentDeliveryPolicy(
+          currentSystem.agentDeliveryPolicy,
+          nonEmpty["agentDeliveryPolicy"]
+        );
+      }
       if (typeof updates.teamspaceId === "string") {
         const teamspace = await client.getTeamspace(updates.teamspaceId);
         if (!teamspace) {
@@ -32688,6 +32713,16 @@ function safeParseErrorBody(body) {
     return void 0;
   }
 }
+var ORG_ROLE_PRECEDENCE = ["admin", "member", "client"];
+function asLegacyMember(m) {
+  const roles = Array.isArray(m["roles"]) ? m["roles"] : [];
+  const rank = (role2) => {
+    const i = ORG_ROLE_PRECEDENCE.indexOf(role2);
+    return i === -1 ? ORG_ROLE_PRECEDENCE.length : i;
+  };
+  const role = [...roles].sort((a, b) => rank(a) - rank(b) || a.localeCompare(b))[0];
+  return { ...m, role: role ?? "" };
+}
 function createHttpClient(config2) {
   const url = new URL(config2.apiBaseUrl);
   const isLocalhost = url.hostname === "localhost" || url.hostname === "127.0.0.1";
@@ -32802,21 +32837,37 @@ function createHttpClient(config2) {
     getMembership: async (userId, orgId) => {
       return { userId, orgId, role: "member" };
     },
-    // Org membership management
+    // Org membership management — the generic member API, shared with the
+    // teamspace, account and system tiers (rev-4551c900). The org-only routes
+    // this used to call were a second implementation over the same Grant rows
+    // that skipped the lockout rule.
+    //
+    // Those routes also set a name, title and responsibility on the User
+    // record. Nothing does that any more: a name is written on first sign-in
+    // from the identity provider, and the fields are gone from the tool
+    // schemas rather than accepted and silently dropped.
     listOrgMembers: async (orgId) => {
-      const result = await request("GET", `/api/organizations/${encodeURIComponent(orgId)}/members`);
-      return result ?? [];
+      const result = await request("GET", `/api/orgs/${encodeURIComponent(orgId)}/members`);
+      return (result?.members ?? []).map(asLegacyMember);
     },
     addOrgMember: async (orgId, input) => {
-      const result = await request("POST", `/api/organizations/${encodeURIComponent(orgId)}/members`, input);
-      return result;
+      const result = await request("POST", `/api/orgs/${encodeURIComponent(orgId)}/members`, {
+        userId: input.email.toLowerCase(),
+        roles: [input.role ?? "member"]
+      });
+      return asLegacyMember(result?.member ?? {});
     },
     updateOrgMember: async (orgId, userId, input) => {
-      const result = await request("PUT", `/api/organizations/${encodeURIComponent(orgId)}/members/${encodeURIComponent(userId)}`, input);
-      return result;
+      if (!input.role) {
+        const current = await request("GET", `/api/orgs/${encodeURIComponent(orgId)}/members`);
+        const found = (current?.members ?? []).find((m) => String(m["userId"]).toLowerCase() === userId.toLowerCase());
+        return found ? asLegacyMember(found) : void 0;
+      }
+      const result = await request("PATCH", `/api/orgs/${encodeURIComponent(orgId)}/members/${encodeURIComponent(userId)}`, { roles: [input.role] });
+      return asLegacyMember(result?.member ?? {});
     },
     removeOrgMember: async (orgId, userId) => {
-      await request("DELETE", `/api/organizations/${encodeURIComponent(orgId)}/members/${encodeURIComponent(userId)}`);
+      await request("DELETE", `/api/orgs/${encodeURIComponent(orgId)}/members/${encodeURIComponent(userId)}`);
     },
     // Both renameOrganization and updateOrganization PATCH the same endpoint.
     // Server-side routing determines which service runs based on which fields are present.
@@ -32846,7 +32897,7 @@ function createHttpClient(config2) {
       try {
         const result = await request(
           "POST",
-          `/api/organizations/${encodeURIComponent(sourceOrgId)}/transfer-project`,
+          `/api/organizations/${encodeURIComponent(sourceOrgId)}/transfer-system`,
           { systemId, targetOrgId }
         );
         const system = result?.system ? {
@@ -32903,7 +32954,11 @@ function createHttpClient(config2) {
       const result = await request("GET", `/api/systems/${encodeURIComponent(systemId)}/facts`);
       return result?.data;
     },
-    saveSystemFact: (input) => request("PUT", `/api/systems/${encodeURIComponent(String(input.projectId))}/facts`, input),
+    saveSystemFact: (input) => {
+      const effectiveId = input.systemId ?? input.projectId;
+      if (!effectiveId) throw new Error("saveSystemFact: systemId (or legacy projectId) is required");
+      return request("PUT", `/api/systems/${encodeURIComponent(String(effectiveId))}/facts`, input);
+    },
     transitionSystemLifecycleState: async (systemId, targetState, options) => {
       return request("POST", `/api/systems/${encodeURIComponent(systemId)}/lifecycle/transitions`, {
         targetState,
@@ -33136,12 +33191,14 @@ function createHttpClient(config2) {
       return result?.note ?? result;
     },
     createNote: async (input) => {
-      const result = await request("POST", `/api/systems/${encodeURIComponent(input.projectId ?? "")}/notes`, input);
+      const effectiveSystemId = input.systemId;
+      if (!effectiveSystemId) throw new Error("createNote: systemId is required");
+      const result = await request("POST", `/api/systems/${encodeURIComponent(effectiveSystemId)}/notes`, input);
       return result?.note ?? result;
     },
     updateNote: async (noteId, input) => {
       const existing = await request("GET", `/api/notes/${encodeURIComponent(noteId)}`);
-      const systemId = existing?.projectId ?? existing?.note?.projectId;
+      const systemId = existing?.systemId ?? existing?.note?.systemId;
       if (systemId) {
         const result2 = await request("PATCH", `/api/systems/${encodeURIComponent(systemId)}/notes/${encodeURIComponent(noteId)}`, input);
         return result2?.note ?? result2;
@@ -33182,20 +33239,20 @@ function createHttpClient(config2) {
     },
     reassignNote: async (noteId, targetBeatId, reason, _actorId, targetRevisionId) => {
       const note = await request("GET", `/api/notes/${encodeURIComponent(noteId)}`);
-      if (!note?.note) return void 0;
+      if (!note?.note?.systemId) return void 0;
       const result = await request(
         "POST",
-        `/api/systems/${encodeURIComponent(note.note.projectId)}/notes/${encodeURIComponent(noteId)}/reassign`,
+        `/api/systems/${encodeURIComponent(note.note.systemId)}/notes/${encodeURIComponent(noteId)}/reassign`,
         { targetBeatId, targetRevisionId, reason }
       );
       return result?.note ?? result;
     },
     removeNote: async (noteId, reason, _actorId) => {
       const note = await request("GET", `/api/notes/${encodeURIComponent(noteId)}`);
-      if (!note?.note) return void 0;
+      if (!note?.note?.systemId) return void 0;
       const result = await request(
         "POST",
-        `/api/systems/${encodeURIComponent(note.note.projectId)}/notes/${encodeURIComponent(noteId)}/remove`,
+        `/api/systems/${encodeURIComponent(note.note.systemId)}/notes/${encodeURIComponent(noteId)}/remove`,
         { reason }
       );
       return result?.note ?? result;
@@ -34145,7 +34202,9 @@ function createHttpClient(config2) {
         ...parentGroupId !== void 0 && { parentGroupId },
         ...createdBy !== void 0 && { createdBy }
       };
-      const url2 = input.projectId ? `/api/systems/${encodeURIComponent(input.projectId)}/deliverable-groups` : `/api/teamspaces/${encodeURIComponent(input.teamspaceId)}/deliverable-groups`;
+      const inputRec = input;
+      const effectiveSystemId = inputRec.systemId ?? input.projectId;
+      const url2 = effectiveSystemId ? `/api/systems/${encodeURIComponent(effectiveSystemId)}/deliverable-groups` : `/api/teamspaces/${encodeURIComponent(input.teamspaceId)}/deliverable-groups`;
       const result = await request("POST", url2, body);
       if (result === void 0) throw new Error(`Resource not found when creating DeliverableGroup`);
       return result;
@@ -34965,7 +35024,7 @@ function loadConfig() {
   };
 }
 async function main() {
-  console.error(`[harmonica-mcp] v${"4.0.1"} starting\u2026`);
+  console.error(`[harmonica-mcp] v${"4.1.0"} starting\u2026`);
   const config2 = loadConfig();
   const client = createHttpClient({
     apiBaseUrl: config2.apiBaseUrl,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@codazen/harmonica-mcp",
-  "version": "4.0.1",
+  "version": "4.1.0",
   "description": "MCP server for Harmonica — connect any MCP-compatible AI assistant to Harmonica",
   "license": "MIT",
   "bin": {