Expand unit 6b: includes, computed constants, inline functions - #55
Merged
Conversation
…tions A two-corpus census (tibetswap b82e921, chia-gaming cdab9c9) found source-level import of shared definitions in every production puzzle file, tibetswap defining 124 of 136 functions inline, chia-gaming using computed compile-time constants 36 times, and let effectively absent at two uses, both in test files. The census also corrected the deferral's premise: chia-gaming's source imports are universal (about 200 import and include statements across 19 puzzle files), and its compiled-sibling hash plumbing lives in its build layer, not the language. Unit 6b is therefore resequenced ahead of unit 7 so the benchmark puzzles share their tree-hash helpers instead of pasting them four times, and unit 7's style constraint is widened to admit the new surfaces. assign stays held on unit 7 evidence with a pre-registered trigger, and the compiled-sibling import stays deferred. Decision by Evan, 2026-08-16, skeleton change under ground rule 6.
chia-gaming and tibetswap back the unit 6b census, and the script now clones them so the survey is reproducible from a fresh checkout.
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.
What changed
Docs only, plus one line-item in the reference fetch script. No spec or implementation contact, so no spec section applies. The unit 6b bullet in
docs/execution-plan.mdis rewritten: scope expands from a bare include mechanism to source-level include of shared constants and functions, computed compile-time constants, and inline functions, and the unit is resequenced ahead of unit 7. Unit 7's style constraint is widened to admit the new surfaces.tools/fetch-references.shgains the two corpora the decision cites.Why
A census of the two deployed-puzzle corpora (tibetswap
b82e921, chia-gamingcdab9c9) found:defconst) 36 times.letis effectively unused: two occurrences across both corpora, both in test files, so it stays out.assignappears only in chia-gaming (about 20 on-chain uses), so it stays held on unit 7 evidence, now with a pre-registered trigger.The census also corrected the prior deferral's premise. chia-gaming did not abandon source-level import: its imports are universal, and the compiled-sibling hash plumbing the old bullet described lives in its build layer, not its language. That import mechanism stays deferred, with
bitlisp-compile -Tcovering the need manually.Decision by Evan, 2026-08-16, skeleton change under ground rule 6.
Read order
docs/execution-plan.md(the unit 6b bullet, then the unit 7 style constraint)tools/fetch-references.shVerify independently