Skip to content

Prioritize MCP error status and keep type fixture runtime safe - #341

Merged
aruintelligence merged 1 commit into
mainfrom
fix/mcp-error-precedence
Oct 14, 2026
Merged

aruintelligence merged 1 commit into
mainfrom
fix/mcp-error-precedence

Conversation

@aruintelligence

Copy link
Copy Markdown
Owner

Changes

  • Classify MCP isError: true as tool_error even when a result also claims input_required.
  • Add a conflicting-status regression case and clarify the result-status contract.
  • Keep deliberate TypeScript negative examples type-checked without executing them under Node versions that discover .mts files as tests.

Verification

  • node --test test/mcp-call-tool-gate.test.js (5 passed)
  • npm test (764 passed)
  • git diff --check

@aruintelligence
aruintelligence merged commit ee29e38 into main Oct 14, 2026
62 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant