Teach MCP clients the calendar's date conventions - #239
Merged
Merged
Conversation
Expand the server instructions from one sentence into a short guide: which jurisdictions each tradition covers, New vs Old calendar, and the civil-vs-church date offset (get_day takes the civil date; search_saints returns the church month/day, 13 days behind the civil date on the Old calendar). Also covers chaining search_saints into get_day and which result fields to prefer. Also list douay-rheims in get_day's translation docs. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
instructionsgrow from one sentence into a ~330-word guide. Clients such as Claude Code put this text into the agent's system prompt when they connect. It covers:slavicmeans OCA/ROCOR;greekmeans GOARCH/Antiochian.get_dayalways takes the civil date, and withcalendar=julianit returns the church date, 13 days earlier.search_saintsreturns the church month/day, so Old-calendar users have to add 13 days before callingget_day. Feasts tied to Pascha aren't found by saint search.search_saints, thenget_dayfor the saint's full life and the readings.summary_title; the fasting fields, with a note that they follow the strict typikon;abbreviated_reading_indices; stories are HTML.get_day's docstring now listsdouay-rheimsas a translation option.Test plan
mcp_svctests pass in Docker (9 tests)get_day(2027, 1, 7, calendar='julian')returns 2026-12-25, Nativity of Christ/mcpand check the new instructions appear🤖 Generated with Claude Code