@rulvar/cli 1.34.0 → 1.36.0

package/dist/index.d.ts

@@ -248,11 +248,23 @@ interface CreateWorkerOptions {
   owner?: string;
   /**
   * The store's lease ttl; the worker renews at ttl/3 (the normative
-  * bound). Default: the Appendix A reference 60000 ms.
-  * MUST match the store's configured ttl.
+  * bound). An integer between 1 and 2147483647 ms, refused as a
+  * ConfigError at construction. MUST match the store's configured ttl:
+  * when the store exposes the optional `leaseTtlMs` capability
+  * (SqliteStore does), the match is VERIFIED at construction and a
+  * mismatch is a ConfigError; a store without the capability is
+  * trusted. Omitted, the worker ADOPTS the store's exposed ttl, falling
+  * back to the Appendix A reference 60000 ms.
   */
   ttlMs?: number;
-  /** Idle sweep cadence for start(); default 1000 ms. */
+  /**
+  * Idle sweep cadence for start(); default 1000 ms. An integer between
+  * 1 and 2147483647 ms, refused as a ConfigError at construction (an
+  * overflow or a value that is not finite would collapse to the 1 ms floor
+  * and storm the store; v1.35.0 review P2-4). Zero is not a manual
+  * mode: drive sweeps directly with worker.sweep() instead of
+  * start().
+  */
   pollMs?: number;
   /**
   * The OQ-21 interim channel: original in-process run arguments are not

package/dist/index.js

@@ -583,9 +583,17 @@ function createWorker(engine, options) {
 	const concurrency = options.concurrency ?? 1;
 	if (!Number.isInteger(concurrency) || concurrency < 1) throw new ConfigError(`createWorker concurrency must be a positive integer, got ${concurrency}`);
 	const owner = options.owner ?? workerIdentity();
-	const ttlMs = options.ttlMs ?? 6e4;
+	const requireTimerMs = (value, site) => {
+		if (!Number.isInteger(value) || value < 1 || value > 2147483647) throw new ConfigError(`${site} must be an integer between 1 and 2147483647 ms; got ${String(value)}`);
+	};
+	const storeTtlMs = store.leaseTtlMs;
+	if (storeTtlMs !== void 0 && !Number.isInteger(storeTtlMs)) throw new ConfigError(`the store's leaseTtlMs capability must report an integer; got ${String(storeTtlMs)}`);
+	const ttlMs = options.ttlMs ?? storeTtlMs ?? 6e4;
+	requireTimerMs(ttlMs, "createWorker ttlMs");
+	if (storeTtlMs !== void 0 && ttlMs !== storeTtlMs) throw new ConfigError(`createWorker ttlMs ${String(ttlMs)} does not match the store's configured lease ttl ${String(storeTtlMs)} (the store exposes leaseTtlMs; omit createWorker ttlMs to adopt it)`);
 	const renewMs = Math.max(1, Math.floor(ttlMs / 3));
 	const pollMs = options.pollMs ?? 1e3;
+	requireTimerMs(pollMs, "createWorker pollMs");
 	const registry = buildDeriverRegistry(options.extraDerivers);
 	const active = /* @__PURE__ */ new Map();
 	/**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rulvar/cli",
-  "version": "1.34.0",
+  "version": "1.36.0",
   "description": "Rulvar shell: run/resume/runs/inspect/plan/kb commands, TUI progress, createServer, createWorker, OTel exporter.",
   "type": "module",
   "license": "Apache-2.0",
@@ -22,17 +22,17 @@
     "access": "public"
   },
   "dependencies": {
-    "@rulvar/core": "1.34.0"
+    "@rulvar/core": "1.36.0"
   },
   "devDependencies": {
     "@types/node": "^22.20.0",
     "tsdown": "^0.22.3",
     "typescript": "~6.0.3",
-    "@rulvar/store-sqlite": "1.34.0",
-    "@rulvar/planner": "1.34.0",
-    "@rulvar/plan": "1.34.0",
-    "@rulvar/evals": "1.34.0",
-    "@rulvar/testing": "1.34.0"
+    "@rulvar/testing": "1.36.0",
+    "@rulvar/store-sqlite": "1.36.0",
+    "@rulvar/evals": "1.36.0",
+    "@rulvar/planner": "1.36.0",
+    "@rulvar/plan": "1.36.0"
   },
   "bin": {
     "rulvar": "./dist/cli.js"