@pipeworx/mcp-datacube-sk 0.1.1 → 0.1.3

package/README.md

@@ -2,7 +2,7 @@
 
 DATAcube — Statistical Office of the Slovak Republic (data.statistics.sk).
 
-Part of [Pipeworx](https://pipeworx.io) — an MCP gateway connecting AI agents to 1679+ live data sources.
+Part of [Pipeworx](https://pipeworx.io) — an MCP gateway connecting AI agents to 1684+ live data sources.
 
 ## Tools
 
@@ -56,7 +56,7 @@ directly, instead of just this one's:
 }
 ```
 
-Both URLs reach the same gateway and the same 1679+ data sources. The
+Both URLs reach the same gateway and the same 1684+ data sources. The
 only difference is which pack's tools are listed **directly**; `ask_pipeworx`
 reaches all of them from either one.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pipeworx/mcp-datacube-sk",
-  "version": "0.1.1",
+  "version": "0.1.3",
   "description": "DATAcube — Statistical Office of the Slovak Republic (data.statistics.sk).",
   "type": "module",
   "main": "src/index.ts",
@@ -26,7 +26,7 @@
     "@cloudflare/workers-types": "^4.20260405.1"
   },
   "pipeworx": {
-    "sourceHash": "v1-e9caf9eac19b163f18a0c3fd9af30b6e048956390825895be371c14bf9a10fd3",
-    "sourceCommit": "839ab210d969ef7f822cd747ea516f6bb49e9231"
+    "sourceHash": "v1-1f5767dfe4a92bdc524970d0bd33d2502d4fd95a7ec3e33a948e5bfff971addc",
+    "sourceCommit": "a00767651388cfd017fdeac82188622721db2051"
   }
 }

package/server.json

@@ -3,7 +3,7 @@
   "name": "io.github.pipeworx-io/datacube-sk",
   "title": "Datacube Sk",
   "description": "DATAcube — Statistical Office of the Slovak Republic (data.statistics.sk).",
-  "version": "0.1.1",
+  "version": "0.1.3",
   "websiteUrl": "https://pipeworx.io/packs/datacube-sk",
   "repository": {
     "url": "https://github.com/pipeworx-io/mcp-datacube-sk",

package/src/index.ts

@@ -452,7 +452,42 @@ async function fetchWithTimeout(
         ),
       );
     }
-    throw err;
+    // Fleet #2382. Everything that isn't a timeout/abort here is a genuine
+    // NETWORK-LEVEL failure — DNS resolution, connection refused, TLS handshake,
+    // Cloudflare's own "Network connection lost." — meaning `fetch()` itself
+    // threw and no HTTP response of any kind was ever received. Until this fix
+    // that raw exception was rethrown VERBATIM: a bare `TypeError: fetch failed`
+    // (or the Workers-runtime equivalent) names no upstream, carries no class
+    // token, and reads exactly like a defect in OUR code — because it says
+    // nothing about the call at all. It landed in `error`, the tier that means
+    // "Pipeworx has a defect", for every one of the (at the time of writing)
+    // ~470 packs that call this helper directly with no wrapper of their own.
+    //
+    // `dexscreener` hit this independently (fleet #1579) and fixed it with a
+    // bespoke per-pack try/catch around `fetchWithTimeout`. That fix is correct
+    // but only covers one pack; every other caller of this shared helper still
+    // leaked the raw exception. Moving the same fix HERE — the one place that
+    // already carries the timeout case — covers every pack that uses
+    // `fetchWithTimeout` without a wrapper, for free, and without widening
+    // `classifyToolError`'s regex list: the fix is giving the message a proper
+    // `upstream_down:` token at the point the two facts (no response was ever
+    // received, and which host we were trying to reach) are actually in hand,
+    // not teaching the classifier to guess from prose after the fact.
+    //
+    // Safe on the same grounds as the timeout branch above: no argument a
+    // caller passes can make `fetch()` itself throw a connection-level error,
+    // so this is always an availability failure, never a caller mistake. Same
+    // `markInternalOrigin` treatment — an origin we run that never answered is
+    // still ours, not a third party's outage.
+    const raw = err instanceof Error ? err.message : String(err);
+    throw new Error(
+      markInternalOrigin(
+        `upstream_down: could not reach ${name} at all (${raw.slice(0, 160)}). ` +
+          `No request reached ${name}, so this says NOTHING about whether the arguments you passed ` +
+          'are valid — do not re-check them on the strength of this error. Retry shortly.',
+        url,
+      ),
+    );
   }
 }
 
@@ -795,17 +830,61 @@ async function getData(args: Record<string, unknown>): Promise<unknown> {
     throw new Error('selections must be a non-empty array of strings (one per dimension, in dataset_dimensions order). Use dataset_dimensions to find dimension order and valid codes.');
   }
   const path = (sel as string[]).map((s) => encodeURIComponent(s.trim())).join('/');
-  return dcGet(`/dataset/${encodeURIComponent(cube)}/${path}?lang=${lang}`);
+  let out: { size?: number[]; id?: string[]; value?: unknown[] | Record<string, unknown> };
+  try {
+    out = (await dcGet(`/dataset/${encodeURIComponent(cube)}/${path}?lang=${lang}`)) as typeof out;
+  } catch (err) {
+    // DATAcube answers an unknown cube with 400 {"status_message":"Name API not
+    // foud: <cube>"} (sic). That threw as a Pipeworx defect (fleet #2496).
+    if (err instanceof DcHttpError && err.status === 400 && /not fou?n?d/i.test(err.body)) {
+      return {
+        error: 'not_found',
+        message: `DATAcube has no cube "${cube}". Call list_datasets({query: "..."}) to find the cube code.`,
+        cube,
+      };
+    }
+    throw err;
+  }
+  // A selection code the cube does not have is NOT an error upstream: DATAcube
+  // returns a normal JSON-stat dataset with that dimension's size 0 and no
+  // values — a plain success with nothing in it (fleet #2496). Say so.
+  const size = Array.isArray(out?.size) ? out.size : [];
+  const zeroDims = (out?.id ?? []).filter((_, i) => size[i] === 0);
+  const noValues = Array.isArray(out?.value) ? out.value.length === 0 : out?.value != null && Object.keys(out.value).length === 0;
+  if (zeroDims.length > 0 || noValues) {
+    return {
+      ...out,
+      empty_reason: 'no_match',
+      note:
+        `DATAcube cube "${cube}" returned no values for selections ${JSON.stringify(sel)}` +
+        `${zeroDims.length ? ` — nothing matched in dimension(s) ${zeroDims.join(', ')}` : ''}. ` +
+        `Check the codes with dataset_dimensions({cube: "${cube}"}).`,
+    };
+  }
+  return out;
 }
 
 async function dcGet(path: string): Promise<unknown> {
   const res = await pwFetch(`${BASE}${path}`, {
     headers: { Accept: 'application/json', 'User-Agent': UA },
   });
-  if (!res.ok) throw new Error(`DATAcube: ${res.status} ${await res.text().then((t) => t.slice(0, 200))}`);
+  if (!res.ok) {
+    const body = await res.text().then((t) => t.slice(0, 200));
+    throw new DcHttpError(res.status, body, `DATAcube: ${res.status} ${body}`);
+  }
   return res.json();
 }
 
+class DcHttpError extends Error {
+  readonly status: number;
+  readonly body: string;
+  constructor(status: number, body: string, message: string) {
+    super(message);
+    this.status = status;
+    this.body = body;
+  }
+}
+
 function cubeFromHref(href: string | undefined): string | undefined {
   if (!href) return undefined;
   const m = href.match(/\/dataset\/([^/?]+)/);

package/src/server.ts

@@ -9,7 +9,7 @@ import { CallToolRequestSchema, ListToolsRequestSchema } from '@modelcontextprot
 import pack from './index.js';
 
 const server = new Server(
-  { name: '@pipeworx/mcp-datacube-sk', version: '0.1.1' },
+  { name: '@pipeworx/mcp-datacube-sk', version: '0.1.3' },
   { capabilities: { tools: {} } },
 );