dsh-labscript 0.3.0 → 0.3.1

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## 0.3.1
+
+- The DSH-side bridge budget (`requestTimeoutMs`) now defaults to 180 s instead of 60 s, and the
+  `labs_doctor` probe uses it too. It has to stay above runmanager's own client timeout, which
+  now comes from labconfig (`[timeouts] communication_timeout`): at 60 s both budgets were equal,
+  so a slow RPC would be killed by the host before runmanager could report the real error.
+
 ## 0.3.0
 
 Implements the upstream request written from the field (labscript-suite workflow terms and

package/README.md

@@ -107,7 +107,8 @@ value next to `mapped_run_folder`.
 (`[servers] runmanager`, `[ports] runmanager` — default 42523 — and
 `[timeouts] communication_timeout` — default 60 s), the same way `runmanager.remote.Client`
 resolves them. Configured values override labconfig. The Plugins page leaves host/port blank
-for that reason; typing them pins them.
+for that reason; typing them pins them. The DSH-side bridge budget (`requestTimeoutMs`,
+default 180 s) stays above that timeout, so a slow RPC is not cut short by the host.
 
 `engage` controls how a submitted run is watched: `timeoutS` (default 900) is the
 total wait budget of one `labs_wait`, `noShotS` (default 25) makes the wait return

package/lib/runtime.js

@@ -69,7 +69,10 @@ function normalizeConfig(raw) {
     mock: input.mock || {},
     simulator: input.simulator || {},
     dshHome: input.dshHome || '',
-    requestTimeoutMs: Number(input.requestTimeoutMs || 60000),
+    // Must stay above runmanager's own client timeout (labconfig [timeouts]
+    // communication_timeout, default 60 s): the bridge is killed when this expires, which
+    // would hide the real error behind a generic timeout.
+    requestTimeoutMs: Number(input.requestTimeoutMs || 180000),
   }
 }
 
@@ -303,7 +306,7 @@ export function createRuntime(ctx, rawConfig) {
     }
     if (config.python) {
       try {
-        const data = await runBridge('doctor', {}, { timeoutMs: 60000 })
+        const data = await runBridge('doctor', {}, { timeoutMs: config.requestTimeoutMs })
         for (const check of data.checks || []) checks.push(check)
       } catch (error) {
         add('python-bridge', 'fail', error.message, error.detail && error.detail.hint ? error.detail.hint : 'run the bridge manually to see the traceback')

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-labscript",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "description": "Control labscript-suite experiments from DeepSeek Harness: runmanager globals and scans, submits and waits, YAML fixed-operation protocols with postprocess write-back, panel snapshots, a built-in mock backend and a read-only doctor.",
   "type": "module",
   "main": "lib/index.js",