@mcoda/codali 0.1.109 → 0.1.111

package/dist/gateway/LocalGatewayProvider.js

@@ -5,6 +5,31 @@ import { ClaudeCliProvider } from "../providers/ClaudeCliProvider.js";
 import { CodexCliProvider } from "../providers/CodexCliProvider.js";
 import { OllamaRemoteProvider } from "../providers/OllamaRemoteProvider.js";
 import { OpenAiCompatibleProvider } from "../providers/OpenAiCompatibleProvider.js";
+/**
+ * Attaches the scheduling priority mswarm reads from `scheduling.priority`,
+ * without disturbing any runner settings the agent already carries.
+ */
+const buildLocalRunner = (nested, baseUrl, priority) => {
+    const existing = isRecord(nested.localRunner)
+        ? nested.localRunner
+        : undefined;
+    if (!isMswarmEndpoint(baseUrl)) {
+        return existing;
+    }
+    const existingExtra = isRecord(existing?.extraBody) ? existing.extraBody : {};
+    const existingScheduling = isRecord(existingExtra.scheduling) ? existingExtra.scheduling : {};
+    return {
+        ...(existing ?? {}),
+        extraBody: {
+            ...existingExtra,
+            scheduling: {
+                ...existingScheduling,
+                // An explicit value in the agent config wins over the default.
+                priority: existingScheduling.priority ?? priority ?? CODALI_MSWARM_PRIORITY,
+            },
+        },
+    };
+};
 /**
  * Request timeout for a locally-hosted model.
  *
@@ -14,6 +39,17 @@ import { OpenAiCompatibleProvider } from "../providers/OpenAiCompatibleProvider.
  * unavailable" — a timeout wearing the costume of an outage.
  */
 const LOCAL_MODEL_TIMEOUT_MS = 600000;
+/**
+ * Scheduling priority Codali asks mswarm for.
+ *
+ * mswarm sorts ascending, so a lower number runs sooner, and its nodes reserve
+ * capacity for priority <= -1. Codali runs are interactive — someone is waiting
+ * at a terminal or in a chat window — so they should not queue behind batch
+ * work. -10 sits comfortably inside the reserved band without claiming the
+ * extreme of the -100..100 range, leaving room for anything genuinely more
+ * urgent.
+ */
+export const CODALI_MSWARM_PRIORITY = -10;
 const isRecord = (value) => Boolean(value && typeof value === "object" && !Array.isArray(value));
 const readString = (record, keys) => {
     for (const key of keys) {
@@ -125,9 +161,7 @@ export const createProviderForAssignment = (assignment, options = {}) => {
         timeoutMs: options.timeoutMs ?? LOCAL_MODEL_TIMEOUT_MS,
         runnerKind: readString(nested, ["runnerKind", "runner_kind"]),
         authMode: readString(nested, ["authMode", "auth_mode"]),
-        localRunner: isRecord(nested.localRunner)
-            ? nested.localRunner
-            : undefined,
+        localRunner: buildLocalRunner(nested, baseUrl, options.priority),
         supportsTools: assignment.candidate.supportsTools ?? readBoolean(raw, ["supportsTools"]),
         supportsJsonSchema: assignment.candidate.supportsJsonSchema ?? readBoolean(raw, ["supportsJsonSchema"]),
         supportsStreaming: assignment.candidate.supportsStreaming ?? readBoolean(raw, ["supportsStreaming"]),

package/dist/gateway/LocalGatewayTaskRunner.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"LocalGatewayTaskRunner.d.ts","sourceRoot":"","sources":["../../src/gateway/LocalGatewayTaskRunner.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,QAAQ,EAGT,MAAM,+BAA+B,CAAC;AACvC,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,0BAA0B,CAAC;AAC7D,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,uBAAuB,CAAC;AAEzD,OAAO,KAAK,EAEV,+BAA+B,EAC/B,gCAAgC,EAChC,6BAA6B,EAE9B,MAAM,0BAA0B,CAAC;AAElC;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA4BG;AAEH,8EAA8E;AAC9E,eAAO,MAAM,uBAAuB,IAAI,CAAC;AAEzC,2EAA2E;AAC3E,eAAO,MAAM,wBAAwB,IAAI,CAAC;AAe1C,MAAM,WAAW,6BAA6B;IAC5C,QAAQ,EAAE,QAAQ,CAAC;IACnB,QAAQ,EAAE,YAAY,CAAC;IACvB,WAAW,EAAE,WAAW,CAAC;IACzB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,+CAA+C;IAC/C,OAAO,CAAC,EAAE,CAAC,KAAK,EAAE,2BAA2B,KAAK,IAAI,CAAC;CACxD;AAED,MAAM,MAAM,2BAA2B,GACnC;IAAE,IAAI,EAAE,YAAY,CAAC;IAAC,MAAM,EAAE,MAAM,CAAC;IAAC,UAAU,EAAE,MAAM,CAAC;IAAC,YAAY,EAAE,MAAM,EAAE,CAAA;CAAE,GAClF;IAAE,IAAI,EAAE,WAAW,CAAC;IAAC,MAAM,EAAE,MAAM,CAAC;IAAC,IAAI,EAAE,MAAM,CAAC;IAAC,IAAI,EAAE,OAAO,CAAA;CAAE,GAClE;IACE,IAAI,EAAE,aAAa,CAAC;IACpB,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,CAAC;IACb,EAAE,EAAE,OAAO,CAAC;IACZ,SAAS,EAAE,MAAM,CAAC;IAClB,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB,GACD;IACE,IAAI,EAAE,UAAU,CAAC;IACjB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,WAAW,GAAG,QAAQ,CAAC;IAC/B,aAAa,EAAE,MAAM,CAAC;IACtB,cAAc,EAAE,MAAM,CAAC;IACvB,UAAU,EAAE,MAAM,CAAC;CACpB,CAAC;AA6DN,qBAAa,sBAAuB,YAAW,6BAA6B;IAC9D,OAAO,CAAC,QAAQ,CAAC,OAAO;gBAAP,OAAO,EAAE,6BAA6B;IAE7D,GAAG,CACP,KAAK,EAAE,+BAA+B,GACrC,OAAO,CAAC,gCAAgC,CAAC;CA+S7C;AAED,eAAO,MAAM,4BAA4B,GACvC,SAAS,6BAA6B,KACrC,sBAA6D,CAAC;AAEjE,eAAO,MAAM,aAAa,UArXD,OAAO,KAAG,KAAK,IAAI,MAAM,CAAC,MAAM,EAAE,OAAO,CAqX7B,CAAC"}
+{"version":3,"file":"LocalGatewayTaskRunner.d.ts","sourceRoot":"","sources":["../../src/gateway/LocalGatewayTaskRunner.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,QAAQ,EAGT,MAAM,+BAA+B,CAAC;AACvC,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,0BAA0B,CAAC;AAC7D,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,uBAAuB,CAAC;AAGzD,OAAO,KAAK,EAEV,+BAA+B,EAC/B,gCAAgC,EAChC,6BAA6B,EAE9B,MAAM,0BAA0B,CAAC;AAElC;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA4BG;AAEH,8EAA8E;AAC9E,eAAO,MAAM,uBAAuB,IAAI,CAAC;AAEzC,2EAA2E;AAC3E,eAAO,MAAM,wBAAwB,IAAI,CAAC;AAe1C,MAAM,WAAW,6BAA6B;IAC5C,QAAQ,EAAE,QAAQ,CAAC;IACnB,QAAQ,EAAE,YAAY,CAAC;IACvB,WAAW,EAAE,WAAW,CAAC;IACzB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,+CAA+C;IAC/C,OAAO,CAAC,EAAE,CAAC,KAAK,EAAE,2BAA2B,KAAK,IAAI,CAAC;CACxD;AAED,MAAM,MAAM,2BAA2B,GACnC;IAAE,IAAI,EAAE,YAAY,CAAC;IAAC,MAAM,EAAE,MAAM,CAAC;IAAC,UAAU,EAAE,MAAM,CAAC;IAAC,YAAY,EAAE,MAAM,EAAE,CAAA;CAAE,GAClF;IAAE,IAAI,EAAE,WAAW,CAAC;IAAC,MAAM,EAAE,MAAM,CAAC;IAAC,IAAI,EAAE,MAAM,CAAC;IAAC,IAAI,EAAE,OAAO,CAAA;CAAE,GAClE;IACE,IAAI,EAAE,aAAa,CAAC;IACpB,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,CAAC;IACb,EAAE,EAAE,OAAO,CAAC;IACZ,SAAS,EAAE,MAAM,CAAC;IAClB,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB,GACD;IACE,IAAI,EAAE,UAAU,CAAC;IACjB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,WAAW,GAAG,QAAQ,CAAC;IAC/B,aAAa,EAAE,MAAM,CAAC;IACtB,cAAc,EAAE,MAAM,CAAC;IACvB,UAAU,EAAE,MAAM,CAAC;CACpB,CAAC;AAsEN,qBAAa,sBAAuB,YAAW,6BAA6B;IAC9D,OAAO,CAAC,QAAQ,CAAC,OAAO;gBAAP,OAAO,EAAE,6BAA6B;IAE7D,GAAG,CACP,KAAK,EAAE,+BAA+B,GACrC,OAAO,CAAC,gCAAgC,CAAC;CAqT7C;AAED,eAAO,MAAM,4BAA4B,GACvC,SAAS,6BAA6B,KACrC,sBAA6D,CAAC;AAEjE,eAAO,MAAM,aAAa,UApYD,OAAO,KAAG,KAAK,IAAI,MAAM,CAAC,MAAM,EAAE,OAAO,CAoY7B,CAAC"}

package/dist/gateway/LocalGatewayTaskRunner.js

@@ -1,3 +1,4 @@
+import { truncateToolResult } from "../tools/TruncateResult.js";
 import { CODALI_GATEWAY_SECURITY_PROMPT_HARDENING } from "./GatewaySecurityPolicy.js";
 /**
  * Bounded worker-task runner.
@@ -69,7 +70,16 @@ const buildTaskMessages = (input, hasTools) => {
         "You are a Codali gateway worker.",
         "Gather evidence for one bounded sub-task. Do not answer the user's overall question.",
         hasTools
-            ? "Call the tools you need in a single batch. You get exactly one opportunity to call tools, so request everything you need at once."
+            ? [
+                "Call the tools you need in a single batch. You get exactly one opportunity to call tools, so request everything you need at once.",
+                // Told only that it had one chance, the worker reliably issued a
+                // single search and stopped. A search ranks whole files; it does not
+                // return the value of a constant or the contents of a directory, so
+                // questions of that shape came back unanswered with the budget for
+                // seven more calls unspent.
+                "One search is rarely enough. In the same batch, add the tools that cover what a search cannot: listing a directory's files, reading a named file, or looking up a symbol's definition.",
+                "Prefer several complementary calls over one broad one — unused calls cost nothing, a missing one costs the answer.",
+            ].join("\n")
             : "No tools are available. Answer from the task description alone or state what is missing.",
         "Report only what the tool results actually support. Never invent file paths, identifiers, figures, or quotations.",
         "If the results are insufficient, say so plainly instead of guessing; another round may be scheduled.",
@@ -246,8 +256,14 @@ export class LocalGatewayTaskRunner {
                     errorCode: result.error?.code,
                     errorMessage: result.error?.message,
                 },
+                // Structural where the tool returned data: a 258 KB web-research
+                // payload cut at a character count is unparseable, and the model
+                // then reports it found nothing. Applied here so it covers docdex
+                // and built-in tools too, not just the connectors.
                 output: result.ok
-                    ? truncate(result.output)
+                    ? (result.data !== undefined
+                        ? truncateToolResult(result.data, result.output, MAX_TOOL_OUTPUT_CHARS).text
+                        : truncate(result.output))
                     : `Tool failed (${result.error?.code ?? "unknown"}): ${result.error?.message ?? ""}`,
             };
         }));

package/dist/gateway/RelevanceJudge.d.ts

@@ -10,5 +10,16 @@ export interface RelevanceJudgeOptions {
     provider: Provider;
     maxTokens?: number;
 }
+/**
+ * Whether any result title shares a distinctive word with the question.
+ *
+ * A cheap pre-check that exists because the judge was answering obvious cases
+ * wrongly and expensively: asked whether `LocalGatewayTaskRunner.ts` could
+ * answer "what does LocalGatewayTaskRunner do", a small model said no, which
+ * forced a 45-second web search for a local class. Lexical overlap settles
+ * that case without a model call, leaving the judge for genuinely ambiguous
+ * results — which is where it earns its keep.
+ */
+export declare const titlesShareTermsWithQuery: (query: string, titles: readonly string[]) => boolean;
 export declare const createRelevanceJudge: (options: RelevanceJudgeOptions) => RelevanceJudge;
 //# sourceMappingURL=RelevanceJudge.d.ts.map

package/dist/gateway/RelevanceJudge.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"RelevanceJudge.d.ts","sourceRoot":"","sources":["../../src/gateway/RelevanceJudge.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,+BAA+B,CAAC;AAC9D,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,gCAAgC,CAAC;AAyBrE;;;;GAIG;AACH,eAAO,MAAM,qBAAqB,GAAI,SAAS,OAAO,KAAG,MAAM,EAqB9D,CAAC;AAEF,MAAM,WAAW,qBAAqB;IACpC,QAAQ,EAAE,QAAQ,CAAC;IACnB,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AAED,eAAO,MAAM,oBAAoB,GAC/B,SAAS,qBAAqB,KAC7B,cAsCF,CAAC"}
+{"version":3,"file":"RelevanceJudge.d.ts","sourceRoot":"","sources":["../../src/gateway/RelevanceJudge.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,+BAA+B,CAAC;AAC9D,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,gCAAgC,CAAC;AAyBrE;;;;GAIG;AACH,eAAO,MAAM,qBAAqB,GAAI,SAAS,OAAO,KAAG,MAAM,EAqB9D,CAAC;AAEF,MAAM,WAAW,qBAAqB;IACpC,QAAQ,EAAE,QAAQ,CAAC;IACnB,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AAoCD;;;;;;;;;GASG;AACH,eAAO,MAAM,yBAAyB,GACpC,OAAO,MAAM,EACb,QAAQ,SAAS,MAAM,EAAE,KACxB,OAWF,CAAC;AAEF,eAAO,MAAM,oBAAoB,GAC/B,SAAS,qBAAqB,KAC7B,cAyCF,CAAC"}

package/dist/gateway/RelevanceJudge.js

@@ -46,11 +46,65 @@ export const summarizeResultTitles = (results) => {
     }
     return titles;
 };
+/**
+ * Words too common to signal relevance on their own.
+ *
+ * Not a keyword rule about any subject — just the connectives that appear in
+ * every question and would otherwise match any file path.
+ */
+const STOPWORDS = new Set([
+    "what", "which", "where", "when", "does", "did", "the", "and", "for", "with",
+    "from", "that", "this", "how", "why", "who", "are", "was", "were", "its",
+    "class", "file", "code", "show", "list", "find", "tell", "give", "about",
+    "purpose", "exists", "existence", "summary", "summarize", "explain",
+]);
+/**
+ * Splits text into comparable words, breaking CamelCase as well as punctuation.
+ *
+ * A symbol name is one word to a human and several to a path. Asked "which file
+ * defines CodaliGatewayPlannerError", the question yields the single term
+ * `codaligatewayplannererror`, while `gateway/GatewayPlanner.ts` flattens to
+ * `gateway gatewayplanner ts` — no overlap, so the correct file was discarded.
+ * Splitting both sides on case boundaries puts `gateway` and `planner` on each
+ * side, and the match succeeds.
+ */
+const wordsIn = (value) => value
+    // Insert a break between a lowercase/digit and a following uppercase, so
+    // `GatewayPlanner` becomes `Gateway Planner` before the lowercase pass.
+    .replace(/([a-z0-9])([A-Z])/g, "$1 $2")
+    // And between an acronym and a following word: `HTTPClient` -> `HTTP Client`.
+    .replace(/([A-Z]+)([A-Z][a-z])/g, "$1 $2")
+    .toLowerCase()
+    .split(/[^a-z0-9]+/)
+    .filter(Boolean);
+/**
+ * Whether any result title shares a distinctive word with the question.
+ *
+ * A cheap pre-check that exists because the judge was answering obvious cases
+ * wrongly and expensively: asked whether `LocalGatewayTaskRunner.ts` could
+ * answer "what does LocalGatewayTaskRunner do", a small model said no, which
+ * forced a 45-second web search for a local class. Lexical overlap settles
+ * that case without a model call, leaving the judge for genuinely ambiguous
+ * results — which is where it earns its keep.
+ */
+export const titlesShareTermsWithQuery = (query, titles) => {
+    const terms = wordsIn(query).filter((word) => word.length > 3 && !STOPWORDS.has(word));
+    if (terms.length === 0)
+        return false;
+    // Whole words only. Splitting both sides the same way already lets a symbol
+    // in the question meet the same symbol in a path, so a substring pass would
+    // add nothing but false matches ("string" hitting `StringUtils.ts`).
+    const haystack = ` ${wordsIn(titles.join(" ")).join(" ")} `;
+    return terms.some((term) => haystack.includes(` ${term} `));
+};
 export const createRelevanceJudge = (options) => async ({ query, results }) => {
     const titles = summarizeResultTitles(results);
     // Nothing to judge; the caller's presence check already decided.
     if (titles.length === 0)
         return true;
+    // An obvious match needs no model call, and no chance of getting it wrong.
+    if (titlesShareTermsWithQuery(query, titles))
+        return true;
     const response = await options.provider.generate({
         messages: [
             {

package/dist/providers/OpenAiCompatibleProvider.d.ts

@@ -1,4 +1,5 @@
 import type { Provider, ProviderConfig, ProviderRequest, ProviderResponse } from "./ProviderTypes.js";
+export declare const fetchWithTransientRetry: (url: string, init: RequestInit, controller?: AbortController) => Promise<Response>;
 export declare class OpenAiCompatibleProvider implements Provider {
     private config;
     /** OpenAI-style function calling is supported and parsed from `tool_calls`. */

package/dist/providers/OpenAiCompatibleProvider.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"OpenAiCompatibleProvider.d.ts","sourceRoot":"","sources":["../../src/providers/OpenAiCompatibleProvider.ts"],"names":[],"mappings":"AAOA,OAAO,KAAK,EACV,QAAQ,EACR,cAAc,EAEd,eAAe,EACf,gBAAgB,EAGjB,MAAM,oBAAoB,CAAC;AAsH5B,qBAAa,wBAAyB,YAAW,QAAQ;IAS3C,OAAO,CAAC,MAAM;IAR1B,+EAA+E;IAC/E,QAAQ,CAAC,iBAAiB,QAAQ;IAElC,IAAI,SAAuB;IAC3B,OAAO,CAAC,WAAW,CAAoC;IACvD,OAAO,CAAC,QAAQ,CAAsB;IACtC,OAAO,CAAC,sBAAsB,CAAoC;gBAE9C,MAAM,EAAE,cAAc;IAWpC,QAAQ,CAAC,OAAO,EAAE,eAAe,GAAG,OAAO,CAAC,gBAAgB,CAAC;IAsMnE,OAAO,CAAC,WAAW;CAepB"}
+{"version":3,"file":"OpenAiCompatibleProvider.d.ts","sourceRoot":"","sources":["../../src/providers/OpenAiCompatibleProvider.ts"],"names":[],"mappings":"AAOA,OAAO,KAAK,EACV,QAAQ,EACR,cAAc,EAEd,eAAe,EACf,gBAAgB,EAGjB,MAAM,oBAAoB,CAAC;AAgJ5B,eAAO,MAAM,uBAAuB,GAClC,KAAK,MAAM,EACX,MAAM,WAAW,EACjB,aAAa,eAAe,KAC3B,OAAO,CAAC,QAAQ,CAclB,CAAC;AAEF,qBAAa,wBAAyB,YAAW,QAAQ;IAS3C,OAAO,CAAC,MAAM;IAR1B,+EAA+E;IAC/E,QAAQ,CAAC,iBAAiB,QAAQ;IAElC,IAAI,SAAuB;IAC3B,OAAO,CAAC,WAAW,CAAoC;IACvD,OAAO,CAAC,QAAQ,CAAsB;IACtC,OAAO,CAAC,sBAAsB,CAAoC;gBAE9C,MAAM,EAAE,cAAc;IAWpC,QAAQ,CAAC,OAAO,EAAE,eAAe,GAAG,OAAO,CAAC,gBAAgB,CAAC;IA0MnE,OAAO,CAAC,WAAW;CAepB"}

package/dist/providers/OpenAiCompatibleProvider.js

@@ -80,6 +80,49 @@ const buildPromptOnlyFormatInstruction = (format) => {
     }
     return undefined;
 };
+/**
+ * A single dropped connection must not fail a whole run.
+ *
+ * Self-hosted llama.cpp behind a relay closes sockets under load, surfacing as
+ * a bare `TypeError: fetch failed` with no status. Measured over fifty live
+ * questions this took out four runs outright — the caller saw an empty result,
+ * not a slow one. Retried, each of them succeeded.
+ *
+ * Deliberately narrow: only connection-level errors are retried. An HTTP error
+ * status is the server's considered answer and repeating it just wastes a
+ * minute, and an abort is our own timeout firing, where retrying would double
+ * the wait the caller already declined.
+ */
+const TRANSIENT_RETRIES = 2;
+const TRANSIENT_RETRY_DELAY_MS = 750;
+const isTransientNetworkError = (error) => {
+    if (error instanceof Error && error.name === "AbortError")
+        return false;
+    const cause = error?.cause;
+    const code = cause?.code ?? error?.code;
+    if (typeof code === "string") {
+        return ["ECONNRESET", "ECONNREFUSED", "EPIPE", "ETIMEDOUT", "UND_ERR_SOCKET"].includes(code);
+    }
+    return error instanceof TypeError && /fetch failed|network|socket/i.test(error.message);
+};
+export const fetchWithTransientRetry = async (url, init, controller) => {
+    let lastError;
+    for (let attempt = 0; attempt <= TRANSIENT_RETRIES; attempt += 1) {
+        try {
+            return await fetch(url, init);
+        }
+        catch (error) {
+            lastError = error;
+            // Our own timeout already fired; there is no time left to spend.
+            if (controller?.signal.aborted || !isTransientNetworkError(error))
+                throw error;
+            if (attempt === TRANSIENT_RETRIES)
+                break;
+            await new Promise((resolve) => setTimeout(resolve, TRANSIENT_RETRY_DELAY_MS * (attempt + 1)));
+        }
+    }
+    throw lastError;
+};
 export class OpenAiCompatibleProvider {
     constructor(config) {
         this.config = config;
@@ -156,12 +199,12 @@ export class OpenAiCompatibleProvider {
         const timeoutMs = this.config.timeoutMs ?? 60000;
         const timeout = setTimeout(() => controller.abort(), timeoutMs);
         try {
-            const response = await fetch(url, {
+            const response = await fetchWithTransientRetry(url, {
                 method: "POST",
                 headers,
                 body: JSON.stringify(body),
                 signal: controller.signal,
-            });
+            }, controller);
             if (!response.ok) {
                 const errorBody = await response.text();
                 throw new Error(`OpenAI-compatible error ${response.status}: ${errorBody}`);

package/dist/tools/ToolRegistry.d.ts

@@ -1,9 +1,13 @@
-import { type ToolCapability, type ToolContext, type ToolDefinition, type ToolDescriptor, type ToolExecutionResult } from "./ToolTypes.js";
+import { type ToolCapability, type ToolContext, type ToolDefinition, type ToolDescriptor, type ToolExecutionResult, type ToolSchemaDefinition } from "./ToolTypes.js";
 /**
  * Projects a registered tool into its model-facing descriptor. Kept as a free
  * function so connector adapters can reuse the same defaulting rules.
  */
 export declare const toolDescriptorFor: (tool: ToolDefinition) => ToolDescriptor;
+export declare const sanitizeToolArguments: (value: unknown, schema: ToolSchemaDefinition | undefined) => {
+    value: unknown;
+    dropped: string[];
+};
 export declare class ToolRegistry {
     private tools;
     register(tool: ToolDefinition): void;

package/dist/tools/ToolRegistry.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"ToolRegistry.d.ts","sourceRoot":"","sources":["../../src/tools/ToolRegistry.ts"],"names":[],"mappings":"AAAA,OAAO,EAIL,KAAK,cAAc,EACnB,KAAK,WAAW,EAChB,KAAK,cAAc,EACnB,KAAK,cAAc,EAGnB,KAAK,mBAAmB,EAGzB,MAAM,gBAAgB,CAAC;AAExB;;;GAGG;AACH,eAAO,MAAM,iBAAiB,GAAI,MAAM,cAAc,KAAG,cAOvD,CAAC;AA4QH,qBAAa,YAAY;IACvB,OAAO,CAAC,KAAK,CAAqC;IAElD,QAAQ,CAAC,IAAI,EAAE,cAAc,GAAG,IAAI;IAOpC,IAAI,IAAI,cAAc,EAAE;IAIxB,QAAQ,IAAI,KAAK,CAAC,IAAI,CAAC,cAAc,EAAE,MAAM,GAAG,aAAa,GAAG,aAAa,CAAC,CAAC;IAQ/E;;;;OAIG;IACH,OAAO,CAAC,KAAK,CAAC,EAAE,SAAS,MAAM,EAAE,GAAG,cAAc,EAAE;IAOpD,2EAA2E;IAC3E,YAAY,CAAC,KAAK,CAAC,EAAE,SAAS,MAAM,EAAE,GAAG,GAAG,CAAC,cAAc,EAAE,cAAc,EAAE,CAAC;IAaxE,OAAO,CAAC,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,OAAO,EAAE,OAAO,EAAE,WAAW,GAAG,OAAO,CAAC,mBAAmB,CAAC;CAkC/F"}
+{"version":3,"file":"ToolRegistry.d.ts","sourceRoot":"","sources":["../../src/tools/ToolRegistry.ts"],"names":[],"mappings":"AAAA,OAAO,EAIL,KAAK,cAAc,EACnB,KAAK,WAAW,EAChB,KAAK,cAAc,EACnB,KAAK,cAAc,EAGnB,KAAK,mBAAmB,EACxB,KAAK,oBAAoB,EAE1B,MAAM,gBAAgB,CAAC;AAExB;;;GAGG;AACH,eAAO,MAAM,iBAAiB,GAAI,MAAM,cAAc,KAAG,cAOvD,CAAC;AA4HH,eAAO,MAAM,qBAAqB,GAChC,OAAO,OAAO,EACd,QAAQ,oBAAoB,GAAG,SAAS,KACvC;IAAE,KAAK,EAAE,OAAO,CAAC;IAAC,OAAO,EAAE,MAAM,EAAE,CAAA;CAwBrC,CAAC;AAyLF,qBAAa,YAAY;IACvB,OAAO,CAAC,KAAK,CAAqC;IAElD,QAAQ,CAAC,IAAI,EAAE,cAAc,GAAG,IAAI;IAOpC,IAAI,IAAI,cAAc,EAAE;IAIxB,QAAQ,IAAI,KAAK,CAAC,IAAI,CAAC,cAAc,EAAE,MAAM,GAAG,aAAa,GAAG,aAAa,CAAC,CAAC;IAQ/E;;;;OAIG;IACH,OAAO,CAAC,KAAK,CAAC,EAAE,SAAS,MAAM,EAAE,GAAG,cAAc,EAAE;IAOpD,2EAA2E;IAC3E,YAAY,CAAC,KAAK,CAAC,EAAE,SAAS,MAAM,EAAE,GAAG,GAAG,CAAC,cAAc,EAAE,cAAc,EAAE,CAAC;IAaxE,OAAO,CAAC,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,OAAO,EAAE,OAAO,EAAE,WAAW,GAAG,OAAO,CAAC,mBAAmB,CAAC;CAwC/F"}

package/dist/tools/ToolRegistry.js

@@ -79,6 +79,78 @@ const validateSchema = (schema, path = "$") => {
     }
     return undefined;
 };
+/**
+ * Repairs arguments that are nearly right before validating them.
+ *
+ * A worker gets one batch of tool calls, so a rejected call is not retried — it
+ * takes the whole sub-task down. Measured against the live suite, small models
+ * lost calls to two trivial mistakes: naming a field `depth` when the schema
+ * says `maxDepth`, and sending `"2"` where a number was wanted. Both were
+ * reported as `tool_invalid_args` and the question came back unanswered.
+ *
+ * Two repairs, both safe:
+ *
+ * - **Unknown properties are dropped.** The handler would ignore them anyway;
+ *   failing instead only converts a usable call into no call at all. The tool
+ *   still runs with its documented defaults, which beats returning nothing.
+ * - **Scalar strings are coerced** to the declared type when they parse
+ *   exactly. `"2"` becomes `2`; `"two"` stays a string and is still rejected.
+ *
+ * Nothing here weakens validation: a missing required argument, or a value that
+ * cannot be coerced, fails exactly as before.
+ */
+const coerceScalar = (value, schema) => {
+    if (typeof value !== "string")
+        return value;
+    const types = readSchemaTypes(schema);
+    if (!types || types.some((type) => type === "string"))
+        return value;
+    const trimmed = value.trim();
+    if (types.includes("number") || types.includes("integer")) {
+        const parsed = Number(trimmed);
+        if (trimmed !== "" && Number.isFinite(parsed)) {
+            if (types.includes("integer") && !Number.isInteger(parsed))
+                return value;
+            return parsed;
+        }
+    }
+    if (types.includes("boolean")) {
+        if (trimmed.toLowerCase() === "true")
+            return true;
+        if (trimmed.toLowerCase() === "false")
+            return false;
+    }
+    return value;
+};
+export const sanitizeToolArguments = (value, schema) => {
+    const dropped = [];
+    const walk = (input, node, path) => {
+        if (!node)
+            return input;
+        if (Array.isArray(input)) {
+            return node.items ? input.map((entry, index) => walk(entry, node.items, `${path}[${index}]`)) : input;
+        }
+        if (!isObject(input))
+            return coerceScalar(input, node);
+        const properties = node.properties;
+        if (!properties)
+            return input;
+        const output = {};
+        for (const [key, entry] of Object.entries(input)) {
+            const propertySchema = properties[key];
+            if (!propertySchema) {
+                if (node.additionalProperties === true)
+                    output[key] = entry;
+                else
+                    dropped.push(path ? `${path}.${key}` : key);
+                continue;
+            }
+            output[key] = walk(entry, propertySchema, path ? `${path}.${key}` : key);
+        }
+        return output;
+    };
+    return { value: walk(value, schema, ""), dropped };
+};
 const runtimeTypeOf = (value) => {
     if (value === null)
         return "null";
@@ -315,12 +387,17 @@ export class ToolRegistry {
         if (schemaError) {
             return { ok: false, output: "", error: schemaError };
         }
-        const validationError = tool.inputSchema ? validateValue(args, tool.inputSchema) : undefined;
+        const sanitized = tool.inputSchema
+            ? sanitizeToolArguments(args, tool.inputSchema)
+            : { value: args, dropped: [] };
+        const validationError = tool.inputSchema
+            ? validateValue(sanitized.value, tool.inputSchema)
+            : undefined;
         if (validationError) {
             return { ok: false, output: "", error: validationError };
         }
         try {
-            const result = await tool.handler(args, context);
+            const result = await tool.handler(sanitized.value, context);
             return { ok: true, output: result.output, data: result.data };
         }
         catch (error) {

package/dist/tools/docdex/DocdexTools.d.ts

@@ -1,5 +1,18 @@
 import type { ToolDefinition } from "../ToolTypes.js";
 import type { DocdexClient } from "../../docdex/DocdexClient.js";
+/**
+ * Rewrites a path into the repo-relative form docdex accepts.
+ *
+ * Docdex answers `invalid path` for a leading slash, a `./` prefix, or an
+ * absolute path, and a model asked to list a repository reaches for exactly
+ * those. That took out every "which files are in X" question: the tool was
+ * chosen correctly and then rejected on a formatting detail. Normalising here
+ * is the adapter's job — the caller meant the same directory either way.
+ *
+ * Returns undefined for "the whole repository", which is what docdex wants when
+ * no path is given.
+ */
+export declare const toRepoRelativePath: (value: string | undefined, workspaceRoot?: string) => string | undefined;
 /**
  * Judges whether search results actually answer the question.
  *

package/dist/tools/docdex/DocdexTools.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"DocdexTools.d.ts","sourceRoot":"","sources":["../../../src/tools/docdex/DocdexTools.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,iBAAiB,CAAC;AACtD,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,8BAA8B,CAAC;AAwEjE;;;;;;;;;;;;GAYG;AACH,MAAM,MAAM,cAAc,GAAG,CAAC,KAAK,EAAE;IACnC,KAAK,EAAE,MAAM,CAAC;IACd,OAAO,EAAE,OAAO,CAAC;CAClB,KAAK,OAAO,CAAC,OAAO,CAAC,CAAC;AAEvB,MAAM,WAAW,iBAAiB;IAChC,cAAc,CAAC,EAAE,cAAc,CAAC;CACjC;AAED,eAAO,MAAM,iBAAiB,GAC5B,QAAQ,YAAY,EACpB,UAAS,iBAAsB,KAC9B,cAAc,EAAgE,CAAC"}
+{"version":3,"file":"DocdexTools.d.ts","sourceRoot":"","sources":["../../../src/tools/docdex/DocdexTools.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,iBAAiB,CAAC;AACtD,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,8BAA8B,CAAC;AAuFjE;;;;;;;;;;;GAWG;AACH,eAAO,MAAM,kBAAkB,GAC7B,OAAO,MAAM,GAAG,SAAS,EACzB,gBAAgB,MAAM,KACrB,MAAM,GAAG,SAWX,CAAC;AAoHF;;;;;;;;;;;;GAYG;AACH,MAAM,MAAM,cAAc,GAAG,CAAC,KAAK,EAAE;IACnC,KAAK,EAAE,MAAM,CAAC;IACd,OAAO,EAAE,OAAO,CAAC;CAClB,KAAK,OAAO,CAAC,OAAO,CAAC,CAAC;AAEvB,MAAM,WAAW,iBAAiB;IAChC,cAAc,CAAC,EAAE,cAAc,CAAC;CACjC;AAED,eAAO,MAAM,iBAAiB,GAC5B,QAAQ,YAAY,EACpB,UAAS,iBAAsB,KAC9B,cAAc,EAAgE,CAAC"}