feat(execution): add validated context transformation - #7
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.
Summary
Completes the validated transformation, atomic execution, and post-commit finalization chain for
0.2.0-dev.5:D0-D4 remain zero-mutation. D5 is the sole deterministic message-context commit boundary. D6 is model-free derived observation and has no rollback authority.
Milestone progress
RecoveryVerifierValidatedPlan -> ExecutablePlanpreflightTransformationCandidategenerationD6 finalization boundary
ContextManager.estimateComponents()outputCOMMITTEDExecutionResultcontextGenerationmust equal D5's committed generationinventoryBeforeContextInventory.synchronize()rebuilds from committed messages without replacing the registryactualReductionTokens = before.totalTokens - after.totalTokensExecutionReportEXECUTION_FINALIZATION_FAILEDwith no actual-reduction claimEXECUTION_ABORTED, relabels D5, or rolls back committed messagesPreserved invariants
ValidatedPlan != ExecutablePlanValidatedTransformationalone is not an execution capabilityplanner-v1, inventory selection, budgets, protocols, policy, PAR/IPR, and telemetry remain hash-frozenExecutionReport -> STOP; no D7 scopeAccounting note
“Actual” means the observed signed difference under ContextOS's frozen-compatible canonical ContextManager estimator. It includes message serialization overhead, tool schemas/tool choice, and fixed prompt overhead; it does not claim tokenizer-exact llama.cpp backend counts.
Validation
npm.cmd test: 172 tests, 171 passed, 1 Windows symlink capability skipgit diff --checkaa59f4d: emptyDocumentation
Updated English and Traditional Chinese README, changelog, architecture, execution contract, technical report, and RFC.
This PR remains Draft as the complete dev.5 D0-D6 freeze candidate.