Simplify loop-engineering for Astra, Fable and Sol orchestration - #41
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.
Loop-engineering mixed model behavior with host capabilities and repeated conflicting instructions across its references. The docs now have one portable workflow with explicit Astra/Fable guidance, Sol orchestration, and an evidence-preserving fallback for other models. Model identity grants neither tool access nor write isolation.
Reviewed all 11 existing instruction documents and added one focused model reference. Reduced total documentation from 14,466 to 6,910 words. Removed historical anecdotes, mandatory questionnaires, hidden-reasoning requests and repeated workflow prose. Corrected scheduler status, changed-goal recovery, Codex-only skill discovery, and successor driver configuration so radar and queue probes survive resume. A scoped wave can finish without archiving unrelated project work.
Replaced 20 wording-only state tests, one obsolete sibling wording assertion and a duplicated static prose checklist with executable documentation examples and reference graph checks. All 26 retained state tests are AST-identical; runtime sources and packaged code are unchanged. The examples now verify completion evidence, bound successors with both probes, and verdict exit handling. Added a Codex-only-root resolver fixture. Retained the existing executable whole-project completion recipe and its real-vault tests for success, orphan refusal and rejected pushes.
Validation:
Worklog: generic-loop-engineering / loop-model-contracts, loop-docs-optimization, loop-model-portability. Current model references are linked from the maintained model guide. Word-count reduction is measured, not a claimed token or quality benchmark.