@roadmapperai/mcp 0.6.0 → 0.7.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@roadmapperai/mcp",
-  "version": "0.6.0",
+  "version": "0.7.1",
   "description": "Roadmapper AI MCP server — exposes a planning surface (themes, capabilities, tasks, sprints, PRs) to coding agents via stdio JSON-RPC. Pairs with the Roadmapper AI workspace at dashboard.roadmapperai.com.",
   "keywords": [
     "mcp",

package/server.mjs

@@ -1902,7 +1902,14 @@ async function callTool(name, args) {
       session.themesListedAt = Date.now();
       return suggestThemeFor(args, projected);
     case "link_pr":
-      return linkPR(args, projected, seed, wsId);
+      // Pass null as the seed reference. linkPR's only use of seed
+      // is to look up legacy seed-bundled PR entries on a task; in
+      // the live path the canonical PRs live on the projected task,
+      // so seed is never the source of truth. Passing the previously-
+      // undefined identifier `seed` raised "seed is not defined" at
+      // runtime for every link_pr call — caught by the live test
+      // drive that exposed bug #5.
+      return linkPR(args, projected, null, wsId);
     case "archive_task":
       return archiveLifecycle("task", "archive", args, wsId);
     case "archive_capability":
@@ -1943,7 +1950,16 @@ async function proposeTask(args, projected, wsId) {
   if (!cap) return errorResult(`Capability ${args.capabilityId} not found.`);
   const titleErr = validateName(args.title, 5);
   if (titleErr) return errorResult(titleErr);
-  if (args.effort && !VALID_EFFORTS.has(args.effort))
+  // Effort is required end-to-end: the schema flags it, and the
+  // Postgres RPC refuses without it. We also raise here so older
+  // clients that ignore the schema get a clear error message before
+  // a round-trip.
+  if (!args.effort) {
+    return errorResult(
+      "effort is required (one of XS, S, M, L, XL). Always size the task — XS (≤2h), S (≤1d), M (~1-3d), L (~1-2w), XL (>2w)."
+    );
+  }
+  if (!VALID_EFFORTS.has(args.effort))
     return errorResult(`Invalid effort ${args.effort}.`);
   if (args.priority && !VALID_PRIORITIES.has(args.priority))
     return errorResult(`Invalid priority ${args.priority}.`);
@@ -1960,7 +1976,7 @@ async function proposeTask(args, projected, wsId) {
   )
     return errorResult(`expectedScope must be a positive number, got ${args.expectedScope}.`);
 
-  const effort = args.effort ?? "M";
+  const effort = args.effort;
   const start = todayISO();
   // Target dates are day-resolution; round up so sub-day estimates
   // (XS=0.25, S=0.5) still nudge the target at least one day out.
@@ -2269,10 +2285,32 @@ function suggestCapabilityFor(args, projected) {
     (c) => effectiveCapabilityStatus(c, projected.tasks) !== "delivered"
   );
   const query = tokenize(desc);
+  // Build a richer haystack per capability: include the parent
+  // theme's name+description (vocabulary often overlaps with the
+  // query) and the titles of attached tasks (the *implementation*
+  // language, which is closer to how engineers describe new work
+  // than the bet-style capability outcome statement). This makes
+  // matches that previously scored near zero hit the 0.2+ band
+  // where the model takes them seriously.
+  const themeById = new Map(
+    (projected.themes ?? []).map((t) => [t.id, t])
+  );
   const ranked = activeCaps
     .map((c) => {
+      const theme = themeById.get(c.pillarId);
+      const taskTitles = (projected.tasks ?? [])
+        .filter((t) => t.capabilityId === c.id)
+        .map((t) => t.title)
+        .join(" ");
       const hay = tokenize(
-        `${c.name} ${c.description ?? ""} ${c.outcome ?? ""}`
+        [
+          c.name,
+          c.description ?? "",
+          c.outcome ?? "",
+          theme?.name ?? "",
+          theme?.description ?? "",
+          taskTitles,
+        ].join(" ")
       );
       return { capability: c, score: jaccardScore(query, hay) };
     })