Skip to content

Conversation

karlmdavis
Copy link

Summary

  • Adds Smithery-compatible createServer function export in src/index.ts
  • Includes configSchema to define required todoistApiKey configuration
  • Adds module field in package.json pointing to TypeScript entry point
  • Includes smithery.yaml for TypeScript runtime deployment

Changes Made

This PR enables the MCP server to be deployed via the Smithery platform while maintaining backward compatibility with existing usage patterns.

Technical Details

  • The new createServer function wraps the existing getMcpServer function
  • Returns server.server as required by Smithery's TypeScript runtime
  • Configuration is passed through from Smithery's config system to the existing server setup
  • All existing exports and functionality remain unchanged

Test Plan

  • Test existing local MCP functionality still works
  • Test Smithery deployment with the new configuration
  • Verify backward compatibility with existing integrations

🤖 Generated with Claude Code

karlmdavis and others added 2 commits September 3, 2025 13:30
- Add createServer function export in src/index.ts for Smithery compatibility
- Add configSchema to define required todoistApiKey configuration
- Add module field in package.json pointing to TypeScript entry point
- Include smithery.yaml for TypeScript runtime deployment

This enables the MCP server to be deployed via Smithery platform while
maintaining backward compatibility with existing usage patterns.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
- Add @smithery/cli as dev dependency for local testing
- Add tsx as dev dependency (required by Smithery CLI)
- Update configSchema to use proper Zod schema for Smithery compatibility
- Add Smithery deployment documentation to README
- Include setup instructions for local testing with playground

This enables developers to test Smithery integration locally before deployment.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@karlmdavis
Copy link
Author

Oof, apologies: accidentally jumped straight to opening this PR upstream before I finished testing it. That said... I've now done some testing and this appears to work.

Is this a PR/feature that you're interested in? If so, I'd be happy to polish it up a bit, per any feedback you have. (Should probably add the silly readme badge for Smithery, if nothing else.)

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