Repository navigation
Retain Turn admission declarations - #574
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
1 participant
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.
Active Turn messages and function results were checked against the latest Runtime heartbeat, allowing later declarations to change what an already admitted Turn could accept. Retain the admission declaration for the Turn’s delivery lifetime, including confirmed no-effect repreparation, while preserving fresh admission and final preclaim checks for new work.
Runtime steering now checks its existing declaration snapshot before native calls and retains assignment fencing and input receipts. Remove the unused live-heartbeat delivery helper and function-exchange kind field; no protocol fields, storage, or adapter loops change.
Validation: fresh independent blind review found no actionable issues. Real gateway/database regression cases cover narrowing, widening, unavailability, fresh Turns, images/function results and repreparation; all 11 scenarios passed 50 race-enabled repetitions. Runtime steering race50, focused suites/vet, runtime contract and hygiene passed. Independent review also ran full dispatch/execution race tests and repeated integration tests. Focused checks passed again after merging current cutover. Live model loops are unchanged; final integrated public qualification remains separate.
Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.