@rulvar/planner 1.22.0 → 1.24.0

package/dist/index.js

@@ -1,5 +1,5 @@
 import { n as compileScript, r as scriptDiagnosticsOf, t as SANDBOX_GLOBALS } from "./compile-BtCY9wfO.js";
-import { ConfigError, InMemoryStore, SandboxError, ScriptRejected, createEngine, createSandboxBridge, defineWorkflow, toJournalValue } from "@rulvar/core";
+import { ConfigError, InMemoryStore, SANDBOX_AGENT_OPT_KEYS, SandboxError, ScriptRejected, createEngine, createSandboxBridge, defineWorkflow, toJournalValue } from "@rulvar/core";
 import { createHash } from "node:crypto";
 import { Linter } from "eslint";
 import { toJsonDiagnostics, workflowsConfig } from "eslint-plugin-rulvar";
@@ -11,6 +11,21 @@ import { MessageChannel, Worker } from "node:worker_threads";
 * pure constant, byte-stable across runs; plan() (M6-T05) composes it
 * with profileCard(registry) and the goal. The usage examples are
 * distilled from the runnable examples/ corpus (M5-T09).
+*
+* The agent opts line is GENERATED from the runtime allowlist
+* (`SANDBOX_AGENT_OPT_KEYS` in @rulvar/core), so the card can never
+* drift from what the sandbox bridge actually accepts (v1.22.0 review
+* 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).",
@@ -18,11 +33,18 @@ const CARD = [
 	"",
 	"Available globals (the COMPLETE list; nothing else exists):",
 	"- agent(prompt, opts?) -> Promise<value>: one subagent call.",
-	"  opts (JSON only): { agentType?, model?, effort?, schema?, tools?, onError?, label?, key?, estCost?, limits?, escalation?, fallback?, result? }",
+	`  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.",
+	"  - 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.",
 	"  opts: { settle?: true } returns settled outcomes instead of throwing on the first failure.",
 	"- pipeline(items, ...stageFns, opts?) -> Promise<results[]>: streams items through stages.",
@@ -42,7 +64,11 @@ const CARD = [
 	"- Date.now() and Math.random() ARE the seeded shims; still prefer now() and random().",
 	"- Never put functions inside option objects; options are pure JSON.",
 	"- Policies are declarative JSON rule tables; ladders are JSON.",
-	"- Identical calls with identical arguments journal as ONE result; pass { key } to repeat deliberately.",
+	"- Every call journals as its OWN operation: identical calls share a content key but take",
+	"  sequential ordinals (0, 1, 2, ...), so repeats always RUN, never dedupe.",
+	"- For deliberate repeats still pass a distinguishing { key }: it binds each result to its",
+	"  call by identity instead of position, so editing or reordering the script cannot attach",
+	"  an old result to the wrong call on resume.",
 	"",
 	"Patterns (from the runnable corpus):",
 	"- Adversarial panel:",
@@ -295,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.22.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.22.0",
-    "@rulvar/core": "1.22.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.22.0"
+    "@rulvar/testing": "1.24.0"
   },
   "repository": {
     "type": "git",