fix: eth_getTransactionCount to return the actual count, not parent's

CHANGELOG.md

@@ -41,6 +41,8 @@
 
 - [#5055](https://github.com/ChainSafe/forest/issues/5055) Fixed an issue where Forest fails on duplicate drand entries. This would only happen on new devnets.
 
+- [#6103](https://github.com/ChainSafe/forest/pull/6103) Fixed `eth_getTransactionCount` to return the nonce of the requested tipset and not its parent.
+
 ## Forest v0.30.1 "Laurelin"
 
 Mandatory release for mainnet node operators. It sets the NV27 _Golden Week_ network upgrade to epoch `5_348_280` which corresponds to `Wed 24 Sep 23:00:00 UTC 2025`. It also includes a few improvements that help with snapshot generation and inspection.

src/rpc/methods/eth.rs

@@ -2252,7 +2252,10 @@ impl RpcMethod<2> for EthGetTransactionCount {
             block_param.clone(),
             ResolveNullTipset::TakeOlder,
         )?;
-        let state = StateTree::new_from_root(ctx.store_owned(), ts.parent_state())?;
+
+        let (state_cid, _) = ctx.state_manager.tipset_state(&ts).await?;
+
+        let state = StateTree::new_from_root(ctx.store_owned(), &state_cid)?;
         let actor = state.get_required_actor(&addr)?;
         if is_evm_actor(&actor.code) {
             let evm_state = evm::State::load(ctx.store(), actor.code, actor.state)?;

src/state_manager/mod.rs

@@ -393,7 +393,7 @@ impl<DB> StateManager<DB>
 where
     DB: Blockstore + Send + Sync + 'static,
 {
-    /// Returns the pair of (parent state root, message receipt root). This will
+    /// Returns the pair of (state root, message receipt root). This will
     /// either be cached or will be calculated and fill the cache. Tipset
     /// state for a given tipset is guaranteed not to be computed twice.
     pub async fn tipset_state(self: &Arc<Self>, tipset: &Arc<Tipset>) -> anyhow::Result<CidPair> {