refactor(contracts): remove redundant if case; move struct to interface

Author: 0xjei

packages/contracts/contracts/src/AdvancedChecker.sol

@@ -1,18 +1,7 @@
 // SPDX-License-Identifier: MIT
 pragma solidity ^0.8.20;
 
-import {IAdvancedChecker, Check} from "./interfaces/IAdvancedChecker.sol";
-
-/// @notice Tracks validation status for pre, main, and post checks.
-/// @dev Used to maintain check state in AdvancedPolicy.
-struct CheckStatus {
-    /// @dev Pre-check completion status.
-    bool pre;
-    /// @dev Number of completed main checks.
-    uint8 main;
-    /// @dev Post-check completion status.
-    bool post;
-}
+import {IAdvancedChecker, Check, CheckStatus} from "./interfaces/IAdvancedChecker.sol";
 
 /// @title AdvancedChecker.
 /// @notice Multi-phase validation checker with pre, main, and post checks.

packages/contracts/contracts/src/AdvancedPolicy.sol

@@ -77,9 +77,6 @@ abstract contract AdvancedPolicy is IAdvancedPolicy, Policy {
         } else if (checkType == Check.POST) {
             // Handle POST check.
             if (SKIP_POST) revert CannotPostCheckWhenSkipped();
-            if (!status.pre) {
-                revert PreCheckNotEnforced();
-            }
             if (status.main == 0) {
                 revert MainCheckNotEnforced();
             }

packages/contracts/contracts/src/interfaces/IAdvancedChecker.sol

@@ -12,6 +12,17 @@ enum Check {
     POST
 }
 
+/// @notice Tracks validation status for pre, main, and post checks.
+/// @dev Used to maintain check state in AdvancedPolicy.
+struct CheckStatus {
+    /// @dev Pre-check completion status.
+    bool pre;
+    /// @dev Number of completed main checks.
+    uint8 main;
+    /// @dev Post-check completion status.
+    bool post;
+}
+
 /// @title IAdvancedChecker.
 /// @notice Defines multi-phase validation system interface.
 /// @dev Implement this for custom validation logic with pre/main/post checks.