fix: correct Monarch domain to monarch.com + graceful fallback for clients without elicitation#78
Open
shmuelsash wants to merge 1 commit into
Conversation
…ients without elicitation - Update the user-facing token instructions and login form description from the retired app.monarchmoney.com domain to app.monarch.com (auth.py, tools/auth.py). - When an MCP client does not support elicitation (returns -32601 Method not found), return a clear message pointing the user to monarch_login_with_token instead of letting the -32601 bubble up as an opaque error. Applied to login, MFA, and token-login flows.
shmuelsash
force-pushed
the
fix/monarch-domain-and-elicitation-fallback
branch
from
July 9, 2026 20:47
544bedf to
575071e
Compare
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.
What
Two correctness/robustness fixes:
Domain migration in user-facing text. Monarch retired
app.monarchmoney.comin favour ofapp.monarch.com. The token instructions (TokenFormdescription and themonarch_login_with_tokendocstring) still point users at the old domain, where thetokenlocal-storage key no longer lives. Updated toapp.monarch.com.Graceful handling for MCP clients without elicitation. Some clients don't implement the elicitation protocol and return
-32601 Method not foundwhenctx.elicit(...)is called. Today that surfaces as an opaque error. This catches it in the login, MFA, and token-login flows and returns a clear message telling the user to usemonarch_login_with_tokeninstead (with step-by-step token retrieval).Notes
-32601case and re-raises anything else.