8000 feat: add support for passing options to requestSampling by OrenMe · Pull Request #125 · punkpeye/fastmcp · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat: add support for passing options to requestSampling #125

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Jun 18, 2025

Conversation

OrenMe
Copy link
Contributor
@OrenMe OrenMe commented Jun 17, 2025

Solves #122

Note: for some reason I wasn't able to import the RequestOptions from the protocol file

@punkpeye
Copy link
Owner

@OrenMe just need help passing linting

@OrenMe
Copy link
Contributor Author
OrenMe commented Jun 18, 2025

Hi @punkpeye , thank u for reviewing
I pushed the fix and returned usage of type system
There is inherit issue with the return type due to mismatch between payload of content, TextContent has text filed while AudioContent and ImageContent doesn't have it, but this is not related to my change

Also ran the linter and prettier which moved the SamplingResponse

@punkpeye punkpeye merged commit 08dcf0b into punkpeye:main Jun 18, 2025
1 check passed
@punkpeye
Copy link
Owner

thank you

@OrenMe
Copy link
Contributor Author
OrenMe commented Jun 18, 2025

Thank u! Love the project

@OrenMe OrenMe deleted the orenm/requestSamplingOptions branch June 18, 2025 16:47
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.

2 participants
0