docs(RFD): add proposal for forking at a specific message#629
docs(RFD): add proposal for forking at a specific message#629SteffenDE wants to merge 1 commit intoagentclientprotocol:mainfrom
Conversation
benbrandt
left a comment
There was a problem hiding this comment.
I think inclusive makes sense here.
Do we want to say something like this only works on agent messages?
Or, should we allow the agent to provide context on a given fork point?
Do we basically take user messages as the only valid fork points?
I think this could also solve the "edit previous message" case, but maybe then we want a way to say "don't create a new session, just truncate" which only some agents might support (codex would for example) just thinking about the edit use case that you may not always want to fully fork, but maybe we don't have to solve that right now
I think we can let the agent respond with an error if it cannot fork from a specific message, but assume that by default any message id can be used as fork point. The edit previous message is our main use case for Tidewave. I think it should still be a full fork. If clients want a "rewind current session" or similar, that should rather be a different method in my opinion, e.g. |
|
I think we could go with any message, but I also wonder if the agent should indicate a given message is a forkable point? So if you see some property on the message, you know it can fork from there and the client can indicate as such? |
|
I'm wondering if we should have this as a field on the capability, like If it's on the message, the question would be where to put it. The |
Based on #233 and #536.
The fork RFD already mentioned forking at a specific message. Since we now have the message-id RFD, we can extend fork with this capability to allow features like editing messages, regenerating responses, retroactively branching, etc.