Skip to content

feat: add protocol fee configuration#151

Open
TUPM96 wants to merge 1 commit into
sorosave-protocol:mainfrom
TUPM96:codex/protocol-fee-18
Open

feat: add protocol fee configuration#151
TUPM96 wants to merge 1 commit into
sorosave-protocol:mainfrom
TUPM96:codex/protocol-fee-18

Conversation

@TUPM96
Copy link
Copy Markdown

@TUPM96 TUPM96 commented May 25, 2026

/claim #18

Summary

  • add protocol fee configuration backed by instance storage: protocol_fee_bps and protocol_treasury
  • add protocol-admin setters/getters, including a 10,000 bps upper bound
  • deduct the fee during distribute_payout, send it to the treasury, and pay the remainder to the round recipient
  • add coverage for defaults, authorization, invalid fee rates, and payout fee accounting

Validation

  • cargo fmt --check
  • cargo test --package sorosave
  • cargo clippy --package sorosave --all-targets -- -D warnings
  • git diff --check

Payout details can be provided privately on acceptance.

Copilot AI review requested due to automatic review settings May 25, 2026 10:30
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants