Uh oh!
There was an error while loading. Please reload this page.
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>
phil65
commented
Oct 11, 2025
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. |
PsiACE
commented
Oct 11, 2025
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 |
Uh oh!
There was an error while loading. Please reload this page.
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