feat(commitment_marketplace): cannot-buy-own-listing-additional-edge-cases-and-fuzz-inputs

[Tijesunimi004]

Closes #273. Added checks and SafeMath integrations to mitigate integer overflows and prevent purchasing or offering on self-owned listings.

[drips-wave]

@Tijesunimi004 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

[Copilot]

Pull request overview

This PR updates the commitment_marketplace Soroban contract to tighten “cannot buy/offer on own listing” edge cases and to use shared safe-math helpers in fee calculations.

Changes:

• Add self-owned listing/auction checks to make_offer and a self-offer guard to accept_offer.
• Replace fee arithmetic in buy_nft, accept_offer, and end_auction with shared_utils::math::SafeMath.
• Add tests covering the new self-owned offer/accept edge cases.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 5 comments.

File	Description
contracts/commitment_marketplace/src/lib.rs	Adds self-owned offer checks; switches fee computation to SafeMath; adds overflow-safe ends_at calculation.
contracts/commitment_marketplace/src/tests.rs	Adds new panic-assertion tests for self-owned listing/auction offer cases.
contracts/commitment_marketplace/Cargo.toml	Adds a path dependency on shared_utils.
Cargo.lock	Updates lockfile to include shared_utils dependency for the marketplace crate.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.