@codazen/harmonica-mcp 0.30.0 → 0.32.0

package/dist/index.js

@@ -22877,20 +22877,98 @@ function registerCheckTools(server, ctx, client) {
   );
   server.tool(
     "list_checks",
-    "List persisted checks for a project, beat, or revision. Shows score trends over time. Filter by check type.",
+    "List persisted checks for a project, beat, beat version, or revision. Shows score trends over time. Filter by check type.",
     {
       projectId: external_exports.string().describe("The project ID"),
       beatId: external_exports.string().optional().describe("List checks for this specific beat"),
+      beatVersionId: external_exports.string().optional().describe("List checks for this specific beat version"),
       revisionId: external_exports.string().optional().describe("List checks for this specific revision"),
       checkType: external_exports.enum(["beat_quality", "plan_quality", "build_quality", "pii_scan", "portfolio_coherence"]).optional().describe("Filter by check type")
     },
-    async ({ projectId, beatId, revisionId, checkType }) => {
+    async ({ projectId, beatId, beatVersionId, revisionId, checkType }) => {
       try {
+        const filters = [
+          revisionId ? "revisionId" : null,
+          beatVersionId ? "beatVersionId" : null,
+          beatId ? "beatId" : null
+        ].filter(Boolean);
+        if (filters.length > 1) {
+          return {
+            content: [{
+              type: "text",
+              text: `Only one of revisionId, beatVersionId, beatId may be supplied (got: ${filters.join(", ")}). Pick the target scope and resend.`
+            }],
+            isError: true
+          };
+        }
+        if (revisionId) {
+          if (revisionId.startsWith("bv-")) {
+            return {
+              content: [{
+                type: "text",
+                text: "revisionId must start with 'rev-', got a Beat Version ID (bv-prefix). Use beatVersionId parameter instead."
+              }],
+              isError: true
+            };
+          }
+          if (!revisionId.startsWith("rev-")) {
+            return {
+              content: [{
+                type: "text",
+                text: `revisionId must start with 'rev-', got '${revisionId}'. Pass the full Revision UUID (rev-xxxxxxxx-...).`
+              }],
+              isError: true
+            };
+          }
+        }
+        if (beatVersionId) {
+          if (beatVersionId.startsWith("rev-")) {
+            return {
+              content: [{
+                type: "text",
+                text: "beatVersionId must start with 'bv-', got a Revision ID (rev-prefix). Use revisionId parameter instead."
+              }],
+              isError: true
+            };
+          }
+          if (!beatVersionId.startsWith("bv-")) {
+            return {
+              content: [{
+                type: "text",
+                text: `beatVersionId must start with 'bv-', got '${beatVersionId}'. Pass the full Beat Version UUID (bv-xxxxxxxx-...).`
+              }],
+              isError: true
+            };
+          }
+        }
+        if (beatId) {
+          if (beatId.startsWith("rev-")) {
+            return {
+              content: [{
+                type: "text",
+                text: "beatId must not be a Revision ID (rev-prefix). Use revisionId parameter instead."
+              }],
+              isError: true
+            };
+          }
+          if (beatId.startsWith("bv-")) {
+            return {
+              content: [{
+                type: "text",
+                text: "beatId must not be a Beat Version ID (bv-prefix). Use beatVersionId parameter instead."
+              }],
+              isError: true
+            };
+          }
+        }
         let checks;
         let scope;
         if (revisionId) {
           checks = await client.listRevisionChecks(revisionId, projectId, checkType);
           scope = `revision ${revisionId}`;
+        } else if (beatVersionId) {
+          checks = await client.listBeatVersionChecks(beatVersionId, projectId, checkType);
+          scope = `beat_version ${beatVersionId}`;
         } else if (beatId) {
           checks = await client.listBeatChecks(beatId, projectId, checkType);
           scope = `beat ${beatId}`;
@@ -25282,6 +25360,63 @@ function registerRevisionLifecycleTools(server, ctx, client) {
       }
     }
   );
+  server.tool(
+    "create_pr_for_revision",
+    "Open a Harmonica-initiated DRAFT pull request for a revision and link it back via revision.scm. Cuts a branch (agent/rev-<short>-<slug>), bootstraps it with an empty commit, opens a draft PR whose body starts with the `Harmonica: rev-<uuid>` marker, then records the PR on the revision. Rejects if the revision already has a linked PR, or if the project has no repo configured. Dry-run by default \u2014 call with `confirm: true` to actually create the branch and PR. Leaves the revision in `draft`; transition it to `open` when the PR is marked ready for review.",
+    {
+      revisionId: external_exports.string().describe("The revision ID (e.g., rev-abc123)"),
+      confirm: external_exports.boolean().optional().describe("Set true to create the branch + draft PR. Omitted/false performs a dry-run that only describes what would happen \u2014 no GitHub or DB writes.")
+    },
+    async ({ revisionId, confirm }) => {
+      try {
+        const revision = await client.getRevision(revisionId);
+        if (!revision) {
+          return { content: [{ type: "text", text: `Revision not found: "${revisionId}"` }], isError: true };
+        }
+        await assertBeatInOrg(client, revision.beatId, ctx.orgId);
+        if (!confirm) {
+          const shortId = revisionId.split("-").slice(0, 2).join("-");
+          const lines2 = [
+            "**Dry run** \u2014 no PR created. Re-run with `confirm: true` to proceed.",
+            "",
+            `**Revision:** ${revisionId}`,
+            `**Title:** ${revision.title}`
+          ];
+          if (revision.scm) {
+            lines2.push("", `\u26A0 This revision already has a linked PR (#${revision.scm.number} \u2014 ${revision.scm.url}). create_pr_for_revision would be rejected.`);
+          } else {
+            const project = await client.getProject(revision.projectId);
+            if (!project?.repoOwner || !project?.repoName || !project?.repoDefaultBranch) {
+              lines2.push("", "\u26A0 The project has no repo configured (repoOwner/repoName/repoDefaultBranch). create_pr_for_revision would be rejected.");
+            } else {
+              lines2.push(
+                "",
+                `Would open a draft PR against \`${project.repoOwner}/${project.repoName}:${project.repoDefaultBranch}\` on a new \`agent/${shortId}-\u2026\` branch, with body starting \`Harmonica: ${revisionId}\`.`
+              );
+            }
+          }
+          return { content: [{ type: "text", text: lines2.join("\n") }] };
+        }
+        const result = await client.createPrForRevision(revisionId);
+        if (!result.success) {
+          return { content: [{ type: "text", text: `Failed to create PR (${result.error.code}): ${result.error.message}` }], isError: true };
+        }
+        const lines = [
+          "Draft PR created and linked to the revision.",
+          "",
+          `**Revision:** ${result.result.revisionId}`,
+          `**PR:** #${result.result.prNumber} \u2014 ${result.result.prUrl}`,
+          `**Branch:** ${result.result.branch}`,
+          "",
+          "The revision stays in `draft`. Transition it to `open` once the PR is marked ready for review."
+        ];
+        return { content: [{ type: "text", text: lines.join("\n") }] };
+      } catch (err) {
+        const message = err instanceof Error ? err.message : String(err);
+        return { content: [{ type: "text", text: `Failed to create PR for revision: ${message}` }], isError: true };
+      }
+    }
+  );
 }
 
 // ../../libs/harmonica-services/src/mcp/tools/session-tools.ts
@@ -26504,9 +26639,10 @@ function createHttpClient(config2) {
   }
   const DEFAULT_TIMEOUT_MS = parseInt(process.env.MCP_HTTP_TIMEOUT_MS || "30000", 10);
   const LONG_RUNNING_TIMEOUT_MS = parseInt(process.env.MCP_HTTP_LONG_TIMEOUT_MS || "180000", 10);
+  const getApiKey = () => typeof config2.apiKey === "function" ? config2.apiKey() : config2.apiKey;
   async function request(method, path, body, timeoutMs) {
     const headers = {
-      Authorization: `Bearer ${config2.apiKey}`
+      Authorization: `Bearer ${getApiKey()}`
     };
     if (body !== void 0) {
       headers["Content-Type"] = "application/json";
@@ -26913,6 +27049,24 @@ function createHttpClient(config2) {
         return { success: false, error: { code: "TRANSITION_FAILED", message } };
       }
     },
+    createPrForRevision: async (revisionId) => {
+      try {
+        const result = await request(
+          "POST",
+          `/api/revisions/${encodeURIComponent(revisionId)}/pr`
+        );
+        if (result === void 0) {
+          return { success: false, error: { code: "REVISION_NOT_FOUND", message: "Revision not found" } };
+        }
+        return { success: true, result };
+      } catch (err) {
+        if (err instanceof ApiError) {
+          const parsed = safeParseErrorBody(err.body);
+          return { success: false, error: { code: parsed?.code ?? "PR_CREATE_FAILED", message: parsed?.error ?? err.message } };
+        }
+        return { success: false, error: { code: "PR_CREATE_FAILED", message: err instanceof Error ? err.message : String(err) } };
+      }
+    },
     reconcileRevisionToState: async (revisionId, targetState, options) => {
       const forwardPath = ["planning", "building", "ready_for_drop", "in_staging", "ready_for_production", "live"];
       const revision = await request("GET", `/api/revisions/${encodeURIComponent(revisionId)}`);
@@ -27019,7 +27173,7 @@ function createHttpClient(config2) {
         res = await fetch(url2, {
           method: "GET",
           headers: {
-            Authorization: `Bearer ${config2.apiKey}`,
+            Authorization: `Bearer ${getApiKey()}`,
             Accept: "text/event-stream"
           },
           signal: controller.signal
@@ -27450,6 +27604,14 @@ function createHttpClient(config2) {
       );
       return res.checks;
     },
+    listBeatVersionChecks: async (beatVersionId, projectId, checkType) => {
+      const qs = checkType ? `?type=${encodeURIComponent(checkType)}` : "";
+      const res = await request(
+        "GET",
+        `/api/projects/${encodeURIComponent(projectId)}/beat-versions/${encodeURIComponent(beatVersionId)}/checks${qs}`
+      );
+      return res.checks;
+    },
     listDropChecks: async (dropId, _projectId, checkType) => {
       const qs = checkType ? `?type=${encodeURIComponent(checkType)}` : "";
       const res = await request(
@@ -27697,7 +27859,7 @@ function loadConfig() {
   };
 }
 async function main() {
-  console.error(`[harmonica-mcp] v${"0.30.0"} starting\u2026`);
+  console.error(`[harmonica-mcp] v${"0.32.0"} starting\u2026`);
   const config2 = loadConfig();
   const client = createHttpClient({
     apiBaseUrl: config2.apiBaseUrl,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@codazen/harmonica-mcp",
-  "version": "0.30.0",
+  "version": "0.32.0",
   "description": "MCP server for Harmonica — connect Claude to your Harmonica projects",
   "license": "MIT",
   "bin": {