@wrongstack/core 0.299.0 → 0.300.0

package/dist/execution/index.js

@@ -3979,7 +3979,11 @@ async function completeBrainLlm(target, input) {
   return (await completeBrainLlmDetailed(target, input)).text;
 }
 async function completeBrainLlmDetailed(target, input) {
-  const signal = AbortSignal.timeout(input.timeoutMs);
+  if (input.signal?.aborted) {
+    throw new DOMException("Brain call aborted before it started.", "AbortError");
+  }
+  const timeoutSignal = AbortSignal.timeout(input.timeoutMs);
+  const signal = input.signal ? AbortSignal.any([input.signal, timeoutSignal]) : timeoutSignal;
   const response = await target.provider.complete(
     {
       model: target.model,
@@ -5174,22 +5178,29 @@ function buildCouncilJudgeUserPrompt(question, votes, opts = {}) {
     "</council-ballots>"
   ].filter(Boolean).join("\n\n");
 }
-function normalizeOptions(options) {
-  if (!options) return [];
+function validateCouncilOptions(options) {
+  const errors = [];
   const seen = /* @__PURE__ */ new Set();
-  return options.map((option) => {
+  for (const option of options ?? []) {
     const id = option.id.trim();
-    const label = option.label.trim();
-    if (!id) throw new Error("buildCouncilQuestionPrompt: option id must not be empty.");
-    if (!label) throw new Error(`buildCouncilQuestionPrompt: option "${id}" needs a label.`);
-    if (seen.has(id)) throw new Error(`buildCouncilQuestionPrompt: duplicate option id "${id}".`);
+    if (!id) errors.push("Every option must have a non-empty `id`.");
+    if (!option.label.trim()) errors.push(`Option "${id || "<empty>"}" must have a label.`);
+    if (seen.has(id)) errors.push(`Duplicate option id "${id}".`);
     seen.add(id);
-    return {
-      id,
-      label,
-      ...option.consequence?.trim() ? { consequence: option.consequence.trim() } : {}
-    };
-  });
+  }
+  return errors;
+}
+function normalizeOptions(options) {
+  if (!options) return [];
+  const errors = validateCouncilOptions(options);
+  if (errors.length > 0) {
+    throw new Error(`buildCouncilQuestionPrompt: ${errors.join("; ")}`);
+  }
+  return options.map((option) => ({
+    id: option.id.trim(),
+    label: option.label.trim(),
+    ...option.consequence?.trim() ? { consequence: option.consequence.trim() } : {}
+  }));
 }
 function requiredInstruction(path17) {
   const text = readBundledInstructionText(path17);
@@ -5297,6 +5308,8 @@ function requireFraction(value, label) {
 var COUNCIL_REFUSAL_OPTION_ID = "council_refuse";
 var DEFAULT_COUNCIL_MAX_CONCURRENCY = 3;
 var MAX_COUNCIL_CONCURRENCY = 8;
+var OVERALL_TIMEOUT_REASON = "Council overall timeout exceeded.";
+var CALL_CANCELLED_REASON = "Cancelled.";
 var CouncilOrchestrator = class {
   caller;
   personas;
@@ -5307,6 +5320,17 @@ var CouncilOrchestrator = class {
   fallbackProfileManager;
   seatCaller;
   judgeCaller;
+  /**
+   * Normalized ad-hoc profiles keyed by the caller's config object identity.
+   * The Brain adapter reuses ONE profile object for every decision, so this
+   * avoids re-validating + re-freezing it on every ask() without caching
+   * string-keyed registry lookups (those are already O(1)).
+   *
+   * Hosts must treat ad-hoc profile configs as IMMUTABLE once passed to
+   * ask(): the cache is keyed by object identity and never invalidated, so
+   * mutating a cached profile would silently serve the first snapshot.
+   */
+  profileCache = /* @__PURE__ */ new WeakMap();
   constructor(opts) {
     if (!opts.caller && !opts.seatCaller && !opts.judgeCaller) {
       throw new Error(
@@ -5325,13 +5349,33 @@ var CouncilOrchestrator = class {
     this.seatCaller = opts.seatCaller;
     this.judgeCaller = opts.judgeCaller;
   }
-  async ask(question) {
-    const startedAt = Date.now();
-    const profile = resolveCouncilProfile(question.profile, {
+  /**
+   * Resolve the effective profile for a question. String ids and the default
+   * go through the registry (already O(1)); ad-hoc config objects are
+   * normalized once per stable object identity and cached, because hosts such
+   * as the Brain adapter pass the same profile object on every ask().
+   */
+  resolveProfile(profile) {
+    if (typeof profile === "string" || profile === void 0) {
+      return resolveCouncilProfile(profile, {
+        registry: this.profiles,
+        personas: this.personas,
+        defaultProfile: this.defaultProfile
+      });
+    }
+    const cached = this.profileCache.get(profile);
+    if (cached) return cached;
+    const resolved = resolveCouncilProfile(profile, {
       registry: this.profiles,
       personas: this.personas,
       defaultProfile: this.defaultProfile
     });
+    this.profileCache.set(profile, resolved);
+    return resolved;
+  }
+  async ask(question) {
+    const startedAt = Date.now();
+    const profile = this.resolveProfile(question.profile);
     validateRefusalCollision(question, this.refusalOptionId);
     const timeoutSignal = AbortSignal.timeout(profile.overallTimeoutMs);
     const signal = question.signal ? AbortSignal.any([question.signal, timeoutSignal]) : timeoutSignal;
@@ -5348,11 +5392,16 @@ var CouncilOrchestrator = class {
         try {
           return await this.callSeat(question, profile, seat, i, signal, usage);
         } catch (error) {
+          const timedOut = signal.aborted && !question.signal?.aborted;
           return {
             seatId: seat.id,
             persona: seat.persona,
-            status: signal.aborted ? "cancelled" : "failed",
-            error: errorMessage(error)
+            // Only the caller's own cancel is a "cancelled" vote; the overall
+            // budget expiring is a failure (timeout), matching the envelope.
+            status: question.signal?.aborted ? "cancelled" : "failed",
+            // Canonical text for aborted-by-budget or cancelled seats, so one
+            // event does not surface a different string per code path.
+            error: question.signal?.aborted ? CALL_CANCELLED_REASON : timedOut ? OVERALL_TIMEOUT_REASON : errorMessage(error)
           };
         }
       }
@@ -5362,7 +5411,7 @@ var CouncilOrchestrator = class {
     if (question.signal?.aborted) {
       return resultEnvelope({
         status: "cancelled",
-        reason: "Council call cancelled.",
+        reason: CALL_CANCELLED_REASON,
         resolution: "none",
         votes,
         profile,
@@ -5375,14 +5424,17 @@ var CouncilOrchestrator = class {
     if (timeoutSignal.aborted) {
       return resultEnvelope({
         status: "failed",
-        reason: "Council overall timeout exceeded.",
+        reason: OVERALL_TIMEOUT_REASON,
         resolution: "none",
         votes,
         profile,
         usage,
         startedAt,
         warnings,
-        errors: [...errors, "Council overall timeout exceeded."]
+        // Seat-prefixed errors normally carry the canonical timeout text, but
+        // a signal-blind caller can resolve valid votes even after the budget
+        // expired — append the standalone entry only when nothing carries it.
+        errors: errors.some((entry) => entry.includes(OVERALL_TIMEOUT_REASON)) ? errors : [...errors, OVERALL_TIMEOUT_REASON]
       });
     }
     if (!question.options || question.options.length === 0) {
@@ -5409,7 +5461,17 @@ var CouncilOrchestrator = class {
     );
   }
   async callSeat(question, profile, seat, seatIndex, signal, usage) {
-    if (signal.aborted) return cancelledVote(seat);
+    if (signal.aborted) {
+      return question.signal?.aborted ? cancelledVote(seat) : {
+        seatId: seat.id,
+        persona: seat.persona,
+        status: "failed",
+        ...seat.target?.providerId ? { provider: seat.target.providerId } : {},
+        ...seat.target?.model ? { model: seat.target.model } : {},
+        durationMs: 0,
+        error: OVERALL_TIMEOUT_REASON
+      };
+    }
     let persona;
     try {
       persona = this.personas.require(seat.persona);
@@ -5435,12 +5497,15 @@ var CouncilOrchestrator = class {
     });
     const metadata = callMetadata(result);
     if (result.error) {
+      const timedOut = signal.aborted && !question.signal?.aborted;
       return {
         seatId: seat.id,
         persona: seat.persona,
-        status: signal.aborted ? "cancelled" : "failed",
+        status: question.signal?.aborted ? "cancelled" : "failed",
         ...metadata,
-        error: result.error
+        // Canonical text for cancelled or aborted-by-budget seats, so one
+        // cancel/timeout event does not surface a raw provider string.
+        error: question.signal?.aborted ? CALL_CANCELLED_REASON : timedOut ? OVERALL_TIMEOUT_REASON : result.error
       };
     }
     const parsed = parseVote(result.text, question, this.refusalOptionId);
@@ -5539,9 +5604,27 @@ var CouncilOrchestrator = class {
       signal,
       usage
     );
+    if (signal.aborted) {
+      const cancelled = question.signal?.aborted === true;
+      const reason = cancelled ? CALL_CANCELLED_REASON : OVERALL_TIMEOUT_REASON;
+      return resultEnvelope({
+        status: cancelled ? "cancelled" : "failed",
+        reason,
+        resolution: "none",
+        votes,
+        profile,
+        usage,
+        startedAt,
+        warnings,
+        errors: errors.some((entry) => entry.includes(reason)) ? errors : [...errors, reason],
+        judgeUsed: true
+      });
+    }
     if (!judged.ok) {
       return resultEnvelope({
-        status: signal.aborted ? "cancelled" : "abstained",
+        // User cancel -> cancelled; overall budget expired mid-judge -> failed;
+        // otherwise the judge simply failed -> abstained (can't decide).
+        status: question.signal?.aborted ? "cancelled" : signal.aborted ? "failed" : "abstained",
         reason: judged.error,
         resolution: "none",
         votes,
@@ -5601,6 +5684,19 @@ var CouncilOrchestrator = class {
       });
     }
     if (!profile.judge) {
+      if (divergentStances(valid)) {
+        return resultEnvelope({
+          status: "abstained",
+          reason: "Council produced multiple distinct stances and has no judge to reconcile them.",
+          resolution: "none",
+          votes,
+          profile,
+          usage,
+          startedAt,
+          warnings,
+          errors
+        });
+      }
       const first = valid[0];
       if (!first) {
         return resultEnvelope({
@@ -5637,9 +5733,30 @@ var CouncilOrchestrator = class {
       signal,
       usage
     );
+    if (signal.aborted) {
+      const cancelled = question.signal?.aborted === true;
+      const reason = cancelled ? CALL_CANCELLED_REASON : OVERALL_TIMEOUT_REASON;
+      return resultEnvelope({
+        status: cancelled ? "cancelled" : "failed",
+        reason,
+        resolution: "none",
+        votes,
+        profile,
+        usage,
+        startedAt,
+        warnings,
+        errors: errors.some((entry) => entry.includes(reason)) ? errors : [...errors, reason],
+        judgeUsed: true
+      });
+    }
     if (!judged.ok) {
       return resultEnvelope({
-        status: signal.aborted ? "cancelled" : "failed",
+        // User cancel -> cancelled; overall budget expired mid-judge -> failed;
+        // otherwise the judge simply failed -> failed (open questions cannot
+        // abstain for a judge failure — 'abstained' is reserved for quorum
+        // failure and stance divergence; the option path maps this same
+        // plain-judge-failure case to 'abstained' instead).
+        status: question.signal?.aborted ? "cancelled" : "failed",
         reason: judged.error,
         resolution: "none",
         votes,
@@ -5678,18 +5795,28 @@ var CouncilOrchestrator = class {
       signal,
       usage
     });
-    if (result.error) return { ok: false, error: result.error };
+    if (result.error) {
+      if (signal.aborted && !question.signal?.aborted) {
+        return { ok: false, error: OVERALL_TIMEOUT_REASON };
+      }
+      if (question.signal?.aborted) {
+        return { ok: false, error: CALL_CANCELLED_REASON };
+      }
+      return { ok: false, error: result.error };
+    }
     return parseJudge(result.text, question, this.refusalOptionId);
   }
   /**
-   * Resolve the effective LLM caller for a call. Voter seats use
-   * `seatCaller(seatIndex)` when wired. Judge seats (seatIndex undefined)
-   * use `judgeCaller` if set, otherwise `seatCaller(0)` if set, otherwise
-   * the shared `caller`.
+   * Resolve the effective LLM caller for a call. Voter seats (defined
+   * seatIndex) use `seatCaller(seatIndex)` when wired, otherwise the shared
+   * `caller` — a seat never falls through to the judge caller. Judge seats
+   * (seatIndex undefined) use `judgeCaller` if set, otherwise `seatCaller(0)`
+   * if set, otherwise the shared `caller`.
    */
   resolveCaller(seatIndex) {
-    if (seatIndex !== void 0 && this.seatCaller) {
-      return this.seatCaller(seatIndex);
+    if (seatIndex !== void 0) {
+      if (this.seatCaller) return this.seatCaller(seatIndex);
+      return this.caller ?? this.judgeCaller;
     }
     if (this.judgeCaller) return this.judgeCaller;
     if (this.seatCaller) return this.seatCaller(0);
@@ -5698,6 +5825,7 @@ var CouncilOrchestrator = class {
   async safeCall(input) {
     const effectiveCaller = this.resolveCaller(input.seatIndex);
     const resolvedTarget = this.resolveCouncilTarget(input.target);
+    const startedAt = Date.now();
     try {
       const result = await effectiveCaller.call({
         system: input.system,
@@ -5714,8 +5842,17 @@ var CouncilOrchestrator = class {
       addUsage(input.usage, result);
       return result;
     } catch (error) {
-      input.usage.calls += 1;
-      return emptyCallResult(errorMessage(error));
+      const failed = {
+        text: "",
+        model: resolvedTarget?.model ?? "",
+        provider: resolvedTarget?.providerId ?? "",
+        tokens: { input: 0, output: 0, total: 0 },
+        durationMs: Math.max(0, Date.now() - startedAt),
+        fromFallback: false,
+        error: errorMessage(error)
+      };
+      addUsage(input.usage, failed);
+      return failed;
     }
   }
   /**
@@ -5747,47 +5884,77 @@ var CouncilOrchestrator = class {
     };
   }
 };
-function parseVote(text, question, refusalOptionId) {
+function parseCouncilResponse(text, question, refusalOptionId, opts) {
+  const roleLabel = opts.role === "judge" ? "Judge" : "Voter";
   const parsed = parseObject(text);
   if (!parsed.ok && (!question.options || question.options.length === 0)) {
     const fallback = text.trim();
-    if (fallback) return { ok: true, vote: { stance: fallback } };
-    return { ok: false, error: "Voter returned an empty response." };
+    if (fallback) return { ok: true, value: { [opts.freeTextField]: fallback } };
+    return { ok: false, error: `${roleLabel} returned an empty response.` };
   }
   if (!parsed.ok) return parsed;
   const rationale = optionalString(parsed.value["rationale"]);
   if (question.options && question.options.length > 0) {
     const optionId = optionalString(parsed.value["optionId"]);
-    const allowed = /* @__PURE__ */ new Set([...question.options.map((option) => option.id.trim()), refusalOptionId]);
+    const allowed = /* @__PURE__ */ new Set([
+      ...question.options.map((option) => option.id.trim()),
+      refusalOptionId
+    ]);
     if (!optionId || !allowed.has(optionId)) {
-      return { ok: false, error: "Voter returned an unknown or missing optionId." };
+      return { ok: false, error: `${roleLabel} returned an unknown or missing optionId.` };
     }
-    return { ok: true, vote: { optionId, ...rationale ? { rationale } : {} } };
+    return { ok: true, value: { optionId, ...rationale ? { rationale } : {} } };
+  }
+  const freeText = optionalString(parsed.value[opts.freeTextField]);
+  if (!freeText) {
+    return {
+      ok: false,
+      error: `${roleLabel} returned an empty or missing ${opts.freeTextField}.`
+    };
   }
-  const stance = optionalString(parsed.value["stance"]);
-  if (!stance) return { ok: false, error: "Voter returned an empty or missing stance." };
-  return { ok: true, vote: { stance, ...rationale ? { rationale } : {} } };
+  return { ok: true, value: { [opts.freeTextField]: freeText, ...rationale ? { rationale } : {} } };
 }
-function parseJudge(text, question, refusalOptionId) {
-  const parsed = parseObject(text);
-  if (!parsed.ok && (!question.options || question.options.length === 0)) {
-    const fallback = text.trim();
-    if (fallback) return { ok: true, value: { answer: fallback } };
-    return { ok: false, error: "Judge returned an empty response." };
+function divergentStances(valid) {
+  const seen = /* @__PURE__ */ new Set();
+  for (const vote of valid) {
+    const normalized = vote.stance.trim().replace(/^["'`]+|["'`]+$/g, "").toLowerCase().replace(/[.!?;:,]+$/g, "").replace(/\s+/g, " ").trim();
+    if (!normalized) return true;
+    seen.add(normalized);
+    if (seen.size > 1) return true;
   }
+  return false;
+}
+function parseVote(text, question, refusalOptionId) {
+  const parsed = parseCouncilResponse(text, question, refusalOptionId, {
+    role: "voter",
+    freeTextField: "stance"
+  });
   if (!parsed.ok) return parsed;
-  const rationale = optionalString(parsed.value["rationale"]);
-  if (question.options && question.options.length > 0) {
-    const optionId = optionalString(parsed.value["optionId"]);
-    const allowed = /* @__PURE__ */ new Set([...question.options.map((option) => option.id.trim()), refusalOptionId]);
-    if (!optionId || !allowed.has(optionId)) {
-      return { ok: false, error: "Judge returned an unknown or missing optionId." };
+  const { optionId, stance, rationale } = parsed.value;
+  return {
+    ok: true,
+    vote: {
+      ...optionId ? { optionId } : {},
+      ...stance ? { stance } : {},
+      ...rationale ? { rationale } : {}
     }
-    return { ok: true, value: { optionId, ...rationale ? { rationale } : {} } };
-  }
-  const answer = optionalString(parsed.value["answer"]);
-  if (!answer) return { ok: false, error: "Judge returned an empty or missing answer." };
-  return { ok: true, value: { answer, ...rationale ? { rationale } : {} } };
+  };
+}
+function parseJudge(text, question, refusalOptionId) {
+  const parsed = parseCouncilResponse(text, question, refusalOptionId, {
+    role: "judge",
+    freeTextField: "answer"
+  });
+  if (!parsed.ok) return parsed;
+  const { optionId, answer, rationale } = parsed.value;
+  return {
+    ok: true,
+    value: {
+      ...optionId ? { optionId } : {},
+      ...answer ? { answer } : {},
+      ...rationale ? { rationale } : {}
+    }
+  };
 }
 function parseObject(text) {
   const trimmed = text.trim();
@@ -5831,7 +5998,7 @@ function callMetadata(result) {
   };
 }
 function addUsage(usage, result) {
-  usage.calls += 1;
+  usage.calls += Math.max(1, result.attempts ?? 1);
   usage.inputTokens += result.tokens.input;
   usage.outputTokens += result.tokens.output;
   usage.totalTokens += result.tokens.total;
@@ -5840,21 +6007,40 @@ function usageResult(usage, startedAt) {
   return Object.freeze({ ...usage, durationMs: Math.max(0, Date.now() - startedAt) });
 }
 function cancelledVote(seat) {
-  return { seatId: seat.id, persona: seat.persona, status: "cancelled", error: "Cancelled." };
+  return {
+    seatId: seat.id,
+    persona: seat.persona,
+    status: "cancelled",
+    ...seat.target?.providerId ? { provider: seat.target.providerId } : {},
+    ...seat.target?.model ? { model: seat.target.model } : {},
+    durationMs: 0,
+    error: CALL_CANCELLED_REASON
+  };
 }
 function distinctTargetCount(votes, profile) {
-  const keys = votes.filter((vote) => vote.status === "valid").map(
-    (vote) => profile.distinctness === "provider" ? vote.provider : `${vote.provider ?? ""}/${vote.model ?? ""}`
-  ).filter(Boolean);
-  return new Set(keys).size;
+  return new Set(distinctTargetKeys(votes, profile)).size;
+}
+function distinctTargetKeys(votes, profile) {
+  const keys = [];
+  for (const vote of votes) {
+    if (vote.status !== "valid") continue;
+    const provider = vote.provider?.trim() ?? "";
+    const model = vote.model?.trim() ?? "";
+    if (profile.distinctness === "provider") {
+      if (provider) keys.push(provider);
+    } else if (provider || model) {
+      keys.push(`${provider}/${model}`);
+    }
+  }
+  return keys;
 }
 function distinctnessWarnings(votes, profile) {
   if (profile.distinctness === "none") return [];
-  const valid = votes.filter((vote) => vote.status === "valid");
-  const distinct = distinctTargetCount(valid, profile);
-  if (valid.length > 1 && distinct < valid.length) {
+  const keys = distinctTargetKeys(votes, profile);
+  const distinct = new Set(keys).size;
+  if (keys.length > 1 && distinct < keys.length) {
     return [
-      `Council distinctness policy "${profile.distinctness}" was not met: ${distinct} distinct target(s) served ${valid.length} valid vote(s).`
+      `Council distinctness policy "${profile.distinctness}" was not met: ${distinct} distinct target(s) served ${keys.length} valid vote(s).`
     ];
   }
   return [];
@@ -5890,17 +6076,6 @@ async function mapConcurrent(items, concurrency, worker) {
   await Promise.all(Array.from({ length: Math.min(concurrency, items.length) }, () => run()));
   return results;
 }
-function emptyCallResult(error) {
-  return {
-    text: "",
-    model: "",
-    provider: "",
-    tokens: { input: 0, output: 0, total: 0 },
-    durationMs: 0,
-    fromFallback: false,
-    error
-  };
-}
 function optionalString(value) {
   return typeof value === "string" && value.trim() ? value.trim() : void 0;
 }
@@ -5947,52 +6122,58 @@ function resolvePersonaRegistry(voters) {
   });
   return { registry, personaIds };
 }
+function makeSeatCallerForVoter(target) {
+  return {
+    async call(input) {
+      const startedAt = Date.now();
+      try {
+        const result = await completeBrainLlmDetailed(
+          { provider: target.provider, model: input.model ?? target.model },
+          {
+            system: typeof input.system === "string" ? input.system : Array.isArray(input.system) ? input.system.map((b) => b.text).join("\n") : "",
+            user: input.userPrompt ?? "",
+            timeoutMs: input.timeoutMs ?? 15e3,
+            maxTokens: input.maxTokens,
+            // Forward the orchestrator's signal (overall budget + caller
+            // cancellation) so an in-flight seat call is interrupted when
+            // the council times out or is cancelled instead of running to
+            // its per-call timeout. Previously this was dropped and a
+            // cancelled council waited for every seat to finish.
+            signal: input.signal
+          }
+        );
+        const inputTokens = result.usage?.input ?? 0;
+        const outputTokens = result.usage?.output ?? 0;
+        return {
+          text: result.text,
+          model: target.model,
+          provider: target.provider.id,
+          tokens: {
+            input: inputTokens,
+            output: outputTokens,
+            total: inputTokens + outputTokens
+          },
+          durationMs: Date.now() - startedAt,
+          fromFallback: false
+        };
+      } catch (error) {
+        return {
+          text: "",
+          model: target.model,
+          provider: target.provider.id,
+          tokens: { input: 0, output: 0, total: 0 },
+          durationMs: 0,
+          fromFallback: false,
+          error: error instanceof Error ? error.message : String(error)
+        };
+      }
+    }
+  };
+}
 function createCouncilBrainArbiter(opts) {
   if (opts.voters.length === 0) {
     throw new Error("createCouncilBrainArbiter: at least one voter is required.");
   }
-  function makeSeatCallerForVoter(target) {
-    return {
-      async call(input) {
-        const startedAt = Date.now();
-        try {
-          const result = await completeBrainLlmDetailed(
-            { provider: target.provider, model: input.model ?? target.model },
-            {
-              system: typeof input.system === "string" ? input.system : Array.isArray(input.system) ? input.system.map((b) => b.text).join("\n") : "",
-              user: input.userPrompt ?? "",
-              timeoutMs: input.timeoutMs ?? 15e3,
-              maxTokens: input.maxTokens
-            }
-          );
-          const inputTokens = result.usage?.input ?? 0;
-          const outputTokens = result.usage?.output ?? 0;
-          return {
-            text: result.text,
-            model: target.model,
-            provider: target.provider.id,
-            tokens: {
-              input: inputTokens,
-              output: outputTokens,
-              total: inputTokens + outputTokens
-            },
-            durationMs: Date.now() - startedAt,
-            fromFallback: false
-          };
-        } catch (error) {
-          return {
-            text: "",
-            model: target.model,
-            provider: target.provider.id,
-            tokens: { input: 0, output: 0, total: 0 },
-            durationMs: 0,
-            fromFallback: false,
-            error: error instanceof Error ? error.message : String(error)
-          };
-        }
-      }
-    };
-  }
   const seatCaller = (seatIndex) => {
     const voter = opts.voters[seatIndex];
     if (!voter) {
@@ -6030,13 +6211,23 @@ function createCouncilBrainArbiter(opts) {
     model: opts.judge.model,
     role: "judge"
   } : false;
+  const perCallTimeoutMs = opts.decisionTimeoutMs ?? 15e3;
+  const effectiveConcurrency = Math.max(
+    1,
+    Math.min(opts.maxConcurrency ?? DEFAULT_COUNCIL_MAX_CONCURRENCY, seats.length)
+  );
+  const overallTimeoutMs = Math.max(
+    DEFAULT_COUNCIL_OVERALL_TIMEOUT_MS,
+    perCallTimeoutMs * (Math.ceil(seats.length / effectiveConcurrency) + 1)
+  );
   const profile = {
     id: "brain-council-adapter",
     seats,
     judge: judgeTarget,
     quorumFraction: opts.quorumFraction ?? 0.5,
     approvalFraction: opts.approvalFraction ?? 0.5,
-    perCallTimeoutMs: opts.decisionTimeoutMs ?? 15e3,
+    perCallTimeoutMs,
+    overallTimeoutMs,
     ...opts.judgeMaxTokens !== void 0 ? { judgeMaxTokens: opts.judgeMaxTokens } : {},
     distinctness: opts.distinctness ?? "none"
   };
@@ -15965,8 +16156,17 @@ var DefaultMultiAgentCoordinator = class _DefaultMultiAgentCoordinator extends E
       durationMs: 0
     };
     this.completedResults.push(synthetic);
+    this.trimCompletedResults();
     this.emit("task.completed", { task, result: synthetic });
   }
+  trimCompletedResults() {
+    if (this.completedResults.length > _DefaultMultiAgentCoordinator.MAX_COMPLETED_RESULTS) {
+      this.completedResults.splice(
+        0,
+        this.completedResults.length - _DefaultMultiAgentCoordinator.MAX_COMPLETED_RESULTS
+      );
+    }
+  }
   async runDispatched(subagentId, task) {
     const subagent = this.subagents.get(subagentId);
     if (!subagent) return;
@@ -16100,12 +16300,7 @@ var DefaultMultiAgentCoordinator = class _DefaultMultiAgentCoordinator extends E
   }
   recordCompletion(result) {
     this.completedResults.push(result);
-    if (this.completedResults.length > _DefaultMultiAgentCoordinator.MAX_COMPLETED_RESULTS) {
-      this.completedResults.splice(
-        0,
-        this.completedResults.length - _DefaultMultiAgentCoordinator.MAX_COMPLETED_RESULTS
-      );
-    }
+    this.trimCompletedResults();
     this.totalIterations += result.iterations;
     if (this.inFlight > 0) {
       this.inFlight--;
@@ -20411,6 +20606,7 @@ var OneShotOrchestrator = class {
         tokens: { input: 0, output: 0, total: 0 },
         durationMs: Math.round(performance.now() - startedAt),
         fromFallback: false,
+        attempts: 0,
         error: "No provider or model could be resolved. Check your config."
       };
     }
@@ -20425,6 +20621,7 @@ var OneShotOrchestrator = class {
         tokens: { input: 0, output: 0, total: 0 },
         durationMs: Math.round(performance.now() - startedAt),
         fromFallback: false,
+        attempts: 0,
         error: `Cannot build provider "${target.providerId}": ${err instanceof Error ? err.message : String(err)}`
       };
     }
@@ -20437,11 +20634,13 @@ var OneShotOrchestrator = class {
     let fromFallback = false;
     let lastError;
     let fallbackEligible = false;
+    let attempts = 0;
     if (tracker && !tracker.isAvailable(target.providerId, target.model) || !evaluateModelCalendar(config.modelAvailabilitySchedule, target.providerId, target.model).allowed) {
       this.opts.logger?.debug(
         `one-shot: primary "${target.providerId}/${target.model}" is blocked \u2014 trying fallback`
       );
     } else {
+      attempts += 1;
       const primaryAttempt = await this.tryCall(
         provider,
         request,
@@ -20456,10 +20655,17 @@ var OneShotOrchestrator = class {
         tracker?.recordSuccess(target.providerId, target.model);
         servingProviderId = provider.id;
         servingModel = target.model;
-        return this.buildResult(result, servingProviderId, servingModel, false, startedAt);
+        return this.buildResult(result, servingProviderId, servingModel, false, startedAt, attempts);
       }
       if (!fallbackEligible || chain.length === 0) {
-        return this.buildErrorResult(lastError, target.providerId, target.model, false, startedAt);
+        return this.buildErrorResult(
+          lastError,
+          target.providerId,
+          target.model,
+          false,
+          startedAt,
+          attempts
+        );
       }
     }
     const estimatedTokens = estimateRequestTokens(request.messages, request.system, []).total;
@@ -20487,6 +20693,7 @@ var OneShotOrchestrator = class {
       }
       servingProviderId = fbProvider.id;
       servingModel = entry.model;
+      attempts += 1;
       const attempt = await this.tryCall(
         fbProvider,
         this.buildRequest(input, entry.model),
@@ -20497,7 +20704,14 @@ var OneShotOrchestrator = class {
       if (attempt.response) {
         tracker?.recordSuccess(entry.providerId, entry.model);
         fromFallback = true;
-        return this.buildResult(attempt.response, servingProviderId, servingModel, true, startedAt);
+        return this.buildResult(
+          attempt.response,
+          servingProviderId,
+          servingModel,
+          true,
+          startedAt,
+          attempts
+        );
       }
       lastError = attempt.error;
     }
@@ -20506,7 +20720,8 @@ var OneShotOrchestrator = class {
       servingProviderId,
       servingModel,
       fromFallback,
-      startedAt
+      startedAt,
+      attempts
     );
   }
   // ── Private helpers ─────────────────────────────────────────────
@@ -20630,7 +20845,7 @@ var OneShotOrchestrator = class {
     }
   }
   /** Build a success result from a provider Response. */
-  buildResult(response, servingProviderId, servingModel, fromFallback, startedAt) {
+  buildResult(response, servingProviderId, servingModel, fromFallback, startedAt, attempts) {
     const textBlocks = response.content.filter(isTextBlock);
     const text = textBlocks.map((b) => b.text).join("\n").trim();
     return {
@@ -20644,11 +20859,12 @@ var OneShotOrchestrator = class {
       },
       durationMs: Math.round(performance.now() - startedAt),
       fromFallback,
+      attempts,
       stopReason: response.stopReason
     };
   }
   /** Build a total-failure error result. */
-  buildErrorResult(error, servingProviderId, servingModel, fromFallback, startedAt) {
+  buildErrorResult(error, servingProviderId, servingModel, fromFallback, startedAt, attempts) {
     return {
       text: "",
       model: servingModel,
@@ -20656,6 +20872,7 @@ var OneShotOrchestrator = class {
       tokens: { input: 0, output: 0, total: 0 },
       durationMs: Math.round(performance.now() - startedAt),
       fromFallback,
+      attempts,
       error: error instanceof Error ? error.message : String(error ?? "Unknown error")
     };
   }