Repository navigation
Settle retained worker feedback without replaying execution - #61
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.
A worker can finish its model calls, retain valid feedback, and pause for factory-owned verification while an earlier completion hold still blocks resumption. Repeating the model call would duplicate execution and lose the relationship to the original result.
Add an explicit native settlement API. A candidate verifies the complete intent, checkpoint, response and consume chain against the current run, model, task, source and operator. A confirmed request appends an immutable completion receipt and updates only the corresponding hold projection in one transaction. The receipt preserves the original hold. It grants no new execution, tool call, budget, resume or Git action.
The API rejects unknown outcomes, incomplete custody, changed authority and unsupported restored custody. Every returned receipt is screened against current credentials. The contract covers stored typed WorkerFeedback; ordinary reports without feedback require a separate producer integration and are documented as unsupported here.
Validation: 87 focused host tests; four controlled PostgreSQL cases pass, including atomic rollback, competing requests, revoked access and unknown outcomes. These use controlled provider responses and do not establish live CRM acceptance. The first PostgreSQL run exposed an incorrect split-worker fixture; that failure is retained, and the successor uses the existing combined-owner fixture pattern. The complete publication source matches the tested successor. Ruff, mypy, strict documentation, package contents and isolated import smoke pass. Full hosted CI is pending.