Repository navigation
[6/7] feat: reuse workspace context and navigate agent output - #136
Draft
BIackFIame wants to merge 69 commits into
Draft
BIackFIame wants to merge 69 commits into
BIackFIame wants to merge 69 commits into
Conversation
This was referenced Oct 7, 2026
Contributor
Author
|
@codex review Please review this PR for correctness, regressions and security issues. Review only; post findings on GitHub without changing code. This is part 6/7 of a cumulative series. Focus on changes introduced after predecessor |
…cklog/1-orchestration
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.
Summary
Part 6/7 of the CanvasTTY agent-workflow series. Context previews, session snapshots, presets, output search, deliberate shared input and layout actions make workspaces easier to resume and navigate. Approved project memory is delivered through the existing plugin API. This part also includes renderer/loading fixes and the source Electron dependency update.
Review scope and dependencies
Depends on: #135.
This is a cumulative upstream PR: GitHub's Files changed tab includes earlier unmerged parts because all PRs target upstream
main; cross-fork stacked bases are unsupported. Review this part in the isolated predecessor-to-head diff. The intended merge order follows the numbered series; do not merge a later part independently of its predecessors.The Context plugin owns memory storage and tools; its main is published at 63458cf.
Validation and current blockers
Tested feature head:
e2c0b767ae2c5c91e20e13daa5f2a654aaf6029e. Local helpers, typecheck, build and secret audit passed; full suite: 2,022 passed, 10 skipped, zero failures. Historical fork CI passed on macOS, Linux and Windows using the exact candidate tree under a temporary workflow overlay. That overlay is not included here.Draft: merge preview against updated upstream
6faadcdcfound conflicts insrc/preload/index.ts,src/renderer/src/features/workspace/WorkspaceCanvas.tsx,src/shared/contracts.ts. These historical results do not validate a conflict-resolved integration with current main. Preserve the newly accepted material/terminal-link behavior, resolve the overlaps, and rerun checks before marking ready.Historical fixtures covered approved memory delivery and one output-search match. Snapshot/preset, broadcast and layout behavior have automated coverage; the full visible-app workflow and a ten-terminal search benchmark were not repeated.
Series order
Parts 2–7 are cumulative Draft PRs against upstream main, with an isolated layer diff in each description. Their tested feature heads currently conflict with newer upstream changes; integration and fresh checks are required before readiness. Part 7 also retains its independent experimental acceptance gate.