From 60ca21e359227c3e4427d52f5f2cecff980feac1 Mon Sep 17 00:00:00 2001 From: DooseWayo <“doosewayo94@gmail.com”> Date: Sun, 27 Sep 2026 16:25:27 +0100 Subject: [PATCH] fix(common): use canonical BPS_DENOMINATOR in split.rs split.rs declared its own private `const BPS_DENOMINATOR: i128 = 10_000;` instead of importing the one centralized in mergefi-common's lib.rs (#163). compute_split now uses `crate::BPS_DENOMINATOR`, so the fee/payout math can't silently drift from the shared constant. Co-Authored-By: Claude Opus 5.5 --- contracts/common/src/split.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/contracts/common/src/split.rs b/contracts/common/src/split.rs index 7d4fcf6..e322b97 100644 --- a/contracts/common/src/split.rs +++ b/contracts/common/src/split.rs @@ -17,7 +17,7 @@ use soroban_sdk::{Address, Env, Vec}; -const BPS_DENOMINATOR: i128 = 10_000; +use crate::BPS_DENOMINATOR; /// The computed protocol fee and each recipient's absolute payout amount. pub struct Payouts {