Skip to content

Conversation

@patricios-space
Copy link
Collaborator

@patricios-space patricios-space commented Dec 26, 2025

@patricios-space patricios-space marked this pull request as ready for review December 26, 2025 18:08
@patricios-space patricios-space requested a review from a team as a code owner December 26, 2025 18:08
Copy link
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

This PR refactors ownership-related bindings by extracting the Ownable2Step struct from the common package into a dedicated ownable2step package. The new package provides a more comprehensive implementation including storage types, message types, exit codes, and getter functions for ownership management in TON smart contracts.

Key Changes:

  • Created a new ownable2step package with complete bindings for two-step ownership functionality
  • Replaced all references from common.Ownable2Step to ownable2step.Storage across the codebase
  • Added support for ownership transfer messages, error codes, and getter methods

Reviewed changes

Copilot reviewed 24 out of 24 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pkg/ccip/bindings/ownable2step/types.go Defines ownership message types, exit codes, and error constants
pkg/ccip/bindings/ownable2step/storage.go Defines the Storage struct for two-step ownership state
pkg/ccip/bindings/ownable2step/reader.go Implements getter functions for owner and pending owner with role prefixing
pkg/ccip/bindings/ownable2step/exitcode_string.go Auto-generated stringer implementation for ExitCode enum
pkg/ccip/bindings/router/router.go Updates Storage and RMNRemote to use ownable2step.Storage, adds RMNOwnableMessage type
pkg/ccip/bindings/router/reader.go Adds getter functions for RMN owner and pending owner
pkg/ccip/bindings/common/common.go Removes the Ownable2Step struct definition
pkg/ccip/model/router.go Updates import and type references to use ownable2step.Storage
pkg/ccip/model/onramp.go Updates import and type references to use ownable2step.Storage
pkg/ccip/model/offramp.go Updates import and type references to use ownable2step.Storage
pkg/ccip/model/fee_quoter.go Updates import and type references to use ownable2step.Storage
pkg/ccip/bindings/receiver/receiver.go Updates import and type references to use ownable2step.Storage
pkg/ccip/bindings/onramp/onramp_test.go Updates test cases to use ownable2step.Storage
pkg/ccip/bindings/onramp/onramp.go Updates Storage struct to use ownable2step.Storage
pkg/ccip/bindings/offramp/offramp.go Updates Storage struct to use ownable2step.Storage
pkg/ccip/bindings/feequoter/fee_quoter.go Updates Storage struct to use ownable2step.Storage
pkg/bindings/mcms/mcms/types.go Updates Data struct to use ownable2step.Storage
pkg/bindings/mcms/mcms/storage.go Updates import and initializes Data with ownable2step.Storage
pkg/bindings/examples/counter/types.go Updates ContractData to use ownable2step.Storage
integration-tests/txm/txm_test.go Updates test setup to use ownable2step.Storage
integration-tests/tracetracking/integration_test.go Updates test data initialization to use ownable2step.Storage
integration-tests/smoke/logpoller/helper/event_emitter.go Updates contract data initialization to use ownable2step.Storage
deployment/mcms/sequence/deploy_mcms.go Updates MCMS deployment to use ownable2step.Storage
deployment/ccip/sequence/deploy_ccip.go Updates all CCIP contract deployments to use ownable2step.Storage

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

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