benjivers/jev-mcp Public Public: anyone can read this; nobody can change it.

// Contract and regression tests for the tool surface.

//

// These run the real built server over stdio against a local stand-in for the

// TypeSafe API, so each assertion is about what the server actually sends and

// returns. Every test named "regression" pins a defect that was confirmed on

// the wire before it was fixed.

import assert from "node:assert/strict";

import test from "node:test";

import { payload, startMock, withClient } from "./helpers.mjs";

const TOOLS = ["jev_ask", "jev_check", "jev_classify", "jev_models", "jev_score"];

// ── Contract ────────────────────────────────────────────────────────────────

test("advertises exactly the five judgment tools", async () => {

  await withClient({ withKey: false }, async (client) => {

    const { tools } = await client.listTools();

    assert.deepEqual(tools.map((t) => t.name).sort(), TOOLS);

  });

});

test("every tool documents itself and declares both input and output schemas", async () => {

  await withClient({ withKey: false }, async (client) => {

    const { tools } = await client.listTools();

    for (const tool of tools) {

      assert.ok(tool.description?.length > 40, `${tool.name} needs a real description`);

      assert.ok(tool.inputSchema, `${tool.name} has no inputSchema`);

      assert.equal(tool.inputSchema.type, "object");

      assert.ok(tool.outputSchema, `${tool.name} has no outputSchema, so callers get an opaque string`);

    }

  });

});

test("classify requires caller-supplied options, so the model cannot invent one", async () => {

  await withClient({ withKey: false }, async (client) => {

    const { tools } = await client.listTools();

    const classify = tools.find((t) => t.name === "jev_classify");

    for (const field of ["state", "question", "options"]) {

      assert.ok(classify.inputSchema.required.includes(field), `${field} must be required`);

    }

    const scoreTool = tools.find((t) => t.name === "jev_score");

    assert.ok(scoreTool.inputSchema.required.includes("levels"));

    assert.equal(scoreTool.inputSchema.properties.levels.minItems, 2);

  });

});

test("a missing API key fails loudly and says what to do about it", async () => {

  await withClient({ withKey: false }, async (client) => {

    const result = await client.callTool({

      name: "jev_check",

      arguments: { state: "anything", question: "Is this a greeting?" },

    });

    assert.equal(result.isError, true, "must report an error, not a fabricated answer");

    const { error } = payload(result);

    assert.equal(error.kind, "no_api_key");

    assert.equal(error.retryable, false);

    assert.match(error.hint, /environment/i);

  });

});

test("falls back to a key file when the environment carries no key", async () => {

  // An MCP server inherits the client's environment, not your shell's, so a key

  // exported from a shell profile never arrives. The file fallback is the only

  // route that reaches every spawn path, which makes it worth pinning here.

  const { mkdtempSync, writeFileSync } = await import("node:fs");

  const { tmpdir } = await import("node:os");

  const { join } = await import("node:path");

  const keyFile = join(mkdtempSync(join(tmpdir(), "jev-key-")), "key");

  writeFileSync(keyFile, "  value-for-the-local-stand-in\n", { mode: 0o600 });

  const mock = await startMock();

  try {

    await withClient({ baseUrl: mock.url, withKey: false, env: { JEV_KEY_FILE: keyFile } }, async (client) => {

      const result = await client.callTool({

        name: "jev_check",

        arguments: { state: "s", question: "Is this fine?" },

      });

      assert.notEqual(result.isError, true, "the key file should satisfy the credential check");

      assert.equal(typeof payload(result).probability_yes, "number");

      assert.equal(mock.requests.length, 1, "the request must actually reach the API");

    });

  } finally {

    await mock.close();

  }

});

// ── Regressions confirmed on the wire ───────────────────────────────────────

test("regression: an option the caller names 'none' is never overwritten", async () => {

  const mock = await startMock();

  const callerMeaning = "Already resolved; no action needed.";

  try {

    await withClient({ baseUrl: mock.url }, async (client) => {

      const result = await client.callTool({

        name: "jev_classify",

        arguments: {

          state: "s",

          question: "What should happen next?",

          options: { none: callerMeaning, escalate: "Send to a human" },

        },

      });

      const sent = mock.only().questions.classify.criteria;

      assert.equal(sent.none, callerMeaning, "the caller's meaning must reach the model intact");

      assert.equal(sent.none_of_these, "None of the other options fits.");

      const body = payload(result);

      assert.equal(body.none_option, "none_of_these", "the caller must be told which key means no-match");

    });

  } finally {

    await mock.close();

  }

});

test("regression: add_none false sends exactly the caller's options", async () => {

  const mock = await startMock();

  try {

    await withClient({ baseUrl: mock.url }, async (client) => {

      const result = await client.callTool({

        name: "jev_classify",

        arguments: { state: "s", question: "q", options: { a: "first", b: "second" }, add_none: false },

      });

      assert.deepEqual(Object.keys(mock.only().questions.classify.criteria).sort(), ["a", "b"]);

      assert.equal(payload(result).none_option, null);

    });

  } finally {

    await mock.close();

  }

});

test("regression: an empty or oversized option set fails before any request is sent", async () => {

  const mock = await startMock();

  try {

    await withClient({ baseUrl: mock.url }, async (client) => {

      const empty = await client.callTool({

        name: "jev_classify",

        arguments: { state: "s", question: "q", options: {}, add_none: false },

      });

      assert.equal(empty.isError, true);

      assert.match(payload(empty).error.message, /at least one option/);

      const tooMany = Object.fromEntries(Array.from({ length: 256 }, (_, i) => [`o${i}`, null]));

      const oversized = await client.callTool({

        name: "jev_classify",

        arguments: { state: "s", question: "q", options: tooMany, add_none: false },

      });

      assert.equal(oversized.isError, true);

      assert.match(payload(oversized).error.message, /at most 255 options/);

      assert.equal(mock.requests.length, 0, "neither request should reach the API");

    });

  } finally {

    await mock.close();

  }

});

test("regression: duplicate question ids fail instead of silently dropping a question", async () => {

  const mock = await startMock();

  try {

    await withClient({ baseUrl: mock.url }, async (client) => {

      const result = await client.callTool({

        name: "jev_ask",

        arguments: {

          state: "s",

          questions: [

            { id: "dup", type: "check", question: "first" },

            { id: "dup", type: "check", question: "second" },

          ],

        },

      });

      assert.equal(result.isError, true);

      assert.match(payload(result).error.message, /Duplicate question ids: "dup"/);

      assert.equal(mock.requests.length, 0, "no partial request should be sent");

    });

  } finally {

    await mock.close();

  }

});

test("regression: jev_ask honours add_none and passes yes/no meanings", async () => {

  const mock = await startMock();

  try {

    await withClient({ baseUrl: mock.url }, async (client) => {

      await client.callTool({

        name: "jev_ask",

        arguments: {

          state: "s",

          questions: [

            { id: "pick", type: "classify", question: "q", options: { a: "d", b: "d" }, add_none: false },

            { id: "flag", type: "check", question: "q", yes_means: "It is urgent", no_means: "It can wait" },

          ],

        },

      });

      const sent = mock.only().questions;

      assert.deepEqual(Object.keys(sent.pick.criteria).sort(), ["a", "b"], "add_none:false must be honoured");

      assert.deepEqual(sent.flag.criteria, { true: "It is urgent", false: "It can wait" });

    });

  } finally {

    await mock.close();

  }

});

test("regression: a verbose SDK log level does not corrupt the JSON-RPC stream", async () => {

  const mock = await startMock();

  try {

    await withClient({ baseUrl: mock.url, env: { TYPESAFE_LOG_LEVEL: "debug" } }, async (client) => {

      const result = await client.callTool({

        name: "jev_check",

        arguments: { state: "s", question: "Is this fine?" },

      });

      assert.notEqual(result.isError, true, "debug logging must not break the connection");

      assert.equal(typeof payload(result).probability_yes, "number");

    });

  } finally {

    await mock.close();

  }

});

// ── Behaviour ───────────────────────────────────────────────────────────────

test("results carry structured content matching the text block", async () => {

  const mock = await startMock();

  try {

    await withClient({ baseUrl: mock.url }, async (client) => {

      const result = await client.callTool({

        name: "jev_classify",

        arguments: { state: "s", question: "q", options: { a: "d", b: "d" } },

      });

      assert.ok(result.structuredContent, "clients should get typed data, not a JSON string");

      assert.deepEqual(result.structuredContent, payload(result));

      assert.equal(typeof result.structuredContent.probabilities.a, "number");

    });

  } finally {

    await mock.close();

  }

});

test("confidence drives the recommended action", async () => {

  const mock = await startMock();

  try {

    await withClient({ baseUrl: mock.url }, async (client) => {

      const args = { state: "s", question: "q", options: { a: "d", b: "d" } };

      mock.state.confidence = 0.95;

      assert.equal(payload(await client.callTool({ name: "jev_classify", arguments: args })).action, "act");

      mock.state.confidence = 0.6;

      assert.equal(payload(await client.callTool({ name: "jev_classify", arguments: args })).action, "review");

      mock.state.confidence = 0.2;

      assert.equal(payload(await client.callTool({ name: "jev_classify", arguments: args })).action, "abstain");

      mock.state.confidence = 0.6;

      const strict = await client.callTool({ name: "jev_classify", arguments: { ...args, act_above: 0.5 } });

      assert.equal(payload(strict).action, "act", "caller thresholds must override the defaults");

    });

  } finally {

    await mock.close();

  }

});

test("a near-even probability is reported as uncertain, not rounded to a yes", async () => {

  const mock = await startMock();

  try {

    await withClient({ baseUrl: mock.url }, async (client) => {

      mock.state.noul = 0.52;

      const result = await client.callTool({

        name: "jev_check",

        arguments: { state: "s", question: "Is this urgent?" },

      });

      const body = payload(result);

      assert.equal(body.probability_yes, 0.52);

      assert.equal(body.verdict, "uncertain");

    });

  } finally {

    await mock.close();

  }

});

test("oversized state is rejected before it costs anything", async () => {

  const mock = await startMock();

  try {

    await withClient({ baseUrl: mock.url, env: { JEV_MAX_STATE_CHARS: "50" } }, async (client) => {

      const result = await client.callTool({

        name: "jev_check",

        arguments: { state: "x".repeat(200), question: "q" },

      });

      assert.equal(result.isError, true);

      assert.match(payload(result).error.message, /above the 50 limit/);

      assert.equal(mock.requests.length, 0);

    });

  } finally {

    await mock.close();

  }

});

test("an unusable timeout falls back instead of disabling the timeout", async () => {

  const mock = await startMock();

  try {

    await withClient({ baseUrl: mock.url, env: { JEV_TIMEOUT_MS: "not-a-number" } }, async (client) => {

      const result = await client.callTool({

        name: "jev_check",

        arguments: { state: "s", question: "q" },

      });

      assert.notEqual(result.isError, true, "a malformed timeout must not break the server");

    });

  } finally {

    await mock.close();

  }

});

test("API failures are classified so a caller can tell retryable from terminal", async () => {

  const mock = await startMock();

  try {

    await withClient({ baseUrl: mock.url }, async (client) => {

      mock.state.status = 401;

      const unauthorized = await client.callTool({ name: "jev_check", arguments: { state: "s", question: "q" } });

      assert.equal(unauthorized.isError, true);

      const { error } = payload(unauthorized);

      assert.equal(error.kind, "authentication");

      assert.equal(error.retryable, false);

      assert.equal(error.requestId, "req_test_123", "the request id support needs must survive");

    });

  } finally {

    await mock.close();

  }

});

test("jev_models reports the active model and what the key can use", async () => {

  const mock = await startMock();

  try {

    await withClient({ baseUrl: mock.url }, async (client) => {

      const body = payload(await client.callTool({ name: "jev_models", arguments: {} }));

      assert.equal(body.active_model, "jev-latest");

      assert.equal(body.models[0].name, "jev-latest");

    });

  } finally {

    await mock.close();

  }

});

test("one jev_ask call batches every question into a single request", async () => {

  const mock = await startMock();

  try {

    await withClient({ baseUrl: mock.url }, async (client) => {

      const result = await client.callTool({

        name: "jev_ask",

        arguments: {

          state: { ticket: "My card was charged twice." },

          questions: [

            { id: "billing", type: "check", question: "Is this about billing?" },

            { id: "team", type: "classify", question: "Which team?", options: { billing: "d", tech: "d" } },

            { id: "anger", type: "score", question: "How angry?", levels: ["calm", "annoyed", "furious"] },

          ],

        },

      });

      assert.equal(mock.requests.length, 1, "three questions must cost one round trip");

      assert.deepEqual(Object.keys(mock.only().questions).sort(), ["anger", "billing", "team"]);

      const body = payload(result);

      assert.deepEqual(Object.keys(body.answers).sort(), ["anger", "billing", "team"]);

      assert.equal(body.answers.team.action, "act", "choice answers carry a gate");

      assert.equal(body.answers.billing.action, undefined, "a noul has no confidence to gate on");

    });

  } finally {

    await mock.close();

  }

});