fixes: anvil, forge(fmt, backtraces) backports to v1.4.3 (#12217)

* fix(fmt): inline call options when they fit (#12172)

* fix(fmt): inline call options when they fit

* fix: ensure closure only when extra box is opened

* fix(anvil): get account info from db when node block > fork block (#12175)

fix(anvil): return account info from db when requested block > fork block

* fix(fmt): do not add underscores on fractional part with exponent (#12195)

* fix(fmt): do not add underscores on fractional part with exponent

* Update crates/fmt/src/state/common.rs

Co-authored-by: DaniPopes <[email protected]>

---------

Co-authored-by: grandizzy <[email protected]>
Co-authored-by: DaniPopes <[email protected]>

* fix(forge): show backtrace line only on -vvvvv (#12211)

* fix(forge): show backtrace line only on -vvvvv

* fix tests

* Update docs

* Fix

---------

Co-authored-by: 0xrusowsky <[email protected]>

* chore: bump v1.4.3 (#12213)

* chore: fix tests

* chore: fix clippy (#12167)

* fix(coverage): do not account abstract contract items (#12202)

* fix(coverage): exclude only virtual fns without impl (#12216)

---------

Co-authored-by: 0xrusowsky <[email protected]>
Co-authored-by: DaniPopes <[email protected]>
Co-authored-by: Matthias Seitz <[email protected]>

Author: 4 people

Cargo.lock

@@ -29,7 +29,7 @@ members = [
 resolver = "2"
 
 [workspace.package]
-version = "1.4.2"
+version = "1.4.3"
 edition = "2024"
 # Remember to update clippy.toml as well
 rust-version = "1.89"

Cargo.toml

@@ -772,18 +772,30 @@ impl EthApi {
         node_info!("eth_getAccountInfo");
 
         if let Some(fork) = self.get_fork() {
+            let block_request = self.block_request(block_number).await?;
             // check if the number predates the fork, if in fork mode
-            if let BlockRequest::Number(number) = self.block_request(block_number).await?
-                && fork.predates_fork_inclusive(number)
-            {
-                // if this predates the fork we need to fetch balance, nonce, code individually
-                // because the provider might not support this endpoint
-                let balance = fork.get_balance(address, number).map_err(BlockchainError::from);
-                let code = fork.get_code(address, number).map_err(BlockchainError::from);
-                let nonce = self.get_transaction_count(address, Some(number.into()));
-                let (balance, code, nonce) = try_join!(balance, code, nonce)?;
-
-                return Ok(alloy_rpc_types::eth::AccountInfo { balance, nonce, code });
+            if let BlockRequest::Number(number) = block_request {
+                trace!(target: "node", "get_account_info: fork block {}, requested block {number}", fork.block_number());
+                return if fork.predates_fork(number) {
+                    // if this predates the fork we need to fetch balance, nonce, code individually
+                    // because the provider might not support this endpoint
+                    let balance = fork.get_balance(address, number).map_err(BlockchainError::from);
+                    let code = fork.get_code(address, number).map_err(BlockchainError::from);
+                    let nonce = self.get_transaction_count(address, Some(number.into()));
+                    let (balance, code, nonce) = try_join!(balance, code, nonce)?;
+
+                    Ok(alloy_rpc_types::eth::AccountInfo { balance, nonce, code })
+                } else {
+                    // Anvil node is at the same block or higher than the fork block,
+                    // return account info from backend to reflect current state.
+                    let account_info = self.backend.get_account(address).await?;
+                    let code = self.backend.get_code(address, Some(block_request)).await?;
+                    Ok(alloy_rpc_types::eth::AccountInfo {
+                        balance: account_info.balance,
+                        nonce: account_info.nonce,
+                        code,
+                    })
+                };
             }
         }
 

crates/anvil/src/eth/api.rs

@@ -1621,7 +1621,7 @@ async fn test_fork_get_account() {
 
 #[tokio::test(flavor = "multi_thread")]
 async fn test_fork_get_account_info() {
-    let (_api, handle) = spawn(fork_config()).await;
+    let (api, handle) = spawn(fork_config()).await;
     let provider = handle.http_provider();
 
     let info = provider
@@ -1654,6 +1654,23 @@ async fn test_fork_get_account_info() {
             code: Default::default(),
         }
     );
+
+    // Mine and check account info at new block number, see https://github.com/foundry-rs/foundry/issues/12148
+    api.evm_mine(None).await.unwrap();
+    let info = provider
+        .get_account_info(address!("0x19e53a7397bE5AA7908fE9eA991B03710bdC74Fd"))
+        // predates fork
+        .number(BLOCK_NUMBER + 1)
+        .await
+        .unwrap();
+    assert_eq!(
+        info,
+        AccountInfo {
+            balance: U256::from(14352720829244098514u64),
+            nonce: 6690,
+            code: Default::default(),
+        }
+    );
 }
 
 fn assert_hardfork_config(

crates/anvil/tests/it/fork.rs

@@ -11,7 +11,7 @@ use std::fmt;
 #[derive(Clone, Debug, PartialEq, Eq)]
 pub enum Item {
     Data(Vec<u8>),
-    Array(Vec<Item>),
+    Array(Vec<Self>),
 }
 
 impl Encodable for Item {