@codazen/harmonica-mcp 4.0.1 → 4.0.2

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 };
         }
@@ -32688,6 +32682,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 +32806,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 +32866,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 +32923,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 +33160,15 @@ function createHttpClient(config2) {
       return result?.note ?? result;
     },
     createNote: async (input) => {
-      const result = await request("POST", `/api/systems/${encodeURIComponent(input.projectId ?? "")}/notes`, input);
+      const inputRec = input;
+      const effectiveSystemId = inputRec.systemId ?? input.projectId;
+      if (!effectiveSystemId) throw new Error("createNote: systemId (or legacy projectId) 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 +33209,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 +34172,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 +34994,7 @@ function loadConfig() {
   };
 }
 async function main() {
-  console.error(`[harmonica-mcp] v${"4.0.1"} starting\u2026`);
+  console.error(`[harmonica-mcp] v${"4.0.2"} 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.0.2",
   "description": "MCP server for Harmonica — connect any MCP-compatible AI assistant to Harmonica",
   "license": "MIT",
   "bin": {