Skip to content

docs: add OAuth/route lifecycle design notes, add AGENTS.md - #78

Open
timcharper wants to merge 1 commit into
tbxark:masterfrom
timcharper:docs/oauth-lifecycle-design
Open

docs: add OAuth/route lifecycle design notes, add AGENTS.md#78
timcharper wants to merge 1 commit into
tbxark:masterfrom
timcharper:docs/oauth-lifecycle-design

Conversation

@timcharper

Copy link
Copy Markdown
Contributor

Adds docs/OAUTH_LIFECYCLE.md: a design doc covering how OAuth-authorized routes behave today (the known 404-until-restart failure mode after running -authorize), and a target design for closing it without introducing new failure modes. Also adds a repo AGENTS.md pointing at it and the other docs.

This is the first commit of a larger proposed series - see the draft #77 for the full 4-commit stack (extract connect/mount, always-mount + fail-fast 503, lazy connect-on-request with no restart needed) that implements the design described here. Submitting this piece on its own, non-draft, since it's a complete, independently useful change (a reference doc) and is meant to establish direction for the rest of the series before those follow-on commits are split into their own PRs.

🤖 Generated with Claude Code

Documents current mount-at-startup-only behavior, the resulting
restart-to-reauthorize gap, and the target design for closing it
(always mount, fail fast locally with no credential, single-flight
connect, disk as sole source of truth) as a stack of small
incremental changes rather than one large rewrite.

Also adds AGENTS.md (none existed) indexing the docs and pointing
at build/format/test commands.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@timcharper

Copy link
Copy Markdown
Contributor Author

@tbxark would love your thoughts on this direction; basically, I want to remove the need to restart the server when I run -authorize (refresh tokens occasionally time out). With a cache-less design, -authorize just writes new state to the disk and the next request from mcp-server just sees and uses it.

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