Skip to content

docs(time): fix response examples#4174

Open
Sean-Kenneth-Doherty wants to merge 2 commits into
modelcontextprotocol:mainfrom
Sean-Kenneth-Doherty:codex/fix-time-readme-examples
Open

docs(time): fix response examples#4174
Sean-Kenneth-Doherty wants to merge 2 commits into
modelcontextprotocol:mainfrom
Sean-Kenneth-Doherty:codex/fix-time-readme-examples

Conversation

@Sean-Kenneth-Doherty
Copy link
Copy Markdown

@Sean-Kenneth-Doherty Sean-Kenneth-Doherty commented May 16, 2026

Summary

  • add the day_of_week field to the Time server response examples
  • fix the conversion example so the New York to Tokyo datetime values and time_difference are internally consistent
  • add a regression test that parses the README response JSON blocks and compares them with frozen TimeServer output

Validation

Run from src/time:

  • uv run pytest
  • uv run ruff check .

Local result: 39 passed, ruff clean.

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