docs(core): the guide and specs state the executed-call refreeze and hook turn granularity - #256
Merged
Conversation
…hook turn granularity Three surfaces still said the refreeze needs a SUCCESSFUL mutating call, which #242 dropped: the frozen prompt's guide and the activation sentences of --spec tools and --spec skills. The permissions, settings, and memory surfaces already said every/any/the-writing call, so two specs disagreed about the same event, and a model whose write-then-selftest exited nonzero got a receipt saying its tool is live beside a guide saying it is not. The guide also lumped hooks into "apply on their next use": hooks are discovered per turn, so a new hook applies from the next turn, and the receipt and --spec hooks already say so. The failed-call nuance lives in the two specs (pay-on-read); the guide carries the trigger only, inside the frozen prompt byte budget. Drift tests ban the stale phrase on every static surface and pin the guide's two claims.
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.
Why
Round-7 contract audit (F2, F5). PR #242 made the mid-turn refreeze fire after every EXECUTED mutating call, a failed one included, because a bash write-then-selftest that exits nonzero has still persisted the file. Three surfaces still described the old trigger: the frozen prompt's self-extension guide,
--spec tools, and--spec skills("after a successful mutating call"), while--spec permissions,--spec settings, and--spec memoryalready state the executed/any/writing-call truth. Inside one turn a model could hold a receipt saying its new tool is live and a guide saying the refreeze it needed did not happen.The guide also said "Hooks, permissions, and templates apply on their next use". Hooks are discovered per turn; the receipt ("approved hooks apply from the next turn") and
--spec hooks("re-discovered every turn") both say so. Lumping the three invited the install-the-gate-then-prove-it shape at a granularity hooks do not have.Summary
frozen_prompt_fits_token_budgetstill passes; the first draft that carried the nuance in the prompt failed it and was tightened).--spec toolsand--spec skills: "after every executed mutating call (a failed call that wrote the file still activates it)".extensions_touchednow says executed, citing fix(core): refreeze after a mutating call that wrote a file and then failed #242.Test Plan
the_guide_states_executed_call_and_turn_granular_hooks: pins the two guide claims and bans the stale phrase. Red against the old wording.no_spec_surface_requires_a_successful_mutating_call: iterates every static--specsurface, bans the phrase everywhere. Red against the old tools/skills sentences.Greptile Summary
This change updates the self-extension guide and rendered specifications to accurately explain activation timing. Tools and skills refresh after every executed mutating call, including a call that writes a file and then fails; permissions and templates apply on their next use; and newly installed hooks are discovered for the next turn.
The focused runtime test confirmed that a tool written by a script that exits nonzero is included in the following model request. The focused hook test confirmed that the currently discovered hook set remains in effect during a turn and that a fresh discovery controls the following turn. The updated guide and specification assertions passed. No defects were found.
Confidence Score: 5/5
Safe to merge: the product-facing documentation and static specifications now match the exercised runtime behavior.
Focused runtime checks covered a failed mutating writer, next-request refreeze, and hook behavior across a turn boundary; guide and specification regression checks also passed. No actionable findings remain.
Files Needing Attention: No files need follow-up attention.
What T-Rex did
Reviews (1): Last reviewed commit: "docs(core): the guide and specs state th..." | Re-trigger Greptile