Skip to content

docs(sampling): clarify client-side sampling context#2487

Closed
MukundaKatta wants to merge 1 commit intomodelcontextprotocol:mainfrom
MukundaKatta:codex/document-sampling-context
Closed

docs(sampling): clarify client-side sampling context#2487
MukundaKatta wants to merge 1 commit intomodelcontextprotocol:mainfrom
MukundaKatta:codex/document-sampling-context

Conversation

@MukundaKatta
Copy link
Copy Markdown

Summary

  • explain that ctx.session.create_message(...) routes sampling through the connected client rather than exposing a direct server-side model handle
  • document how to use model_preferences, include_context, and the client-side sampling callback together
  • update the README callback example to use ClientRequestContext and sync the snippet source files with the clarified guidance

Testing

  • not run (docs/snippets only)

Closes #1205

@MukundaKatta
Copy link
Copy Markdown
Author

Closing out — pre-commit failures and I'm not pushing updates. Thanks.

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.

Controlling Context in Client for Sampling Requests

1 participant