@aborruso/ckan-mcp-server 0.4.21 → 0.4.23

package/LOG.md

@@ -2,6 +2,16 @@
 
 ## 2026-01-26
 
+### Release v0.4.23
+
+- Fix MQA quality score maximum from 450 to 405 (correct max: 100+100+110+75+20)
+- Files modified: `src/tools/quality.ts:442`, `tests/integration/quality.test.ts:279,302`
+
+### Release v0.4.22
+
+- Fix MQA identifier normalization to handle dot separators (e.g. `c_g273:D.1727` -> `c_g273-d-1727`)
+- Workers deploy: https://ckan-mcp-server.andy-pr.workers.dev (2026-01-26)
+
 ### Release v0.4.21
 
 - Fix metrics parsing in Workers by switching to fetch and mocking fetch in tests

package/dist/index.js

@@ -2105,6 +2105,7 @@ Returns:
 import { z as z8 } from "zod";
 import axios2 from "axios";
 var MQA_API_BASE = "https://data.europa.eu/api/mqa/cache/datasets";
+var MQA_METRICS_BASE = "https://data.europa.eu/api/hub/repo/datasets";
 var ALLOWED_SERVER_PATTERNS = [
   /^https?:\/\/(www\.)?dati\.gov\.it/i
 ];
@@ -2112,7 +2113,7 @@ function isValidMqaServer(serverUrl) {
   return ALLOWED_SERVER_PATTERNS.some((pattern) => pattern.test(serverUrl));
 }
 function normalizeMqaIdentifier(identifier) {
-  return identifier.trim().replace(/:/g, "-").replace(/-+/g, "-").toLowerCase();
+  return identifier.trim().replace(/:/g, "-").replace(/\./g, "-").replace(/-+/g, "-").toLowerCase();
 }
 function buildMqaIdCandidates(identifier) {
   const base = normalizeMqaIdentifier(identifier);
@@ -2125,6 +2126,109 @@ function buildMqaIdCandidates(identifier) {
   }
   return candidates;
 }
+var DIMENSION_MAX = {
+  accessibility: 100,
+  findability: 100,
+  interoperability: 110,
+  reusability: 75,
+  contextuality: 20
+};
+function parseScoreValue(value) {
+  if (!value || typeof value !== "object") {
+    return void 0;
+  }
+  const raw = value["@value"];
+  if (typeof raw === "number") {
+    return raw;
+  }
+  if (typeof raw === "string") {
+    const parsed = Number(raw);
+    return Number.isFinite(parsed) ? parsed : void 0;
+  }
+  return void 0;
+}
+function decodeMetricsPayload(payload) {
+  if (payload && typeof payload === "object" && "@graph" in payload) {
+    return payload;
+  }
+  if (typeof payload === "string") {
+    try {
+      return JSON.parse(payload);
+    } catch {
+      return void 0;
+    }
+  }
+  if (payload instanceof ArrayBuffer) {
+    try {
+      const text = new TextDecoder().decode(new Uint8Array(payload));
+      return JSON.parse(text);
+    } catch {
+      return void 0;
+    }
+  }
+  if (ArrayBuffer.isView(payload)) {
+    try {
+      const view = payload;
+      const text = new TextDecoder().decode(new Uint8Array(view.buffer, view.byteOffset, view.byteLength));
+      return JSON.parse(text);
+    } catch {
+      return void 0;
+    }
+  }
+  return payload;
+}
+function extractMetricsScores(metricsData) {
+  const scores = {};
+  const parsed = decodeMetricsPayload(metricsData);
+  if (!parsed || typeof parsed !== "object") {
+    return scores;
+  }
+  const graph = parsed["@graph"];
+  if (!Array.isArray(graph)) {
+    return scores;
+  }
+  for (const node of graph) {
+    if (!node || typeof node !== "object") {
+      continue;
+    }
+    const metricRef = node["dqv:isMeasurementOf"];
+    if (!metricRef) {
+      continue;
+    }
+    const metricId = typeof metricRef === "string" ? metricRef : metricRef["@id"];
+    if (typeof metricId !== "string") {
+      continue;
+    }
+    const value = parseScoreValue(node["dqv:value"]);
+    if (value === void 0) {
+      continue;
+    }
+    if (metricId.endsWith("#accessibilityScoring")) {
+      scores.accessibility = value;
+    } else if (metricId.endsWith("#findabilityScoring")) {
+      scores.findability = value;
+    } else if (metricId.endsWith("#interoperabilityScoring")) {
+      scores.interoperability = value;
+    } else if (metricId.endsWith("#reusabilityScoring")) {
+      scores.reusability = value;
+    } else if (metricId.endsWith("#contextualityScoring")) {
+      scores.contextuality = value;
+    } else if (metricId.endsWith("#scoring")) {
+      scores.total = value;
+    }
+  }
+  return scores;
+}
+function findNonMaxDimensions(scores) {
+  const nonMax = [];
+  Object.keys(DIMENSION_MAX).forEach((dimension) => {
+    const value = scores[dimension];
+    if (typeof value === "number" && value < DIMENSION_MAX[dimension]) {
+      nonMax.push(dimension);
+    }
+  });
+  return nonMax;
+}
 async function getMqaQuality(serverUrl, datasetId) {
   const dataset = await makeCkanRequest(
     serverUrl,
@@ -2138,6 +2242,7 @@ async function getMqaQuality(serverUrl, datasetId) {
   }
   for (const europeanId of candidates) {
     const mqaUrl = `${MQA_API_BASE}/${europeanId}`;
+    const metricsUrl = `${MQA_METRICS_BASE}/${europeanId}/metrics`;
     try {
       const response = await axios2.get(mqaUrl, {
         timeout: 3e4,
@@ -2145,7 +2250,41 @@ async function getMqaQuality(serverUrl, datasetId) {
           "User-Agent": "CKAN-MCP-Server/1.0"
         }
       });
-      return response.data;
+      let metricsPayload;
+      try {
+        const metricsResponse = await fetch(metricsUrl, {
+          headers: {
+            "User-Agent": "CKAN-MCP-Server/1.0"
+          }
+        });
+        if (!metricsResponse.ok) {
+          throw new Error(`MQA metrics error: ${metricsResponse.status} ${metricsResponse.statusText}`);
+        }
+        try {
+          metricsPayload = await metricsResponse.json();
+        } catch {
+          metricsPayload = await metricsResponse.text();
+        }
+      } catch (metricsError) {
+        if (metricsError instanceof Error) {
+          throw new Error(`MQA metrics error: ${metricsError.message}`);
+        }
+        throw metricsError;
+      }
+      const scores = extractMetricsScores(metricsPayload);
+      const resultEntry = response.data?.result?.results?.[0];
+      const portalId = resultEntry?.info?.["dataset-id"] || europeanId;
+      const breakdown = {
+        scores,
+        nonMaxDimensions: findNonMaxDimensions(scores),
+        metricsUrl,
+        mqaUrl,
+        portalId
+      };
+      return {
+        mqa: response.data,
+        breakdown
+      };
     } catch (error) {
       if (axios2.isAxiosError(error)) {
         if (error.response?.status === 404) {
@@ -2222,9 +2361,11 @@ function metricAvailability(section, availabilityKey, statusKey) {
   return void 0;
 }
 function normalizeQualityData(data) {
-  const resultEntry = data?.result?.results?.[0];
+  const mqaData = data?.mqa ?? data;
+  const breakdown = data?.breakdown;
+  const resultEntry = mqaData?.result?.results?.[0];
   if (!resultEntry || typeof resultEntry !== "object") {
-    return data;
+    return { ...data, breakdown };
   }
   return {
     id: resultEntry.info?.["dataset-id"],
@@ -2247,7 +2388,12 @@ function normalizeQualityData(data) {
       category: metricBoolean(resultEntry.findability, "categoryAvailability") !== void 0 ? { available: metricBoolean(resultEntry.findability, "categoryAvailability") } : void 0,
       spatial: metricBoolean(resultEntry.findability, "spatialAvailability") !== void 0 ? { available: metricBoolean(resultEntry.findability, "spatialAvailability") } : void 0,
       temporal: metricBoolean(resultEntry.findability, "temporalAvailability") !== void 0 ? { available: metricBoolean(resultEntry.findability, "temporalAvailability") } : void 0
-    }
+    },
+    contextuality: {
+      byteSize: metricAvailability(resultEntry.contextuality, "byteSizeAvailability"),
+      rights: metricAvailability(resultEntry.contextuality, "rightsAvailability")
+    },
+    breakdown
   };
 }
 function formatQualityMarkdown(data, datasetId) {
@@ -2256,7 +2402,32 @@ function formatQualityMarkdown(data, datasetId) {
   lines.push(`# Quality Metrics for Dataset: ${datasetId}`);
   lines.push("");
   if (normalized.info?.score !== void 0) {
-    lines.push(`**Overall Score**: ${normalized.info.score}/405`);
+    lines.push(`**Overall Score**: ${normalized.info.score}/450`);
+    lines.push("");
+  }
+  if (normalized.breakdown?.scores) {
+    lines.push("## Dimension Scores");
+    const scores = normalized.breakdown.scores;
+    const order = [
+      ["accessibility", "Accessibility", DIMENSION_MAX.accessibility],
+      ["findability", "Findability", DIMENSION_MAX.findability],
+      ["interoperability", "Interoperability", DIMENSION_MAX.interoperability],
+      ["reusability", "Reusability", DIMENSION_MAX.reusability],
+      ["contextuality", "Contextuality", DIMENSION_MAX.contextuality]
+    ];
+    for (const [key, label, max] of order) {
+      const value = scores[key];
+      if (typeof value === "number") {
+        const isMax = value >= max;
+        const status = isMax ? "\u2705" : "\u26A0\uFE0F";
+        lines.push(`- ${label}: ${value}/${max} ${status}${isMax ? "" : ` (max ${max})`}`);
+      }
+    }
+    if (normalized.breakdown.nonMaxDimensions.length > 0) {
+      lines.push(`- Non-max dimension(s): ${normalized.breakdown.nonMaxDimensions.join(", ")}`);
+    } else if (Object.keys(scores).length > 0) {
+      lines.push("- Non-max dimension(s): none");
+    }
     lines.push("");
   }
   if (normalized.accessibility) {
@@ -2308,16 +2479,27 @@ function formatQualityMarkdown(data, datasetId) {
     }
     lines.push("");
   }
+  if (normalized.contextuality) {
+    lines.push("## Contextuality");
+    if (normalized.contextuality.byteSize !== void 0) {
+      lines.push(`- Byte Size: ${normalized.contextuality.byteSize.available ? "\u2713" : "\u2717"} Available`);
+    }
+    if (normalized.contextuality.rights !== void 0) {
+      lines.push(`- Rights: ${normalized.contextuality.rights.available ? "\u2713" : "\u2717"} Available`);
+    }
+    lines.push("");
+  }
   lines.push("---");
-  const portalId = normalized.id || datasetId;
+  const portalId = normalized.breakdown?.portalId || normalized.id || datasetId;
   lines.push(`Portal: https://data.europa.eu/data/datasets/${portalId}/quality?locale=it`);
-  lines.push(`Source: ${MQA_API_BASE}/${portalId}`);
+  lines.push(`MQA source: ${MQA_API_BASE}/${portalId}`);
+  lines.push(`Metrics endpoint: ${normalized.breakdown?.metricsUrl || `${MQA_METRICS_BASE}/${portalId}/metrics`}`);
   return lines.join("\n");
 }
 function registerQualityTools(server2) {
   server2.tool(
     "ckan_get_mqa_quality",
-    "Get MQA (Metadata Quality Assurance) quality metrics for a dataset on dati.gov.it. Returns quality score and detailed metrics (accessibility, reusability, interoperability, findability) from data.europa.eu. Only works with dati.gov.it server.",
+    "Get MQA (Metadata Quality Assurance) quality metrics for a dataset on dati.gov.it. Returns quality score and detailed metrics (accessibility, reusability, interoperability, findability, contextuality) from data.europa.eu. Only works with dati.gov.it server.",
     {
       server_url: z8.string().url().describe("Base URL of dati.gov.it (e.g., https://www.dati.gov.it/opendata)"),
       dataset_id: z8.string().describe("Dataset ID or name"),
@@ -2865,7 +3047,7 @@ var registerAllPrompts = (server2) => {
 function createServer() {
   return new McpServer({
     name: "ckan-mcp-server",
-    version: "0.4.17"
+    version: "0.4.22"
   });
 }
 function registerAll(server2) {