Juglipaff / fair-reward-distributor / 28776381712

100%

push

github

feat!: remove pre-hooks; internal API now takes uint128 directly

BREAKING CHANGE: `_preStake` / `_preWithdraw` / `_preDistribute` are
removed. `_stake`, `_withdraw`, `_distribute` now accept `uint128`
liquidity directly, so consumers must narrow (e.g. via
`SafeCast.toUint128`) at the external boundary. `InsufficientStake` is
renamed to `InsufficientLiquidity` and now carries the raw `uint256`.

Rationale: the pre-hooks did no work that a consumer's external entry
function can't do more directly. Removing them enforces the narrowing
invariant in the type system rather than in an overridable hook,
eliminates one virtual dispatch per user action, and cuts the mandatory
integration boilerplate.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

12 of 12 branches covered (100.0%)

Branch coverage included in aggregate %.

25 of 25 new or added lines in 2 files covered. (100.0%)

90 of 90 relevant lines covered (100.0%)

33.09 hits per line