Closed bitcoinbrisbane closed 4 weeks ago
[!CAUTION]
Review failed
The pull request is closed.
This pull request introduces several changes across multiple files, primarily focusing on the transition from a whitelist mechanism to a Merkle tree-based proof system in the DynamicEater
contract. The README.md
file is updated with new command examples and a new section for example proofs. The MergeFactory
contract is modified to remove the WithMarket
merge type, and the MergeWithMarket
contract is deleted. Additionally, new dependencies related to Merkle trees are added, and test cases are updated to reflect these changes.
File | Change Summary |
---|---|
README.md |
Updated command examples by removing the --asset parameter from simple-swap and mint-nft-position commands. Added a new section titled "Example proofs." |
contracts/core/merge/DynamicEater.sol |
Added bytes32 public merkleRoot; , replaced mapping(address => uint256) public whiteList; , removed addWhiteList , added setMerkleRoot , mergeWithProof , and _validateLeaf . Updated onlyWhiteListed modifier. |
contracts/core/merge/MergeFactory.sol |
Updated MergeType enum to include DynamicEater , removed createMerge , and updated createMergeWith to accept a MergeType parameter. |
contracts/core/merge/MergeWithMarket.sol |
Contract removed. |
contracts/libraries/BondingCurve.sol |
Updated comments in calculateTokensOut for clarity without altering functionality. |
data/proof.json |
New JSON file added containing an array of objects with address and proofHex fields. |
ignition/modules/MergeWithMarket.ts |
Module removed. |
package.json |
Added new dependencies: @openzeppelin/merkle-tree , keccak256 , and merkletreejs . |
scripts/merklegen.js |
Implemented Merkle tree functionality using OpenZeppelin library, created a Merkle tree from a whitelist, and logged the root hash and proofs. |
test/DynamicEater.test.ts |
Modified tests to focus on merging functionality with the Merkle root and added new tests for merging with proof. |
test/MarketMerge.test.ts |
Skipped the "MarketMerge" test suite, noting that the contracts have been deleted. |
README.md
file, adding command examples and updating existing commands.README.md
file with changes to command examples.--asset
parameter from commands in the README.md
file.MergeFactory
contract and its functionalities related to merging operations.Eater
contract, relating to merging functionality.MergeWithMarket
contract and its functionalities in the context of merging operations.eat-all-the-tokens
, SC
🐰 In the meadow, changes bloom,
A Merkle tree dispels the gloom.
Commands refined, proofs in sight,
Hop along, it's a joyous night!
With every merge, the tokens play,
A brighter path, let’s hop away! 🌟
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?
Summary by CodeRabbit
New Features
Bug Fixes
Documentation
Chores
Tests