Repository navigation
fix(heartbeat): keep checkpoint capture independent of input display - #6659
Closed
loopx-agent wants to merge 1 commit into
Closed
loopx-agent wants to merge 1 commit into
loopx-agent wants to merge 1 commit into
Conversation
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
Collaborator
Author
|
Superseded by standalone main-targeted #6661 at 9f1fc4c. The capture/input isolation patch is independent of the planning-horizon proposal; the research integration will compose the exact PR heads. Earlier integration-head validation remains bound to its original revision and is not presented as new-head validation. No branch was deleted or force-pushed. |
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.
Checkpoint handoff silently selected TurnEnvelope even when the benchmark display option was off: the worker requested a capture root, and the heartbeat renderer appended
--turn-envelopeand switched its dispatcher. Capture now preserves the ordinary dispatcher/quota projection. Only an explicitheartbeat-prompt --turn-envelope(with a capture root), or the existing explicit worker option, selects the short view. Generated selection and regeneration commands retain the chosen transport without changing admission or settlement owners.This changes the previous capture-root-only renderer behavior; the protocol/runtime guides document the migration. Full private captures and checkpoint handoff remain available in both display modes. Receipt projection now follows the explicit display option, and historical receipts are preserved. The existing helper is simplified rather than retaining a checkpoint-to-display compatibility mapping; no new strategy or permission owner is introduced.
Validation: a regression fails on the previous implicit activation; 247 focused tests pass on the candidate, including real worker → renderer → quota/selection/replay → single settlement with display on/off and handoff on/off. Another 247 optional Harbor/native tests are skipped in that local environment, not counted as passes. Semantic vocabulary smoke, public boundary check and changed-runtime Ruff pass; an unchanged test file has five pre-existing Ruff E702 findings. The clean exact head also passes the complete isolated non-root Linux qualification on the current pinned provider: six settled work wakes, pending pre-replan capture, ordinary and stride receivers, automatic full recapture, another receiver’s actual work, and the generic Core Goal journey without benchmark imports. The six underlying tests pass with no skips, using a real native binary, HTTP/SQLite/Docker and deterministic local inference; this proves mechanics, not model research gain. Ordinary-package validation is still being completed. No paid model or benchmark task run is launched.
Base: the latest owner-selected research integration,
codex/loopx-research-integration-20261016-03at57e909e694b66e34784241839a350bad6d763531. This PR contains only the input/capture separation and its focused validation/docs. Independent exact-head review and merge authorization remain separate from test success.