feat: ✨ Integrate VerificationPayloads #6
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes
This PR introduces flexible vault proof verification and simplifies the codebase architecture:
Proof Verification System: Added
ProofManagerandFlexibleVaultProofResolverclasses to handle merkle root validation for flexible vault operations. Supports multiple verification blobs viaflexible_vault_proof_blobparameter.Local-Only Proof Strategy: Removed remote proof fetching capabilities. Proofs must now be provided as local JSON blobs, eliminating network dependencies and improving reliability.
Configuration Consolidation: Merged
BridgeConfiginto unifiedEVMClientConfig, reducing configuration complexity. Addeddisable_call_verificationflag for testing scenarios.Codebase Simplification: Major refactoring across core modules (
base.py,utils.py,types.py) removing ~1300 lines while maintaining functionality. Consolidated response types into singleResponseclass.Enhanced Bridge Support: Improved CCTP bridge implementation with better error handling and transaction state management. Added comprehensive test coverage for bridge operations.
Test Coverage: Added
test_flexible_vault.pyandtest_cctp_bridge.pywith extensive unit tests for new proof verification logic and bridge functionality.Key behavioral change: Clients must now supply proof blobs directly rather than fetching from remote endpoints, improving performance and reducing external dependencies.