fix: set the proper max buffer size for the blockfetch and txsubmission protocols

[etorreborre]

According to the network specification.

Summary by CodeRabbit

• Chores
  • Adjusted network buffer limits for block fetching and transaction submission to optimize memory usage and improve data transmission stability and throughput during syncing and submission.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

Adjusts buffer limits in two protocol modules: BlockFetch reduces max_buffer from 25,000,000 to 2,500,000 bytes for both initiator and responder registrations; TxSubmission increases max_buffer from 5,760 to 2,500,000 bytes. No other logic or public signatures changed.

Changes

Cohort / File(s)	Summary
BlockFetch buffer reduction crates/amaru-protocols/src/blockfetch/mod.rs	Lowers max_buffer in register_blockfetch_initiator and register_blockfetch_responder from 25,000,000 → 2,500,000 bytes.
TxSubmission buffer increase crates/amaru-protocols/src/tx_submission/mod.rs	Raises max_buffer in register_tx_submission from 5,760 → 2,500,000 bytes.

Estimated Code Review Effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

Buffers trimmed and buffed like a retro arcade sprite,
One goes small, the other roars into the night,
Code stays neat, no drama in sight,
Merge it, ship it — press Start and take flight ✨🎮

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: adjusting max buffer sizes for blockfetch and txsubmission protocols, which is exactly what the changeset implements.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines	Coverage Δ
crates/amaru-protocols/src/blockfetch/mod.rs	88.73% <ø> (ø)
crates/amaru-protocols/src/tx_submission/mod.rs	89.18% <100.00%> (ø)

... and 6 files with indirect coverage changes

🚀 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.

[rkuhn]

We can set this limit, it is the one written in the network spec, but ultimately this limit depends on how many blocks are in the range that we request. The pessimistic assumption is that we get all the blocks from the server before we can start processing them, meaning that they must fit into the muxer buffer. It was my understanding that blocks are currently smaller than 100kB and will get bigger with Leios. We’ll have to update this limit once we settle on our block fetching strategy.