fix: mint-cleanup

[KevinMB0220]

✅ Pull Request: Refactor and Fixes for reputation-system Module

📋 Description

This PR addresses and resolves multiple compilation errors and reference issues in the reputation-system module. It removes duplicate or invalid imports, aligns all types with the correct soroban_sdk primitives, and eliminates local conflicts with the NFTError enum by using the shared definition from nft-core. Additionally, it cleans up unused imports and fixes environment ownership issues.

🛠 Changes Made

• ✅ Removed the local NFTError enum and replaced it with the centralized version from nft_core.
• ✅ Fixed all missing imports from soroban_sdk (Env, Address, Vec, Map, String, Symbol, and macros #[contract], #[contractimpl], #[contracttype]).
• ✅ Resolved incorrect usage of Vec::from_array and String::from_str, ensuring env is cloned before being moved.
• ✅ Updated the NFTCore::mint_nft call to include the missing badge_metadata argument.
• ✅ Cleaned up unused use statements to reduce warnings and improve clarity.

📸 Evidence

🔧 Before:

• ❌ cargo build failed with over 40 type and macro resolution errors.
• ❌ NFTError was duplicated and conflicting across modules.
• ❌ env was moved early, causing borrow issues.

✅ After:

• ✔️ The recognition-system module compiles without errors.
• ✔️ Type mismatches, visibility problems, and ownership issues have been resolved.
• ✔️ The code is now clean, consistent, and easier to maintain.

🔗 Related Issue

This pull request closes #25 “Refactor recognition-system module to fix types and compilation errors.”

---

Grabacion.de.pantalla.2025-05-03.181341.mp4

Thank you for reviewing this PR. Feedback is appreciated to further improve the system.