Skip to content

[CLD-1916]: feat(evm): port link token state code#13

Closed
graham-chainlink wants to merge 1 commit intomainfrom
ggoh/cld-1916/link-token-state
Closed

[CLD-1916]: feat(evm): port link token state code#13
graham-chainlink wants to merge 1 commit intomainfrom
ggoh/cld-1916/link-token-state

Conversation

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

Ports EVM LinkToken state loading/view generation logic into this repo to support reading LINK/Static LINK token deployments from an address bundle (TypeAndVersion map), aligning with the CLD changesets/view layer.

Changes:

  • Adds LinkTokenState and StaticLinkTokenState with Generate*View helpers.
  • Adds MaybeLoadLinkTokenChainState and MaybeLoadStaticLinkTokenState to bind on-chain wrappers from address/type+version inputs.
  • Introduces unit tests covering nil bindings, duplicate contract instances, and version mismatch behavior.

Reviewed changes

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

File Description
pkg/family/evm/link_state.go Implements LINK & Static LINK state structs, view generation, and chain-state loaders using TypeAndVersion address maps.
pkg/family/evm/link_state_test.go Adds focused unit tests for nil bindings, dedupe error behavior, and version mismatch handling.

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

Comment thread pkg/family/evm/link_state.go Outdated
Comment thread pkg/family/evm/link_state.go Outdated
Comment thread pkg/family/evm/link_state.go Outdated
Comment thread pkg/family/evm/link_state.go Outdated
Comment thread pkg/family/evm/link_state.go Outdated
Comment thread pkg/family/evm/link_state_test.go
@graham-chainlink graham-chainlink force-pushed the ggoh/cld-1916/link-token-state branch from ca15072 to 7987d67 Compare April 23, 2026 06:38
@graham-chainlink graham-chainlink marked this pull request as ready for review April 23, 2026 06:40
@graham-chainlink graham-chainlink requested a review from a team as a code owner April 23, 2026 06:40
Copilot AI review requested due to automatic review settings April 23, 2026 06:40
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

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


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

Comment thread pkg/family/evm/link_state_test.go
Comment thread pkg/family/evm/link_state_test.go
Comment thread pkg/family/evm/link_state_test.go
Comment thread pkg/family/evm/link_state_test.go
@graham-chainlink graham-chainlink force-pushed the ggoh/cld-1916/link-token-state branch from 7987d67 to 5bb816b Compare April 23, 2026 06:48
@graham-chainlink graham-chainlink enabled auto-merge (squash) April 23, 2026 09:33
Copy link
Copy Markdown
Contributor

@ecPablo ecPablo left a comment

Choose a reason for hiding this comment

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

@graham-chainlink graham-chainlink enabled auto-merge (squash) April 24, 2026 02:27
@graham-chainlink graham-chainlink deleted the ggoh/cld-1916/link-token-state branch April 24, 2026 02:31
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.

3 participants