Skip to content

Bound sponsor caps and fix payout argument encoding - #744

Closed
toxxxxicc wants to merge 1 commit into
MergeFi:mainfrom
toxxxxicc:fix/448-449-453-454-resource-bounds
Closed

toxxxxicc wants to merge 1 commit into
MergeFi:mainfrom
toxxxxicc:fix/448-449-453-454-resource-bounds

Conversation

@toxxxxicc

@toxxxxicc toxxxxicc commented Oct 19, 2026 •

Copy link
Copy Markdown

Summary

Caps configured sponsor counts at 50 for escrow and milestones, replaces unchecked dust casts with checked conversions, and adds tuple-vector argument support to the invoke script.

Related issues

Closes #448
Closes #449
Closes #453
Closes #454

Changes

  • Add a shared sponsor ceiling and typed initialization errors, with exact-limit and over-limit tests for both contracts.
  • Check conversion of refund/split dust to u32 before iterating.
  • Encode recipients:<address>=<bps>,... as a vector of (Address, u32) vectors and add parser tests.
  • Document the limits and invocation syntax; update CLI error-name mappings.

Test plan

  • Existing tests pass (make test) (Not run per request.)
  • New tests added for changed behaviour.
  • Manually verified against a local node / testnet where applicable (Not run.)

Checklist

  • Branch is up to date with main (based on origin/main).
  • No debug / dead code left in.
  • cargo fmt and cargo clippy are clean (Not run per request.)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@toxxxxicc
toxxxxicc requested a review from chonilius as a code owner October 19, 2026 14:52
@vercel

vercel Bot commented Oct 19, 2026

Copy link
Copy Markdown

@Rickyy1017 is attempting to deploy a commit to the chonilius' projects Team on Vercel.

A member of the Team first needs to authorize it.

@rikkk-j

rikkk-j commented Oct 19, 2026

Copy link
Copy Markdown

Superseded by #746. The replacement is submitted from the fork associated with the active rikkk-j account.

@toxxxxicc

Copy link
Copy Markdown
Author

Superseded by #746, now submitted from rikkk-j/contracts.

@toxxxxicc toxxxxicc closed this Oct 19, 2026
@toxxxxicc
toxxxxicc deleted the fix/448-449-453-454-resource-bounds branch October 19, 2026 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants