ma-agents 3.15.1 → 3.15.3

package/lib/bmad-extension-plugin/.claude-plugin/marketplace.json

@@ -37,7 +37,7 @@
       "name": "ma-skills",
       "source": "./",
       "description": "ma-agents extension module providing enterprise SDLC personas and operational workflow skills.",
-      "version": "3.15.1",
+      "version": "3.15.3",
       "author": {
         "name": "Alon Mayaffit"
       },

package/lib/knowledge-atlas/dist/atlas.cjs

@@ -2106,7 +2106,9 @@ function classifyFile(fileId) {
   if (/^\d+-\d+-[a-z0-9][a-z0-9-]*\.md$/.test(basename4)) return "story";
   if (/product-brief/.test(basename4)) return "product-brief";
   if (lowerFileId.includes("/briefs/")) return "brief";
-  if (/prd/.test(basename4) || /product-requirements/.test(basename4)) return "prd";
+  if ((/prd/.test(basename4) || /product-requirements/.test(basename4)) && !/(validation|report|checklist|review|cohesion|assessment)/.test(basename4)) {
+    return "prd";
+  }
   if (/architecture/.test(basename4)) return "architecture";
   if (/epic/.test(basename4)) return "epics";
   if (basename4.endsWith(".md") || basename4.endsWith(".yaml") || basename4.endsWith(".yml")) {
@@ -12154,14 +12156,23 @@ function renderDocumentPage(fileRec, manifest, projectRoot, navFlags, resolveAbs
   const statusBadgesHtml = renderStatusBadges(fileRec.statuses);
   const existingIds = /* @__PURE__ */ new Set();
   for (const m of renderedHtml.matchAll(/\sid="([^"]+)"/g)) existingIds.add(m[1]);
-  const anchorTargets = [];
+  const escRe = (s) => s.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
+  const inlinePlacements = [];
+  const fallbackAnchors = [];
   for (const node of fileNodes) {
-    if (node.anchor && !existingIds.has(node.anchor)) {
-      anchorTargets.push(`<span class="entity-anchor" id="${esc4(node.anchor)}"></span>`);
-      existingIds.add(node.anchor);
-    }
-  }
-  const anchorTargetsHtml = anchorTargets.join("\n");
+    const a = node.anchor;
+    if (!a || existingIds.has(a)) continue;
+    existingIds.add(a);
+    const m = new RegExp('<a\\b[^>]*href="[^"]*#' + escRe(a) + '"', "i").exec(renderedHtml);
+    if (m) inlinePlacements.push({ idx: m.index, anchor: a });
+    else fallbackAnchors.push(a);
+  }
+  inlinePlacements.sort((x, y) => y.idx - x.idx);
+  for (const p of inlinePlacements) {
+    const span = `<span class="entity-anchor" id="${esc4(p.anchor)}"></span>`;
+    renderedHtml = renderedHtml.slice(0, p.idx) + span + renderedHtml.slice(p.idx);
+  }
+  const anchorTargetsHtml = fallbackAnchors.map((a) => `<span class="entity-anchor" id="${esc4(a)}"></span>`).join("\n");
   const backLinkHtml = '<a class="back-link" href="../index.html">&larr; Back to Home</a>';
   const pageBody = [
     '<div class="content-wrap">',
@@ -13061,6 +13072,7 @@ function buildAppScript() {
   // mouse (finding #8: tooltip was shown with display:block but no left/top,
   // so it rendered at the bottom of <main>).
   var lastHoverNode = null;
+  var selectedLink = null; // edge highlighted on click: drawn bold + its two endpoints ringed
 
   function getFilteredData() {
     var nodes = allNodes;
@@ -13183,6 +13195,18 @@ function buildAppScript() {
         ctx.fillStyle = n.color || '#888';
         traceNodeShapePath(ctx, n.shape, n.x, n.y, size);
         ctx.fill();
+        // Ring the two endpoints of the selected edge so both ends are easy to
+        // find, however far apart they sit in the layout.
+        if (selectedLink) {
+          var s = selectedLink.source, t = selectedLink.target;
+          if (n === s || n === t) {
+            ctx.beginPath();
+            ctx.arc(n.x, n.y, size + 3.5, 0, 2 * Math.PI);
+            ctx.strokeStyle = '#111827';
+            ctx.lineWidth = 1.8;
+            ctx.stroke();
+          }
+        }
       })
       // Matching hit-test paint: without this, force-graph's click/hover
       // detection falls back to a generic area that no longer matches the
@@ -13196,12 +13220,30 @@ function buildAppScript() {
       })
       .linkSource('source')
       .linkTarget('target')
-      .linkColor(function(e) { return e.color || '#aaa'; })
+      // Selected edge is drawn dark + thick so you can trace it end to end.
+      .linkColor(function(e) { return e === selectedLink ? '#111827' : (e.color || '#aaa'); })
+      .linkWidth(function(e) { return e === selectedLink ? 3.5 : (e.dashed ? 0.5 : 1); })
       // Story 29.8 - dashed stroke for weak references edges; solid otherwise.
       .linkLineDash(function(e) { return e.dashed ? [4, 3] : null; })
-      .linkDirectionalArrowLength(4)
+      .linkDirectionalArrowLength(function(e) { return e === selectedLink ? 7 : 4; })
       .linkDirectionalArrowRelPos(1)
       .linkCurvature(0.1)
+      // Click an edge to highlight it (bold + ringed endpoints); click it again
+      // or click empty space to clear. Re-applying the accessors forces an
+      // immediate repaint of the new selection state.
+      .onLinkClick(function(e) {
+        selectedLink = (selectedLink === e) ? null : e;
+        graph.linkWidth(graph.linkWidth()).linkColor(graph.linkColor());
+      })
+      .onLinkHover(function(e) {
+        if (container) container.style.cursor = e ? 'pointer' : 'default';
+        if (!tooltip) return;
+        if (!e) { tooltip.style.display = 'none'; return; }
+        var s = (e.source && e.source.label) || e.source;
+        var t = (e.target && e.target.label) || e.target;
+        tooltip.style.display = 'block';
+        tooltip.textContent = s + '  \u2192  ' + t + (e.kind ? '  [' + e.kind + ']' : '');
+      })
       .onNodeClick(function(n) {
         // Navigate to the reading view via the server-resolved doc page (the
         // 'page' field, e.g. "docs/<slug>.html"), NOT the raw fileId \u2014 the
@@ -13242,6 +13284,10 @@ function buildAppScript() {
       .onBackgroundClick(function() {
         if (tooltip) tooltip.style.display = 'none';
         lastHoverNode = null;
+        if (selectedLink) {
+          selectedLink = null;
+          graph.linkWidth(graph.linkWidth()).linkColor(graph.linkColor());
+        }
       });
   }
 

package/lib/knowledge-atlas/src/render/document.ts

@@ -299,16 +299,36 @@ export function renderDocumentPage(
   // to a heading slug (`story-11-oauth2-integration`). We emit an invisible
   // anchor target for every file node whose anchor is not already a heading id,
   // so every cross-link `…#anchor` resolves to a real location on the page.
+  // Place each entity's anchor at the FIRST place that entity is mentioned in
+  // the body (its first linkified occurrence: an <a … href="…#anchor">), so a
+  // cross-link or graph-node click lands where the entity is actually discussed.
+  // Previously every anchor was emitted as one clump at the top of the article,
+  // so every #anchor (e.g. all 9 epics) resolved to the same spot. Anchors with
+  // no linked mention fall back to the top clump.
   const existingIds = new Set<string>();
   for (const m of renderedHtml.matchAll(/\sid="([^"]+)"/g)) existingIds.add(m[1]!);
-  const anchorTargets: string[] = [];
+  const escRe = (s: string) => s.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
+  const inlinePlacements: { idx: number; anchor: string }[] = [];
+  const fallbackAnchors: string[] = [];
   for (const node of fileNodes) {
-    if (node.anchor && !existingIds.has(node.anchor)) {
-      anchorTargets.push(`<span class="entity-anchor" id="${esc(node.anchor)}"></span>`);
-      existingIds.add(node.anchor);
-    }
+    const a = node.anchor;
+    if (!a || existingIds.has(a)) continue;
+    existingIds.add(a);
+    // First <a> tag whose href ends with this exact anchor fragment.
+    const m = new RegExp('<a\\b[^>]*href="[^"]*#' + escRe(a) + '"', 'i').exec(renderedHtml);
+    if (m) inlinePlacements.push({ idx: m.index, anchor: a });
+    else fallbackAnchors.push(a);
+  }
+  // Insert the invisible anchor spans just before their target links. Apply from
+  // the last position to the first so earlier indices stay valid.
+  inlinePlacements.sort((x, y) => y.idx - x.idx);
+  for (const p of inlinePlacements) {
+    const span = `<span class="entity-anchor" id="${esc(p.anchor)}"></span>`;
+    renderedHtml = renderedHtml.slice(0, p.idx) + span + renderedHtml.slice(p.idx);
   }
-  const anchorTargetsHtml = anchorTargets.join('\n');
+  const anchorTargetsHtml = fallbackAnchors
+    .map(a => `<span class="entity-anchor" id="${esc(a)}"></span>`)
+    .join('\n');
 
   // Back link
   const backLinkHtml = '<a class="back-link" href="../index.html">&larr; Back to Home</a>';

package/lib/knowledge-atlas/src/render/graph.ts

@@ -648,6 +648,7 @@ function buildAppScript(): string {
   // mouse (finding #8: tooltip was shown with display:block but no left/top,
   // so it rendered at the bottom of <main>).
   var lastHoverNode = null;
+  var selectedLink = null; // edge highlighted on click: drawn bold + its two endpoints ringed
 
   function getFilteredData() {
     var nodes = allNodes;
@@ -770,6 +771,18 @@ function buildAppScript(): string {
         ctx.fillStyle = n.color || '#888';
         traceNodeShapePath(ctx, n.shape, n.x, n.y, size);
         ctx.fill();
+        // Ring the two endpoints of the selected edge so both ends are easy to
+        // find, however far apart they sit in the layout.
+        if (selectedLink) {
+          var s = selectedLink.source, t = selectedLink.target;
+          if (n === s || n === t) {
+            ctx.beginPath();
+            ctx.arc(n.x, n.y, size + 3.5, 0, 2 * Math.PI);
+            ctx.strokeStyle = '#111827';
+            ctx.lineWidth = 1.8;
+            ctx.stroke();
+          }
+        }
       })
       // Matching hit-test paint: without this, force-graph's click/hover
       // detection falls back to a generic area that no longer matches the
@@ -783,12 +796,30 @@ function buildAppScript(): string {
       })
       .linkSource('source')
       .linkTarget('target')
-      .linkColor(function(e) { return e.color || '#aaa'; })
+      // Selected edge is drawn dark + thick so you can trace it end to end.
+      .linkColor(function(e) { return e === selectedLink ? '#111827' : (e.color || '#aaa'); })
+      .linkWidth(function(e) { return e === selectedLink ? 3.5 : (e.dashed ? 0.5 : 1); })
       // Story 29.8 - dashed stroke for weak references edges; solid otherwise.
       .linkLineDash(function(e) { return e.dashed ? [4, 3] : null; })
-      .linkDirectionalArrowLength(4)
+      .linkDirectionalArrowLength(function(e) { return e === selectedLink ? 7 : 4; })
       .linkDirectionalArrowRelPos(1)
       .linkCurvature(0.1)
+      // Click an edge to highlight it (bold + ringed endpoints); click it again
+      // or click empty space to clear. Re-applying the accessors forces an
+      // immediate repaint of the new selection state.
+      .onLinkClick(function(e) {
+        selectedLink = (selectedLink === e) ? null : e;
+        graph.linkWidth(graph.linkWidth()).linkColor(graph.linkColor());
+      })
+      .onLinkHover(function(e) {
+        if (container) container.style.cursor = e ? 'pointer' : 'default';
+        if (!tooltip) return;
+        if (!e) { tooltip.style.display = 'none'; return; }
+        var s = (e.source && e.source.label) || e.source;
+        var t = (e.target && e.target.label) || e.target;
+        tooltip.style.display = 'block';
+        tooltip.textContent = s + '  →  ' + t + (e.kind ? '  [' + e.kind + ']' : '');
+      })
       .onNodeClick(function(n) {
         // Navigate to the reading view via the server-resolved doc page (the
         // 'page' field, e.g. "docs/<slug>.html"), NOT the raw fileId — the
@@ -829,6 +860,10 @@ function buildAppScript(): string {
       .onBackgroundClick(function() {
         if (tooltip) tooltip.style.display = 'none';
         lastHoverNode = null;
+        if (selectedLink) {
+          selectedLink = null;
+          graph.linkWidth(graph.linkWidth()).linkColor(graph.linkColor());
+        }
       });
   }
 

package/lib/knowledge-atlas/src/scan/scanner.ts

@@ -126,8 +126,20 @@ export function classifyFile(fileId: string): DocType {
   // e.g. briefs/epic-overview.md must be 'brief', not 'epics'.
   if (lowerFileId.includes('/briefs/')) return 'brief';
 
-  // PRD: filename contains 'prd' or matches product-requirements
-  if (/prd/.test(basename) || /product-requirements/.test(basename)) return 'prd';
+  // PRD: filename contains 'prd' or matches product-requirements — but NOT a
+  // derivative document *about* the PRD (validation report, checklist, review,
+  // cohesion/assessment report). Those score or critique the requirements; the
+  // requirements themselves live in the PRD proper. Classifying e.g.
+  // `prd-validation-report.md` as 'prd' made it a rival definition-site, so an
+  // FR the PRD actually defines could anchor in the report instead — the real
+  // Prophesee bug where FR1/FR3/FR21 links pointed at prd-validation-report.html
+  // rather than the PRD. Let those fall through to 'report'.
+  if (
+    (/prd/.test(basename) || /product-requirements/.test(basename)) &&
+    !/(validation|report|checklist|review|cohesion|assessment)/.test(basename)
+  ) {
+    return 'prd';
+  }
 
   // Architecture document
   if (/architecture/.test(basename)) return 'architecture';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ma-agents",
-  "version": "3.15.1",
+  "version": "3.15.3",
   "description": "NPX tool to install skills for AI coding agents (Claude Code, Gemini, Copilot, Kilocode, Cline, Cursor, Roo Code)",
   "main": "index.js",
   "bin": {