Skip to content

docs: add pre-registered OAuth clients to v3-features#3129

Merged
jlowin merged 1 commit intomainfrom
docs/v3-features-static-client
Feb 10, 2026
Merged

docs: add pre-registered OAuth clients to v3-features#3129
jlowin merged 1 commit intomainfrom
docs/v3-features-static-client

Conversation

@jlowin
Copy link
Copy Markdown
Member

@jlowin jlowin commented Feb 10, 2026

Adds the static client registration feature (merged in #3086) to the v3 feature tracking document.

@marvin-context-protocol marvin-context-protocol Bot added documentation Updates to docs, examples, or guides. Primary change is documentation-related. v3 Targeted for FastMCP 3 labels Feb 10, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Feb 10, 2026

Walkthrough

This pull request adds a documentation section titled "Pre-Registered OAuth Clients" to the v3-features.mdx file. The section documents that the OAuth client helper now accepts client_id and client_secret parameters for pre-registered clients, allowing them to bypass the Dynamic Client Registration flow. The documentation includes sample code demonstrating usage, guidance on when to use static credentials, and a reference link. The identical section appears twice within the document.

Possibly related PRs

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive The description is minimal but related to the change. However, it lacks the required checklist items and self-review confirmation specified in the repository's description template. Complete the Contributors and Review checklists from the template, including issue reference, workflow confirmation, testing notes, and self-review acknowledgment.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: adding documentation about pre-registered OAuth clients to the v3-features document.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch docs/v3-features-static-client

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments
docs/development/v3-notes/v3-features.mdx (1)

127-140: Code example uses placeholder credentials.

client_id="my-registered-app" and client_secret="my-secret" are generic placeholders. Consider using more realistic-looking values (e.g., a UUID-style client ID) to better illustrate real-world usage, consistent with the coding guidelines for MDX code examples.

As per coding guidelines, "Use realistic data instead of placeholder values in MDX code examples".


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@jlowin jlowin merged commit eeb1785 into main Feb 10, 2026
15 checks passed
@jlowin jlowin deleted the docs/v3-features-static-client branch February 10, 2026 01:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Updates to docs, examples, or guides. Primary change is documentation-related. v3 Targeted for FastMCP 3

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant