Repository navigation
fix(web): preserve task context and capability routing - #286
Merged
Merged
Conversation
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.
Workbench flows could retain the wrong context after the frontend roadmap changes: extension settings returned to Inspector instead of the requested input area, new-draft image checks disagreed with the model chosen for creation, and unavailable historical Run directories silently opened the current/source directory. Desktop bootstrap also omitted the backend's GitHub review capability.
This patch opens the same task's composer while retaining its draft and attachments, aligns default image eligibility with selectable model routing, fails closed with a retry when a Run's directory cannot be confirmed, and forwards the missing Desktop capability. Go ownership, protocols and execution permissions remain unchanged. It also fixes a pre-existing transcript test race by waiting for the observed React Query update without weakening its assertions.
Related to #257. The acceptance report records the four reproduced defects, remaining candidates and evidence boundaries; this PR does not close the broader roadmap.
Validation after merging latest main
4de4a916837142bc3100e78f492b7b5b67ce2c57, including #288 and #293 (current head1e614dbd031a6911abd23c55bbd148f53ce4f2be):docs/PROJECT_MEMORY.mdwas resolved by preserving all dated checkpoints from both branches. The automatic application-navigation merge retains this PR's composer destination and the new recovery routing.Original implementation evidence at
452b6fdcfd35c5049abc8b9e82c9390913cf6a5e(the browser experiments below were not repeated for this merge):452b6fdcfd35c5049abc8b9e82c9390913cf6a5e: affected-check selection, Go checks, TypeScript console and the Go control-plane gate. Unaffected checks were skipped by the existing classifier.npm test -- --maxWorkers=4). Four app-navigation, three image-routing, historical workspace and Desktop capability regressions failed before their respective fixes.The browser MCP peer is a deterministic local fixture. Native WebView2/GitHub login, real-model image/tool execution and the full model-to-preview/Git delivery journey are outside this acceptance slice.