Remove backwards compatibility from Batcher

[arr00]

Summary by CodeRabbit

• Refactor

  • Streamlined batch dispatch logic by optimizing internal implementation.

• Documentation

  • Added clarification regarding compatibility considerations with ERC7984ERC20Wrapper versions during batch operations.

[netlify]

✅ Deploy Preview for confidential-tokens ready!

Name	Link
🔨 Latest commit	b9778dc
🔍 Latest deploy log	https://app.netlify.com/projects/confidential-tokens/deploys/69af077e75eb83000854051d
😎 Deploy Preview	https://deploy-preview-321--confidential-tokens.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 2dd6e1ff-9aa9-4044-90d7-857734158ad3

📥 Commits

Reviewing files that changed from the base of the PR and between d7ecc2c and b9778dc.

📒 Files selected for processing (1)

• contracts/finance/BatcherConfidential.sol

---

Walkthrough

The BatcherConfidential contract is refactored to inline the _calculateUnwrapAmount helper function directly into the dispatchBatch method. Explanatory comments are added regarding ERC7984ERC20Wrapper compatibility and potential dispatch failures.

Changes

Cohort / File(s)	Summary
Function Inlining & Documentation contracts/finance/BatcherConfidential.sol	Removes internal _calculateUnwrapAmount(euint64) helper function and inlines its logic into dispatchBatch. Adds clarifying comments about ERC7984ERC20Wrapper compatibility concerns.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 One helper function, now set free,
Its logic flows where it should be,
Inlined with care in dispatch's call,
Cleaner code, and notes for all,
Comments bright—now nothing's small! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'Remove backwards compatibility from Batcher' is concise and directly reflects the main change—removing the _calculateUnwrapAmount internal function and consolidating unwrap logic inline in dispatchBatch, eliminating legacy compatibility patterns.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch chore/batcher-remove-backwards-comp

---

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