Skip to content

refactor(common): consolidate EVM UDFs under unified udfs module#1957

Merged
LNSD merged 1 commit intomainfrom
lnsd/refactor-common-relocate-udfs
Mar 12, 2026
Merged

refactor(common): consolidate EVM UDFs under unified udfs module#1957
LNSD merged 1 commit intomainfrom
lnsd/refactor-common-relocate-udfs

Conversation

@LNSD
Copy link
Contributor

@LNSD LNSD commented Mar 11, 2026

Flatten the evm::udfs submodule hierarchy into the top-level udfs module to simplify the module structure and remove the unnecessary evm nesting layer.

  • Move all EVM UDF modules (evm_decode_*, evm_encode_*, evm_topic, evm_function_params, shift_units) from evm::udfs to udfs
  • Relocate EthCallUdfsCache from standalone ethcall_udfs_cache module into udfs::eth_call
  • Consolidate UDF registration logic into the udfs module root
  • Update import paths across all consuming crates, tests, and feature docs

Flatten the `evm::udfs` submodule hierarchy into the top-level `udfs` module to simplify the module structure and remove the unnecessary `evm` nesting layer.

- Move all EVM UDF modules (`evm_decode_*`, `evm_encode_*`, `evm_topic`, `evm_function_params`, `shift_units`) from `evm::udfs` to `udfs`
- Relocate `EthCallUdfsCache` from standalone `ethcall_udfs_cache` module into `udfs::eth_call`
- Consolidate UDF registration logic into the `udfs` module root
- Update import paths across all consuming crates, tests, and feature docs

Signed-off-by: Lorenzo Delgado <lorenzo@edgeandnode.com>
@LNSD LNSD requested review from Theodus and leoyvens March 11, 2026 23:36
@LNSD LNSD self-assigned this Mar 11, 2026
Copy link
Member

@Theodus Theodus left a comment

Choose a reason for hiding this comment

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

LGTM

@LNSD LNSD merged commit f332cd6 into main Mar 12, 2026
8 checks passed
@LNSD LNSD deleted the lnsd/refactor-common-relocate-udfs branch March 12, 2026 00:13
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