@codazen/harmonica-mcp 4.0.2 → 4.2.0

package/dist/index.js

@@ -26329,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;
@@ -26344,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(),
@@ -26612,6 +26623,7 @@ var CHECK_TARGET_TYPES = [
   "revision",
   "beat_version",
   "project",
+  "system",
   "drop",
   "organization"
 ];
@@ -26818,13 +26830,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({
@@ -27816,7 +27828,7 @@ function registerNoteTools(server, ctx, client) {
           const noteId = await client.getNextNoteId(projectCode, systemId);
           note = await client.createNote({
             noteId,
-            projectId: systemId,
+            systemId,
             beatId,
             revisionId,
             beatVersionId,
@@ -28157,9 +28169,9 @@ ${trimmed}` : `# ${title}`;
 }
 
 // ../../libs/harmonica-services/src/mcp/tools/notebook-tools.ts
-var SCOPE_TYPE_VALUES = ["org", "teamspace", "project", "account"];
+var SCOPE_TYPE_VALUES = ["org", "teamspace", "system", "account", "project"];
 var scopeRefSchema = external_exports.object({
-  scopeType: external_exports.enum(SCOPE_TYPE_VALUES).describe("Scope type \u2014 org, teamspace, project, or account"),
+  scopeType: external_exports.enum(SCOPE_TYPE_VALUES).describe("Scope type \u2014 org, teamspace, system, or account"),
   scopeId: external_exports.string().min(1).describe("ID of the scope entity")
 });
 function formatNotebook(nb) {
@@ -28384,7 +28396,7 @@ ${formatTree(tree.root)}` }] };
       if (!scopeProjectId) {
         try {
           const bindings = await client.getNotebookScopeBindings(notebookId);
-          scopeProjectId = bindings?.find((b) => b.scopeType === "project")?.scopeId;
+          scopeProjectId = bindings?.find((b) => b.scopeType === "system" || b.scopeType === "project")?.scopeId;
         } catch {
         }
       }
@@ -28550,7 +28562,7 @@ ${formatNotebook(updated)}` }] };
   );
   server.tool(
     "set_notebook_scopes",
-    "Replace all scope bindings for a Notebook. Pass an empty array to remove all bindings. Bindable scope types: org, teamspace, project. (projectContainer is reserved.)",
+    "Replace all scope bindings for a Notebook. Pass an empty array to remove all bindings. Bindable scope types: org, teamspace, system. (systemContainer is reserved.)",
     {
       notebookId: external_exports.string().describe("The Notebook ID"),
       scopes: external_exports.array(scopeRefSchema).max(45).describe("New complete set of scope bindings (max 45)")
@@ -28915,7 +28927,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,
@@ -29575,6 +29587,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];
@@ -29582,6 +29595,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) {
@@ -29958,9 +29984,10 @@ Accepts the full PR metadata (number, url, branch, state, optional mergeSha). Fi
       revisionId: external_exports.string().describe("The Revision ID whose PR to review (e.g., rev-abc123)"),
       autoMerge: external_exports.boolean().optional().describe("Merge automatically once converged. Default false \u2192 stop at ready-for-review."),
       maxIterations: external_exports.number().int().min(1).max(3).optional().describe("Review rounds (default and hard cap: 3)."),
-      confirm: external_exports.boolean().optional().describe("Set true to start the review. Omitted/false performs a dry-run \u2014 no session, scan, or task is created.")
+      confirm: external_exports.boolean().optional().describe("Set true to start the review. Omitted/false performs a dry-run \u2014 no session, scan, or task is created."),
+      requestingSessionId: external_exports.string().min(1).optional().describe("Your current Session ID (sess-<uuid>). Pass it so the review Session nests under yours and reports its outcome back to you when it finishes \u2014 otherwise you have no way to learn how the review turned out.")
     },
-    async ({ revisionId, autoMerge, maxIterations, confirm }) => {
+    async ({ revisionId, autoMerge, maxIterations, confirm, requestingSessionId }) => {
       try {
         const revision = await client.getRevision(revisionId);
         if (!revision) {
@@ -29971,7 +29998,7 @@ Accepts the full PR metadata (number, url, branch, state, optional mergeSha). Fi
         if ("refusal" in latitude) {
           return { content: [{ type: "text", text: latitude.refusal }], isError: true };
         }
-        const result = await client.requestCodeReview({ revisionId, autoMerge, maxIterations, confirm });
+        const result = await client.requestCodeReview({ revisionId, autoMerge, maxIterations, confirm, requestingSessionId });
         if (!result.success) {
           return { content: [{ type: "text", text: `Failed to request code review (${result.error.code}): ${result.error.message}` }], isError: true };
         }
@@ -30685,7 +30712,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(
       "",
@@ -30733,13 +30760,17 @@ function assertSnapshotShape(val, sourceHint) {
 }
 
 // ../../libs/harmonica-services/src/mcp/tools/subscription-tools.ts
-var ENTITY_TYPES = ["project", "beat", "revision"];
+var ENTITY_TYPES = ["system", "beat", "revision"];
+var entityTypeField = external_exports.preprocess(
+  (v) => v === "project" ? "system" : v,
+  external_exports.enum(ENTITY_TYPES)
+);
 function registerSubscriptionTools(server, ctx, client) {
   server.tool(
     "subscribe_to_entity",
-    "Follow a Project, Beat, or Revision to receive notifications when it changes. You are auto-subscribed to things you own \u2014 use this to follow additional entities.",
+    "Follow a System, Beat, or Revision to receive notifications when it changes. You are auto-subscribed to things you own \u2014 use this to follow additional entities.",
     {
-      entityType: external_exports.enum(ENTITY_TYPES).describe("Type of entity to follow"),
+      entityType: entityTypeField.describe("Type of entity to follow"),
       entityId: external_exports.string().describe("The entity ID (system ID, beat ID, or revision ID)"),
       systemId: external_exports.string().describe("The system ID (for access control)")
     },
@@ -30761,9 +30792,9 @@ function registerSubscriptionTools(server, ctx, client) {
   );
   server.tool(
     "unsubscribe_from_entity",
-    "Stop following a Project, Beat, or Revision. Use durable=true to prevent auto-resubscribe if you are later re-assigned.",
+    "Stop following a System, Beat, or Revision. Use durable=true to prevent auto-resubscribe if you are later re-assigned.",
     {
-      entityType: external_exports.enum(ENTITY_TYPES).describe("Type of entity to unfollow"),
+      entityType: entityTypeField.describe("Type of entity to unfollow"),
       entityId: external_exports.string().describe("The entity ID"),
       durable: external_exports.boolean().optional().default(false).describe("If true, prevents auto-resubscribe on reassignment")
     },
@@ -30959,8 +30990,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) {
@@ -33160,9 +33197,8 @@ function createHttpClient(config2) {
       return result?.note ?? result;
     },
     createNote: async (input) => {
-      const inputRec = input;
-      const effectiveSystemId = inputRec.systemId ?? input.projectId;
-      if (!effectiveSystemId) throw new Error("createNote: systemId (or legacy projectId) is required");
+      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;
     },
@@ -34994,7 +35030,7 @@ function loadConfig() {
   };
 }
 async function main() {
-  console.error(`[harmonica-mcp] v${"4.0.2"} starting\u2026`);
+  console.error(`[harmonica-mcp] v${"4.2.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.2",
+  "version": "4.2.0",
   "description": "MCP server for Harmonica — connect any MCP-compatible AI assistant to Harmonica",
   "license": "MIT",
   "bin": {