fix(genvm): wire named fee accounting - #1750
Merged
Merged
Conversation
Contributor
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
kp2pml30
force-pushed
the
genvm/fee-fix
branch
from
September 3, 2026 06:59
3355f1a to
6b3ffdd
Compare
kp2pml30
force-pushed
the
genvm/fee-fix
branch
from
September 3, 2026 16:33
6b3ffdd to
e1447d3
Compare
kp2pml30
changed the title
kp2pml30
force-pushed
the
genvm/fee-fix
branch
from
September 4, 2026 05:25
e1447d3 to
880d798
Compare
kp2pml30
marked this pull request as ready for review
September 4, 2026 07:08
kp2pml30
force-pushed
the
genvm/fee-fix
branch
from
September 9, 2026 11:01
880d798 to
115271c
Compare
kp2pml30
force-pushed
the
genvm/fee-fix
branch
4 times, most recently
from
September 10, 2026 09:17
cbbc8cc to
a55e4d3
Compare
Integrate the GenVM named fee-bucket protocol across execution, settlement, Studio configuration, and fee reporting. GenVM now reports consumption as a name-keyed map instead of a positional list and gains a submitted-message count bucket; the reporting path accepts both shapes so legacy receipts keep decoding. Preserve compatibility for legacy unmetered messages and leader public data while pinning the matching GenVM revision. The v0.6 economics on main supersede the fee model this change was authored against, so it keeps main's pricing, overlay and phase-bound rules and maps the GenVM gas-data keys onto them. Repin GenVM to v0.6.0-rc5, whose v0.2 executor surfaces emitted envelopes under the canonical empty entry key, so v0.2 -> v0.3 messages are delivered instead of rejected. The gasless estimate helper wrote the method name under the legacy "method" key, which GenVM's closed entry key set rejects as malformed_entry; it now uses the same key as the SDK and the rest of the suite. Claude-Session: https://claude.ai/code/session_01U8TZ2YgVcjVV714P45m59i
kp2pml30
force-pushed
the
genvm/fee-fix
branch
from
September 10, 2026 13:43
a55e4d3 to
4f67dd8
Compare
4 tasks
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
1 participant
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.
Delivery Context
Depends-On: genlayerlabs/genvm-manager#31
Outcome
Validation
I had to fix a bunch of unrelated bugs