Move#46
Merged
Merged
Conversation
- Implemented secure storage for MVSM modules in the VM, replacing file-based storage with RocksDB. - Added error handling and validation for empty keys in storage operations. - Improved staking logic with additional checks for zero amounts and existing stakes. - Enhanced reward processing with overflow protection and validation for block heights. - Refactored staking state save/load functions to include comprehensive error handling. - Introduced logging for better traceability of storage operations and staking processes. - Added limits to prevent excessive memory usage during key retrieval in storage.
- Added `publish.rs` for handling deployment of Move modules to the blockchain, including signature verification and gas estimation. - Introduced `storage.rs` to manage VM state persistence using RocksDB, including comprehensive metadata and module summaries. - Created utility functions in `utils.rs` for path management, object ID generation, and module dependency retrieval. - Defined `VMModule` structure in `vm_module.rs` for representing Move modules, with functions for loading and serializing modules to secure storage. - Established `vm_state.rs` to manage the VM state, including module registration and state retrieval with variations. - Developed `vm_transaction.rs` for creating and executing VM transactions, including conversion from blockchain transactions. - Enhanced `process_vm_transaction` in `simulation/mod.rs` to handle VM function calls and integrate with the blockchain transaction system. - Updated transaction processing logic to accommodate VM transactions and ensure proper execution and error handling.
✅ Deploy Preview for movekari canceled.
|
✅ Deploy Preview for whitepaperkari canceled.
|
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
This pull request introduces significant updates to the Move module deployment and interaction functionality, streamlines the CLI commands, and enhances documentation for Move VM operations. The most important changes include removing the
--mvsm-fileparameter, simplifying thePublishstruct and its execution logic, renaming an example file for clarity, and adding comprehensive Move VM API documentation.Move CLI Updates:
--mvsm-fileparameter from thehandle_move_commandfunction to simplify the command structure. (crates/command/src/move_cli/mod.rs, [1] [2]Publishstruct inpublish.rsto reduce the default gas budget, simplify descriptions, and refactor the deployment logic for better modularity and error handling. (crates/kari-move/src/base/publish.rs, crates/kari-move/src/base/publish.rsL6-R216)File and Dependency Adjustments:
sign_verifyexample tokey_sign_verifyfor clarity. (crates/core/wallet/key/Cargo.toml, crates/core/wallet/key/Cargo.tomlL48-R49)mona-crypto,mona-blockchain, andpanorama) fromCargo.tomlto clean up the project. (crates/kari/Cargo.toml, crates/kari/Cargo.tomlL19-L22)Documentation Enhancements:
crates/rpc/rpc-api/1.Blockchain.md, crates/rpc/rpc-api/1.Blockchain.mdR187-R297)Summary
Summary about this PR