chore: change ActionArgs::default to ::new

[jonasXchen]

avoid using ::default, instead use ::new

Summary by CodeRabbit

• Refactor
  • Updated constructor method naming convention for the ActionArgs API to follow standard practices.

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

[coderabbitai]

📝 Walkthrough

Walkthrough

The pull request renames the associated constructor method on ActionArgs from default(data: Vec<u8>) -> Self to new(data: Vec<u8>) -> Self in the magicblock-magic-program-api/src/args.rs file. The initialization logic remains unchanged, with escrow_index set to 255 and the provided data assigned. This is purely a method signature rename with no behavioral changes.

Possibly related PRs

• feat: ActionArgs::default() to abstract escrow_index #662: Directly modifies the ActionArgs constructor in the same file, adding a default() constructor, getters, and a with_escrow_index builder method to complement the renamed new() constructor.

Suggested reviewers

• taco-paco

✨ 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 jonasXchen/action-args

---

📜 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 aedccd9 and 678c4ea.

📒 Files selected for processing (1)

• magicblock-magic-program-api/src/args.rs (1 hunks)

🔇 Additional comments (1)

magicblock-magic-program-api/src/args.rs (1)

16-21: Verification complete: method rename is correctly implemented.

The change from default to new is correct and has been properly applied. Ripgrep search confirms no remaining calls to ActionArgs::default exist in the codebase. Current usages in test-integration/programs/flexi-counter/src/processor/schedule_intent.rs (lines 73, 112) use struct literal syntax (ActionArgs { ... }), which is valid and doesn't require the new method. No impl Default for ActionArgs exists, which is appropriate since the constructor requires a parameter.

---

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.