Repository navigation
Conversation
Signed-off-by: aihop <4408344+hhyykk@users.noreply.github.com>
This branch has not been deployed
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.
Goal And Delivered Outcome
When explicit action selection is deferred or rejected and no scheduler is configured, quota can pass
resolve_scheduler_execution_context(None)into the recovery command renderer. The renderer raisesValueError, so callers receive a generic quota/receipt failure instead of the intended read-only reentry command._selection_recovery_commandnow uses the existing neighboring projection policy: omit unrenderable optional scheduler arguments while retaining registry, runtime, Goal, Agent, Turn and capability arguments. Valid declared scheduler profiles remain present. The scheduler validator and typed admission/settlement rules are unchanged; a refused selection still cannot execute work or spend quota.mainat3dd3c550afecfa0c525a34e8744a3cf06774ac94.Author Declaration
interaction_contract.py::_selection_recovery_commandScope And Continuation
Complete within recovery command projection. Existing Python CLI glue remains the owner; no parallel decision logic, schema, provider, configuration or authority is introduced. Related broader refactoring is unnecessary: the shared renderer and existing neighboring fallback already supply the behavior. An independently reviewable repair; no rollout or merge is performed.
Validation
12a5c685db2bad0ab6b36d55e19b32dbf5020abd(same source tested before commit).test_unadmitted_selection_construction.py: 56 passed; adjacent scheduler context, action-selection conflict and selected quota-settlement cases: 261 passed, 76 deselectedloopx canary premerge --from-git-diff --git-diff-base origin/main: 5 direct checks and 18 selected checks, no failures or advisory failuresloopx checkscoped to both files: no errors or warningsThese checks cover all three recovery-command callers, optional scheduler handling, reentry identity and refusal boundaries. They do not certify every host integration or complete operational qualification. Hosted CI and exact-head maintainer review remain separate; no merge authorization is inferred.
Frontend / Visual Evidence
UI impact: none. CLI recovery projection only; no visual surface changed.
Type of Change / LoopX Area
Bug fix and regression test; control plane / host runtime integration.
Technical Direction
Bounded S10 reliability/diagnostics repair under the existing overall roadmap.
Shared-authority RFC fixture impact
N/A: no authority-provider, schema, promotion, persistence or compatibility projection changes.
Boundary Checklist