@pipeworx/mcp-cbs-nl 0.1.1 → 0.1.3

package/README.md

@@ -4,7 +4,7 @@ Statistics Netherlands (CBS) StatLine — the Dutch national statistics office's
 open-data catalogue, over OData v3. Population, labour, energy, prices, trade and
 several thousand other official tables, in Dutch and English.
 
-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
 
@@ -82,7 +82,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-cbs-nl",
-  "version": "0.1.1",
+  "version": "0.1.3",
   "description": "Statistics Netherlands (CBS / StatLine) OData MCP.",
   "type": "module",
   "main": "src/index.ts",
@@ -26,7 +26,7 @@
     "@cloudflare/workers-types": "^4.20260405.1"
   },
   "pipeworx": {
-    "sourceHash": "v1-46e09e644b80bcb40b905088be867d5527e237f7218a8e75295732b0128424f1",
-    "sourceCommit": "60f04b68c347e970c6be94a53439d1364061fc06"
+    "sourceHash": "v1-1f4013c99a6c89d86557fa722f1ed30372a6b8f380159c808b093354aeb4e965",
+    "sourceCommit": "a00767651388cfd017fdeac82188622721db2051"
   }
 }

package/server.json

@@ -3,7 +3,7 @@
   "name": "io.github.pipeworx-io/cbs-nl",
   "title": "Cbs Nl",
   "description": "Statistics Netherlands (CBS / StatLine) OData MCP.",
-  "version": "0.1.1",
+  "version": "0.1.3",
   "websiteUrl": "https://pipeworx.io/packs/cbs-nl",
   "repository": {
     "url": "https://github.com/pipeworx-io/mcp-cbs-nl",

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,
+      ),
+    );
   }
 }
 
@@ -729,6 +764,32 @@ const tools: McpToolExport['tools'] = [
 ];
 
 async function callTool(name: string, args: Record<string, unknown>): Promise<unknown> {
+  try {
+    return await dispatch(name, args);
+  } catch (err) {
+    // CBS answers an unknown table id (and an unknown dimension endpoint) with
+    // a 404 ASP.NET page titled "The resource cannot be found." That threw as a
+    // bare `CBS: 404 …` and booked as a Pipeworx defect with "retry the same
+    // tool" (fleet #2496). It is the caller naming a thing CBS does not have.
+    if (err instanceof CbsHttpError && err.status === 404 && /resource cannot be found/i.test(err.body)) {
+      const table = typeof args.table === 'string' ? args.table : '';
+      const what =
+        name === 'dimension_values'
+          ? `no table "${table}" or no dimension "${String(args.dimension ?? '')}" in it`
+          : `no table "${table}"`;
+      return {
+        error: 'not_found',
+        message:
+          `CBS StatLine has ${what}. Call search_tables({keyword: "..."}) to find the table id` +
+          `${name === 'dimension_values' ? ', then table_dimensions for its dimension names' : ''}.`,
+        table,
+      };
+    }
+    throw err;
+  }
+}
+
+async function dispatch(name: string, args: Record<string, unknown>): Promise<unknown> {
   PROXY =
     typeof args._proxyUrl === 'string' && typeof args._proxyToken === 'string'
       ? { url: args._proxyUrl, token: args._proxyToken }
@@ -763,7 +824,23 @@ async function callTool(name: string, args: Record<string, unknown>): Promise<un
       if (typeof args.select === 'string' && args.select.trim()) params.set('$select', args.select);
       if (typeof args.filter === 'string' && args.filter.trim()) params.set('$filter', args.filter);
       if (args.skip !== undefined) params.set('$skip', String(numArg(args.skip, 0)));
-      return cbsGet(`${TABLE}/${tableId(args)}/TypedDataSet?${params.toString()}`);
+      const out = (await cbsGet(`${TABLE}/${tableId(args)}/TypedDataSet?${params.toString()}`)) as {
+        value?: unknown[];
+      };
+      // `value: []` is an honest no-match (the filter or skip matched no rows)
+      // and used to go out as a bare empty success (fleet #2496).
+      if (Array.isArray(out?.value) && out.value.length === 0) {
+        return {
+          ...out,
+          empty_reason: 'no_match',
+          note:
+            `CBS table "${tableId(args)}" has no rows matching` +
+            `${typeof args.filter === 'string' && args.filter.trim() ? ` filter ${args.filter}` : ' this request'}` +
+            `${args.skip !== undefined ? ` at skip ${String(args.skip)}` : ''}. ` +
+            `Check the codes with dimension_values (e.g. dimension "Periods") before filtering.`,
+        };
+      }
+      return out;
     }
     default:
       throw new Error(`Unknown tool: ${name}`);
@@ -786,7 +863,9 @@ async function cbsGet(url: string): Promise<unknown> {
   if (!res.ok) {
     const body = await res.text().catch(() => '');
     const title = body.match(/<title>([\s\S]*?)<\/title>/i)?.[1]?.trim();
-    throw new Error(
+    throw new CbsHttpError(
+      res.status,
+      body,
       `CBS: ${res.status}${title ? ` ${title}` : ''} — ${url} — ${body.replace(/\s+/g, ' ').slice(0, 300)}`,
     );
   }
@@ -799,6 +878,16 @@ async function cbsGet(url: string): Promise<unknown> {
   }
 }
 
+class CbsHttpError extends Error {
+  readonly status: number;
+  readonly body: string;
+  constructor(status: number, body: string, message: string) {
+    super(message);
+    this.status = status;
+    this.body = body;
+  }
+}
+
 function tableId(args: Record<string, unknown>): string {
   return reqStr(args, 'table', '"37296eng"').trim().replace(/[^A-Za-z0-9_]/g, '');
 }

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-cbs-nl', version: '0.1.1' },
+  { name: '@pipeworx/mcp-cbs-nl', version: '0.1.3' },
   { capabilities: { tools: {} } },
 );