-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add mcp time server #44
Conversation
952cfa3
to
23d0c59
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is awesome—thanks for the submission!
Left some questions and suggestions, but would love to get this in. 🙏
ceb22e5
to
ecad3f6
Compare
82ec907
to
d0256cf
Compare
d0256cf
to
8947ed0
Compare
8947ed0
to
d37ce3c
Compare
Hey, I've added the necessary changes. Can you take a look? :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good! Thank you so much for the work!
You probably want to change the toplevel readme as well to include the timeserver
…rver Add mcp time server
Description
This merge requests adding a new mcp time server with the following tools:
get_current_time
- Get current time in IANA timezoneconvert_time
- Convert time between IANA timezonesServer Details
Motivation and Context
LLM doesn't know what time is it ;)
And people always struggle to calculate the time between timezones
How Has This Been Tested?
Has been tested with MCP Inspector and Claude with uvx
Breaking Changes
New MCP configuration
Types of changes
Checklist
Additional context