Skip to content

Conversation

@matteo8p
Copy link
Contributor

Currently, when you run npm i, it triggers a build script that breaks. See this issue.
#32

There was a mismatch in zod versions being used. The SDK requires zod v4 but the project impored zod v3. We update the imports in app-bridge.ts and types.ts to use zod v4.

After:
Screenshot 2025-11-27 at 1 43 41 PM

Motivation and Context

Fixes broken npm i

How Has This Been Tested?

I ran npm i and confirmed that the build works and dependencies are properly installed.

Breaking Changes

NA

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

NA

@matteo8p matteo8p mentioned this pull request Nov 27, 2025
@matteo8p matteo8p changed the title Fix zod issue Fix zod mismatch issue Nov 27, 2025
@matteo8p matteo8p changed the title Fix zod mismatch issue Fix zod mismatch issue. Fixes npm i Nov 27, 2025
Copy link
Collaborator

@ochafik ochafik left a comment

Choose a reason for hiding this comment

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

Thanks @matteo8p !

@ochafik ochafik merged commit 192d29f into modelcontextprotocol:main Nov 27, 2025
@fredericbarthelet fredericbarthelet mentioned this pull request Nov 28, 2025
9 tasks
gabe4coding pushed a commit to gabe4coding/ext-apps that referenced this pull request Dec 12, 2025
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.

2 participants