feat(web): fork from a selected assistant response#1
Closed
nassimna wants to merge 2 commits into
Closed
Conversation
Owner
Author
|
Superseded by #2, which contains only the agent thread-controls delta on top of pingdotgg#4248. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What Changed
thread/fork, OpenCodesession.forkwith the selected message ID, and Claude session resume/fork at the selected assistant UUID.Why
The header action could only fork the latest conversation state. Users exploring an earlier answer need the fork to start at that exact response, and silently approximating the boundary would give the provider different context from the history shown in T3 Code.
Placing the action beneath each response makes the fork point explicit and matches the existing message-action interaction model. Provider-specific message mapping keeps the visible transcript and native provider session aligned.
Dependency
This PR is stacked on the agent-managed side-thread foundation in pingdotgg/t3code#4246. It should be reviewed as the 16-file delta from
agent/forked-side-threads, then rebased onto upstreammainafter that PR merges.Verification
vp test runon the seven affected orchestration, provider-adapter, and message-timeline test files: 213 tests passed.UI Evidence
Before: latest-state fork in the header
After: fork action beneath the selected response
Forked side chat and provider follow-up
Interaction
Short selected-response fork and follow-up recording
Checklist