@rulvar/planner 1.23.0 → 1.24.0

package/dist/index.js

@@ -18,6 +18,14 @@ import { MessageChannel, Worker } from "node:worker_threads";
 * P2-4: the hand-maintained list had silently fallen three options
 * behind, and the card claimed identical calls journal as one result,
 * which the ordinal semantics contradict).
+*
+* The tools and model semantics are pinned by the v1.23.0 review:
+* string entries of `tools` are registered TOOLSET names (the profile
+* card prints the valid set; profile names there are agentType values,
+* and runtime resolution rejects them as unknown toolsets before any
+* provider call), and `model`/`routing` are normally omitted because
+* the profile card never names models, so the planner has no valid ref
+* source unless the goal text itself supplies an allowlist.
 */
 const CARD = [
 	"You write ONE JavaScript async-function BODY (the workflow script).",
@@ -27,11 +35,14 @@ const CARD = [
 	"- agent(prompt, opts?) -> Promise<value>: one subagent call.",
 	`  opts (JSON only): { ${SANDBOX_AGENT_OPT_KEYS.map((key) => `${key}?`).join(", ")} }`,
 	"  - schema: a JSON Schema LITERAL (never a library value); the result is the validated object.",
-	"  - tools: an array of registered profile NAMES (strings only).",
+	"  - tools: an array of registered TOOLSET names, exactly the ones the profile card lists;",
+	"    never agent profile names (an unknown name is a typed error before any call).",
 	"  - onError: 'throw' | 'null' (default 'null'; a failed call yields null and the loss is recorded).",
-	"  - model: a string; agentType: a registered profile name from the profile card.",
-	"  - routing: a JSON map of invocation role to a model ref from the profile card",
-	"    (e.g. { summarize: '<model ref>' }); wins over the profile's own routing.",
+	"  - agentType: a registered profile name from the profile card.",
+	"  - model and routing: normally OMIT BOTH; the host profiles and routing decide models,",
+	"    and the profile card never names any. Use them only when the goal text itself lists",
+	"    allowed model refs (then routing maps invocation role to one of them, e.g.",
+	"    { summarize: '<allowed ref>' }, and wins over the profile's own routing).",
 	"  - memoizeOutcome: true journals this exact outcome for replay on resume EVEN when it failed (default: failures rerun).",
 	"  - replay: 'cache' | 'never'; 'never' forces a fresh live call on every resume, 'cache' reuses any matching prior.",
 	"- parallel(fns, opts?) -> Promise<values[]>: concurrent branches; fns is an array of async functions.",
@@ -310,6 +321,10 @@ var WorkerSandboxRunner = class {
 				}
 				if (message.t === "fail") {
 					const wire = message.error;
+					if (wire.code === "config") {
+						settleErr(new ConfigError(wire.message));
+						return;
+					}
 					const rebuilt = new Error(wire.message);
 					rebuilt.name = wire.code;
 					settleErr(rebuilt);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rulvar/planner",
-  "version": "1.23.0",
+  "version": "1.24.0",
   "description": "Rulvar flagship hybrid mode: plan agent, compileScript, WorkerSandboxRunner, self-repair loop.",
   "type": "module",
   "license": "Apache-2.0",
@@ -23,14 +23,14 @@
   },
   "dependencies": {
     "eslint": "^9.39.4",
-    "eslint-plugin-rulvar": "1.23.0",
-    "@rulvar/core": "1.23.0"
+    "@rulvar/core": "1.24.0",
+    "eslint-plugin-rulvar": "1.24.0"
   },
   "devDependencies": {
     "@types/node": "^22.20.0",
     "tsdown": "^0.22.3",
     "typescript": "~6.0.3",
-    "@rulvar/testing": "1.23.0"
+    "@rulvar/testing": "1.24.0"
   },
   "repository": {
     "type": "git",