feature(writer): add support for external decoded data sources

zerosnacks commented 4 months ago

Add nodes_mut for mutability of nodes, required for populating added CallTrace fields
Adds decoded.label, decoded.return_data, decoded.call_data, wrapped in the DecodedCallTrace container type.
Wraps the raw LogData in a CallLog to accompany it with additional decoded data fields: decoded.name, decoded.params, wrapped in the DecodedCallLog container type.

decoded.contract_name has not been added as a field as it is only used in Foundry's gas report, not in the current trace render.

It is likely that some stylistic updates will be followed up relatively quickly

zerosnacks commented 4 months ago

Still, open for feedback

zerosnacks commented 4 months ago

Formatting has been fixed, Foundry tests are now passing: https://github.com/foundry-rs/foundry/pull/8224

DaniPopes commented 4 months ago

Can you add tests here too? Using expect!, with also decoded fields

zerosnacks commented 4 months ago

Can you add tests here too? Using expect!, with also decoded fields

Added, please note that it shows the application of patches, not necessarily the full decoding as implemented in Foundry

paradigmxyz / revm-inspectors