@basein/runner 0.3.21 → 0.3.23

package/dist/replay/derive.d.ts

@@ -53,16 +53,6 @@ export interface DeriveContext {
      * into the session only after the prompt hook (issue 1212).
      */
     agentDay?: () => AgentDay | undefined;
-    /**
-     * A reading for a scenario that can follow a plan's decision, made while the
-     * agent decides (the service's offer1.md O4): the plan, and its decision
-     * step. With it the service reads this turn's results without a live call,
-     * and the prompt as the user's request. Only the service reads it.
-     */
-    atDecision?: {
-        scenarioId: string;
-        stepIndex: number;
-    };
 }
 /**
  * The service, which derives when this runner has no key of its own

package/dist/replay/derive.js

@@ -347,9 +347,6 @@ async function deriveViaService(opts, service, schema, keys) {
                 // 2023-11-30 …"), which the service turns into the day it reads from
                 // (issue 1212). Older services ignore it.
                 ...(opts.todaySaid ? { todaySaid: opts.todaySaid } : {}),
-                // A reading made at a plan's decision (offer1.md O4): the service then
-                // takes this turn's results with no live call. Older services ignore it.
-                ...(opts.atDecision ? { atDecision: opts.atDecision } : {}),
             }),
             signal: opts.signal ?? timeoutSignal(),
         });

package/dist/replay/handover.d.ts

@@ -63,13 +63,7 @@ export type HandoverStopReason =
  * neither true nor false: nothing says whether this request needs the
  * step, and nothing is guessed. The step did not run.
  */
- | "test_failed"
-/**
- * A sub-task's last step was left out by its test (AC5), and the sub-task
- * could not hand its results back to its caller: the body that maps them
- * threw. Its steps that ran did their work; the one left out did not run.
- */
- | "result_map";
+ | "test_failed";
 export interface HandoverNoteInput {
     intent: string;
     kind: FallbackKind;
@@ -92,8 +86,7 @@ export interface HandoverNoteInput {
     ranSteps?: number[];
     /**
      * The steps before the stop that their tests left out, 1-based and
-     * inclusive (a sub-task left out whole spans several): named, so the model
-     * neither thinks they ran nor runs them.
+     * inclusive: named, so the model neither thinks they ran nor runs them.
      */
     leftOut?: Array<{
         from: number;
@@ -101,8 +94,6 @@ export interface HandoverNoteInput {
         toolName: string;
         text: string | null;
     }>;
-    /** For `result_map`: the sub-task whose results could not go back. */
-    subTask?: string | null;
     /**
      * Of {@link executed}, how many are the model's own tool calls above — the
      * steps a steered session ran before a divergence took over. Only read when
@@ -132,37 +123,11 @@ export interface HandoverNoteInput {
     remainingTools: string[];
     params?: Readonly<Record<string, unknown>>;
     delivery: HandoverDelivery;
-    /**
-     * The plan stopped in front of a *call* it cannot run (segmented.md
-     * R-CALL-29, R-CALL-36). The agent is told what that stretch of work was and
-     * why it is not running, so it does that part itself rather than wondering
-     * what is missing.
-     */
-    unusable?: {
-        reason: string;
-        callIntent?: string | null;
-        segmentId?: string | null;
-        ownerScenarioId?: string | null;
-    };
-    /**
-     * The scenario that owns the stopped step, when it is not the one that was
-     * matched — a parked step inside a called segment (R-CALL-30).
-     */
-    ownerScenarioId?: string | null;
-    /** The sub-task the stopped step belongs to, when it is inside one. */
-    inSubTask?: string | null;
     /**
      * For `other_branch`: the stopped step's test in words — what the steps
      * from there on are for, which this request is not (autocorrect.md AC5).
      */
     whenText?: string | null;
-    /**
-     * For `decision`: the live request the plan serves, the user's own words.
-     * Its sentences with a condition word are quoted, so the agent sees the
-     * choice that is its to make (the service's segmentsbreakdown.md SB-18).
-     * Read for no other kind.
-     */
-    request?: string | null;
 }
 export declare function buildHandoverNote(n: HandoverNoteInput): string;
 /** "Step 2", "Steps 1–3", "Steps 1, 3 and 5–6": 1-based numbers in order, runs joined. */

package/dist/replay/handover.js

@@ -11,24 +11,11 @@
  * and a tool's own payload is already in front of it where it ran.
  */
 import { truncate } from "./bundle.js";
-import { conditionSentences } from "./condition-words.js";
 /** Hard ceiling on a note. Long enough to carry derived params, short enough to read. */
 export const MAX_HANDOVER_NOTE = 4_000;
 const MAX_ERROR_CHARS = 500;
 const MAX_PARAMS_CHARS = 1_500;
-/** What the note says about each reason a call cannot run (R-CALL-36). */
-const UNUSABLE_LINE = {
-    switched_off: "whose recorded scenario is switched off",
-    not_ready: "whose recorded scenario is not ready",
-    missing: "whose recorded scenario was deleted",
-    stale: "whose recorded scenario is out of date",
-    unresolved: "with no matching recorded scenario right now",
-    depth: "whose recorded scenario is nested too deep to run here",
-    unsupported: "this runner cannot run",
-};
 export function buildHandoverNote(n) {
-    if (n.kind === "decision")
-        return decisionNote(n);
     const intent = n.intent.length > 240 ? `${n.intent.slice(0, 240)}…` : n.intent;
     const human = n.stepIndex + 1;
     const tool = n.stepToolName ? ` (${n.stepToolName})` : "";
@@ -38,91 +25,6 @@ export function buildHandoverNote(n) {
     ];
     if (intent)
         lines.push(`Intent: ${intent}`);
-    lines.push(...ranLines(n));
-    if (n.kind === "unusable_call") {
-        // A call is a stretch of work, not a tool: the agent is told what the
-        // sub-task was, not which tool did not run.
-        const what = n.unusable?.callIntent ? ` ("${truncate(n.unusable.callIntent, 200)}")` : "";
-        const why = UNUSABLE_LINE[n.unusable?.reason ?? ""] ?? "that cannot run here";
-        lines.push(`Step ${human} of ${n.totalSteps} is a sub-task${what} ${why}.`);
-    }
-    else if (n.kind === "known_bad_step") {
-        const times = n.failureCount ? ` ${n.failureCount} times` : "";
-        // A parked step inside a called segment names the scenario that owns it:
-        // that is where the repair happens, and the agent's own step numbering has
-        // no row for it (R-CALL-30).
-        const owner = n.inSubTask
-            ? `, in sub-task "${truncate(n.inSubTask, 120)}"${n.ownerScenarioId ? `, scenario ${n.ownerScenarioId}` : ""}`
-            : "";
-        lines.push(`Step ${human} of ${n.totalSteps}${tool.replace(/\)$/, `${owner})`)} did not run: it has failed${times} before, so the scenario stops in front of it.`);
-    }
-    else if (n.kind === "nondeterministic_step") {
-        // The calculation could not write code for this step's input: it is a
-        // judgement, and the agent is the one to make it (plan-services.md D8).
-        lines.push(`Step ${human} of ${n.totalSteps}${tool} did not run: its input needs a judgement the scenario cannot compute. Make that choice yourself and carry on.`);
-    }
-    else if (n.kind === "other_branch") {
-        // The plan holds one half of an "if … otherwise" and this request is the
-        // other (autocorrect.md AC5). The steps from here are not wrong, they are
-        // not this request's: the model is told what they are for, never to run
-        // them, and the list of their tools is left out below for that reason.
-        const why = n.whenText ? `they are only for when ${n.whenText}, and that is not so for this request` : "this request is the other case of the task, which they are not for";
-        lines.push(n.stepIndex >= n.totalSteps
-            ? "None of the scenario's steps was needed for this request. Do the task as this request needs it."
-            : `Step ${human} of ${n.totalSteps}${tool} and the steps after it did not run: ${why}. Do the rest of the task yourself, as this request needs it.`);
-    }
-    else {
-        lines.push(stepFailedLine(n, human, tool));
-    }
-    // Not for the other half: those tools are the ones this request must not
-    // run, and naming them invites exactly that.
-    if (n.remainingTools.length > 0 && n.kind !== "other_branch") {
-        lines.push(`The remaining planned steps were: ${n.remainingTools.join(", ")}.`);
-    }
-    if (n.params && Object.keys(n.params).length > 0) {
-        let json;
-        try {
-            json = JSON.stringify(n.params);
-        }
-        catch {
-            json = "[unserializable]";
-        }
-        lines.push(`Parameters derived for this request: ${truncate(json, MAX_PARAMS_CHARS)}`);
-    }
-    return truncate(lines.join("\n"), MAX_HANDOVER_NOTE);
-}
-/**
- * The note of a plan stopped at its decision (the service's
- * segmentsbreakdown.md SB-18): what ran, and the request's own words for the
- * choice the agent now makes. Nothing that names the recorded half: no
- * intent, no remaining steps, no parameters, and never the condition the
- * service keeps (`stop.why`). Each would hint which way the recording went.
- */
-function decisionNote(n) {
-    // Where the results of the steps that ran are, as the ran-steps line says.
-    const above = Math.max(0, Math.min(n.executedAbove ?? 0, n.executed));
-    const delivery = above >= n.executed ? "above" : n.delivery;
-    const from = n.executed === 0
-        ? ""
-        : delivery === "above"
-            ? " from the results above"
-            : delivery === "below"
-                ? above > 0
-                    ? " from the results above and below"
-                    : " from the results below"
-                : "";
-    const quote = conditionSentences(n.request);
-    const ask = quote ? `This request asks for a decision: “${quote}”.` : "This task has a decision to make here.";
-    const lines = [
-        "[BaseInstRunner calculated replay] The calculated scenario for this request ran the steps before a decision, and stopped there.",
-        ...ranLines(n),
-        `${ask} Decide${from} which way applies, then continue the task, calling tools as needed.`,
-    ];
-    return truncate(lines.join("\n"), MAX_HANDOVER_NOTE);
-}
-/** What ran before the stop, and what of it the model may assume: the lines every note carries. */
-function ranLines(n) {
-    const lines = [];
     // Steps left out by their tests sit among those before the stop (AC5): the
     // ones that ran are then named, never "steps 1–N", which would say a step
     // ran that did not.
@@ -188,7 +90,44 @@ function ranLines(n) {
         lines.push(`Step ${n.inFlight.stepIndex + 1} of ${n.totalSteps}${inTool} was started and has not reported back: ` +
             "it may not have run, or may still complete. Check whether its work happened before you repeat it.");
     }
-    return lines;
+    if (n.kind === "known_bad_step") {
+        const times = n.failureCount ? ` ${n.failureCount} times` : "";
+        lines.push(`Step ${human} of ${n.totalSteps}${tool} did not run: it has failed${times} before, so the scenario stops in front of it.`);
+    }
+    else if (n.kind === "nondeterministic_step") {
+        // The calculation could not write code for this step's input: it is a
+        // judgement, and the agent is the one to make it (plan-services.md D8).
+        lines.push(`Step ${human} of ${n.totalSteps}${tool} did not run: its input needs a judgement the scenario cannot compute. Make that choice yourself and carry on.`);
+    }
+    else if (n.kind === "other_branch") {
+        // The plan holds one half of an "if … otherwise" and this request is the
+        // other (autocorrect.md AC5). The steps from here are not wrong, they are
+        // not this request's: the model is told what they are for, never to run
+        // them, and the list of their tools is left out below for that reason.
+        const why = n.whenText ? `they are only for when ${n.whenText}, and that is not so for this request` : "this request is the other case of the task, which they are not for";
+        lines.push(n.stepIndex >= n.totalSteps
+            ? "None of the scenario's steps was needed for this request. Do the task as this request needs it."
+            : `Step ${human} of ${n.totalSteps}${tool} and the steps after it did not run: ${why}. Do the rest of the task yourself, as this request needs it.`);
+    }
+    else {
+        lines.push(stepFailedLine(n, human, tool));
+    }
+    // Not for the other half: those tools are the ones this request must not
+    // run, and naming them invites exactly that.
+    if (n.remainingTools.length > 0 && n.kind !== "other_branch") {
+        lines.push(`The remaining planned steps were: ${n.remainingTools.join(", ")}.`);
+    }
+    if (n.params && Object.keys(n.params).length > 0) {
+        let json;
+        try {
+            json = JSON.stringify(n.params);
+        }
+        catch {
+            json = "[unserializable]";
+        }
+        lines.push(`Parameters derived for this request: ${truncate(json, MAX_PARAMS_CHARS)}`);
+    }
+    return truncate(lines.join("\n"), MAX_HANDOVER_NOTE);
 }
 /** "Step 2", "Steps 1–3", "Steps 1, 3 and 5–6": 1-based numbers in order, runs joined. */
 export function stepsPhrase(numbers) {
@@ -240,13 +179,6 @@ function stepFailedLine(n, human, tool) {
             `could not read its output${detail ? ` (${detail})` : ""}, so it stops there. Do not run it again.`);
     }
     switch (n.stopReason) {
-        case "result_map": {
-            // Nothing of the stopped position ran: its sub-task's last step was left
-            // out, and what failed is handing the sub-task's results on.
-            const what = n.subTask ? ` "${truncate(n.subTask, 120)}"` : "";
-            return (`The sub-task${what} could not hand back its results${detail ? ` (${detail})` : ""}, ` +
-                "so the scenario stops after it. Do the rest of the task yourself.");
-        }
         case "test_failed":
             // The test could not answer, so whether the step is needed is open: the
             // model decides, as it would without the scenario.

package/dist/replay/logic.d.ts

@@ -88,15 +88,4 @@ export declare function evalToolInputLogic(code: string, parameters: Record<stri
 export declare function evalToolOutputLogic(code: string, toolOutput: string, parameters: Record<string, unknown>, intent: string, respParams?: Record<string, unknown>): Record<string, unknown>;
 /** Build the final response model from everything the steps accumulated. */
 export declare function evalResponseParamsLogic(code: string, respParams: Record<string, unknown>, parameters: Record<string, unknown>, intent: string): Record<string, unknown>;
-/**
- * The two mapping bodies of a call (segmented.md R-CALL-20).
- *
- * `paramMapLogic` builds the called segment's parameters from the caller's
- * state; `resultMapLogic` turns what the segment emitted into the keys the
- * caller's later steps read. Both belong to the *(call, segment) pair* rather
- * than to either side alone, which is why two chains calling one segment carry
- * two of them.
- */
-export declare function evalParamMapLogic(code: string, parameters: Record<string, unknown>, intent: string, respParams?: Record<string, unknown>): Record<string, unknown>;
-export declare function evalResultMapLogic(code: string, out: Record<string, unknown>, parameters: Record<string, unknown>, intent: string, respParams?: Record<string, unknown>): Record<string, unknown>;
 //# sourceMappingURL=logic.d.ts.map

package/dist/replay/logic.js

@@ -177,28 +177,4 @@ export function evalResponseParamsLogic(code, respParams, parameters, intent) {
         intent,
     ]), "responseParamsLogic");
 }
-/**
- * The two mapping bodies of a call (segmented.md R-CALL-20).
- *
- * `paramMapLogic` builds the called segment's parameters from the caller's
- * state; `resultMapLogic` turns what the segment emitted into the keys the
- * caller's later steps read. Both belong to the *(call, segment) pair* rather
- * than to either side alone, which is why two chains calling one segment carry
- * two of them.
- */
-export function evalParamMapLogic(code, parameters, intent, respParams = {}) {
-    return assertPlainObject(run("paramMapLogic", ["parameters", "intent", "respParams"], code, [
-        parameters,
-        intent,
-        respParams,
-    ]), "paramMapLogic");
-}
-export function evalResultMapLogic(code, out, parameters, intent, respParams = {}) {
-    return assertPlainObject(run("resultMapLogic", ["out", "parameters", "intent", "respParams"], code, [
-        out,
-        parameters,
-        intent,
-        respParams,
-    ]), "resultMapLogic");
-}
 //# sourceMappingURL=logic.js.map