gasp-standard-mcp 1.7.1 → 1.8.0

package/README.md

@@ -56,6 +56,7 @@ Restart your MCP client after adding the config.
 - 60 temporal mappings (grain, frequency, period for every metric node)
 - Access control classifications for every graph metric (sensitivity tier + access scope)
 - CEL `change_nature` field for revenue quality analysis (Adjusted NRR/GRR)
+- Per-department cross-references (e.g. NRR under Customer Success, Gross Margin under Finance) now resolve to the full canonical content via slug lookup
 - Sources: the canonical [GASP Standard](https://gaspwiki.com) files
 
 ## Examples

package/build/loader.js

@@ -97,6 +97,57 @@ export function loadStandard() {
             }
         }
     }
+    // Enrich cross-reference stubs with canonical content fields.
+    //
+    // Some metrics intentionally appear in multiple department files — e.g. NRR
+    // is defined canonically in core.md and listed as a "See [Core](core.md#...)"
+    // stub in customer-success.md. Per-department records are needed to drive
+    // department-scoped routes (e.g. /v1/metrics/customer-success/...), but
+    // consumers of standard.json that look up by slug expect every record to
+    // carry full content. Without this pass, Map-based lookups silently get the
+    // empty stub instead of the canonical record.
+    //
+    // Strategy: keep stub identity (department, category, name, slug, rawMarkdown)
+    // but copy any content field that's empty on the stub from the canonical.
+    const canonicalBySlug = new Map();
+    for (const m of metrics) {
+        if (m.slug && m.definition && !canonicalBySlug.has(m.slug)) {
+            canonicalBySlug.set(m.slug, m);
+        }
+    }
+    for (const m of metrics) {
+        if (m.definition)
+            continue; // already canonical or non-stub
+        const seeMatch = m.rawMarkdown.match(/See \[([^\]]+)\]\(([^)]+)\)/);
+        if (!seeMatch)
+            continue;
+        const link = seeMatch[2];
+        const anchor = link.includes("#") ? link.split("#")[1] : null;
+        if (!anchor)
+            continue;
+        const canon = canonicalBySlug.get(anchor) ||
+            canonicalBySlug.get(m.slug) ||
+            [...canonicalBySlug.values()].find((c) => c.slug.includes(anchor));
+        if (!canon || canon === m)
+            continue;
+        if (!m.definition)
+            m.definition = canon.definition;
+        if (!m.formulas?.length)
+            m.formulas = [...canon.formulas];
+        if (!m.benchmarks)
+            m.benchmarks = canon.benchmarks;
+        if (!m.insight)
+            m.insight = canon.insight;
+        if (!m.commonMistakes?.length && canon.commonMistakes)
+            m.commonMistakes = [...canon.commonMistakes];
+        if (!m.sources?.length)
+            m.sources = [...canon.sources];
+        if (!m.aliases?.length)
+            m.aliases = [...canon.aliases];
+        // dualLens and dualLensNote are intentionally NOT enriched — they are a
+        // Core-only construct (asserted by data-integrity tests). Consumers that
+        // need dual-lens content should resolve via slug to the Core record.
+    }
     // Load query templates
     const queryTemplatePath = join(standardDir, "query-templates.json");
     let queryTemplates;