plumbbob 0.9.5 → 0.10.1

package/dist/verbs/agent.js

@@ -1,17 +1,17 @@
-// `plumbbob agent <subcommand>` — the doorway to user-authored agents: anything
+// `plumbbob agent <subcommand>`: the doorway to user-authored agents: anything
 // executable that speaks the JSON envelope contract (JSON in on stdin, JSON out
-// on stdout, prose streamed on stderr). `agent list` walks the two agent tiers —
-// the repo's tracked `.plumbbob/agents/<name>/`, then the personal
-// `~/.plumbbob/agents/<name>/` — and prints each resolvable agent. `agent run
+// on stdout, prose streamed on stderr). `agent list` walks the two agent tiers
+// (the repo's tracked `.plumbbob/agents/<name>/`, then the personal
+// `~/.plumbbob/agents/<name>/`) and prints each resolvable agent. `agent run
 // <name> [--step N] [--mode before|build|after]` composes the StepContext,
 // spawns the manifest command, streams its stderr live, captures and validates
 // the child's envelope, re-emits it on this verb's own stdout (machine) with the
 // human summary on stderr, lands `parked[]` through the build-log, and appends
-// the envelope to the handoff ledger (`builds/<slug>/handoff.json` — untracked,
+// the envelope to the handoff ledger (`builds/<slug>/handoff.json`: untracked,
 // step-scoped, cleared at checkpoint) so later runs can thread it back in.
-// There is deliberately no code path here to checkpoint, flip a step, or chain
-// agents — the subprocess boundary keeps the human as the clock by
-// construction, not by policy. A thin read-write shell: resolution,
+// No code path here can checkpoint, flip a step, or chain agents, and that
+// absence is deliberate; the subprocess boundary keeps the human as the clock
+// by construction, not by policy. A thin read-write shell: resolution,
 // composition, and spawn mechanics live in lib/agents.ts.
 import { readFileSync, writeFileSync } from 'node:fs';
 import { join } from 'node:path';
@@ -50,10 +50,10 @@ function list(cwd, _args) {
  * Run one named agent, or a slot's harness-bound agents, against the step in flight.
  *
  * With a name (or `--agent` flag) it runs exactly that agent, failing loud on a
- * miss — the user who typed the name asked for it specifically. With no name it
+ * miss; the user who typed the name asked for it specifically. With no name it
  * runs whatever the build's harness.json binds to the requested slot. Either
- * way it composes the StepContext, spawns, and applies side effects — never
- * advancing the loop.
+ * way it composes the StepContext, spawns, and applies side effects (never
+ * advancing the loop).
  */
 async function run(cwd, args) {
     const root = findRepoRoot(cwd);
@@ -72,7 +72,7 @@ async function run(cwd, args) {
         process.stderr.write('plumbbob: no step to run against — pass --step N, or `plumbbob build N` first.\n');
         return 1;
     }
-    // A `--agent <path>` flag still needs a name — it labels an explicit run (the
+    // A `--agent <path>` flag still needs a name; it labels an explicit run (the
     // handoff ledger and the human summary key on the name), it does not name one.
     if (parsed.name === undefined && parsed.flagPath !== undefined) {
         process.stderr.write('plumbbob: --agent needs an agent name too, for the run label. Try: plumbbob agent run reviewer --agent ./path --step 3.\n');
@@ -94,10 +94,10 @@ async function run(cwd, args) {
  * One agent's full run: resolve it, pick its slot, compose the StepContext, spawn, report.
  *
  * `ambient` marks a harness-bound run whose resolution or slot mismatch
- * degrades to a warning so a batch keeps going — a binding is ambient
+ * degrades to a warning so a batch keeps going; a binding is ambient
  * configuration the loop must survive without; an explicit ask
  * (`ambient: false`) fails loud on the same miss. A run that actually starts
- * and fails (non-zero exit, timeout, …) is a hard failure either way — the
+ * and fails (non-zero exit, timeout, …) is a hard failure either way: the
  * softening covers a *missing* agent, never a broken one.
  */
 async function runOne(root, slug, step, spec) {
@@ -126,17 +126,17 @@ async function runOne(root, slug, step, spec) {
         mode,
         settings: {
             // The envelope reports the `auto` setting to agents as information only.
-            // The checkpoint latch never reads it — a model can write a settings
+            // The checkpoint latch never reads it; a model can write a settings
             // file, so a standing `auto` cannot be a self-approval grant; approval
             // comes only from what the human literally typed. A `true` here informs
             // an agent, it does not self-approve.
             auto: resolveBoolean(root, 'auto', false),
             agentTimeout: resolveNumber(root, 'agentTimeout', 0),
             // Hand this agent its own config block over the envelope's existing
-            // `settings` field — no new envelope field, no new verb; the config just
+            // `settings` field: no new envelope field, no new verb; the config just
             // rides here. Resolution: settings.json's agentConfig[name], with the
             // untracked personal overlay (settings.local.json) replacing the project
-            // entry whole — no deep merge — and {} when neither defines it.
+            // entry whole (no deep merge) and {} when neither defines it.
             agent: resolveRecord(root, 'agentConfig')[spec.name] ?? {},
         },
     });
@@ -159,7 +159,7 @@ async function runOne(root, slug, step, spec) {
 /**
  * No name given: run the agents the build's harness.json binds to the requested slot.
  *
- * Bindings merge as a ladder — the per-step harness entry beats the harness
+ * Bindings merge as a ladder: the per-step harness entry beats the harness
  * `defaults`, which beat the settings-level `agents` key; the first level that
  * names the slot wins, replace not append. A missing bound agent degrades to a
  * warning; an absent harness, or one that binds nothing to this slot, is a
@@ -216,7 +216,7 @@ function degrade(ambient, hard, soft) {
  *
  * A value flag missing its value, or `--step` given a non-number, is a loud
  * error (returned as a string) rather than a silent default. An undeclared
- * `--flag` never arrives here — `run` screens argv against the verb spec in
+ * `--flag` never arrives here; `run` screens argv against the verb spec in
  * cli-core.ts and refuses before dispatch.
  */
 function parseRunArgs(args) {
@@ -254,7 +254,7 @@ function parseRunArgs(args) {
 /**
  * Resolve the slot the agent runs in.
  *
- * An explicit `--mode` must name a real slot AND one the manifest declares —
+ * An explicit `--mode` must name a real slot AND one the manifest declares;
  * an undeclared slot is refused loud, because the user asked for that exact
  * run. With no `--mode`, a single-slot agent uses its only slot; a multi-slot
  * agent must be told which one (harness bindings pick the slot from the
@@ -282,11 +282,11 @@ function resolveMode(flag, manifest) {
 /**
  * Map the run outcome to reporting and side effects.
  *
- * A failed run — non-zero exit, out-of-contract stdout, timeout, interrupt, or a
- * shell that never started — reports on stderr and stops with NO side effects:
+ * A failed run (non-zero exit, out-of-contract stdout, timeout, interrupt, or a
+ * shell that never started) reports on stderr and stops with NO side effects:
  * the envelope of a failed child is not authoritative. A clean run lands
  * `parked[]`, records the envelope in the handoff ledger, prints the human
- * summary on stderr, and re-emits the machine envelope on stdout — nothing else,
+ * summary on stderr, and re-emits the machine envelope on stdout, nothing else,
  * ever (the stream discipline: stdout carries the envelope alone).
  */
 function report(root, slug, name, mode, step, result) {
@@ -321,7 +321,7 @@ function failureLine(name, result) {
 /**
  * Land each parked concern through the build-log's Park list.
  *
- * The agent never writes .plumbbob/ itself — parked lines only reach the
+ * The agent never writes .plumbbob/ itself; parked lines only reach the
  * build-log through this verb. A build-log with no "## Park list" section, or no
  * build-log at all, warns once rather than losing the lines silently.
  */
@@ -351,7 +351,7 @@ function applyParked(root, slug, parked) {
 /**
  * The human-facing summary printed on stderr.
  *
- * A headline plus, for a halt, the route out — `blocked` unblocks and re-runs,
+ * A headline plus, for a halt, the route out: `blocked` unblocks and re-runs,
  * `drift` sends the plan to /plumbbob:refine. The machine envelope on stdout carries
  * the same status for the calling skill; this stderr copy is the terminal read.
  */

package/dist/verbs/build.js

@@ -1,9 +1,9 @@
-// `plumbbob build <n>` — enter a step: read step n's seam from intent.md and
+// `plumbbob build <n>`: enter a step: read step n's seam from intent.md and
 // write the normalized SEAM + STEP control files (flat, untracked, per-build).
-// The seam is the step's edit grant — the exact paths and `dir/` prefixes the
+// The seam is the step's edit grant: the exact paths and `dir/` prefixes the
 // step expects to touch, for orientation, never a lock. The STEP file is the
 // in-flight signal (the dashboard derives the BUILD phase from it); this verb
-// never checkpoints — only `checkpoint` commits.
+// never checkpoints; only `checkpoint` commits.
 import { readFileSync, writeFileSync } from 'node:fs';
 import { findRepoRoot } from "../lib/git.js";
 import { hasSession, intentPath, resolveBuild, seamPath, stampStepStat, stampTick, stepPath } from "../lib/sidecar.js";
@@ -11,7 +11,7 @@ import { parseStepSeam } from "../lib/intent.js";
 import { parseSteps } from "../lib/orient.js";
 import { stepLabel, syncBuildLogState } from "../lib/buildlogsync.js";
 /**
- * Enter step n — or the next undone step — and write its SEAM/STEP markers.
+ * Enter step n (or the next undone step) and write its SEAM/STEP markers.
  *
  * Refuses a missing session, a malformed step argument, an `N-M` range (a
  * `/plumbbob:build` skill feature, not a CLI one), and a seam that fails to parse.
@@ -27,13 +27,13 @@ export function build(cwd, args) {
     const { build: slug, rest } = resolveBuild(root, args);
     const raw = rest.find((a) => !a.startsWith('--'));
     // A step range like `1-3` is a `/plumbbob:build` skill affordance (auto-approve
-    // through the range, then pause), not a CLI capability — the CLI records one
+    // through the range, then pause), not a CLI capability: the CLI records one
     // in-flight step at a time. Name it rather than bounce off the generic usage.
     if (raw !== undefined && /^\d+-\d*$/.test(raw)) {
         process.stderr.write(`plumbbob: build takes one step number; \`${raw}\` step ranges are a \`/plumbbob:build\` feature (auto-approve through the range, then pause). Try \`plumbbob build ${raw.split('-')[0]}\`.\n`);
         return 1;
     }
-    // An explicit arg that isn't a positive integer is a usage error — caught before
+    // An explicit arg that isn't a positive integer is a usage error: caught before
     // reading intent.md so the message doesn't depend on the plan being present.
     if (raw !== undefined && (!/^\d+$/.test(raw) || Number(raw) < 1)) {
         process.stderr.write('plumbbob: build needs a step number. Try: plumbbob build 2.\n');
@@ -65,14 +65,14 @@ export function build(cwd, args) {
     writeFileSync(stepPath(root, slug), `${step}\n`);
     // Stamp the turn ledger on entry (TICK = TURN): the checkpoint latch demands
     // a human turn after this point before the step may land. Skipped when the
-    // ledger is dormant — a host with no hooks grows no TURN/TICK files.
+    // ledger is dormant: a host with no hooks grows no TURN/TICK files.
     stampTick(root, slug);
     // The wall-clock receipt starts here: checkpoint stamps landedAt, and the
     // pair becomes the step's duration in the finish report.
     stampStepStat(root, slug, step, 'startedAt', new Date().toISOString());
     // The build-log's top half is CLI-owned so it never lies: show this step in
-    // flight — Current step plus the ☐/☑ mirror, re-rendered from intent.md.
-    // Best-effort — a missing or hand-edited build-log never blocks the build.
+    // flight: Current step plus the ☐/☑ mirror, re-rendered from intent.md.
+    // Best-effort: a missing or hand-edited build-log never blocks the build.
     const title = parseSteps(intent).find((s) => s.n === step)?.title ?? null;
     syncBuildLogState(root, slug, stepLabel(step, title));
     const picked = raw === undefined ? ' (next undone)' : '';

package/dist/verbs/check.js

@@ -1,14 +1,14 @@
-// `plumbbob check` — run the heavy gate and report, with NO state change. This
+// `plumbbob check`: run the heavy gate and report, with NO state change. This
 // is the read-only half of the verify tick: `/plumbbob:verify` runs it before the
 // pause so the human approves on a known-green check. The gate itself lives in
 // lib/check.ts: checkride (our sibling check-runner package) by default, or a
-// spawn command named by the `check` setting — resolved through the settings
-// ladder (flag → local overlay → tracked settings.json → default) — for repos
+// spawn command named by the `check` setting (resolved through the settings
+// ladder (flag → local overlay → tracked settings.json → default)) for repos
 // that gate through something else. Exits with the check's own code (0 = green,
 // 1 = red, 2 = the gate itself broke).
 //
 // Narrowing flags for the iteration loop (`check --bail --only types,lint`) map
-// straight onto checkride's run flags. Only this verb takes them — the
+// straight onto checkride's run flags. Only this verb takes them: the
 // checkpoint gate is always full-fat.
 import { findRepoRoot } from "../lib/git.js";
 import { hasSession } from "../lib/sidecar.js";
@@ -35,7 +35,7 @@ export async function check(cwd, args = []) {
  * Only the declared flags are read here; an undeclared one never reaches this
  * function, because `run` screens argv against the verb spec in cli-core.ts and
  * refuses before dispatch. This used to tolerate unknown args on the grounds
- * that running the gate mattered more than flag hygiene — but the same tolerance
+ * that running the gate mattered more than flag hygiene, but the same tolerance
  * on the mutating verbs let `checkpoint --help` commit, so the screen is now
  * central and uniform.
  */
@@ -69,7 +69,7 @@ function slotList(args, flag) {
 /**
  * The one-line human verdict for a check exit code.
  *
- * A broken gate (exit 2) reports distinctly from red — a misconfigured harness
+ * A broken gate (exit 2) reports distinctly from red: a misconfigured harness
  * must never read as broken code, though both block a checkpoint.
  */
 function verdictLine(code) {

package/dist/verbs/checkpoint.js

@@ -1,18 +1,19 @@
-// `plumbbob checkpoint [<n>] [-m <msg>]` — the commit tick that lands a step.
+// `plumbbob checkpoint [<n>] [-m <msg>]`: the commit tick that lands a step.
 // Executor-agnostic and author-blind: it reads the diff, not who wrote it, so a
 // `/plumbbob:build` run, your own hands, a vibe session, or another harness all
 // checkpoint identically. It does NOT require a STEP marker (the flat control
 // file recording the step in flight): the step is whatever you pass, else the
 // in-flight STEP, else the next undone step in intent.md. It evaluates the
 // approval latch, gates on a green check, commits any pending work (or records
-// the existing HEAD when the tree is already clean — the human's commit skill
+// the existing HEAD when the tree is already clean: the human's commit skill
 // may have committed first), records the SHA, flips the intent checkbox to
-// `[x]`, and clears the STEP/SEAM markers — returning the dashboard to the
+// `[x]`, and clears the STEP/SEAM markers: returning the dashboard to the
 // DESIGN boundary.
 import { appendFileSync, readFileSync, rmSync, writeFileSync } from 'node:fs';
 import { commit, findRepoRoot, headSha, isDirty, stageAll, stagePath, stagedPaths, stagedStat } from "../lib/git.js";
 import { activeBuild, buildFolder, buildLogPath, buildScope, bumpStepStat, checkpointsPath, clearHandoff, clearTick, hasSession, intentPath, readStats, refreshExcludes, seamPath, stampStepStat, stepPath, } from "../lib/sidecar.js";
 import { runCheck } from "../lib/check.js";
+import { readCommitBody } from "../lib/commitbody.js";
 import { checkLatch } from "../lib/latch.js";
 import { markStepDone, parseSteps } from "../lib/orient.js";
 import { parseBuildScope, parseStepSeam, scopeDrift } from "../lib/intent.js";
@@ -22,7 +23,7 @@ import { AT_BOUNDARY, syncBuildLogState } from "../lib/buildlogsync.js";
 /**
  * Land a step: latch, check gate, commit, record, return to the boundary.
  *
- * Exit 1 on any refusal — no session, no resolvable step, a latched tick, or a
+ * Exit 1 on any refusal: no session, no resolvable step, a latched tick, or a
  * red/broken check; each refusal says what unblocks it.
  */
 export async function checkpoint(cwd, args) {
@@ -31,7 +32,7 @@ export async function checkpoint(cwd, args) {
         process.stderr.write('plumbbob: no active session. Run `plumbbob start "<title>"` first.\n');
         return 1;
     }
-    // Before either staging path — `-A` below, and the plan commit's folder-scoped
+    // Before either staging path: `-A` below, and the plan commit's folder-scoped
     // add, which also leans on the excludes to skip the in-flight markers inside
     // the build folder.
     refreshExcludes(root);
@@ -43,10 +44,18 @@ export async function checkpoint(cwd, args) {
         process.stderr.write('plumbbob: no step to checkpoint — pass a number, or plan a step in intent.md first.\n');
         return 1;
     }
-    // The approval latch runs before the check gate — cheap first. The tick may
+    // Checked before the latch and the check gate: a socket stdin can never
+    // deliver the requested body, so there is no point latching or running a
+    // ~55s check only to discover that afterward.
+    const bodyResult = readCommitBody(args);
+    if (!bodyResult.ok) {
+        process.stderr.write(bodyResult.message);
+        return 1;
+    }
+    // The approval latch runs before the check gate: cheap first. The tick may
     // not land without a human turn since the step was entered, a one-turn grant
     // the human typed, or a dormant ledger. A refusal is not an error: the
-    // message is the pause affordance — exit 1 and hand the turn back.
+    // message is the pause affordance: exit 1 and hand the turn back.
     const latch = checkLatch(root, step);
     if (!latch.allow) {
         process.stderr.write(latch.message);
@@ -69,7 +78,7 @@ export async function checkpoint(cwd, args) {
     if (isDirty(root)) {
         stageAll(root);
         warnScopeDrift(root, step);
-        const body = withMarker(`plumbbob step ${step}`, bodyArg(args) ?? fallbackBody(root, step));
+        const body = withMarker(`plumbbob step ${step}`, bodyResult.body ?? fallbackBody(root, step));
         sha = commit(root, messageArg(args) ?? subjectForStep(root, step), body);
     }
     else {
@@ -86,8 +95,8 @@ export async function checkpoint(cwd, args) {
     syncBuildLogState(root, activeBuild(root), AT_BOUNDARY);
     rmSync(seamPath(root), { force: true });
     rmSync(stepPath(root), { force: true });
-    clearHandoff(root); // the agent-run handoff ledger is step-scoped — clear it with the markers.
-    clearTick(root); // the step's entry stamp is spent — the next `build <n>` re-stamps.
+    clearHandoff(root); // the agent-run handoff ledger is step-scoped; clear it with the markers.
+    clearTick(root); // the step's entry stamp is spent; the next `build <n>` re-stamps.
     process.stdout.write(`plumbbob: step ${step} checkpointed — ${sha.slice(0, 9)}. Back at the boundary.\n`);
     return 0;
 }
@@ -97,33 +106,46 @@ export async function checkpoint(cwd, args) {
  *
  * Giving the plan its own commit keeps the first step's diff from absorbing the
  * scaffold, so `git log` reads baseline → plan → steps. No check gate (there is
- * no code work to verify yet), no intent flip, no step markers — the plan lives
+ * no code work to verify yet), no intent flip, no step markers: the plan lives
  * entirely in DESIGN. An optional `--body` (stdin heredoc) rides after the
  * `plumbbob plan` marker line; the folder is whitelisted artifact plane, so
  * there is no scope drift to warn about.
  */
 function checkpointPlan(root, args) {
     // The latch covers the plan commit too, keyed on the TICK that `start`
-    // stamped: the plan pause is a real pause. No step number — a step-range
+    // stamped: the plan pause is a real pause. No step number: a step-range
     // grant does not speak to a plan.
     const latch = checkLatch(root, null);
     if (!latch.allow) {
         process.stderr.write(latch.message);
         return 1;
     }
-    stagePath(root, buildFolder(root));
-    const sha = commit(root, planSubject(root), withMarker('plumbbob plan', bodyArg(args) ?? undefined));
+    const bodyResult = readCommitBody(args);
+    if (!bodyResult.ok) {
+        process.stderr.write(bodyResult.message);
+        return 1;
+    }
+    // A repo that gitignores the sidecar (its `.gitignore`, not plumbbob's own
+    // info/exclude) makes this folder-scoped add impossible; stagePath skips it
+    // and reports false, and the plan still lands as a valid record-only commit:
+    // its body carries the plan, the files stay untracked, and `--allow-empty`
+    // (in commit) makes the empty diff legal. History still reads baseline → plan
+    // → steps.
+    const staged = stagePath(root, buildFolder(root));
+    const sha = commit(root, planSubject(root), withMarker('plumbbob plan', bodyResult.body ?? undefined));
     appendFileSync(checkpointsPath(root), `plan ${sha}\n`);
     // Landing the plan consumes `start`'s entry stamp: a later hand-built diff
     // (no `build <n>`) must find no stale TICK and stay guidance-governed.
     clearTick(root);
-    process.stdout.write(`plumbbob: plan committed — ${sha.slice(0, 9)}. Baseline → plan → steps.\n`);
+    process.stdout.write(staged
+        ? `plumbbob: plan committed — ${sha.slice(0, 9)}. Baseline → plan → steps.\n`
+        : `plumbbob: plan committed (record-only — sidecar is gitignored; the plan rides the commit message, the files stay untracked) — ${sha.slice(0, 9)}. Baseline → plan → steps.\n`);
     return 0;
 }
 /**
  * Compose the plan commit's CLI-owned subject: `chore(<scope>): plan`.
  *
- * The scope resolves through the build-default fallback chain — a bare
+ * The scope resolves through the build-default fallback chain: a bare
  * `chore: plan` when none resolves. The `plumbbob plan` identifier rides the
  * body marker line, never the subject.
  */
@@ -134,7 +156,7 @@ function planSubject(root) {
  * Resolve the step being checkpointed, or null when none can be determined.
  *
  * Explicit arg > in-flight STEP file > first undone step in intent.md. A `-m`
- * value is a message, never a step — `checkpoint -m "2"` must not read as
+ * value is a message, never a step: `checkpoint -m "2"` must not read as
  * step 2.
  */
 function resolveStep(root, args) {
@@ -154,10 +176,10 @@ function resolveStep(root, args) {
     }
 }
 /**
- * Warn — never refuse — when the staged tree reaches beyond the step's seam.
+ * Warn (never refuse) when the staged tree reaches beyond the step's seam.
  *
  * Guidance, not a gate: the checkpoint captures the drift and says so. The seam
- * (the step's edit grant — exact paths or `dir/` prefixes) comes from the
+ * (the step's edit grant: exact paths or `dir/` prefixes) comes from the
  * in-flight SEAM file when a build is live, else the step's declared seam in
  * intent.md. Plumbbob's own artifact plane is whitelisted inside `scopeDrift`,
  * so the `[x]` flip and build-log line this very checkpoint stages never read
@@ -177,7 +199,7 @@ function warnScopeDrift(root, step) {
  *
  * The normalized SEAM file `build` wrote is authoritative while a build is
  * live; fall back to the step's declared seam parsed from intent.md. Empty when
- * neither resolves — the caller then skips the drift warning rather than
+ * neither resolves: the caller then skips the drift warning rather than
  * flagging the whole tree.
  */
 function seamTokens(root, step) {
@@ -191,14 +213,14 @@ function seamTokens(root, step) {
         }
     }
     catch {
-        // no SEAM file — fall through to the declared seam.
+        // no SEAM file: fall through to the declared seam.
     }
     return seamForStep(root, step);
 }
 /**
  * Flip the step's checkbox in intent.md to `[x]`.
  *
- * Best-effort bookkeeping — the checkpoint SHA is the source of truth — but the
+ * Best-effort bookkeeping (the checkpoint SHA is the source of truth) but the
  * dashboard reads intent.md, so a swallowed failure here would make orientation
  * lie; the catch warns and asks for a hand flip instead.
  */
@@ -216,7 +238,7 @@ function flipIntent(root, step) {
  *
  * The build's history accrues at each checkpoint instead of being reconstructed
  * at finish; the step's title is lifted from intent.md when still present.
- * Best-effort: a missing or odd build-log never blocks a checkpoint — the
+ * Best-effort: a missing or odd build-log never blocks a checkpoint; the
  * `checkpoints` SHA is the source of truth.
  */
 function logCheckpoint(root, step, sha) {
@@ -236,7 +258,7 @@ function logCheckpoint(root, step, sha) {
 /**
  * The compact stats receipt riding the Log line, or null when nothing accrued.
  *
- * Only what happened is shown — a clean first-try step gets no suffix at all.
+ * Only what happened is shown: a clean first-try step gets no suffix at all.
  * Wall-clock needs both stamps: a hand-built step never ran `build <n>`, so it
  * has no startedAt.
  */
@@ -280,7 +302,7 @@ function subjectForStep(root, step) {
  * The build-default Conventional scope, or null for a bare subject.
  *
  * The `**Scope:**` header field in intent.md wins when authored and filled;
- * else the build slug with its date prefix stripped; else null — a build with
+ * else the build slug with its date prefix stripped; else null: a build with
  * neither field keeps producing bare subjects.
  */
 function buildDefaultScope(root) {
@@ -291,7 +313,7 @@ function buildDefaultScope(root) {
         }
     }
     catch {
-        // no intent.md yet — fall through to the slug rung.
+        // no intent.md yet: fall through to the slug rung.
     }
     return buildScope(activeBuild(root));
 }
@@ -313,27 +335,6 @@ function messageArg(args) {
     const i = args.indexOf('-m');
     return i !== -1 && i + 1 < args.length ? (args[i + 1] ?? null) : null;
 }
-/**
- * Read the `--body` commit body from stdin, or null when absent or empty.
- *
- * The stdin heredoc lets a skill compose proportional prose the CLI never
- * could; null either way hands over to the deterministic fallback body. Reading
- * fd 0 blocks until EOF, which the heredoc supplies; a read error (no stdin
- * attached) degrades to the fallback, and an interactive TTY — which would
- * never send EOF — skips the read instead of hanging.
- */
-function bodyArg(args) {
-    if (!args.includes('--body') || process.stdin.isTTY === true) {
-        return null;
-    }
-    try {
-        const raw = readFileSync(0, 'utf8').trimEnd();
-        return raw.length > 0 ? raw : null;
-    }
-    catch {
-        return null;
-    }
-}
 /**
  * The deterministic fallback commit body: the step's done-when, its seam, and
  * the staged diffstat.