test: strengthen coding standards enforcement for error handling and pagination

[cursor]

Summary

Audited the codebase against docs/coding-standards.md and closed two guardrail gaps:

• Error handling (§6): Added tests/coding-standards/error-handling.test.ts to ensure API-facing tool handlers use errorResult() for known errors and throw toMcpError(err) for unexpected failures.
• Pagination defaults (§8): Added tests/coding-standards/pagination.test.ts to lock harness_list to page=0, size=20, max=100.

Fix

harness_schema was the only API handler that swallowed unexpected errors via return errorResult(...) in its catch block. It now follows the same isUserError / isUserFixableApiError / throw toMcpError pattern as the other handlers.

Verification

• pnpm standards:check — 64 tests pass (was 60)
• pnpm test — 2241 tests pass

Notes

The pasted automation prompt referenced 10 tools and Zod v3; the repo's canonical standards (docs/coding-standards.md) correctly document 11 consolidated tools (including harness_schema) and Zod v4. Existing CI already runs pnpm standards:check on every PR.

  

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ thisrohangupta
❌ cursoragent
_{You have signed the CLA already but the status is still pending? Let us recheck it.}