Skip to content

fix: declare zod runtime dependencies#4378

Open
mturac wants to merge 1 commit into
modelcontextprotocol:mainfrom
mturac:fix/zod-runtime-dependencies
Open

fix: declare zod runtime dependencies#4378
mturac wants to merge 1 commit into
modelcontextprotocol:mainfrom
mturac:fix/zod-runtime-dependencies

Conversation

@mturac

@mturac mturac commented Jun 15, 2026

Copy link
Copy Markdown

Summary

  • add zod as a direct runtime dependency for the memory and sequential-thinking servers
  • add the same direct dependency for the filesystem server, which also imports zod directly
  • update the npm lockfile so published package installs include the dependency metadata

Fixes #4330

Testing

  • npm install
  • npm run build --workspace src/memory && npm run build --workspace src/sequentialthinking && npm run build --workspace src/filesystem
  • npm test --workspace src/memory && npm test --workspace src/sequentialthinking && npm test --workspace src/filesystem
  • packed the three affected workspaces and installed the tarballs in a clean temp project
  • verified zod resolves from each installed package and that short startup smokes no longer hit a missing zod module error
  • git diff --check

@mturac mturac force-pushed the fix/zod-runtime-dependencies branch from 0671ad9 to 8866415 Compare June 16, 2026 15:24
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.

server-memory and server-sequential-thinking fail at startup due to missing zod dependency

1 participant