Skip to content

Add per-call max_fee_bps slippage guard to deduct #498

Description

@greatest0fallt1me

Description

This is a smart-contract issue for the GrantFox campaign. Allow callers to specify a max acceptable fee in basis points; revert if exceeded.

Requirements and Context

  • Param max_fee_bps
  • Revert with Slippage error
  • Default = u16::MAX
  • Test slippage path
  • Must be secure, tested, and documented
  • Should be efficient and easy to review

Suggested Execution

  1. Fork the repo and create a branch
    git checkout -b feature/max-fee-bps
  2. Implement changes
    • contracts/vault/src/lib.rs
    • contracts/vault/src/errors.rs
  3. Test and commit
    • Run the repo's standard test suite and lint
    • Cover edge cases; include output in the PR

Example commit message

feat: max_fee_bps slippage guard on deduct

Acceptance Criteria

  • Param accepted
  • Revert correct
  • Default safe
  • Tests cover

Guidelines

  • Minimum 95% test coverage with cargo test
  • require_auth on every state-changing entrypoint
  • Overflow-safe math; no unwrap() in production paths
  • Clear NatSpec-style /// rustdoc
  • Timeframe: 96 hours

Metadata

Metadata

Labels

GRANTFOX OSSGrantFox open-source campaign taskOFFICIAL CAMPAIGNOfficial GrantFox campaign issueStellar WaveStellar Wave Program issueenhancementNew feature or improvementsecuritySecurity hardeningsmart-contractSoroban smart-contract work

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions