Skip to content

added gnosis script for role grant#152

Open
shinchann221 wants to merge 1 commit into
devfrom
paras/script-add
Open

added gnosis script for role grant#152
shinchann221 wants to merge 1 commit into
devfrom
paras/script-add

Conversation

@shinchann221

@shinchann221 shinchann221 commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Note

Medium Risk
Grants an on-chain role to a specific relayer via multisig tooling; no contract logic changes, but incorrect relayer or role wiring would affect who can emit ramp volume events.

Overview
Adds RampVolumeEmitter contract addresses to Optimism (chain 10) dev and mainnet deployment manifests, and introduces a Forge Gnosis helper to authorize the backend relayer when inline grantRole was skipped at deploy.

GrantRampVolumeEmitterRole.s.sol builds a cash-controller Safe batch that calls RoleRegistry.grantRole with RAMP_VOLUME_EMITTER_ROLE (read from the deployed emitter) for the fixed relayer address, using addresses from the deployment file. It writes output/GrantRampVolumeEmitterRole-{chainId}.json and follows the same pattern as other scripts/gnosis-txs/* utilities. The PR includes a generated bundle for OP mainnet (chain 10).

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

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@shinchann221
shinchann221 changed the base branch from master to dev June 19, 2026 17:56

@certora-run certora-run Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Verification Results

  • Group ID: d6a70ea2-9877-4da2-902d-e8ad44b26b95
  • Commit: e5b01c6
Job Result VERIFIED Link
EtherFiSafe.conf 14 Link
CashModuleCore.conf --rule "p03" 2 Link

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.

1 participant