Skip to content

chore: add impl-only deploy script for MoolahVaultManager (BSC mainnet)#189

Merged
razww merged 1 commit into
masterfrom
chore/deploy-moolah-vault-manager-impl
Jun 8, 2026
Merged

chore: add impl-only deploy script for MoolahVaultManager (BSC mainnet)#189
razww merged 1 commit into
masterfrom
chore/deploy-moolah-vault-manager-impl

Conversation

@ricklista

Copy link
Copy Markdown
Contributor

Summary

Adds an impl-only Foundry deploy script for MoolahVaultManager on BSC mainnet. Deploys a fresh MoolahVaultManager implementation (constructed with the production Moolah address) for later proxy wiring via a timelock upgrade.

Change type

  • Migration / deploy script

Contracts changed

File Type
script/deploy_moolahVaultManager_impl.sol (MoolahVaultManagerImplDeploy) new

Interface changes

None — script only; no protocol contract modified.

Storage layout

N/A — deploy script, not an upgradeable contract.

Access control

Unchanged. Script broadcasts from the auto-selected deployer key (_deployerKey() from DeployBase).

Risk assessment

Area Risk Note
Storage collision 🟢 None No storage; impl deploy only
Fund safety 🟢 None Deploys an unwired implementation; no funds touched
Access control 🟢 None No role/owner changes
External call safety 🟢 None Only new MoolahVaultManager(moolah)

Deployment

  • Chain: BSC mainnet
  • Script: script/deploy_moolahVaultManager_impl.sol
  • Moolah: 0x8F73b65B4caAf64FBA2aF91cC5D4a2A1318E5D8C (hardcoded)
  • Deployer key auto-selected by chain ID. Impl is inert until a separate timelock proxy upgrade points the proxy at it.

Test plan

  • Script compiles (forge build script/deploy_moolahVaultManager_impl.sol --via-ir)
  • Deploy on BSC mainnet and record impl address; wire via timelock upgrade runbook (out of scope for this PR)

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

@razww razww left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@razww razww merged commit 308cc16 into master Jun 8, 2026
4 of 6 checks passed
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