-
Notifications
You must be signed in to change notification settings - Fork 9.9k
Description
Request: Add tool annotations — @modelcontextprotocol/server-time
Context
The time server currently provides zero annotations on its 2 tools. Both tools are pure read-only computations with no side effects. For reference, @modelcontextprotocol/server-filesystem annotates all 14 of its tools. We're requesting the same treatment here.
Current state — 0/2 tools annotated
| # | Tool | readOnlyHint |
destructiveHint |
idempotentHint |
openWorldHint |
|---|---|---|---|---|---|
| 1 | get_current_time |
— | — | — | — |
| 2 | convert_time |
— | — | — | — |
Suggested annotations
| # | Tool | readOnlyHint |
destructiveHint |
idempotentHint |
openWorldHint |
|---|---|---|---|---|---|
| 1 | get_current_time |
true |
false |
true |
false |
| 2 | convert_time |
true |
false |
true |
false |
Rationale
get_current_time → readOnlyHint: true, idempotentHint: true, openWorldHint: false
Returns the current time in a specified IANA timezone. Pure read from the system clock. No state mutation, no external calls.
convert_time → readOnlyHint: true, idempotentHint: true, openWorldHint: false
Converts a time between IANA timezones. Pure computation with no side effects, no state, no external calls.
Impact
This is the simplest annotation addition across all reference servers — ~10 lines of metadata across 2 tool registrations. Both tools are unambiguously read-only. No behavior changes. Completing this brings the time server in line with the filesystem server's annotation coverage.