Skip to content

feat: polygon & base contract deployment addresses#70

Merged
nvtaveras merged 3 commits into
mainfrom
feat/polygonBaseContractDeployment
May 7, 2026
Merged

feat: polygon & base contract deployment addresses#70
nvtaveras merged 3 commits into
mainfrom
feat/polygonBaseContractDeployment

Conversation

@nvtaveras
Copy link
Copy Markdown
Contributor

@nvtaveras nvtaveras commented May 7, 2026

Summary

  • Add deployment addresses (USDC/EURC, MigrationMultisig, ProtocolFeeRecipient, FeeSetter, ReserveSafe) to Polygon (137) and Base (8453) in the addressbook
  • Skip DeployMockAggregators early when no mock aggregators are configured for the target chain
  • Log deployed MockChainlinkAggregator addresses for visibility

Test plan

  • Verify addressbook entries match the canonical addresses for Polygon and Base
  • Run DeployMockAggregators against a chain without mock aggregator configs and confirm it skips cleanly
  • Run DeployMockAggregators against a chain with mock aggregator configs and confirm deployments + logs

Note

Medium Risk
Moderate risk because it updates chain-specific contract/token addresses used for deployments/configuration, where mistakes can misroute funds or permissions; script changes are small and mostly add guardrails/logging.

Overview
Adds Polygon (137) and Base (8453) entries to .treb/addressbook.json for deployed contracts/tokens (e.g., USDC/EURC, MigrationMultisig, ReserveSafe, FeeSetter, ProtocolFeeRecipient).

Updates DeployMockAggregators.s.sol to early-exit when no mock aggregator configs exist for the target chain, and to log each deployed MockChainlinkAggregator address for easier verification.

Reviewed by Cursor Bugbot for commit 72858b9. Bugbot is set up for automated code reviews on this repo. Configure here.

@nvtaveras nvtaveras merged commit 9912a30 into main May 7, 2026
5 checks passed
@nvtaveras nvtaveras deleted the feat/polygonBaseContractDeployment branch May 7, 2026 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants