toolroll 0.9.52 → 0.9.53

package/dist/mate-tools.js

@@ -43,7 +43,7 @@ import { parseAcceptanceCriteria } from "./scope.js";
 import { diagnoseTaskDispatch, withDispatchDiagnoses } from "./dispatch.js";
 import { agentChoicesFor, routeOfTask, INSTALLATION_SCOPE } from "./agentconfig.js";
 import { isNewModel, modelWords, priceWords, runtimeStates, seenModels } from "./model-catalog.js";
-import { agentsSummary, chosenWords, PHASES, postureWords, RISK_CHOICES, riskConsequence, riskTitle, routeProblems, sameSpec, sizeSourceWords, sizeWords, specWords } from "./phase-routing.js";
+import { agentsSummary, chosenWords, PHASES, postureWords, routeProblems, sameSpec, sizeSourceWords, sizeWords, specWords } from "./phase-routing.js";
 import { TOOL_CATALOG, discoverTools, projectToolsOf, secretsSetFor, toolCommandLine, toolStanding } from "./project-tools.js";
 import { deciderOf, durationWords, FLOW_KIND_WORDS, FLOW_TEMPLATES, flowFromSteps, stepsFor, withKeptSteps } from "./flows.js";
 import { parseContract, toModelSchema } from "./contracts/contract.js";
@@ -320,8 +320,8 @@ function scopeStandingOf(store, taskId) {
 const ROLE_WORD = { plan: "planner", build: "builder", repair: "repair", review: "reviewer" };
 const ROLE_OF_WORD = { planner: "plan", plan: "plan", builder: "build", build: "build", repair: "repair", reviewer: "review", review: "review" };
 /**
- * THE AGENTS A TASK RUNS UNDER, as chat reads them (v48): the declared
- * risk and what it means, the same one-line summary the task page and CLI
+ * THE AGENTS A TASK RUNS UNDER, as chat reads them (v48): the task's size,
+ * the same one-line summary the task page and CLI
  * print, each role's exact agent with its reasons, the standing of those
  * agents (approved / awaiting approval / recommended), and — for a
  * proposal — only the configured, role-valid choices an operator could
@@ -334,14 +334,11 @@ export function agentsOver(store, taskId, now) {
         return null;
     const routed = routeOfTask(store, taskId, ref, now);
     const scope = store.getScope(taskId);
-    const risk = ref.riskLevel ?? scope?.riskLevel ?? "routine";
-    const editable = !store.hasLiveClaim(ref.id, now) && store.activeTournamentTerms(ref.id) === null;
-    const editableWhy = store.hasLiveClaim(ref.id, now) ? "this task is running — its agents cannot change under a live claim" : store.activeTournamentTerms(ref.id) !== null ? "tournament terms decide the agents while the contest is open" : null;
+    const editable = !store.hasLiveClaim(ref.id, now);
+    const editableWhy = store.hasLiveClaim(ref.id, now) ? "this task is running — its agents cannot change under a live claim" : null;
     const base = {
         task: taskId,
-        risk: { level: risk, title: riskTitle(risk), consequence: riskConsequence(risk) },
         size: ref.sizing == null ? null : { size: ref.sizing.size, risky: ref.sizing.risky, reason: ref.sizing.reason, source: sizeSourceWords(ref.sizing.source) },
-        riskChoices: RISK_CHOICES.map(one => ({ risk: one.risk, title: one.title, consequence: one.consequence })),
         editable,
         editableWhy,
         approval: scope === null ? "none" : scopeStandingOf(store, taskId),
@@ -1536,7 +1533,7 @@ const MATE_TOOL_HANDLERS = {
         },
     },
     get_agents: {
-        description: "Read current roles, risk, approval, configured alternatives and the organisation policy (allowed providers, models, tools, permission ceiling; changed only on Settings → Policy) before propose_agents.",
+        description: "Read current roles, size, approval, configured alternatives and the organisation policy (allowed providers, models, tools, permission ceiling; changed only on Settings → Policy) before propose_agents.",
         handle: (ctx, args) => {
             const taskId = taskIdOf(args);
             if (taskId === null)
@@ -1697,8 +1694,6 @@ const MATE_TOOL_HANDLERS = {
                 return { ok: false, message: `guidance is plain text up to ${LIMITS.note} characters` };
             if (task.state === "done" || task.state === "cancelled")
                 return { ok: false, message: "that task is finished, so guidance has no next attempt to reach" };
-            if (ctx.store.openContestFor(ref.id) !== null)
-                return { ok: false, message: "agents are racing on that task — wait until the comparison finishes" };
             const id = ctx.draft("steer", { task: taskId, taskTitle: task.title, repoId: ref.repoId, note: args["note"] });
             if (id === null)
                 return tooMany();
@@ -1790,14 +1785,13 @@ const MATE_TOOL_HANDLERS = {
         },
     },
     propose_agents: {
-        description: "Read get_agents. Change risk, size (small: fast model, no plan; large or risky: strongest agents), configured role model, or clear override; stales approval, refuses running work.",
+        description: "Read get_agents. Change size (small: fast model, no plan; large or risky: strongest agents), configured role model, or clear override; stales approval, refuses running work.",
         handle: (ctx, args) => {
             const taskId = taskIdOf(args);
             const ref = taskId === null ? null : admittedRef(ctx, taskId);
             const task = taskId === null ? null : ctx.store.getTask(taskId);
             if (taskId === null || ref === null || task === null)
                 return notFound();
-            const risk = args["risk"];
             const roleWord = args["role"];
             const phase = roleWord === undefined ? null : typeof roleWord === "string" ? ROLE_OF_WORD[roleWord] ?? null : null;
             if (roleWord !== undefined && (phase === null || phase === "review"))
@@ -1808,8 +1802,8 @@ const MATE_TOOL_HANDLERS = {
             const risky = args["risky"];
             const sized = ctx.store.refForId(ref.id)?.sizing ?? null;
             const sizeChange = size === undefined && risky === undefined ? null : { size: size ?? sized?.size ?? "medium", risky: risky ?? sized?.risky ?? false };
-            if (risk === undefined && phase === null && sizeChange === null)
-                return { ok: false, message: "say what changes: a risk, a size, or a role with an agent (or clear: true)" };
+            if (phase === null && sizeChange === null)
+                return { ok: false, message: "say what changes: a size, or a role with an agent (or clear: true)" };
             if (phase !== null && !clear && (agent === null || typeof agent !== "object"))
                 return { ok: false, message: "a role change names an agent from get_agents, or clear: true" };
             if (phase === null && (clear || agent !== undefined))
@@ -1818,8 +1812,6 @@ const MATE_TOOL_HANDLERS = {
                 return { ok: false, message: "why is plain text ≤400" };
             if (ctx.store.hasLiveClaim(ref.id, ctx.now))
                 return { ok: false, message: "a worker is building that task right now — its agents cannot change under it" };
-            if (ctx.store.activeTournamentTerms(ref.id) !== null)
-                return { ok: false, message: "tournament terms decide this task's agents while the contest is open" };
             const view = agentsOver(ctx.store, taskId, ctx.now);
             if (view === null)
                 return notFound();
@@ -1840,19 +1832,16 @@ const MATE_TOOL_HANDLERS = {
                 }
                 chosen = { provider: match.provider, model: match.model };
                 const currentLeg = view["agents"].find(one => one.role === ROLE_WORD[phase]);
-                if (currentLeg !== undefined && sameSpec(currentLeg, chosen) && risk === undefined)
+                if (currentLeg !== undefined && sameSpec(currentLeg, chosen))
                     return { ok: false, message: `${specWords(chosen)} already runs the ${ROLE_WORD[phase]} role` };
             }
             if (phase !== null && clear && !(current?.routeOverrides ?? []).some(one => one.phase === phase)) {
                 return { ok: false, message: `nobody chose a ${ROLE_WORD[phase]} for this task by hand — there is nothing to clear` };
             }
-            if (risk !== undefined && phase === null && risk === view["risk"].level)
-                return { ok: false, message: `this task is already declared ${riskTitle(risk).toLowerCase()}` };
             const id = ctx.draft("agents", {
                 task: taskId,
                 taskTitle: task.title,
                 repoId: ref.repoId,
-                ...(risk === undefined ? {} : { risk, riskConsequence: riskConsequence(risk) }),
                 ...(sizeChange === null ? {} : { size: sizeChange.size, risky: sizeChange.risky }),
                 ...(phase === null ? {} : { phase, role: ROLE_WORD[phase] }),
                 ...(chosen === null ? {} : { provider: chosen.provider, model: chosen.model }),
@@ -1870,7 +1859,6 @@ const MATE_TOOL_HANDLERS = {
                     proposal: id,
                     kind: "agents",
                     task: taskId,
-                    ...(risk === undefined ? {} : { risk }),
                     ...(sizeChange === null ? {} : { size: sizeChange.size, risky: sizeChange.risky }),
                     ...(phase === null ? {} : { role: ROLE_WORD[phase], ...(chosen === null ? { clear: true } : { agent: chosen }) }),
                     awaiting: view["approval"] === "approved" ? "the operator's confirmation — the current approval will then need renewing" : "the operator's confirmation",

package/dist/modes.d.ts

@@ -27,7 +27,8 @@ export type ModeTerms = {
     /** Explicit opt-in: a verified planner may approve a plan that preserves
      * the signer's pre-authorized filed contract and execution terms exactly. */
     planAuto: boolean;
-    /** Attended mint without the per-mint password — signer only (D8). */
+    /** Historical signed field (watched sessions without a per-session
+     * password, removed in v115); new modes leave it false and nothing reads it. */
     quickMint: boolean;
     /** Historical signed field; new modes leave it false and no worker consumes
      * it. Automatic review is the project's review switch (review-switch.ts),
@@ -47,15 +48,13 @@ export type ModeTerms = {
      * "automerge" = the mode's signature substitutes for the per-merge
      * human authorization, through the grant machinery only (D1/E1). */
     publication: "notify" | "automerge";
-    /** Whether this mode AUTHORIZES automatic PAID fallback (a
-     * subscription->api-key or api-key->api-key switch that spends). v30,
-     * fallback chains R8: legacy modes default FALSE — a paid substitution
-     * must be an explicit, freshly-signed grant. A subscription->subscription
-     * fallback is not "paid" and needs no grant. */
+    /** Historical signed term (v30-v114: a grant for automatic paid
+     * fallback; fallback chains were removed in v115). Always false on new
+     * modes; retained for digest and audit compatibility. */
     allowPaidFallback: boolean;
     /** Whether this mode AUTHORIZES the bounded repair loop to draft AND
      * auto-approve its own repair attempts (v40, evidence-review-v1) — the
-     * allowPaidFallback precedent, verbatim: legacy modes default FALSE, a
+     * same precedent as the old paid-fallback grant: legacy modes default FALSE, a
      * new authority is never inherited, only freshly signed. Without it, a
      * short/refuted run with named unresolved criteria still gets exactly
      * one drafted repair — it simply waits unapproved. */

package/dist/modes.js

@@ -24,7 +24,7 @@ export function presetTerms(name, absoluteExpiry) {
             permissionDefault: "safe",
             autoApproveFiling: false,
             planAuto: false,
-            quickMint: true,
+            quickMint: false,
             reviewAuto: false,
             reviewRetryAuto: false,
             perAttemptBudgetMicrousd: null,
@@ -42,7 +42,7 @@ export function presetTerms(name, absoluteExpiry) {
             permissionDefault: "escalated",
             autoApproveFiling: true,
             planAuto: false,
-            quickMint: true,
+            quickMint: false,
             reviewAuto: false,
             reviewRetryAuto: false,
             perAttemptBudgetMicrousd: null,
@@ -108,10 +108,8 @@ export function modeTermsFromJson(json) {
         measured !== undefined &&
         runs !== undefined &&
         (t["publication"] === "notify" || t["publication"] === "automerge") &&
-        // allowPaidFallback: a legacy mode has NO such field — that MUST read
-        // as false (R8: a paid substitution is only ever an explicit,
-        // freshly-signed grant). A present value must be a strict boolean;
-        // anything else is a bad envelope, null.
+        // allowPaidFallback (historical): absent reads as false; a present
+        // value must be a strict boolean, anything else is a bad envelope.
         (t["allowPaidFallback"] === undefined || typeof t["allowPaidFallback"] === "boolean") &&
         // repairAuto/repairMaxAttempts: the SAME precedent, verbatim (v40). A
         // legacy mode has NO such fields — that MUST read as false/0, never
@@ -155,11 +153,8 @@ export function modeWords(terms) {
         terms.autoApproveFiling
             ? "every scope YOU file — signed-in console or credentialed CLI — is approved the moment you file it; while this mode is active, your signed-in browser session becomes a spend credential for this repository"
             : "filings still wait for their own approval ceremony",
-        terms.quickMint
-            ? "you start watched sessions without re-typing your password — the confirm screen still shows every term"
-            : "watched sessions keep the per-session password",
         terms.planAuto
-            ? "plans for your pre-authorized filings auto-approve only when the goal, exclusions, paths, acceptance criteria, risk, budget, and agent route remain exactly unchanged; provide a goal, paths and acceptance criteria upfront; amendments and unresolved questions still wait for you"
+            ? "plans for your pre-authorized filings auto-approve only when the goal, exclusions, paths, acceptance criteria, budget, and agent route remain exactly unchanged; provide a goal, paths and acceptance criteria upfront; amendments and unresolved questions still wait for you"
             : "planner-generated plans wait for your approval",
         terms.name === "hands-off"
             ? "automatic review is on for this project unless you turn it off (`toolroll review off --repo <path>`): each finished build whose check passes (or that finishes, with checks Off) gets one read-only review by the project's review agent; a HIGH finding sends it back once as a revision filed under this mode, a HIGH on that revision comes to you, MEDIUM and LOW findings come to you as suggested follow-ups, and a review that fails or times out never holds the work — it reaches you marked not reviewed"
@@ -178,9 +173,8 @@ export function modeWords(terms) {
         terms.publication === "automerge"
             ? "pull requests merge THEMSELVES when CI is seen green on the exact commit — you are told afterwards (requires a merge-capable publication grant)"
             : "merges wait for you — even where a grant could merge on its own, while this mode is active",
-        terms.allowPaidFallback
-            ? "when a subscription is exhausted mid-build, an approved fallback that spends (an API key) may run automatically — spend moves to that account"
-            : "automatic fallback never switches to a paid API key on its own; a subscription that runs out stops and waits for you",
+        ...(terms.allowPaidFallback
+            ? ["a historical paid-fallback grant is retained on record but no longer used"] : []),
         ...(terms.repairAuto
             ? ["historical automatic repair grants are retained on record but no longer schedule work"] : []),
         ...(terms.chatApprove

package/dist/operate-remote-arguments.js

@@ -10,7 +10,7 @@ const args = (positionals = '', values = '', references = {}) => ({
  * digest. Neither dereferences a conversation, task or proposal. Creation IDs are never looked up. */
 export const REMOTE_ARGUMENTS = new Map([
     ...['status', 'ready', 'grants', 'gaps', 'sync', 'runner list', 'coordinator list', 'incident list',
-        'routine list', 'intake show', 'intake run', 'template list', 'webhook status', 'review show',
+        'intake show', 'intake run', 'template list', 'webhook status', 'review show',
         'chat-approval show', 'chat-approval off'].map(row => [row, args()]),
     ['integrations', args('', 'saved')],
     // --person and --token-name are names compared inside the reader's own view; --token stays a refused credential.
@@ -31,7 +31,7 @@ export const REMOTE_ARGUMENTS = new Map([
     ['task next', args('id:task', 'key undo')],
     ['task steer', args('id:task', 'note')],
     ['task assign', args('id:task', 'key runner anyone')],
-    ['task scope', args('id:task', 'key goal not touches budget-usd risk acceptance candidate compare model provider race race-count race-per-usd race-total-usd repair-model')],
+    ['task scope', args('id:task', 'key goal not touches budget-usd risk acceptance candidate model provider repair-model')],
     ['task plan', args('id:task', 'key provider model')],
     ['task hold', args('id:task', 'key reason until')],
     ['task unhold', args('id:task', 'key')],
@@ -47,13 +47,9 @@ export const REMOTE_ARGUMENTS = new Map([
     ['cap list', args()], ['cap add', args('name', 'kind probe expires')],
     // These objects are installation-scoped at the outer boundary; only an all-project principal reaches them.
     ['incident resolve', args('incident')],
-    ['routine show', args('id')],
-    ['routine add', args('name', 'goal schedule acceptance ceiling budget-usd not touches require')],
-    ...['refresh', 'pause', 'resume', 'run-now'].map(action => [`routine ${action}`, args('name')]),
     ['config show', args('phase?')], ['verify show', args()],
     ['intake preview', args()], ['intake pr-comments', args('', 'limit')],
     ['template show', args('name')],
-    ['contest show', args('id')], ['contest exclude', args('contest ordinal')],
     ...['search', 'impact'].map(action => [`knowledge ${action}`, args('query', 'base')]),
     ['knowledge refresh', args('', 'base')],
     // Decision reads and supersession are already project-bound in project-memory.ts. Proposal IDs are

package/dist/operate-remote.js

@@ -43,10 +43,9 @@ export const REMOTE_SCOPES = new Map([
     ["task list", listing],
     ["runner list", installation], ["coordinator list", installation], ["outbox list", installation], ["incident list", installation], ["incident resolve", installation],
     ["cap list", repo], ["cap add", repo],
-    ...["list", "show", "add", "refresh", "pause", "resume", "run-now"].map(action => [`routine ${action}`, installation]),
     ["config show", installation], ["verify show", repo], ["intake show", installation], ["intake run", installation], ["intake preview", installation],
-    ["intake pr-comments", installation], ["template list", installation], ["template show", installation], ["contest show", installation],
-    ["contest exclude", installation], ["webhook status", installation], ["review show", installation],
+    ["intake pr-comments", installation], ["template list", installation], ["template show", installation],
+    ["webhook status", installation], ["review show", installation],
     ["chat-approval show", self], ["chat-approval off", self],
     // The person's own remote actions, or everyone's within their projects for an approver (remote-audit.ts).
     ["audit", self],
@@ -63,11 +62,11 @@ export const REMOTE_SCOPES = new Map([
  */
 export const REMOTE_ALL_PROJECTS_ONLY = new Map([
     // Settings, connections and acts that belong to the installation as a whole, not to any one project.
-    ...["integrations", "grants", "sync", "incident resolve", "routine add", "routine refresh", "routine pause", "routine resume", "routine run-now",
-        "config show", "intake run", "contest exclude", "webhook status"].map(row => [row, "installation-wide"]),
+    ...["integrations", "grants", "sync", "incident resolve",
+        "config show", "intake run", "webhook status"].map(row => [row, "installation-wide"]),
     // Cross-project lists and reads a limited person is still refused; scoping them is later work.
-    ...["runner list", "coordinator list", "outbox list", "incident list", "routine list", "routine show", "intake show", "intake preview",
-        "intake pr-comments", "template list", "template show", "contest show", "review show"].map(row => [row, "not yet scoped"]),
+    ...["runner list", "coordinator list", "outbox list", "incident list", "intake show", "intake preview",
+        "intake pr-comments", "template list", "template show", "review show"].map(row => [row, "not yet scoped"]),
 ]);
 /** The projects a listing or task reference may disclose: null for someone with access to all of them. */
 export function remoteLensOf(store, scope, allows) {

package/dist/operate.d.ts

@@ -73,10 +73,6 @@ export type OperateOptions = {
     integrationIo?: Partial<IntegrationIo>;
     /** Injected by tests: the mate's provider fetch, key environment, and stdin lines. */
     mateSeams?: MateCliSeams;
-    /** Injected by tests: a held-session coordinator, so a `tick` exercises
-     * the attended road exactly as a co-located `up` would (production wires
-     * one only inside `up`). */
-    heldCoordinator?: import("./held.js").HeldSessionCoordinator;
     /** Test seam for the short indexed wait loop. */
     waitSleep?: (milliseconds: number) => Promise<void>;
     /** Injected by tests: `onboard`'s home folder, terminal and probes. */
@@ -110,8 +106,6 @@ export declare const TASK_ACTIONS: readonly ["add", "list", "show", "wait", "sta
 export declare const PUBLISH_ACTIONS: readonly ["setup", "grant", "revoke", "status", "unblock", "rearm", "merge", "refire"];
 export declare const CONFIG_ACTIONS: readonly ["show", "set", "clear"];
 export declare const APPROVER_ACTIONS: readonly ["list", "add"];
-export declare const ROUTINE_ACTIONS: readonly ["list", "add", "show", "approve", "refresh", "pause", "resume", "run-now"];
-export declare const CONTEST_ACTIONS: readonly ["show", "exclude"];
 export declare const PEOPLE_ACTIONS: readonly ["list", "invite", "projects", "revoke"];
 export declare const KEYS_ACTIONS: readonly ["status", "set", "clear", "verify", "auth"];
 /**