Ahmad Kaouk 4a16de1061 fix: resolve forge build warnings (#398) ... ## Summary ### Configuration - Remove deprecated `deny_warnings` config key from foundry.toml - Add global `[lint]` config to suppress naming convention warnings for AVS/EL/ERC patterns (`mixed-case-function`, `mixed-case-variable`) ### DataHavenServiceManager Refactoring - Rename immutable variables to SCREAMING_SNAKE_CASE (`_allocationManager` → `_ALLOCATION_MANAGER`, `_rewardsCoordinator` → `_REWARDS_COORDINATOR`) - Wrap modifier logic in internal functions (`_checkRewardsInitiator`, `_checkValidator`, `_checkAllocationManager`) to reduce contract size - Add `_toAddress` helper with assembly for safe bytes-to-address conversion ### Safe Typecasting - Replace direct typecasts with OpenZeppelin's SafeCast library in deploy scripts and test utilities - Use `.toUint32()`, `.toUint64()`, `.toUint160()` for overflow-protected conversions - Replace `bytes32("wrong origin")` string cast with hex literal in test deployer ### Code Cleanup - Remove 25+ unused imports across script and test files - Convert plain imports to named imports for better clarity - Use `SafeERC20.safeTransfer()` for token transfers in tests - Change `view` to `pure` where appropriate ## Test plan - [x] `forge build` completes with no warnings - [x] `forge test` passes all 10 tests		2026-01-22 09:48:27 -03:00
..
PermissionControllerMock.sol	fix: Resolve Foundry build errors and apply code formatting (#241)	2025-10-20 11:20:59 +03:00
RewardsCoordinatorMock.sol	fix: resolve forge build warnings (#398)	2026-01-22 09:48:27 -03:00
SnowbridgeGatewayMock.sol	refactor(contracts): Harden DataHavenServiceManager with input validation and code cleanup (#395)	2026-01-20 10:32:32 +00:00