feat(dpp): token distribution model #2447
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the WalkthroughThis pull request introduces a comprehensive overhaul of the token distribution system in the Dash Platform Protocol (DPP) Rust implementation. The changes focus on restructuring how token configurations, distributions, and rules are managed. A new Changes
Sequence DiagramsequenceDiagram
participant TokenConfig as Token Configuration
participant DistributionRules as Distribution Rules
participant MintTransition as Token Mint Transition
TokenConfig->>DistributionRules: Get distribution rules
DistributionRules-->>TokenConfig: Return rules
MintTransition->>DistributionRules: Check new tokens destination
DistributionRules-->>MintTransition: Provide destination identity
MintTransition->>MintTransition: Validate and process token mint
Possibly related PRs
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🔭 Outside diff range comments (1)
packages/rs-drive-abci/src/execution/validation/state_transition/state_transitions/batch/mod.rs (1)
Line range hint
1-11800: Security considerationsThe code has several important security features but could benefit from additional safeguards:
Token Operations:
- Verify all token operations have proper authorization checks
- Add rate limiting for token operations
- Consider adding emergency pause functionality
Group Management:
- Add timeouts for group operations
- Implement quorum requirements for critical operations
- Add member removal capabilities
Configuration Updates:
- Add delays for critical configuration changes
- Implement multi-sig requirements for high-risk operations
Consider implementing these additional security measures to enhance the token distribution model's security.
🧹 Nitpick comments (27)
packages/rs-dpp/src/data_contract/associated_token/token_perpetual_distribution/distribution_function/mod.rs (1)
88-109: Exponential function usage is well explained.
No immediate issues, though in calling code, consider limiting large exponents to evade numeric overflow.packages/rs-drive-abci/src/execution/validation/state_transition/state_transitions/batch/mod.rs (5)
10205-10205: Fix line indentationThe line is incorrectly indented compared to surrounding code.
+ use dpp::data_contract::associated_token::token_distribution_rules::accessors::v0::TokenDistributionRulesV0Setters;
10978-10984: Simplify chained method callsThe code has multiple chained method calls that can be simplified for better readability.
- token_configuration - .distribution_rules_mut() - .set_minting_allow_choosing_destination(false); - token_configuration - .distribution_rules_mut() - .set_new_tokens_destination_identity(Some(identity.id())); + let rules = token_configuration.distribution_rules_mut(); + rules.set_minting_allow_choosing_destination(false); + rules.set_new_tokens_destination_identity(Some(identity.id()));
11081-11084: Simplify chained method callsSimilar to above, the chained method calls can be simplified.
- token_configuration - .distribution_rules_mut() - .set_minting_allow_choosing_destination(false); - token_configuration - .distribution_rules_mut() - .set_new_tokens_destination_identity(Some(identity.id())); + let rules = token_configuration.distribution_rules_mut(); + rules.set_minting_allow_choosing_destination(false); + rules.set_new_tokens_destination_identity(Some(identity.id()));
11181-11184: Simplify chained method callsSimilar to above, the chained method calls can be simplified.
- token_configuration - .distribution_rules_mut() - .set_minting_allow_choosing_destination(false); - token_configuration - .distribution_rules_mut() - .set_new_tokens_destination_identity(Some(identity.id())); + let rules = token_configuration.distribution_rules_mut(); + rules.set_minting_allow_choosing_destination(false); + rules.set_new_tokens_destination_identity(Some(identity.id()));
Line range hint
1-11800: Overall architecture feedbackThe implementation demonstrates a robust token distribution model with several key strengths:
- Comprehensive test coverage for various token operations
- Strong validation of token operations
- Support for group-based authorization
- Proper handling of token freezing/unfreezing
- Well-implemented token configuration updates
However, there are some areas that could be improved:
Error Handling:
- Consider adding more specific error types for token operations
- Add error context information for better debugging
Documentation:
- Add documentation for complex token operations
- Document the group authorization model
Code Organization:
- Consider splitting large test modules into smaller files
- Extract common test setup code into helper functions
packages/rs-dpp/src/data_contract/associated_token/token_distribution_rules/mod.rs (1)
18-26: Consider adding documentation for the Display implementation.While the implementation is correct, adding documentation would help explain the formatting choices and make it clear that it delegates to the inner type's Display implementation.
+/// Implements Display for TokenDistributionRules by delegating to the inner type's +/// Display implementation. This ensures consistent formatting across versions. impl fmt::Display for TokenDistributionRules {packages/rs-dpp/src/data_contract/associated_token/token_perpetual_distribution/mod.rs (2)
10-15: LGTM! Consider adding documentation.The enum is well-structured with proper versioning. Consider adding documentation to explain the purpose of
TokenPerpetualDistributionand why it needsPartialOrd.+/// Represents perpetual token distribution rules with versioning support. +/// Implements PartialOrd to allow comparison of distribution configurations. #[derive(Serialize, Deserialize, Encode, Decode, Debug, Clone, PartialEq, Eq, PartialOrd, From)]
17-25: Consider creating a macro for versioned enum pattern.This Display implementation is identical to
TokenDistributionRules. Consider creating a macro to reduce boilerplate for versioned enums./// Example macro (to be placed in a common utility module): macro_rules! impl_versioned_display { ($type:ident) => { impl fmt::Display for $type { fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result { match self { Self::V0(v0) => write!(f, "{}", v0) } } } }; } // Usage: impl_versioned_display!(TokenPerpetualDistribution);packages/rs-dpp/src/data_contract/associated_token/token_pre_programmed_distribution/v0/mod.rs (1)
10-14: Add documentation and consider performance implications.The nested BTreeMap structure is well-chosen for ordered timestamps, but consider:
- Adding documentation to explain the structure and its performance characteristics
- For large datasets, consider implementing methods to efficiently batch process distributions
+/// Represents pre-programmed token distributions organized by timestamps. +/// Uses BTreeMap for ordered access with O(log n) complexity. +/// - Outer map: Timestamp -> Distribution map +/// - Inner map: Identifier -> Token amount for each recipient #[derive(Serialize, Deserialize, Decode, Encode, Debug, Clone, PartialEq, Eq)] #[serde(rename_all = "camelCase")] pub struct TokenPreProgrammedDistributionV0 { + /// Nested map structure for timestamp-based token distributions. + /// Performance note: Operations have O(log n) complexity where n is the number of timestamps. pub distributions: BTreeMap<TimestampMillis, BTreeMap<Identifier, TokenAmount>>, }packages/rs-dpp/src/lib.rs (1)
82-83: Add documentation for interval types.While the type aliases are well-chosen, they would benefit from documentation explaining:
- The purpose and use case for each interval type
- Any constraints or valid value ranges
- The relationship between these intervals
+ /// Represents an interval in epochs. Valid values are non-zero positive integers. pub type EpochInterval = u16; + /// Represents an interval in block heights. Must be greater than zero. pub type BlockHeightInterval = u64; + /// Represents a time interval in milliseconds. Must be greater than zero. pub type TimestampMillisInterval = u64;Also applies to: 86-87, 91-92
packages/rs-dpp/src/data_contract/associated_token/token_perpetual_distribution/v0/mod.rs (2)
11-27: Consider enhancing enum documentation with examples.The enum variants are well-documented with basic descriptions. Consider adding examples for each variant to illustrate typical usage scenarios and parameter values.
#[derive(Serialize, Deserialize, Decode, Encode, Debug, Clone, PartialEq, Eq, PartialOrd)] pub enum RewardDistributionType { - /// A fixed amount of tokens is emitted every n blocks + /// A fixed amount of tokens is emitted every n blocks + /// Example: BlockFixed(100, 1000) // Emit 1000 tokens every 100 blocks BlockFixed(BlockHeightInterval, TokenAmount),
38-70: Consider optimizing string formatting in Display implementation.The current implementation creates intermediate String allocations for None cases. Consider using references to static strings to reduce allocations.
- None => "None".to_string(), + None => "None",packages/rs-dpp/src/data_contract/associated_token/token_distribution_rules/v0/mod.rs (1)
48-79: Consider optimizing string formatting in Display implementation.Similar to the previous file, the implementation creates unnecessary String allocations for None cases.
- None => "None".to_string(), + None => "None",packages/rs-dpp/src/data_contract/change_control_rules/mod.rs (1)
130-138: Consider adding documentation for the Display implementation.The Display implementation is correct but would benefit from documentation explaining the string format produced by the v0 implementation.
Apply this diff to add documentation:
+/// Implements Display for ChangeControlRules by delegating to the inner type's Display implementation. +/// The string format follows the format defined by the v0 implementation. impl fmt::Display for ChangeControlRules { fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result { match self { ChangeControlRules::V0(v0) => { write!(f, "{}", v0) //just pass through } } } }packages/rs-dpp/src/data_contract/associated_token/token_distribution_rules/v0/accessors.rs (1)
1-114: LGTM! Consider adding examples to documentation.The implementation is well-structured with clear separation of concerns between getters and setters. The code follows Rust best practices and provides comprehensive access to distribution rules.
Consider adding usage examples to the documentation for complex scenarios, such as:
// Example: Setting up perpetual distribution let mut rules = TokenDistributionRulesV0::default(); rules.set_perpetual_distribution(Some(TokenPerpetualDistribution::new(...))); rules.set_perpetual_distribution_rules(ChangeControlRules::new(...));packages/rs-dpp/src/data_contract/change_control_rules/v0/mod.rs (1)
163-181: LGTM! Consider adding Display implementation tests.The Display implementation is well-structured and properly formats all fields. The output format is clear and readable.
Consider adding tests to verify the Display implementation:
#[test] fn test_change_control_rules_display() { let rules = ChangeControlRulesV0 { authorized_to_make_change: AuthorizedActionTakers::NoOne, admin_action_takers: AuthorizedActionTakers::NoOne, changing_authorized_action_takers_to_no_one_allowed: true, changing_admin_action_takers_to_no_one_allowed: false, self_changing_admin_action_takers_allowed: true, }; let expected = "ChangeControlRulesV0 {\n authorized_to_make_change: NoOne,\n ...}"; assert!(rules.to_string().starts_with(expected)); }packages/rs-dpp/src/data_contract/associated_token/token_distribution_rules/accessors/mod.rs (1)
98-158: Consider adding validation in setters.While the setter implementation is clean, consider adding validation logic to ensure the rules being set are consistent with the token's configuration.
For example, you could add validation in
set_perpetual_distribution:fn set_perpetual_distribution( &mut self, perpetual_distribution: Option<TokenPerpetualDistribution>, ) { + // Validate the distribution configuration + if let Some(dist) = &perpetual_distribution { + // Add validation logic here + } match self { TokenDistributionRules::V0(v0) => v0.set_perpetual_distribution(perpetual_distribution), } }packages/rs-dpp/src/data_contract/associated_token/token_configuration/accessors/mod.rs (1)
72-82: Good architectural decision to consolidate distribution rules.The addition of
distribution_rulesanddistribution_rules_mutmethods provides a cleaner interface for managing token distribution configurations. This change improves modularity and maintainability.Consider documenting the migration path for existing implementations that might be using the removed identity management methods.
packages/rs-dpp/src/data_contract/associated_token/token_configuration/v0/accessors.rs (2)
130-138: Thorough implementation of group position collection.The code properly integrates distribution rules into the group position collection process. However, consider extracting the group position collection logic into a separate method for better maintainability.
Consider refactoring to:
fn all_used_group_positions(&self) -> BTreeSet<GroupContractPosition> { let mut group_positions = BTreeSet::new(); + self.collect_distribution_rule_positions(&mut group_positions); + // ... rest of the implementation } + +fn collect_distribution_rule_positions(&self, positions: &mut BTreeSet<GroupContractPosition>) { + add_from_change_control_rules( + self.distribution_rules.new_tokens_destination_identity_rules(), + ); + add_from_change_control_rules( + self.distribution_rules.minting_allow_choosing_destination_rules(), + ); + add_from_change_control_rules(self.distribution_rules.perpetual_distribution_rules()); +}
180-182: Consider adding debug assertions for distribution rules.The setter for distribution rules could benefit from debug assertions to catch potential issues early in development.
fn set_distribution_rules(&mut self, rules: TokenDistributionRules) { + debug_assert!(matches!(rules, TokenDistributionRules::V0(_)), + "Attempting to set non-V0 distribution rules on V0 configuration"); self.distribution_rules = rules; }packages/rs-drive-abci/src/execution/validation/state_transition/state_transitions/data_contract_update/mod.rs (1)
1306-1524: Well-structured test cases for token immutability.The test suite comprehensively covers token immutability requirements:
- Preventing token removal
- Preventing token modification
- Proper error handling and validation
The tests are well-organized with clear setup, execution, and verification phases.
However, consider adding the following test cases for completeness:
- Attempting to modify multiple tokens simultaneously
- Edge cases with token positions (e.g., non-existent positions)
packages/rs-dpp/src/data_contract/associated_token/token_pre_programmed_distribution/mod.rs (1)
16-24: Consider enhancing the Display implementation.While the current implementation is functional, consider adding version information to the display output for better debugging.
impl fmt::Display for TokenPreProgrammedDistribution { fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result { match self { TokenPreProgrammedDistribution::V0(v0) => { - write!(f, "{}", v0) //just pass through + write!(f, "V0({})", v0) } } } }packages/rs-drive-abci/src/execution/validation/state_transition/state_transitions/batch/action_validation/token/token_mint_transition_action/state_v0/mod.rs (4)
Line range hint
95-103: Improve error message specificity.The error message for corrupted drive state could be more informative by including the context of the operation.
- "token {} total supply not found", + "token {} total supply not found during mint validation",
Line range hint
61-91: Consider extracting supply validation logic.The max supply validation logic is complex enough to warrant its own method. This would improve readability and make the code more maintainable.
impl TokenMintTransitionAction { fn validate_max_supply( &self, token_total_supply: u64, max_supply: u64, ) -> Result<SimpleConsensusValidationResult, Error> { match token_total_supply.checked_add(self.mint_amount()) { Some(total_supply_after_mint) if total_supply_after_mint <= max_supply => { Ok(SimpleConsensusValidationResult::new()) } _ => Ok(SimpleConsensusValidationResult::new_with_error( ConsensusError::StateError(StateError::TokenMintPastMaxSupplyError( TokenMintPastMaxSupplyError::new( self.token_id(), self.mint_amount(), token_total_supply, max_supply, ), )), )), } } }
Line range hint
106-132: Extract recipient validation to a helper function.The recipient validation logic could be moved to a separate method to improve code organization and reusability.
impl TokenMintTransitionAction { fn validate_recipient( &self, platform: &PlatformStateRef, owner_id: Identifier, transaction: TransactionArg, execution_context: &mut StateTransitionExecutionContext, platform_version: &PlatformVersion, ) -> Result<SimpleConsensusValidationResult, Error> { let recipient = self.identity_balance_holder_id(); if recipient == owner_id { return Ok(SimpleConsensusValidationResult::new()); } let balance = platform.drive.fetch_identity_balance( recipient.to_buffer(), transaction, platform_version, )?; execution_context.add_operation(ValidationOperation::RetrieveIdentity( RetrieveIdentityInfo::only_balance(), )); match balance { Some(_) => Ok(SimpleConsensusValidationResult::new()), None => Ok(SimpleConsensusValidationResult::new_with_error( ConsensusError::StateError(StateError::RecipientIdentityDoesNotExistError( RecipientIdentityDoesNotExistError::new(recipient), )), )), } } }
Line range hint
32-145: Consider adding const assertions for numeric limits.The implementation handles overflow checks well but could benefit from const assertions to validate numeric limits at compile time.
Add the following at the module level:
const _: () = assert!(std::u64::MAX > 1_000_000_000_000, "u64 must be large enough to handle token amounts");
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
Cargo.lockis excluded by!**/*.lock
📒 Files selected for processing (40)
packages/rs-dpp/Cargo.toml(1 hunks)packages/rs-dpp/src/data_contract/associated_token/mod.rs(1 hunks)packages/rs-dpp/src/data_contract/associated_token/token_configuration/accessors/mod.rs(3 hunks)packages/rs-dpp/src/data_contract/associated_token/token_configuration/accessors/v0/mod.rs(3 hunks)packages/rs-dpp/src/data_contract/associated_token/token_configuration/methods/apply_token_configuration_item/v0/mod.rs(2 hunks)packages/rs-dpp/src/data_contract/associated_token/token_configuration/methods/authorized_action_takers_for_configuration_item/v0/mod.rs(2 hunks)packages/rs-dpp/src/data_contract/associated_token/token_configuration/methods/can_apply_token_configuration_item/v0/mod.rs(5 hunks)packages/rs-dpp/src/data_contract/associated_token/token_configuration/methods/validate_token_configuration_update/v0/mod.rs(4 hunks)packages/rs-dpp/src/data_contract/associated_token/token_configuration/v0/accessors.rs(4 hunks)packages/rs-dpp/src/data_contract/associated_token/token_configuration/v0/mod.rs(4 hunks)packages/rs-dpp/src/data_contract/associated_token/token_configuration_item.rs(4 hunks)packages/rs-dpp/src/data_contract/associated_token/token_distribution_rules/accessors/mod.rs(1 hunks)packages/rs-dpp/src/data_contract/associated_token/token_distribution_rules/accessors/v0/mod.rs(1 hunks)packages/rs-dpp/src/data_contract/associated_token/token_distribution_rules/mod.rs(1 hunks)packages/rs-dpp/src/data_contract/associated_token/token_distribution_rules/v0/accessors.rs(1 hunks)packages/rs-dpp/src/data_contract/associated_token/token_distribution_rules/v0/mod.rs(1 hunks)packages/rs-dpp/src/data_contract/associated_token/token_perpetual_distribution/distribution_function/encode.rs(1 hunks)packages/rs-dpp/src/data_contract/associated_token/token_perpetual_distribution/distribution_function/mod.rs(1 hunks)packages/rs-dpp/src/data_contract/associated_token/token_perpetual_distribution/mod.rs(1 hunks)packages/rs-dpp/src/data_contract/associated_token/token_perpetual_distribution/v0/mod.rs(1 hunks)packages/rs-dpp/src/data_contract/associated_token/token_pre_programmed_distribution/mod.rs(1 hunks)packages/rs-dpp/src/data_contract/associated_token/token_pre_programmed_distribution/v0/mod.rs(1 hunks)packages/rs-dpp/src/data_contract/change_control_rules/mod.rs(2 hunks)packages/rs-dpp/src/data_contract/change_control_rules/v0/mod.rs(2 hunks)packages/rs-dpp/src/lib.rs(1 hunks)packages/rs-dpp/src/state_transition/state_transitions/document/batch_transition/batched_transition/token_mint_transition/v0/v0_methods.rs(2 hunks)packages/rs-dpp/src/state_transition/state_transitions/document/batch_transition/batched_transition/token_transition.rs(2 hunks)packages/rs-dpp/src/state_transition/state_transitions/document/batch_transition/v1/v0_methods.rs(0 hunks)packages/rs-drive-abci/src/execution/validation/state_transition/state_transitions/batch/action_validation/token/token_burn_transition_action/state_v0/mod.rs(1 hunks)packages/rs-drive-abci/src/execution/validation/state_transition/state_transitions/batch/action_validation/token/token_destroy_frozen_funds_transition_action/state_v0/mod.rs(1 hunks)packages/rs-drive-abci/src/execution/validation/state_transition/state_transitions/batch/action_validation/token/token_emergency_action_transition_action/state_v0/mod.rs(0 hunks)packages/rs-drive-abci/src/execution/validation/state_transition/state_transitions/batch/action_validation/token/token_freeze_transition_action/state_v0/mod.rs(0 hunks)packages/rs-drive-abci/src/execution/validation/state_transition/state_transitions/batch/action_validation/token/token_mint_transition_action/state_v0/mod.rs(1 hunks)packages/rs-drive-abci/src/execution/validation/state_transition/state_transitions/batch/action_validation/token/token_unfreeze_transition_action/state_v0/mod.rs(1 hunks)packages/rs-drive-abci/src/execution/validation/state_transition/state_transitions/batch/mod.rs(7 hunks)packages/rs-drive-abci/src/execution/validation/state_transition/state_transitions/data_contract_update/mod.rs(2 hunks)packages/rs-drive-abci/tests/strategy_tests/token_tests.rs(3 hunks)packages/rs-drive/src/drive/contract/insert/insert_contract/v1/mod.rs(2 hunks)packages/rs-drive/src/drive/tokens/estimated_costs/for_token_balances/v0/mod.rs(0 hunks)packages/rs-drive/src/state_transition_action/batch/batched_transition/token_transition/token_mint_transition_action/v0/transformer.rs(5 hunks)
💤 Files with no reviewable changes (4)
- packages/rs-drive/src/drive/tokens/estimated_costs/for_token_balances/v0/mod.rs
- packages/rs-drive-abci/src/execution/validation/state_transition/state_transitions/batch/action_validation/token/token_freeze_transition_action/state_v0/mod.rs
- packages/rs-drive-abci/src/execution/validation/state_transition/state_transitions/batch/action_validation/token/token_emergency_action_transition_action/state_v0/mod.rs
- packages/rs-dpp/src/state_transition/state_transitions/document/batch_transition/v1/v0_methods.rs
✅ Files skipped from review due to trivial changes (1)
- packages/rs-dpp/src/data_contract/associated_token/mod.rs
🧰 Additional context used
📓 Learnings (1)
packages/rs-dpp/src/data_contract/associated_token/token_configuration/v0/mod.rs (1)
Learnt from: QuantumExplorer
PR: dashpay/platform#2432
File: packages/rs-dpp/src/data_contract/associated_token/token_configuration/v0/mod.rs:222-225
Timestamp: 2025-01-20T16:20:59.791Z
Learning: In the Dash Platform codebase, TokenAmount from crate::balances::credits is compatible with u64 when used for token base supply.
🔇 Additional comments (64)
packages/rs-dpp/src/data_contract/associated_token/token_configuration/methods/apply_token_configuration_item/v0/mod.rs (10)
3-5: Imports look consistent.
These imports seamlessly integrateTokenDistributionRulesV0GettersandTokenDistributionRulesV0Settersinto the existing token configuration flows with no obvious issues.
42-43: No concerns with direct setter usage.
Callingset_new_tokens_destination_identityhere is straightforward. Ensure that any specialized validation of theidentityis done upstream if necessary.
48-49: Authorized action takers updated properly.
Usingnew_tokens_destination_identity_rules_mut()is consistent with the modular approach to distribution rules.
53-54: Admin action takers updated properly.
This closely mirrors patterns used in previous blocks.
58-59: Allow choosing destination is set correctly.
Again, no functional or style issues are evident.
64-65: Control group assignment is consistent.
Matches earlier patterns for control group modifications.
71-73: Admin group setters are consistent with design.
No issues.
75-78: Perpetual distribution inclusion is logically consistent.
Callingset_perpetual_distributionintegrates well with the rest of the distribution rules.
79-83: Perpetual distribution control group matches pattern.
No concerns.
84-86: Perpetual distribution admin group also follows pattern.
Assigning admin action takers remains consistent and readable.packages/rs-dpp/src/data_contract/associated_token/token_perpetual_distribution/distribution_function/mod.rs (8)
1-5: Imports and module structure are appropriate.
Usingordered_float::NotNanhelps ensure safe handling of floating-point values for distribution logic.
8-28: Excellent documentation for LinearInteger.
The doc comments clearly explain usage, formula, and examples. This is highly maintainable.
29-49: Clear extension for LinearFloat.
Floating-point slope adds flexibility, andNotNan<f64>is a good safeguard.
50-70: PolynomialInteger variant is clearly documented.
Consider verifying input domain constraints for exponents (n >= 1) in calling code to avoid edge cases.
71-87: PolynomialFloat extends polynomial curves to fractional powers.
Be mindful of complex number edges ifn< 0. Possibly enforce domain constraints at a higher layer.
110-130: Logarithmic function is functionally comprehensive.
Logs require positivex. Ensure domain checks in upstream logic.
131-143: Stepwise approach offers advanced schedule control.
The vector-based approach is flexible. Confirm step intervals align with block or epoch definitions.
145-196: Display implementation is thorough.
Formatting each variant fosters clearer logs and error messages. This is a clean and well-structured approach.packages/rs-dpp/src/data_contract/associated_token/token_perpetual_distribution/distribution_function/encode.rs (4)
1-5: Imports are correct for serialization.
No concerns regarding re-exports or side-effects.
6-54: Encode trait implementation is consistent and thorough.
Assigning a variant index is straightforward, and usinginto_inner()forNotNanis suitably handled.
56-120: Decode trait implementation effectively guards against NaN.
Returning a decode error on invalid floats is essential for robust domain handling.
122-179: BorrowDecode trait supports efficient decoding.
Mirroring theDecodelogic while preserving references is a nice optimization. This matches Rust best practices for reducing copies.packages/rs-dpp/src/data_contract/associated_token/token_distribution_rules/mod.rs (1)
8-13: LGTM! Well-structured versioned enum.The
TokenDistributionRulesenum is well-designed with proper versioning support through serde tags and follows semantic versioning conventions.packages/rs-dpp/src/data_contract/associated_token/token_pre_programmed_distribution/v0/mod.rs (1)
17-28: LGTM! Well-formatted Display implementation.The Display implementation provides clear, hierarchical formatting of the distribution data.
packages/rs-dpp/src/data_contract/associated_token/token_distribution_rules/accessors/v0/mod.rs (1)
6-73: Well-structured accessor traits with comprehensive documentation!The traits provide a clean and well-documented interface for managing token distribution rules. The separation of getters and setters, consistent method naming, and appropriate use of mutability demonstrate good design practices.
packages/rs-dpp/src/data_contract/associated_token/token_distribution_rules/v0/mod.rs (1)
38-46: Verify if the default change control rules are too restrictive.The default implementation sets all permissions to
NoOneand disables all changes. This might be too restrictive for typical use cases and could lead to tokens being permanently locked down.Consider if a more permissive default would be more appropriate, or if this is intentionally restrictive for security reasons.
packages/rs-dpp/src/state_transition/state_transitions/document/batch_transition/batched_transition/token_mint_transition/v0/v0_methods.rs (1)
85-91: Clean integration with the new token distribution rules!The changes effectively integrate the new distribution rules architecture while maintaining proper error handling and type safety.
packages/rs-dpp/src/data_contract/associated_token/token_configuration/accessors/v0/mod.rs (3)
3-3: LGTM! Import added for new TokenDistributionRules type.The import aligns with the new distribution rules functionality being introduced.
31-35: LGTM! New getter methods for distribution rules.The addition of both immutable and mutable accessors for distribution rules follows Rust's best practices for encapsulation.
80-81: LGTM! Setter method for distribution rules.The setter method complements the getter methods, providing a complete interface for managing distribution rules.
packages/rs-drive-abci/src/execution/validation/state_transition/state_transitions/batch/action_validation/token/token_unfreeze_transition_action/state_v0/mod.rs (1)
4-4: Verify error handling after UnauthorizedTokenActionError removal.The removal of UnauthorizedTokenActionError import suggests a change in error handling strategy. Please ensure that unauthorized actions are still properly handled, possibly through a different mechanism.
packages/rs-drive-abci/src/execution/validation/state_transition/state_transitions/batch/action_validation/token/token_burn_transition_action/state_v0/mod.rs (1)
4-4: Verify error handling consistency across token actions.The removal of UnauthorizedTokenActionError import aligns with the changes in token_unfreeze_transition_action. Please ensure that error handling is consistent across all token actions.
✅ Verification successful
Error handling is consistent across token actions
The removal of UnauthorizedTokenActionError import aligns with the codebase's error handling patterns. The verification shows consistent error handling across all token actions, including token burn and unfreeze transitions, with proper use of ConsensusError::StateError wrapper and standardized error types.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Search for error handling patterns in token action files fd -e rs "token.*transition_action" -x rg -A 5 "ConsensusError::StateError" # Search for validation patterns in token action files fd -e rs "token.*transition_action" -x ast-grep --pattern 'validate_group_action($$$)'Length of output: 164
Script:
#!/bin/bash # Search for error-related imports in token action files fd "token.*transition_action.*\.rs$" -x rg -l "use.*Error" # Search for specific error patterns in token files fd "token.*transition_action.*\.rs$" -x rg -B 2 -A 2 "Error::" # Search for validate methods and their implementations ast-grep --pattern 'fn validate($$$) -> Result<$$$> { $$$ }'Length of output: 8825
Script:
#!/bin/bash # Search for token-related error types and their usage fd -e rs -p ".*token.*transition_action.*" packages/rs-drive-abci/src/execution/validation/state_transition -x rg -l "Error" # Search for specific error handling in token action files fd -e rs -p ".*token.*transition_action.*" packages/rs-drive-abci/src/execution/validation/state_transition -x rg -B 2 -A 2 "IdentityDoesNotHaveEnoughTokenBalanceError|UnauthorizedTokenActionError" # Look for validation methods in token action files fd -e rs -p ".*token.*transition_action.*" packages/rs-drive-abci/src/execution/validation/state_transition -x ast-grep --pattern 'fn validate($$$) -> Result<$$$, $$$> { $$$ }'Length of output: 8972
packages/rs-dpp/src/data_contract/change_control_rules/mod.rs (1)
14-14: LGTM! Import added for Display trait implementation.The import is required for implementing the Display trait.
packages/rs-drive-abci/src/execution/validation/state_transition/state_transitions/batch/action_validation/token/token_destroy_frozen_funds_transition_action/state_v0/mod.rs (1)
4-4: LGTM! Verify error handling after UnauthorizedTokenActionError removal.The removal of
UnauthorizedTokenActionErroraligns with the shift to token distribution rules for authorization. The remaining error handling throughIdentityTokenAccountNotFrozenErroris sufficient.Let's verify that
UnauthorizedTokenActionErroris not used elsewhere in the codebase:✅ Verification successful
Verified: UnauthorizedTokenActionError removal is correct for frozen funds destruction
The error removal is specific to frozen funds destruction validation, where authorization is now handled through token distribution rules. The error remains in use for other token-related validations where needed.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Search for any remaining usage of UnauthorizedTokenActionError rg "UnauthorizedTokenActionError"Length of output: 6535
packages/rs-dpp/src/data_contract/associated_token/token_configuration/methods/authorized_action_takers_for_configuration_item/v0/mod.rs (2)
25-27: Consistent shift to admin_action_takers for authorization.The change from
authorized_to_make_change_action_takers()toadmin_action_takers()across multiple configuration items improves consistency in authorization handling.Also applies to: 35-37, 86-88, 92-94, 98-100, 104-106, 118-120
41-55: Well-integrated perpetual distribution authorization.The new cases for perpetual distribution properly integrate with the existing authorization framework, maintaining consistency with other configuration items.
packages/rs-dpp/src/data_contract/associated_token/token_distribution_rules/accessors/mod.rs (1)
12-96: Well-structured implementation of TokenDistributionRulesV0Getters.The implementation provides a comprehensive and well-documented API for accessing token distribution rules. The pattern matching ensures type safety and proper handling of the V0 variant.
packages/rs-drive/src/drive/contract/insert/insert_contract/v1/mod.rs (1)
223-225: LGTM!The code correctly retrieves the destination identity from the distribution rules, with a sensible fallback to the contract owner ID.
packages/rs-dpp/src/data_contract/associated_token/token_configuration_item.rs (3)
40-42: LGTM!The new variants follow a consistent pattern and use appropriate types for managing perpetual distribution configuration.
73-94: LGTM!The indices are correctly assigned and maintain sequential ordering after adding the new variants.
124-134: LGTM!The
Displayimplementation for the new variants is consistent with the existing pattern.packages/rs-dpp/src/data_contract/associated_token/token_configuration/v0/mod.rs (4)
6-7: LGTM!The imports are correctly versioned and follow the module hierarchy.
38-39: LGTM!The new field is well-documented and uses the appropriate type for managing token distribution rules.
92-100: LGTM!The
Displayimplementation correctly includes the new field with consistent formatting.
140-169: LGTM!The initialization follows the "most restrictive" pattern consistently, with appropriate default values for all sub-fields.
packages/rs-dpp/src/data_contract/associated_token/token_configuration/methods/can_apply_token_configuration_item/v0/mod.rs (3)
79-104: LGTM!The permission checks for perpetual distribution changes follow the established pattern and correctly access the rules through the distribution_rules field.
106-107: LGTM!The access path for new tokens destination identity rules is consistently updated to use the distribution_rules field.
Also applies to: 112-113, 123-124
135-136: LGTM!The access path for minting allow choosing destination rules is consistently updated to use the distribution_rules field.
Also applies to: 141-142, 154-155
packages/rs-drive-abci/tests/strategy_tests/token_tests.rs (2)
8-11: LGTM: Import changes align with the new token distribution model.The imports have been updated to include both
TokenConfigurationV0GettersandTokenConfigurationV0Setters, along with the newTokenDistributionRulesV0Setters. This reflects the architectural change of moving token distribution logic into a dedicated module.
76-78: LGTM: Token configuration updates follow the new distribution rules pattern.The token configuration is now accessed through the distribution rules, which is consistent with the architectural changes. The same pattern is correctly applied in both test functions.
Also applies to: 226-228
packages/rs-dpp/src/data_contract/associated_token/token_configuration/methods/validate_token_configuration_update/v0/mod.rs (3)
4-4: LGTM: New import for distribution rules accessors.The addition of
TokenDistributionRulesV0Gettersimport aligns with the architectural changes.
87-108: LGTM: Distribution rules validation follows consistent patterns.The validation logic for
new_tokens_destination_identityandminting_allow_choosing_destinationhas been updated to use the distribution rules accessors while maintaining the same validation patterns and error handling.Also applies to: 123-144
158-186: LGTM: New perpetual distribution validation added.The new validation for perpetual distribution follows the same pattern as other validations, maintaining consistency in error handling and validation logic.
packages/rs-dpp/src/state_transition/state_transitions/document/batch_transition/batched_transition/token_transition.rs (2)
Line range hint
8-11: LGTM: Import changes align with the new token distribution model.The addition of
TokenDistributionRulesV0Gettersimport is consistent with the architectural changes.
348-348: LGTM: Token configuration access updated to use distribution rules.The access to
new_tokens_destination_identityhas been updated to use the distribution rules accessors while maintaining the same error handling and message.packages/rs-drive/src/state_transition_action/batch/batched_transition/token_transition/token_mint_transition_action/v0/transformer.rs (2)
15-15: LGTM: Import changes align with the new token distribution model.The addition of
TokenDistributionRulesV0Gettersimport is consistent with the architectural changes.
121-122: LGTM: Token configuration access consistently updated across both methods.The access to token configuration has been updated to use distribution rules accessors in both methods while maintaining the same validation and error handling logic.
Also applies to: 153-156, 298-299, 330-333
packages/rs-drive-abci/src/execution/validation/state_transition/state_transitions/data_contract_update/mod.rs (1)
1180-1183: LGTM! Clean import organization.The new imports are well-organized and specifically scoped for token-related functionality.
packages/rs-dpp/src/data_contract/associated_token/token_pre_programmed_distribution/mod.rs (2)
1-8: LGTM! Well-organized imports and module structure.The imports are clean and the module organization follows Rust conventions.
9-14: Well-implemented versioned enum with proper derive macros.The
TokenPreProgrammedDistributionenum is well-designed with:
- Proper serialization/deserialization support
- Version tagging for forward compatibility
- Clean derive implementations
packages/rs-dpp/Cargo.toml (1)
17-17: Verify the ordered-float version compatibility.The addition of
ordered-floatwith serde support is appropriate. However, please verify that version 4.6.0 is compatible with the rest of the ecosystem.✅ Verification successful
ordered-float v4.6.0 is properly integrated and compatible
The dependency is well-integrated within the rs-dpp package, using the NotNan type appropriately for mathematical operations in distribution functions. The serde feature is correctly utilized for serialization, and there are no compatibility issues with other dependencies.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Check for potential compatibility issues with ordered-float cargo tree --package ordered-float cargo tree --package serde | grep ordered-floatLength of output: 278
Script:
#!/bin/bash # Search for ordered-float in all Cargo.toml files echo "=== Checking ordered-float in Cargo.toml files ===" rg "ordered-float.*=" -A 2 -B 2 "Cargo.toml$" echo -e "\n=== Checking ordered-float usage in Rust files ===" rg "ordered_float::" -A 2 -B 2 "\.rs$" echo -e "\n=== Checking specific ordered-float types usage ===" rg "NotNan|OrderedFloat" "\.rs$"Length of output: 612
Script:
#!/bin/bash # Find and examine all Cargo.toml files in the packages directory echo "=== Checking ordered-float in Cargo.toml files ===" fd Cargo.toml packages/ -x cat {} | grep -A 2 -B 2 "ordered-float" echo -e "\n=== Checking Rust files for ordered-float usage ===" fd .rs$ packages/ -x rg -l "use.*ordered_float|NotNan|OrderedFloat" {}Length of output: 755
Script:
#!/bin/bash echo "=== Checking ordered-float usage patterns ===" rg -A 3 -B 3 "ordered_float|NotNan|OrderedFloat" packages/rs-dpp/src/data_contract/associated_token/token_perpetual_distribution/distribution_function/Length of output: 19030
packages/rs-drive-abci/src/execution/validation/state_transition/state_transitions/batch/action_validation/token/token_mint_transition_action/state_v0/mod.rs (3)
Line range hint
22-31: LGTM! Well-structured trait definition.The trait is well-defined with appropriate visibility modifiers and follows Rust naming conventions.
Line range hint
32-145: Overall implementation is robust and secure.The token minting validation implementation is thorough, covering all necessary checks including authorization, supply limits, and recipient validation. The code uses safe arithmetic operations and proper error handling. While functional, it could benefit from the suggested refactoring to improve maintainability.
5-5: Verify the removal of UnauthorizedTokenActionError.The removal of
UnauthorizedTokenActionErrorimport suggests a change in error handling for unauthorized actions. Let's verify that authorization errors are properly handled elsewhere.✅ Verification successful
Verified: UnauthorizedTokenActionError import removal is safe
The authorization checks are properly handled through the base class's
validate_group_action()method, which is called viaself.base().validate_group_action(). The removal of this import is part of a clean refactoring that centralizes authorization handling in the base implementation.🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Search for UnauthorizedTokenActionError usage and its potential replacement echo "Searching for UnauthorizedTokenActionError usage:" rg "UnauthorizedTokenActionError" -A 5 echo -e "\nSearching for potential new authorization error types:" ast-grep --pattern 'struct $name:ident { $$$ }' | rg -A 5 ".*Error.*Token.*Auth|.*Token.*Auth.*Error"Length of output: 22772
Script:
#!/bin/bash # Check if mint operations are validated through the base class echo "Checking token mint validation inheritance:" ast-grep --pattern 'impl $_ for TokenMintTransitionActionStateV0 { $$$ }' echo -e "\nChecking base class validation usage:" rg -A 5 "validate_authorization|check_authorization" packages/rs-drive-abci/src/execution/validation/state_transition/state_transitions/batch/action_validation/token/token_mint_transition_action/state_v0/Length of output: 472
Script:
#!/bin/bash # Get the complete content of the mint transition action file echo "Checking mint transition action implementation:" cat packages/rs-drive-abci/src/execution/validation/state_transition/state_transitions/batch/action_validation/token/token_mint_transition_action/state_v0/mod.rs echo -e "\nChecking for trait implementations:" rg -B 2 -A 5 "trait.*TokenMintTransitionAction|trait.*TokenBaseTransitionAction" packages/rs-drive-abci/src/execution/validation/state_transition/state_transitions/batch/action_validation/token/Length of output: 16762
commit 6776651 Author: QuantumExplorer <[email protected]> Date: Sat Mar 1 22:23:41 2025 +0700 chore: update to latest dash core 37 (#2483) commit 1501103 Merge: a7c7a0f da17fc5 Author: Ivan Shumkov <[email protected]> Date: Thu Feb 27 14:21:41 2025 +0700 chore: merge master and resolve conflicts (#2481) commit da17fc5 Author: pshenmic <[email protected]> Date: Thu Feb 27 13:31:51 2025 +0700 feat(js-dash-sdk): fix tests after merge commit c7e40cb Merge: c57e8b2 f9eb069 Author: Ivan Shumkov <[email protected]> Date: Thu Feb 27 09:35:02 2025 +0700 Merge remote-tracking branch 'origin/chore/merge-master' into chore/merge-master commit c57e8b2 Author: Ivan Shumkov <[email protected]> Date: Thu Feb 27 09:34:40 2025 +0700 test(dpp): fix assertion with the same value commit 045b6fa Author: Ivan Shumkov <[email protected]> Date: Thu Feb 27 09:32:33 2025 +0700 chore(dpp): remove unnecessary type conversion commit 8160ccd Author: Ivan Shumkov <[email protected]> Date: Thu Feb 27 09:31:32 2025 +0700 chore: remove duplicated commented code commit f9eb069 Merge: 05d0085 a7c7a0f Author: pshenmic <[email protected]> Date: Wed Feb 26 20:03:00 2025 +0700 Merge branch 'v2.0-dev' into chore/merge-master commit a7c7a0f Author: pshenmic <[email protected]> Date: Wed Feb 26 19:52:02 2025 +0700 build: bump rust version to 1.85 (#2480) commit 05d0085 Merge: bcf1785 196976c Author: Ivan Shumkov <[email protected]> Date: Wed Feb 26 18:03:38 2025 +0700 Merge branch 'master' into v2.0-dev commit bcf1785 Author: lklimek <[email protected]> Date: Fri Feb 21 08:43:35 2025 +0100 feat: wasm sdk build proof-of-concept (#2405) Co-authored-by: Ivan Shumkov <[email protected]> commit 5e32426 Author: Paul DeLucia <[email protected]> Date: Thu Feb 20 19:22:52 2025 +0700 fix: token already paused unpaused and frozen validation (#2466) commit 374a036 Author: Ivan Shumkov <[email protected]> Date: Thu Feb 20 17:46:57 2025 +0700 test: fix slowdown of JS SDK unit tests (#2475) commit 1fed09b Author: Ivan Shumkov <[email protected]> Date: Thu Feb 20 13:46:36 2025 +0700 fix(dpp): invalid feature flag usage (#2477) commit 33507bb Author: Paul DeLucia <[email protected]> Date: Thu Feb 20 13:18:55 2025 +0700 fix: destroy frozen funds used wrong identity and proof verification (#2467) commit 91a9766 Author: Ivan Shumkov <[email protected]> Date: Wed Feb 19 16:57:32 2025 +0700 feat(sdk): return state transition execution error (#2454) commit cb915a7 Author: Ivan Shumkov <[email protected]> Date: Wed Feb 19 16:46:54 2025 +0700 test: fix token history contract tests (#2470) commit 04276d5 Author: Ivan Shumkov <[email protected]> Date: Tue Feb 18 21:00:05 2025 +0700 fix: xss vulnerability in mocha (#2469) commit 196976c Author: pshenmic <[email protected]> Date: Fri Feb 14 18:50:08 2025 +0700 fix(sdk)!: bigint for uint64 values (#2443) commit 0bd29a6 Author: pshenmic <[email protected]> Date: Fri Feb 14 17:29:35 2025 +0700 feat(dpp): extra methods for state transitions in wasm (#2462) commit 1eae781 Author: pshenmic <[email protected]> Date: Fri Feb 14 15:29:17 2025 +0700 chore(platform): npm audit fix (#2463) commit ddf4e67 Author: Ivan Shumkov <[email protected]> Date: Fri Feb 14 11:28:08 2025 +0700 test: fix `fetchProofForStateTransition` tests and warnings (#2460) commit d88ea46 Author: Ivan Shumkov <[email protected]> Date: Fri Feb 14 09:52:53 2025 +0700 fix(dpp): invalid imports and tests (#2459) commit 82e4d4c Merge: 125cfe7 4becf5f Author: Paul DeLucia <[email protected]> Date: Thu Feb 13 19:05:51 2025 +0700 fix: check if token is paused on token transfers (#2458) commit 4becf5f Author: pauldelucia <[email protected]> Date: Thu Feb 13 18:34:24 2025 +0700 add costs commit 907971d Merge: 9026669 125cfe7 Author: Paul DeLucia <[email protected]> Date: Thu Feb 13 18:05:06 2025 +0700 Merge branch 'v2.0-dev' into feat/token-paused-validation commit 125cfe7 Merge: 91f65c6 c286ec0 Author: Ivan Shumkov <[email protected]> Date: Thu Feb 13 15:51:46 2025 +0700 Merge branch 'v2.0-dev' into v2.0-tokens-dev commit 9026669 Author: pauldelucia <[email protected]> Date: Thu Feb 13 13:41:19 2025 +0700 feat: check if token is paused on token transfers commit c286ec0 Author: pshenmic <[email protected]> Date: Wed Feb 12 15:41:21 2025 +0700 feat(sdk): add option to request all keys (#2445) commit 91f65c6 Merge: d6b40e6 1a1c50b Author: Paul DeLucia <[email protected]> Date: Wed Feb 12 12:04:58 2025 +0700 fix: wrong order of parameters in UnauthorizedTokenActionError (#2456) commit 1a1c50b Author: pauldelucia <[email protected]> Date: Wed Feb 12 11:51:31 2025 +0700 fix: wrong order of parameters in UnauthorizedTokenActionError commit 26aff36 Author: lklimek <[email protected]> Date: Tue Feb 11 13:06:54 2025 +0100 build: bump Alpine version to 3.21 (#2074) commit 9daa195 Author: Ivan Shumkov <[email protected]> Date: Tue Feb 11 14:38:55 2025 +0700 ci: use github-hosted arm runner for release workflow (#2452) commit 2b1c252 Author: Paul DeLucia <[email protected]> Date: Tue Feb 4 16:40:34 2025 +0700 fix: proof result error for credit transfers in sdk (#2451) commit d6b40e6 Author: QuantumExplorer <[email protected]> Date: Tue Feb 4 06:49:03 2025 +0700 feat(platform): token distribution part two (#2450) commit 93f7d44 Author: Ivan Shumkov <[email protected]> Date: Wed Jan 29 14:07:55 2025 +0700 fix(dpp): invalid feature flag instructions (#2448) commit 6d5af88 Author: QuantumExplorer <[email protected]> Date: Mon Jan 27 16:59:39 2025 +0700 feat(dpp): token distribution model (#2447) commit e735313 Author: Ivan Shumkov <[email protected]> Date: Mon Jan 27 14:24:26 2025 +0700 feat: add token transitions to SDK and DAPI (#2434) commit 0743be2 Author: pshenmic <[email protected]> Date: Sun Jan 26 22:00:40 2025 +0700 feat(dpp): extra methods for state transitions in wasm (#2401) commit f609bcf Merge: 3733f56 cbddb8d Author: Ivan Shumkov <[email protected]> Date: Fri Jan 24 18:16:38 2025 +0700 Merge branch 'v2.0-dev' into v2.0-tokens-dev commit cbddb8d Author: QuantumExplorer <[email protected]> Date: Fri Jan 24 17:59:16 2025 +0700 chore(platform): make bls sig compatibility an optional feature (#2440) Co-authored-by: Ivan Shumkov <[email protected]> commit 764684b Author: Ivan Shumkov <[email protected]> Date: Fri Jan 24 17:57:27 2025 +0700 chore: ignore deprecated `lodash.get` (#2441) commit 3733f56 Author: QuantumExplorer <[email protected]> Date: Thu Jan 23 09:16:12 2025 +0700 feat(platform)!: enhance token configuration and validation mechanisms (#2439) commit 2480ceb Author: QuantumExplorer <[email protected]> Date: Wed Jan 22 16:33:13 2025 +0700 chore: dapi grpc queries (#2437) commit c9ab154 Author: QuantumExplorer <[email protected]> Date: Wed Jan 22 15:50:25 2025 +0700 feat(platform)!: improved token validation and token config update transition (#2435) commit d9647cc Author: QuantumExplorer <[email protected]> Date: Tue Jan 21 10:28:58 2025 +0700 feat: get proofs for tokens (#2433) commit e5964b8 Author: QuantumExplorer <[email protected]> Date: Mon Jan 20 23:31:50 2025 +0700 feat: group queries (#2432) commit 0220302 Author: QuantumExplorer <[email protected]> Date: Sun Jan 19 14:43:51 2025 +0700 feat(platform): proof verification for many queries and a few more queries (#2431) commit cd1527d Author: QuantumExplorer <[email protected]> Date: Fri Jan 17 19:39:37 2025 +0700 fix(dpp)!: wrapping overflow issue (#2430) commit fd7ee85 Merge: d7143cc e4e156c Author: Ivan Shumkov <[email protected]> Date: Thu Jan 16 21:45:47 2025 +0700 Merge branch 'master' into v1.9-dev commit e4e156c Author: QuantumExplorer <[email protected]> Date: Thu Jan 16 18:11:57 2025 +0700 chore(release): update change log and release v1.8.0 (#2427) Co-authored-by: Ivan Shumkov <[email protected]> commit 55a1e03 Author: QuantumExplorer <[email protected]> Date: Thu Jan 16 15:30:42 2025 +0700 feat(platform)!: token base support (#2383) commit 59bf0af Author: QuantumExplorer <[email protected]> Date: Thu Jan 16 13:10:39 2025 +0700 chore(release): bump to v1.8.0-rc.2 (#2426) commit 410eb09 Author: QuantumExplorer <[email protected]> Date: Thu Jan 16 06:31:26 2025 +0700 fix(drive-abci): rebroadcasting should not only take first 2 quorums too (#2425) commit 2abce8e Author: Ivan Shumkov <[email protected]> Date: Wed Jan 15 22:51:58 2025 +0700 chore(release): update changelog and bump version to 1.8.0-rc.1 (#2423) commit ad5f604 Author: Ivan Shumkov <[email protected]> Date: Wed Jan 15 22:14:13 2025 +0700 chore: update bls library (#2424) commit c6feb5b Author: QuantumExplorer <[email protected]> Date: Wed Jan 15 18:57:49 2025 +0700 feat(platform)!: distribute prefunded specialized balances after vote (#2422) Co-authored-by: Ivan Shumkov <[email protected]> commit 94dcbb2 Author: Ivan Shumkov <[email protected]> Date: Wed Jan 15 05:51:45 2025 +0700 chore(drive): increase withdrawal limits to 2000 Dash per day (#2287) commit 6a0aede Author: Ivan Shumkov <[email protected]> Date: Tue Jan 14 21:42:59 2025 +0700 chore: fix test suite configuration script (#2402) commit e94b7bb Author: QuantumExplorer <[email protected]> Date: Tue Jan 14 19:23:46 2025 +0700 fix(drive-abci): document purchase on mutable document from different epoch had issue (#2420) commit 4ee57a6 Author: Ivan Shumkov <[email protected]> Date: Tue Jan 14 19:12:20 2025 +0700 fix(drive): more than one key was returned when expecting only one result (#2421) commit be5cd6d Author: Ivan Shumkov <[email protected]> Date: Mon Jan 13 15:12:33 2025 +0700 fix(sdk): failed to deserialize consensus error (#2410) commit e07271e Author: Ivan Shumkov <[email protected]> Date: Mon Jan 13 14:57:08 2025 +0700 chore: resolve NPM audit warnings (#2417) commit a809df7 Author: QuantumExplorer <[email protected]> Date: Sun Jan 12 09:21:48 2025 +0700 test: unify identity versioned cost coverage (#2416) commit 6d637fe Author: Paul DeLucia <[email protected]> Date: Fri Dec 27 09:42:04 2024 -0500 fix: try DriveDocumentQuery from DocumentQuery start field (#2407) commit cfd9c4d Author: Ivan Shumkov <[email protected]> Date: Thu Dec 19 18:30:06 2024 +0700 chore(release): update changelog and bump version to 1.8.0-dev.2 (#2404) commit fecda31 Merge: 37d5732 fc7d994 Author: Ivan Shumkov <[email protected]> Date: Thu Dec 19 15:33:45 2024 +0700 Merge branch 'master' into v1.8-dev commit fc7d994 Author: Ivan Shumkov <[email protected]> Date: Thu Dec 19 14:40:44 2024 +0700 chore(release): update changelog and bump version to 1.7.1 (#2403) commit adcd3b8 Author: QuantumExplorer <[email protected]> Date: Thu Dec 19 09:54:07 2024 +0300 fix!: emergency hard fork to fix masternode voting (#2397) commit 37d5732 Author: Ivan Shumkov <[email protected]> Date: Wed Dec 18 22:24:37 2024 +0700 fix(dashmate): some group commands fail with mtime not found (#2400) commit 01a5b7a Author: Ivan Shumkov <[email protected]> Date: Wed Dec 18 20:44:44 2024 +0700 refactor(dpp): using deprecated param to init wasm module (#2399) commit c5f5878 Author: Ivan Shumkov <[email protected]> Date: Wed Dec 18 18:04:14 2024 +0700 fix(dashmate): local network starting issues (#2394) commit 71c41ff Author: Ivan Shumkov <[email protected]> Date: Wed Dec 18 18:03:55 2024 +0700 perf(dpp): reduce JS binding size by 3x (#2396) commit 21ec393 Author: lklimek <[email protected]> Date: Wed Dec 18 10:47:58 2024 +0100 build!: update rust to 1.83 - backport #2393 to v1.7 (#2398) commit d7143cc Author: lklimek <[email protected]> Date: Wed Dec 18 08:53:53 2024 +0100 build!: optimize for x86-64-v3 cpu microarchitecture (Haswell+) (#2374) commit d318b1c Author: lklimek <[email protected]> Date: Tue Dec 17 14:56:15 2024 +0100 build: bump wasm-bindgen to 0.2.99 (#2395) commit 889d192 Author: Ivan Shumkov <[email protected]> Date: Tue Dec 17 19:25:58 2024 +0700 chore(release): update changelog and bump version to 1.8.0-dev.1 (#2391) commit 8185d21 Author: lklimek <[email protected]> Date: Tue Dec 17 10:47:53 2024 +0100 feat(sdk)!: allow setting CA cert (#1924) commit 82a6217 Author: lklimek <[email protected]> Date: Tue Dec 17 02:51:18 2024 +0100 build!: update rust to 1.83 (#2393) commit 494054a Author: QuantumExplorer <[email protected]> Date: Mon Dec 16 13:47:58 2024 +0300 refactor(platform): replace bls library (#2257) Co-authored-by: Lukasz Klimek <[email protected]> commit 4c203e4 Author: lklimek <[email protected]> Date: Mon Dec 16 10:38:34 2024 +0100 test(sdk): generate test vectors using testnet (#2381) commit 0ff6b27 Author: lklimek <[email protected]> Date: Mon Dec 16 10:37:35 2024 +0100 chore: remove deprecated check_network_version.sh (#2084) commit b265bb8 Author: lklimek <[email protected]> Date: Fri Dec 13 13:25:40 2024 +0100 ci: fix artifact upload issue on release build (#2389) commit 40ae73f Author: Ivan Shumkov <[email protected]> Date: Fri Dec 13 17:35:40 2024 +0700 chore(release): update changelog and bump version to 1.7.0 (#2387) commit 257e3da Author: Ivan Shumkov <[email protected]> Date: Fri Dec 13 15:44:10 2024 +0700 chore(dashmate)!: update Core to version 22 (#2384) commit 19a4c6d Author: Ivan Shumkov <[email protected]> Date: Thu Dec 12 18:30:14 2024 +0700 chore(dashmate): set tenderdash version to 1 (#2385) commit 0e9d4dc Author: lklimek <[email protected]> Date: Thu Dec 12 11:39:35 2024 +0100 chore: address vulnerabilty GHSA-mwcw-c2x4-8c55 (#2382) Co-authored-by: Ivan Shumkov <[email protected]> commit bdae90c Author: Ivan Shumkov <[email protected]> Date: Thu Dec 12 13:36:04 2024 +0700 chore(dashmate): increase subsidy for devnet (#2353)
Summary by CodeRabbit
Release Notes
New Features
Improvements
Technical Updates
Checklist:
For repository code-owners and collaborators only