CMCDragonkai
CMCDragonkai
We should incorporate: 1. Unit test reports that go straight to the PR comment. 2. Benchmark reports that go straight to the PR comment, while also showing how benchmark compares...
@joshuakarp should this part of our feature freeze?
Seems like there are some new services that are primarily accessed over a discord bot or even wechat bot. Then bot-apps are bootstrapped over existing conversational interfaces. We can see...
A quick way to solve this is to move all of our comment descriptions of the exceptions into the actual `description` property and also use the appropriate exit code from...
I suggest a single GRPC connection error exception that is out into grpc/errors.ts. This is better than creating separate exceptions in client and agent domains. It can be rethrown as...
Oh wait I re-read the issue and therefore to might make sense to catch and specialise it to client domain error or agent domain error. Because certain ops will do...
Can you flesh out the task list in this issue based on what we want to end up doing.
#310 enables the ability to resolve this issue, but this would have to be addressed in a later PR following https://github.com/MatrixAI/js-polykey/issues/276#issuecomment-1017027663
The `GRPCClient` is now exposing the `destroyCallback` which is being used by `NodeConnection`. Similarly the `GRPCClientClient` will be destroyed if the underlying GRPC channel is shutdown. This means any usage...
The original spec relies alot on the way GRPC worked. GRPC isn't relevant anymore as we changed to using js-rpc on https://github.com/MatrixAI/Polykey/tree/4ddd9dd726995e4431a09b26b764f5d53fd5e4ac (October the 6th 2023). However having good errors...