Skip to content

Add authorized caller to ops gen and RMN 2.1.0#2061

Open
carte7000 wants to merge 3 commits into
mainfrom
add-authorized-callers-to-ops-gen
Open

Add authorized caller to ops gen and RMN 2.1.0#2061
carte7000 wants to merge 3 commits into
mainfrom
add-authorized-callers-to-ops-gen

Conversation

@carte7000
Copy link
Copy Markdown
Contributor

No description provided.

@carte7000 carte7000 marked this pull request as ready for review May 12, 2026 12:29
@carte7000 carte7000 requested review from a team as code owners May 12, 2026 12:29
Copilot AI review requested due to automatic review settings May 12, 2026 12:29
@carte7000 carte7000 requested review from a team as code owners May 12, 2026 12:29
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds support for “authorized callers” access control in the EVM operations generator, and applies it to the RMN curse operation for version 2.1.0 so direct execution eligibility matches the on-chain AuthorizedCallers gating.

Changes:

  • Introduce a new authorized_callers access mode in operations-gen, wiring it to contract.IsAuthorizedCaller.
  • Add a generic IsAuthorizedCaller helper that checks getAllAuthorizedCallers on the target contract.
  • Update RMN 2.1.0 generated ops + generator config so curse uses the new authorized-caller check instead of allowing all callers.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
chains/evm/deployment/v2_1_0/operations/rmn/rmn.go Switch RMN curse/curse0 writes to use contract.IsAuthorizedCaller gating.
chains/evm/deployment/utils/operations/contract/callers.go Add AuthorizedCallersContract + IsAuthorizedCaller implementation.
chains/evm/deployment/operations_gen_config.yaml Configure RMN curse as authorized_callers.
chains/evm/cmd/operations-gen/README.md Document the new authorized_callers access mode.
chains/evm/cmd/operations-gen/main.go Teach generator to parse authorized_callers and emit IsAuthorizedCaller in write ops.
Files not reviewed (1)
  • chains/evm/deployment/v2_1_0/operations/rmn/rmn.go: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread chains/evm/cmd/operations-gen/main.go
Comment thread chains/evm/cmd/operations-gen/README.md
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown

Metric add-authorized-callers-to-ops-gen main
Coverage 70.1% 69.8%

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