Skip to content

Create new instance for fee calculation (backport #3051)#3053

Merged
jusbar23 merged 1 commit intorelease/protocol/v9.xfrom
mergify/bp/release/protocol/v9.x/pr-3051
Aug 22, 2025
Merged

Create new instance for fee calculation (backport #3051)#3053
jusbar23 merged 1 commit intorelease/protocol/v9.xfrom
mergify/bp/release/protocol/v9.x/pr-3051

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Aug 22, 2025

Changelist

  • Fix issue with fee calculation for ORRS

Test Plan

  • Test cases

Author/Reviewer Checklist

  • If this PR has changes that result in a different app state given the same prior state and transaction list, manually add the state-breaking label.
  • If the PR has breaking postgres changes to the indexer add the indexer-postgres-breaking label.
  • If this PR isn't state-breaking but has changes that modify behavior in PrepareProposal or ProcessProposal, manually add the label proposal-breaking.
  • If this PR is one of many that implement a specific feature, manually label them all feature:[feature-name].
  • If you wish to for mergify-bot to automatically create a PR to backport your change to a release branch, manually add the label backport/[branch-name].
  • Manually add any of the following labels: refactor, chore, bug.

Summary by CodeRabbit

  • Bug Fixes

    • Prevented unintended mutation of net fees during revenue-share calculations by using mutation‑safe copies and adjusting subtraction logic; added a post‑aggregation validation that logs and aborts if total shared fees exceed net fees.
  • Tests

    • Added comprehensive tests covering taker/maker order‑router rev shares, unconditional rev shares, maker rebates, and affiliate‑whitelist interactions to validate aggregation, allocation, and edge cases.

This is an automatic backport of pull request #3051 done by [Mergify](https://mergify.com).

@mergify mergify bot requested a review from a team as a code owner August 22, 2025 14:40
@mergify mergify bot mentioned this pull request Aug 22, 2025
6 tasks
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Aug 22, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

Support

Need help? Join our Discord community for assistance with any issues or questions.

CodeRabbit Commands (Invoked using PR/Issue comments)

Type @coderabbitai help to get the list of available commands.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Status, Documentation and Community

  • Visit our Status Page to check the current availability of CodeRabbit.
  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

@jusbar23 jusbar23 merged commit d1effa6 into release/protocol/v9.x Aug 22, 2025
24 checks passed
@jusbar23 jusbar23 deleted the mergify/bp/release/protocol/v9.x/pr-3051 branch August 22, 2025 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

1 participant

Comments