fix: align StateMinerProvingDeadline with Lotus

CHANGELOG.md

@@ -70,6 +70,9 @@
 - [#4610](https://github.com/ChainSafe/forest/issues/4610) Fixed incorrect
   structure in the `Filecoin.MinerGetBaseInfo` RPC method.
 
+- [#4635](https://github.com/ChainSafe/forest/pull/4635) Fixed bug in
+  `StateMinerProvingDeadline`.
+
 ## Forest 0.19.2 "Eagle"
 
 Non-mandatory release that includes a fix for the Prometheus-incompatible

scripts/tests/api_compare/filter-list

@@ -8,5 +8,3 @@
 !Filecoin.StateCall
 # CustomCheckFailed in Forest: https://github.com/ChainSafe/forest/issues/4446
 !Filecoin.StateCirculatingSupply
-# CustomCheckFailed in Forest: https://github.com/ChainSafe/forest/issues/4630
-!Filecoin.StateMinerProvingDeadline

scripts/tests/api_compare/filter-list-offline

@@ -23,5 +23,3 @@
 # Offline server won't provide correct results for finality-related methods
 !Filecoin.EthGetBlockByNumber
 !eth_getBlockByNumber
-# CustomCheckFailed in Forest: https://github.com/ChainSafe/forest/issues/4630
-!Filecoin.StateMinerProvingDeadline

src/rpc/methods/state.rs

@@ -719,7 +719,11 @@ impl RpcMethod<2> for StateMinerProvingDeadline {
         let state: miner::State = ctx
             .state_manager
             .get_actor_state_from_address(&ts, &address)?;
-        Ok(ApiDeadlineInfo(state.deadline_info(policy, ts.epoch())))
+        Ok(ApiDeadlineInfo(
+            state
+                .recorded_deadline_info(policy, ts.epoch())
+                .next_not_elapsed(),
+        ))
     }
 }
 

src/shim/actors/miner.rs

@@ -5,11 +5,15 @@ mod partition;
 mod state;
 
 use cid::Cid;
-use fil_actor_interface::miner::State;
+use fil_actor_interface::{
+    miner::{DeadlineInfo, State},
+    Policy,
+};
 use fil_actors_shared::fvm_ipld_bitfield::BitField;
 use fvm_ipld_blockstore::Blockstore;
 
 use crate::rpc::types::{SectorOnChainInfo, SectorPreCommitOnChainInfo};
+use crate::shim::clock::ChainEpoch;
 use crate::utils::db::CborStoreExt as _;
 
 pub trait MinerStateExt {
@@ -31,6 +35,8 @@ pub trait MinerStateExt {
         store: &BS,
         sector_number: u64,
     ) -> anyhow::Result<Option<SectorPreCommitOnChainInfo>>;
+
+    fn recorded_deadline_info(&self, policy: &Policy, current_epoch: ChainEpoch) -> DeadlineInfo;
 }
 
 pub trait PartitionExt {

src/shim/actors/miner/state.rs

@@ -2,6 +2,9 @@
 // SPDX-License-Identifier: Apache-2.0, MIT
 
 use anyhow::Context as _;
+use fil_actor_interface::{convert::*, Policy};
+
+use crate::shim::clock::ChainEpoch;
 
 use super::*;
 
@@ -180,4 +183,30 @@ impl MinerStateExt for State {
                 .map(SectorPreCommitOnChainInfo::from),
         })
     }
+
+    /// Returns deadline calculations for the state recorded proving period and deadline.
+    /// This is out of date if the a miner does not have an active miner cron
+    fn recorded_deadline_info(&self, policy: &Policy, current_epoch: ChainEpoch) -> DeadlineInfo {
+        match self {
+            State::V8(st) => st
+                .recorded_deadline_info(&from_policy_v13_to_v9(policy), current_epoch)
+                .into(),
+            State::V9(st) => st
+                .recorded_deadline_info(&from_policy_v13_to_v9(policy), current_epoch)
+                .into(),
+            State::V10(st) => st
+                .recorded_deadline_info(&from_policy_v13_to_v10(policy), current_epoch)
+                .into(),
+            State::V11(st) => st
+                .recorded_deadline_info(&from_policy_v13_to_v11(policy), current_epoch)
+                .into(),
+            State::V12(st) => st
+                .recorded_deadline_info(&from_policy_v13_to_v12(policy), current_epoch)
+                .into(),
+            State::V13(st) => st.recorded_deadline_info(policy, current_epoch).into(),
+            State::V14(st) => st
+                .recorded_deadline_info(&from_policy_v13_to_v14(policy), current_epoch)
+                .into(),
+        }
+    }
 }