@eir-labs/coltrane 0.16.0 → 0.17.0

package/dist/src/cli.d.ts

@@ -31,7 +31,7 @@ export interface CliIO {
     /** Injected for tests. Production reads stdin for `--input -`. */
     stdin?: () => string;
 }
-export declare const USAGE = "coltrane 0.16.0\n\n  coltrane validate                     load the genome; exit non-zero on load errors\n  coltrane dispatch <standard>          run a standard\n  coltrane monitor <gig-id>             report a gig's progress\n  coltrane logs <gig-id>                per-chair logs for a gig\n  coltrane abort <gig-id>               stop a running gig\n  coltrane trace <output-id>            walk an output's provenance\n  coltrane simulate <standard>          cost/shape a standard without running it\n  coltrane health                       engine + store health\n  coltrane serve                        run the MCP server on stdio\n  coltrane enqueue <standard>           queue a gig on the LOCAL file queue (no store, no key)\n                                        (env: COLTRANE_QUEUE_DIR \u2014 its absence is the whole\n                                         difference between local and hosted)\n  coltrane work                         claim one queued gig from the org store and run it\n  coltrane work --check                  report whether this box's drain environment is configured\n                                        and exit without claiming (0 ready, 1 not ready)\n                                        (env: COLTRANE_STORE_URL, COLTRANE_STORE_ANON,\n                                         COLTRANE_AGENT_TOKEN; results drain via\n                                         COLTRANE_DRAIN_URL + COLTRANE_DRAIN_KEY;\n                                         checkpoints under COLTRANE_WORKER_CHECKPOINTS,\n                                         default ~/.coltrane/worker-checkpoints;\n                                         exit 0 complete or parked, 1 failed, 3 queue empty)\n\nOptions\n  --input <json|@file|->                dispatch payload; @file reads a file, - reads stdin\n  --depth <skim|standard|deep>          tighten the per-chair turn cap\n  --budget <dollars>                    per-gig ceiling; the run stops when it is gone\n  --reuse                               allow chair-level reuse of prior sealed outputs\n  --resume <gig-id>                     continue a gig that died mid-pipeline\n  --approve <role>=<json|@file>         a human chair's verdict; repeat once per chair\n  --as <name>                           who is approving; sealed as the verdict's author\n  --wait                                block until the run finishes\n  --follow                              poll until the gig reaches a terminal state\n  --direction <upstream|downstream|both>  for trace\n  --json                                print the raw result to stdout\n  --genome <path>                       genome root (default: $COLTRANE_GENOME or cwd)\n  --help, --version\n\nA dispatch that reaches a chair a HUMAN holds parks: it names the waiting chair and exits 0,\nbecause a gig waiting on a person is not a failed gig. Approve it on the resume \u2014\n\n  coltrane dispatch <standard> --resume <gig> \\\n      --approve approve=@verdict.json --as eugene\n\n--input is NOT required on an approve-only resume: when every remaining chair is human the\ncheckpoint's recorded payload stands and the omission inherits it. If you DO pass --input it is\nchecked against the checkpoint, and a disagreement still refuses the resume.\n";
+export declare const USAGE = "coltrane 0.17.0\n\n  coltrane validate                     load the genome; exit non-zero on load errors\n  coltrane dispatch <standard>          run a standard\n  coltrane monitor <gig-id>             report a gig's progress\n  coltrane logs <gig-id>                per-chair logs for a gig\n  coltrane abort <gig-id>               stop a running gig\n  coltrane trace <output-id>            walk an output's provenance\n  coltrane simulate <standard>          cost/shape a standard without running it\n  coltrane health                       engine + store health\n  coltrane serve                        run the MCP server on stdio\n  coltrane enqueue <standard>           queue a gig on the LOCAL file queue (no store, no key)\n                                        (env: COLTRANE_QUEUE_DIR \u2014 its absence is the whole\n                                         difference between local and hosted)\n  coltrane work                         claim one queued gig from the org store and run it\n  coltrane work --check                  report whether this box's drain environment is configured\n                                        and exit without claiming (0 ready, 1 not ready)\n                                        (env: COLTRANE_STORE_URL, COLTRANE_STORE_ANON,\n                                         COLTRANE_AGENT_TOKEN; results drain via\n                                         COLTRANE_DRAIN_URL + COLTRANE_DRAIN_KEY;\n                                         checkpoints under COLTRANE_WORKER_CHECKPOINTS,\n                                         default ~/.coltrane/worker-checkpoints;\n                                         exit 0 complete or parked, 1 failed, 3 queue empty)\n\nOptions\n  --input <json|@file|->                dispatch payload; @file reads a file, - reads stdin\n  --depth <skim|standard|deep>          tighten the per-chair turn cap\n  --budget <dollars>                    per-gig ceiling; the run stops when it is gone\n  --reuse                               allow chair-level reuse of prior sealed outputs\n  --resume <gig-id>                     continue a gig that died mid-pipeline\n  --approve <role>=<json|@file>         a human chair's verdict; repeat once per chair\n  --as <name>                           who is approving; sealed as the verdict's author\n  --wait                                block until the run finishes\n  --follow                              poll until the gig reaches a terminal state\n  --direction <upstream|downstream|both>  for trace\n  --json                                print the raw result to stdout\n  --genome <path>                       genome root (default: $COLTRANE_GENOME or cwd)\n  --help, --version\n\nA dispatch that reaches a chair a HUMAN holds parks: it names the waiting chair and exits 0,\nbecause a gig waiting on a person is not a failed gig. Approve it on the resume \u2014\n\n  coltrane dispatch <standard> --resume <gig> \\\n      --approve approve=@verdict.json --as eugene\n\n--input is NOT required on an approve-only resume: when every remaining chair is human the\ncheckpoint's recorded payload stands and the omission inherits it. If you DO pass --input it is\nchecked against the checkpoint, and a disagreement still refuses the resume.\n";
 interface Parsed {
     cmd: string | undefined;
     positional: string[];

package/dist/src/run_deps.d.ts

@@ -25,7 +25,7 @@
  * own fails closed. That is the correct posture for a box running work nobody is watching.
  */
 import { type ToolProviderRegistry } from "./tool_providers.js";
-import type { BudgetInput } from "./runtime.js";
+import type { BudgetInput, RunDeps } from "./runtime.js";
 /**
  * Every engine tool as an in-house provider, tagged with the engine's own MCP server.
  *
@@ -50,3 +50,63 @@ export declare function engineToolProviders(): ToolProviderRegistry;
  */
 export declare function drainBudget(input: Record<string, unknown> | undefined): BudgetInput;
 export declare function drainTimeoutMs(): number;
+/**
+ * WHICH REPOSITORY THIS GIG'S WORKING TREE IS. One home for the decision, shared by every door — the
+ * drain reads it as `resolveWorkingRepo(claim)` and the dispatch door as
+ * `resolveWorkingRepo({ input }, args['repo_url'])`, so a change-request's repository resolves the
+ * SAME way however the gig arrived.
+ *
+ * The order is the fix, in three tiers:
+ *   1. the TYPED input `repository` — a first-class field in every code-touching standard's input
+ *      contract (domain_types/change-request.json, change-context.json). The correct value was always
+ *      being passed; a direct dispatch was the door that ignored it.
+ *   2. an EXPLICIT repo_url argument — the dispatch door's named override, honoured beneath the typed
+ *      input and above the org default so `repo_url` on gig_dispatch keeps working as a fallback.
+ *   3. the org column (`claim.repo_url`) — the store's per-organization default, so existing
+ *      single-repo deployments keep working unchanged.
+ *
+ * Null is a normal answer, not a degraded one — a standard whose contract names no repository touches
+ * no tree and mints no git credential, which is already correct for research work.
+ *
+ * `explicitRepoUrl` has a default initializer so `Function.length === 1`: the drain calls this with one
+ * argument and tests/the_repo_is_typed_input pins that arity. `claim` is typed structurally (no
+ * ClaimedGig import) so this shared module keeps its no-new-dependency posture.
+ */
+export declare function resolveWorkingRepo(claim: {
+    input?: unknown;
+    repo_url?: string | null | undefined;
+}, explicitRepoUrl?: string | undefined): string | null;
+/**
+ * The SHARED run-deps every gig runs under, assembled once so the four call sites cannot drift.
+ *
+ * WHY THIS EXISTS. `runGig` takes its dependencies as a bag of optionals, and `runGig` has four call
+ * sites (server.ts sync + async dispatch, worker.ts drain, chart.ts movement), each of which used to
+ * hand-assemble its own body. Read one and each omission looks deliberate; read them side by side and
+ * they had already diverged — a wire added to one branch was carried to the other by a comment asking
+ * the next reader to remember. This function is that shared body: every door builds its run-deps from
+ * here, so a wire is INHERITED, not copied by reminder.
+ *
+ * The fields that legitimately DIFFER per door are supplied by explicit ARGUMENT, never defaulted:
+ *   · `mcpServerConfigs` — empty on the drain (a freshly cloned, untrusted repo must not declare
+ *     servers for the seat reading it; see the header above) and the bootstrap map on the server. A
+ *     default would silently hand one door the wrong value — the exact class of hand-carried
+ *     divergence this whole change exists to eliminate — so it is a REQUIRED argument.
+ *   · the venue trio (`venue`/`venues`/`venueRealizer`) and `repoUrl` — conditionally spread so an
+ *     absent one is threaded to nothing, keeping the venue-less / repo-less path byte-identical.
+ *
+ * DOOR-SPECIFIC fields (gig_id, signal, onProgress, depth, reuse/human wiring, resume_from,
+ * seed_outputs, chart, checkpoints) are NOT here: each call site spreads its own onto this result. The
+ * assembler unifies exactly the wires that diverged and no more.
+ */
+export type AssembleRunDepsArgs = Pick<RunDeps, "outputs" | "ledger" | "invoke" | "model_version" | "skills" | "skill_dirs" | "evals" | "budget" | "toolProviders" | "venue" | "venues" | "venueRealizer"> & {
+    /**
+     * The enforcement environment, supplied per door: {} on the drain, the bootstrap map on the server
+     * (which is itself `Record | undefined`). REQUIRED — the property may hold undefined but may never be
+     * OMITTED, so every door states this wire at its call site rather than inheriting a silent default;
+     * that omission is the exact hand-carried divergence this assembler exists to foreclose.
+     */
+    mcpServerConfigs: Readonly<Record<string, unknown>> | undefined;
+    /** The repository this run operates on, already resolved via `resolveWorkingRepo`. Null → not threaded. */
+    repoUrl?: string | null | undefined;
+};
+export declare function assembleRunDeps(args: AssembleRunDepsArgs): RunDeps;

package/dist/src/run_deps.js

@@ -71,4 +71,56 @@ export function drainTimeoutMs() {
     const env = Number(process.env["COLTRANE_GIG_TIMEOUT_MS"]);
     return Number.isFinite(env) && env > 0 ? env : DEFAULT_DRAIN_TIMEOUT_MS;
 }
+/**
+ * WHICH REPOSITORY THIS GIG'S WORKING TREE IS. One home for the decision, shared by every door — the
+ * drain reads it as `resolveWorkingRepo(claim)` and the dispatch door as
+ * `resolveWorkingRepo({ input }, args['repo_url'])`, so a change-request's repository resolves the
+ * SAME way however the gig arrived.
+ *
+ * The order is the fix, in three tiers:
+ *   1. the TYPED input `repository` — a first-class field in every code-touching standard's input
+ *      contract (domain_types/change-request.json, change-context.json). The correct value was always
+ *      being passed; a direct dispatch was the door that ignored it.
+ *   2. an EXPLICIT repo_url argument — the dispatch door's named override, honoured beneath the typed
+ *      input and above the org default so `repo_url` on gig_dispatch keeps working as a fallback.
+ *   3. the org column (`claim.repo_url`) — the store's per-organization default, so existing
+ *      single-repo deployments keep working unchanged.
+ *
+ * Null is a normal answer, not a degraded one — a standard whose contract names no repository touches
+ * no tree and mints no git credential, which is already correct for research work.
+ *
+ * `explicitRepoUrl` has a default initializer so `Function.length === 1`: the drain calls this with one
+ * argument and tests/the_repo_is_typed_input pins that arity. `claim` is typed structurally (no
+ * ClaimedGig import) so this shared module keeps its no-new-dependency posture.
+ */
+export function resolveWorkingRepo(claim, explicitRepoUrl = undefined) {
+    const typed = claim.input?.repository;
+    if (typeof typed === "string" && typed.trim().length > 0)
+        return typed;
+    if (typeof explicitRepoUrl === "string" && explicitRepoUrl.trim().length > 0)
+        return explicitRepoUrl;
+    const orgDefault = claim.repo_url;
+    return typeof orgDefault === "string" && orgDefault.trim().length > 0 ? orgDefault : null;
+}
+export function assembleRunDeps(args) {
+    return {
+        outputs: args.outputs,
+        ledger: args.ledger,
+        invoke: args.invoke,
+        model_version: args.model_version,
+        skills: args.skills,
+        skill_dirs: args.skill_dirs,
+        evals: args.evals,
+        budget: args.budget,
+        toolProviders: args.toolProviders,
+        mcpServerConfigs: args.mcpServerConfigs,
+        // The venue trio and the repository, threaded only when present so an absent one is threaded to
+        // nothing — runGig's `deps.venue !== undefined` gate stays untripped and the venue-less path is
+        // byte-identical.
+        ...(args.venue ? { venue: args.venue } : {}),
+        ...(args.venues ? { venues: args.venues } : {}),
+        ...(args.venueRealizer ? { venueRealizer: args.venueRealizer } : {}),
+        ...(args.repoUrl ? { repoUrl: args.repoUrl } : {}),
+    };
+}
 //# sourceMappingURL=run_deps.js.map

package/dist/src/run_deps.js.map

@@ -1 +1 @@
-{"version":3,"file":"run_deps.js","sourceRoot":"","sources":["../../src/run_deps.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;GAyBG;AAEH,OAAO,EAAE,SAAS,EAAE,MAAM,UAAU,CAAC;AACrC,OAAO,EAAE,iBAAiB,EAAgD,MAAM,qBAAqB,CAAC;AAGtG;;;;;;;GAOG;AACH,MAAM,UAAU,mBAAmB;IACjC,OAAO,IAAI,GAAG,CACZ,SAAS,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,EAAE,IAAI,EAAE,CAAC,CAAC,IAAI,EAAE,IAAI,EAAE,UAAmB,EAAE,MAAM,EAAE,iBAAiB,EAAE,CAAC,CAAC,CACvG,CAAC;AACJ,CAAC;AAED,0DAA0D;AAC1D,MAAM,qBAAqB,GAAG,IAAI,CAAC;AAEnC;;;;;;;;;;;;GAYG;AACH,MAAM,UAAU,WAAW,CAAC,KAA0C;IACpE,MAAM,KAAK,GAAI,KAAK,EAAE,CAAC,QAAQ,CAAuC,EAAE,CAAC,SAAS,CAAC,CAAC;IACpF,IAAI,OAAO,KAAK,KAAK,QAAQ,IAAI,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,IAAI,KAAK,GAAG,CAAC;QAAE,OAAO,EAAE,OAAO,EAAE,KAAK,EAAE,CAAC;IAEhG,MAAM,GAAG,GAAG,MAAM,CAAC,OAAO,CAAC,GAAG,CAAC,wBAAwB,CAAC,CAAC,CAAC;IAC1D,OAAO,EAAE,OAAO,EAAE,MAAM,CAAC,QAAQ,CAAC,GAAG,CAAC,IAAI,GAAG,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,qBAAqB,EAAE,CAAC;AACpF,CAAC;AAED;;;;;;GAMG;AACH,MAAM,wBAAwB,GAAG,EAAE,GAAG,EAAE,GAAG,IAAI,CAAC;AAEhD,MAAM,UAAU,cAAc;IAC5B,MAAM,GAAG,GAAG,MAAM,CAAC,OAAO,CAAC,GAAG,CAAC,yBAAyB,CAAC,CAAC,CAAC;IAC3D,OAAO,MAAM,CAAC,QAAQ,CAAC,GAAG,CAAC,IAAI,GAAG,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,wBAAwB,CAAC;AAC1E,CAAC"}
+{"version":3,"file":"run_deps.js","sourceRoot":"","sources":["../../src/run_deps.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;GAyBG;AAEH,OAAO,EAAE,SAAS,EAAE,MAAM,UAAU,CAAC;AACrC,OAAO,EAAE,iBAAiB,EAAgD,MAAM,qBAAqB,CAAC;AAGtG;;;;;;;GAOG;AACH,MAAM,UAAU,mBAAmB;IACjC,OAAO,IAAI,GAAG,CACZ,SAAS,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,EAAE,IAAI,EAAE,CAAC,CAAC,IAAI,EAAE,IAAI,EAAE,UAAmB,EAAE,MAAM,EAAE,iBAAiB,EAAE,CAAC,CAAC,CACvG,CAAC;AACJ,CAAC;AAED,0DAA0D;AAC1D,MAAM,qBAAqB,GAAG,IAAI,CAAC;AAEnC;;;;;;;;;;;;GAYG;AACH,MAAM,UAAU,WAAW,CAAC,KAA0C;IACpE,MAAM,KAAK,GAAI,KAAK,EAAE,CAAC,QAAQ,CAAuC,EAAE,CAAC,SAAS,CAAC,CAAC;IACpF,IAAI,OAAO,KAAK,KAAK,QAAQ,IAAI,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,IAAI,KAAK,GAAG,CAAC;QAAE,OAAO,EAAE,OAAO,EAAE,KAAK,EAAE,CAAC;IAEhG,MAAM,GAAG,GAAG,MAAM,CAAC,OAAO,CAAC,GAAG,CAAC,wBAAwB,CAAC,CAAC,CAAC;IAC1D,OAAO,EAAE,OAAO,EAAE,MAAM,CAAC,QAAQ,CAAC,GAAG,CAAC,IAAI,GAAG,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,qBAAqB,EAAE,CAAC;AACpF,CAAC;AAED;;;;;;GAMG;AACH,MAAM,wBAAwB,GAAG,EAAE,GAAG,EAAE,GAAG,IAAI,CAAC;AAEhD,MAAM,UAAU,cAAc;IAC5B,MAAM,GAAG,GAAG,MAAM,CAAC,OAAO,CAAC,GAAG,CAAC,yBAAyB,CAAC,CAAC,CAAC;IAC3D,OAAO,MAAM,CAAC,QAAQ,CAAC,GAAG,CAAC,IAAI,GAAG,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,wBAAwB,CAAC;AAC1E,CAAC;AAED;;;;;;;;;;;;;;;;;;;;;GAqBG;AACH,MAAM,UAAU,kBAAkB,CAChC,KAAgE,EAChE,kBAAsC,SAAS;IAE/C,MAAM,KAAK,GAAI,KAAK,CAAC,KAA8C,EAAE,UAAU,CAAC;IAChF,IAAI,OAAO,KAAK,KAAK,QAAQ,IAAI,KAAK,CAAC,IAAI,EAAE,CAAC,MAAM,GAAG,CAAC;QAAE,OAAO,KAAK,CAAC;IACvE,IAAI,OAAO,eAAe,KAAK,QAAQ,IAAI,eAAe,CAAC,IAAI,EAAE,CAAC,MAAM,GAAG,CAAC;QAAE,OAAO,eAAe,CAAC;IACrG,MAAM,UAAU,GAAG,KAAK,CAAC,QAAQ,CAAC;IAClC,OAAO,OAAO,UAAU,KAAK,QAAQ,IAAI,UAAU,CAAC,IAAI,EAAE,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,IAAI,CAAC;AAC5F,CAAC;AAkDD,MAAM,UAAU,eAAe,CAAC,IAAyB;IACvD,OAAO;QACL,OAAO,EAAE,IAAI,CAAC,OAAO;QACrB,MAAM,EAAE,IAAI,CAAC,MAAM;QACnB,MAAM,EAAE,IAAI,CAAC,MAAM;QACnB,aAAa,EAAE,IAAI,CAAC,aAAa;QACjC,MAAM,EAAE,IAAI,CAAC,MAAM;QACnB,UAAU,EAAE,IAAI,CAAC,UAAU;QAC3B,KAAK,EAAE,IAAI,CAAC,KAAK;QACjB,MAAM,EAAE,IAAI,CAAC,MAAM;QACnB,aAAa,EAAE,IAAI,CAAC,aAAa;QACjC,gBAAgB,EAAE,IAAI,CAAC,gBAAgB;QACvC,gGAAgG;QAChG,gGAAgG;QAChG,kBAAkB;QAClB,GAAG,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,KAAK,EAAE,IAAI,CAAC,KAAK,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;QAC5C,GAAG,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,MAAM,EAAE,IAAI,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;QAC/C,GAAG,CAAC,IAAI,CAAC,aAAa,CAAC,CAAC,CAAC,EAAE,aAAa,EAAE,IAAI,CAAC,aAAa,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;QACpE,GAAG,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,OAAO,EAAE,IAAI,CAAC,OAAO,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;KACnD,CAAC;AACJ,CAAC"}

package/dist/src/server.js

@@ -24,6 +24,7 @@ import { FileLedger, LedgerError, LEDGER_SCHEMA_VERSION, defaultLedgerPath, } fr
 import { sealDrill } from "./seal_drill.js";
 import { standardSimulate } from "./simulate.js";
 import { runGig, BudgetExhausted, GigAborted, ResumeRefused, partialGigUsage, partialBudgetState } from "./runtime.js";
+import { assembleRunDeps, resolveWorkingRepo } from "./run_deps.js";
 import { createCheckpointStore, createReuseStore } from "./reuse.js";
 import { makeClaudeInvoker, killLiveChairChildren } from "./claude_invoker.js";
 import { dockerComposeRealizer } from "./venue_realizer.js";
@@ -1017,23 +1018,31 @@ async function runImpl(slug, args, deps, approval) {
                 // Reached only when the target is a standard: the chart branch above always returns, and an
                 // unresolvable standard slug returned before the preflights. So there is exactly one here.
                 const standard = targetStandards[0];
+                // The repository this dispatch resolves for the run, through the ONE shared resolver both doors
+                // use (run_deps.ts): the TYPED input.repository is authoritative, and the explicit repo_url
+                // argument is honoured only BENEATH it (non-goal — repo_url on gig_dispatch keeps working as a
+                // fallback). This is the C2/C3 fix: a change-request carrying a typed `repository` is now
+                // honoured by a direct dispatch exactly as the drain honours it, instead of being ignored.
+                const dispatchRepoUrl = resolveWorkingRepo({ input: gigInput }, repoUrl);
+                // The SHARED run-deps for BOTH dispatch branches, built ONCE from the assembler so the wait:true
+                // and async paths cannot diverge by construction — a wire is inherited, not carried across by a
+                // reminder comment. The venue trio and repository are threaded (conditionally) inside the
+                // assembler; `mcpServerConfigs` is stated here as the explicit argument it requires. Each branch
+                // spreads its own door-specific fields (gig_id, onProgress, signal, depth, reuse/human wiring).
+                const dispatchDeps = assembleRunDeps({
+                    outputs: deps.outputs, ledger: deps.ledger, invoke: deps.invoke,
+                    model_version: deps.model_version, skills: deps.skills, skill_dirs: deps.skill_dirs,
+                    evals: deps.evals, budget,
+                    toolProviders: deps.toolProviders, mcpServerConfigs: deps.mcpServerConfigs, // dispatch preflight resolves against the invoker's environment
+                    venue, venues: deps.venues, venueRealizer: deps.venueRealizer,
+                    repoUrl: dispatchRepoUrl,
+                });
                 // Synchronous mode (opt-in via wait:true) — block, return the manifest. The
                 // deterministic test path and any caller that wants the answer in one call.
                 if (wait) {
                     try {
                         const res = await runGig(standard, gigInput, {
-                            outputs: deps.outputs, ledger: deps.ledger, invoke: deps.invoke,
-                            model_version: deps.model_version, skills: deps.skills, skill_dirs: deps.skill_dirs, evals: deps.evals, budget,
-                            toolProviders: deps.toolProviders, mcpServerConfigs: deps.mcpServerConfigs, // dispatch preflight resolves against the invoker's environment
-                            // THE ROOM THIS DISPATCH NAMED, threaded so runGig realizes it or refuses fail-closed —
-                            // the same conditional-spread trio the chart path uses (server.ts:935-941). `venue` is
-                            // threaded only when non-empty so runGig's `deps.venue !== undefined` gate is never
-                            // tripped by an absent one; `venues`/`venueRealizer` come from bootstrap (ServerDeps).
-                            ...(venue ? { venue } : {}),
-                            ...(deps.venues ? { venues: deps.venues } : {}),
-                            ...(deps.venueRealizer ? { venueRealizer: deps.venueRealizer } : {}),
-                            // The repository the run names, threaded so runGig populates the room's tree from it.
-                            ...(repoUrl ? { repoUrl } : {}),
+                            ...dispatchDeps,
                             ...(depth ? { depth } : {}), ...reuseWiring, ...humanWiring,
                         });
                         return {
@@ -1116,19 +1125,11 @@ async function runImpl(slug, args, deps, approval) {
                         catch { /* best-effort */ }
                     }
                 };
+                // The SAME assembled base as the wait:true branch — the venue trio and the repository wire are
+                // INHERITED from `dispatchDeps`, not copied here by a reminder to keep the two branches in step.
+                // A wire added to the shared assembler reaches this default async path by construction.
                 const runPromise = runGig(standard, gigInput, {
-                    outputs: deps.outputs, ledger: deps.ledger, invoke: deps.invoke,
-                    model_version: deps.model_version, skills: deps.skills, skill_dirs: deps.skill_dirs, evals: deps.evals, budget,
-                    toolProviders: deps.toolProviders, mcpServerConfigs: deps.mcpServerConfigs, // dispatch preflight resolves against the invoker's environment
-                    // Same venue trio as the sync path above — the DEFAULT dispatch mode must honour a named
-                    // room too, or the fix covers only the deterministic wait:true test path and leaves the
-                    // path the product actually dispatches through discarding the venue.
-                    ...(venue ? { venue } : {}),
-                    ...(deps.venues ? { venues: deps.venues } : {}),
-                    ...(deps.venueRealizer ? { venueRealizer: deps.venueRealizer } : {}),
-                    // Same repository wire as the sync path above — the default async dispatch must populate a
-                    // named room's tree too, not only the deterministic wait:true path.
-                    ...(repoUrl ? { repoUrl } : {}),
+                    ...dispatchDeps,
                     gig_id: gigId, onProgress, signal: controller.signal, ...(depth ? { depth } : {}), ...reuseWiring, ...humanWiring,
                 });
                 // A REFUSED resume must be answered in THIS reply, not discovered later by polling. The