Skip to content

Conversation

@dancoombs
Copy link
Collaborator

Proposed Changes

  • Add support for new entrypoints to builder
  • Remove ops from pool by id instead of by hash to cover the case of a bad hash being produced by a EIP-7702 initcode packed user op. This may lead to a situation where the wrong op gets removed from the pool (if the old op has been replaced), however this is reasonable to do as removing ops from a reverting sender is healthy for the mempool.

@codecov
Copy link

codecov bot commented Dec 23, 2025

Codecov Report

❌ Patch coverage is 17.74194% with 102 lines in your changes missing coverage. Please review.
✅ Project coverage is 59.85%. Comparing base (e5997cf) to head (9e51e16).

Files with missing lines Patch % Lines
crates/pool/src/server/remote/server.rs 0.00% 20 Missing ⚠️
bin/rundler/src/cli/builder.rs 0.00% 18 Missing ⚠️
crates/pool/src/server/local.rs 0.00% 17 Missing ⚠️
crates/pool/src/server/remote/client.rs 0.00% 16 Missing ⚠️
crates/builder/src/task.rs 0.00% 10 Missing ⚠️
bin/rundler/src/cli/mod.rs 0.00% 9 Missing ⚠️
crates/builder/src/bundle_sender.rs 64.28% 5 Missing ⚠️
crates/aggregators/pbh/src/proxy.rs 0.00% 4 Missing ⚠️
crates/rpc/src/rundler.rs 0.00% 2 Missing ⚠️
bin/rundler/src/cli/proxy.rs 0.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

Files with missing lines Coverage Δ
crates/builder/src/bundle_proposer.rs 94.33% <100.00%> (ø)
crates/types/src/pool/traits.rs 100.00% <100.00%> (ø)
crates/types/src/proxy.rs 100.00% <100.00%> (ø)
bin/rundler/src/cli/proxy.rs 0.00% <0.00%> (ø)
crates/rpc/src/rundler.rs 0.00% <0.00%> (ø)
crates/aggregators/pbh/src/proxy.rs 0.00% <0.00%> (ø)
crates/builder/src/bundle_sender.rs 77.18% <64.28%> (+0.02%) ⬆️
bin/rundler/src/cli/mod.rs 0.00% <0.00%> (ø)
crates/builder/src/task.rs 2.66% <0.00%> (-0.03%) ⬇️
crates/pool/src/server/remote/client.rs 0.00% <0.00%> (ø)
... and 3 more
Flag Coverage Δ
unit-tests 59.85% <17.74%> (-0.03%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
rundler binary 0.00% <0.00%> (ø)
builder 75.70% <54.54%> (-0.03%) ⬇️
dev ∅ <ø> (∅)
pool 65.72% <0.00%> (-0.15%) ⬇️
provider 23.97% <ø> (ø)
rpc 27.71% <0.00%> (ø)
sim 81.19% <ø> (ø)
tasks ∅ <ø> (∅)
types 68.86% <100.00%> (ø)
utils 51.38% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@dancoombs
Copy link
Collaborator Author

TODO: remove the by id code - a later PR fixes the hash calculation for 7702 by pulling the auth from the transaction.

@dancoombs dancoombs merged commit 63cf456 into danc/new-uo-pool Jan 6, 2026
9 of 10 checks passed
@dancoombs dancoombs deleted the danc/new-uo-builder branch January 6, 2026 15:09
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.

1 participant