Richard's Claude Code

The if (!handler) continue guard prevents a crash when a handler is missing, but the test still asserts expect(spy).toHaveBeenCalled() (line 72) for every handler that returns a successful (non-error) response. get_agent_card_details reads from the in-memory tools array and never calls TruveraClient.request, so its handler succeeds with isError: false, the early-exit at line 69 is not taken, and the spy assertion at line 72 fires and fails.

The test describe block is titled "every tool handler should call TruveraClient.request" — a false assumption for this tool.

Fix: add a tool-name guard before the spy assertion, e.g.:

const NO_REQUEST_TOOLS = new Set(['get_agent_card_details']);
if (!NO_REQUEST_TOOLS.has(tool.name)) {
  expect(spy).toHaveBeenCalled();
}

Richard's Claude Code

The switch from mcp-remote to native type: "http" requires VS Code 1.99+ (released March 2025). Older installs fail silently — no error is surfaced to the user, the tools just don't appear. The README doesn't mention a minimum version.

Worth adding a one-liner to the VS Code setup section, e.g.: "Requires VS Code 1.99 or later. Earlier versions should use the manual mcp-remote config."

Richard's Claude Code

The Docker support | ✅ Included row was removed from the status table just above here, implying Docker is gone. But docker:build and docker:run scripts still exist in package.json, and a Dockerfile still exists in the repo. A developer reading this table will conclude Docker is unsupported and either avoid it or re-implement it unnecessarily.

Suggest restoring the row (perhaps updated to reflect current state), or adding a note in the Known Limitations section if Docker support is intentionally being deprecated.