[CLD-1777]: feat(operations-gen): add evm support#910
[CLD-1777]: feat(operations-gen): add evm support#910graham-chainlink wants to merge 1 commit intomainfrom
Conversation
|
There was a problem hiding this comment.
file mainly copied from https://github.com/smartcontractkit/chainlink-ccip/blob/main/chains/evm/cmd/operations-gen/main.go and adapted to the new design and added tests.
I will update the ported code in chainlink-ccip to point to this one when this PR is merged.
graham-chainlink
changed the title
There was a problem hiding this comment.
Pull request overview
Adds EVM support to the tools/operations-gen CLI so it can generate type-safe Go operation wrappers from Solidity ABIs/bytecode, using an EVM-specific handler and template.
Changes:
- Registers the
evmchain-family handler and introduces EVM extraction/generation logic. - Adds an EVM operations template plus supporting utilities (type mapping, ABI parsing, overload handling).
- Adds documentation and unit tests for key EVM helpers.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| tools/operations-gen/README.md | Documents the CLI, config schema, and input/output layouts for EVM generation. |
| tools/operations-gen/main.go | Enables EVM by registering evmHandler in chainFamilies. |
| tools/operations-gen/evm.go | Implements EVM handler: config decoding, ABI/bytecode reading, ABI parsing, template data preparation, and codegen helpers. |
| tools/operations-gen/evm_test.go | Adds unit tests for EVM naming/type mapping utilities and overload handling. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Add evm support for the operations-gen cli, basically ported the code from https://github.com/smartcontractkit/chainlink-ccip/blob/main/chains/evm/cmd/operations-gen/main.go and adapted it to the new design JIRA: https://smartcontract-it.atlassian.net/browse/CLD-1777
graham-chainlink
force-pushed
the
ggoh/cld-1777/evm-support-operations-gen
branch
from
3df479d to
0d43e7b
Compare
|
| var nameOverrides = map[string]string{ | ||
| "OnRamp": "onramp", | ||
| "OffRamp": "offramp", | ||
| } |
There was a problem hiding this comment.
I don't think we actually want to support this product specific overrides. Can ythey do it in configuration instead?
jkongie
left a comment
jkongie
left a comment
There was a problem hiding this comment.
We should add more documentation to this
| ```bash | ||
| task generate:operations CONFIG=/path/to/operations_gen_config.yaml | ||
| ``` |
There was a problem hiding this comment.
Is this applicable? task seems like a repo specific wrapper
| input: | ||
| base_path: "." # Directory containing abi/ and bytecode/ subdirectories | ||
|
|
||
| output: | ||
| base_path: "." # Directory where generated operations/ folders are written |
There was a problem hiding this comment.
Can we always assume that the contracts will be in a single base path?
Add evm support for the operations-gen cli, basically ported the code from https://github.com/smartcontractkit/chainlink-ccip/blob/main/chains/evm/cmd/operations-gen/main.go and adapted it to the new design
JIRA: https://smartcontract-it.atlassian.net/browse/CLD-1777