refactor(agent): derive recovery state from record queries - #7784
Open
christianklotz wants to merge 9 commits into
Open
refactor(agent): derive recovery state from record queries#7784christianklotz wants to merge 9 commits into
christianklotz wants to merge 9 commits into
Conversation
christianklotz
marked this pull request as ready for review
August 7, 2026 15:24
christianklotz
marked this pull request as draft
August 7, 2026 15:31
davidbrai
reviewed
Aug 7, 2026
christianklotz
marked this pull request as ready for review
August 7, 2026 15:42
davidbrai
reviewed
Aug 7, 2026
badlogic
reviewed
Aug 7, 2026
| Reduction may additionally perform point lookups for provisioned entry ids and bounded branch lookups for effective model, thinking, and active-tool configuration at the operation anchor. These are indexed lookups, not extra history scans. Every scan is bounded by the open operation or the still-relevant idle queue, not by total session history or another lane's activity. | ||
| **Potential lookup-efficiency gap:** finding the latest run is linear in the number of consecutive non-run operation starts after it. Geometric limits return fewer than twice that many starts, and normal histories have only a few intervening compaction or navigation operations. The worst case remains a lane with many non-run operations and no recent run; backends may optimize that case internally if measurements justify it. | ||
|
|
||
| Reduction may additionally perform point lookups for provisioned entry ids and bounded branch lookups for effective model, thinking, and active-tool configuration at the operation anchor. After discovery identifies the recovery boundary, these reads are bounded by the open operation or the still-relevant idle queue, not by another lane's activity. |
Collaborator
There was a problem hiding this comment.
this is no longer needed, as config (model, thinking level, active tools) have moved into lanes. reduction is literally only entry id look ups now. will push new harness-v2.md in a bit, so you can review.
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
3 participants
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
findOpenOperations(), and derive recovery state through boundedfindRecords()callsEntryQuery.customTypeto custom entries and align the harness designTesting
npm run check./test.sh: 15 unrelated coding-agent failures from the missingTuiMainScreenexport, plus one footer watcher timeout