Skip to content

Commit

Permalink
feat: add types for flat call tracer
Browse files Browse the repository at this point in the history
  • Loading branch information
mattsse committed Sep 14, 2024
1 parent cbfe3d9 commit e9f3c73
Show file tree
Hide file tree
Showing 2 changed files with 52 additions and 3 deletions.
33 changes: 33 additions & 0 deletions crates/rpc-types-trace/src/geth/call.rs
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
//! Geth call tracer types.

use crate::parity::LocalizedTransactionTrace;
use alloy_primitives::{Address, Bytes, B256, U256};
use serde::{Deserialize, Serialize};

Expand Down Expand Up @@ -88,6 +89,38 @@ impl CallConfig {
}
}

/// The response object for `debug_traceTransaction` with `"tracer": "flatCallTracer"`.
///
/// That is equivalent to parity's [`LocalizedTransactionTrace`]
/// <https://github.com/ethereum/go-ethereum/blob/0dd173a727dd2d2409b8e401b22e85d20c25b71f/eth/tracers/native/call_flat.go#L62-L62>
pub type FlatCallFrame = LocalizedTransactionTrace;

/// The configuration for the flat call tracer.
#[derive(Clone, Copy, Debug, Default, PartialEq, Eq, Serialize, Deserialize)]
#[serde(rename_all = "camelCase")]
pub struct FlatCallConfig {
/// If true, call tracer converts errors to parity format
#[serde(default, skip_serializing_if = "Option::is_none")]
pub convert_parity_errors: Option<bool>,
/// If true, call tracer includes calls to precompiled contracts
#[serde(default, skip_serializing_if = "Option::is_none")]
pub include_precompiles: Option<bool>,
}

impl FlatCallConfig {
/// Converts errors to parity format.
pub const fn parity_errors(mut self) -> Self {
self.convert_parity_errors = Some(true);
self
}

/// Include calls to precompiled contracts.
pub const fn with_precompiles(mut self) -> Self {
self.include_precompiles = Some(true);
self
}
}

#[cfg(test)]
mod tests {
use super::*;
Expand Down
22 changes: 19 additions & 3 deletions crates/rpc-types-trace/src/geth/mod.rs
Original file line number Diff line number Diff line change
@@ -1,14 +1,16 @@
//! Geth tracing types.

use crate::geth::mux::{MuxConfig, MuxFrame};
use crate::geth::{
call::FlatCallFrame,
mux::{MuxConfig, MuxFrame},
};
use alloy_primitives::{Bytes, B256, U256};
use alloy_rpc_types_eth::{state::StateOverride, BlockOverrides};
use serde::{de::DeserializeOwned, ser::SerializeMap, Deserialize, Serialize, Serializer};
use std::{collections::BTreeMap, time::Duration};

// re-exports
pub use self::{
call::{CallConfig, CallFrame, CallLogFrame},
call::{CallConfig, CallFrame, CallLogFrame, FlatCallConfig},
four_byte::FourByteFrame,
noop::NoopFrame,
pre_state::{
Expand Down Expand Up @@ -118,6 +120,8 @@ pub enum GethTrace {
Default(DefaultFrame),
/// The response for call tracer
CallTracer(CallFrame),
/// The response for the flat call tracer
FlatCallTracer(FlatCallFrame),
/// The response for four byte tracer
FourByteTracer(FourByteFrame),
/// The response for pre-state byte tracer
Expand Down Expand Up @@ -246,6 +250,10 @@ pub enum GethDebugBuiltInTracerType {
/// with the top-level call at root and sub-calls as children of the higher levels.
#[serde(rename = "callTracer")]
CallTracer,
/// Tracks all call frames of a transaction and returns them in a flat format, i.e. as opposed
/// to the nested format of `callTracer`.
#[serde(rename = "flatCallTracer")]
FlatCallTracer,
/// The prestate tracer has two modes: prestate and diff. The prestate mode returns the
/// accounts necessary to execute a given transaction. diff mode returns the differences
/// between the transaction's pre and post-state (i.e. what changed because the transaction
Expand Down Expand Up @@ -311,6 +319,14 @@ impl GethDebugTracerConfig {
self.from_value()
}

/// Returns the [FlatCallConfig] if it is a call config.
pub fn into_flat_call_config(self) -> Result<FlatCallConfig, serde_json::Error> {
if self.0.is_null() {
return Ok(Default::default());
}
self.from_value()
}

/// Returns the raw json value
pub fn into_json(self) -> serde_json::Value {
self.0
Expand Down

0 comments on commit e9f3c73

Please sign in to comment.