simt-dev 0.7.1 → 0.8.0

package/src/bench/run.mjs

@@ -1,769 +1,88 @@
+// Run benchmark tasks across arms, and re-grade a stored run.
+//
+// The orchestrator owns the plan and nothing else: an arm is materialised by the registry
+// in `arms.mjs`, a harness is driven by the adapter record in `adapters.mjs`, a workspace
+// is built by `workspace.mjs`, a verdict comes from the task's own oracle in `graders.mjs`,
+// and a record is written by `records.mjs` (review 0003 Q78). What is left here is the
+// order those five things happen in.
 import { existsSync } from 'node:fs';
-import { lstat, mkdir, mkdtemp, readdir, readFile, rm, stat, unlink, writeFile } from 'node:fs/promises';
+import { mkdir, mkdtemp, readdir, readFile, rm, stat, writeFile } from 'node:fs/promises';
 import { tmpdir } from 'node:os';
 import { dirname, join, resolve, sep } from 'node:path';
-import { fileURLToPath } from 'node:url';
 
-import { assembleBrief } from '../brief/assemble.mjs';
-import { DEFAULT_BRIEF_BYTES } from '../brief/packet.mjs';
-import { loadRegistry } from '../graph/load.mjs';
 import { extractOpenedSkills, matchSkillsForPaths } from '../graph/compile.mjs';
-import { MANIFEST_FILE, STATE_DIR } from '../name.mjs';
+import { loadRegistry } from '../graph/load.mjs';
 import { LOCAL_DIR } from '../state.mjs';
+import { packageVersion } from '../package-version.mjs';
 import { readManifest } from '../projection/manifest.mjs';
-import { plan } from '../projection/project.mjs';
-import { parse as parseRegion } from '../projection/markers.mjs';
-import { gradeWorkspace } from './grade.mjs';
-import { resolveVaultDir } from './mine.mjs';
-import { spawnBounded } from './process.mjs';
-import { loadTasks } from './tasks.mjs';
-import { ablateSectionInWorkspace, parseAblationArmName } from './ablate.mjs';
-import { applyVariantInWorkspace, parseVariantArmName } from './variants.mjs';
-
-const BENCH_DIR = dirname(fileURLToPath(import.meta.url));
-const ADAPTERS_PATH = join(BENCH_DIR, 'adapters.json');
-
-export const BUILTIN_ARMS = new Set(['bare', 'current', 'kernel', 'brief']);
+import {
+  ADAPTER_SCHEMA_VERSION,
+  adapterIsolation,
+  applyPatch,
+  executeAdapter,
+  loadAdapters,
+} from './adapters.mjs';
+import { BUILTIN_ARMS, describeArm, parseArmSpec } from './arms.mjs';
+import { gradeWithOracle } from './graders.mjs';
+// Importing the tests grader registers the oracle it grades for; the runner asks the
+// registry rather than the module, so a second oracle needs no change here.
+import './grade.mjs';
+import { resolveVaultDir } from '../testing/vault.mjs';
+import {
+  RECORD_SCHEMA_VERSION,
+  corpusDigestOf,
+  experimentOf,
+  runIdFor,
+  runRecord,
+} from '../measure/records.mjs';
+import { DEFAULT_BOOTSTRAP_SEED } from '../measure/stats.mjs';
+import { loadTasks, readTask } from '../measure/tasks.mjs';
+import { mapPool, prepareBaseTree, prepareIsolatedWorkspace } from './workspace.mjs';
 
 const DEFAULT_REPS = 3;
 const DEFAULT_MAX_TURNS = 25;
 const DEFAULT_AGENT_TIMEOUT_MS = 120_000;
 const DEFAULT_CONCURRENCY = 4;
 
-let cachedAdapters = null;
-
-/**
- * Load the declarative harness adapter table (`src/bench/adapters.json`).
- *
- * @returns {Promise<Record<string, object>>}
- */
-export async function loadAdapters() {
-  if (cachedAdapters) return cachedAdapters;
-  const raw = JSON.parse(await readFile(ADAPTERS_PATH, 'utf8'));
-  cachedAdapters = raw.adapters;
-  return cachedAdapters;
-}
-
-function sanitizedEnv(extra = {}) {
-  const env = { ...process.env, SIMT_DEV_WORKSPACE: 'off', ...extra };
-  for (const key of Object.keys(env)) {
-    if (key.startsWith('NODE_TEST')) delete env[key];
-  }
-  return env;
-}
-
-/**
- * What every benchmarked agent is allowed to inherit from the operator's shell.
- *
- * `PATH` and the locale are what a CLI needs to run at all. Nothing here names a
- * credential or a state directory: those come from the adapter's own `env` list.
- */
-const ENV_BASELINE = [
-  'PATH',
-  'LANG',
-  'LC_ALL',
-  'LC_CTYPE',
-  'TZ',
-  'TERM',
-  'SHELL',
-  'TMPDIR',
-  'TEMP',
-  'TMP',
-  'SystemRoot',
-  'WINDIR',
-  'USER',
-];
-
-/** The isolation levels a run may record; `sandbox` is declared, not implemented. */
-const ISOLATION_LEVELS = new Set(['none', 'env', 'sandbox']);
-
-/**
- * The environment an arm's agent sees: the baseline, the adapter's declared keys,
- * and nothing else.
- *
- * The agent used to inherit everything the operator had — `SIMT_DEV_VAULT_DIR`,
- * `XDG_CACHE_HOME`, `HOME`, and every credential in the shell. A run that can read
- * the vault can read the hidden tests it is graded against, and a run sharing the
- * operator's HOME writes into their own configuration, so the number it produced
- * described the machine rather than the arm (review 0003 Q49). `home` is the run's
- * own slot, which is removed when the run ends.
- *
- * @param {object} adapter an entry from `adapters.json`
- * @param {{home?: string, extra?: Record<string, string>, base?: NodeJS.ProcessEnv}} [options]
- * @returns {Record<string, string>}
- */
-export function agentEnv(adapter, { home, extra = {}, base = process.env } = {}) {
-  const allowed = new Set([...ENV_BASELINE, ...(adapter?.env ?? [])]);
-  const env = {};
-  for (const key of allowed) {
-    if (typeof base[key] === 'string') env[key] = base[key];
-  }
-  for (const [key, value] of Object.entries(extra ?? {})) {
-    if (typeof value === 'string') env[key] = value;
-  }
-  if (typeof home === 'string' && home.length > 0) env.HOME = home;
-  env.SIMT_DEV_WORKSPACE = 'off';
-  for (const key of Object.keys(env)) {
-    if (key.startsWith('NODE_TEST')) delete env[key];
-  }
-  return env;
-}
-
-/**
- * The isolation an adapter's runs provide, as the record states it.
- *
- * `none` is a built-in that scripts the workspace, `env` is a CLI driven with a
- * scrubbed environment and a slot-local HOME, and `sandbox` is reserved for a
- * real filesystem sandbox — nothing claims it yet, so a record never says it.
- *
- * @param {object} adapter
- * @returns {'none'|'env'|'sandbox'}
- */
-export function adapterIsolation(adapter) {
-  const declared = adapter?.isolation;
-  return ISOLATION_LEVELS.has(declared) ? declared : 'none';
-}
-
-function runCmd(cmd, args, { cwd, env, input = null, timeoutMs = DEFAULT_AGENT_TIMEOUT_MS } = {}) {
-  return spawnBounded(cmd, args, { cwd, env: sanitizedEnv(env), input, timeoutMs });
-}
-
-function getByPath(obj, pathStr) {
-  if (!obj || typeof obj !== 'object') return undefined;
-  const parts = pathStr.split('.');
-  let cur = obj;
-  for (const part of parts) {
-    if (cur === null || cur === undefined || typeof cur !== 'object') return undefined;
-    cur = cur[part];
-  }
-  return cur;
-}
-
-function firstNumeric(obj, paths = []) {
-  for (const p of paths) {
-    const val = getByPath(obj, p);
-    if (typeof val === 'number' && Number.isFinite(val)) return val;
-  }
-  return null;
-}
-
-function emptyUsage() {
-  return {
-    usage: { input: 0, cacheWrite: 0, cacheRead: 0, output: 0, turns: 0, toolCalls: 0 },
-    costUsd: null,
-  };
-}
-
-function parseJsonObject(text) {
-  const trimmed = String(text ?? '').trim();
-  if (!trimmed) return null;
-  try {
-    return JSON.parse(trimmed);
-  } catch {
-    const lines = trimmed.split('\n').reverse();
-    for (const line of lines) {
-      const s = line.trim();
-      if (s.startsWith('{') && s.endsWith('}')) {
-        try {
-          return JSON.parse(s);
-        } catch {
-          // try earlier line
-        }
-      }
-    }
-    const start = trimmed.indexOf('{');
-    const end = trimmed.lastIndexOf('}');
-    if (start !== -1 && end > start) {
-      try {
-        return JSON.parse(trimmed.slice(start, end + 1));
-      } catch {
-        return null;
-      }
-    }
-    return null;
-  }
-}
-
-/**
- * Parse token usage, turns, tool calls and USD cost from a harness's output.
- *
- * Uses the field paths declared in `src/bench/adapters.json` and supports single-JSON
- * outputs (`claude`, `gemini`, `cursor`), JSONL event streams (`codex`, `opencode`),
- * and `--analytics-log` JSONL files (`aider`).
- *
- * @param {string|object} adapterOrId
- * @param {string} stdoutText
- * @param {string} [analyticsText]
- * @returns {{usage: {input: number, cacheWrite: number, cacheRead: number, output: number, turns: number, toolCalls: number}, costUsd: number|null}}
- */
-export function parseAdapterUsage(adapterOrId, stdoutText = '', analyticsText = '') {
-  const result = emptyUsage();
-  const adapter =
-    typeof adapterOrId === 'string'
-      ? (cachedAdapters?.[adapterOrId] ?? { id: adapterOrId, format: 'json', fields: {} })
-      : (adapterOrId ?? { id: 'unknown', format: 'json', fields: {} });
-
-  if (adapter.format === 'builtin') {
-    if (adapter.id === 'gold') {
-      result.usage.turns = 1;
-      result.usage.toolCalls = 1;
-    }
-    return result;
-  }
-
-  const fields = adapter.fields ?? {};
-
-  if (adapter.format === 'json') {
-    const obj = parseJsonObject(stdoutText);
-    if (!obj || typeof obj !== 'object') return result;
-
-    result.usage.input = firstNumeric(obj, fields.input) ?? 0;
-    result.usage.cacheWrite = firstNumeric(obj, fields.cacheWrite) ?? 0;
-    result.usage.cacheRead = firstNumeric(obj, fields.cacheRead) ?? 0;
-    result.usage.output = firstNumeric(obj, fields.output) ?? 0;
-    result.usage.turns = firstNumeric(obj, fields.turns) ?? 0;
-    result.usage.toolCalls = firstNumeric(obj, fields.toolCalls) ?? 0;
-    result.costUsd = firstNumeric(obj, fields.costUsd) ?? null;
-
-    // Gemini CLI also nests per-model token totals under `stats.models.<name>.tokens`.
-    const models = obj?.stats?.models;
-    if (models && typeof models === 'object') {
-      let promptSum = 0;
-      let cachedSum = 0;
-      let candidatesSum = 0;
-      for (const m of Object.values(models)) {
-        promptSum += Number(m?.tokens?.prompt ?? 0) || 0;
-        cachedSum += Number(m?.tokens?.cached ?? 0) || 0;
-        candidatesSum += Number(m?.tokens?.candidates ?? 0) || 0;
-      }
-      if (result.usage.input === 0 && promptSum > 0) result.usage.input = promptSum;
-      if (result.usage.cacheRead === 0 && cachedSum > 0) result.usage.cacheRead = cachedSum;
-      if (result.usage.output === 0 && candidatesSum > 0) result.usage.output = candidatesSum;
-    }
-
-    return result;
-  }
-
-  const rawStream = adapter.format === 'analytics-log' && analyticsText ? analyticsText : stdoutText;
-  const lines = String(rawStream ?? '')
-    .split('\n')
-    .map((l) => l.trim())
-    .filter(Boolean);
-
-  for (const line of lines) {
-    let event;
-    try {
-      event = JSON.parse(line);
-    } catch {
-      continue;
-    }
-    if (!event || typeof event !== 'object') continue;
-
-    const input = firstNumeric(event, fields.input);
-    const cacheWrite = firstNumeric(event, fields.cacheWrite);
-    const cacheRead = firstNumeric(event, fields.cacheRead);
-    const output = firstNumeric(event, fields.output);
-    const turns = firstNumeric(event, fields.turns);
-    const toolCalls = firstNumeric(event, fields.toolCalls);
-    const cost = firstNumeric(event, fields.costUsd);
-
-    if (input !== null) result.usage.input += input;
-    if (cacheWrite !== null) result.usage.cacheWrite += cacheWrite;
-    if (cacheRead !== null) result.usage.cacheRead += cacheRead;
-    if (output !== null) result.usage.output += output;
-    if (cost !== null) result.costUsd += cost;
-
-    if (turns !== null) {
-      result.usage.turns = Math.max(result.usage.turns, turns);
-    } else if (
-      event.type === 'turn.completed' ||
-      event.type === 'step_finish' ||
-      event.event === 'message_send'
-    ) {
-      result.usage.turns += 1;
-    }
-
-    if (toolCalls !== null) {
-      result.usage.toolCalls = Math.max(result.usage.toolCalls, toolCalls);
-    } else if (
-      event.type === 'tool_use' ||
-      (event.type === 'item.completed' &&
-        (event.item?.type === 'command_execution' ||
-          event.item?.type === 'tool_call' ||
-          event.item?.type === 'mcp_tool_call'))
-    ) {
-      result.usage.toolCalls += 1;
-    }
-  }
-
-  if (result.costUsd !== null) result.costUsd = Number(result.costUsd.toFixed(6));
-  return result;
-}
-
-/**
- * Parse an `--arm` specification (`bare`, `current`, `kernel`, `brief`, or `name=path/to/arm.json`).
- *
- * @param {string} rawSpec
- * @param {string} [cwd]
- * @returns {Promise<{name: string, kind: 'builtin'|'custom', base: string|null, write: Record<string, string>, remove: string[], env: Record<string, string>}>}
- */
-export async function parseArmSpec(rawSpec, cwd = process.cwd()) {
-  const spec = String(rawSpec ?? '').trim();
-  if (!spec) throw new Error('empty --arm specification');
-
-  const eq = spec.indexOf('=');
-  if (eq === -1) {
-    const ablateSpec = parseAblationArmName(spec);
-    if (ablateSpec) {
-      return {
-        ...ablateSpec,
-        write: {},
-        remove: [],
-        env: {},
-      };
-    }
-    const variantSpec = parseVariantArmName(spec);
-    if (variantSpec) {
-      return {
-        ...variantSpec,
-        write: {},
-        remove: [],
-        env: {},
-      };
-    }
-    if (!BUILTIN_ARMS.has(spec)) {
-      throw new Error(
-        `unknown arm "${spec}"; built-in arms are: ${[...BUILTIN_ARMS].join(', ')} (or pass --arm name=path/to/arm.json)`,
-      );
-    }
-    return {
-      name: spec,
-      kind: 'builtin',
-      base: spec,
-      write: {},
-      remove: [],
-      env: {},
-    };
-  }
-
-  const name = spec.slice(0, eq).trim();
-  const relPath = spec.slice(eq + 1).trim();
-  if (!name || !/^[A-Za-z0-9._-]+$/.test(name)) {
-    throw new Error(`invalid custom arm name "${name}"`);
-  }
-  if (!relPath) {
-    throw new Error(`custom arm "${name}" is missing a JSON file path`);
-  }
-
-  const fullPath = resolve(cwd, relPath);
-  let data;
-  try {
-    data = JSON.parse(await readFile(fullPath, 'utf8'));
-  } catch (error) {
-    throw new Error(`could not read arm file ${relPath}: ${error.message ?? error}`);
-  }
-  if (!data || typeof data !== 'object' || Array.isArray(data)) {
-    throw new Error(`arm file ${relPath} must contain a JSON object`);
-  }
-  if (data.base !== undefined && data.base !== null && !BUILTIN_ARMS.has(data.base)) {
-    throw new Error(`arm file ${relPath} has unknown base "${data.base}"`);
-  }
-  if (data.write !== undefined && (typeof data.write !== 'object' || data.write === null || Array.isArray(data.write))) {
-    throw new Error(`arm file ${relPath} field "write" must be an object mapping paths to strings`);
-  }
-  if (data.remove !== undefined && !Array.isArray(data.remove)) {
-    throw new Error(`arm file ${relPath} field "remove" must be an array of paths`);
-  }
-  if (data.env !== undefined && (typeof data.env !== 'object' || data.env === null || Array.isArray(data.env))) {
-    throw new Error(`arm file ${relPath} field "env" must be an object mapping env keys to strings`);
-  }
-
-  const write = {};
-  for (const [k, v] of Object.entries(data.write ?? {})) write[k] = String(v);
-  const remove = (data.remove ?? []).map((p) => String(p));
-  const env = {};
-  for (const [k, v] of Object.entries(data.env ?? {})) env[k] = String(v);
-
-  return {
-    name,
-    kind: 'custom',
-    base: data.base ?? null,
-    write,
-    remove,
-    env,
-  };
-}
-
-function assertContained(workDir, relPath) {
-  const target = resolve(workDir, relPath);
-  if (target !== workDir && !target.startsWith(`${workDir}${sep}`)) {
-    throw new Error(`arm path "${relPath}" escapes workspace`);
-  }
-  return target;
-}
-
-async function stripRegionFile(filePath, style) {
-  let text;
-  try {
-    text = await readFile(filePath, 'utf8');
-  } catch {
-    return false;
-  }
-  const state = parseRegion(text, style);
-  if (state.status !== 'ok') return false;
-  const remaining = `${state.before.trimEnd()}\n${state.after.trimStart()}`.trim();
-  if (!remaining) {
-    await unlink(filePath).catch(() => {});
-  } else {
-    await writeFile(filePath, `${remaining}\n`, 'utf8');
-  }
-  return true;
-}
-
-async function stripBareArm(workDir) {
-  await stripRegionFile(join(workDir, 'AGENTS.md'), 'html');
-  await stripRegionFile(join(workDir, '.editorconfig'), 'hash');
-  await stripRegionFile(join(workDir, '.gitattributes'), 'hash');
-
-  const surfaceDirs = [
-    '.cursor/rules',
-    '.claude/rules',
-    '.github/instructions',
-    '.agents/rules',
-  ];
-  for (const relDir of surfaceDirs) {
-    const absDir = join(workDir, relDir);
-    const entries = await readdir(absDir).catch(() => []);
-    for (const entry of entries) {
-      const full = join(absDir, entry);
-      const text = await readFile(full, 'utf8').catch(() => null);
-      if (text !== null && parseRegion(text, 'html').status === 'ok') {
-        await unlink(full).catch(() => {});
-      }
-    }
-  }
-
-  for (const rel of ['.githooks', '.mcp.json', '.cursor/mcp.json', '.vscode/mcp.json', 'docs/lessons']) {
-    await rm(join(workDir, rel), { recursive: true, force: true }).catch(() => {});
-  }
-
-  if (!existsSync(join(workDir, 'AGENTS.md'))) {
-    for (const mirror of ['CLAUDE.md', 'GEMINI.md']) {
-      const full = join(workDir, mirror);
-      const st = await lstat(full).catch(() => null);
-      if (st?.isSymbolicLink()) {
-        await unlink(full).catch(() => {});
-      }
-    }
-  }
+function runsBaseDir(root) {
+  return join(root, LOCAL_DIR, 'bench', 'runs');
 }
 
-async function applyProjectionMode(workDir, mode) {
-  const manifestRes = await readManifest(workDir);
-  const agentsText = await readFile(join(workDir, 'AGENTS.md'), 'utf8').catch(() => null);
-  const hasRegion = agentsText !== null && parseRegion(agentsText, 'html').status === 'ok';
-
-  // If the repository is neither adopted via manifest nor carrying a generated region,
-  // `current` leaves its hand-written files as-is, while `kernel` projects a kernel
-  // region only when the repo is adopted or already carries a region.
-  if (!manifestRes.exists && !hasRegion) return;
-
-  const manifestPath = manifestRes.path ?? join(workDir, MANIFEST_FILE);
-  const hadManifest = manifestRes.exists;
-  const originalManifestBytes = hadManifest ? await readFile(manifestPath, 'utf8').catch(() => null) : null;
-
-  const baseManifest = manifestRes.data ?? {
-    schemaVersion: 1,
-    projection: { sha: '0000000000000000000000000000000000000000', sources: [] },
-    gates: { preCommit: [], prePush: [] },
-  };
-  const nextManifest = { ...baseManifest };
-  if (mode === 'kernel') nextManifest.mode = 'kernel';
-  else delete nextManifest.mode;
-
-  await writeFile(manifestPath, `${JSON.stringify(nextManifest, null, 2)}\n`, 'utf8');
-
-  const p = await plan(workDir);
-  for (const t of p.targets ?? []) {
-    if (t.action !== 'unchanged' && typeof t.text === 'string') {
-      await mkdir(dirname(t.path), { recursive: true });
-      await writeFile(t.path, t.text, 'utf8');
-    }
-  }
-
-  if (!hadManifest) {
-    await unlink(manifestPath).catch(() => {});
-  } else if (originalManifestBytes !== null && mode !== 'kernel') {
-    await writeFile(manifestPath, originalManifestBytes, 'utf8');
-  }
+/** Two string lists hold the same members, ignoring order and duplicates. */
+function sameStringSet(a, b) {
+  const left = [...new Set(a)].sort();
+  const right = [...new Set(b)].sort();
+  return left.length === right.length && left.every((value, index) => value === right[index]);
 }
 
 /**
- * The `brief` arm's context: the packet `simt brief` delivers, assembled by the same
- * function the command and the prompt-submit hook call.
+ * The experiment a batch is measured under: the conditions every record in it shares.
  *
- * The arm used to re-implement the packet from `registry.skills` — a property the
- * registry does not have, so every run reported "(none matched)" while the real
- * command resolved skills — and wrote it to a file no agent was told to read. An arm
- * meant to measure the packet measured a different artefact (review 0003 Q49).
+ * Recorded rather than inferred, because two batches that differ in rule text, corpus or
+ * adapter were not the same experiment and a report must be able to say so.
  */
-async function buildBriefPrefix(workDir, task) {
-  // A packet is built from git's answer; a directory that is not a repository has none,
-  // and asking would print a failure into a bench log for no measurement.
-  if (!existsSync(join(workDir, '.git'))) return '';
-  const assembled = await assembleBrief({
-    root: workDir,
-    task: String(task?.prompt ?? ''),
-    paths: '',
-    budget: DEFAULT_BRIEF_BYTES,
-    build: true,
-    federate: false,
-  }).catch(() => null);
-  if (!assembled || assembled.error || assembled.cold) return '';
-  return assembled.packed?.text ?? '';
+async function batchExperiment(root, { adapter, tasks, isolation, model }) {
+  const manifest = await readManifest(root).catch(() => ({ data: null }));
+  const projection = manifest?.data?.projection ?? {};
+  return experimentOf({
+    simtVersion: packageVersion(),
+    rulesDigest: projection.sha ?? projection.simtDevSha ?? null,
+    adapterVersion: `${adapter.id}@${ADAPTER_SCHEMA_VERSION}${model ? `:${model}` : ''}`,
+    corpusDigest: corpusDigestOf(tasks),
+    seed: DEFAULT_BOOTSTRAP_SEED,
+    isolation,
+  });
 }
 
 /**
- * Materialise an experimental arm (`bare`, `current`, `kernel`, `brief`, or custom) in `workDir`.
+ * Run benchmark tasks across one or more arms and repetitions.
  *
- * @param {string} workDir
- * @param {{name: string, kind?: string, base?: string|null, write?: Record<string, string>, remove?: string[], env?: Record<string, string>}} armSpec
- * @param {object} [task]
- * @returns {Promise<{env: Record<string, string>, promptPrefix: string}>}
- */
-export async function materialiseArm(workDir, armSpec, task = {}) {
-  let promptPrefix = '';
-  const ablate = armSpec.ablate ?? parseAblationArmName(armSpec.name)?.ablate ?? null;
-  const variant = armSpec.variant ?? parseVariantArmName(armSpec.name)?.variant ?? null;
-  const base = armSpec.base ?? (BUILTIN_ARMS.has(armSpec.name) ? armSpec.name : null);
-  if (ablate) {
-    await ablateSectionInWorkspace(workDir, ablate);
-  } else if (variant) {
-    await applyVariantInWorkspace(workDir, variant);
-  } else if (base === 'bare') {
-    await stripBareArm(workDir);
-  } else if (base === 'current') {
-    await applyProjectionMode(workDir, 'full');
-  } else if (base === 'kernel') {
-    await applyProjectionMode(workDir, 'kernel');
-  } else if (base === 'brief') {
-    await applyProjectionMode(workDir, 'full');
-    promptPrefix = await buildBriefPrefix(workDir, task);
-  }
-
-  for (const relPath of armSpec.remove ?? []) {
-    const target = assertContained(workDir, relPath);
-    await rm(target, { recursive: true, force: true }).catch(() => {});
-  }
-
-  for (const [relPath, content] of Object.entries(armSpec.write ?? {})) {
-    const target = assertContained(workDir, relPath);
-    await mkdir(dirname(target), { recursive: true });
-    await writeFile(target, String(content), 'utf8');
-  }
-
-  return { env: { ...(armSpec.env ?? {}) }, promptPrefix };
-}
-
-/**
- * Export `baseSha` from `root` via `git archive` into a fresh temp directory, materialise
- * `armSpec`, and initialize a single-commit git repository so `git rev-list --all` has
- * exactly one commit and the object store shares nothing with `root`.
+ * A preview answers the plan and writes nothing; `apply` prepares one base tree per task,
+ * materialises each arm over a copy of it, drives the adapter, grades through the task's
+ * oracle, and stores the record beside the agent's patch.
  *
  * @param {string} root
- * @param {string} baseSha
- * @param {object} armSpec
- * @param {object} [task]
- * @param {{parentDir?: string}} [options]
- * @returns {Promise<{slotDir: string, workDir: string, initialHead: string, env: Record<string, string>, promptPrefix: string}>}
- */
-export async function prepareIsolatedWorkspace(root, baseSha, armSpec, task = {}, { parentDir } = {}) {
-  const slotDir = await mkdtemp(join(parentDir ?? tmpdir(), 'bench-run-'));
-  const workDir = join(slotDir, 'workspace');
-  await mkdir(workDir, { recursive: true });
-
-  const archiveRes = await runCmd('git', ['archive', '--format=tar', baseSha], { cwd: root });
-  if (archiveRes.code !== 0) {
-    await rm(slotDir, { recursive: true, force: true }).catch(() => {});
-    throw new Error(
-      `git archive failed for ${baseSha}: ${archiveRes.stderr.toString('utf8').trim()}`,
-    );
-  }
-
-  const tarRes = await runCmd('tar', ['-xf', '-', '-C', workDir], {
-    cwd: workDir,
-    input: archiveRes.stdout,
-  });
-  if (tarRes.code !== 0) {
-    await rm(slotDir, { recursive: true, force: true }).catch(() => {});
-    throw new Error(`tar unpack failed for ${baseSha}: ${tarRes.stderr.toString('utf8').trim()}`);
-  }
-
-  const rootLessonsDir = join(root, 'docs', 'lessons');
-  const rootLessonFiles = (await readdir(rootLessonsDir).catch(() => [])).filter((f) => f.endsWith('.md'));
-  if (rootLessonFiles.length > 0) {
-    const destLessonsDir = join(workDir, 'docs', 'lessons');
-    await mkdir(destLessonsDir, { recursive: true });
-    for (const file of rootLessonFiles) {
-      const target = join(destLessonsDir, file);
-      if (!existsSync(target)) {
-        const text = await readFile(join(rootLessonsDir, file), 'utf8');
-        await writeFile(target, text, 'utf8');
-      }
-    }
-  }
-
-  const rootProfilesDir = join(root, STATE_DIR, 'profiles');
-  const rootProfileFiles = (await readdir(rootProfilesDir).catch(() => [])).filter((f) => f.endsWith('.md'));
-  if (rootProfileFiles.length > 0) {
-    const destProfilesDir = join(workDir, STATE_DIR, 'profiles');
-    await mkdir(destProfilesDir, { recursive: true });
-    for (const file of rootProfileFiles) {
-      const text = await readFile(join(rootProfilesDir, file), 'utf8');
-      await writeFile(join(destProfilesDir, file), text, 'utf8');
-    }
-  }
-
-  await runCmd('git', ['init', '-q'], { cwd: workDir });
-  await runCmd('git', ['config', 'user.name', 'bench'], { cwd: workDir });
-  await runCmd('git', ['config', 'user.email', 'bench@local'], { cwd: workDir });
-  await runCmd('git', ['config', 'commit.gpgsign', 'false'], { cwd: workDir });
-
-  // The repository exists before an arm materialises: the `brief` arm assembles its
-  // packet from git's answer, which a directory that is not yet a repository cannot give.
-  const { env, promptPrefix } = await materialiseArm(workDir, armSpec, task);
-  // Assembling a packet writes tool state. It belongs to no arm, and leaving it in the
-  // base commit would make the `brief` workspace differ by an index as well as by context.
-  await rm(join(workDir, LOCAL_DIR), { recursive: true, force: true }).catch(() => {});
-
-  await runCmd('git', ['add', '-A'], { cwd: workDir });
-  const commitRes = await runCmd(
-    'git',
-    ['commit', '-qm', 'base', '--allow-empty', '--no-verify'],
-    { cwd: workDir },
-  );
-  if (commitRes.code !== 0) {
-    await rm(slotDir, { recursive: true, force: true }).catch(() => {});
-    throw new Error(`git commit failed in isolated workspace: ${commitRes.stderr.toString('utf8').trim()}`);
-  }
-
-  const headRes = await runCmd('git', ['rev-parse', 'HEAD'], { cwd: workDir });
-  const initialHead = headRes.stdout.toString('utf8').trim();
-
-  return { slotDir, workDir, initialHead, env, promptPrefix };
-}
-
-function buildAdapterArgs(adapter, { prompt, turns, model, analyticsLog }) {
-  const replaceToken = (token) =>
-    String(token)
-      .replaceAll('{prompt}', prompt)
-      .replaceAll('{turns}', String(turns))
-      .replaceAll('{model}', String(model ?? ''))
-      .replaceAll('{analyticsLog}', String(analyticsLog ?? ''));
-
-  const args = (adapter.args ?? []).map(replaceToken);
-  if (model && model !== 'default' && model !== adapter.id && Array.isArray(adapter.modelArgs)) {
-    args.push(...adapter.modelArgs.map(replaceToken));
-  }
-  return args;
-}
-
-/**
- * Apply a patch to a workspace, falling back to a three-way merge.
- *
- * The gold oracle and the regrade path land operator-supplied bytes the same way. They
- * used to disagree (`--binary` on one side only), so a patch that applied under one read
- * as unreproducible under the other (review 0003 Q59).
- */
-async function applyPatch(workDir, patch, { timeoutMs = DEFAULT_AGENT_TIMEOUT_MS } = {}) {
-  if (!patch || patch.length === 0) return { code: 0 };
-  const applied = await runCmd('git', ['apply', '--binary', '-'], {
-    cwd: workDir,
-    input: patch,
-    timeoutMs,
-  });
-  if (applied.code === 0) return applied;
-  return runCmd('git', ['apply', '--3way', '-'], { cwd: workDir, input: patch, timeoutMs });
-}
-
-export async function executeAdapter(
-  adapter,
-  workDir,
-  task,
-  { vaultRoot, model, maxTurns = DEFAULT_MAX_TURNS, env, timeoutMs, slotDir, promptPrefix = '' },
-) {
-  if (adapter.id === 'noop') {
-    return {
-      ...parseAdapterUsage(adapter, ''),
-      transcript: '',
-      agent: { exitCode: 0, timedOut: false, wallMs: 0 },
-    };
-  }
-  if (adapter.id === 'gold') {
-    const startedAt = Date.now();
-    const goldPatch = await readFile(join(vaultRoot, task.id, 'gold.patch'));
-    await applyPatch(workDir, goldPatch, { timeoutMs });
-    return {
-      ...parseAdapterUsage(adapter, ''),
-      transcript: '',
-      agent: { exitCode: 0, timedOut: false, wallMs: Date.now() - startedAt },
-    };
-  }
-
-  const analyticsLog = join(slotDir, 'analytics.jsonl');
-  // The arm's context arrives as a prefix rather than a file the agent must be told to
-  // read, so the prompt an adapter receives is the whole thing it was given. A `gold`
-  // or `noop` run never reaches this line: it scripts the workspace instead.
-  const prompt = [String(promptPrefix ?? ''), String(task.prompt ?? '')]
-    .filter((part) => part.length > 0)
-    .join('\n\n');
-  const args = buildAdapterArgs(adapter, { prompt, turns: maxTurns, model, analyticsLog });
-  const outcome = await runCmd(adapter.bin, args, {
-    cwd: workDir,
-    env: agentEnv(adapter, { home: slotDir, extra: env }),
-    timeoutMs,
-  });
-  // A binary that could not start produced no run. Grading its empty workspace would
-  // record an unresolved arm instead of the missing harness it is (review 0003 Q47).
-  if (outcome.spawnError) {
-    throw new Error(`harness ${adapter.id} could not start ${adapter.bin}: ${outcome.spawnError}`);
-  }
-  const stdoutText = outcome.stdout.toString('utf8');
-  const analyticsText = adapter.format === 'analytics-log' ? await readFile(analyticsLog, 'utf8').catch(() => '') : '';
-  return {
-    ...parseAdapterUsage(adapter, stdoutText, analyticsText),
-    transcript: `${stdoutText}\n${analyticsText}`,
-    // A run's own exit code, deadline and wall time belong in its record: the old one
-    // stored none, so a timed-out run was indistinguishable from a completed one.
-    agent: { exitCode: outcome.code, timedOut: outcome.timedOut, wallMs: outcome.wallMs },
-  };
-}
-
-// loadTasks lives in tasks.mjs (the advice engine became a second reader, so the loader
-// was extracted rather than copied — ADR-002).
-
-function runIdFor(harness, armName, rep) {
-  return `${harness}-${armName}-r${rep}`;
-}
-
-async function mapPool(items, concurrency, fn) {
-  const results = new Array(items.length);
-  let cursor = 0;
-  async function worker() {
-    while (true) {
-      const idx = cursor;
-      cursor += 1;
-      if (idx >= items.length) return;
-      results[idx] = await fn(items[idx], idx);
-    }
-  }
-  const count = Math.max(1, Math.min(concurrency, items.length));
-  await Promise.all(Array.from({ length: count }, () => worker()));
-  return results;
-}
-
-/**
- * Run benchmark tasks across one or more arms and repetitions.
+ * @param {object} [options]
  */
 export async function runBench(
   root,
@@ -785,9 +104,7 @@ export async function runBench(
   const adapters = adapterTable ?? (await loadAdapters());
   const adapter = adapters[harness];
   if (!adapter) {
-    throw new Error(
-      `unknown harness "${harness}"; known harnesses: ${Object.keys(adapters).sort().join(', ')}`,
-    );
+    throw new Error(`unknown harness "${harness}"; known harnesses: ${Object.keys(adapters).sort().join(', ')}`);
   }
 
   if (!Number.isInteger(reps) || reps < 1) {
@@ -808,23 +125,19 @@ export async function runBench(
     );
   }
 
-  const resolvedModel = model ?? (adapter.kind === 'builtin' ? adapter.id : 'default');
-  // A built-in harness scripts the workspace instead of driving an agent, so its
-  // numbers calibrate the grader and are never an arm's measurement (review 0003 Q49).
-  const measured = adapter.kind !== 'builtin';
+  const resolvedModel = model ?? (adapter.kind === 'oracle' ? adapter.id : 'default');
+  // An oracle harness scripts the workspace instead of driving an agent, so its numbers
+  // calibrate the grader and are never an arm's measurement (review 0003 Q49).
+  const measured = adapter.kind !== 'oracle';
   const isolation = adapterIsolation(adapter);
+  const experiment = await batchExperiment(root, { adapter, tasks, isolation, model: resolvedModel });
+  const timeoutForRun = timeoutMs ?? adapter.timeoutMs;
+
   const plannedRuns = [];
   for (const armSpec of armSpecs) {
     for (let rep = 1; rep <= reps; rep += 1) {
-      const runId = runIdFor(harness, armSpec.name, rep);
-      for (const task of tasks) {
-        plannedRuns.push({
-          runId,
-          armSpec,
-          rep,
-          task,
-        });
-      }
+      const runId = runIdFor({ harness, model: resolvedModel, arm: armSpec.name, rep });
+      for (const task of tasks) plannedRuns.push({ runId, armSpec, rep, task });
     }
   }
 
@@ -834,33 +147,32 @@ export async function runBench(
       root,
       harness,
       model: resolvedModel,
-      arms: armSpecs.map((a) => a.name),
+      arms: armSpecs.map((arm) => arm.name),
+      armDescriptions: armSpecs.map((arm) => `${arm.name}: ${describeArm(arm)}`),
       reps,
       taskCount: tasks.length,
-      runs: plannedRuns.map((item) => ({
-        schemaVersion: 1,
-        runId: item.runId,
-        task: item.task.id,
-        arm: item.armSpec.name,
-        harness,
-        model: resolvedModel,
-        rep: item.rep,
-        measured,
-        isolation,          status: 'planned',
-          resolved: null,
-          gatesPassed: null,
-          usage: emptyUsage().usage,
-        costUsd: null,
-        diff: { files: 0, outsideBlastRadius: 0 },
-        probe: null,
-      })),
+      experiment,
+      runs: plannedRuns.map((item) =>
+        runRecord({
+          task: item.task,
+          arm: item.armSpec.name,
+          harness,
+          model: resolvedModel,
+          rep: item.rep,
+          measured,
+          isolation,
+          experiment: { ...experiment, model: resolvedModel },
+          status: 'planned',
+          usage: { input: 0, cacheWrite: 0, cacheRead: 0, output: 0, turns: 0, toolCalls: 0 },
+        }),
+      ),
     };
   }
 
-  const runsBaseDir = join(root, LOCAL_DIR, 'bench', 'runs');
+  const baseRunsDir = runsBaseDir(root);
   for (const armSpec of armSpecs) {
     for (let rep = 1; rep <= reps; rep += 1) {
-      const runDir = join(runsBaseDir, runIdFor(harness, armSpec.name, rep));
+      const runDir = join(baseRunsDir, runIdFor({ harness, model: resolvedModel, arm: armSpec.name, rep }));
       await mkdir(runDir, { recursive: true });
       await writeFile(join(runDir, '.arm.json'), `${JSON.stringify(armSpec, null, 2)}\n`, 'utf8');
     }
@@ -868,19 +180,28 @@ export async function runBench(
 
   const regRes = await loadRegistry().catch(() => ({ registry: { skills: [] } }));
   const registrySkills = regRes?.registry?.skills ?? [];
-  const knownSkillNames = new Set(registrySkills.map((s) => s.name));
+  const knownSkillNames = new Set(registrySkills.map((skill) => skill.name));
   const parentDir = await mkdtemp(join(tmpdir(), 'bench-batch-'));
   try {
+    // One export per task, copied per run: the harness is what the batch measures, not
+    // the cost of unpacking the same commit once per arm and repetition.
+    const bases = new Map();
+    for (const task of tasks) {
+      if (bases.has(task.base)) continue;
+      try {
+        bases.set(task.base, await prepareBaseTree(root, task.base, parentDir));
+      } catch {
+        bases.set(task.base, null);
+      }
+    }
+
     const records = await mapPool(plannedRuns, concurrency, async (item) => {
       let slotDir = null;
       try {
-        const prepared = await prepareIsolatedWorkspace(
-          root,
-          item.task.base,
-          item.armSpec,
-          item.task,
-          { parentDir },
-        );
+        const prepared = await prepareIsolatedWorkspace(root, item.task.base, item.armSpec, item.task, {
+          parentDir,
+          baseDir: bases.get(item.task.base) ?? undefined,
+        });
         slotDir = prepared.slotDir;
         const { workDir, initialHead, env, promptPrefix } = prepared;
         const { usage, costUsd, transcript, agent } = await executeAdapter(adapter, workDir, item.task, {
@@ -888,33 +209,29 @@ export async function runBench(
           model: resolvedModel,
           maxTurns,
           env,
-          timeoutMs,
+          timeoutMs: timeoutForRun,
           slotDir,
           promptPrefix,
         });
 
-        const grade = await gradeWorkspace(workDir, item.task, {
-          vaultRoot,
-          initialHead,
-          timeoutMs,
-        });
+        const grade = await gradeWithOracle(workDir, item.task, { vaultRoot, initialHead, timeoutMs: timeoutForRun });
         const expectedSkills = matchSkillsForPaths(registrySkills, grade.goldFiles ?? []);
         const openedSkills =
-          adapter.id === 'gold'
-            ? expectedSkills
-            : adapter.id === 'noop'
-              ? []
-              : extractOpenedSkills(transcript, knownSkillNames);
-
-        const record = {
-          schemaVersion: 1,
-          task: item.task.id,
+          adapter.kind === 'oracle'
+            ? adapter.oracle?.from === 'vault'
+              ? expectedSkills
+              : []
+            : extractOpenedSkills(transcript, knownSkillNames);
+
+        const record = runRecord({
+          task: item.task,
           arm: item.armSpec.name,
           harness,
           model: resolvedModel,
           rep: item.rep,
           measured,
           isolation,
+          experiment: { ...experiment, model: resolvedModel },
           status: 'ok',
           agent,
           resolved: grade.resolved,
@@ -924,10 +241,9 @@ export async function runBench(
           costUsd,
           diff: grade.diff,
           skills: { expected: expectedSkills, opened: openedSkills },
-          probe: null,
-        };
+        });
 
-        const runDir = join(runsBaseDir, item.runId);
+        const runDir = join(baseRunsDir, item.runId);
         await Promise.all([
           writeFile(join(runDir, `${item.task.id}.json`), `${JSON.stringify(record, null, 2)}\n`, 'utf8'),
           writeFile(join(runDir, `${item.task.id}.patch`), grade.agentPatch),
@@ -938,29 +254,22 @@ export async function runBench(
         // One failed run is one exclusion, not a lost batch: the old path exited 3 and
         // left a half-written run behind (review 0003 Q47).
         return {
-          schemaVersion: 1,
+          ...runRecord({
+            task: item.task,
+            arm: item.armSpec.name,
+            harness,
+            model: resolvedModel,
+            rep: item.rep,
+            measured,
+            isolation,
+            experiment: { ...experiment, model: resolvedModel },
+            status: 'error',
+            error: String(error?.message ?? error),
+          }),
           runId: item.runId,
-          task: item.task.id,
-          arm: item.armSpec.name,
-          harness,
-          model: resolvedModel,
-          rep: item.rep,
-          measured,
-          isolation,
-          status: 'error',
-          error: String(error?.message ?? error),
-          resolved: false,
-          gatesPassed: false,
-          f2pPassed: [],
-          usage: emptyUsage().usage,
-          costUsd: null,
-          diff: { files: 0, outsideBlastRadius: 0 },
-          probe: null,
         };
       } finally {
-        if (slotDir !== null) {
-          await rm(slotDir, { recursive: true, force: true }).catch(() => {});
-        }
+        if (slotDir !== null) await rm(slotDir, { recursive: true, force: true }).catch(() => {});
       }
     });
 
@@ -969,9 +278,11 @@ export async function runBench(
       root,
       harness,
       model: resolvedModel,
-      arms: armSpecs.map((a) => a.name),
+      arms: armSpecs.map((arm) => arm.name),
+      armDescriptions: armSpecs.map((arm) => `${arm.name}: ${describeArm(arm)}`),
       reps,
       taskCount: tasks.length,
+      experiment,
       runs: records,
     };
   } finally {
@@ -979,49 +290,40 @@ export async function runBench(
   }
 }
 
-/** Two string lists hold the same members, ignoring order and duplicates. */
-function sameStringSet(a, b) {
-  const left = [...new Set(a)].sort();
-  const right = [...new Set(b)].sort();
-  return left.length === right.length && left.every((value, index) => value === right[index]);
-}
-
 async function resolveRegradeTargets(root, target) {
-  const runsBaseDir = join(root, LOCAL_DIR, 'bench', 'runs');
+  const base = runsBaseDir(root);
   const direct = resolve(root, target);
-  const candidate = existsSync(direct) ? direct : join(runsBaseDir, target);
+  const candidate = existsSync(direct) ? direct : join(base, target);
 
   if (target === 'all') {
-    const dirs = (await readdir(runsBaseDir, { withFileTypes: true }).catch(() => []))
-      .filter((d) => d.isDirectory())
-      .map((d) => join(runsBaseDir, d.name))
+    const dirs = (await readdir(base, { withFileTypes: true }).catch(() => []))
+      .filter((entry) => entry.isDirectory())
+      .map((entry) => join(base, entry.name))
       .sort();
     const files = [];
     for (const dir of dirs) {
       const entries = (await readdir(dir).catch(() => []))
-        .filter((f) => f.endsWith('.json') && !f.startsWith('.'))
+        .filter((file) => file.endsWith('.json') && !file.startsWith('.'))
         .sort();
       for (const entry of entries) files.push(join(dir, entry));
     }
     return files;
   }
 
-  const st = await stat(candidate).catch(() => null);
-  if (!st) {
-    throw new Error(`stored run not found: ${target}`);
-  }
-  if (st.isFile()) return [candidate];
-  if (st.isDirectory()) {
+  const targetStat = await stat(candidate).catch(() => null);
+  if (!targetStat) throw new Error(`stored run not found: ${target}`);
+  if (targetStat.isFile()) return [candidate];
+  if (targetStat.isDirectory()) {
     const entries = (await readdir(candidate).catch(() => []))
-      .filter((f) => f.endsWith('.json') && !f.startsWith('.'))
+      .filter((file) => file.endsWith('.json') && !file.startsWith('.'))
       .sort();
-    return entries.map((f) => join(candidate, f));
+    return entries.map((file) => join(candidate, file));
   }
   return [];
 }
 
 /**
- * Re-grade a stored run (or run directory) deterministically from its saved agent patch
+ * Re-grade a stored run (or run directory) deterministically from its saved agent patch,
  * and verify that the reproduced grade matches the stored record.
  *
  * @param {string} root
@@ -1039,28 +341,24 @@ export async function regradeRun(
   } = {},
 ) {
   const files = await resolveRegradeTargets(root, target);
-  if (files.length === 0) {
-    throw new Error(`no stored task runs found under ${target}`);
-  }
+  if (files.length === 0) throw new Error(`no stored task runs found under ${target}`);
 
-  const tasksDir = join(root, LOCAL_DIR, 'bench', 'runs');
-  const taskDefsDir = join(root, STATE_DIR, 'bench', 'tasks');
+  const base = runsBaseDir(root);
   const parentDir = await mkdtemp(join(tmpdir(), 'bench-regrade-'));
 
   try {
     const results = await mapPool(files, concurrency, async (runFilePath) => {
       const stored = JSON.parse(await readFile(runFilePath, 'utf8'));
       const runDir = dirname(runFilePath);
-      const runId = runDir.startsWith(tasksDir)
-        ? runDir.slice(tasksDir.length + 1)
-        : dirname(runFilePath).split(sep).pop();
+      const runId = runDir.startsWith(base) ? runDir.slice(base.length + 1) : runDir.split(sep).pop();
 
-      const taskPath = join(taskDefsDir, `${stored.task}.json`);
-      const task = JSON.parse(await readFile(taskPath, 'utf8'));
+      const task = await readTask(root, stored.task);
+      if (!task) throw new Error(`task descriptor missing for ${stored.task}`);
+      if (!task.base) throw new Error(`task ${stored.task} names no base commit`);
 
       const armMetaPath = join(runDir, '.arm.json');
       const armSpec = await readFile(armMetaPath, 'utf8')
-        .then((s) => JSON.parse(s))
+        .then((text) => JSON.parse(text))
         .catch(() => ({
           name: stored.arm,
           kind: BUILTIN_ARMS.has(stored.arm) ? 'builtin' : 'custom',
@@ -1073,24 +371,16 @@ export async function regradeRun(
       const patchPath = runFilePath.replace(/\.json$/, '.patch');
       const agentPatch = await readFile(patchPath).catch(() => Buffer.alloc(0));
 
-      const { slotDir, workDir, initialHead } = await prepareIsolatedWorkspace(
-        root,
-        task.base,
-        armSpec,
-        task,
-        { parentDir },
-      );
+      const { slotDir, workDir, initialHead } = await prepareIsolatedWorkspace(root, task.base, armSpec, task, {
+        parentDir,
+      });
       try {
         await applyPatch(workDir, agentPatch, { timeoutMs });
 
-        const grade = await gradeWorkspace(workDir, task, {
-          vaultRoot,
-          initialHead,
-          timeoutMs,
-        });
+        const grade = await gradeWithOracle(workDir, task, { vaultRoot, initialHead, timeoutMs });
 
-        // The verdicts alone cannot tell "the same tests passed" from "the same number
-        // of tests passed"; the fail-to-pass set is compared too (review 0003 Q59).
+        // The verdicts alone cannot tell "the same tests passed" from "the same number of
+        // tests passed"; the fail-to-pass set is compared too (review 0003 Q59).
         const reproduced =
           grade.resolved === stored.resolved &&
           grade.gatesPassed === stored.gatesPassed &&
@@ -1106,15 +396,8 @@ export async function regradeRun(
           diff: grade.diff,
         };
 
-        if (apply) {
-          await writeFile(runFilePath, `${JSON.stringify(updated, null, 2)}\n`, 'utf8');
-        }
-
-        return {
-          ...updated,
-          runId,
-          reproduced,
-        };
+        if (apply) await writeFile(runFilePath, `${JSON.stringify(updated, null, 2)}\n`, 'utf8');
+        return { ...updated, runId, reproduced };
       } finally {
         await rm(slotDir, { recursive: true, force: true }).catch(() => {});
       }
@@ -1125,8 +408,8 @@ export async function regradeRun(
       root,
       target,
       regraded: results.length,
-      reproducedCount: results.filter((r) => r.reproduced).length,
-      allReproduced: results.every((r) => r.reproduced),
+      reproducedCount: results.filter((result) => result.reproduced).length,
+      allReproduced: results.every((result) => result.reproduced),
       runs: results,
     };
   } finally {