-
Notifications
You must be signed in to change notification settings - Fork 2
plan: cloud architecture #474
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
base: main
Are you sure you want to change the base?
Conversation
jonathanlab
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, nice work on this! Left some questions about the replay functionality and how this ties into our current set up with ACP.
| - Review changes, pull them locally, continue | ||
| - Feels like delegating to a colleague | ||
|
|
||
| Most cloud agent implementations force you to choose one or the other. The goal here is to support both seamlessly—and let you switch between them without friction. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just want to flag that this goal is so consequential we could sell even a mediocre product very easily if we hit it.
|
Great stuff @joshsny 🐐, really like the direction of this! 🔥 Some thoughts: I really like using Temporal for lifecycle only, we are using it in ph-ai as hot-path message bus (with redis streams in front) and we are incurring on latency and additional complexity that here is simply not needed. On the On the "local wins" approach, I agree on the direction but perhaps we would need some kind of "caching" layer but maybe this is an early optimization problem. Maybe too technical at this stage but the watcher may need strong ignore rules and operation batching, let's say that we run a package manager command, the file watcher would have a ton of events in the same path so we might want to avoid emitting all of those at once. I am mostly thinking about possible ordering of changes, as in: multiple writes can arrive out of order with respect to the local client but we can define this in another passage. Q: what kind of ack are you thinking for |
I’m going to be looking at the cloud experience in array next, so wanted to bash out some thoughts on the architecture.
Will use this as a place to iterate and get thoughts.