simt-dev 0.4.0 → 0.6.0

package/src/hooks/shim.mjs

@@ -18,6 +18,7 @@
 
 import { renderLocator } from '../locate.mjs';
 import { BIN_ENV, CLI, MANIFEST_FILE, PACKAGE_NAME } from '../name.mjs';
+import { localInstall } from '../distribute/release.mjs';
 import { packageVersion } from '../package-version.mjs';
 
 /**
@@ -49,8 +50,8 @@ export function fileFor(event) {
  * @param {string} event one of EVENTS
  * @param {{npm?: boolean, version?: string}} [opts]
  *   `npm` is whether the repository has a package.json: it decides the remedy's install
- *   (a devDependency where one exists, a pinned global install where there is no
- *   node_modules to pin in). `version` defaults to the running package's own version,
+ *   (a devDependency where one exists, the release binary into the repository where
+ *   there is no node_modules to pin in). `version` defaults to the running package's own version,
  *   so a shim generated by 0.2.0 names 0.2.0 in its remedy and CI install.
  * @returns {string} POSIX sh source, LF-terminated, executable
  */
@@ -62,8 +63,9 @@ export function renderShim(event, { npm = true, version } = {}) {
   // The install that finds the binary when node_modules and PATH both come up empty.
   // A repository with a package.json pins simt-dev as a devDependency, so the remedy
   // installs it there; one without (Gradle, Connect IQ) has no node_modules to pin in,
-  // so the remedy is a pinned global install - the same pinned bytes CI gets.
-  const install = npm ? `npm i -D ${PACKAGE_NAME}@${v}` : `npm i -g ${PACKAGE_NAME}@${v}`;
+  // so the remedy downloads the release binary into the repository - the same pinned
+  // bytes CI gets, and one that needs no npm and no node on the machine (ADR-098).
+  const install = npm ? `npm i -D ${PACKAGE_NAME}@${v}` : localInstall(v);
   const D = '$';
   return `#!/bin/sh
 ${SHIM_MARKER}
@@ -75,7 +77,8 @@ ${SHIM_MARKER}
 # This file only locates the dispatcher and hands over. It resolves in order:
 #   1. ${D}SIMT_DEV_BIN    explicit override, for CI images and odd checkouts
 #   2. node_modules/       the pinned project dependency
-#   3. ${CLI} on PATH       a global install
+#   3. .simt/bin/          the release binary this repository downloaded
+#   4. ${CLI} on PATH       a global install
 # If none resolves it REFUSES rather than exiting 0. A hook that cannot verify
 # anything must never report success - that is what teaches authors to reach for
 # --no-verify.
@@ -102,7 +105,9 @@ ${renderLocator(`hooks run ${event} --root "${D}ROOT"`, {
  */
 export function renderEvidenceShim({ npm = true, version } = {}) {
   const v = version ?? packageVersion();
-  const install = npm ? `npm i -D ${PACKAGE_NAME}@${v}` : `npm i -g ${PACKAGE_NAME}@${v}`;
+  // The same split as renderShim: a devDependency where one exists, the release
+  // binary into the repository where there is no node_modules to pin in.
+  const install = npm ? `npm i -D ${PACKAGE_NAME}@${v}` : localInstall(v);
   const D = '$';
   return `#!/bin/sh
 ${SHIM_MARKER}

package/src/index/govern.mjs

@@ -56,7 +56,7 @@ export function sourceRoots(files) {
 /**
  * One Affects token.
  *
- * A prefixed command (`simt-dev init`, `simt hooks run`) is a command only when
+ * A prefixed command (`simt init`, `simt hooks run`) is a command only when
  * that command exists. The tool name on its own is not a command and not a path.
  * Anything else is a path or a glob, at the repo root and then under each source
  * root. A token that is none of those is unresolved.

package/src/index/map.mjs

@@ -285,7 +285,7 @@ export async function embedTaskQuery(root, task, { embed = false, allowRemote =
  * }} [opts]
  */
 export async function measureRecall(root, opts = {}) {
-  const tasksDir = join(root, '.simt-dev', 'bench', 'tasks');
+  const tasksDir = join(root, '.simt', 'bench', 'tasks');
   let names;
   try {
     names = (await readdir(tasksDir)).filter((name) => name.endsWith('.json')).sort();

package/src/index/persist.mjs

@@ -7,8 +7,8 @@
 import { mkdir, readFile, stat, writeFile } from 'node:fs/promises';
 import { dirname, join } from 'node:path';
 
-export const SQLITE_FILE = '.simt-dev/index.sqlite';
-export const JSONL_FILE = '.simt-dev/index.jsonl';
+export const SQLITE_FILE = '.simt/index.sqlite';
+export const JSONL_FILE = '.simt/index.jsonl';
 export const SCHEMA_V = '3';
 
 const SCHEMA = `

package/src/index/store.mjs

@@ -4,7 +4,7 @@
 // otherwise. Either one is a cache: `git ls-files -s` can rebuild it, so a missing
 // file is not a lost database. A blob that has not changed is not read again.
 
-import { execFile } from 'node:child_process';
+import { execFile, spawn } from 'node:child_process';
 import { stat } from 'node:fs/promises';
 import { dirname, join } from 'node:path';
 
@@ -387,9 +387,9 @@ export async function packageOf(root, filePath, cache) {
   }
 }
 
-function git(root, args, input, binary = false) {
+function git(root, args, input) {
   return new Promise((resolve, reject) => {
-    const child = execFile('git', args, { cwd: root, encoding: binary ? 'buffer' : 'utf8', maxBuffer: 64 * 1024 * 1024 }, (error, stdout, stderr) => {
+    const child = execFile('git', args, { cwd: root, encoding: 'utf8', maxBuffer: 64 * 1024 * 1024 }, (error, stdout, stderr) => {
       if (error) reject(Object.assign(error, { stdout, stderr }));
       else resolve(stdout);
     });
@@ -477,28 +477,58 @@ function normalise(filePath, got) {
   return { symbols, imports: Array.isArray(got.imports) ? got.imports : [], edges };
 }
 
+/**
+ * The body of each listed blob, read as one `cat-file --batch` stream.
+ *
+ * Streamed and parsed incrementally rather than buffered: the combined output of a
+ * repository whose tracked content exceeds execFile's maxBuffer (64 MiB) used to
+ * reject the whole index with "stdout maxBuffer length exceeded" — an ~82 MiB
+ * Android checkout died on first run for exactly that reason.
+ */
 export async function blobTexts(root, blobs) {
   const unique = [...new Set(blobs)];
   const texts = new Map();
   if (unique.length === 0) return texts;
-  const stdout = await git(root, ['cat-file', '--batch'], `${unique.join('\n')}\n`, true);
-  let offset = 0;
-  while (offset < stdout.length) {
-    const nl = stdout.indexOf(0x0a, offset);
-    if (nl < 0) break;
-    const header = stdout.toString('utf8', offset, nl);
-    const parts = header.split(' ');
-    const size = Number(parts[2]);
-    if (parts[1] !== 'blob' || !Number.isFinite(size)) {
-      offset = nl + 1;
-      continue;
+  const child = spawn('git', ['cat-file', '--batch'], { cwd: root });
+  child.stdin.end(`${unique.join('\n')}\n`);
+  let pending = null;
+  const onChunk = (chunk) => {
+    pending = pending === null ? chunk : Buffer.concat([pending, chunk]);
+    let consumed = 0;
+    for (;;) {
+      const nl = pending.indexOf(0x0a, consumed);
+      if (nl < 0) break;
+      const parts = pending.toString('utf8', consumed, nl).split(' ');
+      const size = Number(parts[2]);
+      if (parts[1] !== 'blob' || !Number.isFinite(size)) {
+        consumed = nl + 1;
+        continue;
+      }
+      const end = nl + 1 + size + 1;
+      if (pending.length < end) break; // the body is not fully arrived yet
+      const start = nl + 1;
+      const body = pending.subarray(start, start + size);
+      consumed = end;
+      texts.set(parts[0], body.includes(0) ? null : body.toString('utf8'));
     }
-    const start = nl + 1;
-    const body = stdout.subarray(start, start + size);
-    offset = start + size + 1;
-    texts.set(parts[0], body.includes(0) ? null : body.toString('utf8'));
-  }
-  return texts;
+    // Drop the consumed prefix. Copy the tail once it has grown past a MiB, so the
+    // released bytes are actually freed instead of staying alive behind a view into
+    // the buffer they came from.
+    if (consumed > 0) {
+      const tail = pending.subarray(consumed);
+      pending = tail.length === 0 ? null : consumed >= 1 << 20 ? Buffer.from(tail) : tail;
+    }
+  };
+  return new Promise((resolve, reject) => {
+    let stderr = '';
+    child.stderr.on('data', (d) => { stderr += d; });
+    child.stdout.on('data', onChunk);
+    child.on('error', reject);
+    child.on('close', (code) => {
+      if (code !== 0) reject(new Error(`git cat-file --batch exited ${code}${stderr ? `: ${stderr.trim()}` : ''}`));
+      else resolve(texts);
+    });
+  });
 }
 
 async function blobSizes(root, blobs) {

package/src/ledger/index.mjs

@@ -22,7 +22,7 @@ import {
 
 const exec = promisify(execFile);
 
-const LEDGER_FILE = '.simt-dev/ledger.jsonl';
+const LEDGER_FILE = '.simt/ledger.jsonl';
 
 /** Phrases that claim a run happened. `unverified` and `not verified` are removed
  *  first, because both contain the word and neither is a claim. */
@@ -76,7 +76,7 @@ export async function recordRun(root, { cmd, exit, ms, output }) {
         }
       : {}),
   };
-  await mkdir(join(root, '.simt-dev'), { recursive: true });
+  await mkdir(join(root, '.simt'), { recursive: true });
   await appendFile(join(root, LEDGER_FILE), `${JSON.stringify(line)}\n`);
 
   if (exit === 0 && typeof cmd === 'string') {

package/src/local/client.mjs

@@ -26,6 +26,7 @@ function guardEndpoint(endpoint, { allowRemote = false } = {}) {
   if (!model) {
     return { ok: false, error: 'local model endpoint is missing `model`', refusedRemote: false };
   }
+  const apiKey = typeof endpoint.apiKey === 'string' && endpoint.apiKey.length > 0 ? endpoint.apiKey : null;
   const loopback = isLoopbackUrl(rawUrl);
   if (!loopback && !allowRemote) {
     return {
@@ -39,6 +40,7 @@ function guardEndpoint(endpoint, { allowRemote = false } = {}) {
     url: rawUrl,
     baseUrl: resolveEndpointUrl(rawUrl, ''),
     model,
+    apiKey,
     timeoutMs: Number.isInteger(endpoint.timeoutMs) && endpoint.timeoutMs > 0
       ? endpoint.timeoutMs
       : DEFAULT_LOCAL_TIMEOUT_MS,
@@ -46,13 +48,20 @@ function guardEndpoint(endpoint, { allowRemote = false } = {}) {
   };
 }
 
-async function requestJson(url, { method = 'GET', body = undefined, timeoutMs = DEFAULT_LOCAL_TIMEOUT_MS, fetchImpl = globalThis.fetch } = {}) {
+async function requestJson(url, { method = 'GET', body = undefined, timeoutMs = DEFAULT_LOCAL_TIMEOUT_MS, apiKey = null, fetchImpl = globalThis.fetch } = {}) {
   const controller = new AbortController();
   const timer = setTimeout(() => controller.abort(), timeoutMs);
   try {
+    const headers = {
+      ...(body !== undefined ? { 'content-type': 'application/json' } : {}),
+      accept: 'application/json',
+    };
+    if (typeof apiKey === 'string' && apiKey.length > 0) {
+      headers.authorization = `Bearer ${apiKey}`;
+    }
     const response = await fetchImpl(url, {
       method,
-      headers: body !== undefined ? { 'content-type': 'application/json', accept: 'application/json' } : { accept: 'application/json' },
+      headers,
       body: body !== undefined ? JSON.stringify(body) : undefined,
       signal: controller.signal,
     });
@@ -121,6 +130,7 @@ export async function localModels(endpoint, { allowRemote = false, fetchImpl = g
   const res = await requestJson(targetUrl, {
     method: 'GET',
     timeoutMs: guard.timeoutMs,
+    apiKey: guard.apiKey,
     fetchImpl,
   });
   if (!res.ok) {
@@ -200,6 +210,7 @@ export async function localChat(
       stream: false,
     },
     timeoutMs: guard.timeoutMs,
+    apiKey: guard.apiKey,
     fetchImpl,
   });
 
@@ -216,12 +227,14 @@ export async function localChat(
   }
 
   const choice = Array.isArray(res.data.choices) ? res.data.choices[0] : null;
-  const content =
-    typeof choice?.message?.content === 'string'
-      ? choice.message.content
-      : typeof choice?.text === 'string'
-        ? choice.text
-        : null;
+  const message = choice?.message ?? {};
+  // Prefer the final answer. A reasoning model (omlx, Qwen3-thinking) emits its
+  // output in `reasoning_content` and leaves `content` empty or absent when the
+  // budget is tiny (e.g. doctor's 1-token ping), so fall back to it, then the
+  // legacy `text` field. Whitespace-only output counts as absent.
+  const content = [message.content, message.reasoning_content, choice?.text]
+    .map((value) => (typeof value === 'string' && value.trim().length > 0 ? value : null))
+    .find((value) => value !== null) ?? null;
 
   if (content === null) {
     return {
@@ -275,6 +288,7 @@ export async function localEmbed(
       input: Array.isArray(input) ? input.map(String) : String(input ?? ''),
     },
     timeoutMs: guard.timeoutMs,
+    apiKey: guard.apiKey,
     fetchImpl,
   });
 

package/src/local/config.mjs

@@ -8,8 +8,13 @@
 //     "embed": { "url": "http://127.0.0.1:11434/v1", "model": "nomic-embed-code" }
 //   }
 //
+// Each role also accepts an optional `apiKey` (a non-empty string). When present it is
+// sent as an `Authorization: Bearer <key>` header, for servers that require a token
+// (omlx, llama.cpp `--api-key`, OpenAI-compatible gateways). Omitted by default.
+//
 // Non-loopback URLs are refused unless `--allow-remote` is explicitly passed (ADR-075).
 
+import { EXIT, fail } from '../output/index.mjs';
 import { MANIFEST_FILE } from '../name.mjs';
 import { readManifest } from '../projection/manifest.mjs';
 
@@ -113,6 +118,17 @@ function parseEndpoint(role, entry, { allowRemote = false } = {}) {
     };
   }
 
+  // Optional bearer credential for servers that require authentication (omlx,
+  // llama.cpp `--api-key`, OpenAI-compatible gateways). Absent by default: most
+  // local servers (Ollama, LM Studio) accept unauthenticated loopback requests.
+  let apiKey = '';
+  if (entry.apiKey !== undefined) {
+    if (typeof entry.apiKey !== 'string' || entry.apiKey.trim().length === 0) {
+      return { endpoint: null, finding: `\`local.${role}.apiKey\` must be a non-empty string when set` };
+    }
+    apiKey = entry.apiKey;
+  }
+
   const loopback = isLoopbackHost(parsedUrl.hostname);
   const baseUrl = resolveEndpointUrl(rawUrl, '');
   const blocked = !loopback && !allowRemote;
@@ -125,6 +141,7 @@ function parseEndpoint(role, entry, { allowRemote = false } = {}) {
         baseUrl,
         model,
         timeoutMs,
+        apiKey,
         loopback: false,
         blocked: true,
       },
@@ -140,6 +157,7 @@ function parseEndpoint(role, entry, { allowRemote = false } = {}) {
       baseUrl,
       model,
       timeoutMs,
+      apiKey,
       loopback,
       blocked: false,
     },
@@ -232,3 +250,37 @@ export async function readLocalConfig(root, { allowRemote = false } = {}) {
     manifestError: null,
   };
 }
+
+/**
+ * The guard every model consumer shares (R37's distill, R39's critic and bench).
+ *
+ * A `--local`-shaped call without a usable chat endpoint is a configuration
+ * error, never a silent fallback to the deterministic path: a caller who asked
+ * for the model and got the deterministic answer would read that answer as the
+ * model's — and the reverse, reading model output as deterministic, is worse.
+ * Second real user arrived with the critic, so the guard moved here from the
+ * distiller's `runEngine` (ADR-002).
+ *
+ * Returns `{ chat }` on success or `{ exit }` with the failure already shaped
+ * (the same pass-through shape the command layer returns).
+ */
+export function chatEndpoint(config) {
+  if (config.manifestError !== null) {
+    return { exit: fail(`${MANIFEST_FILE}: ${config.manifestError}`, EXIT.CONFIG) };
+  }
+  if (!config.configured || !config.chat || config.findings.length > 0) {
+    const detail = !config.configured
+      ? 'no local model is configured'
+      : config.remoteBlocked
+        ? 'the configured chat endpoint is non-loopback and was refused'
+        : 'the local configuration has errors';
+    return {
+      exit: fail(
+        `--local needs a usable local chat endpoint: ${detail}`,
+        EXIT.CONFIG,
+        `configure local.chat in ${MANIFEST_FILE} (plan 0003 R37), or run without --local for the deterministic path`,
+      ),
+    };
+  }
+  return { chat: config.chat };
+}

package/src/locate.mjs

@@ -12,6 +12,7 @@
 // this is a block and not a script.
 
 import { BIN_ENV, BIN_REL, CLI, PACKAGE_NAME } from './name.mjs';
+import { LOCAL_BIN } from './distribute/release.mjs';
 
 /**
  * @param {string} command arguments to hand the tool, with `$ROOT` spelled as a shell
@@ -24,6 +25,13 @@ import { BIN_ENV, BIN_REL, CLI, PACKAGE_NAME } from './name.mjs';
  * @returns {string} POSIX sh source
  */
 export function renderLocator(command, { subject = CLI, remedy = `REMEDY run: npm i -D ${PACKAGE_NAME}` } = {}) {
+  // The remedy is emitted single-quoted (see the assignment at the bottom of the
+  // block), so a quote in it would end that string and re-open the shell to
+  // whatever follows. Refuse at render time: the failure is a template bug, and
+  // it should be found where the string is written, not in a hook on a machine.
+  if (remedy.includes("'")) {
+    throw new Error(`the remedy is embedded single-quoted and cannot contain a quote: ${remedy}`);
+  }
   const D = '$';
   return `# ${D}${BIN_ENV} must name ONE program: an executable on PATH, or a .mjs entry
 # point to hand to node. It cannot be a command line, because there is no safe way
@@ -59,6 +67,15 @@ if [ -f "${D}ROOT/node_modules/${PACKAGE_NAME}/${BIN_REL}" ]; then
   exec node "${D}ROOT/node_modules/${PACKAGE_NAME}/${BIN_REL}" ${command}
 fi
 
+# The binary this repository downloaded for itself: the pinned bytes, no node.
+# It is checked before PATH so a repository's own copy wins over whatever a
+# machine happens to have installed, the same way node_modules wins - and the
+# dispatcher still compares the version it runs against the pin, so a stale
+# copy is refused rather than trusted.
+if [ -x "${D}ROOT/${LOCAL_BIN}" ]; then
+  exec "${D}ROOT/${LOCAL_BIN}" ${command}
+fi
+
 if command -v ${CLI} >/dev/null 2>&1; then
   exec ${CLI} ${command}
 fi
@@ -68,7 +85,11 @@ echo "${CLI}: cannot locate ${subject}." >&2
 # unbound variable aborts the shell, so an unguarded ${D}${BIN_ENV} here would
 # kill the message that exists to explain the failure - and this block only runs
 # when nothing resolved, which is the one moment the reader needs it.
-echo "  looked for: ${D}{${BIN_ENV}:-<unset>}, ${D}ROOT/node_modules/${PACKAGE_NAME}, ${CLI} on PATH" >&2
+echo "  looked for: ${D}{${BIN_ENV}:-<unset>}, ${D}ROOT/node_modules/${PACKAGE_NAME}, ${D}ROOT/${LOCAL_BIN}, ${CLI} on PATH" >&2
+# The remedy is assigned single-quoted and echoed, so a command in it - the binary
+# download derives its platform with $(uname) - stays text here and runs only when
+# the reader runs it. A remedy in double quotes would be half-executed by this echo.
+remedy='${remedy}'
 echo "${remedy}" >&2
 exit 1
 `;

package/src/mcp/prompts.mjs

@@ -0,0 +1,118 @@
+// Two prompt templates for the two jobs this server exists for.
+//
+// A prompt is a message template, not an answer. The server is read-only, so a prompt
+// cannot implement anything either: what it CAN do is hand a calling agent a working set
+// of instructions that name the tools to call and the order to call them in, which is the
+// part a model otherwise has to discover by trial. The text therefore speaks ABOUT the
+// tools and never runs them.
+
+/**
+ * `simt-brief`: start a task from the pre-assembled packet.
+ *
+ * The brief tool is the entry point every other surface points at, so the one prompt a
+ * client needs before it knows the model is "start there". Required `task` degrades to a
+ * visible placeholder rather than an error: a client that ignored the `required` flag at
+ * least gets a message that says so instead of a broken stream, and the agent reading it
+ * knows to ask before calling anything.
+ */
+const SIMT_BRIEF = {
+  name: 'simt-brief',
+  title: 'Work a task from the simt brief',
+  description: 'Start a task from the pre-assembled brief packet, and work from what it selected and why.',
+  arguments: [
+    { name: 'task', description: "the task, in the user's own words", required: true },
+  ],
+};
+
+/**
+ * `simt-slice`: implement one slice of the working plan.
+ *
+ * `id` is optional and omission is the DEFAULT reading - the next slice whose
+ * dependencies are complete is what "implement the next slice" means, and a plan whose
+ * numbering a reader does not know yet has no id to give.
+ */
+const SIMT_SLICE = {
+  name: 'simt-slice',
+  title: 'Implement a plan slice',
+  description: 'Implement the next ready slice of the working plan, or the named one.',
+  arguments: [
+    { name: 'id', description: 'slice id; omitted means the next ready one', required: false },
+  ],
+};
+
+export const PROMPTS = [SIMT_BRIEF, SIMT_SLICE];
+
+/**
+ * The prompt names as words, exported for the tool-vocabulary derivation in
+ * `src/toolwords.mjs`: a name this tool serves over MCP is this tool's own name, and the
+ * estate guard that reads that derivation would otherwise read `simt-brief` the way it
+ * reads a repository (ADR-080).
+ */
+export const PROMPT_WORDS = PROMPTS.map((prompt) => prompt.name);
+
+/**
+ * The prompt named, or null when there is no such name.
+ *
+ * `null`, not a throw: the caller decides what a missing prompt is worth (the server
+ * answers with the method error a client expects), and `getPrompt` stays a lookup.
+ */
+export function getPrompt(name, args = {}) {
+  const brief = name === SIMT_BRIEF.name;
+  const slice = name === SIMT_SLICE.name;
+  if (!brief && !slice) return null;
+
+  // Prompts degrade on missing arguments instead of failing: a template with an unfilled
+  // variable is still readable, and the agent it reaches is exactly the one that can ask
+  // the user what was meant.
+  const id = typeof args?.id === 'string' && args.id.length > 0 ? args.id : null;
+  const text = brief
+    ? briefText(typeof args?.task === 'string' && args.task.trim() !== '' ? args.task : null)
+    : sliceText(id);
+
+  return {
+    description: brief ? SIMT_BRIEF.description : SIMT_SLICE.description,
+    messages: [{ role: 'user', content: { type: 'text', text } }],
+  };
+}
+
+function briefText(task) {
+  const ask = task ?? '(none given - ask the user what to work before calling anything)';
+  return `Work the task through this repository's simt tools before touching code.
+
+The task: ${ask}
+
+1. Call the \`brief\` tool with this task. It returns the pre-assembled packet: the kind
+   of task, the commands that kind needs, the files the index ranks highest, the tests
+   covering them, the decisions governing them and the skills that apply.
+2. Work from the packet. Where something you need was cut to fit the budget, the packet
+   names the tool that fetches it - call that tool instead of guessing.
+3. Make the change, then prove it by running the packet's focused test before anything
+   broader.
+
+Report what changed and the evidence, not a summary of the effort.
+`;
+}
+
+function sliceText(id) {
+  const ask = id
+    ? `id ${id}`
+    : 'the next one whose dependencies are complete';
+  const lead = id
+    ? `Call the \`plan_next\` tool with id "${id}" for the design and acceptance checks of exactly that slice.`
+    : 'Call the `plan_next` tool with no id: it returns the next queued slice whose dependencies are complete, with that slice\'s design section only.';
+  return `Implement one slice of this repository's plan, reading it through the simt tools.
+
+The slice: ${ask}
+
+1. ${lead}
+2. Read \`simt://kernel\` first: the rules this repository packs for its stack, and the
+   ones it defers to \`simt rules show\`. They govern how you work here.
+3. Implement in the design's language, writing the regression test its acceptance check
+   asks for as you go. A slice is done when its checks hold, not when the code runs.
+4. Run the tests the design names, then the full suite - the last both in place and
+   from a copy at a path with no siblings, because a suite that only passes at home has
+   not passed.
+5. Commit locally with a conventional subject and a body that records why, and report
+   the sha. Push only on explicit user approval.
+`;
+}