Plan: correct two claims in the Phase 2 reference notes - #17
Merged
Conversation
Post-merge review of the notes (five reviewers, findings verified against CHIP-0025 and CHIP-0049) surfaced two wording defects worth correcting before Phase 2 consumes them. First, the message-condition genealogy was wrong and pointed the costing question at the wrong precedent. Our tx-scoped SEND/RECV_MESSAGE conditions port Chia's SEND_MESSAGE and RECEIVE_MESSAGE (CHIP-0025), the announcements' successors, and CHIP-0049's always-priced exception enumerates only the four announcement codes, leaving Chia's own message conditions on the free tier. The note now states the split precedent explicitly instead of implying announcement pricing carries over. The free tier's per-coin-spend granularity is also now stated, since the multiplication across spends is the DoS-relevant part of that decision. Second, obligation 2 commits to superlinear per-condition pricing, not to a stated pricing-over-caps philosophy, so the gloss claiming that commitment is softened to consistency. The taproot bullet's headline now separates what is ratified (no EC arithmetic in the VM, the D2 muladd decline) from what is open (the condition-layer form, owed to CONDITIONS.md v0).
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
Two corrections and one precision fix to the Phase 2 reference block that PR #15 added to
docs/execution-plan.md, from a post-merge review Evan requested. No spec, implementation, or vector changes, and no skeleton element touched.How to verify
git diff main~1 -- docs/execution-plan.mdshows the original block plus this correction as one readable unit.docs/bitcoin-script-successor-evaluation.mdsection 7.spec/VM.mdsection 8, D2 entry.PATH=.venv/bin:$PATH ci/lint/lint.sh