scenescout 3.22.0 → 3.23.0

package/dist/engine/ci.js

@@ -16,6 +16,7 @@ import { markdownCell } from "./check.js";
 import { parseLimitFlag } from "./limits.js";
 import { isWorthALook, redactSecrets } from "./memory.js";
 import { SARIF_ANCHOR_FALLBACKS, checkSarifAnchor, sarifLocation } from "./sarif.js";
+import { fromRunLine, resolveFromRun } from "./from-run.js";
 // ── options ─────────────────────────────────────────────────────────────────
 export const PROVIDERS = ["anthropic", "openai"];
 /** Where each provider's key is read from. Nothing else is: no flag, no file, no input. */
@@ -93,6 +94,9 @@ export const CI_OPTION_NAMES = [
     "show",
     "compare-url",
     "dedup",
+    "from-run",
+    "from-run-mode",
+    "from-run-turns-per-page",
     "sarif-file-anchor",
 ];
 /** The longest --show description: it becomes a line of the model's prompt. */
@@ -115,7 +119,8 @@ export function checkBaseUrl(raw) {
         return { ok: true, url: u.toString().replace(/\/+$/, "") };
     return { ok: false, error: "--base-url must be https (plain http only to 127.0.0.1 or localhost): the API key is sent to it" };
 }
-export function parseCiArgs(args, cwd) {
+/** `env` is read for the earlier run alone (SCENESCOUT_FROM_RUN, SCENESCOUT_FROM_RUN_MODE), since a flag wins over it. */
+export function parseCiArgs(args, cwd, env = {}) {
     const positional = [];
     const flags = new Map();
     for (let i = 0; i < args.length; i++) {
@@ -271,6 +276,13 @@ export function parseCiArgs(args, cwd) {
     const anchor = flags.has("sarif-file-anchor") ? checkSarifAnchor(flags.get("sarif-file-anchor")) : undefined;
     if (anchor && !anchor.ok)
         return anchor;
+    const runFlags = { path: flags.get("from-run"), mode: flags.get("from-run-mode"), turnsPerPage: flags.get("from-run-turns-per-page") };
+    if (show && (runFlags.path !== undefined || runFlags.mode !== undefined || runFlags.turnsPerPage !== undefined))
+        return { ok: false, error: "--from-run starts an exploration from an earlier run, and --show explores nothing: give one or the other" };
+    // A run asked to show an element explores nothing, so an earlier run named in the environment does not apply to it.
+    const fromRun = show ? { ok: true } : resolveFromRun(runFlags, env);
+    if (!fromRun.ok)
+        return fromRun;
     const resolve = (p) => (p.startsWith("/") || /^[A-Za-z]:[\\/]/.test(p) ? p : `${cwd.replace(/[\\/]$/, "")}/${p}`);
     return {
         ok: true,
@@ -296,6 +308,7 @@ export function parseCiArgs(args, cwd) {
             ...(navTimeout.value !== undefined ? { navTimeoutMs: navTimeout.value } : {}),
             dedup: dedup,
             ...(anchor ? { sarifFileAnchor: anchor.value } : {}),
+            ...(fromRun.fromRun ? { fromRun: { ...fromRun.fromRun, path: resolve(fromRun.fromRun.path), given: fromRun.fromRun.path } } : {}),
         },
     };
 }
@@ -691,6 +704,7 @@ export function ciKickoff(o) {
     return [
         `Run an exploratory test session following the method.`,
         `Target: ${o.url}`,
+        ...(o.fromRunLines ?? []),
         `Project directory (for scout_scan): ${o.projectDir}`,
         `Level: ${o.level}`,
         `Write mode: ${o.mode}`,
@@ -760,6 +774,7 @@ export function ciSummaryMarkdown(r, secrets = []) {
         `| Model | ${r.provider} ${cell(r.model, secrets)}, effort ${r.effort} |`,
         ...(r.lanes ? [`| Lanes | ${lanesCell(r.lanes, secrets)} |`] : []),
         ...(r.dedup ? [`| Finding dedup | ${dedupLine(r.dedup)} |`] : []),
+        ...(r.fromRun ? [`| Started from | ${cell(fromRunLine(r.fromRun), secrets)} |`] : []),
         `| Usage | ${usageLine(r.spend, r.model, r.endedAt, r.price)} |`,
         ``,
     ];
@@ -785,6 +800,27 @@ export function ciSummaryMarkdown(r, secrets = []) {
     lines.push(r.capture ? `The pictures are in shots/.` : `The full report, with repro steps and the gap ledger, is report.md.`, ``);
     return lines.join("\n");
 }
+/** A run record with every text a page or a person wrote in it passed through `clean`, as everything else ci.json holds is. */
+function cleanRecord(r, clean) {
+    const all = (xs) => xs.map(clean);
+    return {
+        ...r,
+        knownRoutes: all(r.knownRoutes),
+        visited: all(r.visited),
+        lanes: r.lanes.map((l) => ({ session: l.session, routes: all(l.routes) })),
+        steps: r.steps.map((s) => ({ ...s, route: clean(s.route), ...(s.target !== undefined ? { target: clean(s.target) } : {}) })),
+        left: r.left.map((l) => ({
+            ...l,
+            route: clean(l.route),
+            unexercised: all(l.unexercised),
+            forms: all(l.forms),
+            unchosen: l.unchosen.map((d) => ({ key: clean(d.key), options: all(d.options) })),
+        })),
+        gaps: all(r.gaps),
+        ...(r.assigned ? { assigned: all(r.assigned) } : {}),
+        ...(r.prefixes ? { prefixes: r.prefixes.map((n) => ({ ...n, target: clean(n.target), ...(n.why !== undefined ? { why: clean(n.why) } : {}) })) } : {}),
+    };
+}
 function lanesCell(l, secrets) {
     if (l.oneLoop)
         return `${l.asked} asked; explored in one loop: ${cell(l.oneLoop, secrets)}`;
@@ -876,6 +912,8 @@ export function ciSummaryJson(r, version, secrets = []) {
         })),
         ...(r.capture ? { capture: cleanCapture(r.capture, clean) } : {}),
         ...(r.lanes ? { lanes: lanesJson(r.lanes, clean) } : {}),
+        ...(r.fromRun ? { fromRun: { mode: r.fromRun.mode, source: clean(r.fromRun.source), runId: r.fromRun.runId, recordAt: r.fromRun.recordAt } } : {}),
+        ...(r.record ? { record: cleanRecord(r.record, clean) } : {}),
     };
 }
 /** The lanes as ci.json holds them. Module paths and lane names come from the app's routes: redacted like the rest. */