[TheHalalHunter]

Summary

Resolves four related issues in a single PR to keep the contract API changes atomic and reviewable together.

---

#550 — Unify pause function naming

• Renamed \emergency_pause/\emergency_unpause\ → \pause/\unpause\ in \credential-nft\ and \progress-tracker\ to match \learn-token\
• Updated all tests, integration tests, scripts, examples, and README API tables

#551 — Add RBAC to credential-nft and progress-tracker

• Ported \AdminRole\ enum (Admin, Minter, Pauser) from \learn-token\ to both contracts
• Added \grant_role,
  evoke_role, \has_role\ public functions
• Gated \mint_credential\ on Minter/Admin, \pause/\unpause\ on Pauser/Admin, \upgrade\ on Admin
• Admin role granted to initial admin on \initialize\
• Added comprehensive RBAC tests to both contracts

#552 — Typed ContractError returns in learn-token

• All 15 \ContractError\ variants are now used (previously 11 were dead code)
• Newly wired variants: \ZeroAddress, \RewardCapped\
• Converted all public state-mutating functions from \panic!\ to \Result<_, ContractError>\
• Removed
  equire_not_paused\ (replaced by \ensure_not_paused\ returning \Result)
• Updated all string-message \should_panic\ tests to typed error assertions

#553 — Typed ContractError returns in credential-nft

• Added 5 new variants: \ContractPaused, \NotFound, \DuplicateCredential, \InvalidScore, \InvalidMetadataUri\
• Converted all public functions in \lib.rs, \mint.rs, and \�erify.rs\ to return \Result\
• Updated all string-message \should_panic\ tests across \credential_tests.rs, \error_message_tests.rs, and \security_double_spending_tests.rs\

Testing

All string-message #[should_panic]\ tests have been converted to typed error assertions. Cargo is not installed in the development environment — please run \cargo test --workspace\ as part of CI review.

Commits

• \�c2c330\ refactor: rename emergency_pause/emergency_unpause to pause/unpause (Unify pause function naming across contracts #550)
• \8953604\ feat: add RBAC (grant_role/revoke_role/has_role) to credential-nft and progress-tracker (Add role-based access control to credential-nft and progress-tracker #551)
• \d4d9dc9\ refactor: convert learn-token panics to typed ContractError returns (Replace panic-based error handling with typed ContractError returns in learn-token #552)
• \4338f02\ refactor: convert credential-nft panics to typed ContractError returns (Replace panic-based error handling with typed ContractError returns in credential-nft #553)

Closes #550, Closes #551, Closes #552, Closes #553

[drips-wave]

@TheHalalHunter Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits