Skip to content

Fix Build#12

Merged
dominicmacaulay merged 1 commit intomainfrom
fix-build-maybe
Feb 4, 2026
Merged

Fix Build#12
dominicmacaulay merged 1 commit intomainfrom
fix-build-maybe

Conversation

@dominicmacaulay
Copy link
Copy Markdown
Contributor

Why?

Typescript was no able to properly inspect index.js when attempting to build.

What Changed

  • Skip typescript checking on the file

Copilot reasoning:

The TypeScript errors you're seeing are not caused by bugs in your code, but by limitations in TypeScript's type inference system when dealing with:

  1. Deeply nested generic types in the MCP SDK (v1.24.0)
  2. Recursive type definitions that hit TypeScript's recursion limits
  3. Complex Zod schema inference combined with SDK generics

The Real Issues:

  1. Type instantiation recursion: TypeScript tries to deeply infer types through multiple layers (Zod → SDK → Your handlers), hitting its recursion limit
  2. String literal vs string type: Return types require type: "text" (literal) but TypeScript infers type: string
  3. Complex return type matching: The SDK expects very specific return structures that TypeScript struggles to validate

Copy link
Copy Markdown
Member

@dallasbpeters dallasbpeters left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. :shipit:

@dominicmacaulay dominicmacaulay merged commit 22cfc5e into main Feb 4, 2026
3 checks passed
@dominicmacaulay dominicmacaulay deleted the fix-build-maybe branch February 4, 2026 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants