@mcpmesh/sdk 2.2.4 → 2.4.0

package/dist/__tests__/llm-max-iterations.test.js

@@ -0,0 +1,250 @@
+/**
+ * Unit tests for the configurable provider-managed agentic-loop cap — issue #1116.
+ *
+ * Two surfaces are covered:
+ *
+ *  1. Provider resolution (`resolveMaxIterations`): precedence of the
+ *     consumer-forwarded `model_params.max_iterations` over the
+ *     `MESH_LLM_MAX_ITERATIONS` env, the default of 10 when neither is set,
+ *     and the sanitization of invalid inputs.
+ *
+ *  2. Consumer forwarding (`MeshDelegatedProvider.complete()`): when the caller
+ *     passes `options.maxIterations`, it surfaces on the wire request as
+ *     `model_params.max_iterations`; when absent, no such key leaks.
+ *
+ * Parity note: the truncation marker stays the PLAIN content string
+ * "Maximum tool call iterations reached" (matching Python `mesh/helpers.py`);
+ * this PR does not introduce a structured marker.
+ */
+import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
+vi.mock("@mcpmesh/core", () => ({
+    generateTraceId: () => "trace-mock",
+    generateSpanId: () => "span-mock",
+    injectTraceContext: (argsJson) => argsJson,
+    publishSpan: vi.fn(async () => false),
+    parseSseResponse: (s) => s,
+    parseSseResponseToObject: (s) => JSON.parse(s),
+}));
+vi.mock("../http-pool.js", () => ({
+    getDispatcher: () => undefined,
+}));
+import { resolveMaxIterations, envMaxIterations, sanitizeMaxIterations, } from "../llm-provider.js";
+import { MeshDelegatedProvider } from "../llm-agent.js";
+// ----------------------------------------------------------------------------
+// resolveMaxIterations — provider-side precedence + sanitization
+// ----------------------------------------------------------------------------
+describe("resolveMaxIterations — provider loop cap resolution", () => {
+    const ORIGINAL_ENV = process.env.MESH_LLM_MAX_ITERATIONS;
+    afterEach(() => {
+        if (ORIGINAL_ENV === undefined) {
+            delete process.env.MESH_LLM_MAX_ITERATIONS;
+        }
+        else {
+            process.env.MESH_LLM_MAX_ITERATIONS = ORIGINAL_ENV;
+        }
+    });
+    it("model_params override wins over env", () => {
+        process.env.MESH_LLM_MAX_ITERATIONS = "7";
+        expect(resolveMaxIterations(25)).toBe(25);
+    });
+    it("falls back to env when the param is absent", () => {
+        process.env.MESH_LLM_MAX_ITERATIONS = "15";
+        expect(resolveMaxIterations(undefined)).toBe(15);
+    });
+    it("defaults to 10 when neither param nor env is set", () => {
+        delete process.env.MESH_LLM_MAX_ITERATIONS;
+        expect(resolveMaxIterations(undefined)).toBe(10);
+    });
+    it("accepts a numeric string from the env", () => {
+        delete process.env.MESH_LLM_MAX_ITERATIONS;
+        process.env.MESH_LLM_MAX_ITERATIONS = "3";
+        expect(resolveMaxIterations(undefined)).toBe(3);
+    });
+    it("floors a fractional value to an integer", () => {
+        delete process.env.MESH_LLM_MAX_ITERATIONS;
+        expect(resolveMaxIterations(4.9)).toBe(4);
+    });
+    // #1116 fractional hole: floor-BEFORE-validate. 0.5 floors to 0, which is
+    // not > 0, so it must fall back to the default — NOT return a zero cap that
+    // would disable the loop.
+    it("falls back to 10 for a fractional value that floors to zero (0.5)", () => {
+        delete process.env.MESH_LLM_MAX_ITERATIONS;
+        expect(resolveMaxIterations(0.5)).toBe(10);
+    });
+    it("falls back to 10 for a fractional value that floors to zero (0.9)", () => {
+        delete process.env.MESH_LLM_MAX_ITERATIONS;
+        expect(resolveMaxIterations(0.9)).toBe(10);
+    });
+    it.each([
+        ["zero", 0],
+        ["negative", -5],
+        ["NaN", NaN],
+        ["non-numeric string", "abc"],
+        ["null", null],
+        ["object", {}],
+    ])("falls back to 10 for an invalid value (%s)", (_label, value) => {
+        delete process.env.MESH_LLM_MAX_ITERATIONS;
+        expect(resolveMaxIterations(value)).toBe(10);
+    });
+    it("an invalid env falls back to 10", () => {
+        process.env.MESH_LLM_MAX_ITERATIONS = "not-a-number";
+        expect(resolveMaxIterations(undefined)).toBe(10);
+    });
+    // Parity (#1116): the Gemini AI-SDK-managed loop sets
+    // `requestOptions.maxSteps = resolvedMaxIterations`, the SAME value the
+    // manual provider-managed loop uses as its cap. We assert the resolution
+    // the Gemini path consumes here rather than the wired `maxSteps`, since
+    // observing the latter would require a full `vi.mock("ai")` generateText
+    // harness that this file deliberately does not build.
+    it("Gemini maxSteps path consumes the forwarded resolved cap", () => {
+        delete process.env.MESH_LLM_MAX_ITERATIONS;
+        expect(resolveMaxIterations(25)).toBe(25);
+    });
+    it("Gemini maxSteps path falls back to 10 when the cap is absent", () => {
+        delete process.env.MESH_LLM_MAX_ITERATIONS;
+        expect(resolveMaxIterations(undefined)).toBe(10);
+    });
+});
+// ----------------------------------------------------------------------------
+// envMaxIterations — single source of truth for env parsing
+// ----------------------------------------------------------------------------
+describe("envMaxIterations — env parse + validation", () => {
+    const ORIGINAL_ENV = process.env.MESH_LLM_MAX_ITERATIONS;
+    afterEach(() => {
+        if (ORIGINAL_ENV === undefined) {
+            delete process.env.MESH_LLM_MAX_ITERATIONS;
+        }
+        else {
+            process.env.MESH_LLM_MAX_ITERATIONS = ORIGINAL_ENV;
+        }
+    });
+    it("returns undefined when the env is unset", () => {
+        delete process.env.MESH_LLM_MAX_ITERATIONS;
+        expect(envMaxIterations()).toBeUndefined();
+    });
+    it.each([
+        ["zero", "0", undefined],
+        ["empty string", "", undefined],
+        ["non-numeric", "abc", undefined],
+        ["fractional below one", "0.5", undefined],
+        ["positive integer", "15", 15],
+        ["fractional floors down", "4.9", 4],
+        ["scientific notation (Number semantics)", "1e2", 100],
+    ])("parses %s → %s", (_label, raw, expected) => {
+        process.env.MESH_LLM_MAX_ITERATIONS = raw;
+        expect(envMaxIterations()).toBe(expected);
+    });
+});
+// ----------------------------------------------------------------------------
+// sanitizeMaxIterations — single source of truth for cap normalization
+// ----------------------------------------------------------------------------
+describe("sanitizeMaxIterations — value normalization", () => {
+    it.each([
+        ["positive integer", 5, 5],
+        ["fractional floors down", 4.9, 4],
+        ["numeric string", "15", 15],
+        ["scientific notation (Number semantics)", "1e2", 100],
+    ])("normalizes a valid value (%s) → %s", (_label, value, expected) => {
+        expect(sanitizeMaxIterations(value)).toBe(expected);
+    });
+    it.each([
+        ["undefined", undefined],
+        ["null", null],
+        ["zero", 0],
+        ["negative", -3],
+        ["fractional below one", 0.5],
+        ["NaN", NaN],
+        ["non-numeric string", "abc"],
+        ["empty string", ""],
+        ["object", {}],
+    ])("rejects an invalid value (%s) → undefined", (_label, value) => {
+        expect(sanitizeMaxIterations(value)).toBeUndefined();
+    });
+});
+// ----------------------------------------------------------------------------
+// MeshDelegatedProvider.complete() — consumer forwards maxIterations
+// ----------------------------------------------------------------------------
+const ENDPOINT = "http://provider.local:9001";
+const FN_BUFFERED = "process_chat";
+describe("MeshDelegatedProvider.complete() — maxIterations forwarding", () => {
+    let originalFetch;
+    beforeEach(() => {
+        originalFetch = globalThis.fetch;
+    });
+    afterEach(() => {
+        globalThis.fetch = originalFetch;
+        vi.restoreAllMocks();
+    });
+    function mockJsonResponse(body) {
+        return {
+            ok: true,
+            status: 200,
+            statusText: "OK",
+            headers: {
+                get: (name) => name.toLowerCase() === "content-type" ? "application/json" : null,
+            },
+            text: async () => JSON.stringify(body),
+            json: async () => body,
+        };
+    }
+    function mcpToolResponse(payload) {
+        return {
+            jsonrpc: "2.0",
+            id: 1,
+            result: {
+                content: [{ type: "text", text: JSON.stringify(payload) }],
+            },
+        };
+    }
+    const STUB_COMPLETION = { role: "assistant", content: "ok" };
+    it("forwards options.maxIterations as model_params.max_iterations", async () => {
+        let capturedBody;
+        const fetchMock = vi.fn(async (_url, init) => {
+            capturedBody = init.body;
+            return mockJsonResponse(mcpToolResponse(STUB_COMPLETION));
+        });
+        globalThis.fetch = fetchMock;
+        const provider = new MeshDelegatedProvider(ENDPOINT, FN_BUFFERED, false);
+        const messages = [{ role: "user", content: "hi" }];
+        await provider.complete("anthropic/claude-sonnet-4-5", messages, undefined, {
+            maxIterations: 25,
+        });
+        const body = JSON.parse(capturedBody);
+        const modelParams = body.params.arguments.request.model_params;
+        expect(modelParams.max_iterations).toBe(25);
+    });
+    it("typed maxIterations wins over an escape-hatch modelParams.max_iterations", async () => {
+        let capturedBody;
+        const fetchMock = vi.fn(async (_url, init) => {
+            capturedBody = init.body;
+            return mockJsonResponse(mcpToolResponse(STUB_COMPLETION));
+        });
+        globalThis.fetch = fetchMock;
+        const provider = new MeshDelegatedProvider(ENDPOINT, FN_BUFFERED, false);
+        const messages = [{ role: "user", content: "hi" }];
+        await provider.complete("anthropic/claude-sonnet-4-5", messages, undefined, {
+            maxIterations: 25,
+            modelParams: { max_iterations: 3 },
+        });
+        const body = JSON.parse(capturedBody);
+        const modelParams = body.params.arguments.request.model_params;
+        expect(modelParams.max_iterations).toBe(25);
+    });
+    it("does not emit max_iterations when maxIterations is absent", async () => {
+        let capturedBody;
+        const fetchMock = vi.fn(async (_url, init) => {
+            capturedBody = init.body;
+            return mockJsonResponse(mcpToolResponse(STUB_COMPLETION));
+        });
+        globalThis.fetch = fetchMock;
+        const provider = new MeshDelegatedProvider(ENDPOINT, FN_BUFFERED, false);
+        const messages = [{ role: "user", content: "hi" }];
+        await provider.complete("anthropic/claude-sonnet-4-5", messages, undefined, {
+            maxOutputTokens: 256,
+        });
+        const body = JSON.parse(capturedBody);
+        const modelParams = body.params.arguments.request.model_params;
+        expect(modelParams.max_iterations).toBeUndefined();
+    });
+});
+//# sourceMappingURL=llm-max-iterations.test.js.map

package/dist/__tests__/llm-max-iterations.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"llm-max-iterations.test.js","sourceRoot":"","sources":["../../src/__tests__/llm-max-iterations.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;GAiBG;AAEH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,UAAU,EAAE,SAAS,EAAE,MAAM,QAAQ,CAAC;AAEzE,EAAE,CAAC,IAAI,CAAC,eAAe,EAAE,GAAG,EAAE,CAAC,CAAC;IAC9B,eAAe,EAAE,GAAG,EAAE,CAAC,YAAY;IACnC,cAAc,EAAE,GAAG,EAAE,CAAC,WAAW;IACjC,kBAAkB,EAAE,CAAC,QAAgB,EAAE,EAAE,CAAC,QAAQ;IAClD,WAAW,EAAE,EAAE,CAAC,EAAE,CAAC,KAAK,IAAI,EAAE,CAAC,KAAK,CAAC;IACrC,gBAAgB,EAAE,CAAC,CAAS,EAAE,EAAE,CAAC,CAAC;IAClC,wBAAwB,EAAE,CAAC,CAAS,EAAE,EAAE,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC;CACvD,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,iBAAiB,EAAE,GAAG,EAAE,CAAC,CAAC;IAChC,aAAa,EAAE,GAAG,EAAE,CAAC,SAAS;CAC/B,CAAC,CAAC,CAAC;AAEJ,OAAO,EACL,oBAAoB,EACpB,gBAAgB,EAChB,qBAAqB,GACtB,MAAM,oBAAoB,CAAC;AAC5B,OAAO,EAAE,qBAAqB,EAAE,MAAM,iBAAiB,CAAC;AAGxD,+EAA+E;AAC/E,iEAAiE;AACjE,+EAA+E;AAE/E,QAAQ,CAAC,qDAAqD,EAAE,GAAG,EAAE;IACnE,MAAM,YAAY,GAAG,OAAO,CAAC,GAAG,CAAC,uBAAuB,CAAC;IAEzD,SAAS,CAAC,GAAG,EAAE;QACb,IAAI,YAAY,KAAK,SAAS,EAAE,CAAC;YAC/B,OAAO,OAAO,CAAC,GAAG,CAAC,uBAAuB,CAAC;QAC7C,CAAC;aAAM,CAAC;YACN,OAAO,CAAC,GAAG,CAAC,uBAAuB,GAAG,YAAY,CAAC;QACrD,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qCAAqC,EAAE,GAAG,EAAE;QAC7C,OAAO,CAAC,GAAG,CAAC,uBAAuB,GAAG,GAAG,CAAC;QAC1C,MAAM,CAAC,oBAAoB,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IAC5C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4CAA4C,EAAE,GAAG,EAAE;QACpD,OAAO,CAAC,GAAG,CAAC,uBAAuB,GAAG,IAAI,CAAC;QAC3C,MAAM,CAAC,oBAAoB,CAAC,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACnD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kDAAkD,EAAE,GAAG,EAAE;QAC1D,OAAO,OAAO,CAAC,GAAG,CAAC,uBAAuB,CAAC;QAC3C,MAAM,CAAC,oBAAoB,CAAC,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACnD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uCAAuC,EAAE,GAAG,EAAE;QAC/C,OAAO,OAAO,CAAC,GAAG,CAAC,uBAAuB,CAAC;QAC3C,OAAO,CAAC,GAAG,CAAC,uBAAuB,GAAG,GAAG,CAAC;QAC1C,MAAM,CAAC,oBAAoB,CAAC,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAClD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yCAAyC,EAAE,GAAG,EAAE;QACjD,OAAO,OAAO,CAAC,GAAG,CAAC,uBAAuB,CAAC;QAC3C,MAAM,CAAC,oBAAoB,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAC5C,CAAC,CAAC,CAAC;IAEH,0EAA0E;IAC1E,4EAA4E;IAC5E,0BAA0B;IAC1B,EAAE,CAAC,mEAAmE,EAAE,GAAG,EAAE;QAC3E,OAAO,OAAO,CAAC,GAAG,CAAC,uBAAuB,CAAC;QAC3C,MAAM,CAAC,oBAAoB,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IAC7C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mEAAmE,EAAE,GAAG,EAAE;QAC3E,OAAO,OAAO,CAAC,GAAG,CAAC,uBAAuB,CAAC;QAC3C,MAAM,CAAC,oBAAoB,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IAC7C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,IAAI,CAAC;QACN,CAAC,MAAM,EAAE,CAAC,CAAC;QACX,CAAC,UAAU,EAAE,CAAC,CAAC,CAAC;QAChB,CAAC,KAAK,EAAE,GAAG,CAAC;QACZ,CAAC,oBAAoB,EAAE,KAA0B,CAAC;QAClD,CAAC,MAAM,EAAE,IAAyB,CAAC;QACnC,CAAC,QAAQ,EAAE,EAAuB,CAAC;KACpC,CAAC,CAAC,4CAA4C,EAAE,CAAC,MAAM,EAAE,KAAK,EAAE,EAAE;QACjE,OAAO,OAAO,CAAC,GAAG,CAAC,uBAAuB,CAAC;QAC3C,MAAM,CAAC,oBAAoB,CAAC,KAAK,CAAC,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IAC/C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iCAAiC,EAAE,GAAG,EAAE;QACzC,OAAO,CAAC,GAAG,CAAC,uBAAuB,GAAG,cAAc,CAAC;QACrD,MAAM,CAAC,oBAAoB,CAAC,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACnD,CAAC,CAAC,CAAC;IAEH,sDAAsD;IACtD,wEAAwE;IACxE,yEAAyE;IACzE,wEAAwE;IACxE,yEAAyE;IACzE,sDAAsD;IACtD,EAAE,CAAC,0DAA0D,EAAE,GAAG,EAAE;QAClE,OAAO,OAAO,CAAC,GAAG,CAAC,uBAAuB,CAAC;QAC3C,MAAM,CAAC,oBAAoB,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IAC5C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8DAA8D,EAAE,GAAG,EAAE;QACtE,OAAO,OAAO,CAAC,GAAG,CAAC,uBAAuB,CAAC;QAC3C,MAAM,CAAC,oBAAoB,CAAC,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACnD,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,+EAA+E;AAC/E,4DAA4D;AAC5D,+EAA+E;AAE/E,QAAQ,CAAC,2CAA2C,EAAE,GAAG,EAAE;IACzD,MAAM,YAAY,GAAG,OAAO,CAAC,GAAG,CAAC,uBAAuB,CAAC;IAEzD,SAAS,CAAC,GAAG,EAAE;QACb,IAAI,YAAY,KAAK,SAAS,EAAE,CAAC;YAC/B,OAAO,OAAO,CAAC,GAAG,CAAC,uBAAuB,CAAC;QAC7C,CAAC;aAAM,CAAC;YACN,OAAO,CAAC,GAAG,CAAC,uBAAuB,GAAG,YAAY,CAAC;QACrD,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yCAAyC,EAAE,GAAG,EAAE;QACjD,OAAO,OAAO,CAAC,GAAG,CAAC,uBAAuB,CAAC;QAC3C,MAAM,CAAC,gBAAgB,EAAE,CAAC,CAAC,aAAa,EAAE,CAAC;IAC7C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,IAAI,CAAC;QACN,CAAC,MAAM,EAAE,GAAG,EAAE,SAAS,CAAC;QACxB,CAAC,cAAc,EAAE,EAAE,EAAE,SAAS,CAAC;QAC/B,CAAC,aAAa,EAAE,KAAK,EAAE,SAAS,CAAC;QACjC,CAAC,sBAAsB,EAAE,KAAK,EAAE,SAAS,CAAC;QAC1C,CAAC,kBAAkB,EAAE,IAAI,EAAE,EAAE,CAAC;QAC9B,CAAC,wBAAwB,EAAE,KAAK,EAAE,CAAC,CAAC;QACpC,CAAC,wCAAwC,EAAE,KAAK,EAAE,GAAG,CAAC;KACvD,CAAC,CAAC,gBAAgB,EAAE,CAAC,MAAM,EAAE,GAAG,EAAE,QAAQ,EAAE,EAAE;QAC7C,OAAO,CAAC,GAAG,CAAC,uBAAuB,GAAG,GAAG,CAAC;QAC1C,MAAM,CAAC,gBAAgB,EAAE,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;IAC5C,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,+EAA+E;AAC/E,uEAAuE;AACvE,+EAA+E;AAE/E,QAAQ,CAAC,6CAA6C,EAAE,GAAG,EAAE;IAC3D,EAAE,CAAC,IAAI,CAAC;QACN,CAAC,kBAAkB,EAAE,CAAC,EAAE,CAAC,CAAC;QAC1B,CAAC,wBAAwB,EAAE,GAAG,EAAE,CAAC,CAAC;QAClC,CAAC,gBAAgB,EAAE,IAAI,EAAE,EAAE,CAAC;QAC5B,CAAC,wCAAwC,EAAE,KAAK,EAAE,GAAG,CAAC;KACvD,CAAC,CAAC,oCAAoC,EAAE,CAAC,MAAM,EAAE,KAAK,EAAE,QAAQ,EAAE,EAAE;QACnE,MAAM,CAAC,qBAAqB,CAAC,KAAK,CAAC,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;IACtD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,IAAI,CAAC;QACN,CAAC,WAAW,EAAE,SAAS,CAAC;QACxB,CAAC,MAAM,EAAE,IAAI,CAAC;QACd,CAAC,MAAM,EAAE,CAAC,CAAC;QACX,CAAC,UAAU,EAAE,CAAC,CAAC,CAAC;QAChB,CAAC,sBAAsB,EAAE,GAAG,CAAC;QAC7B,CAAC,KAAK,EAAE,GAAG,CAAC;QACZ,CAAC,oBAAoB,EAAE,KAAK,CAAC;QAC7B,CAAC,cAAc,EAAE,EAAE,CAAC;QACpB,CAAC,QAAQ,EAAE,EAAE,CAAC;KACf,CAAC,CAAC,2CAA2C,EAAE,CAAC,MAAM,EAAE,KAAK,EAAE,EAAE;QAChE,MAAM,CAAC,qBAAqB,CAAC,KAAK,CAAC,CAAC,CAAC,aAAa,EAAE,CAAC;IACvD,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,+EAA+E;AAC/E,qEAAqE;AACrE,+EAA+E;AAE/E,MAAM,QAAQ,GAAG,4BAA4B,CAAC;AAC9C,MAAM,WAAW,GAAG,cAAc,CAAC;AAEnC,QAAQ,CAAC,6DAA6D,EAAE,GAAG,EAAE;IAC3E,IAAI,aAA2B,CAAC;IAEhC,UAAU,CAAC,GAAG,EAAE;QACd,aAAa,GAAG,UAAU,CAAC,KAAK,CAAC;IACnC,CAAC,CAAC,CAAC;IAEH,SAAS,CAAC,GAAG,EAAE;QACb,UAAU,CAAC,KAAK,GAAG,aAAa,CAAC;QACjC,EAAE,CAAC,eAAe,EAAE,CAAC;IACvB,CAAC,CAAC,CAAC;IAEH,SAAS,gBAAgB,CAAC,IAAY;QACpC,OAAO;YACL,EAAE,EAAE,IAAI;YACR,MAAM,EAAE,GAAG;YACX,UAAU,EAAE,IAAI;YAChB,OAAO,EAAE;gBACP,GAAG,EAAE,CAAC,IAAY,EAAE,EAAE,CACpB,IAAI,CAAC,WAAW,EAAE,KAAK,cAAc,CAAC,CAAC,CAAC,kBAAkB,CAAC,CAAC,CAAC,IAAI;aACpE;YACD,IAAI,EAAE,KAAK,IAAI,EAAE,CAAC,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC;YACtC,IAAI,EAAE,KAAK,IAAI,EAAE,CAAC,IAAI;SACA,CAAC;IAC3B,CAAC;IAED,SAAS,eAAe,CAAC,OAAe;QACtC,OAAO;YACL,OAAO,EAAE,KAAK;YACd,EAAE,EAAE,CAAC;YACL,MAAM,EAAE;gBACN,OAAO,EAAE,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,IAAI,CAAC,SAAS,CAAC,OAAO,CAAC,EAAE,CAAC;aAC3D;SACF,CAAC;IACJ,CAAC;IAED,MAAM,eAAe,GAAG,EAAE,IAAI,EAAE,WAAW,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC;IAE7D,EAAE,CAAC,+DAA+D,EAAE,KAAK,IAAI,EAAE;QAC7E,IAAI,YAAgC,CAAC;QACrC,MAAM,SAAS,GAAG,EAAE,CAAC,EAAE,CAAC,KAAK,EAAE,IAAY,EAAE,IAAiB,EAAE,EAAE;YAChE,YAAY,GAAG,IAAI,CAAC,IAAc,CAAC;YACnC,OAAO,gBAAgB,CAAC,eAAe,CAAC,eAAe,CAAC,CAAC,CAAC;QAC5D,CAAC,CAAC,CAAC;QACH,UAAU,CAAC,KAAK,GAAG,SAAoC,CAAC;QAExD,MAAM,QAAQ,GAAG,IAAI,qBAAqB,CAAC,QAAQ,EAAE,WAAW,EAAE,KAAK,CAAC,CAAC;QACzE,MAAM,QAAQ,GAAiB,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC,CAAC;QAEjE,MAAM,QAAQ,CAAC,QAAQ,CAAC,6BAA6B,EAAE,QAAQ,EAAE,SAAS,EAAE;YAC1E,aAAa,EAAE,EAAE;SAClB,CAAC,CAAC;QAEH,MAAM,IAAI,GAAG,IAAI,CAAC,KAAK,CAAC,YAAa,CAAC,CAAC;QACvC,MAAM,WAAW,GAAG,IAAI,CAAC,MAAM,CAAC,SAAS,CAAC,OAAO,CAAC,YAAuC,CAAC;QAC1F,MAAM,CAAC,WAAW,CAAC,cAAc,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IAC9C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,0EAA0E,EAAE,KAAK,IAAI,EAAE;QACxF,IAAI,YAAgC,CAAC;QACrC,MAAM,SAAS,GAAG,EAAE,CAAC,EAAE,CAAC,KAAK,EAAE,IAAY,EAAE,IAAiB,EAAE,EAAE;YAChE,YAAY,GAAG,IAAI,CAAC,IAAc,CAAC;YACnC,OAAO,gBAAgB,CAAC,eAAe,CAAC,eAAe,CAAC,CAAC,CAAC;QAC5D,CAAC,CAAC,CAAC;QACH,UAAU,CAAC,KAAK,GAAG,SAAoC,CAAC;QAExD,MAAM,QAAQ,GAAG,IAAI,qBAAqB,CAAC,QAAQ,EAAE,WAAW,EAAE,KAAK,CAAC,CAAC;QACzE,MAAM,QAAQ,GAAiB,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC,CAAC;QAEjE,MAAM,QAAQ,CAAC,QAAQ,CAAC,6BAA6B,EAAE,QAAQ,EAAE,SAAS,EAAE;YAC1E,aAAa,EAAE,EAAE;YACjB,WAAW,EAAE,EAAE,cAAc,EAAE,CAAC,EAAE;SACnC,CAAC,CAAC;QAEH,MAAM,IAAI,GAAG,IAAI,CAAC,KAAK,CAAC,YAAa,CAAC,CAAC;QACvC,MAAM,WAAW,GAAG,IAAI,CAAC,MAAM,CAAC,SAAS,CAAC,OAAO,CAAC,YAAuC,CAAC;QAC1F,MAAM,CAAC,WAAW,CAAC,cAAc,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IAC9C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2DAA2D,EAAE,KAAK,IAAI,EAAE;QACzE,IAAI,YAAgC,CAAC;QACrC,MAAM,SAAS,GAAG,EAAE,CAAC,EAAE,CAAC,KAAK,EAAE,IAAY,EAAE,IAAiB,EAAE,EAAE;YAChE,YAAY,GAAG,IAAI,CAAC,IAAc,CAAC;YACnC,OAAO,gBAAgB,CAAC,eAAe,CAAC,eAAe,CAAC,CAAC,CAAC;QAC5D,CAAC,CAAC,CAAC;QACH,UAAU,CAAC,KAAK,GAAG,SAAoC,CAAC;QAExD,MAAM,QAAQ,GAAG,IAAI,qBAAqB,CAAC,QAAQ,EAAE,WAAW,EAAE,KAAK,CAAC,CAAC;QACzE,MAAM,QAAQ,GAAiB,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC,CAAC;QAEjE,MAAM,QAAQ,CAAC,QAAQ,CAAC,6BAA6B,EAAE,QAAQ,EAAE,SAAS,EAAE;YAC1E,eAAe,EAAE,GAAG;SACrB,CAAC,CAAC;QAEH,MAAM,IAAI,GAAG,IAAI,CAAC,KAAK,CAAC,YAAa,CAAC,CAAC;QACvC,MAAM,WAAW,GAAG,IAAI,CAAC,MAAM,CAAC,SAAS,CAAC,OAAO,CAAC,YAAuC,CAAC;QAC1F,MAAM,CAAC,WAAW,CAAC,cAAc,CAAC,CAAC,aAAa,EAAE,CAAC;IACrD,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/__tests__/llm-mesh-error-mapping.test.d.ts

@@ -0,0 +1,16 @@
+/**
+ * Unit tests for #1116 item-2 regression fixes: routing TS LLM calls through
+ * the shared callMcpTool must preserve the behavior the hand-rolled paths had.
+ *
+ * Covered:
+ *  - Fix 1: MeshDelegatedProvider.complete() maps a timeout to LLMAPIError(408).
+ *           callMcpTool re-throws the AbortError as a plain Error ("MCP call
+ *           timed out after <N>ms"), so complete() now matches on the message.
+ *  - Fix 2: callMcpTool surfaces a tool-level isError result by throwing, so
+ *           complete() re-wraps it into LLMAPIError and createLlmToolProxy into
+ *           ToolExecutionError instead of returning the error text as success.
+ *  - Fix 3: createLlmToolProxy returns null (not "") when the tool returns no
+ *           content.
+ */
+export {};
+//# sourceMappingURL=llm-mesh-error-mapping.test.d.ts.map

package/dist/__tests__/llm-mesh-error-mapping.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"llm-mesh-error-mapping.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/llm-mesh-error-mapping.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;GAaG"}

package/dist/__tests__/llm-mesh-error-mapping.test.js

@@ -0,0 +1,135 @@
+/**
+ * Unit tests for #1116 item-2 regression fixes: routing TS LLM calls through
+ * the shared callMcpTool must preserve the behavior the hand-rolled paths had.
+ *
+ * Covered:
+ *  - Fix 1: MeshDelegatedProvider.complete() maps a timeout to LLMAPIError(408).
+ *           callMcpTool re-throws the AbortError as a plain Error ("MCP call
+ *           timed out after <N>ms"), so complete() now matches on the message.
+ *  - Fix 2: callMcpTool surfaces a tool-level isError result by throwing, so
+ *           complete() re-wraps it into LLMAPIError and createLlmToolProxy into
+ *           ToolExecutionError instead of returning the error text as success.
+ *  - Fix 3: createLlmToolProxy returns null (not "") when the tool returns no
+ *           content.
+ */
+import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
+import { MeshDelegatedProvider, createLlmToolProxy } from "../llm-agent.js";
+import { LLMAPIError, ToolExecutionError } from "../errors.js";
+vi.mock("@mcpmesh/core", () => ({
+    generateTraceId: () => "trace-mock",
+    generateSpanId: () => "span-mock",
+    injectTraceContext: (argsJson) => argsJson,
+    publishSpan: vi.fn(async () => false),
+    parseSseResponse: (s) => s,
+    parseSseResponseToObject: (s) => JSON.parse(s),
+    awaitJobCancel: vi.fn(() => new Promise(() => { })),
+    matchesPropagateHeader: () => false,
+}));
+vi.mock("../http-pool.js", () => ({
+    getDispatcher: () => undefined,
+}));
+const ENDPOINT = "http://provider.local:9000";
+const FUNCTION = "mesh_complete";
+function jsonResponse(result) {
+    const body = JSON.stringify({ jsonrpc: "2.0", id: "x", result });
+    return {
+        ok: true,
+        status: 200,
+        statusText: "OK",
+        text: async () => body,
+        headers: {
+            get: (name) => name.toLowerCase() === "content-type" ? "application/json" : null,
+        },
+    };
+}
+describe("MeshDelegatedProvider.complete() error mapping (#1116 item-2)", () => {
+    let originalFetch;
+    beforeEach(() => {
+        originalFetch = globalThis.fetch;
+    });
+    afterEach(() => {
+        globalThis.fetch = originalFetch;
+        vi.restoreAllMocks();
+        delete process.env.MESH_PROVIDER_TIMEOUT_MS;
+    });
+    it("Fix 1: maps a timeout to LLMAPIError with status 408", async () => {
+        process.env.MESH_PROVIDER_TIMEOUT_MS = "10";
+        // fetch hangs until aborted; reject with AbortError when the signal fires.
+        const fetchMock = vi.fn((_url, init) => {
+            const signal = init.signal;
+            return new Promise((_resolve, reject) => {
+                signal.addEventListener("abort", () => {
+                    const err = new Error("aborted");
+                    err.name = "AbortError";
+                    reject(err);
+                });
+            });
+        });
+        globalThis.fetch = fetchMock;
+        const provider = new MeshDelegatedProvider(ENDPOINT, FUNCTION);
+        const err = await provider
+            .complete("mesh-delegated", [{ role: "user", content: "hi" }])
+            .then(() => null)
+            .catch((e) => e);
+        expect(err).toBeInstanceOf(LLMAPIError);
+        expect(err.statusCode).toBe(408);
+        expect(err.message).toMatch(/timed out/i);
+    });
+    it("Fix 2: maps a tool-level isError result to LLMAPIError(0)", async () => {
+        const fetchMock = vi.fn(async () => jsonResponse({
+            isError: true,
+            content: [{ type: "text", text: "provider blew up" }],
+        }));
+        globalThis.fetch = fetchMock;
+        const provider = new MeshDelegatedProvider(ENDPOINT, FUNCTION);
+        const err = await provider
+            .complete("mesh-delegated", [{ role: "user", content: "hi" }])
+            .then(() => null)
+            .catch((e) => e);
+        expect(err).toBeInstanceOf(LLMAPIError);
+        expect(err.statusCode).toBe(0);
+        expect(err.message).toContain("provider blew up");
+    });
+});
+describe("createLlmToolProxy error/empty mapping (#1116 item-2)", () => {
+    let originalFetch;
+    const toolInfo = {
+        functionName: "do_thing",
+        capability: "doer",
+        endpoint: ENDPOINT,
+        agentId: "agent-1",
+    };
+    beforeEach(() => {
+        originalFetch = globalThis.fetch;
+    });
+    afterEach(() => {
+        globalThis.fetch = originalFetch;
+        vi.restoreAllMocks();
+    });
+    it("Fix 2: maps a tool-level isError result to ToolExecutionError", async () => {
+        const fetchMock = vi.fn(async () => jsonResponse({
+            isError: true,
+            content: [{ type: "text", text: "tool failed" }],
+        }));
+        globalThis.fetch = fetchMock;
+        const proxy = createLlmToolProxy(toolInfo);
+        const err = await proxy({}).then(() => null).catch((e) => e);
+        expect(err).toBeInstanceOf(ToolExecutionError);
+        expect(err.message).toContain("tool failed");
+    });
+    it("Fix 3: returns null when the tool returns no content", async () => {
+        const fetchMock = vi.fn(async () => jsonResponse({ content: [] }));
+        globalThis.fetch = fetchMock;
+        const proxy = createLlmToolProxy(toolInfo);
+        const result = await proxy({});
+        expect(result).toBeNull();
+    });
+    it("returns parsed JSON for non-empty content", async () => {
+        const fetchMock = vi.fn(async () => jsonResponse({ content: [{ type: "text", text: '{"ok":true}' }] }));
+        globalThis.fetch = fetchMock;
+        const proxy = createLlmToolProxy(toolInfo);
+        const result = await proxy({});
+        expect(result).toEqual({ ok: true });
+    });
+});
+//# sourceMappingURL=llm-mesh-error-mapping.test.js.map

package/dist/__tests__/llm-mesh-error-mapping.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"llm-mesh-error-mapping.test.js","sourceRoot":"","sources":["../../src/__tests__/llm-mesh-error-mapping.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;GAaG;AAEH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,UAAU,EAAE,SAAS,EAAE,MAAM,QAAQ,CAAC;AACzE,OAAO,EAAE,qBAAqB,EAAE,kBAAkB,EAAE,MAAM,iBAAiB,CAAC;AAC5E,OAAO,EAAE,WAAW,EAAE,kBAAkB,EAAE,MAAM,cAAc,CAAC;AAE/D,EAAE,CAAC,IAAI,CAAC,eAAe,EAAE,GAAG,EAAE,CAAC,CAAC;IAC9B,eAAe,EAAE,GAAG,EAAE,CAAC,YAAY;IACnC,cAAc,EAAE,GAAG,EAAE,CAAC,WAAW;IACjC,kBAAkB,EAAE,CAAC,QAAgB,EAAE,EAAE,CAAC,QAAQ;IAClD,WAAW,EAAE,EAAE,CAAC,EAAE,CAAC,KAAK,IAAI,EAAE,CAAC,KAAK,CAAC;IACrC,gBAAgB,EAAE,CAAC,CAAS,EAAE,EAAE,CAAC,CAAC;IAClC,wBAAwB,EAAE,CAAC,CAAS,EAAE,EAAE,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC;IACtD,cAAc,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,IAAI,OAAO,CAAO,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;IACxD,sBAAsB,EAAE,GAAG,EAAE,CAAC,KAAK;CACpC,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,iBAAiB,EAAE,GAAG,EAAE,CAAC,CAAC;IAChC,aAAa,EAAE,GAAG,EAAE,CAAC,SAAS;CAC/B,CAAC,CAAC,CAAC;AAEJ,MAAM,QAAQ,GAAG,4BAA4B,CAAC;AAC9C,MAAM,QAAQ,GAAG,eAAe,CAAC;AAEjC,SAAS,YAAY,CAAC,MAAe;IACnC,MAAM,IAAI,GAAG,IAAI,CAAC,SAAS,CAAC,EAAE,OAAO,EAAE,KAAK,EAAE,EAAE,EAAE,GAAG,EAAE,MAAM,EAAE,CAAC,CAAC;IACjE,OAAO;QACL,EAAE,EAAE,IAAI;QACR,MAAM,EAAE,GAAG;QACX,UAAU,EAAE,IAAI;QAChB,IAAI,EAAE,KAAK,IAAI,EAAE,CAAC,IAAI;QACtB,OAAO,EAAE;YACP,GAAG,EAAE,CAAC,IAAY,EAAE,EAAE,CACpB,IAAI,CAAC,WAAW,EAAE,KAAK,cAAc,CAAC,CAAC,CAAC,kBAAkB,CAAC,CAAC,CAAC,IAAI;SACpE;KACqB,CAAC;AAC3B,CAAC;AAED,QAAQ,CAAC,+DAA+D,EAAE,GAAG,EAAE;IAC7E,IAAI,aAA2B,CAAC;IAEhC,UAAU,CAAC,GAAG,EAAE;QACd,aAAa,GAAG,UAAU,CAAC,KAAK,CAAC;IACnC,CAAC,CAAC,CAAC;IAEH,SAAS,CAAC,GAAG,EAAE;QACb,UAAU,CAAC,KAAK,GAAG,aAAa,CAAC;QACjC,EAAE,CAAC,eAAe,EAAE,CAAC;QACrB,OAAO,OAAO,CAAC,GAAG,CAAC,wBAAwB,CAAC;IAC9C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sDAAsD,EAAE,KAAK,IAAI,EAAE;QACpE,OAAO,CAAC,GAAG,CAAC,wBAAwB,GAAG,IAAI,CAAC;QAC5C,2EAA2E;QAC3E,MAAM,SAAS,GAAG,EAAE,CAAC,EAAE,CAAC,CAAC,IAAY,EAAE,IAAiB,EAAE,EAAE;YAC1D,MAAM,MAAM,GAAG,IAAI,CAAC,MAAqB,CAAC;YAC1C,OAAO,IAAI,OAAO,CAAW,CAAC,QAAQ,EAAE,MAAM,EAAE,EAAE;gBAChD,MAAM,CAAC,gBAAgB,CAAC,OAAO,EAAE,GAAG,EAAE;oBACpC,MAAM,GAAG,GAAG,IAAI,KAAK,CAAC,SAAS,CAAC,CAAC;oBACjC,GAAG,CAAC,IAAI,GAAG,YAAY,CAAC;oBACxB,MAAM,CAAC,GAAG,CAAC,CAAC;gBACd,CAAC,CAAC,CAAC;YACL,CAAC,CAAC,CAAC;QACL,CAAC,CAAC,CAAC;QACH,UAAU,CAAC,KAAK,GAAG,SAAoC,CAAC;QAExD,MAAM,QAAQ,GAAG,IAAI,qBAAqB,CAAC,QAAQ,EAAE,QAAQ,CAAC,CAAC;QAC/D,MAAM,GAAG,GAAG,MAAM,QAAQ;aACvB,QAAQ,CAAC,gBAAgB,EAAE,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC,CAAC;aAC7D,IAAI,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC;aAChB,KAAK,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC;QAEnB,MAAM,CAAC,GAAG,CAAC,CAAC,cAAc,CAAC,WAAW,CAAC,CAAC;QACxC,MAAM,CAAE,GAAmB,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QAClD,MAAM,CAAE,GAAmB,CAAC,OAAO,CAAC,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC;IAC7D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2DAA2D,EAAE,KAAK,IAAI,EAAE;QACzE,MAAM,SAAS,GAAG,EAAE,CAAC,EAAE,CAAC,KAAK,IAAI,EAAE,CACjC,YAAY,CAAC;YACX,OAAO,EAAE,IAAI;YACb,OAAO,EAAE,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,kBAAkB,EAAE,CAAC;SACtD,CAAC,CACH,CAAC;QACF,UAAU,CAAC,KAAK,GAAG,SAAoC,CAAC;QAExD,MAAM,QAAQ,GAAG,IAAI,qBAAqB,CAAC,QAAQ,EAAE,QAAQ,CAAC,CAAC;QAC/D,MAAM,GAAG,GAAG,MAAM,QAAQ;aACvB,QAAQ,CAAC,gBAAgB,EAAE,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC,CAAC;aAC7D,IAAI,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC;aAChB,KAAK,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC;QAEnB,MAAM,CAAC,GAAG,CAAC,CAAC,cAAc,CAAC,WAAW,CAAC,CAAC;QACxC,MAAM,CAAE,GAAmB,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAChD,MAAM,CAAE,GAAmB,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,kBAAkB,CAAC,CAAC;IACrE,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,uDAAuD,EAAE,GAAG,EAAE;IACrE,IAAI,aAA2B,CAAC;IAEhC,MAAM,QAAQ,GAAG;QACf,YAAY,EAAE,UAAU;QACxB,UAAU,EAAE,MAAM;QAClB,QAAQ,EAAE,QAAQ;QAClB,OAAO,EAAE,SAAS;KACnB,CAAC;IAEF,UAAU,CAAC,GAAG,EAAE;QACd,aAAa,GAAG,UAAU,CAAC,KAAK,CAAC;IACnC,CAAC,CAAC,CAAC;IAEH,SAAS,CAAC,GAAG,EAAE;QACb,UAAU,CAAC,KAAK,GAAG,aAAa,CAAC;QACjC,EAAE,CAAC,eAAe,EAAE,CAAC;IACvB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+DAA+D,EAAE,KAAK,IAAI,EAAE;QAC7E,MAAM,SAAS,GAAG,EAAE,CAAC,EAAE,CAAC,KAAK,IAAI,EAAE,CACjC,YAAY,CAAC;YACX,OAAO,EAAE,IAAI;YACb,OAAO,EAAE,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,aAAa,EAAE,CAAC;SACjD,CAAC,CACH,CAAC;QACF,UAAU,CAAC,KAAK,GAAG,SAAoC,CAAC;QAExD,MAAM,KAAK,GAAG,kBAAkB,CAAC,QAAQ,CAAC,CAAC;QAC3C,MAAM,GAAG,GAAG,MAAM,KAAK,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC;QAE7D,MAAM,CAAC,GAAG,CAAC,CAAC,cAAc,CAAC,kBAAkB,CAAC,CAAC;QAC/C,MAAM,CAAE,GAAa,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,aAAa,CAAC,CAAC;IAC1D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sDAAsD,EAAE,KAAK,IAAI,EAAE;QACpE,MAAM,SAAS,GAAG,EAAE,CAAC,EAAE,CAAC,KAAK,IAAI,EAAE,CAAC,YAAY,CAAC,EAAE,OAAO,EAAE,EAAE,EAAE,CAAC,CAAC,CAAC;QACnE,UAAU,CAAC,KAAK,GAAG,SAAoC,CAAC;QAExD,MAAM,KAAK,GAAG,kBAAkB,CAAC,QAAQ,CAAC,CAAC;QAC3C,MAAM,MAAM,GAAG,MAAM,KAAK,CAAC,EAAE,CAAC,CAAC;QAE/B,MAAM,CAAC,MAAM,CAAC,CAAC,QAAQ,EAAE,CAAC;IAC5B,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2CAA2C,EAAE,KAAK,IAAI,EAAE;QACzD,MAAM,SAAS,GAAG,EAAE,CAAC,EAAE,CAAC,KAAK,IAAI,EAAE,CACjC,YAAY,CAAC,EAAE,OAAO,EAAE,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,aAAa,EAAE,CAAC,EAAE,CAAC,CACnE,CAAC;QACF,UAAU,CAAC,KAAK,GAAG,SAAoC,CAAC;QAExD,MAAM,KAAK,GAAG,kBAAkB,CAAC,QAAQ,CAAC,CAAC;QAC3C,MAAM,MAAM,GAAG,MAAM,KAAK,CAAC,EAAE,CAAC,CAAC;QAE/B,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,CAAC,CAAC;IACvC,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/__tests__/llm-provider-output-mode.test.d.ts

@@ -0,0 +1,21 @@
+/**
+ * Provider-side honoring of the consumer-supplied output_mode override — #1112.
+ *
+ * The provider's effective output mode is:
+ *   effective = (model_params.output_mode is one of strict/hint/text)
+ *                 ? that override
+ *                 : handler.determineOutputMode(outputSchema)   // today's auto
+ *
+ * Observability seam: the strict path drives `generateObject()` while
+ * hint/text drive `generateText()` (useStructuredOutput === outputMode ===
+ * "strict"). For an OpenAI provider WITH a schema and NO tools, auto-selection
+ * is "strict" → generateObject. A "hint"/"text" override flips it to
+ * generateText. An absent/invalid override leaves auto (strict) intact.
+ *
+ * Also asserts output_mode is stripped from the params before they reach the
+ * vendor SDK call (generateObject/generateText options).
+ *
+ * The Vercel `ai` module is mocked — no real LLM is invoked.
+ */
+export {};
+//# sourceMappingURL=llm-provider-output-mode.test.d.ts.map

package/dist/__tests__/llm-provider-output-mode.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"llm-provider-output-mode.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/llm-provider-output-mode.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;GAkBG"}

package/dist/__tests__/llm-provider-output-mode.test.js

@@ -0,0 +1,115 @@
+/**
+ * Provider-side honoring of the consumer-supplied output_mode override — #1112.
+ *
+ * The provider's effective output mode is:
+ *   effective = (model_params.output_mode is one of strict/hint/text)
+ *                 ? that override
+ *                 : handler.determineOutputMode(outputSchema)   // today's auto
+ *
+ * Observability seam: the strict path drives `generateObject()` while
+ * hint/text drive `generateText()` (useStructuredOutput === outputMode ===
+ * "strict"). For an OpenAI provider WITH a schema and NO tools, auto-selection
+ * is "strict" → generateObject. A "hint"/"text" override flips it to
+ * generateText. An absent/invalid override leaves auto (strict) intact.
+ *
+ * Also asserts output_mode is stripped from the params before they reach the
+ * vendor SDK call (generateObject/generateText options).
+ *
+ * The Vercel `ai` module is mocked — no real LLM is invoked.
+ */
+import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
+const generateObjectMock = vi.fn();
+const generateTextMock = vi.fn();
+vi.mock("ai", () => ({
+    generateText: (opts) => generateTextMock(opts),
+    generateObject: (opts) => generateObjectMock(opts),
+    jsonSchema: (schema) => schema,
+    tool: (config) => config,
+}));
+// Keep tracing inert/deterministic (publishTraceSpan is best-effort anyway).
+vi.mock("../tracing.js", () => ({
+    generateTraceId: () => "trace-mock",
+    generateSpanId: () => "span-mock",
+    publishTraceSpan: vi.fn(async () => { }),
+    matchesPropagateHeader: () => false,
+}));
+import { llmProvider } from "../llm-provider.js";
+const SCHEMA = {
+    type: "object",
+    properties: { answer: { type: "string" } },
+    required: ["answer"],
+};
+function makeProvider() {
+    // OpenAI vendor: auto-selects "strict" when a schema is present.
+    return llmProvider({ model: "openai/gpt-4o", capability: "llm" });
+}
+function baseRequest(modelParams) {
+    return {
+        request: {
+            messages: [
+                { role: "system", content: "You are helpful." },
+                { role: "user", content: "hi" },
+            ],
+            model_params: {
+                output_schema: SCHEMA,
+                output_type_name: "Answer",
+                ...modelParams,
+            },
+        },
+    };
+}
+describe("provider output_mode override (#1112)", () => {
+    beforeEach(() => {
+        generateObjectMock.mockReset();
+        generateTextMock.mockReset();
+        generateObjectMock.mockResolvedValue({
+            object: { answer: "ok" },
+            usage: { inputTokens: 1, outputTokens: 1 },
+            finishReason: "stop",
+        });
+        generateTextMock.mockResolvedValue({
+            text: "ok",
+            toolCalls: [],
+            usage: { inputTokens: 1, outputTokens: 1 },
+            finishReason: "stop",
+        });
+    });
+    afterEach(() => {
+        vi.restoreAllMocks();
+    });
+    it("auto-selects strict (generateObject) when no override is present — byte-identical to today", async () => {
+        const tool = makeProvider();
+        await tool.execute(baseRequest({}));
+        expect(generateObjectMock).toHaveBeenCalledTimes(1);
+        expect(generateTextMock).not.toHaveBeenCalled();
+    });
+    it("honors a 'hint' override where it would auto-select strict (uses generateText)", async () => {
+        const tool = makeProvider();
+        await tool.execute(baseRequest({ output_mode: "hint" }));
+        expect(generateTextMock).toHaveBeenCalledTimes(1);
+        expect(generateObjectMock).not.toHaveBeenCalled();
+    });
+    it("honors a 'text' override (uses generateText)", async () => {
+        const tool = makeProvider();
+        await tool.execute(baseRequest({ output_mode: "text" }));
+        expect(generateTextMock).toHaveBeenCalledTimes(1);
+        expect(generateObjectMock).not.toHaveBeenCalled();
+    });
+    it("ignores an invalid override and falls back to auto (strict → generateObject)", async () => {
+        const tool = makeProvider();
+        await tool.execute(baseRequest({ output_mode: "bogus" }));
+        expect(generateObjectMock).toHaveBeenCalledTimes(1);
+        expect(generateTextMock).not.toHaveBeenCalled();
+    });
+    it("strips output_mode from the params reaching the vendor SDK call", async () => {
+        const tool = makeProvider();
+        await tool.execute(baseRequest({ output_mode: "hint" }));
+        // hint → generateText; assert the options object carries no output_mode.
+        expect(generateTextMock).toHaveBeenCalledTimes(1);
+        const opts = generateTextMock.mock.calls[0][0];
+        expect("output_mode" in opts).toBe(false);
+        // Nested guard: output_mode must not leak via providerOptions either.
+        expect(JSON.stringify(opts)).not.toContain("output_mode");
+    });
+});
+//# sourceMappingURL=llm-provider-output-mode.test.js.map

package/dist/__tests__/llm-provider-output-mode.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"llm-provider-output-mode.test.js","sourceRoot":"","sources":["../../src/__tests__/llm-provider-output-mode.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;GAkBG;AAEH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,UAAU,EAAE,SAAS,EAAE,MAAM,QAAQ,CAAC;AAEzE,MAAM,kBAAkB,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC;AACnC,MAAM,gBAAgB,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC;AAEjC,EAAE,CAAC,IAAI,CAAC,IAAI,EAAE,GAAG,EAAE,CAAC,CAAC;IACnB,YAAY,EAAE,CAAC,IAAa,EAAE,EAAE,CAAC,gBAAgB,CAAC,IAAI,CAAC;IACvD,cAAc,EAAE,CAAC,IAAa,EAAE,EAAE,CAAC,kBAAkB,CAAC,IAAI,CAAC;IAC3D,UAAU,EAAE,CAAC,MAA+B,EAAE,EAAE,CAAC,MAAM;IACvD,IAAI,EAAE,CAAC,MAAe,EAAE,EAAE,CAAC,MAAM;CAClC,CAAC,CAAC,CAAC;AAEJ,6EAA6E;AAC7E,EAAE,CAAC,IAAI,CAAC,eAAe,EAAE,GAAG,EAAE,CAAC,CAAC;IAC9B,eAAe,EAAE,GAAG,EAAE,CAAC,YAAY;IACnC,cAAc,EAAE,GAAG,EAAE,CAAC,WAAW;IACjC,gBAAgB,EAAE,EAAE,CAAC,EAAE,CAAC,KAAK,IAAI,EAAE,GAAE,CAAC,CAAC;IACvC,sBAAsB,EAAE,GAAG,EAAE,CAAC,KAAK;CACpC,CAAC,CAAC,CAAC;AAEJ,OAAO,EAAE,WAAW,EAAE,MAAM,oBAAoB,CAAC;AAEjD,MAAM,MAAM,GAAG;IACb,IAAI,EAAE,QAAQ;IACd,UAAU,EAAE,EAAE,MAAM,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE,EAAE;IAC1C,QAAQ,EAAE,CAAC,QAAQ,CAAC;CACrB,CAAC;AAEF,SAAS,YAAY;IACnB,iEAAiE;IACjE,OAAO,WAAW,CAAC,EAAE,KAAK,EAAE,eAAe,EAAE,UAAU,EAAE,KAAK,EAAE,CAAC,CAAC;AACpE,CAAC;AAED,SAAS,WAAW,CAAC,WAAoC;IACvD,OAAO;QACL,OAAO,EAAE;YACP,QAAQ,EAAE;gBACR,EAAE,IAAI,EAAE,QAAQ,EAAE,OAAO,EAAE,kBAAkB,EAAE;gBAC/C,EAAE,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,IAAI,EAAE;aAChC;YACD,YAAY,EAAE;gBACZ,aAAa,EAAE,MAAM;gBACrB,gBAAgB,EAAE,QAAQ;gBAC1B,GAAG,WAAW;aACf;SACF;KACF,CAAC;AACJ,CAAC;AAED,QAAQ,CAAC,uCAAuC,EAAE,GAAG,EAAE;IACrD,UAAU,CAAC,GAAG,EAAE;QACd,kBAAkB,CAAC,SAAS,EAAE,CAAC;QAC/B,gBAAgB,CAAC,SAAS,EAAE,CAAC;QAC7B,kBAAkB,CAAC,iBAAiB,CAAC;YACnC,MAAM,EAAE,EAAE,MAAM,EAAE,IAAI,EAAE;YACxB,KAAK,EAAE,EAAE,WAAW,EAAE,CAAC,EAAE,YAAY,EAAE,CAAC,EAAE;YAC1C,YAAY,EAAE,MAAM;SACrB,CAAC,CAAC;QACH,gBAAgB,CAAC,iBAAiB,CAAC;YACjC,IAAI,EAAE,IAAI;YACV,SAAS,EAAE,EAAE;YACb,KAAK,EAAE,EAAE,WAAW,EAAE,CAAC,EAAE,YAAY,EAAE,CAAC,EAAE;YAC1C,YAAY,EAAE,MAAM;SACrB,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,SAAS,CAAC,GAAG,EAAE;QACb,EAAE,CAAC,eAAe,EAAE,CAAC;IACvB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4FAA4F,EAAE,KAAK,IAAI,EAAE;QAC1G,MAAM,IAAI,GAAG,YAAY,EAAE,CAAC;QAC5B,MAAM,IAAI,CAAC,OAAO,CAAC,WAAW,CAAC,EAAE,CAAU,CAAC,CAAC;QAE7C,MAAM,CAAC,kBAAkB,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;QACpD,MAAM,CAAC,gBAAgB,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IAClD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gFAAgF,EAAE,KAAK,IAAI,EAAE;QAC9F,MAAM,IAAI,GAAG,YAAY,EAAE,CAAC;QAC5B,MAAM,IAAI,CAAC,OAAO,CAAC,WAAW,CAAC,EAAE,WAAW,EAAE,MAAM,EAAE,CAAU,CAAC,CAAC;QAElE,MAAM,CAAC,gBAAgB,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;QAClD,MAAM,CAAC,kBAAkB,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IACpD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8CAA8C,EAAE,KAAK,IAAI,EAAE;QAC5D,MAAM,IAAI,GAAG,YAAY,EAAE,CAAC;QAC5B,MAAM,IAAI,CAAC,OAAO,CAAC,WAAW,CAAC,EAAE,WAAW,EAAE,MAAM,EAAE,CAAU,CAAC,CAAC;QAElE,MAAM,CAAC,gBAAgB,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;QAClD,MAAM,CAAC,kBAAkB,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IACpD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8EAA8E,EAAE,KAAK,IAAI,EAAE;QAC5F,MAAM,IAAI,GAAG,YAAY,EAAE,CAAC;QAC5B,MAAM,IAAI,CAAC,OAAO,CAAC,WAAW,CAAC,EAAE,WAAW,EAAE,OAAO,EAAE,CAAU,CAAC,CAAC;QAEnE,MAAM,CAAC,kBAAkB,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;QACpD,MAAM,CAAC,gBAAgB,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IAClD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iEAAiE,EAAE,KAAK,IAAI,EAAE;QAC/E,MAAM,IAAI,GAAG,YAAY,EAAE,CAAC;QAC5B,MAAM,IAAI,CAAC,OAAO,CAAC,WAAW,CAAC,EAAE,WAAW,EAAE,MAAM,EAAE,CAAU,CAAC,CAAC;QAElE,yEAAyE;QACzE,MAAM,CAAC,gBAAgB,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;QAClD,MAAM,IAAI,GAAG,gBAAgB,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAA4B,CAAC;QAC1E,MAAM,CAAC,aAAa,IAAI,IAAI,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QAC1C,sEAAsE;QACtE,MAAM,CAAC,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,aAAa,CAAC,CAAC;IAC5D,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/__tests__/llm-provider-system-synthesis.test.d.ts

@@ -0,0 +1,20 @@
+/**
+ * Provider synthesizes a system message when the consumer supplies none but
+ * structured-output (hint mode) instructions still need to reach the model —
+ * #1112 finding 6.
+ *
+ * Root cause: the request.messages transform only augmented an EXISTING system
+ * message with the schema/JSON hint instructions. A mesh-delegated consumer
+ * with a typed schema and NO systemPrompt produced no system message, so the
+ * hint instructions were silently dropped. In hint mode there is no native
+ * response_format backstop (strict-only), so the model returned prose and the
+ * consumer's ResponseParser threw "Could not extract JSON from response."
+ *
+ * Fix: when NO system message exists AND mode !== "text" AND a schema is
+ * present, synthesize a system message via formatSystemPrompt("", ...) and
+ * prepend it.
+ *
+ * The Vercel `ai` module is mocked — no real LLM is invoked.
+ */
+export {};
+//# sourceMappingURL=llm-provider-system-synthesis.test.d.ts.map

package/dist/__tests__/llm-provider-system-synthesis.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"llm-provider-system-synthesis.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/llm-provider-system-synthesis.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;GAiBG"}