base-commit pinning: baseCommit dispatch field split from ref, branch-o…

[warren-forge-bnnl24]

Summary

feat(runs): base-commit pinning — baseCommit dispatch field split from ref (warren-aaf7)

Agent notes

Split runs.ref's two meanings: ref stays branch-shaped (workspace base for
branch dispatches AND the reap PR base — byte-identical semantics for every
existing ref-dispatch), while a new optional baseCommit (exact 40-hex SHA)
pins the workspace cut point only, never the PR base.

• runs.base_commit column, migrations both dialects (0051 sqlite / 0046 pg)
• spawnRun: baseRef = baseCommit ?? continuation/ref resolution; baseCommit
  frozen on the row and echoed by the projections
• HTTP boundary validation (src/runs/base-commit.ts): ref must pass
  isValidBranchRef; a 40-hex ref is a 400 pointing at baseCommit; baseCommit
  must be exactly 40-hex
• pushCloneDeltasToOrigin skips a SHA ref with a logged reason instead of
  burning a clone_apply_push failure (legacy SHA-ref rows)
• infra-lost retry carries baseCommit forward with ref
• SDK CreateRunInput/DispatchRunInput/RunRow, UI types, warren run
  --base-commit; goldens + cli-reference regenerated

Run

• Warren run: run_9qf0tejnzy6a
• Agent: pi
• Cost: $1.39 (97.5k in / 27.1k out / 11.5M cache-r)

Seeds

• warren-aaf7 — base-commit pinning: baseCommit dispatch field split from ref, branch-or-SHA validation at the HTTP boundary, PR base stays branch-shaped

Commits (1)

• 24b4d6d feat(runs): base-commit pinning — baseCommit dispatch field split from ref (warren-aaf7)

Files changed

.mulch/expertise/tracker.jsonl                     |    1 -
 .mulch/mulch.config.yaml                           |    1 -
 docs/cli-reference.md                              |    1 +
 scripts/check-wire-types.ts                        |    8 -
 .../commands/__golden__/prime/prime-document.json  |    4 +
 .../commands/__golden__/prime/prime-pretty.json    |    2 +-
 src/cli/commands/plan-status.test.ts               |    1 +
 src/cli/commands/run.test.ts                       |    1 +
 src/cli/commands/run.ts                            |    3 +
 src/cli/commands/show.test.ts                      |    1 +
 src/cli/commands/wait.test.ts                      |    1 +
 src/cli/main.ts                                    |    5 +-
 src/client/client.ts                               |    1 +
 src/client/types.ts                                |    6 +
 src/core/wire-tracker.test.ts                      |   49 -
 src/core/wire-tracker.ts                           |  156 --
 src/core/wire.ts                                   |    7 +-
 src/db/migrations/0051_massive_slapstick.sql       |    1 +
 src/db/migrations/meta/0051_snapshot.json          | 1420 ++++++++++++++++++
 src/db/migrations/meta/_journal.json               |    7 +
 .../postgres/0046_normal_imperial_guard.sql        |    1 +
 src/db/migrations/postgres/meta/0046_snapshot.json | 1550 ++++++++++++++++++++
 src/db/migrations/postgres/meta/_journal.json      |    7 +
 src/db/repos/runs.ts                               |    3 +
 src/db/schema/postgres.ts                          |    2 +
 src/db/schema/sqlite.ts                            |    4 +
 src/runs/base-commit.test.ts                       |   54 +
 src/runs/base-commit.ts                            |   67 +
 src/runs/reap/clone-apply.test.ts                  |   21 +
 src/runs/reap/clone-apply.ts                       |   14 +
 src/runs/retry/infra-lost-retry.ts                 |    1 +
 src/runs/spawn/base-commit.test.ts                 |   97 ++
 src/runs/spawn/dispatch.ts                         |    5 +-
 src/runs/spawn/types.ts                            |    7 +
 src/seeds-cli/schema.ts                            |   17 +-
 src/seeds-cli/show.test.ts                         |   12 -
 src/seeds-cli/show.ts                              |   21 +-
 .../handlers/__golden__/envelopes/run-detail.json  |    1 +
 .../handlers/runs.dispatch.base-commit.test.ts     |  188 +++
 src/server/handlers/runs/dispatch.ts               |    9 +-
 src/server/handlers/runs/lifecycle.ts              |    3 +
 src/tracker/contract.ts                            |  128 --
 src/tracker/seeds-tracker.test.ts                  |  270 ----
 src/tracker/seeds-tracker.ts                       |  244 ---
 src/ui/src/api/types.ts                            |   37 +-
 src/ui/tsconfig.app.json                           |    1 -
 46 files changed, 3523 insertions(+), 917 deletions(-)

Prompt

Show prompt

Work seeds issue warren-aaf7. First run `sd show warren-aaf7 --json` from the repo root — the issue body is the full spec; this summary is only a digest.

Context: step 15 of plan pl-a37b (v0.18.0), Track C (external-repo readiness). Track A (dispatch-context log) is merged on main. Track B (IssueTracker cut) is being worked in parallel on other branches — do not touch the seeds-cli/tracker wiring.

Scope: base-commit pinning. Split a `baseCommit` dispatch field from `ref` on the dispatch input (POST /runs and spawnRun), with branch-or-SHA validation at the HTTP boundary. A run dispatched with baseCommit=<sha> materializes the workspace at that commit; the PR base must stay branch-shaped (a SHA base → GitHub 422). CRITICAL regression risk: the reap PR-base resolution (src/runs/reap/run.ts around line 132) — the split must keep ref semantics byte-identical for branch refs, or every ref-dispatch (pr-fixer, conflict repair) breaks. Regenerate docs after route/OpenAPI changes: `bun run gen:docs && bun run gen:openapi` (and gen:cli-ref if the CLI gains a flag).

Quality gates are terminal: `bun run check:all` must be green (12/12) before you commit and report done.

Close the issue with `sd close warren-aaf7 --reason "<what you shipped>"`, then commit everything.

---

🤖 Opened by warren run run_9qf0tejnzy6a