maintenance-pool::withdraw panics (not a typed error) if Treasury is missing — require_treasury().unwrap() #488

Activity

1. chonilius

   added

   bugSomething isn't workingSomething isn't working

   good first issueGood for newcomersGood for newcomers

   Stellar WaveIssues in the Stellar wave programIssues in the Stellar wave program

   on Oct 13, 2026
2. urielit1313-creator commented on Oct 14, 2026

   

   urielit1313-creator

   on Oct 14, 2026

   More actions

   Addressed in PR #628.

   Summary

   • Replaced require_treasury::<DataKey>(&env).unwrap() with require_treasury::<DataKey>(&env).ok_or(Error::NotInitialized)? in maintenance-pool::withdraw, eliminating the unhandled host panic.
   • Added unit test test_withdraw_returns_not_initialized_if_treasury_missing verifying Err(Ok(Error::NotInitialized)) is cleanly returned when Treasury is missing.
   • 100% test pass rate across mergefi-maintenance-pool (43 tests passed) and 0 Clippy warnings.

   ---

   Bounty Payout Address (Base USDC): 0xfE777c43e4B0A24fFA1527eD520d1E7f9ECED483
3. chonilius

   removed

   Stellar WaveIssues in the Stellar wave programIssues in the Stellar wave program

   on Oct 15, 2026