Skip to content

Conversation

prestwich
Copy link
Member

@prestwich prestwich commented Aug 20, 2025

Adds a small wrapper that removes Signet System Transactions from eth_feeHistory caching, to avoid displaying artificially low priority fee histories for blocks that have a high ratio of system to non-system txns

Copy link
Member Author

prestwich commented Aug 20, 2025

@prestwich prestwich changed the title refactor: break ctx into modules fix: improve fee history by removing 0 priority-fee transactions before sending notifications to cache Aug 20, 2025
@prestwich prestwich marked this pull request as ready for review August 20, 2025 15:58
@prestwich prestwich requested a review from a team as a code owner August 20, 2025 15:58
@prestwich prestwich changed the base branch from main to graphite-base/15 August 20, 2025 16:21
@prestwich prestwich changed the base branch from graphite-base/15 to prestwich/split-ctx August 20, 2025 16:21
@prestwich prestwich force-pushed the prestwich/split-ctx branch from 34e94f6 to 35cd9e5 Compare August 20, 2025 16:23
@prestwich prestwich force-pushed the prestwich/fee-est-fix branch from 48087fb to ee741ac Compare August 20, 2025 16:23
@prestwich prestwich changed the base branch from prestwich/split-ctx to graphite-base/15 August 20, 2025 16:37
@prestwich prestwich force-pushed the prestwich/fee-est-fix branch from ee741ac to 54119d5 Compare August 20, 2025 16:37
@prestwich prestwich changed the base branch from graphite-base/15 to main August 20, 2025 16:37
Copy link
Member

@Evalir Evalir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm. truncating is fine as signet system txs are added bottom-of-block

@prestwich prestwich merged commit 00688a7 into main Aug 20, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants