Skip to content

Land #794: entry-point cost budgets, admin multisig decision, mainnet checklist - #815

Open
godamongstmen897 wants to merge 4 commits into
mainfrom
claude/serene-cori-a69qgk
Open

godamongstmen897 wants to merge 4 commits into
mainfrom
claude/serene-cori-a69qgk

Conversation

@godamongstmen897

Copy link
Copy Markdown
Contributor

Merges contributor PR #794 (mathstickz) onto main with these fixes:

Current numbers (gzip-free host metering):

flow instructions write entries write bytes
initialize 204,879 6 1,508
fund 292,583 4 1,196
mark_delivered 151,720 3 448
approve_milestone 385,176 7 1,168
claim_auto_release 397,882 6 1,056
raise_dispute 216,611 5 636
resolve_dispute 432,287 7 1,056

#762 is covered by ADMIN_MULTISIG_DECISION.md, and #763 by MAINNET.md, linked from the README.

Local run: all checks pass with 1234 tests.

Closes #762, closes #763, closes #764


Generated by Claude Code

hunter-baddie and others added 4 commits October 17, 2026 21:48
…checklist

cost_tests did not compile: the crate is no_std (println needs
extern crate std), InvocationResources::instructions is i64 and there is no
memory_read_bytes (mem_bytes is the memory figure). Disputes now require a
delivered milestone (#784), so the dispute flows mark delivery first.

#764 asks the test to fail on an extra storage write in fund; it only
asserted instructions > 0. Add per-entry-point budgets from the current
baselines (exact write entries, ~10% headroom on instructions and write
bytes). Checked: an extra persistent write in fund fails with
'fund: 5 ledger writes exceeds the budget of 4'.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017wV2kkBR3SpUPRe6CVe4te
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants