Repository navigation
Make SI check intent before work and keep authorized work moving - #58
Draft
infotradescout wants to merge 4 commits into
Draft
infotradescout wants to merge 4 commits into
infotradescout wants to merge 4 commits into
Conversation
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.
Selective Intelligence could begin Worker work before checking the user's intended outcome, while a refusal for one tool action could reopen authorization for unrelated work. This PR makes Orchestrator, Worker/Builder, and Objector standing responsibilities: reconstruct intent, challenge a plausible material misreading before Worker dispatch or work, execute the bounded task, and review the result and proof before reporting. A denied action stays scoped to that action; clear reversible work continues. Formal Council roles remain conditional.
The canonical skill, ChatGPT adapter, public plugin description, fallback guide, discovery files, and README now carry the same role order. Checkpoint saving uses short atomic temporary names so deep Windows worktrees can save and resume. The public runtime keeps its 61-entry limit and required activation and execution files; two optional references remain in the full source release but are omitted from that runtime bundle.
Validation on clean native Linux source
6b9cc1e9fa13c31398e6f874ec4a42fe97acaf04: the unchanged SI quality gate passed all five checks. The parent runtime commit5fbdbc42d86ad6a5a4520ec47369237f3944e304also passed discovery bridge 11/11, native pointers 8/8, ChatGPT adapter 59 files with one skill entrypoint, public plugin 4/4, and extracted adapter delivery 25/25; the follow-up commit changes only README. Independent Objector reviews passed the role contract, runtime integration, and README correction. GitHub's Quality job for6b9cc1efailed before any job steps ran, so hosted execution is absent. Fresh model-driven behavior and all-client acceptance remain separate; this PR stays draft and unmerged.