feat(metering): add base_meteredPriorityFeePerGas RPC endpoint #355
+1,849
−872
Conversation
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
niran
force-pushed
the
niran/priority-fee-1a-single-block
branch
from
aa07e74 to
2db0807
Compare
- Add MeteringTestContext for unit tests that don't need a full node - Move meter.rs tests inline using MeteringTestContext (sync tests) - Move block.rs tests inline using MeteringTestContext (sync tests) - Move rpc.rs tests inline using TestHarness with MeteringExtension - Delete tests/ directory (all tests now inline) - Update lib.rs to expose test_utils with #[cfg(any(test, feature = "test-utils"))]
niran
force-pushed
the
niran/metering-test-harness
branch
from
fa14653 to
64ae567
Compare
Address PR review feedback: - Delete test_utils.rs and remove test-utils feature to avoid duplication - Use TestHarness from base-client-node for all tests - Remove envelope_from_signed helper - use direct conversion - Remove unused imports
niran
force-pushed
the
niran/priority-fee-1a-single-block
branch
from
2db0807 to
7f4a2e1
Compare
Add priority fee estimation based on single-block resource consumption: - Add core estimation algorithm (compute_estimate) that determines the minimum priority fee needed for inclusion based on resource demand vs capacity - Add ResourceKind, ResourceLimits, ResourceDemand types for multi-resource estimation - Add base_meteredPriorityFeePerGas RPC endpoint - Document the RPC response format in README.md
niran
force-pushed
the
niran/priority-fee-1a-single-block
branch
from
7f4a2e1 to
07e3ad4
Compare
Collaborator
🟡 Heimdall Review Status
|
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.
Summary
Add a new RPC endpoint
base_meteredPriorityFeePerGasthat meters a bundle and returns a recommended priority fee based on resource consumption.This PR introduces the core estimation algorithm that answers: "What priority fee would my bundle need to pay to displace enough lower-paying transactions to free up the resources I need?"
Changes
base_meteredPriorityFeePerGasRPC endpoint that meters a bundle and estimates priority feesResourceKind,ResourceLimits,ResourceDemand,ResourceEstimatecompute_estimatealgorithm implementing top-down fee estimationestimate_from_transactionsfor single-block estimation from a list of transactionsAlgorithm
The estimation algorithm walks transactions from highest to lowest priority fee, accumulating resource usage until adding another transaction would leave insufficient capacity for the bundle. The last included transaction's fee becomes the threshold.
Response Format
{ "bundleGasPrice": "0x...", "bundleHash": "0x...", "results": [...], "totalGasUsed": 21000, "totalExecutionTimeUs": 1234, "priorityFee": "0x5f5e100", "blocksSampled": 1, "resourceEstimates": [ { "resource": "gasUsed", "thresholdPriorityFee": "0x...", ... }, { "resource": "executionTime", ... }, { "resource": "dataAvailability", ... } ] }Test Plan
compute_estimatecovering congested, uncongested, and edge casesestimate_from_transactions