v0.1.8: add logout MCP tool#6
Merged
Merged
Conversation
Claude Desktop user noted the agent couldn't log them out because the MCP surface had no logout tool — only login_browser + whoami + reads/writes on heating state. The CLI already had `emodul auth logout` (clears token) and `emodul auth forget-password` (clears keychain). Expose them as a single MCP tool. logout(clear_keychain=False) - clear_keychain=False (default): drop token + user_id from config.json, KEEP the keychain password. login_browser can then re-auth without the user typing the password again. - clear_keychain=True: also delete the keychain entry. Auto-refresh on next 401 will fail until login_browser runs again. destructiveHint=true so well-behaved clients prompt for confirmation; idempotentHint=true because re-running has no extra effect. Server instructions updated to list `logout` under AUTH tools (17 total now: 9 read + 5 write + 3 auth).
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.
Why
Claude Desktop user noted the agent couldn't log them out — the MCP surface had `login_browser` + `whoami` + reads/writes on heating state, but no logout. The CLI already had `emodul auth logout` (clears token) and `emodul auth forget-password` (clears keychain), so this just exposes them.
What
New MCP tool:
```
logout(clear_keychain: bool = False)
```
`destructiveHint=true` so well-behaved clients prompt for confirmation. `idempotentHint=true` because re-running has no extra effect.
Updated
Verified
Release
After merge: tag v0.1.8 → OIDC publish.