@bluewombat/slots 0.9.0 → 0.11.0

package/lib/junit.mjs

@@ -0,0 +1,122 @@
+// A JUnit XML report, reduced to what a Gate needs: one entry per test case, what happened to
+// it, and the words of its failure. Playwright, Vitest, Jest, pytest and the JVM runners all
+// write this shape; none of them is asked for more than `testcase` with an optional `failure`,
+// `error` or `skipped` child.
+//
+// Not a general XML parser: it reads the elements above, decodes the five predefined entities and
+// CDATA, and leaves the rest alone.
+
+/**
+ * @typedef {object} TestCase
+ * @property {string} name
+ * @property {string} suite The `classname`, or the enclosing suite's name.
+ * @property {"passed" | "failed" | "skipped"} status
+ * @property {string} message The first line of what went wrong, when it did.
+ * @property {string} detail The failure's own text: the step and the stack.
+ * @property {string[]} attachments Paths the runner says it saved for this case (traces, shots).
+ */
+
+const ENTITIES = { amp: "&", lt: "<", gt: ">", quot: '"', apos: "'" };
+
+/** @param {string} text */
+function decode(text) {
+  return text
+    .replace(/<!\[CDATA\[([\s\S]*?)\]\]>/g, (_, inner) => inner)
+    .replace(/&(#x[0-9a-fA-F]+|#[0-9]+|amp|lt|gt|quot|apos);/g, (whole, code) => {
+      if (code.startsWith("#x")) {
+        return String.fromCodePoint(Number.parseInt(code.slice(2), 16));
+      }
+      if (code.startsWith("#")) {
+        return String.fromCodePoint(Number.parseInt(code.slice(1), 10));
+      }
+      return ENTITIES[/** @type {keyof typeof ENTITIES} */ (code)] ?? whole;
+    });
+}
+
+/** @param {string} tag the opening tag text, `<testcase a="b" …>` */
+function attributesOf(tag) {
+  /** @type {Record<string, string>} */
+  const out = {};
+  for (const match of tag.matchAll(/([\w:.-]+)\s*=\s*("([^"]*)"|'([^']*)')/g)) {
+    out[match[1] ?? ""] = decode(match[3] ?? match[4] ?? "");
+  }
+  return out;
+}
+
+/**
+ * @param {string} xml
+ * @returns {TestCase[]}
+ */
+export function parseJunit(xml) {
+  /** @type {TestCase[]} */
+  const cases = [];
+  // The enclosing suite names, to give a case a home when it has no classname.
+  const events = xml.matchAll(
+    /<testsuite\b[^>]*>|<\/testsuite>|<testcase\b[^>]*\/>|<testcase\b[^>]*>[\s\S]*?<\/testcase>/g,
+  );
+  /** @type {string[]} */
+  const suites = [];
+  for (const event of events) {
+    const text = event[0];
+    if (text.startsWith("</testsuite")) {
+      suites.pop();
+      continue;
+    }
+    if (text.startsWith("<testsuite")) {
+      if (!text.endsWith("/>")) {
+        suites.push(attributesOf(text).name ?? "");
+      }
+      continue;
+    }
+    const open = text.match(/^<testcase\b[^>]*>/)?.[0] ?? text;
+    const attributes = attributesOf(open);
+    const body = text.endsWith("/>") && open === text ? "" : text.slice(open.length);
+    cases.push(caseOf(attributes, body, suites.at(-1) ?? ""));
+  }
+  return cases;
+}
+
+/**
+ * @param {Record<string, string>} attributes
+ * @param {string} body
+ * @param {string} suite
+ * @returns {TestCase}
+ */
+function caseOf(attributes, body, suite) {
+  const failure = body.match(/<(failure|error)\b([^>]*?)(?:\/>|>([\s\S]*?)<\/\1>)/);
+  const skipped = /<skipped\b/.test(body);
+  const detail = failure === null ? "" : decode(failure[3] ?? "").trim();
+  const said = failure === null ? "" : (attributesOf(`<x ${failure[2] ?? ""}>`).message ?? "");
+  const message = (said.length > 0 ? said : (detail.split("\n")[0] ?? "")).trim();
+  /** @type {string[]} */
+  const attachments = [];
+  for (const match of body.matchAll(/\[\[ATTACHMENT\|([^\]]+)\]\]/g)) {
+    attachments.push((match[1] ?? "").trim());
+  }
+  return {
+    name: attributes.name ?? "",
+    suite: attributes.classname ?? suite,
+    status: failure !== null ? "failed" : skipped ? "skipped" : "passed",
+    message,
+    detail,
+    attachments,
+  };
+}
+
+/**
+ * Which of the manifest's scenario ids a test case name carries. An id is found as a whole
+ * token, so `E1` is not in `E10 …`.
+ *
+ * @param {string} name
+ * @param {readonly string[]} ids
+ * @returns {string | undefined}
+ */
+export function scenarioOf(name, ids) {
+  for (const id of ids) {
+    const escaped = id.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
+    if (new RegExp(`(^|[^A-Za-z0-9_])${escaped}($|[^A-Za-z0-9_])`).test(name)) {
+      return id;
+    }
+  }
+  return undefined;
+}

package/lib/output-tail.mjs

@@ -0,0 +1,70 @@
+// The end of a command's output, for a Gate's report: what a person or an agent needs to read
+// to see why a check is red.
+//
+// Test runners and CLIs redraw a progress line (a spinner, "[✓] Pulling schema...") thousands of
+// times. Kept as bytes, those redraws push the real error out of the tail the report is cut from,
+// and the report says nothing but the spinner. So the output is cleaned line by line as it
+// arrives, before anything is dropped for size: escape codes removed, a line redrawn with "\r"
+// reduced to its last state, spinner and progress lines dropped.
+
+// CSI sequences (colors, erase line, cursor moves) and OSC sequences (titles, links).
+const ESC = String.fromCharCode(0x1b);
+const BEL = String.fromCharCode(0x07);
+const ESCAPES = new RegExp(
+  `${ESC}\\[[0-9;?]*[ -/]*[@-~]|${ESC}\\][^${BEL}${ESC}]*(?:${BEL}|${ESC}\\\\)`,
+  "g",
+);
+// A braille glyph is a spinner frame; "[✓] …..." is the line a spinner settles on.
+const SPINNER_FRAME = /[⠀-⣿]/;
+const SPINNER_DONE = /\[[✓✔]\].*\.\.\.\s*$/;
+
+/** One line, cleaned; null when it is only progress noise. */
+export function cleanLine(/** @type {string} */ raw) {
+  const redrawn = raw.split("\r").filter((part) => part.trim().length > 0);
+  const last = (redrawn.length > 0 ? redrawn[redrawn.length - 1] : "") ?? "";
+  const line = last.replace(ESCAPES, "").trimEnd();
+  if (SPINNER_FRAME.test(line) || SPINNER_DONE.test(line)) {
+    return null;
+  }
+  return line;
+}
+
+/**
+ * @param {number} maxChars what `text()` returns at most; older lines are dropped first.
+ */
+export function createOutputTail(maxChars) {
+  let kept = "";
+  /** @type {Map<string, string>} */
+  const partial = new Map();
+
+  const add = (/** @type {string} */ raw) => {
+    const line = cleanLine(raw);
+    if (line === null) {
+      return;
+    }
+    kept += `${line}\n`;
+    if (kept.length > maxChars * 2) {
+      kept = kept.slice(kept.length - maxChars);
+    }
+  };
+
+  return {
+    /** Feed a chunk of one stream; each stream keeps its own unfinished line. */
+    push(/** @type {string} */ stream, /** @type {Buffer | string} */ chunk) {
+      const lines = ((partial.get(stream) ?? "") + chunk.toString()).split("\n");
+      partial.set(stream, lines.pop() ?? "");
+      lines.forEach(add);
+    },
+    /** The cleaned end of the output, cut on a line boundary. */
+    text() {
+      for (const rest of partial.values()) {
+        if (rest.length > 0) {
+          add(rest);
+        }
+      }
+      partial.clear();
+      const cut = kept.length > maxChars ? kept.slice(kept.length - maxChars) : kept;
+      return (kept.length > maxChars ? cut.slice(cut.indexOf("\n") + 1) : cut).trim();
+    },
+  };
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bluewombat/slots",
-  "version": "0.9.0",
+  "version": "0.11.0",
   "type": "module",
   "description": "Shipped mason slots: Builders, Gates, publishers, and refreshers a Project names in its config.",
   "keywords": [
@@ -19,7 +19,8 @@
   },
   "exports": {
     "./builders/*": "./builders/*",
-    "./assembly/*": "./assembly/*",
+    "./ticket/*": "./ticket/*",
+    "./feature/*": "./feature/*",
     "./agents/*": "./agents/*",
     "./gates/*": "./gates/*",
     "./publishers/*": "./publishers/*",
@@ -28,7 +29,8 @@
   },
   "files": [
     "builders",
-    "assembly",
+    "ticket",
+    "feature",
     "agents",
     "lib",
     "gates",
@@ -50,7 +52,7 @@
     "test": "tsx --test $(find test -name '*.test.ts')"
   },
   "dependencies": {
-    "@bluewombat/slot-kit": "^0.9.0",
-    "@pinkpuma/deck": "1.0.0"
+    "@bluewombat/slot-kit": "^0.11.0",
+    "@pinkpuma/deck": "1.1.0"
   }
 }

package/refreshers/git.mjs

@@ -23,6 +23,12 @@ import {
 // fast-forward has been written to by somebody else, and that is a person's
 // problem, not a merge to attempt.
 //
+// A Feature's line cut from the final branch that has no commit of its own is
+// only a stale copy of it: when `--base` names that branch and the line is an
+// ancestor of it, the line is fast-forwarded to it, here and on the remote, so
+// a Ticket starts from the code as it now is. A line with commits of its own
+// is left alone.
+//
 // The remote is `origin`, like the git Publisher's. Takes no option.
 
 const cwd = process.cwd();
@@ -53,6 +59,34 @@ if (fetched.status !== 0) {
   process.exit(0);
 }
 
+const base = take(process.argv, "--base");
+let goal = `origin/${target}`;
+if (base !== undefined && base.trim().length > 0 && base !== target) {
+  const fetchedBase = git(["fetch", "origin", `+refs/heads/${base}:refs/remotes/origin/${base}`]);
+  if (fetchedBase.status !== 0) {
+    emitRefreshRefusal(
+      `Cannot fetch origin/${base} from ${cwd}: ${fetchedBase.stderr.trim().slice(-500)}`,
+    );
+    process.exit(0);
+  }
+  const behindBase =
+    git(["merge-base", "--is-ancestor", `origin/${target}`, `origin/${base}`]).status === 0;
+  if (
+    behindBase &&
+    git(["rev-parse", `origin/${target}`]).stdout !== git(["rev-parse", `origin/${base}`]).stdout
+  ) {
+    // A plain push: never forced, so it only ever adds history.
+    const pushed = git(["push", "origin", `refs/remotes/origin/${base}:refs/heads/${target}`]);
+    if (pushed.status !== 0) {
+      emitRefreshRefusal(
+        `Cannot move origin/${target} up to origin/${base}: ${pushed.stderr.trim().slice(-500)}`,
+      );
+      process.exit(0);
+    }
+    goal = `origin/${base}`;
+  }
+}
+
 const current = git(["symbolic-ref", "--short", "HEAD"]);
 if (current.status !== 0 || current.stdout.trim() !== target) {
   const exists = git(["rev-parse", "--verify", "--quiet", `refs/heads/${target}`]).status === 0;
@@ -65,7 +99,7 @@ if (current.status !== 0 || current.stdout.trim() !== target) {
   }
 }
 
-const merged = git(["merge", "--ff-only", `origin/${target}`]);
+const merged = git(["merge", "--ff-only", goal]);
 if (merged.status !== 0) {
   // The one way a copy stops fast-forwarding is a commit made into it that
   // the reference does not have. The copy is the system's, not a person's:

package/{assembly → ticket}/fix.mjs

@@ -15,7 +15,7 @@ import {
 } from "@bluewombat/slot-kit";
 import { attemptOf } from "../lib/fill.mjs";
 
-const NAME = "assembly fix";
+const NAME = "ticket fix";
 
 const DEFAULT_TEMPLATE = `You are a coding agent working unattended in a checkout of the project.
 
@@ -60,7 +60,7 @@ A person reads that line and decides. It is a correct answer, not a failure.
 
 const report = (take(process.argv, "--report") ?? "").trim();
 if (report.length === 0) {
-  emitFailure("fail-blocking", "The assembly fix command needs --report.");
+  emitFailure("fail-blocking", "The ticket fix command needs --report.");
   process.exit(1);
 }
 

package/{assembly → ticket}/validate.mjs

@@ -15,7 +15,7 @@ import {
 } from "@bluewombat/slot-kit";
 import { attemptOf } from "../lib/fill.mjs";
 
-const NAME = "assembly validate";
+const NAME = "ticket validate";
 
 const DEFAULT_TEMPLATE = `You are a coding agent working unattended in a checkout of the project.
 
@@ -67,7 +67,7 @@ const report = (take(process.argv, "--report") ?? "").trim();
 if (report.length > 0) {
   emitFailure(
     "fail-blocking",
-    "assembly.validate is a read-only review; it does not take --report — that is assembly.fix's job.",
+    "ticket.validate is a read-only review; it does not take --report — that is ticket.fix's job.",
   );
   process.exit(1);
 }
@@ -149,7 +149,7 @@ function answer(run, bin, name) {
     // next round gets the same prompt again.
     emitFailure(
       "fail-retryable",
-      `assembly.validate found no MASON_VERDICT: line in the answer: ${
+      `ticket.validate found no MASON_VERDICT: line in the answer: ${
         said.length > 0 ? said.slice(0, 500) : "(no answer)"
       }`,
     );
@@ -160,9 +160,7 @@ function answer(run, bin, name) {
     const why = verdict.slice(refused[0].length).trim();
     emitFailure(
       "fail-retryable",
-      why.length > 0
-        ? why
-        : "assembly.validate refused the assembled ticket, with no reason given.",
+      why.length > 0 ? why : "ticket.validate refused the assembled ticket, with no reason given.",
     );
     process.exit(1);
   }
@@ -171,7 +169,7 @@ function answer(run, bin, name) {
   }
   emitFailure(
     "fail-retryable",
-    `assembly.validate's MASON_VERDICT: line was neither VALIDATED nor REFUSED: ${verdict.slice(0, 500)}`,
+    `ticket.validate's MASON_VERDICT: line was neither VALIDATED nor REFUSED: ${verdict.slice(0, 500)}`,
   );
   process.exit(1);
 }