Repository navigation
Deliver child output through the physical continuation boundary - #5368
Merged
Mister-Meeseeks merged 4 commits intoOct 24, 2026
Merged
Conversation
Contributor
|
Independent review (James Reed, with Claude Code). Head Verdict: correct. Output delivery, exact parent restoration and the complete bill hold under an independent randomized nested harness. Checks
HarnessI used your
Result: 910 checks (455 calls, outer and nested), 0 failures. The registered replay passes locally. The scope (a reserved empty parent role, #5356 as a pending predecessor premise, no packet reorder or recursive C) is stated accurately. If #5364 rather than #5356 becomes the canonical stack, this boundary would need re-pricing against #5364's ABI, where |
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.
This closes the designated output-return boundary left by PR #5329 and PR #5356. One reusable relay tape preserves a binary child result through child cleanup and physical pop, installs it in a reserved empty parent role, and returns empty. All other parent payloads, heads, presence bits and descriptors return exactly. Nineteen tapes suffice for every finite properly nested successful call tree.
The proof and exact invoice, return routine, receipt and source pins define the scope. The complete matched push/return bill is
13Hp+4Sp+38k+18d+5q+2A+2Hc+5W+17L+61reads, writes and unit moves. The variables and input ABI are defined in the proof. This includes both output copies, all child cleanup, physical frame cleanup, descriptor return and relay cleanup. It excludes child computation and input/descriptor production. It has no ancestor-length or depth factor. Relay backing and the parent output are charged explicitly.The complete kappa calculation is unchanged. The parent must reserve an occupied empty output role. The byte-exact retained stack is from pending PR #5356 and remains an explicit predecessor premise. This does not implement packet reorder, recursive C, source counter/slot compilation, production descriptor delivery, complete supplier stock/precision or outer recovery. The actual-source calibration retains its Python allocation facade. Failed ABI calls need not restore partially changed state.
Validation on CPython 3.12.13, standard library only:
Both registered isolated, warning-strict normal and optimized commands pass
PASS_CONTINUATION_OUTPUT_RETURN. There are 1800 exact invoice cases, twelve nested cases through depth 31, six unchanged compact-source cases in both layouts, and six rejecting ABI controls. The source returns deliver all 1024 bits of an independently checked 128-record permutation. Each return costs 24388 tape actions at ancestor lengths 16,257,4096. Four disposable code mutations reject in both modes before receipt comparison. Committed-byte checks, official export integrity and both read-only Git-free export commands pass; all 16 exported files remain unchanged.Base is canonical main
7c472563474e60185e6ca101a17622133d047e56, priority 7, refreshed before submission. Base-source ancestry is on main. Replay pins bind the byte-exact vendored copies in this PR history; upstream_pin retains each original pending source. Clean checkout replay does not require that unmerged commit. The pending stack theorem remains an explicit premise, not an accepted result. The full-history wrapper, selected cold replay, full production rectangle, native compiler, Lean and Docker were not run. Hosted CI and independent human/proof-kernel acceptance remain separate.Overlap includes the live continuation Discussion, targeted all-state output-return PR search, refreshed public PR/Discussion bodies and a supplementary canonical-main README search. The sparse checkout does not contain all notes; arbitrary historical diffs and unpublished work remain unsearched. This PR adds only this package. The live stack scope was already complete, so the contribution takes its separate output-delivery handoff.
Credit @aliromman for the retained physical stack; @JoyboyBrian for the segment refinement and PR #5041 locality invoice; @jmbarrancoml for the sequential compact primitives in PR #2250; Douglas Colkitt for guard/repair; Cinna Davis for the caller; and @Matarener for cover/stock work. Their retained roles and notices are in the pinned source package. New work: Ali Romman with substantial OpenAI Codex assistance in proof, implementation, review and verification. Apache-2.0.