Skip to content

chore: remove old TxPool implementation#22028

Merged
alexghr merged 2 commits into
merge-train/spartanfrom
pw/remove-old-tx-pool
Mar 26, 2026
Merged

chore: remove old TxPool implementation#22028
alexghr merged 2 commits into
merge-train/spartanfrom
pw/remove-old-tx-pool

Conversation

@PhilWindle

Copy link
Copy Markdown
Collaborator

Summary

  • Removes the old TxPool / AztecKVTxPool implementation that has been fully replaced by TxPoolV2
  • Deletes mem_pools/tx_pool/ directory (24 files, ~5200 lines) including all eviction rules, tests, and benchmarks
  • Removes barrel exports of old TxPool type from p2p/src/index.ts and mem_pools/index.ts
  • Inlines the only shared utility (getTxPriorityFee) into tx_metadata.ts

Test plan

  • yarn build passes
  • All 61 p2p test suites pass (1193 tests)
  • No external packages import the old TxPool type

🤖 Generated with Claude Code

PhilWindle and others added 2 commits March 25, 2026 22:21
The old TxPool (AztecKVTxPool) has been fully replaced by TxPoolV2.
Remove the entire mem_pools/tx_pool directory and its barrel exports.
The only shared utility (getTxPriorityFee) is inlined into tx_metadata.ts.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@alexghr alexghr merged commit fc2c957 into merge-train/spartan Mar 26, 2026
11 checks passed
@alexghr alexghr deleted the pw/remove-old-tx-pool branch March 26, 2026 11:11
github-merge-queue Bot pushed a commit that referenced this pull request Mar 27, 2026
BEGIN_COMMIT_OVERRIDE
fix: only clear provenBlockNumber when it exceeds prune point (#21946)
chore: (A-779) load all accounts before calling
LogService.#getSecretsForSenders (#21923)
fix: align staging-public mana target with testnet/mainnet (#21983)
chore: (A-777) add warn logs for regressive path in block synchronizer
(#21925)
fix: fully validate txs retrieved from tx file store (#21988)
refactor: extract checkpoint proposal handling to ProposalHandler
(#21999)
fix: unbounded memory in calldataRetriever (#22004)
fix(p2p): check peer rate limit before global to prevent quota
starvation (#21997)
fix(p2p): evict expired failed-auth-handshake entries on heartbeat
(#21992)
chore: defensively handle skipPushProposedBlocksToArchiver (#22017)
chore: bump testnet prover resource profile to prod-hi-tps (#22019)
chore: (A-835) remove unused serializer (#22037)
fix(p2p): remove disconnected peers from scoring maps (#22009)
fix(e2e): set anvilSlotsInAnEpoch in slashing tests (#21869)
fix(ethereum): Audit fixes A-810, A-812 (nonce race, isEscapeHatchOpen
logging) (#21948)
chore: remove old TxPool implementation (#22028)
fix: Fix blob encoding when uploaded from proposals (#22045)
chore: Adds /cycle and /fix skills. Also configures linear mcp server
(#22043)
chore: remove validatorReexecute config option (#22024)
fix(sequencer): use last L1 slot of L2 slot as eth_simulateV1 timestamp
(#22023)
docs(simulator): clarify teardown gas billing is intentional (#22057)
chore: revert account loading optimization in log service (#22062)
fix: use DateProvider in PeerScoring (#22070)
fix(aztec.js): preserve extraHashedArgs in DeployMethod.with() (#22053)
fix(p2p): replace process.exit() with graceful shutdown in worker
cleanup (#22046)
chore: merge next (#22089)
fix(stdlib): correct NoteDao size (#22068)
feat: improve blob download from filestores (#22096)
fix: remove stale tx_pool v1 benchmark reference (#22104)
END_COMMIT_OVERRIDE
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