Add PackagedVersion. Remove abstract function and comment

ltyu/sp1-lightclient-ism
Le Yu 3 weeks ago
parent df5731cce5
commit 79b4f8cad7
  1. 3
      solidity/contracts/isms/ccip-read/AbstractCcipReadIsm.sol
  2. 12
      solidity/contracts/isms/ccip-read/StorageProofIsm.sol
  3. 1
      solidity/contracts/libs/StateProofHelpers.sol

@ -7,6 +7,7 @@ import {ICcipReadIsm} from "../../interfaces/isms/ICcipReadIsm.sol";
import {IMailbox} from "../../interfaces/IMailbox.sol"; import {IMailbox} from "../../interfaces/IMailbox.sol";
import {Message} from "../../libs/Message.sol"; import {Message} from "../../libs/Message.sol";
import {AbstractMultisigIsm} from "../multisig/AbstractMultisigIsm.sol"; import {AbstractMultisigIsm} from "../multisig/AbstractMultisigIsm.sol";
import {PackageVersioned} from "contracts/PackageVersioned.sol";
/** /**
* @title AbstractCcipReadIsm * @title AbstractCcipReadIsm
@ -23,7 +24,7 @@ import {AbstractMultisigIsm} from "../multisig/AbstractMultisigIsm.sol";
* message. This functions selector should be provided as the `callbackFunction` payload * message. This functions selector should be provided as the `callbackFunction` payload
* for the OffchainLookup error * for the OffchainLookup error
*/ */
abstract contract AbstractCcipReadIsm is ICcipReadIsm { abstract contract AbstractCcipReadIsm is ICcipReadIsm, PackageVersioned {
// ============ Constants ============ // ============ Constants ============
// solhint-disable-next-line const-name-snakecase // solhint-disable-next-line const-name-snakecase

@ -145,18 +145,6 @@ abstract contract StorageProofIsm is AbstractCcipReadIsm, OwnableUpgradeable {
*/ */
function getHeadStateSlot() public view virtual returns (uint256); function getHeadStateSlot() public view virtual returns (uint256);
/**
* @notice Reverts with the data needed to query for header proofs
* @dev See https://eips.ethereum.org/EIPS/eip-3668 for more information
* @param _message encoded Message that will be included in offchain query
*
* @dev In the future, check if fees have been paid before request a proof from.
* For now this feature is not complete according to the team.
*/
function getOffchainVerifyInfo(
bytes calldata _message
) external view virtual;
/** /**
* @notice Calculates storage key of the source chain DispatchedHook.dispatched mapping * @notice Calculates storage key of the source chain DispatchedHook.dispatched mapping
* @param _messageNonce message nonce * @param _messageNonce message nonce

@ -5,7 +5,6 @@ import {RLPReader} from "@eth-optimism/contracts-bedrock/src/libraries/rlp/RLPRe
import {RLPWriter} from "@eth-optimism/contracts-bedrock/src/libraries/rlp/RLPWriter.sol"; import {RLPWriter} from "@eth-optimism/contracts-bedrock/src/libraries/rlp/RLPWriter.sol";
import {MerkleTrie} from "@eth-optimism/contracts-bedrock/src/libraries/trie/MerkleTrie.sol"; import {MerkleTrie} from "@eth-optimism/contracts-bedrock/src/libraries/trie/MerkleTrie.sol";
/// @notice Storage Proof library inspired by Succinct (https://github.com/succinctlabs)
library StorageProof { library StorageProof {
using RLPReader for RLPReader.RLPItem; using RLPReader for RLPReader.RLPItem;
using RLPReader for bytes; using RLPReader for bytes;

Loading…
Cancel
Save