Summary
The protocol fee bps value is read from persistent storage in multiple contract functions, each constructing the storage key inline. A shared read helper should centralise this to eliminate duplication and make fee config reads consistent.
Scope
- Add
read_protocol_fee_bps(env: &Env) -> u32 helper
- Helper should panic with a descriptive message if the value is absent (contract not initialized)
- Replace all inline protocol fee bps reads with the helper
- Add unit tests: initialized contract returns configured bps, uninitialized contract panics with correct message
Acceptance Criteria
ETA: 12 hours
Coordinate on Telegram
Summary
The protocol fee bps value is read from persistent storage in multiple contract functions, each constructing the storage key inline. A shared read helper should centralise this to eliminate duplication and make fee config reads consistent.
Scope
read_protocol_fee_bps(env: &Env) -> u32helperAcceptance Criteria
ETA: 12 hours
Coordinate on Telegram