Conversation
Signed-off-by: Chojan Shang <chojan.shang@vesoft.com>
Signed-off-by: Chojan Shang <chojan.shang@vesoft.com>
Signed-off-by: Chojan Shang <chojan.shang@vesoft.com>
Signed-off-by: Chojan Shang <chojan.shang@vesoft.com>
Signed-off-by: Chojan Shang <chojan.shang@vesoft.com>
Signed-off-by: Chojan Shang <chojan.shang@vesoft.com>
|
Cool. didnt test this yet, but the last commit already fixed the basic loop for me, I now have a fully functional python agent with filesystem client tools, terminal client tools, MCP support (including MCP -> ACP progress reporting) working. |
Glad the patch works for you—that was exactly why I shipped 0.4.5 yesterday after another user reported the issue.
As for the router: it’s mostly syntactic sugar to make handlers less painful. I get that the getattr-style magic isn’t everyone’s favorite. I’m considering a cleaner, more explicit approach and open to suggestions |
register request/notification methods declaratively, replacing bespoke
dispatch logic
dispatcher, and sender components for clearer responsibilities and easier
injection
wrapping request and notification execution without requiring
configuration
existing tests (tests/test_rpc.py, tests/real_user) passing