fix: don't check for empty account with data

[bmuddha]

Summary by CodeRabbit

• Changes
  • Removed post-execution rent-exemption verification. Accounts are no longer validated for rent-exemption requirements after transaction execution completes, changing how account state validation is handled in the transaction processing pipeline.

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

[coderabbitai]

📝 Walkthrough

Walkthrough

A post-execution rent-exemption verification loop has been removed from the verify_account_states function in magicblock-processor/src/executor/processing.rs. The deleted code previously validated that accounts with nonzero data maintained sufficient lamports to meet minimum rent-exemption requirements. When conditions were unmet, the loop would generate an InsufficientFundsForRent error, log messages, and halt processing. This verification was gated by the presence of a rent collector. The removal eliminates post-execution per-account rent-exemption validation while preserving other verification logic.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch bmuddha/fix/no-empty-account-check

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 710792e and 14e2ea6.

📒 Files selected for processing (1)

• magicblock-processor/src/executor/processing.rs (0 hunks)

💤 Files with no reviewable changes (1)

• magicblock-processor/src/executor/processing.rs

---

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

[github-actions]

Manual Deploy Available

You can trigger a manual deploy of this PR branch to testnet:

Deploy to Testnet 🚀

Alternative: Comment /deploy on this PR to trigger deployment directly.

⚠️ Note: Manual deploy requires authorization. Only authorized users can trigger deployments.

Comment updated automatically when the PR is synchronized.

[snawaz]

@bmuddha

Looks like you didn't wait for CI to complete before merging this PR to master! run_make_ci_lint actually failed.