refactor: cleanup #1
Conversation
WalkthroughThe repository underwent significant changes, including a rebranding from "White Whale" to "MANTRA," updates to the CODEOWNERS file, and the introduction of new GitHub Actions workflows for testing and linting. Several files were streamlined, removing conditional logic related to the "osmosis" feature and simplifying dependency management. Additionally, documentation files were reduced to placeholders, indicating a need for future updates. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant CI/CD
participant Repository
participant Testing
User->>Repository: Create Pull Request
Repository->>CI/CD: Trigger Workflows
CI/CD->>Testing: Run Tests and Lint
Testing-->>CI/CD: Return Results
CI/CD-->>User: Notify Status
Tip We have updated our review workflow to use the Anthropic's Claude family of models. Please share any feedback in the discussion post on our Discord. 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[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
Outside diff range, codebase verification and nitpick comments (1)
contracts/liquidity_hub/pool-manager/src/tests/integration_tests.rs (1)
Line range hint
228-228: Fix the typo in the function name.The function name has a typo - it should be
cant_recreate_existing_pool.Apply this diff to fix the typo:
-fn cant_recreate_existing_poo() { +fn cant_recreate_existing_pool() {
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files ignored due to path filters (1)
Cargo.lockis excluded by!**/*.lock
Files selected for processing (32)
- .github/CODEOWNERS (1 hunks)
- .github/PULL_REQUEST_TEMPLATE.md (3 hunks)
- .github/workflows/ci-test-lints.yaml (1 hunks)
- .github/workflows/pr-convcom-check.yaml (1 hunks)
- .github/workflows/release-artifacts-on-tag.yaml (2 hunks)
- Cargo.toml (3 hunks)
- contracts/liquidity_hub/epoch-manager/Cargo.toml (2 hunks)
- contracts/liquidity_hub/epoch-manager/src/contract.rs (2 hunks)
- contracts/liquidity_hub/incentive-manager/Cargo.toml (2 hunks)
- contracts/liquidity_hub/incentive-manager/src/contract.rs (2 hunks)
- contracts/liquidity_hub/pool-manager/Cargo.toml (2 hunks)
- contracts/liquidity_hub/pool-manager/src/contract.rs (6 hunks)
- contracts/liquidity_hub/pool-manager/src/error.rs (2 hunks)
- contracts/liquidity_hub/pool-manager/src/helpers.rs (10 hunks)
- contracts/liquidity_hub/pool-manager/src/manager/commands.rs (5 hunks)
- contracts/liquidity_hub/pool-manager/src/queries.rs (4 hunks)
- contracts/liquidity_hub/pool-manager/src/swap/commands.rs (1 hunks)
- contracts/liquidity_hub/pool-manager/src/swap/perform_swap.rs (3 hunks)
- contracts/liquidity_hub/pool-manager/src/tests/integration_tests.rs (43 hunks)
- docs/CODE_OF_CONDUCT.md (1 hunks)
- docs/CONTRIBUTING.md (1 hunks)
- docs/SECURITY.md (1 hunks)
- justfile (4 hunks)
- packages/common-testing/Cargo.toml (3 hunks)
- packages/common-testing/src/lib.rs (1 hunks)
- scripts/build_release.sh (2 hunks)
- scripts/deployment/deploy_env/base_env.sh (1 hunks)
- scripts/deployment/deploy_env/mainnets/mantra.env (1 hunks)
- scripts/deployment/deploy_env/mantra_testnet.env (1 hunks)
- scripts/deployment/deploy_env/testnets/mantra.env (1 hunks)
- scripts/get_artifacts_versions.sh (1 hunks)
- scripts/utils/crate_versions.sh (3 hunks)
Files skipped from review due to trivial changes (13)
- .github/CODEOWNERS
- .github/PULL_REQUEST_TEMPLATE.md
- contracts/liquidity_hub/pool-manager/Cargo.toml
- contracts/liquidity_hub/pool-manager/src/error.rs
- contracts/liquidity_hub/pool-manager/src/manager/commands.rs
- docs/CODE_OF_CONDUCT.md
- docs/SECURITY.md
- justfile
- scripts/deployment/deploy_env/mainnets/mantra.env
- scripts/deployment/deploy_env/mantra_testnet.env
- scripts/deployment/deploy_env/testnets/mantra.env
- scripts/get_artifacts_versions.sh
- scripts/utils/crate_versions.sh
Additional context used
Shellcheck
scripts/deployment/deploy_env/base_env.sh
[warning] 13-13: Quote to prevent word splitting/globbing, or split robustly with mapfile or read -a.
(SC2206)
[warning] 13-13: Quote to prevent word splitting/globbing, or split robustly with mapfile or read -a.
(SC2206)
[warning] 13-13: Quote to prevent word splitting/globbing, or split robustly with mapfile or read -a.
(SC2206)
[warning] 13-13: Quote to prevent word splitting/globbing, or split robustly with mapfile or read -a.
(SC2206)
[warning] 13-13: Quote to prevent word splitting/globbing, or split robustly with mapfile or read -a.
(SC2206)
[warning] 16-16: Variable was used as an array but is now assigned a string.
(SC2178)
scripts/build_release.sh
[warning] 12-17: Use spaces, not commas, to separate array elements.
(SC2054)
actionlint
.github/workflows/ci-test-lints.yaml
18-18: the runner of "actions/checkout@v2" action is too old to run on GitHub Actions. update the action's version to fix this issue
(action)
21-21: the runner of "actions-rs/toolchain@v1" action is too old to run on GitHub Actions. update the action's version to fix this issue
(action)
39-39: the runner of "actions/checkout@v2" action is too old to run on GitHub Actions. update the action's version to fix this issue
(action)
42-42: the runner of "actions-rs/toolchain@v1" action is too old to run on GitHub Actions. update the action's version to fix this issue
(action)
50-50: the runner of "actions-rs/cargo@v1" action is too old to run on GitHub Actions. update the action's version to fix this issue
(action)
56-56: the runner of "actions-rs/cargo@v1" action is too old to run on GitHub Actions. update the action's version to fix this issue
(action)
62-62: the runner of "actions-rs/cargo@v1" action is too old to run on GitHub Actions. update the action's version to fix this issue
(action)
Additional comments not posted (47)
packages/common-testing/src/lib.rs (1)
1-1: LGTM!The code changes are approved.
docs/CONTRIBUTING.md (1)
1-3: Verify the plan to update the contributing guidelines.The change suggests that a new version of the contributing guidelines is intended to be created. However, it is not clear when this will be done or what the new guidelines will include.
Please provide more information on the plan to update the contributing guidelines, including:
- When will the new guidelines be added?
- What will be the scope of the new guidelines?
- Who will be responsible for updating the guidelines?
scripts/deployment/deploy_env/base_env.sh (1)
10-10: LGTM!The code changes are approved.
packages/common-testing/Cargo.toml (3)
2-2: LGTM!The package name change from "white-whale-testing" to "common-testing" is approved.
6-6: LGTM!The package description change to remove the reference to "White Whale" is approved.
Line range hint
1-30: Verify the impact of the feature flag removal.Ensure that the removal of the
injective,token_factory,osmosis, andosmosis_token_factoryfeature flags does not break any existing functionality.Run the following script to verify the impact of the feature flag removal:
Verification successful
No Conditional Compilation Found for Removed Feature Flags
The removed feature flags (
injective,token_factory,osmosis,osmosis_token_factory) do not appear in any conditional compilation attributes in the codebase. This suggests that their removal fromCargo.tomlis unlikely to impact the build or functionality directly. However, consider reviewing the context of their occurrences in the codebase to ensure no overlooked dependencies or documentation updates are needed.
- Files with occurrences:
packages/common-testing/src/multi_test/stargate_mock.rscontracts/liquidity_hub/pool-manager/src/tests/integration_tests.rsScripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the impact of the feature flag removal. # Test 1: Search for the usage of the removed feature flags in the codebase. # Expect: No occurrences of the removed feature flags. rg --type rust $'injective|token_factory|osmosis|osmosis_token_factory' # Test 2: Ensure that all existing tests are passing after the feature flag removal. # Expect: All tests should pass. cargo testLength of output: 1920
Script:
#!/bin/bash # Description: Search for conditional compilation attributes using the removed feature flags. # Search for conditional compilation attributes using the removed feature flags. rg --type rust '#\[cfg\(feature = "(injective|token_factory|osmosis|osmosis_token_factory)"\)\]'Length of output: 98
.github/workflows/release-artifacts-on-tag.yaml (2)
18-18: LGTM!The code change is approved.
29-29: LGTM!The code change is approved.
scripts/build_release.sh (4)
9-9: LGTM!The change to the
display_usagefunction aligns with the goal of simplifying the script's usage by removing the need to specify a deployment chain using the-cflag.
19-25: LGTM!The changes introduce architecture-specific handling of the optimizer Docker image, which is a good practice to ensure compatibility across different architectures. The version update of the Docker images likely indicates an upgrade to the underlying tools used for building.
36-36: LGTM!The new comment provides clarity about the purpose of the script, which is to retrieve artifact versions. This change improves the script's readability and maintainability.
12-17: Skipping the Shellcheck warning.The warning about using spaces instead of commas to separate array elements is a false positive. Using spaces to separate array elements is a valid syntax in Bash, and using commas is not required. It is a matter of personal preference or coding style.
Tools
Shellcheck
[warning] 12-17: Use spaces, not commas, to separate array elements.
(SC2054)
.github/workflows/pr-convcom-check.yaml (1)
19-19: LGTM!Updating the toolchain version to
1.80.0is a good practice to leverage the latest improvements and bug fixes. The toolchain version update aligns with the PR objective of cleaning up the codebase.contracts/liquidity_hub/epoch-manager/Cargo.toml (8)
5-5: LGTM!The code change is approved.
6-6: LGTM!The code change is approved.
7-7: LGTM!The code change is approved.
8-8: LGTM!The code change is approved.
9-9: LGTM!The code change is approved.
10-10: LGTM!The code change is approved.
11-11: LGTM!The code change is approved.
Line range hint
1-11: LGTM!The removal of feature declarations is approved.
contracts/liquidity_hub/incentive-manager/Cargo.toml (1)
40-40: LGTM!The dependency change from
white-whale-testingtocommon-testingis approved..github/workflows/ci-test-lints.yaml (1)
1-12: LGTM!The workflow structure looks good. It ensures that the test suite and lints are run on every pull request and push to the main branch.
Cargo.toml (2)
Line range hint
4-11: Verify the impact of removing entries from thememberslist.Several entries have been removed from the
memberslist, including"integration-tests"and various components under"contracts/liquidity_hub". This indicates a potential restructuring or removal of certain modules from the workspace.Please ensure that:
- The removed modules are no longer needed in the project.
- The removal does not break any dependencies or functionality.
- The documentation is updated to reflect the changes in the project structure.
Run the following script to verify the usage of the removed modules:
Verification successful
Verification Successful: No Impact from Removed Modules
The removed modules from the
memberslist inCargo.tomlare not used anywhere in the current codebase. This confirms that their removal does not impact any dependencies or functionality. Ensure that any related documentation is updated to reflect these changes.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the usage of the removed modules. # Test: Search for the usage of the removed modules. Expect: No occurrences. modules=( "integration-tests" "contracts/liquidity_hub/pool-network/*" "contracts/liquidity_hub/fee_collector" "contracts/liquidity_hub/fee_distributor" "contracts/liquidity_hub/whale_lair" "contracts/liquidity_hub/vault-manager" "contracts/liquidity_hub/bonding-manager" ) for module in "${modules[@]}" do echo "Searching for usages of $module..." rg --type rust -i "$module" || echo "No usages found." doneLength of output: 2125
39-40: Verify the impact of updating the dependencies and update references.The paths for
white-whale-stdandwhite-whale-testinghave been replaced with version specifications. This change suggests a move towards more stable and version-controlled dependencies, which could potentially improve the reliability and maintainability of the project.Please ensure that:
- The version update for
white-whale-stddoes not introduce any breaking changes.- All references to
white-whale-testingare updated tocommon-testingin the codebase.Run the following script to verify the references to
white-whale-testing:If references are found, apply the necessary changes to update them to
common-testing.Verification successful
Verification successful: No references to
white-whale-testingfound.The transition from
white-whale-testingtocommon-testinghas been completed successfully, with no remaining references to the old dependency name in the Rust codebase. The update towhite-whale-stdshould also be verified for compatibility, but the reference update is confirmed.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the references to `white-whale-testing`. # Test: Search for the references to `white-whale-testing`. Expect: No occurrences. rg --type rust -i "white-whale-testing" || echo "No references found."Length of output: 90
contracts/liquidity_hub/pool-manager/src/swap/commands.rs (1)
Line range hint
1-92: Verify the removal of the Osmosis fee from the swap output.The AI-generated summary indicates that the conditional compilation block related to the
osmosis_fee_amounthas been removed from the output. However, the provided code does not show the removed lines or old code.To verify the changes, please provide the following:
- The removed lines or old code that included the
osmosis_fee_amountin the output.- The reason for removing the Osmosis fee from the swap output.
- Confirmation that the removal of the Osmosis fee does not affect the correctness of the swap operation.
Once the additional context is provided, I can proceed with a more thorough review of the changes.
contracts/liquidity_hub/pool-manager/src/swap/perform_swap.rs (1)
115-122: LGTM!The code changes are approved. The removal of the
osmosis_fee_assetfield from theSwapResultstruct simplifies the code and eliminates the need for conditional compilation based on the "osmosis" feature flag.contracts/liquidity_hub/incentive-manager/src/contract.rs (2)
17-17: LGTM!The change to the
CONTRACT_NAMEconstant is approved.
Line range hint
173-173: Verify the impact of removing the conditional compilation attribute.Ensure that including the
migratefunction in coverage reports does not have any unintended consequences.Run the following script to verify the impact of the change:
Verification successful
Removal of Conditional Compilation Attribute Verified
The removal of the
#[cfg(not(tarpaulin_include))]attribute from themigratefunction in theincentive-managermodule aligns with the pattern observed in other modules. This change ensures the function is included in coverage reports without affecting functionality. No unintended consequences are expected.
- Location:
contracts/liquidity_hub/incentive-manager/src/contract.rsScripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify that including the `migrate` function in coverage reports does not have any unintended consequences. # Test: Search for the `migrate` function implementation. Expect: Only one occurrence. rg --type rust -A 5 $'pub fn migrate'Length of output: 1920
contracts/liquidity_hub/pool-manager/src/contract.rs (5)
19-19: LGTM!The code change is approved.
23-23: LGTM!The code change is approved.
50-50: LGTM!The code change is approved.
85-85: LGTM!The code change is approved.
190-190: LGTM!The code change is approved.
contracts/liquidity_hub/pool-manager/src/queries.rs (2)
70-77: LGTM!The code changes are approved. The simplification of the
SimulationResponsestructure enhances code readability and maintainability by eliminating the conditional logic related to the "osmosis" feature and providing a uniform response structure.
105-111: LGTM!The code changes are approved. The simplification of the
ReverseSimulationResponsestructure enhances code readability and maintainability by eliminating the conditional logic related to the "osmosis" feature and providing a uniform response structure.Also applies to: 166-172
contracts/liquidity_hub/pool-manager/src/helpers.rs (2)
244-249: LGTM!The code changes are approved. The function has been simplified by removing the conditional logic related to the "osmosis" feature.
258-287: LGTM!The code changes are approved. The function has been simplified by removing the conditional logic related to the "osmosis" feature. The fees are now consistently subtracted from the
return_amount, which simplifies the control flow and reduces code duplication.contracts/liquidity_hub/pool-manager/src/tests/integration_tests.rs (9)
Line range hint
12-16: LGTM!The code changes are approved.
Line range hint
20-138: LGTM!The code changes are approved.
Line range hint
144-183: LGTM!The code changes are approved.
Line range hint
186-226: LGTM!The code changes are approved.
Line range hint
286-450: LGTM!The code changes are approved.
Line range hint
453-526: LGTM!The code changes are approved.
Line range hint
529-617: LGTM!The code changes are approved.
Line range hint
620-812: LGTM!The code changes are approved.
Line range hint
815-906: LGTM!The code changes are approved.
| TXFLAG=(--node $RPC --chain-id $CHAIN_ID --gas-prices 0.35$DENOM --gas auto --gas-adjustment 1.2 -y -b $b_flag --output $output_flag) | ||
| else | ||
| # Using a string for TXFLAG | ||
| TXFLAG="--node $RPC --chain-id $CHAIN_ID --gas-prices 0.15$DENOM --gas auto --gas-adjustment 1.2 -y -b $b_flag --output $output_flag" | ||
| TXFLAG="--node $RPC --chain-id $CHAIN_ID --gas-prices 0.35$DENOM --gas auto --gas-adjustment 1.2 -y -b $b_flag --output $output_flag" |
There was a problem hiding this comment.
Review the increased gas price and address Shellcheck warnings.
-
The gas price for the new networks is set to
0.35$DENOM, which is an increase from the previous value of0.15$DENOMused for other networks. Please ensure that this increase is intentional and consider its implications on transaction costs for these networks. -
Shellcheck has provided the following warnings:
- Quote to prevent word splitting/globbing, or split robustly with mapfile or read -a. (SC2206)
- Variable was used as an array but is now assigned a string. (SC2178)
To address these warnings and improve the script's robustness, consider applying the following changes:
- TXFLAG=(--node $RPC --chain-id $CHAIN_ID --gas-prices 0.35$DENOM --gas auto --gas-adjustment 1.2 -y -b $b_flag --output $output_flag)
+ TXFLAG=(--node "$RPC" --chain-id "$CHAIN_ID" --gas-prices "0.35$DENOM" --gas auto --gas-adjustment 1.2 -y -b "$b_flag" --output "$output_flag")
- TXFLAG="--node $RPC --chain-id $CHAIN_ID --gas-prices 0.35$DENOM --gas auto --gas-adjustment 1.2 -y -b $b_flag --output $output_flag"
+ TXFLAG=(--node "$RPC" --chain-id "$CHAIN_ID" --gas-prices "0.35$DENOM" --gas auto --gas-adjustment 1.2 -y -b "$b_flag" --output "$output_flag")These changes:
- Quote the variables to prevent word splitting/globbing.
- Consistently use an array for
TXFLAGin both cases to avoid the variable usage warning.
Do you want me to open a GitHub issue to track these changes?
Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| TXFLAG=(--node $RPC --chain-id $CHAIN_ID --gas-prices 0.35$DENOM --gas auto --gas-adjustment 1.2 -y -b $b_flag --output $output_flag) | |
| else | |
| # Using a string for TXFLAG | |
| TXFLAG="--node $RPC --chain-id $CHAIN_ID --gas-prices 0.15$DENOM --gas auto --gas-adjustment 1.2 -y -b $b_flag --output $output_flag" | |
| TXFLAG="--node $RPC --chain-id $CHAIN_ID --gas-prices 0.35$DENOM --gas auto --gas-adjustment 1.2 -y -b $b_flag --output $output_flag" | |
| TXFLAG=(--node "$RPC" --chain-id "$CHAIN_ID" --gas-prices "0.35$DENOM" --gas auto --gas-adjustment 1.2 -y -b "$b_flag" --output "$output_flag") | |
| else | |
| # Using a string for TXFLAG | |
| TXFLAG=(--node "$RPC" --chain-id "$CHAIN_ID" --gas-prices "0.35$DENOM" --gas auto --gas-adjustment 1.2 -y -b "$b_flag" --output "$output_flag") |
Tools
Shellcheck
[warning] 13-13: Quote to prevent word splitting/globbing, or split robustly with mapfile or read -a.
(SC2206)
[warning] 13-13: Quote to prevent word splitting/globbing, or split robustly with mapfile or read -a.
(SC2206)
[warning] 13-13: Quote to prevent word splitting/globbing, or split robustly with mapfile or read -a.
(SC2206)
[warning] 13-13: Quote to prevent word splitting/globbing, or split robustly with mapfile or read -a.
(SC2206)
[warning] 13-13: Quote to prevent word splitting/globbing, or split robustly with mapfile or read -a.
(SC2206)
[warning] 16-16: Variable was used as an array but is now assigned a string.
(SC2178)
| test: | ||
| name: Test Suite | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - name: Checkout sources | ||
| uses: actions/checkout@v2 | ||
|
|
||
| - name: Install stable toolchain | ||
| uses: actions-rs/toolchain@v1 | ||
| with: | ||
| profile: minimal | ||
| toolchain: 1.80.0 | ||
| target: wasm32-unknown-unknown | ||
| override: true | ||
|
|
||
| - name: Run cargo-tarpaulin | ||
| uses: actions-rs/tarpaulin@v0.1 | ||
| with: | ||
| version: "0.15.0" | ||
| args: '--locked -- --test-threads 4' | ||
|
|
There was a problem hiding this comment.
Update the versions of the actions.
The "Test Suite" job looks good, but please update the versions of the actions to ensure compatibility and security.
-uses: actions/checkout@v2
+uses: actions/checkout@v3
-uses: actions-rs/toolchain@v1
+uses: actions-rs/toolchain@v2
-uses: actions-rs/tarpaulin@v0.1
+uses: actions-rs/tarpaulin@v0.2Committable suggestion was skipped due to low confidence.
Tools
actionlint
18-18: the runner of "actions/checkout@v2" action is too old to run on GitHub Actions. update the action's version to fix this issue
(action)
21-21: the runner of "actions-rs/toolchain@v1" action is too old to run on GitHub Actions. update the action's version to fix this issue
(action)
| lints: | ||
| name: Lints | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - name: Checkout sources | ||
| uses: actions/checkout@v2 | ||
|
|
||
| - name: Install stable toolchain | ||
| uses: actions-rs/toolchain@v1 | ||
| with: | ||
| profile: minimal | ||
| toolchain: 1.80.0 | ||
| override: true | ||
| components: rustfmt, clippy | ||
|
|
||
| - name: Run cargo fmt | ||
| uses: actions-rs/cargo@v1 | ||
| with: | ||
| command: fmt | ||
| args: --all -- --check | ||
|
|
||
| - name: Run cargo clippy | ||
| uses: actions-rs/cargo@v1 | ||
| with: | ||
| command: clippy | ||
| args: -- -D warnings | ||
|
|
||
| - name: Generate Schema | ||
| uses: actions-rs/cargo@v1 | ||
| with: | ||
| command: schema | ||
| args: --locked | ||
|
|
||
| - name: Schema Changes | ||
| # fails if any changes not committed | ||
| run: git diff --exit-code schema |
There was a problem hiding this comment.
Update the versions of the actions.
The "Lints" job looks good, but please update the versions of the actions to ensure compatibility and security.
-uses: actions/checkout@v2
+uses: actions/checkout@v3
-uses: actions-rs/toolchain@v1
+uses: actions-rs/toolchain@v2
-uses: actions-rs/cargo@v1
+uses: actions-rs/cargo@v2Committable suggestion was skipped due to low confidence.
Tools
actionlint
39-39: the runner of "actions/checkout@v2" action is too old to run on GitHub Actions. update the action's version to fix this issue
(action)
42-42: the runner of "actions-rs/toolchain@v1" action is too old to run on GitHub Actions. update the action's version to fix this issue
(action)
50-50: the runner of "actions-rs/cargo@v1" action is too old to run on GitHub Actions. update the action's version to fix this issue
(action)
56-56: the runner of "actions-rs/cargo@v1" action is too old to run on GitHub Actions. update the action's version to fix this issue
(action)
62-62: the runner of "actions-rs/cargo@v1" action is too old to run on GitHub Actions. update the action's version to fix this issue
(action)
Description and Motivation
Fork cleanup before performing tweaks to the contracts.
Related Issues
Checklist:
Update index.md)cargo fmt --all --.cargo clippy -- -D warnings.just schemas.Summary by CodeRabbit
New Features
Bug Fixes
Documentation
Chores