@alfe.ai/ctrader-mcp 0.3.6 → 0.3.7

package/dist/server.js

@@ -591,23 +591,41 @@ var CTraderError = class extends Error {
 * execution events for other sessions, …).
 */
 const LISTENABLE_EVENTS = new Set([PayloadType.OA_DEPTH_EVENT, PayloadType.OA_SPOT_EVENT]);
-const HEARTBEAT_INTERVAL_MS = 1e4;
+/**
+* Halved from 10s so the read-idle watchdog can detect a wedged socket before
+* `REQUEST_TIMEOUT_MS` fires, while still tolerating two consecutive missed
+* echoes (see `READ_IDLE_TIMEOUT_MS`). cTrader expects a heartbeat at least
+* every 10s and drops an idle connection at ~30s, so 5s is well within
+* protocol expectations. Heartbeats are protocol keepalives, not part of the
+* historical-data payload class that gets rate limited.
+*/
+const HEARTBEAT_INTERVAL_MS = 5e3;
 const REQUEST_TIMEOUT_MS = 2e4;
 const RECONNECT_BASE_MS = 1e3;
 const RECONNECT_MAX_MS = 3e4;
 const MAX_PENDING_REQUESTS = 256;
 /**
-* Read-idle watchdog: cTrader echoes our 10s heartbeats and pushes its own
-* traffic, so a *healthy* socket is never silent for long. If NO inbound byte
-* arrives for this long we treat the socket as silently half-dead (a TCP
-* half-open with no FIN — the OS never fires `close`, so `handleDrop` never
-* runs and every request would otherwise time out at 20s indefinitely) and
-* force a reconnect. Set to 3× the heartbeat interval so a single dropped
-* heartbeat echo doesn't false-positive.
+* Read-idle watchdog: cTrader echoes our heartbeats and pushes its own traffic,
+* so a *healthy* socket is never silent for long. If NO inbound byte arrives
+* for this long we treat the socket as silently half-dead (a TCP half-open with
+* no FIN — the OS never fires `close`, so `handleDrop` never runs and every
+* request would otherwise time out indefinitely) and force a reconnect. Set to
+* 3× the heartbeat interval so two dropped heartbeat echoes don't false-positive.
 */
 const READ_IDLE_TIMEOUT_MS = 3 * HEARTBEAT_INTERVAL_MS;
 /** How often the watchdog checks the read-idle clock. */
-const WATCHDOG_INTERVAL_MS = HEARTBEAT_INTERVAL_MS;
+const WATCHDOG_INTERVAL_MS = HEARTBEAT_INTERVAL_MS / 2;
+/**
+* Detection must beat the request timeout, or the watchdog is useless to the
+* caller that trips it: a wedged socket surfaces an ambiguous `REQUEST_TIMEOUT`
+* (indistinguishable from a slow server) instead of the fast, honest,
+* retryable `CONNECTION_DROPPED` the reconnect path exists to produce.
+*
+* That was the shipped state — 30s idle + 10s poll = up to 40s to detect, vs a
+* 20s request timeout — so the 0.3.1 watchdog never spared the first caller.
+* Worst-case detection is READ_IDLE_TIMEOUT_MS + WATCHDOG_INTERVAL_MS.
+*/
+if (READ_IDLE_TIMEOUT_MS + WATCHDOG_INTERVAL_MS >= REQUEST_TIMEOUT_MS) throw new Error(`ctrader-mcp timing misconfigured: read-idle detection (${String(READ_IDLE_TIMEOUT_MS + WATCHDOG_INTERVAL_MS)}ms) must be faster than REQUEST_TIMEOUT_MS (${String(REQUEST_TIMEOUT_MS)}ms).`);
 function log$1(msg) {
 	process.stderr.write(`[ctrader-mcp] ${msg}\n`);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@alfe.ai/ctrader-mcp",
-  "version": "0.3.6",
+  "version": "0.3.7",
   "description": "cTrader MCP server — full trading (market/limit/stop/stop-limit orders, trailing stops), Level 2 depth, live quotes, trade history, PnL & margin over the cTrader Open API (protobuf/TLS)",
   "type": "module",
   "main": "./dist/server.js",