-
Notifications
You must be signed in to change notification settings - Fork 6
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
Things need to be done in 0.1.5 #23
Comments
Is there any default that needs to be done here? I want it to stabilize and then apply it to my project. |
What do you mean when you say I don't have any idea about new features by now. I wish you can apply it to your project firstly and then we may find out something we forgot to do. |
Because in order to encourage more participants to participate, there will be a barrier in Chinese, although it is good for us to communicate. |
I have a few preliminary suggestions that might help improve this project:
|
@byeblack For transport, there's no standart here beside the SSE and stdio. But we can add an example with For compatibilities with Python SDK, this is important. And I also want to help to solve these problem, could you post a issue on this bug? For wasip2 target, we can implement AsyncRead and AsyncWrite for the wasip2 io. I've tried but it seems like main thread blocked somewhere. I also put WASI compatibility at a high priority. |
@4t145 Thanks for your reply For wasip2 target, I also implemented a simple AsyncRead and AsyncWrite, and tried to verify it with wasmtime, but stdout was blocked at the start of the second response, but stdin still worked normally...
|
@byeblack I think dynamic tool calling has been implemented in a sense? Because the tool calling API is inherently progressive. First, you can manually distribute it when implementing tool_call. Second, even if it is distributed through toolbox, the content stored in toolbox is also Dynamic Trait Object. It's just that when we use macros, the generated things are all static. 也许文档没有很清晰的说明这一点,我们应该在文档里说明清楚。 Maybe the documentation doesn't explain this clearly, we should explain it clearly in the documentation. |
I think WebSocket transport can be used as a puncture replacement for SSE. Although the official discussion did not use WebSocket, we can consider designing it. |
@jokemanfire Wow, that pr has a similay idea with mine in https://github.com/4t145/rmcp/blob/dev/examples/transport/src/http_upgrade.rs |
Features:
Bugs:
Documents:
The text was updated successfully, but these errors were encountered: