e-arveldaja-mcp 0.12.2 → 0.12.3

package/CHANGELOG.md

@@ -2,6 +2,11 @@
 
 ## [Unreleased]
 
+## [0.12.3] - 2026-04-26
+
+### Fixed
+- **Long-running tools crashed the Claude Code MCP transport with "Received a progress notification for an unknown token"** — call sites in `camt-import`, `bank-reconciliation`, `receipt-inbox`, `wise-import`, `analyze-unconfirmed`, `lightyear-investments`, and `api/base-resource` invoked `reportProgress` once per item across loops of 50–69+ entries. Each emit was awaited but landed in the OS stdio buffer; on slow clients the response was matched and the progressToken handler cleared *before* the trailing notifications were drained, and Claude Code treats any "unknown token" progress notification as a fatal transport error (closes the stdio pipe; the user must reconnect via `/mcp`). `reportProgress` now (a) throttles to at most one notification per 100 ms within an invocation, (b) skips the trailing `progress >= total` emit since the response itself signals completion, and (c) honors a `EARVELDAJA_DISABLE_PROGRESS=1` env-var kill switch for environments still affected. Throttle state is per-invocation (WeakMap keyed by the SDK's tool extra), so concurrent tool calls and back-to-back invocations never share a window.
+
 ## [0.12.2] - 2026-04-26
 
 ### Fixed

package/dist/progress.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"progress.d.ts","sourceRoot":"","sources":["../src/progress.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,MAAM,kBAAkB,CAAC;AAErD,MAAM,WAAW,SAAS;IACxB,gBAAgB,EAAE,CAAC,YAAY,EAAE,OAAO,KAAK,OAAO,CAAC,IAAI,CAAC,CAAC;IAC3D,KAAK,CAAC,EAAE;QAAE,aAAa,CAAC,EAAE,MAAM,GAAG,MAAM,CAAA;KAAE,CAAC;CAC7C;AAED,eAAO,MAAM,gBAAgB,8BAAqC,CAAC;AAEnE;;;GAGG;AACH,wBAAsB,cAAc,CAAC,QAAQ,EAAE,MAAM,EAAE,KAAK,CAAC,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAiBpF"}
+{"version":3,"file":"progress.d.ts","sourceRoot":"","sources":["../src/progress.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,MAAM,kBAAkB,CAAC;AAErD,MAAM,WAAW,SAAS;IACxB,gBAAgB,EAAE,CAAC,YAAY,EAAE,OAAO,KAAK,OAAO,CAAC,IAAI,CAAC,CAAC;IAC3D,KAAK,CAAC,EAAE;QAAE,aAAa,CAAC,EAAE,MAAM,GAAG,MAAM,CAAA;KAAE,CAAC;CAC7C;AAED,eAAO,MAAM,gBAAgB,8BAAqC,CAAC;AAyBnE;;;GAGG;AACH,wBAAsB,cAAc,CAAC,QAAQ,EAAE,MAAM,EAAE,KAAK,CAAC,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CA6BpF"}

package/dist/progress.js

@@ -1,16 +1,48 @@
 import { AsyncLocalStorage } from "node:async_hooks";
 export const toolExtraStorage = new AsyncLocalStorage();
+/**
+ * Minimum interval between emitted progress notifications within a single
+ * tool invocation. Tight per-item loops (50 CAMT entries, 69 reconcile rows,
+ * etc.) would otherwise flood the stdio transport. Any notification that
+ * arrives at the client *after* the tool's response is matched gets dropped
+ * with "Received a progress notification for an unknown token" — and on
+ * Claude Code that drop is fatal: the transport is closed and the server
+ * must be reconnected. Throttling cuts the volume and shrinks the race
+ * window; the trailing 100% emit is also skipped because the response itself
+ * already signals completion.
+ *
+ * Set EARVELDAJA_DISABLE_PROGRESS=1 to disable progress entirely on clients
+ * that still mishandle late-arriving events.
+ */
+const PROGRESS_THROTTLE_MS = 100;
+const lastEmitByExtra = new WeakMap();
+function progressDisabled() {
+    const v = process.env.EARVELDAJA_DISABLE_PROGRESS;
+    return v === "1" || v === "true";
+}
 /**
  * Report progress for long-running operations.
  * No-op if the client didn't supply a progress token.
  */
 export async function reportProgress(progress, total) {
+    if (progressDisabled())
+        return;
     const extra = toolExtraStorage.getStore();
     if (!extra)
         return;
     const progressToken = extra?._meta?.progressToken;
     if (progressToken === undefined)
         return;
+    // Skip the trailing 100% notification: the response signals completion,
+    // and the final emit is the most likely to lose the race against it.
+    if (total !== undefined && progress >= total)
+        return;
+    // Throttle within an invocation: at most one notification per window.
+    const now = Date.now();
+    const last = lastEmitByExtra.get(extra) ?? 0;
+    if (now - last < PROGRESS_THROTTLE_MS)
+        return;
+    lastEmitByExtra.set(extra, now);
     try {
         await extra.sendNotification({
             method: "notifications/progress",

package/dist/progress.js.map

@@ -1 +1 @@
-{"version":3,"file":"progress.js","sourceRoot":"","sources":["../src/progress.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,MAAM,kBAAkB,CAAC;AAOrD,MAAM,CAAC,MAAM,gBAAgB,GAAG,IAAI,iBAAiB,EAAa,CAAC;AAEnE;;;GAGG;AACH,MAAM,CAAC,KAAK,UAAU,cAAc,CAAC,QAAgB,EAAE,KAAc;IACnE,MAAM,KAAK,GAAG,gBAAgB,CAAC,QAAQ,EAAE,CAAC;IAC1C,IAAI,CAAC,KAAK;QAAE,OAAO;IACnB,MAAM,aAAa,GAAG,KAAK,EAAE,KAAK,EAAE,aAAa,CAAC;IAClD,IAAI,aAAa,KAAK,SAAS;QAAE,OAAO;IACxC,IAAI,CAAC;QACH,MAAM,KAAK,CAAC,gBAAgB,CAAC;YAC3B,MAAM,EAAE,wBAAwB;YAChC,MAAM,EAAE;gBACN,aAAa;gBACb,QAAQ;gBACR,GAAG,CAAC,KAAK,KAAK,SAAS,IAAI,EAAE,KAAK,EAAE,CAAC;aACtC;SACF,CAAC,CAAC;IACL,CAAC;IAAC,MAAM,CAAC;QACP,2CAA2C;IAC7C,CAAC;AACH,CAAC"}
+{"version":3,"file":"progress.js","sourceRoot":"","sources":["../src/progress.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,MAAM,kBAAkB,CAAC;AAOrD,MAAM,CAAC,MAAM,gBAAgB,GAAG,IAAI,iBAAiB,EAAa,CAAC;AAEnE;;;;;;;;;;;;;GAaG;AACH,MAAM,oBAAoB,GAAG,GAAG,CAAC;AAEjC,MAAM,eAAe,GAAG,IAAI,OAAO,EAAqB,CAAC;AAEzD,SAAS,gBAAgB;IACvB,MAAM,CAAC,GAAG,OAAO,CAAC,GAAG,CAAC,2BAA2B,CAAC;IAClD,OAAO,CAAC,KAAK,GAAG,IAAI,CAAC,KAAK,MAAM,CAAC;AACnC,CAAC;AAED;;;GAGG;AACH,MAAM,CAAC,KAAK,UAAU,cAAc,CAAC,QAAgB,EAAE,KAAc;IACnE,IAAI,gBAAgB,EAAE;QAAE,OAAO;IAC/B,MAAM,KAAK,GAAG,gBAAgB,CAAC,QAAQ,EAAE,CAAC;IAC1C,IAAI,CAAC,KAAK;QAAE,OAAO;IACnB,MAAM,aAAa,GAAG,KAAK,EAAE,KAAK,EAAE,aAAa,CAAC;IAClD,IAAI,aAAa,KAAK,SAAS;QAAE,OAAO;IAExC,wEAAwE;IACxE,qEAAqE;IACrE,IAAI,KAAK,KAAK,SAAS,IAAI,QAAQ,IAAI,KAAK;QAAE,OAAO;IAErD,sEAAsE;IACtE,MAAM,GAAG,GAAG,IAAI,CAAC,GAAG,EAAE,CAAC;IACvB,MAAM,IAAI,GAAG,eAAe,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAC7C,IAAI,GAAG,GAAG,IAAI,GAAG,oBAAoB;QAAE,OAAO;IAC9C,eAAe,CAAC,GAAG,CAAC,KAAK,EAAE,GAAG,CAAC,CAAC;IAEhC,IAAI,CAAC;QACH,MAAM,KAAK,CAAC,gBAAgB,CAAC;YAC3B,MAAM,EAAE,wBAAwB;YAChC,MAAM,EAAE;gBACN,aAAa;gBACb,QAAQ;gBACR,GAAG,CAAC,KAAK,KAAK,SAAS,IAAI,EAAE,KAAK,EAAE,CAAC;aACtC;SACF,CAAC,CAAC;IACL,CAAC;IAAC,MAAM,CAAC;QACP,2CAA2C;IAC7C,CAAC;AACH,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "e-arveldaja-mcp",
-  "version": "0.12.2",
+  "version": "0.12.3",
   "description": "MCP server for Estonian e-arveldaja (e-Financials) API",
   "mcpName": "io.github.iseppo/e-arveldaja-mcp",
   "type": "module",

package/server.json

@@ -6,12 +6,12 @@
     "url": "https://github.com/iseppo/e-arveldaja-mcp",
     "source": "github"
   },
-  "version": "0.12.2",
+  "version": "0.12.3",
   "packages": [
     {
       "registryType": "npm",
       "identifier": "e-arveldaja-mcp",
-      "version": "0.12.2",
+      "version": "0.12.3",
       "transport": {
         "type": "stdio"
       },
@@ -51,6 +51,13 @@
           "isRequired": false,
           "isSecret": false,
           "format": "string"
+        },
+        {
+          "name": "EARVELDAJA_DISABLE_PROGRESS",
+          "description": "Set to \"1\" or \"true\" to disable MCP progress notifications. Workaround for clients that close the stdio transport when a progress notification arrives after the matching response has been processed.",
+          "isRequired": false,
+          "isSecret": false,
+          "format": "string"
         }
       ]
     }