Skip to content

Ar/psr 84 use smart contract trie for key value store#1612

Open
allanbrondum wants to merge 17 commits intofeature/rust_block_statefrom
ar/psr-84-use-smart-contract-trie-for-key-value-store
Open

Ar/psr 84 use smart contract trie for key value store#1612
allanbrondum wants to merge 17 commits intofeature/rust_block_statefrom
ar/psr-84-use-smart-contract-trie-for-key-value-store

Conversation

@allanbrondum
Copy link
Copy Markdown
Collaborator

@allanbrondum allanbrondum commented Apr 8, 2026

Purpose

Change the token key-value state to be implemented by the smart contract trie.

Changes

  • implemented adapter for the smart contract trie in block_state::smart_contract_trie
  • changed state::protocol_level_tokens to use the smart contract trie adapter

Checklist

  • My code follows the style of this project.
  • The code compiles without warnings.
  • I have performed a self-review of the changes.
  • I have documented my code, in particular the intent of the
    hard-to-understand areas.
  • (If necessary) I have updated the CHANGELOG.

@allanbrondum allanbrondum marked this pull request as draft April 8, 2026 19:06
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 10, 2026

Codecov Report

❌ Patch coverage is 94.30199% with 20 lines in your changes missing coverage. Please review.
✅ Project coverage is 63.81%. Comparing base (df755fd) to head (2f0b3db).
⚠️ Report is 22 commits behind head on feature/rust_block_state.

Files with missing lines Patch % Lines
...block-state/src/block_state/smart_contract_trie.rs 96.26% 10 Missing ⚠️
plt/plt-token-module/src/key_value_state.rs 81.48% 5 Missing ⚠️
...ate/src/block_state/state/protocol_level_tokens.rs 75.00% 3 Missing ⚠️
plt/plt-scheduler/src/token_kernel.rs 50.00% 2 Missing ⚠️
Additional details and impacted files
@@                     Coverage Diff                      @@
##           feature/rust_block_state    #1612      +/-   ##
============================================================
+ Coverage                     63.29%   63.81%   +0.51%     
============================================================
  Files                           351      354       +3     
  Lines                         69602    70742    +1140     
  Branches                       3013     3015       +2     
============================================================
+ Hits                          44058    45147    +1089     
- Misses                        22531    22580      +49     
- Partials                       3013     3015       +2     
Files with missing lines Coverage Δ
plt/plt-block-state/src/block_state.rs 96.49% <100.00%> (+13.08%) ⬆️
...ate/src/block_state/types/protocol_level_tokens.rs 100.00% <ø> (ø)
...-scheduler-interface/src/token_kernel_interface.rs 100.00% <ø> (ø)
plt/plt-scheduler/src/queries.rs 100.00% <ø> (ø)
plt/plt-token-module/src/roles.rs 98.66% <100.00%> (ø)
...lt/plt-token-module/src/token_module/initialize.rs 80.28% <100.00%> (+0.57%) ⬆️
plt/plt-scheduler/src/token_kernel.rs 79.89% <50.00%> (ø)
...ate/src/block_state/state/protocol_level_tokens.rs 87.24% <75.00%> (ø)
plt/plt-token-module/src/key_value_state.rs 86.19% <81.48%> (+0.29%) ⬆️
...block-state/src/block_state/smart_contract_trie.rs 96.26% <96.26%> (ø)

... and 8 files with indirect coverage changes

@allanbrondum allanbrondum marked this pull request as ready for review April 10, 2026 05:27
@allanbrondum allanbrondum force-pushed the ar/psr-84-use-smart-contract-trie-for-key-value-store branch from 7a5a954 to b8cb469 Compare April 10, 2026 12:48
Base automatically changed from ar/psr-11-port-the-plt-block-state-to-rust to feature/rust_block_state April 15, 2026 07:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant