mirror of
https://github.com/mcp-use/mcp-use.git
synced 2026-09-14 18:06:01 +08:00
13ca58db78
The step ran `pnpm --filter mcp-use --if-present test:unit`, but the mcp-use package has no test:unit script, so --if-present turned it into a silent pass and 447 tests never ran. Point it at test:run, which is the script that exists and the same one the client step above already uses. Two failures were hiding behind the skip: - usage.test.ts asserted telemetry was sent, but the step sets MCP_USE_ANONYMIZED_TELEMETRY=false, which short-circuits recordUsage. Those cases are about identity persistence, not the opt-out, so capture() now sets the variable itself. The file already tracks and restores it in afterEach. - budgets/runtime-boundary.test.ts compared the builtin list with toEqual, so it depended on module resolution order between node:process and node:http. Sorted, which is what the case means by "loads only". Verified with the step's own env: MCP_USE_ANONYMIZED_TELEMETRY=false pnpm --filter mcp-use test:run gives 43 files, 447 passed, exit 0. Closes #2203